cats_core 1.5.19 → 1.5.22

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

@@ -2,8 +2,8 @@ module Cats
   module Core
     class SwapSerializer < ActiveModel::Serializer
       attributes :id, :reference_no, :swapper, :agreement_date, :issued_commodity_id, :issued_commodity_name,
-        :issued_quantity, :received_commodity_id, :received_commodity_name, :received_quantity,
-        :issued_unit_id, :issued_unit_abbreviation, :received_unit_id, :received_unit_abbreviation
+                 :issued_quantity, :received_commodity_id, :received_commodity_name, :received_quantity,
+                 :issued_unit_id, :issued_unit_abbreviation, :received_unit_id, :received_unit_abbreviation
     end
   end
 end

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

@@ -3,9 +3,7 @@ module Cats
     class AuthorizationService
       def driver_confirm(authorization_id, pin, receipt_number)
         authorization = ReceiptAuthorization.find(authorization_id)
-        if authorization.status == Authorization::AUTHORIZED
-          raise(StandardError, "Authorization not confirmed by storekeeper.")
-        end
+        raise(StandardError, "Authorization not confirmed by storekeeper.") if authorization.status == Authorization::AUTHORIZED
 
         raise(StandardError, "No pin has been generated for receipt.") if authorization.auth_details.nil?
 
@@ -65,14 +63,10 @@ module Cats
 
       def stack(authorization_id)
         authorization = ReceiptAuthorization.find(authorization_id)
-        unless authorization.dispatch.dispatch_status == Dispatch::RECEIVED
-          raise(StandardError, "Dispatch is not received.")
-        end
+        raise(StandardError, "Dispatch is not received.") unless authorization.dispatch.dispatch_status == Dispatch::RECEIVED
 
         total = authorization.transactions.sum(:quantity)
-        if total != authorization.received_quantity
-          raise(StandardError, "Received quantity is not the same as quantity to be stacked.")
-        end
+        raise(StandardError, "Received quantity is not the same as quantity to be stacked.") if total != authorization.received_quantity
 
         authorization.transactions.each(&:commit)
 

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

@@ -57,7 +57,7 @@ module Cats
         end
 
         beneficiaries = RoundBeneficiary.joins(:round_plan_item)
-          .where(round_plan_item: {round_plan_id: plan_id})
+                                        .where(round_plan_item: {round_plan_id: plan_id})
         to_delete = RoundBeneficiary.where(id: ids)
         diff = to_delete - beneficiaries
         raise(StandardError, "Round plan beneficiaries should be from the same plan.") if diff.count.positive?
@@ -76,7 +76,7 @@ module Cats
         end
 
         beneficiaries = RoundBeneficiary.joins(:round_plan_item)
-          .where(round_plan_item: {round_plan_id: plan_id})
+                                        .where(round_plan_item: {round_plan_id: plan_id})
         to_confirm = RoundBeneficiary.where(id: ids)
         diff = to_confirm - beneficiaries
         raise(StandardError, "Round plan beneficiaries should be from the same plan.") if diff.count.positive?

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

@@ -50,7 +50,7 @@ module Cats
           recipients = users.map do |user|
             details = user.details
             if (details.key?("warehouse") && details["warehouse"] == location_id) ||
-                (details.key?("hub") && details["hub"] == location_id)
+               (details.key?("hub") && details["hub"] == location_id)
               user
             end
           end.compact

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

@@ -4,33 +4,31 @@ module Cats
       def search(user, status, authorized: false)
         details = user.details
 
-        unless details["stores"] || details["warehouse"] || details["hub"]
-          raise(StandardError, "User does not have associated location.")
-        end
+        raise(StandardError, "User does not have associated location.") unless details["stores"] || details["warehouse"] || details["hub"]
 
         # Get user's hub
         hub = if details["stores"]
-          Store.find(details["stores"][0]).warehouse.parent
-        elsif details["warehouse"]
-          Location.find(details["warehouse"]).parent
-        else
-          Location.find(details["hub"])
-        end
+                Store.find(details["stores"][0]).warehouse.parent
+              elsif details["warehouse"]
+                Location.find(details["warehouse"]).parent
+              else
+                Location.find(details["hub"])
+              end
 
         if authorized
           return Dispatch.joins(:dispatch_plan_item)
