cats_core 1.4.46 → 1.4.48

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 35e8eb4c24f3e5499dae203f071d84e56efc538665557820ebb0c0c4b56a8d8e
-  data.tar.gz: fe6f7c522a28fcbdc53c0f61dfc7b0844970564de93952c134965055c0501cb3
+  metadata.gz: 6cd51ff40bb6a3dc2aa226e29abbcec7fe47e833095bfabfefbe6e32d4d15a28
+  data.tar.gz: 05726c3a11c58983b9360d9ff8cf4384927148cc67d825e9b21114c823c07e78
 SHA512:
-  metadata.gz: 5fad95b3e3b72ab44e84cb72954d9f825a2ff40be3c615685be577fd49e54fb123f6e6b4be4ad42ea1008bd9f64c2e42091666fad8f1c7e6546cb6368e1ae83b
-  data.tar.gz: 385a523e608cf19f3871495b94d0b3c52621bb14caaaf164daf13c08509e24a64b3fe4a044443d0cef1952a6899ce1e760c9dcfbe04cae29265bb9f8819fd1f3
+  metadata.gz: bea7b46396f1f5ab18cb30d0fbacaa2275f72840fd0fa0d27a0e33cab49be523eb2ddcaa5747d15bb3a5a9120987cf59ee2cae402f2d0e282d6fe72d115f7ac3
+  data.tar.gz: f7a48bf8792e57e5580a44e471a5efbf6e5a66aace12eb5b648685e7105e56b295c218ea6c7f4ad96cb31edadbe568245a3c781b16c1b483a120f777c55bee7b

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,59 @@ 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]
+          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/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

@@ -22,7 +22,7 @@ module Cats
       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' || store.code.start_with?('FDP;')) }
+               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.46'.freeze
+    VERSION = '1.4.48'.freeze
   end
 end

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.46
+  version: 1.4.48
 platform: ruby
 authors:
 - Henock L.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-28 00:00:00.000000000 Z
+date: 2022-06-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_model_serializers
@@ -377,6 +377,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