comee_core 0.1.40 → 0.1.41

data/app/services/comee/core/quotation_request_service.rb

@@ -0,0 +1,69 @@
+module Comee
+  module Core
+    class QuotationRequestService
+      def create_request_with_items(params)
+        request = nil
+        QuotationRequest.transaction do
+          request = QuotationRequest.create!(client_id: params[:client_id], status: QuotationRequest.statuses[:draft])
+          items = params[:items]
+          items.each do |item|
+            item[:price] = 0
+            item[:quotation_request_id] = request.id
+            item[:discount] = 0
+          end
+          QuotationRequestItem.insert_all!(items)
+        end
+        request
+      rescue StandardError
+        raise
+      end
+
+      def submit(id)
+        rfq = QuotationRequest.find(id)
+
+        raise(StandardError, "RFQ should be in draft state.") if QuotationRequest.statuses[rfq.status] != QuotationRequest.statuses[:draft]
+
+        raise(StandardError, "RFQ does not have any items.") if rfq.quotation_request_items.count.zero?
+
+        raise(StandardError, "RFQ contains only canceled items.") if rfq.quotation_request_items.all?(&:canceled?)
+
+        QuotationRequest.transaction do
+          product_ids = rfq.quotation_request_items.map(&:product_id)
+          prices = MasterPrice.where(product_id: product_ids).each_with_object({}) do |price, res|
+            res[price.product_id] = [price.selling_price, price.valid_from, price.valid_to]
+          end
+          items = rfq.quotation_request_items.each_with_object({}) do |item, res|
+            price = prices[item.product_id]
+            res[item.id] = {price: price[0], valid_from: price[1], valid_to: price[2]} if price
+          end
+          QuotationRequestItem.update!(items.keys, items.values)
+          rfq.status = QuotationRequest.statuses[:submitted]
+          rfq.save!
+        end
+        rfq
+      end
+
+      def submit_for_confirmation(id)
+        rfq = QuotationRequest.find(id)
+        if QuotationRequest.statuses[rfq.status] != QuotationRequest.statuses[:submitted]
+          raise(StandardError, "RFQ should be in submitted state.")
+        end
+
+        rfq.status = QuotationRequest.statuses[:awaiting_confirmation]
+        rfq.save!
+        rfq
+      end
+
+      def confirm(id)
+        rfq = QuotationRequest.find(id)
+        if QuotationRequest.statuses[rfq.status] != QuotationRequest.statuses[:awaiting_confirmation]
+          raise(StandardError, "RFQ should be in awaiting_confirmation state.")
+        end
+
+        rfq.status = QuotationRequest.statuses[:confirmed]
+        rfq.save!
+        rfq
+      end
+    end
+  end
+end

data/app/services/comee/core/sales_order_service.rb

