cats_core 1.4.45 → 1.4.49

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 710ffdc5b10d0f97aa5f40aaa8e67a99eaea21120c0b96894d46f86a47b10312
-  data.tar.gz: 3c1b7594b8717c668bb9f38d481afa54057005f524171bd3c088fdf6047344b2
+  metadata.gz: '08b9c239346db9904b66db3df8c7c3461f5c3766c6739b3a039cbc96e3d7fa2e'
+  data.tar.gz: 4838b7d3720606e0581d364888771a39ec2c3c69c164bfebe838e2d8faac9314
 SHA512:
-  metadata.gz: 4b522fa9d586d12f3255b0a9044caff4c8b694e336e3b284a657bc14d2f10a0e10ad3fce4d082784b65435bd73c396b8a593c6c7630bc90b9ac1388bf6f51915
-  data.tar.gz: 46fbc7ca6360f71e4f6cc992f03a3711b302f2f351c7dca8996999cee8c8eaf196321324bf9ed3f6bf16940c04f5224c72bd5124147663a7b4533aeddd9f27bd
+  metadata.gz: 915436bd0e4a48d08809aedc9cbd6317c89ed6c45d2d2356f75805c41f7abb2c756173b0140a84ecad77cf2483058f42106eb31d692f75f80b874c555b59b060
+  data.tar.gz: 324ca7a987fbda8bb7dd50e06c84daeeb58acf0743335f8ce39296dcde740b54503e8caca431b159984b59ad678061c3586838e8607f9dadb12581bf972b289c

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

@@ -25,11 +25,22 @@ module Cats
         render json: { success: true, data: serialize(woredas) }
       end
 
+      def create_fdp
+        service = LocationService.new
+        p = fdp_params
+        fdp = service.create_fdp(p[:code], p[:name], p[:parent_id], p[:description])
+        render json: { success: true, data: serialize(fdp) }
+      end
+
       private
 
       def model_params
         params.require(:payload).permit(:code, :name, :location_type, :description, :parent_id)
       end
+
+      def fdp_params
+        params.require(:payload).permit(:code, :name, :description, :parent_id)
+      end
     end
   end
 end

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

@@ -4,28 +4,60 @@ module Cats
       include Common
 
       def index
-        stores = Store.includes(:store_keeper, :stacks).where(warehouse_id: params[:id])
+        stores = Store.includes(:stacks).where(warehouse_id: params[:id])
         render json: { success: true, data: serialize(stores) }
       end
 
       def filter
-        query = Store.includes(:store_keeper, :stacks).ransack(params[:q])
+        query = Store.includes(:stacks).ransack(params[:q])
         render json: { success: true, data: serialize(query.result) }
       end
 
       def stores_for_hub
         warehouses = Location.find(params[:id]).children
         ids = warehouses.map(&:id)
-        stores = Store.includes(:store_keeper, :stacks).where(warehouse_id: ids)
+        stores = Store.includes(:stacks).where(warehouse_id: ids)
         render json: { success: true, data: serialize(stores) }
       end
 
+      def assign_store
+        store_keeper = User.find(params[:id])
+        store_keeper.details['stores'] ||= []
+        store_keeper.details['stores'] += store_params[:store_ids]
+        store_keeper.save!
+        render json: { success: true, data: serialize(store_keeper) }
+      end
+
+      def unassign_store
+        store_keeper = User.find(params[:id])
+        if store_keeper.details.key?('stores')
+          store_keeper.details['stores'] -= store_params[:store_ids]
+          store_keeper.save!
+          render json: { success: true, data: serialize(store_keeper) }
+        else
+          render json: { success: false, error: 'User has no stores assigned.' }
+        end
+      end
+
+      def fdp
+        store = Store.find(params[:id])
+        location = store.warehouse.parent
+        if location.location_type == Location::FDP
+          render json: { success: true, data: serialize(location) }
+        else
+          render json: { success: false, error: 'No FDP is associated with store.' }
+        end
+      end
+
       private
 
       def model_params
         params.require('payload').permit(:id, :code, :name, :length, :width, :height, :temporary, :has_gangway,
-                                         :gangway_length, :gangway_width, :gangway_corner_dist, :warehouse_id,
-                                         :store_keeper_id)
+                                         :gangway_length, :gangway_width, :gangway_corner_dist, :warehouse_id)
+      end
+
+      def store_params
+        params.require(:payload).permit(store_ids: [])
       end
     end
   end

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