-              .includes({dispatch_plan_item: :destination}, :transporter, :prepared_by, :unit)
-              .where(
-                dispatch_plan_item: {destination: hub, status: DispatchPlanItem::AUTHORIZED},
-                dispatch_status: status
-              )
+                         .includes({dispatch_plan_item: :destination}, :transporter, :prepared_by, :unit)
+                         .where(
+                           dispatch_plan_item: {destination: hub, status: DispatchPlanItem::AUTHORIZED},
+                           dispatch_status: status
+                         )
         end
         Dispatch.joins(:dispatch_plan_item)
-          .includes({dispatch_plan_item: :destination}, :transporter, :prepared_by, :unit)
-          .where(
-            dispatch_plan_item: {destination: hub},
-            dispatch_status: status
-          )
+                .includes({dispatch_plan_item: :destination}, :transporter, :prepared_by, :unit)
+                .where(
+                  dispatch_plan_item: {destination: hub},
+                  dispatch_status: status
+                )
       end
 
       def start(dispatch)
@@ -77,7 +75,7 @@ module Cats
           details = user.details
 
           if (details.key?("warehouse") && hub.child_ids.include?(details["warehouse"])) ||
-              (details.key?("hub") && details["hub"] == location_id)
+             (details.key?("hub") && details["hub"] == location_id)
             user
           end
         end.compact

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

@@ -34,7 +34,7 @@ module Cats
         roles = rule[:recipients]
 
         users = Cats::Core::User.joins(:application_module)
-          .where(application_module: {prefix: app_code}).with_all_roles(*roles)
+                                .where(application_module: {prefix: app_code}).with_all_roles(*roles)
         notifier.deliver(users)
       end
     end

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

@@ -18,7 +18,7 @@ module Cats
           round_rations = rations.select { |r| r.beneficiary_category_id == plan_item.beneficiary_category_id }
           round_rations.each do |ration|
             quantity = (no_of_days / ration.no_of_days * ration.quantity) * plan_item.beneficiaries *
-              round_plan.rounds.count
+                       round_plan.rounds.count
             quantity = quantity.truncate(2)
             unit = ration.unit
             if ration.unit.abbreviation.downcase != "pc"
@@ -120,18 +120,16 @@ module Cats
         end
 
         plans = Cats::Core::RoundPlan.includes(:beneficiary_round_plan_items)
-          .where(beneficiary_round_plan_items: {id: ids})
+                                     .where(beneficiary_round_plan_items: {id: ids})
         raise(StandardError, "Round plan items should be from the same plan.") if plans.count > 1
 
-        unless plans.count.positive? && plan.id == plans[0].id
-          raise(StandardError, "Round plan items are not from the given round plan.")
-        end
+        raise(StandardError, "Round plan items are not from the given round plan.") unless plans.count.positive? && plan.id == plans[0].id
 
         to_delete = Cats::Core::BeneficiaryRoundPlanItem.where(id: ids)
         keys = to_delete.map(&:round_plan_item_id).uniq
 
         item_counts = Cats::Core::BeneficiaryRoundPlanItem.where(round_plan_item_id: keys)
-          .group(:round_plan_item_id).count
+                                                          .group(:round_plan_item_id).count
 
         delete_hash = {}
         keys.each { |k| delete_hash[k] = [] }

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

@@ -12,12 +12,12 @@ module Cats
           warehouses = hub.children
           warehouse_ids = warehouses.map(&:id)
           stores = Cats::Core::Store.joins(:warehouse)
-            .where(
-              cats_core_locations: {
-                location_type: Cats::Core::Location::WAREHOUSE,
-                id: warehouse_ids
-              }
-            ).group_by(&:warehouse_id)
+                                    .where(
+                                      cats_core_locations: {
+                                        location_type: Cats::Core::Location::WAREHOUSE,
+                                        id: warehouse_ids
+                                      }
+                                    ).group_by(&:warehouse_id)
           wh_details = []
           stores.each do |key, value|
             total_space = value.inject(0) { |sum, val| sum + val.available_space }

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

@@ -8,17 +8,17 @@ module Cats
         raise(StandardError, "User does not have associated location.") unless key_available
 
         hub = if details["hub"]
-          Location.find(details["hub"])
-        elsif details["warehouse"]
-          Location.find(details["warehouse"]).parent
-        else
-          Store.find(details["stores"][0]).warehouse.parent
-        end
+                Location.find(details["hub"])
+              elsif details["warehouse"]
+                Location.find(details["warehouse"]).parent
+              else
+                Store.find(details["stores"][0]).warehouse.parent
+              end
 
         allocation_items =
           DispatchPlanItem