@@ -0,0 +1,141 @@
+module Comee
+  module Core
+    class SalesOrderService
+      ##
+      # This method returns { success: true} if all back orders in the +params+ created successfully.
+      # The +params+ contain array of back orders with the corresponding products.
+      # The exact structure of +params+ looks like the following
+      # [{supplier_id: '', order_number: '', order_date: '', delivery_date: '', delivery_address: '', invoice_address: '',
+      #   back_order_items: [{product_id: '', requested_quantity: ''},...] }...]
+      def create_back_orders(params)
+        check_product_prices(params)
+        check_duplicate_order_number(params)
+        bulk_create_back_order(params)
+      end
+
+      ##
+      # This method returns true if all products under each back order in +params+ have active price in the master price table.
+      # The +params+ contain array of back orders with the corresponding products.
+      # The exact structure of +params+ looks like the following
+      # [{supplier_id: '', order_number: '', order_date: '', delivery_date: '', delivery_address: '', invoice_address: '',
+      #   back_order_items: [{product_id: '', requested_quantity: ''},...] }...]
+      def check_product_prices(params)
+        queries = []
+        params.each do |param|
+          param[:back_order_items].map do |bo_item|
+            queries << {supplier_id: param[:supplier_id], product_id: bo_item[:product_id]}
+          end
+        end
+        master_prices = queries.inject(Comee::Core::MasterPrice.none) do |conditions, condition|
+          conditions.or(Comee::Core::MasterPrice.where(condition))
+        end
+
+        raise(StandardError, compose_error_messge(queries, master_prices)) unless queries.count == master_prices.count
+
+        true
+      end
+
+      ##
+      # This method composes an error message when called from check_product_prices.
+      # It recieves +queries+ and +master_prices+ as a parameter
+      #   The +queries+ is an array containing list of conditions that will be joined using or connector
+      #     The +queries+ param looks like [{supplier_id: '', product_id: '', active: true},...]
+      #   The +master_prices+ param contains master prices fetched by the queries param given above
+      # The message is composed by computing difference b/n the queries and master prices params
+      def compose_error_messge(queries, master_prices)
+        result = master_prices.map { |mp| {supplier_id: mp[:supplier_id], product_id: mp[:product_id]} }
+        diff = queries - result
+        product_names = Comee::Core::Product.where(id: diff.map { |dif| dif[:product_id] }).pluck("name").join(", ")
+        supplier_names = Comee::Core::Supplier.where(id: diff.map { |dif| dif[:supplier_id] }).pluck("name").join(", ")
+        "The following products #{product_names} do not have price entry for the following suppliers" \
+        " #{supplier_names} in the master price table or the prices are not valid."
+      end
+
+      ##
+      # This method returns false if there is no duplicate order number, otherwise it raises an error.
+      # The +params+ contain array of back orders with the corresponding products.
+      # The exact structure of +params+ looks like the following
+      # [{supplier_id: '', order_number: '', order_date: '', delivery_date: '', delivery_address: '', invoice_address: '',
+      #   back_order_items: [{product_id: '', requested_quantity: ''},...] }...]
+      # The method first checks if there is duplicate order number within in the +params+ array, and if not then it checks
+      # for duplicate order number from back order table.
+      def check_duplicate_order_number(params)
+        order_numbers = params.map { |param| param[:order_number] }
+        unless order_numbers.uniq.count == order_numbers.count
+          raise(StandardError, "Duplicate order number(s) #{order_numbers.select do |e|
+                                                              order_numbers.count(e) > 1
+                                                            end.uniq.join(', ')} exist in the payload.")
+        end
+
+        duplicate_order_numbers = Comee::Core::BackOrder.where(order_number: order_numbers)
+        unless duplicate_order_numbers.count.zero?
+          raise(StandardError,
+                "Duplicate order number(s) #{duplicate_order_numbers.map(&:order_number).join(', ')} exist in the back order table.")
+        end
+
+        false
+      end
+
+      ##
+      # This method returns items aggregated by product for a supplier.
+      # It recieves +items+, +supplier_id+ and +back_order_id+ as a parameter
+      #   The +items+ is an array containing list of back order items
+      #     The +items+ param looks like [{product_id: '', requested_quantity: ''},...]
+      #   The +supplier_id+ is the id of the supplier under which the items belong to
+      #   The +back_order_id+ is the id of the backer order under which the items belong to
+      def aggregate_back_order_item(items, supplier_id, back_order_id, delivery_date)
+        items.group_by { |item| item[:product_id] }.map do |product_id, aggregated_items|
+          total_quantity = aggregated_items.reduce(0) { |sum, agg_item| agg_item[:requested_quantity] + sum }
+          {product_id: product_id, requested_quantity: total_quantity, supplier_quantity: total_quantity, supplier_id: supplier_id,
+           back_order_id: back_order_id, delivery_date: delivery_date}
+        end
+      end
+
+      ##
+      # This method assigns current purchase price for a product under a given supplier.
+      # It recieves +items+ containing array of products along with the corresponding suppliers
+      # The +items+ param looks like [
+      #   {product_id: '', requested_quantity: '', supplier_quantity: '', supplier_id: '', back_order_id: '' },
+      #   ...
+      # ]
+      # It assigns price by first fetching the price of products in the items array and then iterate through the
+      # items array and find the corresponding price from the fetched price
+      def assign_price_to_items(items)
+        queries = []
+        items.map { |bo_item| queries << {supplier_id: bo_item[:supplier_id], product_id: bo_item[:product_id]} }
+        master_prices = queries.inject(Comee::Core::MasterPrice.none) do |conditions, condition|
+          conditions.or(Comee::Core::MasterPrice.where(condition))
+        end
+        items.each do |item|
+          mp = master_prices.find { |price| price.supplier_id == item[:supplier_id] && price.product_id == item[:product_id] }
+          item[:requested_unit_price] = mp.purchase_price
+          item[:supplier_unit_price] = mp.purchase_price
+          item[:from] = mp.valid_from
+          item[:to] = mp.valid_from
+          item.delete(:supplier_id)
+        end
+      end
+
+      ##
+      # This method returns { success: true} if all back orders in the +params+ created successfully.
+      # The +params+ contain array of back orders with the corresponding products.
+      # The exact structure of +params+ looks like the following
+      # [{supplier_id: '', order_number: '', order_date: '', delivery_date: '', delivery_address: '', invoice_address: '',
+      #   back_order_items: [{product_id: '', requested_quantity: ''},...] }...]
+      # This method uses insert_all method to create back order and its items in bulk
+      def bulk_create_back_order(params)
+        back_orders = params.map { |p| {**p.except(:back_order_items), created_at: Time.now, updated_at: Time.now} }
+        result = Comee::Core::BackOrder.insert_all!(back_orders, returning: %w[id supplier_id delivery_date])
+        aggregated_items = []
+        result.pluck("id", "supplier_id", "delivery_date").each do |r|
+          supplier_detail = params.find { |p| p[:supplier_id].to_i == r[1] }
+          aggregated_items << aggregate_back_order_item(supplier_detail[:back_order_items], r[1], r[0], r[2])
+        end
+        Comee::Core::BackOrderItem.insert_all!(assign_price_to_items(aggregated_items.flatten))
+        {success: true}
+      rescue StandardError => e
+        {success: false, error: e.message}
+      end
+    end
+  end
+end

