cats_core 1.1.1 → 1.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e1410fa75e3f9e4eb96e4c9df8cbd6bb358bb56e4cce6ef42b095aff3b6331b1
-  data.tar.gz: c1529759ffa5ba650bd31df10ae302943afc14e876fff0710a35144cf0841981
+  metadata.gz: 1edf27e2257545cf71d59da4348838a7d413911a3977e505e9ea38788bbf6bc0
+  data.tar.gz: 7b11450abaf98eed4cde168465e0f778be42026895d52883760bbe5cb0b571ae
 SHA512:
-  metadata.gz: 9b31f50e3f08e9a65c1da38520f3afef988613654e5250607e48761274951d3be4975e6b47c704c65ad6211d03081ab1c60b3f6ae1eea6fae191d498bd021110
-  data.tar.gz: 4046501c4c360769d9bc23628f4e067b8c7b55d95f898d57fda4f202f75ddc73fa5d5a60e5f163cf37ab340024b12a28065bb80f8be9eb5f5fb64d81151e8863
+  metadata.gz: 26de9f25dc8533a8ad33089cedacf51feff5b7ecfba6c08bfa55d811e7672cd4892955bccdd853a7281ec677a31478c9e3f8ecc2e421c79cbdeb34559b15c7d2
+  data.tar.gz: 26c4b80b13de8a60a02545382205877c8a252beebf2943ca990860a5bd32652d1152b865a90940dc0a55eebfca1c3bde974bfb6dfa07addaf174c59a5e74674a

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

@@ -52,7 +52,7 @@ module Cats
       end
 
       def search
-        data = @service.search(current_user)
+        data = @service.search(current_user, params[:status])
         render json: { success: true, data: serialize(data) }
       end
 

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

@@ -1,50 +1,20 @@
 module Cats
   module Core
     class LocationsController < ApplicationController
-      before_action :set_location, only: %i[show update]
+      include Common
 
       def index
         locations = Cats::Core::Location.where(location_type: params[:location_type])
-        data = ActiveModelSerializers::SerializableResource.new(locations)
-        render json: { success: true, data: data }
-      end
-
-      def show
-        data = ActiveModelSerializers::SerializableResource.new(@location)
-        render json: { success: true, data: data }
+        render json: { success: true, data: serialize(locations) }
       end
 
       def children
         parent = Cats::Core::Location.find(params[:id])
-        data = ActiveModelSerializers::SerializableResource.new(parent.children)
-        render json: { success: true, data: data }
-      end
-
-      def create
-        obj = Cats::Core::Location.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 @location.update(model_params)
-          data = ActiveModelSerializers::SerializableResource.new(@location)
-          render json: { success: true, data: data }
-        else
-          render json: { success: false, error: @location.errors.full_messages[0] }, status: :unprocessable_entity
-        end
+        render json: { success: true, data: serialize(parent.children) }
       end
 
       private
 
-      def set_location
-        @location = Cats::Core::Location.find(params[:id])
-      end
-
       def model_params
         params.require(:payload).permit(:code, :name, :location_type, :description, :parent_id)
       end

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

@@ -0,0 +1,13 @@
+module Cats
+  module Core
+    class ReceiptTransactionsController < ApplicationController
+      include Common
+
+      private
+
+      def model_params
+        params.require(:payload).permit(:source_id, :destination_id, :transaction_date, :quantity)
+      end
+    end
+  end
+end

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

@@ -8,6 +8,24 @@ module Cats
         render json: { success: true, data: serialize(receipts) }
       end
 
+      def start_stacking
+        receipt = Cats::Core::Receipt.find(params[:id])
+        service = ReceiptService.new
+        result = service.start_stacking(receipt)
+        render json: { success: true, data: serialize(result) }
+      rescue StandardError => e
+        render json: { success: false, error: e.message }
+      end
+
+      def stack
+        receipt = Cats::Core::Receipt.find(params[:id])
+        service = ReceiptService.new
+        result = service.stack(receipt)
+        render json: { success: true, data: serialize(result) }
+      rescue StandardError => e
+        render json: { success: false, error: e.message }
+      end
+
       private
 
       def model_params

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