@@ -60,7 +60,7 @@ module Cats
       def name
         # For this to be efficient, our query should use includes like
         # Commodity.includes(project: { source: commodity_category })
-        project.source.commodity_category.name
+        project.source.commodity_name
       end
 
       def validate_updateability

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

@@ -75,6 +75,21 @@ module Cats
       def start
         self.dispatch_status = STARTED
         save!
+
+        # Check if destination is an FDP and create receipt authorization
+        location = dispatch_plan_item.destination
+        authorized_by = User.find_by(first_name: 'SYSTEM-USER')
+        return unless location.location_type == Location::FDP
+
+        store = location.fdp_store
+        ReceiptAuthorization.create!(
+          dispatch: self,
+          store: store,
+          quantity: quantity,
+          received_quantity: 0,
+          status: Authorization::AUTHORIZED,
+          authorized_by: authorized_by
+        )
       end
 
       def receive

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

@@ -5,11 +5,10 @@ module Cats
       ZONE = 'Zone'.freeze
       WOREDA = 'Woreda'.freeze
       FDP = 'Fdp'.freeze
-      KEBELE = 'Kebele'.freeze
       HUB = 'Hub'.freeze
       WAREHOUSE = 'Warehouse'.freeze
 
-      LOCATION_TYPES = [REGION, ZONE, WOREDA, KEBELE, HUB, WAREHOUSE, FDP].freeze
+      LOCATION_TYPES = [REGION, ZONE, WOREDA, FDP, HUB, WAREHOUSE].freeze
 
       has_ancestry
 
@@ -23,10 +22,9 @@ module Cats
           REGION => [],
           ZONE => [REGION],
           WOREDA => [REGION, ZONE],
-          KEBELE => [REGION, ZONE, WOREDA],
           FDP => [REGION, ZONE, WOREDA],
-          HUB => [REGION, ZONE, WOREDA, KEBELE],
-          WAREHOUSE => [REGION, ZONE, WOREDA, KEBELE, HUB]
+          HUB => [REGION, ZONE, WOREDA, FDP],
+          WAREHOUSE => [REGION, ZONE, WOREDA, FDP, HUB]
         }
 
         return if location_type.nil? || location_type.empty?
@@ -46,6 +44,12 @@ module Cats
         # wrong type of parent to our location.
         errors.add(:location, "cannot have #{parent.location_type} as parent")
       end
+
+      def fdp_store
+        return nil unless location_type == FDP
+
+        Store.find_by(code: "FDP-ST-#{code}")
+      end
     end
   end
 end

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

@@ -21,7 +21,8 @@ module Cats
       validates :stack_status, inclusion: { in: STACK_STATUSES }
       validate :validate_coordinates, :validate_dimensions, :validate_distance_from_wall, :validate_overlap,
                :validate_space_between_stack, :validate_max_height, :validate_max_length, :validate_max_width,
-               :validate_distance_from_ceiling, unless: -> { store && store.code == 'SUP-STORE' }
+               :validate_distance_from_ceiling,
+               unless: -> { store && (store.code == 'SUP-STORE' || store.code.start_with?('FDP-ST')) }
 
       delegate :batch_no, to: :commodity, prefix: true
 

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

@@ -2,7 +2,6 @@ module Cats
   module Core
     class Store < ApplicationRecord
       belongs_to :warehouse, class_name: 'Cats::Core::Location'
-      belongs_to :store_keeper, class_name: 'Cats::Core::User'
       has_many :stacks
 
       validates :code, :name, :length, :width, :height, presence: true
@@ -17,7 +16,6 @@ module Cats
       validate :validate_location
 
       before_create :update_usable_space
-      after_save :update_store_keeper
 
       def validate_location
         return if warehouse.nil?
@@ -25,33 +23,11 @@ module Cats
         errors.add(:warehouse, 'must be a valid warehouse') unless warehouse.location_type == Location::WAREHOUSE
       end
 