data/config/routes.rb

@@ -10,15 +10,56 @@ Comee::Core::Engine.routes.draw do
   end
   resources :suppliers
   resources :products
-  resources :product_types do
+  resources :quotation_requests do
+    collection do
+      post "filter"
+      post "create_with_items", controller: :quotation_requests, action: :create_request_with_items
+    end
+    member do
+      post "submit"
+      post "submit_for_confirmation"
+      post "confirm"
+    end
+  end
+  resources :quotation_request_items do
+    collection do
+      post "filter"
+      post "cancel"
+    end
+  end
+  resources :external_rfqs
+  resources :customer_orders do
+    member do
+      post "submit"
+      post "submit_for_confirmation"
+      post "accept"
+      post "cancel"
+    end
+  end
+  resources :customer_order_items do
+    collection do
+      post "filter"
+      post "cancel"
+    end
+  end
+  resources :customer_orders, only: [] do
     member do
-      get "products", controller: :product_types, action: :products
+      get "order", controller: :customer_orders, action: :order
+      patch "cancel", controller: :customer_orders, action: :cancel
+      patch "seek_confirmation", controller: :customer_orders, action: :seek_confirmation
+      patch "submit", controller: :customer_orders, action: :submit
+      patch "accept", controller: :customer_orders, action: :accept
+      patch "bulk_cancel_items", controller: :customer_orders, action: :bulk_cancel_items
     end
   end
   resources :currencies
   resources :units
   resources :users
-  resources :clients
+  resources :clients do
+    member do
+      get "rfq_products", controller: :quotation_requests, action: :rfq_products_for_client
+    end
+  end
   post "/client_prices/filter", controller: :client_prices, action: :filter
   resources :client_prices
   post "/back_orders/filter", controller: :back_orders, action: :filter
@@ -27,4 +68,17 @@ Comee::Core::Engine.routes.draw do
       post "submit"
     end
   end
+  resources :sales_order_items, except: [:index]
+  post "/sales_orders/filter", controller: :sales_orders, action: :filter
+  resources :sales_orders do
+    collection do
+      post "create_back_order", action: :create_back_order
+    end
+    member do
+      get "items", controller: :sales_order_items, action: :index
+      get "/sales_order/filter/", controller: :sales_orders, action: :filter
+      post "confirm"
+      post "cancel"
+    end
+  end
 end

data/db/migrate/20230728014322_create_comee_core_products.rb

@@ -4,20 +4,12 @@ class CreateComeeCoreProducts < ActiveRecord::Migration[7.0]
       t.string :code, null: false
       t.string :name, null: false
       t.string :description
-      t.jsonb :metadata
-      t.references :product_type,
-                   null: false,
-                   index: {name: "ccpt_on_ccp_indx"},
-                   foreign_key: {to_table: :comee_core_product_types}
-      t.references :unit,
-                   null: false,
-                   index: {name: "unit_on_ccp_indx"},
-                   foreign_key: {to_table: :comee_core_units}
-      t.string "ancestry", collation: "C"
-      t.index "ancestry"
+      t.boolean :leaf, null: false, default: true
+      t.string :ancestry
 
       t.timestamps
     end
     add_index :comee_core_products, :code, unique: true
+    add_index :comee_core_products, :ancestry
   end
 end