@@ -5,6 +5,16 @@ module Cats
 
       validates :reference_no, presence: true, uniqueness: true
       validates :gift_date, presence: true
+
+      after_create :update_donation
+
+      def update_donation
+        donation = self.donation
+        return if donation.active
+
+        donation.active = true
+        donation.save!
+      end
     end
   end
 end

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

@@ -8,6 +8,16 @@ module Cats
       validates :reference_no, uniqueness: true
 
       delegate(:name, to: :commodity_category, prefix: true)
+
+      after_create :update_donation
+
+      def update_donation
+        donation = self.donation
+        return if donation.active
+
+        donation.active = true
+        donation.save!
+      end
     end
   end
 end

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

@@ -4,10 +4,13 @@ module Cats
       # Receipt status
       DRAFT = 'Draft'.freeze
       CONFIRMED = 'Confirmed'.freeze
-      RECEIPT_STATUSES = [DRAFT, CONFIRMED].freeze
+      STACKING = 'Stacking'.freeze
+      STACKED = 'Stacked'.freeze
+      RECEIPT_STATUSES = [DRAFT, CONFIRMED, STACKING, STACKED].freeze
 
       belongs_to :dispatch
       belongs_to :prepared_by, class_name: 'Cats::Core::User'
+      has_many :receipt_transactions, foreign_key: :source_id
 
       validates :quantity, :commodity_status, :status, presence: true
       validates :quantity, numericality: { greater_than: 0 }
@@ -21,19 +24,26 @@ module Cats
       def validate_dispatch_status
         return unless dispatch
 
-        return if dispatch.dispatch_status == Cats::Core::Dispatch::STARTED
+        statuses = [Cats::Core::Dispatch::STARTED, Cats::Core::Dispatch::RECEIVED]
+        return if statuses.include?(dispatch.dispatch_status)
 
-        errors.add(:dispatch, 'Dispatch should be in "Started" state.')
+        errors.add(:dispatch, 'should be in "Started" state.')
       end
 
       def validate_total_quantity
         return unless dispatch && quantity
 
-        received = Cats::Core::Receipt.where(dispatch: dispatch).sum(:quantity)
-        remaining = dispatch.quantity - received
-        return if quantity <= remaining
+        received = dispatch.receipts.sum(:quantity)
+        diff = dispatch.quantity - received
+        if new_record?
+          return if quantity <= diff
 
-        errors.add(:quantity, "Total receipt quantity is higher than dispatch quantity (Max = #{remaining}).")
+          errors.add(:quantity, "total is higher than dispatch quantity (Max = #{diff}).")
+        else
+          return unless diff.negative?
+
+          errors.add(:quantity, "total is higher than dispatch quantity (Max = #{diff.abs}).")
+        end
       end
     end
   end

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

@@ -3,6 +3,8 @@ module Cats
     class ReceiptTransaction < Transaction
       belongs_to :source, class_name: 'Cats::Core::Receipt'
       belongs_to :destination, class_name: 'Cats::Core::Stack'
+
+      delegate(:code, to: :destination, prefix: true)
     end
   end
 end

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

@@ -12,12 +12,14 @@ module Cats
       validates :transaction_date, :quantity, :status, presence: true
       validates :quantity, numericality: { greater_than: 0 }
       validates :status, inclusion: { in: STATUSES }
-      validate :validate_quantity
+      validate :validate_quantity, unless: :skip_quantity_validation
 
       def validate_quantity
         return unless quantity.present? && source.present?
 
-        errors.add(:quantity, 'cannot be more than source quantity') if quantity > source.quantity
+        total = self.class.where(source: source).sum(:quantity)
+        diff = source.quantity - total
+        errors.add(:quantity, "total is higher than source quantity (Max = #{diff}).") if quantity > diff
       end
 
       def commit