-      def store_keeper_name
-        "#{store_keeper.first_name} #{store_keeper.last_name}"
-      end
-
       def update_usable_space
         self.usable_space = (length - 2) * (width - 2)
         self.usable_space -= (gangway_length * gangway_width) if has_gangway
         self.available_space = self.usable_space
       end
-
-      def update_store_keeper
-        return unless store_keeper_id_previously_changed?
-
-        unless id_previously_changed?
-          old_storekeeper = Cats::Core::User.find(store_keeper_id_previously_was)
-          old_storekeeper.details['stores'].delete(id)
-          old_storekeeper.save
-        end
-        if store_keeper.details.key?('stores')
-          store_keeper.details['stores'] << id
-        else
-          store_keeper.details['stores'] = [id]
-        end
-        store_keeper.save
-      end
-
-      delegate(:phone_number, to: :store_keeper, prefix: true)
     end
   end
 end

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

@@ -1,9 +1,8 @@
 module Cats
   module Core
     class StoreSerializer < ActiveModel::Serializer
-      attributes :id, :code, :name, :store_keeper_name, :store_keeper_phone_number, :length, :width, :height,
-                 :temporary, :has_gangway, :gangway_length, :gangway_width, :gangway_corner_dist, :warehouse_id,
-                 :store_keeper_id
+      attributes :id, :code, :name, :length, :width, :height, :temporary, :has_gangway, :gangway_length,
+                 :gangway_width, :gangway_corner_dist, :warehouse_id
 
       has_many :stacks
     end

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

@@ -24,9 +24,42 @@ module Cats
         statuses = authorization.dispatch.receipt_authorizations.map(&:driver_confirmed).uniq
         authorization.dispatch.receive if statuses.length == 1 && statuses[0]
 
+        # Create destination stack and transaction for FDP
+        location = authorization.dispatch.dispatch_plan_item.destination
+        if location.location_type == Location::FDP
+          create_stack(authorization_id)
+          stack(authorization_id)
+        end
+
         authorization
       end
 
+      # A method to create stack and stack transactions for FDP
+      def create_stack(authorization_id)
+        authorization = ReceiptAuthorization.find(authorization_id)
+        commodity = authorization.dispatch.dispatch_plan_item.commodity
+        stack = Stack.create!(
+          code: commodity.batch_no,
+          length: 100,
+          width: 100,
+          height: 100,
+          start_x: 2,
+          start_y: 2,
+          commodity: commodity,
+          store: authorization.store,
+          commodity_status: Commodity::GOOD,
+          stack_status: Stack::ALLOCATED
+        )
+        ReceiptTransaction.create!(
+          receipt_authorization: authorization,
+          destination: stack,
+          quantity: authorization.received_quantity,
+          status: Transaction::DRAFT,
+          transaction_date: Date.today
+        )
+        stack
+      end
+
       def stack(authorization_id)
         authorization = ReceiptAuthorization.find(authorization_id)
         unless authorization.dispatch.dispatch_status == Dispatch::RECEIVED

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

@@ -0,0 +1,25 @@
+module Cats
+  module Core
+    class LocationService
+      # This method creates FDP from location params, but also creates the
+      # related dummy warehouse and store for the FDP
+      def create_fdp(code, name, parent_id, description = nil)
+        ActiveRecord::Base.transaction do
+          fdp = Location.create!(
+            code: code, name: name, description: description, location_type: Location::FDP, parent_id: parent_id
+          )
+          wh_code = "FDP-WH-#{fdp.code}"
+          st_code = "FDP-ST-#{fdp.code}"
+
+          warehouse = Location.create!(
+            code: wh_code, name: wh_code, location_type: Location::WAREHOUSE, parent: fdp
+          )
+          Store.create!(
+            code: st_code, name: st_code, length: 200, width: 200, height: 200, warehouse: warehouse
+          )
+          fdp
+        end
+      end
+    end
+  end
+end

data/config/routes.rb

@@ -41,6 +41,7 @@ Cats::Core::Engine.routes.draw do
 
   get '/regions/:id/woredas', controller: :locations, action: :woredas, as: :region_woredas
   post '/locations/filter', controller: :locations, action: :filter
