cats_core 1.1.37 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f07fe2e24b72c5cf2c4c7d7c9acd92332e6e7a80b4d9305fdffcbb94365fb67e
-  data.tar.gz: 6c343a483c11b17757966c650fa355a541a62483eafa9d6ad5b5428ebb1aef63
+  metadata.gz: ed6937d9d787e6b9cd4861aed52c82cddec427a2607f86e6b76ee835dbf66c0b
+  data.tar.gz: 99f350b323761a10f70058b01cceda92acd6c92d2824f06d011effaad3cd4780
 SHA512:
-  metadata.gz: 5c7a27a3e6babd99f6e331c63d6d79ecfb6af34e5c6ae746ccd538d11e1ea62e48ea9c6b4b84eb5d1500da1466992f2acae40297127e5822c12b4f8ed418a728
-  data.tar.gz: 930ccb77011e1cabc7ce9851fe22584f50346d1494d4556df7df90159fb9f2d99b3340c0f9ff1f7f5cc0a7b334f5274e74b67191d48abecf709f04af27b553b5
+  metadata.gz: 67d812874a6b1680fff22c83dff0ac53193585ac9136e9753ef28db9f0210cb503d9b0f73c19d73aa45b55f042e2ff4e2787af35ef713a0ed12029c982b789a9
+  data.tar.gz: bca2a928d539658137bb2b545ab0ea2f98611312e2c4cd8a0dffa3fe415b19b4771dd97a5550631ccc8689817686affdb0c074a93aeb479a78fbca98d4e758c5

data/app/controllers/cats/core/access_controller.rb

@@ -20,7 +20,7 @@ module Cats
             jwt = Cats::Core::TokenAuthService.issue(payload)
             render json: { token: jwt, user: payload }
           else
-            render json: { error: 'Invalid username or password.' }, status: 400
+            render json: { error: 'Invalid password.' }, status: 400
           end
         else
           render json: { error: 'User does not exist.' }, status: 400

data/app/controllers/cats/core/currencies_controller.rb

@@ -1,43 +1,10 @@
 module Cats
   module Core
     class CurrenciesController < ApplicationController
-      before_action :set_currency, only: %i[show update]
-
-      def index
-        data = ActiveModelSerializers::SerializableResource.new(Cats::Core::Currency.all)
-        render json: { success: true, data: data }
-      end
-
-      def show
-        data = ActiveModelSerializers::SerializableResource.new(@currency)
-        render json: { success: true, data: data }
-      end
-
-      def create
-        obj = Cats::Core::Currency.new(model_params)
-        if obj.save
-          data = ActiveModelSerializers::SerializableResource.new(obj)
-          render json: { success: true, data: data }, status: :created
-        else
-          render json: { success: false, error: obj.errors.full_messages[0] }, status: :unprocessable_entity
-        end
-      end
-
-      def update
-        if @currency.update(model_params)
-          data = ActiveModelSerializers::SerializableResource.new(@currency)
-          render json: { success: true, data: data }
-        else
-          render json: { success: false, error: @currency.errors.full_messages[0] }, status: :unprocessable_entity
-        end
-      end
+      include Common
 
       private
 
-      def set_currency
-        @currency = Cats::Core::Currency.find(params[:id])
-      end
-
       def model_params
         params.require(:payload).permit(:code, :name)
       end

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

@@ -57,7 +57,7 @@ module Cats
       def validate_commodity_status
         return unless commodity
 
-        errors.add(:commodity, 'should be approved first.') unless commodity.approved
+        errors.add(:commodity, 'should be approved first.') unless commodity.status == Commodity::APPROVED
       end
 
       def approve
@@ -68,6 +68,10 @@ module Cats
 
         self.allocation_status = Allocation::APPROVED
         save!
+
+        # If the commodity is all allocated, then we should
+        # set its status to allocated.
+        commodity.allocate
       end
     end
   end

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

@@ -1,7 +1,11 @@
 module Cats
   module Core
     class Commodity < ApplicationRecord
-      after_initialize :set_approved
+      # Statuses
+      DRAFT = 'Draft'.freeze
+      APPROVED = 'Approved'.freeze
+      ALLOCATED = 'Allocated'.freeze
+      STATUSES = [DRAFT, APPROVED, ALLOCATED].freeze
 
       # Commodity statuses
       GOOD = 'Good'.freeze
@@ -17,23 +21,67 @@ module Cats
       belongs_to :unit_of_measure
       belongs_to :source, polymorphic: true
 
+      has_many :allocations
+
       validates :best_use_before, presence: true
       validates :batch_no, presence: true, uniqueness: true
       validates :quantity, presence: true, numericality: { greater_than: 0 }
       validates :volume_per_metric_ton, numericality: { greater_than: 0, allow_nil: true }
       validates :arrival_status, presence: true, inclusion: { in: ARRIVAL_STATUSES }
+      validates :status, presence: true, inclusion: { in: STATUSES }
 
       delegate(:abbreviation, to: :unit_of_measure, prefix: true)
       delegate(:reference_no, to: :source, prefix: true)
 
-      def set_approved
-        return unless new_record?
+      def name
+        source.commodity_category.name
+      end
+
+      def approve
+        raise(StandardError, 'Commodity already approved.') if status == APPROVED
+
+        result = false
+        Cats::Core::Commodity.transaction do
+          self.status = APPROVED
+          save!
+          store = Cats::Core::Store.find_by(code: 'SUP-STORE')
+          raise(StandardError, 'Supplier store does not exist.') unless store
 