@@ -27,10 +29,23 @@ module Cats
           source.save
           destination.save
           self.status = COMMITTED
-          save
+          save!
         end
       end
 
+      def skip_quantity_validation
+        # Quantity validation should be skipped if we are commiting transactions.
+        (
+          instance_of?(Cats::Core::DispatchTransaction) &&
+          destination &&
+          destination.dispatch_status == Cats::Core::Dispatch::STARTED
+        ) || (
+          instance_of?(Cats::Core::ReceiptTransaction) &&
+          source &&
+          source.status == Cats::Core::Receipt::STACKING
+        )
+      end
+
       def set_status
         return unless new_record?
 

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

@@ -0,0 +1,8 @@
+class ReceiptTransactionSerializer < ActiveModel::Serializer
+  attributes :id, :source_id, :dispatch_reference, :destination_id, :destination_code, :quantity, :transaction_date,
+             :status
+
+  def dispatch_reference
+    object.source.dispatch.reference_no
+  end
+end

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

@@ -40,7 +40,7 @@ module Cats
         dispatch
       end
 
-      def search(user)
+      def search(user, status)
         details = user.details
 
         unless details['stores'] || details['warehouse'] || details['hub']
@@ -54,12 +54,12 @@ module Cats
           dispatches = Cats::Core::Dispatch.joins(:allocation_item)
                                            .where(
                                              allocation_item: { destination: hub },
-                                             dispatch_status: Cats::Core::Dispatch::STARTED
+                                             dispatch_status: status
                                            ).or(
                                              Cats::Core::Dispatch.joins(:allocation_item)
                                                                  .where(
                                                                    allocation_item: { destination: warehouse },
-                                                                   dispatch_status: Cats::Core::Dispatch::STARTED
+                                                                   dispatch_status: status
                                                                  )
                                            )
         elsif details['warehouse']
@@ -68,12 +68,12 @@ module Cats
           dispatches = Cats::Core::Dispatch.joins(:allocation_item)
                                            .where(
                                              allocation_item: { destination: hub },
-                                             dispatch_status: Cats::Core::Dispatch::STARTED
+                                             dispatch_status: status
                                            ).or(
                                              Cats::Core::Dispatch.joins(:allocation_item)
                                                                  .where(
                                                                    allocation_item: { destination: warehouse },
-                                                                   dispatch_status: Cats::Core::Dispatch::STARTED
+                                                                   dispatch_status: status
                                                                  )
                                            )
         elsif details['hub']
@@ -81,7 +81,7 @@ module Cats
           dispatches = Cats::Core::Dispatch.joins(:allocation_item)
                                            .where(
                                              allocation_item: { destination: hub },
-                                             dispatch_status: Cats::Core::Dispatch::STARTED
+                                             dispatch_status: status
                                            )
         end
         dispatches
@@ -126,7 +126,7 @@ module Cats
           dispatch.dispatch_status = Cats::Core::Dispatch::RECEIVED
           dispatch.receipts.each { |r| r.status = Cats::Core::Receipt::CONFIRMED }
           dispatch.save
-          dispatch.receipts.each(&:save)
+          dispatch.receipts.each(&:save!)
         end
 
         dispatch

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

@@ -0,0 +1,32 @@
+module Cats
+  module Core
+    class ReceiptService
+      def start_stacking(receipt)
+        raise(StandardError, 'Receipt should be confirmed.') unless receipt.status == Cats::Core::Receipt::CONFIRMED
+
+        raise(StandardError, 'There are no stack assignments in receipt.') if receipt.receipt_transactions.count.zero?
+
+        receipt.status = Cats::Core::Receipt::STACKING
+        receipt.save!
+        receipt
+      end
+
+      def stack(receipt)
+        unless receipt.status == Cats::Core::Receipt::STACKING
+          raise(StandardError, 'Receipt should be in stacking state.')
+        end
+
+        receipt.status = Cats::Core::Receipt::STACKED
+        stacks = receipt.receipt_transactions.map(&:destination)
+        stacks.each { |stack| stack.stack_status = Cats::Core::Stack::ALLOCATED }
+
+        Cats::Core::Receipt.transaction do
+          receipt.receipt_transactions.each(&:commit)
+          stacks.each(&:save!)
+          receipt.save!
+        end
+        receipt
+      end
+    end
+  end
+end

