cats_core 1.2.35 → 1.2.36

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7642c89b1980407bb2366d413f35325fa547cf339131269638d41069112504ce
-  data.tar.gz: a749c65316d24bf37deac90a7b35fa01cf7a463a793a5140f982ad380320d62a
+  metadata.gz: 496524a19290b62ba488dca18f3d4f33c650e6af6702381e698cde2dbd6d9497
+  data.tar.gz: e1ca7c19c73a9eb79aec8a25b4ea833501285dac8d715a4779e618f10e0bd519
 SHA512:
-  metadata.gz: 957c5ce09ad50fe4e0bcef7ded61b11f6de78a3b0ec4c4055c047186c73f27951050d3c708c37bebca4984ea20b2f360e63e106936c9e2611aff4036cb8dd53e
-  data.tar.gz: f5a9ed6b97b4bf5e59e11c7a876c9962fef4eb1194e8e242c578ad77675e969071945f2fc423e7765e00f5daddb492f0262f8a79dd3376324863f408ec0b9817
+  metadata.gz: e600dd3870ed29dc8762c214335c71435761ef87f62370a8408d60aba84a7128355bcf818a94cd24d70f1f727646341d735ecc4b2520c1cdae0ab94b216ac070
+  data.tar.gz: 5028a95e9fd176960f12ebcfab91731737b5928d97f4bcd5a7649325e830d4e1c95817d45351f7e16a61266f1a6eefbdd03977c57393d2ad0c25cb5d34bf54d1

data/app/models/cats/core/transport_bid.rb

@@ -8,23 +8,17 @@ module Cats
       WINNERS_DECLARED = 'Winner Declared'.freeze
       STATUSES = [NEW, OPEN, CLOSED, RANKED, WINNERS_DECLARED].freeze
 
-      # Bid types
-      REGIONAL = 'Regional'.freeze
-      NON_REGIONAL = 'Non Regional'.freeze
-      BID_TYPES = [REGIONAL, NON_REGIONAL].freeze
-
       has_many :transport_bid_items
       has_many :transport_offers
       has_many :offer_items, through: :transport_offers
       belongs_to :region, class_name: 'Cats::Core::Location', optional: true
+      belongs_to :transport_plan
 
       validates :reference_no, presence: true, uniqueness: true
-      validates :start_date, :end_date, :status, :bid_type, :bid_bond_amount, presence: true
+      validates :start_date, :end_date, :status, :bid_bond_amount, presence: true
       validates :bid_bond_amount, numericality: { greater_than_or_equal_to: 0 }
       validates :status, inclusion: { in: STATUSES }
-      validates :bid_type, inclusion: { in: BID_TYPES }
       validate :validate_start_date_against_end_date
-      validate :validate_region
 
       def validate_start_date_against_end_date
         return unless start_date && end_date
@@ -32,16 +26,6 @@ module Cats
         errors.add(:start_date, 'should be before end date.') if start_date >= end_date
       end
 
-      def validate_region
-        return unless bid_type
-
-        errors.add(:region, 'should be null.') if bid_type == NON_REGIONAL && region
-
-        errors.add(:region, 'should not be null.') if bid_type == REGIONAL && !region
-
-        errors.add(:region, 'is not valid.') if region && region.location_type != Location::REGION
-      end
-
       def open
         raise(StandardError, 'Bid is already open.') if status == OPEN
 

data/app/models/cats/core/transport_bid_item.rb

@@ -2,15 +2,13 @@ module Cats
   module Core
     class TransportBidItem < ApplicationRecord
       belongs_to :transport_bid
-      belongs_to :route
-      belongs_to :unit, class_name: 'Cats::Core::UnitOfMeasure', optional: true
+      belongs_to :transport_plan_item
 
       has_many :offer_items
 
       validates :quantity, presence: true, numericality: { greater_than: 0 }
+      validates :transport_plan_item_id, uniqueness: true
 
-      delegate(:name, to: :route, prefix: true)
-      delegate(:name, to: :unit, prefix: true, allow_nil: true)
       delegate(:reference_no, to: :transport_bid, prefix: true)
     end
   end