data/db/migrate/20230811102708_create_comee_core_customer_order_items.rb

@@ -16,6 +16,7 @@ class CreateComeeCoreCustomerOrderItems < ActiveRecord::Migration[7.1]
       t.float :quantity, null: false, default: 0
       t.float :price, null: false, default: 0
       t.date :delivery_date, null: false
+      t.boolean :canceled, default: false
 
       t.timestamps
     end

data/db/migrate/20230812190652_create_comee_core_sales_orders.rb

@@ -8,10 +8,10 @@ class CreateComeeCoreSalesOrders < ActiveRecord::Migration[7.0]
                    foreign_key: {to_table: :comee_core_customer_orders}
       t.float :payment_penalty, null: false, default: 0.0
       t.integer :status, null: false, default: 0
-      t.float :total_price, null: true, default: 0
-      t.float :amount_paid, null: true, default: 0
-      t.string :pallete_note, null: false
-      t.date :handover_date, null: false
+      t.float :total_price, null: false, default: 0
+      t.float :amount_paid, null: false, default: 0
+      t.string :pallete_note
+      t.date :handover_date
       t.string :remark
 
       t.timestamps

data/db/migrate/20230812212844_create_comee_core_sales_order_items.rb

@@ -17,7 +17,8 @@ class CreateComeeCoreSalesOrderItems < ActiveRecord::Migration[7.0]
       t.float :quantity_delivered, null: false, default: 0
       t.float :price, null: false
       t.date :delivery_date, null: false
-      t.integer :eb_number
+      t.date :handover_date
+      t.string :eb_number
 
       t.timestamps
     end

data/db/migrate/20231121094339_create_comee_core_quotation_requests.rb

@@ -0,0 +1,15 @@
+class CreateComeeCoreQuotationRequests < ActiveRecord::Migration[7.1]
+  def change
+    create_table :comee_core_quotation_requests do |t|
+      t.string :reference_no, null: false
+      t.references :client,
+                   null: false,
+                   index: {name: "client_on_ccrfq_indx"},
+                   foreign_key: {to_table: :comee_core_clients}
+      t.integer :status, null: false, default: 0
+
+      t.timestamps
+    end
+    add_index :comee_core_quotation_requests, :reference_no, unique: true
+  end
+end

data/db/migrate/20231121132709_create_comee_core_quotation_request_items.rb

@@ -0,0 +1,27 @@
+class CreateComeeCoreQuotationRequestItems < ActiveRecord::Migration[7.1]
+  def change
+    create_table :comee_core_quotation_request_items do |t|
+      t.references :product,
+                   null: false,
+                   index: {name: "product_on_ccrfqi_indx"},
+                   foreign_key: {to_table: :comee_core_products}
+      t.references :quotation_request,
+                   null: false,
+                   index: {name: "rfq_on_ccrfqi_indx"},
+                   foreign_key: {to_table: :comee_core_quotation_requests}
+      t.float :quantity, null: false
+      t.float :price, null: false, default: 0
+      t.float :discount, null: false, default: 0
+      t.date :expected_delivery_date, null: false
+      t.boolean :canceled, null: false, default: false
+      t.references :unit,
+                   null: false,
+                   index: {name: "unit_on_ccrfqi_indx"},
+                   foreign_key: {to_table: :comee_core_units}
+      t.date :valid_from, null: true
+      t.date :valid_to, null: true
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20231122094424_create_comee_core_external_rfqs.rb

@@ -0,0 +1,10 @@
+class CreateComeeCoreExternalRfqs < ActiveRecord::Migration[7.1]
+  def change
+    create_table :comee_core_external_rfqs do |t|
+      t.integer :status, null: false, default: 0
+      t.jsonb :data, null: false
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20231125122723_create_comee_core_purchase_orders.rb

@@ -0,0 +1,15 @@
+class CreateComeeCorePurchaseOrders < ActiveRecord::Migration[7.1]
+  def change
+    create_table :comee_core_purchase_orders do |t|
+      t.references :supplier,
+                   null: false,
+                   index: {name: "suppliers_on_ccpo_indx"},
+                   foreign_key: {to_table: :comee_core_suppliers}
+      t.date :order_date, default: Date.today
+      t.integer :status, default: 0
+      t.string :delivery_address, null: false
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20231126082907_create_comee_core_purchase_order_items.rb