+  post '/locations/fdp/create', controller: :locations, action: :create_fdp
   resources :locations, except: %i[destroy] do
     member do
       get 'children'
@@ -142,6 +143,18 @@ Cats::Core::Engine.routes.draw do
     action: :storekeeper_authorizations,
     as: :storekeeper_receipt_authorizations
   )
+  post(
+    '/storekeeper/:id/assign_stores',
+    controller: :stores,
+    action: :assign_store,
+    as: :assign_stores_to_storekeeper
+  )
+  post(
+    '/storekeeper/:id/unassign_stores',
+    controller: :stores,
+    action: :unassign_store,
+    as: :unassign_stores_to_storekeeper
+  )
   post(
     '/receipt_authorizations/generate',
     controller: :receipt_authorizations,
@@ -200,6 +213,10 @@ Cats::Core::Engine.routes.draw do
 
   get '/warehouses/:id/stores', controller: :stores, action: :index, as: :stores_warehouse
   get '/hubs/:id/stores', controller: :stores, action: :stores_for_hub, as: :stores_hub
-  resources :stores, only: %i[index show create update]
+  resources :stores, only: %i[index show create update] do
+    member do
+      get 'fdp'
+    end
+  end
   post '/stores/filter', controller: :stores, action: :filter
 end

data/db/migrate/20210717140855_create_cats_core_stores.rb

@@ -17,10 +17,7 @@ class CreateCatsCoreStores < ActiveRecord::Migration[6.1]
                    null: false,
                    index: { name: 'warehouse_on_stores_indx' },
                    foreign_key: { to_table: :cats_core_locations }
-      t.references :store_keeper,
-                   null: false,
-                   index: { name: 'store_keeper_on_store_indx' },
-                   foreign_key: { to_table: :cats_core_users }
+
       t.timestamps
     end
   end

data/lib/cats/core/version.rb

@@ -1,5 +1,5 @@
 module Cats
   module Core
-    VERSION = '1.4.45'.freeze
+    VERSION = '1.4.49'.freeze
   end
 end

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

@@ -2,7 +2,7 @@ FactoryBot.define do
   factory :dispatch_plan_item, class: 'Cats::Core::DispatchPlanItem' do
     reference_no { FFaker::Name.name }
     source factory: :location
-    destination factory: :location
+    destination factory: :woreda
     dispatch_plan
     quantity { 100 }
     unit factory: :unit_of_measure

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

@@ -19,6 +19,15 @@ FactoryBot.define do
   factory :fdp, parent: :woreda, class: 'Cats::Core::Location' do
     location_type { Cats::Core::Location::FDP }
     parent factory: :woreda
+
+    trait :with_store do
+      after(:create) do |obj|
+        wh_code = "FDP-WH-#{obj.code}"
+        st_code = "FDP-ST-#{obj.code}"
+        warehouse = create(:warehouse, code: wh_code, name: wh_code, parent: obj)
+        create(:store, code: st_code, name: st_code, warehouse: warehouse)
+      end
+    end
   end
 
   factory :hub, parent: :location, class: 'Cats::Core::Location' do

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

@@ -11,7 +11,5 @@ FactoryBot.define do
     gangway_width { 4 }
     gangway_corner_dist { 18 }
     warehouse
-    store_keeper factory: :user
-    store_keeper_id { create(:user).id }
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cats_core
 version: !ruby/object:Gem::Version
-  version: 1.4.45
+  version: 1.4.49
 platform: ruby
 authors:
 - Henock L.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-27 00:00:00.000000000 Z
+date: 2022-06-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_model_serializers
@@ -122,6 +122,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 6.0.0
+- !ruby/object:Gem::Dependency
+  name: sablon
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
@@ -377,6 +391,7 @@ files:
 - app/services/cats/core/beneficiary_service.rb
 - app/services/cats/core/dispatch_plan_service.rb
 - app/services/cats/core/dispatch_service.rb
+- app/services/cats/core/location_service.rb
 - app/services/cats/core/menu_service.rb
 - app/services/cats/core/notification_service.rb
 - app/services/cats/core/round_plan_service.rb