data/app/models/cats/core/transport_plan.rb

@@ -0,0 +1,26 @@
+module Cats
+  module Core
+    class TransportPlan < ApplicationRecord
+      # Plan types
+      REGIONAL = 'Regional'.freeze
+      NON_REGIONAL = 'Non Regional'.freeze
+      PLAN_TYPES = [REGIONAL, NON_REGIONAL].freeze
+
+      belongs_to :region, class_name: 'Cats::Core::Location', optional: true
+
+      validates :reference_no, presence: true, uniqueness: true
+      validates :plan_type, presence: true, inclusion: { in: PLAN_TYPES }
+      validate :validate_region
+
+      def validate_region
+        return unless plan_type
+
+        errors.add(:region, 'should be null.') if plan_type == NON_REGIONAL && region
+
+        errors.add(:region, 'should not be null.') if plan_type == REGIONAL && !region
+
+        errors.add(:region, 'is not valid.') if region && region.location_type != Location::REGION
+      end
+    end
+  end
+end

data/app/models/cats/core/transport_plan_item.rb

@@ -0,0 +1,14 @@
+module Cats
+  module Core
+    class TransportPlanItem < ApplicationRecord
+      belongs_to :route
+      belongs_to :transport_plan
+      belongs_to :unit, class_name: 'Cats::Core::UnitOfMeasure'
+
+      validates :quantity, presence: true, numericality: { greater_than: 0 }
+
+      delegate(:name, to: :route, prefix: true)
+      delegate(:name, to: :unit, prefix: true)
+    end
+  end
+end

data/db/migrate/20211215114737_create_cats_core_transport_plans.rb

@@ -0,0 +1,14 @@
+class CreateCatsCoreTransportPlans < ActiveRecord::Migration[6.1]
+  def change
+    create_table :cats_core_transport_plans do |t|
+      t.string :reference_no, unique: true
+      t.string :plan_type, null: false, default: 'Regional'
+      t.references :region,
+                   null: true,
+                   index: { name: 'region_on_tp_indx' },
+                   foreign_key: { to_table: :cats_core_locations }
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20211215114835_create_cats_core_transport_plan_items.rb

@@ -0,0 +1,22 @@
+class CreateCatsCoreTransportPlanItems < ActiveRecord::Migration[6.1]
+  def change
+    create_table :cats_core_transport_plan_items do |t|
+      t.references :route,
+                   null: false,
+                   index: { name: 'route_on_tpi_indx' },
+                   foreign_key: { to_table: :cats_core_routes }
+      t.references :transport_plan,
+                   null: false,
+                   index: { name: 'tp_on_tpi_indx' },
+                   foreign_key: { to_table: :cats_core_transport_plans }
+      t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_tpi_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
+      t.boolean :planned, null: false, default: false
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20211215121151_create_cats_core_transport_bids.rb

@@ -6,12 +6,11 @@ class CreateCatsCoreTransportBids < ActiveRecord::Migration[6.1]
       t.date :start_date, null: false
       t.date :end_date, null: false
       t.string :status, null: false, default: 'New'
-      t.string :bid_type, null: false, default: 'Regional'
-      t.references :region,
-                   null: true,
-                   index: { name: 'region_on_tb_indx' },
-                   foreign_key: { to_table: :cats_core_locations } 
       t.float :bid_bond_amount, null: false, default: 0
+      t.references :transport_plan,
+                   null: false,
+                   index: { name: 'tp_on_tb_indx' },
+                   foreign_key: { to_table: :cats_core_transport_plans } 
 
       t.timestamps
     end

data/db/migrate/20211215124452_create_cats_core_transport_bid_items.rb

@@ -5,17 +5,15 @@ class CreateCatsCoreTransportBidItems < ActiveRecord::Migration[6.1]
                    null: false,
                    index: { name: 'tb_on_tbi_indx' },
                    foreign_key: { to_table: :cats_core_transport_bids }