-            .joins(:dispatch_plan)
-            .where(destination: hub, dispatch_plan: {status: DispatchPlan::APPROVED})
+          .joins(:dispatch_plan)
+          .where(destination: hub, dispatch_plan: {status: DispatchPlan::APPROVED})
 
         allocation_items.map do |item|
           {

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

@@ -4,7 +4,7 @@ module Cats
       def change_password(email, old_password, new_password, password_confirmation)
         raise(StandardError, "Password and confirmation do not match.") unless new_password == password_confirmation
 
-        user = User.find_by(email:)
+        user = User.find_by(email: email)
         raise(StandardError, "User with email '#{email}' could not be found.") unless user
 
         raise(StandardError, "Old password for user '#{email}' is not correct.") unless user.authenticate(old_password)

data/db/migrate/20210715114910_create_cats_core_users.rb

@@ -9,9 +9,9 @@ class CreateCatsCoreUsers < ActiveRecord::Migration[6.1]
       t.jsonb :details, null: false, default: {}
       t.string :phone_number
       t.references :application_module,
-        null: false,
-        index: {name: "am_on_users_indx"},
-        foreign_key: {to_table: :cats_core_application_modules}
+                   null: false,
+                   index: {name: "am_on_users_indx"},
+                   foreign_key: {to_table: :cats_core_application_modules}
 
       t.timestamps
     end

data/db/migrate/20210715120018_create_cats_core_roles.rb

@@ -4,25 +4,25 @@ class CreateCatsCoreRoles < ActiveRecord::Migration[6.1]
       t.string :name
       t.references :resource, polymorphic: true
       t.references :application_module,
-        null: false,
-        index: {name: "am_on_roles_indx"},
-        foreign_key: {to_table: :cats_core_application_modules}
+                   null: false,
+                   index: {name: "am_on_roles_indx"},
+                   foreign_key: {to_table: :cats_core_application_modules}
 
       t.timestamps
     end
 
     create_table(:cats_core_users_cats_core_roles, id: false) do |t|
       t.references :user,
-        null: false,
-        index: {name: "user_on_ur_indx"},
-        foreign_key: {to_table: :cats_core_users}
+                   null: false,
+                   index: {name: "user_on_ur_indx"},
+                   foreign_key: {to_table: :cats_core_users}
       t.references :role,
-        null: false,
-        index: {name: "role_on_ur_indx"},
-        foreign_key: {to_table: :cats_core_roles}
+                   null: false,
+                   index: {name: "role_on_ur_indx"},
+                   foreign_key: {to_table: :cats_core_roles}
     end
 
-    add_index(:cats_core_roles, [:name, :resource_type, :resource_id])
-    add_index(:cats_core_users_cats_core_roles, [:user_id, :role_id], unique: true)
+    add_index(:cats_core_roles, %i[name resource_type resource_id])
+    add_index(:cats_core_users_cats_core_roles, %i[user_id role_id], unique: true)
   end
 end

data/db/migrate/20210715121244_create_cats_core_menus.rb

@@ -4,9 +4,9 @@ class CreateCatsCoreMenus < ActiveRecord::Migration[6.1]
       t.string :label, null: false
       t.string :icon
       t.references :application_module,
-        null: false,
-        index: {name: "am_on_menus_indx"},
-        foreign_key: {to_table: :cats_core_application_modules}
+                   null: false,
+                   index: {name: "am_on_menus_indx"},
+                   foreign_key: {to_table: :cats_core_application_modules}
 
       t.timestamps
     end

data/db/migrate/20210715122141_create_cats_core_menu_items.rb

@@ -5,9 +5,9 @@ class CreateCatsCoreMenuItems < ActiveRecord::Migration[6.1]
       t.string :icon
       t.string :route, null: false
       t.references :menu,
-        null: false,
-        index: {name: "menu_on_mi_indx"},
-        foreign_key: {to_table: :cats_core_menus}
+                   null: false,
+                   index: {name: "menu_on_mi_indx"},
+                   foreign_key: {to_table: :cats_core_menus}
 
       t.timestamps
     end

data/db/migrate/20210715122423_create_cats_core_role_menus.rb

@@ -2,13 +2,13 @@ class CreateCatsCoreRoleMenus < ActiveRecord::Migration[6.1]
   def change
     create_table :cats_core_role_menus do |t|
       t.references :role,
-        null: false,
-        index: {name: "role_on_rm_indx"},
-        foreign_key: {to_table: :cats_core_roles}
+                   null: false,
+                   index: {name: "role_on_rm_indx"},
+                   foreign_key: {to_table: :cats_core_roles}
       t.references :menu,
-        null: false,
-        index: {name: "menu_on_rm_indx"},
-        foreign_key: {to_table: :cats_core_menus}
+                   null: false,
+                   index: {name: "menu_on_rm_indx"},
+                   foreign_key: {to_table: :cats_core_menus}
       t.string :description
 
       t.timestamps
@@ -16,13 +16,13 @@ class CreateCatsCoreRoleMenus < ActiveRecord::Migration[6.1]
 
     create_table(:cats_core_role_menus_menu_items, id: false) do |t|
       t.references :role_menu,
-        null: false,
-        index: {name: "rm_on_ummi_indx"},
-        foreign_key: {to_table: :cats_core_role_menus}
+                   null: false,
+                   index: {name: "rm_on_ummi_indx"},
+                   foreign_key: {to_table: :cats_core_role_menus}
       t.references :menu_item,
-        null: false,
-        index: {name: "mi_on_ummi_indx"},
-        foreign_key: {to_table: :cats_core_menu_items}
+                   null: false,
+                   index: {name: "mi_on_ummi_indx"},
+                   foreign_key: {to_table: :cats_core_menu_items}
     end
   end
 end

data/db/migrate/20210717031810_create_cats_core_plans.rb

@@ -10,9 +10,9 @@ class CreateCatsCorePlans < ActiveRecord::Migration[6.1]
       t.integer :rounds, null: false
 
       t.references :program,
-        null: false,
-        index: {name: "program_on_plan_indx"},
-        foreign_key: {to_table: :cats_core_programs}
+                   null: false,
+                   index: {name: "program_on_plan_indx"},
+                   foreign_key: {to_table: :cats_core_programs}
 
       t.timestamps
     end

data/db/migrate/20210717032024_create_cats_core_plan_items.rb

@@ -2,33 +2,33 @@ class CreateCatsCorePlanItems < ActiveRecord::Migration[6.1]
   def change
     create_table :cats_core_plan_items do |t|
       t.references :plan,
-        null: false,
-        index: {name: "plan_on_plan_items_indx"},
-        foreign_key: {to_table: :cats_core_plans}
+                   null: false,
+                   index: {name: "plan_on_plan_items_indx"},
+                   foreign_key: {to_table: :cats_core_plans}
       t.references :region,
-        null: false,
-        index: {name: "region_on_plan_items_indx"},
-        foreign_key: {to_table: :cats_core_locations}
+                   null: false,
+                   index: {name: "region_on_plan_items_indx"},
+                   foreign_key: {to_table: :cats_core_locations}
       t.references :zone,
-        null: false,
-        index: {name: "zone_on_plan_items_indx"},
-        foreign_key: {to_table: :cats_core_locations}
+                   null: false,
+                   index: {name: "zone_on_plan_items_indx"},
+                   foreign_key: {to_table: :cats_core_locations}
       t.references :woreda,
-        null: false,
-        index: {name: "woreda_on_plan_items_idnx"},
-        foreign_key: {to_table: :cats_core_locations}
+                   null: false,
+                   index: {name: "woreda_on_plan_items_idnx"},
+                   foreign_key: {to_table: :cats_core_locations}
       t.references :fdp,
-        null: false,
-        index: {name: "fdp_on_plan_items_indx"},
-        foreign_key: {to_table: :cats_core_locations}
+                   null: false,
+                   index: {name: "fdp_on_plan_items_indx"},
+                   foreign_key: {to_table: :cats_core_locations}
       t.references :operator,
-        null: false,
-        index: {name: "operator_on_plan_items_indx"},
-        foreign_key: {to_table: :cats_core_operators}
+                   null: false,
+                   index: {name: "operator_on_plan_items_indx"},
+                   foreign_key: {to_table: :cats_core_operators}
 
       t.timestamps
     end
 
-    add_index :cats_core_plan_items, [:plan_id, :fdp_id], unique: true
+    add_index :cats_core_plan_items, %i[plan_id fdp_id], unique: true
   end
 end

data/db/migrate/20210717032260_create_cats_core_beneficiary_categories.rb

@@ -4,9 +4,9 @@ class CreateCatsCoreBeneficiaryCategories < ActiveRecord::Migration[7.0]
       t.string :code, unique: true
       t.string :name, null: false
       t.references :plan,
-        null: false,
-        index: {name: "plan_on_bc_indx"},
-        foreign_key: {to_table: :cats_core_plans}
+                   null: false,
+                   index: {name: "plan_on_bc_indx"},
+                   foreign_key: {to_table: :cats_core_plans}
 
       t.timestamps
     end

data/db/migrate/20210717032270_create_cats_core_rations.rb

@@ -3,17 +3,17 @@ class CreateCatsCoreRations < ActiveRecord::Migration[6.1]
     create_table :cats_core_rations do |t|
       t.string :reference_no, unique: true
       t.references :beneficiary_category,
-        null: false,
-        index: {name: "bc_on_ration_indx"},
-        foreign_key: {to_table: :cats_core_beneficiary_categories}
+                   null: false,
+                   index: {name: "bc_on_ration_indx"},
+                   foreign_key: {to_table: :cats_core_beneficiary_categories}
       t.references :commodity_category,
-        null: false,
-        index: {name: "cc_on_ration_indx"},
-        foreign_key: {to_table: :cats_core_commodity_categories}
+                   null: false,
+                   index: {name: "cc_on_ration_indx"},
+                   foreign_key: {to_table: :cats_core_commodity_categories}
       t.references :unit_of_measure,
-        null: false,
-        index: {name: "uom_on_ration_indx"},
-        foreign_key: {to_table: :cats_core_unit_of_measures}
+                   null: false,
+                   index: {name: "uom_on_ration_indx"},
+                   foreign_key: {to_table: :cats_core_unit_of_measures}
       t.float :quantity, null: false
       t.integer :no_of_days, null: false
 

data/db/migrate/20210717032290_create_cats_core_beneficiary_plan_items.rb

@@ -2,13 +2,13 @@ class CreateCatsCoreBeneficiaryPlanItems < ActiveRecord::Migration[7.0]
   def change
     create_table :cats_core_beneficiary_plan_items do |t|
       t.references :plan_item,
-        null: false,
-        index: {name: "pi_on_bpi_indx"},
-        foreign_key: {to_table: :cats_core_plan_items}
+                   null: false,
+                   index: {name: "pi_on_bpi_indx"},
+                   foreign_key: {to_table: :cats_core_plan_items}
       t.references :beneficiary_category,
-        null: false,
-        index: {name: "bc_on_bpi_indx"},
-        foreign_key: {to_table: :cats_core_beneficiary_categories}
+                   null: false,
+                   index: {name: "bc_on_bpi_indx"},
+                   foreign_key: {to_table: :cats_core_beneficiary_categories}
       t.integer :beneficiaries, null: false
 
       t.integer :rounds
@@ -18,7 +18,7 @@ class CreateCatsCoreBeneficiaryPlanItems < ActiveRecord::Migration[7.0]
     end
     add_index(
       :cats_core_beneficiary_plan_items,
-      [:plan_item_id, :beneficiary_category_id],
+      %i[plan_item_id beneficiary_category_id],
       unique: true,
       name: "pii_on_bci_uniq_indx"
     )

data/db/migrate/20210717032295_create_cats_core_plan_item_details.rb

@@ -2,17 +2,17 @@ class CreateCatsCorePlanItemDetails < ActiveRecord::Migration[6.1]
   def change
     create_table :cats_core_plan_item_details do |t|
       t.references :beneficiary_plan_item,
-        null: false,
-        index: {name: "bpi_on_pid_indx"},
-        foreign_key: {to_table: :cats_core_beneficiary_plan_items}
+                   null: false,
+                   index: {name: "bpi_on_pid_indx"},
+                   foreign_key: {to_table: :cats_core_beneficiary_plan_items}
       t.references :ration,
-        null: false,
-        index: {name: "ration_on_pid_indx"},
-        foreign_key: {to_table: :cats_core_rations}
+                   null: false,
+                   index: {name: "ration_on_pid_indx"},
+                   foreign_key: {to_table: :cats_core_rations}
       t.float :quantity, null: false
 
       t.timestamps
     end
-    add_index(:cats_core_plan_item_details, [:beneficiary_plan_item_id, :ration_id], unique: true, name: "bpii_ri_on_pid_indx")
+    add_index(:cats_core_plan_item_details, %i[beneficiary_plan_item_id ration_id], unique: true, name: "bpii_ri_on_pid_indx")
   end
 end

data/db/migrate/20210717032330_create_cats_core_commodity_donations.rb

@@ -6,22 +6,22 @@ class CreateCatsCoreCommodityDonations < ActiveRecord::Migration[6.1]
       t.string :shipping_reference
       t.date :donated_on, null: false
       t.references :donor,
-        null: false,
-        index: {name: "donor_on_cd_indx"},
-        foreign_key: {to_table: :cats_core_donors}
+                   null: false,
+                   index: {name: "donor_on_cd_indx"},
+                   foreign_key: {to_table: :cats_core_donors}
       t.references :commodity_category,
-        null: false,
-        index: {name: "cc_on_cd_indx"},
-        foreign_key: {to_table: :cats_core_commodity_categories}
+                   null: false,
+                   index: {name: "cc_on_cd_indx"},
+                   foreign_key: {to_table: :cats_core_commodity_categories}
       t.references :plan,
-        null: true,
-        index: {name: "plan_on_cd_indx"},
-        foreign_key: {to_table: :cats_core_plans}
+                   null: true,
+                   index: {name: "plan_on_cd_indx"},
+                   foreign_key: {to_table: :cats_core_plans}
       t.float :quantity, null: false
       t.references :unit,
-        null: false,
-        index: {name: "unit_on_cd_indx"},
-        foreign_key: {to_table: :cats_core_unit_of_measures}
+                   null: false,
+                   index: {name: "unit_on_cd_indx"},
+                   foreign_key: {to_table: :cats_core_unit_of_measures}
 
       t.timestamps
     end

data/db/migrate/20210717032408_create_cats_core_cash_donations.rb

@@ -5,14 +5,14 @@ class CreateCatsCoreCashDonations < ActiveRecord::Migration[7.0]
       t.string :description
       t.date :donated_on, null: false
       t.references :donor,
-        null: false,
-        index: {name: "donor_on_cad_indx"},
-        foreign_key: {to_table: :cats_core_donors}
+                   null: false,
+                   index: {name: "donor_on_cad_indx"},
+                   foreign_key: {to_table: :cats_core_donors}
       t.float :amount, null: false
       t.references :currency,
-        null: false,
-        index: {name: "currency_on_cad_indx"},
-        foreign_key: {to_table: :cats_core_currencies}
+                   null: false,
+                   index: {name: "currency_on_cad_indx"},
+                   foreign_key: {to_table: :cats_core_currencies}
 
       t.timestamps
     end

data/db/migrate/20210717032602_create_cats_core_gift_certificates.rb

@@ -11,23 +11,23 @@ class CreateCatsCoreGiftCertificates < ActiveRecord::Migration[6.1]
       t.string :bill_of_lading_no
       t.float :quantity, null: false
       t.references :commodity_category,
-        null: false,
-        index: {name: "gc_on_cc_indx"},
-        foreign_key: {to_table: :cats_core_commodity_categories}
+                   null: false,
+                   index: {name: "gc_on_cc_indx"},
+                   foreign_key: {to_table: :cats_core_commodity_categories}
       t.references :unit,
-        null: false,
-        index: {name: "unit_on_gc_indx"},
-        foreign_key: {to_table: :cats_core_unit_of_measures}
+                   null: false,
+                   index: {name: "unit_on_gc_indx"},
+                   foreign_key: {to_table: :cats_core_unit_of_measures}
       t.references :destination_warehouse,
-        null: false,
-        index: {name: "dw_on_gc_indx"},
-        foreign_key: {to_table: :cats_core_locations}
+                   null: false,
+                   index: {name: "dw_on_gc_indx"},
+                   foreign_key: {to_table: :cats_core_locations}
       t.float :estimated_price
       t.float :estimated_tax
       t.references :currency,
-        null: false,
-        index: {name: "currency_on_gc_indx"},
-        foreign_key: {to_table: :cats_core_currencies}
+                   null: false,
+                   index: {name: "currency_on_gc_indx"},
+                   foreign_key: {to_table: :cats_core_currencies}
       t.string :registration_no
       t.string :requested_by, null: false
       t.string :request_reference