@@ -0,0 +1,20 @@
+class CreateComeeCorePurchaseOrderItems < ActiveRecord::Migration[7.1]
+  def change
+    create_table :comee_core_purchase_order_items do |t|
+      t.references :purchase_order,
+                   null: false,
+                   index: {name: "pos_on_ccpoitm_indx"},
+                   foreign_key: {to_table: :comee_core_purchase_orders}
+
+      t.references :sales_order_item,
+                   null: false,
+                   index: {name: "soi_on_ccpoitm_indx"},
+                   foreign_key: {to_table: :comee_core_sales_order_items}
+      t.float :quantity, null: false
+      t.date :delivery_date, null: false
+      t.integer :status, null: false, default: 0
+
+      t.timestamps
+    end
+  end
+end

data/lib/comee/core/version.rb

@@ -1,5 +1,5 @@
 module Comee
   module Core
-    VERSION = "0.1.40".freeze
+    VERSION = "0.1.41".freeze
   end
 end

data/lib/comee_core.rb

@@ -1,6 +1,7 @@
 require "comee/core/version"
 require "comee/core/engine"
 require "active_model_serializers"
+require "ancestry"
 require "jwt"
 require "ransack"
 require "noticed"

data/spec/factories/comee/core/customer_order_items.rb

@@ -6,6 +6,7 @@ FactoryBot.define do
     quantity { 1 }
     price { 1.5 }
     delivery_date { Date.current }
+    canceled { false }
 
     trait :with_inventory do
       after(:create) do |item|

data/spec/factories/comee/core/external_rfqs.rb

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :external_rfq, class: "Comee::Core::ExternalRfq" do
+    data { {payload: ""} }
+    status { 1 }
+  end
+end

data/spec/factories/comee/core/products.rb

@@ -1,11 +1,12 @@
 FactoryBot.define do
   factory :product, class: "Comee::Core::Product" do
+    transient do
+      root { Comee::Core::Product.where(code: "ROOT", name: "Products", leaf: false).first_or_create }
+    end
     sequence(:code) { |n| "#{Faker::Alphanumeric.unique.alpha(number: 6)}-#{n}" }
     name { Faker::Name.name }
     description { Faker::Lorem.sentence }
-    metadata { {} }
-    product_type
-    unit
-    ancestry { nil }
+    leaf { true }
+    ancestry { root.id.to_s }
   end
 end

data/spec/factories/comee/core/purchase_order_items.rb

@@ -0,0 +1,9 @@
+FactoryBot.define do
+  factory :purchase_order_item, class: "Comee::Core::PurchaseOrderItem" do
+    purchase_order
+    sales_order_item
+    quantity { 10 }
+    delivery_date { Date.current.advance(days: 7) }
+    status { 0 }
+  end
+end

data/spec/factories/comee/core/purchase_orders.rb

@@ -0,0 +1,8 @@
+FactoryBot.define do
+  factory :purchase_order, class: "Comee::Core::PurchaseOrder" do
+    supplier
+    order_date { Date.current }
+    status { 0 }
+    delivery_address { "Korsu warehouse address" }
+  end
+end

data/spec/factories/comee/core/quotation_request_items.rb

@@ -0,0 +1,14 @@
+FactoryBot.define do
+  factory :quotation_request_item, class: "Comee::Core::QuotationRequestItem" do
+    product
+    quotation_request
+    unit
+    quantity { 10 }
+    price { 0 }
+    discount { 0 }
+    expected_delivery_date { Date.current.advance(days: 15) }
+    canceled { false }
+    valid_from { nil }
+    valid_to { nil }
+  end
+end

data/spec/factories/comee/core/quotation_requests.rb

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :quotation_request, class: "Comee::Core::QuotationRequest" do
+    reference_no { "RFQ-#{client.code}-#{DateTime.now.to_i}" }
+    client
+    status { 0 }
+  end
+end

data/spec/factories/comee/core/sales_order_items.rb

@@ -7,6 +7,7 @@ FactoryBot.define do
     quantity_delivered { 0 }
     price { 50 }
     delivery_date { Date.current.advance(months: 1) }
-    eb_number { 100 }
+    handover_date { Date.current }
+    eb_number { Faker::Lorem.word }
   end
 end

data/spec/factories/comee/core/sales_orders.rb

@@ -7,7 +7,6 @@ FactoryBot.define do
     amount_paid { 0 }
     status { Comee::Core::SalesOrder.statuses[:draft] }
     pallete_note { Faker::Lorem.sentence }
-    handover_date { Date.current }
     remark { Faker::Lorem.sentence }
   end
 end