-      t.references :route,
+      t.references :transport_plan_item,
                    null: false,
-                   index: { name: 'route_on_tbi_indx' },
-                   foreign_key: { to_table: :cats_core_routes }
-      t.references :unit,
-                   null: true,
-                   index: { name: 'unit_on_tbi_indx' },
-                   foreign_key: { to_table: :cats_core_unit_of_measures }
+                   index: { name: 'tpi_on_tbi_indx' },
+                   foreign_key: { to_table: :cats_core_transport_plan_items }
       t.float :quantity, null: false
 
       t.timestamps
     end
+
+    add_index :cats_core_transport_bid_items, :transport_plan_item_id, unique: true
   end
 end

data/lib/cats/core/version.rb

@@ -1,5 +1,5 @@
 module Cats
   module Core
-    VERSION = '1.2.35'.freeze
+    VERSION = '1.2.36'.freeze
   end
 end

data/spec/factories/cats/core/transport_bid_items.rb

@@ -1,8 +1,7 @@
 FactoryBot.define do
   factory :transport_bid_item, class: 'Cats::Core::TransportBidItem' do
     transport_bid
-    route
-    unit factory: :unit_of_measure
+    transport_plan_item
     quantity { 100 }
   end
 end

data/spec/factories/cats/core/transport_bids.rb

@@ -4,8 +4,7 @@ FactoryBot.define do
     description { FFaker::Name.name }
     start_date { Date.today - 1.week }
     end_date { Date.today }
-    bid_type { Cats::Core::TransportBid::REGIONAL }
-    region factory: :location
     bid_bond_amount { 100 }
+    transport_plan
   end
 end

data/spec/factories/cats/core/transport_plan_items.rb

@@ -0,0 +1,9 @@
+FactoryBot.define do
+  factory :transport_plan_item, class: 'Cats::Core::TransportPlanItem' do
+    route
+    transport_plan
+    quantity { 100 }
+    unit factory: :unit_of_measure
+    planned { false }
+  end
+end

data/spec/factories/cats/core/transport_plans.rb

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :transport_plan, class: 'Cats::Core::TransportPlan' do
+    reference_no { FFaker::Name.name }
+    plan_type { Cats::Core::TransportPlan::REGIONAL }
+    region factory: :location
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cats_core
 version: !ruby/object:Gem::Version
-  version: 1.2.35
+  version: 1.2.36
 platform: ruby
 authors:
 - Henock L.
@@ -296,6 +296,8 @@ files:
 - app/models/cats/core/transport_bid_item.rb
 - app/models/cats/core/transport_contract.rb
 - app/models/cats/core/transport_offer.rb
+- app/models/cats/core/transport_plan.rb
+- app/models/cats/core/transport_plan_item.rb
 - app/models/cats/core/transporter.rb
 - app/models/cats/core/unit_of_measure.rb
 - app/models/cats/core/user.rb
@@ -367,6 +369,8 @@ files:
 - db/migrate/20211002050739_create_cats_core_notification_rules.rb
 - db/migrate/20211024063240_add_status_to_cats_core_rhn_requests.rb
 - db/migrate/20211030133752_add_status_to_cats_core_commodities.rb
+- db/migrate/20211215114737_create_cats_core_transport_plans.rb
+- db/migrate/20211215114835_create_cats_core_transport_plan_items.rb
 - db/migrate/20211215121151_create_cats_core_transport_bids.rb
 - db/migrate/20211215124452_create_cats_core_transport_bid_items.rb
 - db/migrate/20211229160125_create_cats_core_transport_offers.rb
@@ -423,6 +427,8 @@ files:
 - spec/factories/cats/core/transport_bids.rb
 - spec/factories/cats/core/transport_contracts.rb
 - spec/factories/cats/core/transport_offers.rb
+- spec/factories/cats/core/transport_plan_items.rb
+- spec/factories/cats/core/transport_plans.rb
 - spec/factories/cats/core/transporters.rb
 - spec/factories/cats/core/unit_of_measures.rb
 - spec/factories/cats/core/users.rb