-        self.approved = false
+          stack = Cats::Core::Stack.create!(
+            code: batch_no,
+            store: store,
+            commodity: self,
+            quantity: quantity,
+            commodity_status: GOOD,
+            stack_status: Stack::ALLOCATED,
+            length: 100,
+            width: 100,
+            height: 10,
+            start_x: 1,
+            start_y: 1
+          )
+          result = stack.id.positive?
+        end
+        result
       end
 
-      def name
-        source.commodity_category.name
+      def allocate(ignore_errors: true)
+        statuses = Allocation.where(commodity_id: id).map(&:allocation_status).uniq
+        unless statuses.count == 1 && statuses[0] == APPROVED
+          return if ignore_errors
+
+          raise(StandardError, 'There are allocations in "Draft" state.')
+        end
+
+        quantity = allocations.sum(:quantity)
+        unless quantity == self.quantity
+          return if ignore_errors
+
+          raise(StandardError, 'Total allocations quantity is not the same as commodity quantity.')
+        end
+
+        self.status = ALLOCATED
+        save!
       end
     end
   end

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

@@ -18,7 +18,7 @@ module Cats
       def validate_commodity_status
         return unless commodity
 
-        errors.add(:commodity, 'should be approved first.') unless commodity.approved
+        errors.add(:commodity, 'should be approved first.') unless commodity.status == Commodity::APPROVED
       end
 
       def validate_quantity
@@ -30,6 +30,11 @@ module Cats
         remaining += quantity_was if quantity_was
         errors.add(:quantity, "exceeds commodity quantity. Maximum allowed is #{remaining}") if quantity > remaining
       end
+
+      def approve
+        self.status = APPROVED
+        save!
+      end
     end
   end
 end

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

@@ -22,6 +22,8 @@ module Cats
       validate :validate_coordinates, :validate_dimensions, :validate_distance_from_wall, :validate_overlap,
                :validate_space_between_stack, unless: -> { store && store.code == 'SUP-STORE' }
 
+      delegate :batch_no, to: :commodity, prefix: true
+
       after_save :update_store_space
 
       def validate_coordinates
@@ -58,8 +60,8 @@ module Cats
         return unless store.present? && length.present? && width.present? && start_x.present? && start_y.present?
 
         rule = stacking_rules
-        if start_x < rule.distance_from_wall || store.length - (start_y + length) < rule.distance_from_wall ||
-           store.width - (start_x + width) < rule.distance_from_wall || start_y < rule.distance_from_wall
+        if start_x < rule.distance_from_wall || store.length - (start_x + length) < rule.distance_from_wall ||
+           store.width - (start_y + width) < rule.distance_from_wall || start_y < rule.distance_from_wall
           errors.add(:base,
                      message: "The stack must be placed #{rule.distance_from_wall} meter away from a store wall")
         end

data/app/serializers/cats/core/commodity_serializer.rb

@@ -3,7 +3,7 @@ module Cats
     class CommoditySerializer < ActiveModel::Serializer
       attributes :id, :name, :batch_no, :description, :unit_of_measure_id, :unit_of_measure_abbreviation, :source_id,
                  :source_type, :source_reference_no, :quantity, :best_use_before, :volume_per_metric_ton,
-                 :arrival_status, :approved
+                 :arrival_status, :status
     end
   end
 end

data/app/services/cats/core/receipt_service.rb

@@ -17,6 +17,7 @@ module Cats
         end
 
         receipt.status = Cats::Core::Receipt::STACKED
+        receipt.save!
         receipt.receipt_transactions.each(&:commit)
         receipt
       end

data/db/migrate/20211030133752_add_status_to_cats_core_commodities.rb

@@ -0,0 +1,6 @@
+class AddStatusToCatsCoreCommodities < ActiveRecord::Migration[6.1]
+  def change
+    add_column :cats_core_commodities, :status, :string, null: false, default: 'Draft'
+    remove_column :cats_core_commodities, :approved, :boolean
+  end
+end

data/lib/cats/core/version.rb

@@ -1,5 +1,5 @@
 module Cats
   module Core
-    VERSION = '1.1.37'.freeze
+    VERSION = '1.2.0'.freeze
   end
 end

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

@@ -2,7 +2,11 @@ FactoryBot.define do
   factory :allocation, class: 'Cats::Core::Allocation' do
     reference_no { FFaker::Name.name }
     rhn_request { nil }
-    commodity
+    commodity do
+      c = create(:commodity)
+      c.approve
+      c
+    end
     source factory: :location
     quantity { 50 }
     commodity_status { Cats::Core::Commodity::GOOD }

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

@@ -8,6 +8,6 @@ FactoryBot.define do
     best_use_before { Date.today + 2.month }
     volume_per_metric_ton { 10 }
     arrival_status { Cats::Core::Commodity::AT_SOURCE }
-    approved { true }
+    status { Cats::Core::Commodity::DRAFT }
   end
 end

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

@@ -1,7 +1,11 @@
 FactoryBot.define do
   factory :rhn_request, class: 'Cats::Core::RhnRequest' do
     reference_no { FFaker::Name.name }
-    commodity
+    commodity do
+      c = create(:commodity)
+      c.approve
+      c
+    end
     quantity { 100 }
     request_date { Date.today }
     requested_by { FFaker::Name.name }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cats_core
 version: !ruby/object:Gem::Version
-  version: 1.1.37
+  version: 1.2.0
 platform: ruby
 authors:
 - Henock L.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-25 00:00:00.000000000 Z
+date: 2021-11-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_model_serializers
@@ -346,6 +346,7 @@ files:
 - db/migrate/20210814175406_create_cats_core_stack_transactions.rb
 - 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
 - lib/cats/core.rb
 - lib/cats/core/engine.rb
 - lib/cats/core/version.rb