data/config/routes.rb

@@ -66,5 +66,11 @@ Cats::Core::Engine.routes.draw do
     end
   end
   resources :dispatch_transactions, except: %i[new edit destroy]
-  resources :receipts, except: %i[index new edit destroy]
+  resources :receipts, except: %i[index new edit destroy] do
+    member do
+      post 'start_stacking'
+      post 'stack'
+    end
+  end
+  resources :receipt_transactions, except: %i[new edit destroy]
 end

data/db/migrate/20210717032330_create_cats_core_donations.rb

@@ -25,6 +25,7 @@ class CreateCatsCoreDonations < ActiveRecord::Migration[6.1]
                    null: true,
                    index: { name: 'uom_on_donation_indx' },
                    foreign_key: { to_table: :cats_core_unit_of_measures }
+      t.boolean :active, null: false, default: false
 
       t.timestamps
     end

data/lib/cats/core/version.rb

@@ -1,5 +1,5 @@
 module Cats
   module Core
-    VERSION = '1.1.1'.freeze
+    VERSION = '1.1.5'.freeze
   end
 end

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

@@ -9,5 +9,6 @@ FactoryBot.define do
     currency { nil }
     commodity_category
     unit_of_measure
+    active { false }
   end
 end

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

@@ -1,5 +1,5 @@
 FactoryBot.define do
-  factory :receipt_transaction do
+  factory :receipt_transaction, class: 'Cats::Core::ReceiptTransaction' do
     source factory: :receipt
     destination factory: :stack
     transaction_date { Date.today }

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

@@ -1,7 +1,7 @@
 FactoryBot.define do
   factory :receipt, class: 'Cats::Core::Receipt' do
     dispatch factory: :dispatch, dispatch_status: Cats::Core::Dispatch::STARTED
-    quantity { 1.5 }
+    quantity { 50 }
     commodity_status { Cats::Core::Commodity::GOOD }
     status { Cats::Core::Receipt::DRAFT }
     remark { FFaker::Name.name }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cats_core
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.5
 platform: ruby
 authors:
 - Henock L.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-25 00:00:00.000000000 Z
+date: 2021-09-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_model_serializers
@@ -225,6 +225,7 @@ files:
 - app/controllers/cats/core/locations_controller.rb
 - app/controllers/cats/core/menus_controller.rb
 - app/controllers/cats/core/notifications_controller.rb
+- app/controllers/cats/core/receipt_transactions_controller.rb
 - app/controllers/cats/core/receipts_controller.rb
 - app/controllers/cats/core/roles_controller.rb
 - app/controllers/cats/core/spaces_controller.rb
@@ -277,11 +278,13 @@ files:
 - app/serializers/cats/core/dispatch_transaction_serializer.rb
 - app/serializers/cats/core/location_serializer.rb
 - app/serializers/cats/core/receipt_serializer.rb
+- app/serializers/cats/core/receipt_transaction_serializer.rb
 - app/serializers/cats/core/role_menu_serializer.rb
 - app/serializers/cats/core/unit_of_measure_serializer.rb
 - app/services/cats/core/dispatch_service.rb
 - app/services/cats/core/menu_service.rb
 - app/services/cats/core/notification_service.rb
+- app/services/cats/core/receipt_service.rb
 - app/services/cats/core/space_service.rb
 - app/services/cats/core/token_auth_service.rb
 - config/routes.rb