spree_cm_commissioner 2.0.1.pre.pre → 2.0.1.pre.pre2

data/app/models/spree_cm_commissioner/notification_taxon.rb

@@ -1,5 +1,5 @@
 module SpreeCmCommissioner
-  class NotificationTaxon < Base
+  class NotificationTaxon < SpreeCmCommissioner::Base
     belongs_to :customer_notification, class_name: 'SpreeCmCommissioner::CustomerNotification'
     belongs_to :taxon, class_name: 'Spree::Taxon'
   end

data/app/models/spree_cm_commissioner/option_type_decorator.rb

@@ -6,7 +6,7 @@ module SpreeCmCommissioner
       base.include SpreeCmCommissioner::ParameterizeName
       base.include SpreeCmCommissioner::OptionTypeAttrType
 
-      base.enum kind: %i[variant product vendor vehicle]
+      base.enum kind: %i[variant product vendor vehicle_type transit]
 
       base.validates :name, presence: true
 
@@ -19,6 +19,16 @@ module SpreeCmCommissioner
       base.scope :promoted, -> { where(promoted: true) }
       base.whitelisted_ransackable_attributes = %w[kind]
 
+      def base.amenities
+        Spree::OptionType.where(kind: 'vehicle_type', name: 'amenities', presentation: 'Amenities', attr_type: 'amenity').first_or_create
+      end
+
+      def base.vehicle
+        Spree::OptionType.where(presentation: 'vehicle', attr_type: 'vehicle_id', kind: 'variant',
+                                name: 'vehicle'
+        ).first_or_create
+      end
+
       def base.rules_option_type
         Spree::OptionType.find_by(name: RULES_OPTION_TYPE_NAME)
       end

data/app/models/spree_cm_commissioner/option_value_vehicle_type.rb

@@ -0,0 +1,8 @@
+require_dependency 'spree_cm_commissioner'
+
+module SpreeCmCommissioner
+  class OptionValueVehicleType < ApplicationRecord
+    belongs_to :vehicle_type, class_name: 'SpreeCmCommissioner::VehicleType', dependent: :destroy
+    belongs_to :option_value, class_name: 'Spree::OptionValue', dependent: :destroy
+  end
+end

data/app/models/spree_cm_commissioner/order_decorator.rb

@@ -2,7 +2,6 @@ module SpreeCmCommissioner
   module OrderDecorator
     def self.prepended(base) # rubocop:disable Metrics/MethodLength,Metrics/AbcSize
       base.include SpreeCmCommissioner::PhoneNumberSanitizer
-      base.include SpreeCmCommissioner::OrderSeatable
       base.include SpreeCmCommissioner::OrderStateMachine
 
       base.scope :subscription, -> { where.not(subscription_id: nil) }
@@ -43,8 +42,6 @@ module SpreeCmCommissioner
       base.has_many :vendors, through: :products, class_name: 'Spree::Vendor'
       base.has_many :taxons, through: :products, class_name: 'Spree::Taxon'
       base.has_many :guests, through: :line_items, class_name: 'SpreeCmCommissioner::Guest'
-      base.has_many :blocks, through: :guests, class_name: 'SpreeCmCommissioner::Block', source: :block
-      base.has_many :reserved_blocks, through: :guests, class_name: 'SpreeCmCommissioner::ReservedBlock'
       base.has_many :guest_card_classes, class_name: 'SpreeCmCommissioner::GuestCardClass', through: :variants
 
       base.delegate :customer, to: :user, allow_nil: true
@@ -61,13 +58,6 @@ module SpreeCmCommissioner
       end
     end
 
-    # override
-    # spree use this method to check stock availability & consider whether :order can continue to next state.
-    def insufficient_stock_lines
-      checker = SpreeCmCommissioner::Stock::OrderAvailabilityChecker.new(self)
-      checker.insufficient_stock_lines
-    end
-
     def ticket_seller_user?
       return false if user.nil?
 
@@ -116,14 +106,7 @@ module SpreeCmCommissioner
     end
 
     def mark_as_archive
-      blocks_canceled = begin
-        cancel_blocks!
-        true
-      rescue StandardError
-        false
-      end
-
-      update(archived_at: Time.current) if blocks_canceled
+      update(archived_at: Time.current)
     end
 
     # overrided
@@ -216,18 +199,6 @@ module SpreeCmCommissioner
 
     private
 
-    def unstock_inventory_in_redis!
-      CmAppLogger.log(label: "#{self.class.name}#unstock_inventory_in_redis!", data: { order_id: id }) do
-        SpreeCmCommissioner::RedisStock::InventoryUpdater.new(line_item_ids).unstock!
-      end
-    end
-
-    def restock_inventory_in_redis!
-      CmAppLogger.log(label: "#{self.class.name}#restock_inventory_in_redis!", data: { order_id: id }) do
-        SpreeCmCommissioner::RedisStock::InventoryUpdater.new(line_item_ids).restock!
-      end
-    end
-
     # override :spree_api
     def webhook_payload_body
       resource_serializer.new(

data/app/models/spree_cm_commissioner/place.rb

@@ -1,5 +1,7 @@
+require_dependency 'spree_cm_commissioner'
+
 module SpreeCmCommissioner
-  class Place < Base
+  class Place < ApplicationRecord
     acts_as_nested_set
 
     validates :reference, presence: true, if: :validate_reference?
@@ -15,6 +17,7 @@ module SpreeCmCommissioner
     has_many :products, through: :product_places
 
     has_many :children, -> { order(:lft) }, class_name: 'SpreeCmCommissioner::Place', foreign_key: :parent_id, dependent: :destroy
+    has_many :vendor_stops, class_name: 'SpreeCmCommissioner::VendorStop', dependent: :destroy, foreign_key: :stop_id
 
     has_many :taxon_places, class_name: 'SpreeCmCommissioner::TaxonPlace', dependent: :destroy
     has_many :taxons, through: :taxon_places

data/app/models/spree_cm_commissioner/product_decorator.rb

@@ -6,6 +6,7 @@ module SpreeCmCommissioner
       base.include SpreeCmCommissioner::KycBitwise
       base.include SpreeCmCommissioner::Metafield
       base.include SpreeCmCommissioner::TenantUpdatable
+      base.include SpreeCmCommissioner::RouteType
 
       base.has_many :variant_kind_option_types, -> { where(kind: :variant).order(:position) },
                     through: :product_option_types, source: :option_type
@@ -32,7 +33,6 @@ module SpreeCmCommissioner
       base.has_one :google_wallet, class_name: 'SpreeCmCommissioner::GoogleWallet', dependent: :destroy
 
       base.has_many :complete_line_items, through: :classifications, source: :line_items
-      base.has_many :inventory_items, through: :variants
       base.has_many :guests, through: :line_items
 
       base.has_many :product_places, class_name: 'SpreeCmCommissioner::ProductPlace', dependent: :destroy
@@ -64,17 +64,20 @@ module SpreeCmCommissioner
       base.before_validation :set_event_id
 
       base.validate :validate_event_taxons, if: -> { taxons.event.present? }
+
       base.validate :validate_product_date, if: -> { available_on.present? && discontinue_on.present? }
-      base.validate :product_type_unchanged, on: :update
+
       base.validates :commission_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 100 }, allow_nil: true
 
-      base.whitelisted_ransackable_attributes = %w[description name slug discontinue_on status vendor_id short_name]
+      base.whitelisted_ransackable_attributes = %w[description name slug discontinue_on status vendor_id short_name route_type]
 
       base.after_update :update_variants_vendor_id, if: :saved_change_to_vendor_id?
       base.after_update :sync_event_id_to_children, if: :saved_change_to_event_id?
 
       base.enum purchasable_on: { both: 0, web: 1, app: 2 }
 
+      base.accepts_nested_attributes_for :trip, allow_destroy: true
+
       base.multi_tenant :tenant, class_name: 'SpreeCmCommissioner::Tenant'
       base.before_save :set_tenant
     end
@@ -111,13 +114,6 @@ module SpreeCmCommissioner
 
       errors.add(:discontinue_on, 'must be after the available on date')
     end
-
-    def product_type_unchanged
-      return if product_type_was.nil?
-      return unless product_type_changed?
-
-      errors.add(:product_type, 'cannot be changed once set')
-    end
   end
 end
 

data/app/models/spree_cm_commissioner/service_calendar.rb

@@ -1,3 +1,5 @@
+require_dependency 'spree_cm_commissioner'
+
 module SpreeCmCommissioner
   class ServiceCalendar < SpreeCmCommissioner::Base
     # exception_rules = [

data/app/models/spree_cm_commissioner/state_decorator.rb

@@ -3,6 +3,7 @@ module SpreeCmCommissioner
     def self.prepended(base)
       base.whitelisted_ransackable_attributes |= %w[name abbr country_id]
       base.has_many :vendors, foreign_key: 'default_state_id', class_name: 'Spree::Vendor', inverse_of: :default_state, dependent: :nullify
+      base.has_many :stops, class_name: 'SpreeCmCommissioner::Place'
 
       def update_total_inventory
         update(total_inventory: vendors.pluck(:total_inventory).compact.sum)

data/app/models/spree_cm_commissioner/stock/availability_checker.rb

@@ -1,48 +1,47 @@
 module SpreeCmCommissioner
   module Stock
     class AvailabilityChecker
-      attr_reader :variant, :options, :error_message
+      attr_reader :variant, :error_message
 
-      def initialize(variant, options = {})
+      def initialize(variant)
         @variant = variant
-        @options = options
         @error_message = nil
       end
 
-      def can_supply?(quantity = 1)
+      def can_supply?(quantity = 1, options = {})
         return false unless variant.available?
         return true unless variant.should_track_inventory?
         return true if variant.backorderable?
         return true if variant.need_confirmation?
 
-        variant_available?(quantity)
-      end
-
-      def variant_available?(quantity = 1)
-        return false if cached_inventory_items.empty?
-
-        cached_inventory_items.all? do |cached_inventory_item|
-          cached_inventory_item.active? && cached_inventory_item.quantity_available >= quantity
-        end
-      end
-
-      def cached_inventory_items
-        return @cached_inventory_items if defined?(@cached_inventory_items)
+        # when delivery required, shipment will dynamically add / remove unit from stock item.
+        # so we can directly check can_supply with stock items directly.
+        return variant.stock_items.sum(:count_on_hand) >= quantity if variant.delivery_required?
 
         if variant.permanent_stock?
-          return [] if options[:from_date].blank? || options[:to_date].blank?
-
-          dates = options[:from_date].to_date..options[:to_date].to_date
-          @cached_inventory_items = builder_klass.new(variant_id: variant.id, dates: dates).call
-          @cached_inventory_items = [] if @cached_inventory_items.size != dates.count
-          @cached_inventory_items
+          permanent_stock_variant_available?(quantity, options)
         else
-          @cached_inventory_items = builder_klass.new(variant_id: variant.id).call
+          variant_available?(quantity, options)
         end
       end
 
-      def builder_klass
-        ::SpreeCmCommissioner::RedisStock::VariantCachedInventoryItemsBuilder
+      def variant_available?(quantity = 1, options = {})
+        query = SpreeCmCommissioner::VariantAvailability::NonPermanentStockQuery.new(
+          variant: variant,
+          except_line_item_id: options[:except_line_item_id]
+        )
+        result = query.available?(quantity)
+        @error_message = query.error_message unless result
+        result
+      end
+
+      def permanent_stock_variant_available?(quantity = 1, options = {})
+        SpreeCmCommissioner::VariantAvailability::PermanentStockQuery.new(
+          variant: variant,
+          from_date: options[:from_date].to_date,
+          to_date: options[:to_date].to_date,
+          except_line_item_id: options[:except_line_item_id]
+        ).available?(quantity)
       end
     end
   end

data/app/models/spree_cm_commissioner/stock/availability_validator_decorator.rb

@@ -3,8 +3,7 @@ module SpreeCmCommissioner
     module AvailabilityValidatorDecorator
       # override
       def item_available?(line_item, quantity)
-        SpreeCmCommissioner::Stock::LineItemAvailabilityChecker.new(line_item)
-                                                               .can_supply?(quantity)
+        SpreeCmCommissioner::Stock::LineItemAvailabilityChecker.new(line_item).can_supply?(quantity)
       end
     end
   end

data/app/models/spree_cm_commissioner/stock/line_item_availability_checker.rb

@@ -8,14 +8,14 @@ module SpreeCmCommissioner
       end
 
       def can_supply?(quantity)
-        ::SpreeCmCommissioner::Stock::AvailabilityChecker.new(line_item.variant, options)
-                                                         .can_supply?(quantity)
+        AvailabilityChecker.new(line_item.variant).can_supply?(quantity, options)
       end
 
       def options
         {
           from_date: line_item.from_date,
-          to_date: line_item.to_date
+          to_date: line_item.to_date,
+          except_line_item_id: line_item.id
         }
       end
     end

data/app/models/spree_cm_commissioner/stock_item_decorator.rb

@@ -3,28 +3,11 @@ module SpreeCmCommissioner
     def self.prepended(base)
       base.has_one :vendor, through: :variant
       base.after_save :update_vendor_total_inventory, if: :saved_change_to_count_on_hand?
-
-      base.after_commit :create_inventory_items, on: :create
-      base.after_commit :adjust_inventory_items_async, on: :destroy
     end
 
     def update_vendor_total_inventory
       SpreeCmCommissioner::VendorJob.perform_later(vendor.id) if vendor.present?
     end
-
-    private
-
-    def create_inventory_items
-      SpreeCmCommissioner::Stock::InventoryItemsGeneratorJob.perform_later(variant_id: variant.id)
-    end
-
-    # When admin delete stock item, it will deduct stock from inventory item
-    def adjust_inventory_items_async
-      params = { variant_id: variant.id, quantity: -count_on_hand }
-      CmAppLogger.log(label: "#{self.class.name}#adjust_inventory_items_async", data: params) do
-        SpreeCmCommissioner::Stock::InventoryItemsAdjusterJob.perform_later(**params)
-      end
-    end
   end
 end
 

data/app/models/spree_cm_commissioner/stop.rb

@@ -0,0 +1,23 @@
+require_dependency 'spree_cm_commissioner'
+
+module SpreeCmCommissioner
+  class Stop < SpreeCmCommissioner::Place
+    belongs_to :branch, class_name: 'SpreeCmCommissioner::Branch'
+    belongs_to :state,  class_name: 'Spree::State', optional: true
+    belongs_to :vendor, class_name: 'Spree::Vendor'
+
+    after_save :add_to_option_value
+
+    def validate_reference?
+      false
+    end
+
+    def add_to_option_value
+      origin = Spree::OptionType.find_by(attr_type: 'origin')&.id
+      destination = Spree::OptionType.find_by(attr_type: 'destination')&.id
+
+      Spree::OptionValue.create(option_type_id: origin, name: name, presentation: id)
+      Spree::OptionValue.create(option_type_id: destination, name: name, presentation: id)
+    end
+  end
+end

data/app/models/spree_cm_commissioner/taxon_decorator.rb

@@ -2,6 +2,7 @@ module SpreeCmCommissioner
   module TaxonDecorator
     def self.prepended(base) # rubocop:disable Metrics/AbcSize,Metrics/MethodLength
       base.include SpreeCmCommissioner::TaxonKind
+      base.include SpreeCmCommissioner::Transit::TaxonBitwise
       base.include SpreeCmCommissioner::ParticipationTypeBitwise
       base.include SpreeCmCommissioner::EventCheckInFlowable
 

data/app/models/spree_cm_commissioner/taxonomy_decorator.rb

@@ -8,6 +8,12 @@ module SpreeCmCommissioner
           Spree::Taxonomy.find_or_create_by(name: 'Businesses', kind: 'category', store: Spree::Store.default)
         end
       end
+
+      def base.place
+        ActiveRecord::Base.connected_to(role: :writing) do
+          Spree::Taxonomy.find_or_create_by(name: 'Place', kind: 'transit', store: Spree::Store.default)
+        end
+      end
     end
   end
 end

data/app/models/spree_cm_commissioner/trip.rb

@@ -1,15 +1,14 @@
+require_dependency 'spree_cm_commissioner'
 module SpreeCmCommissioner
-  class Trip < Base
+  class Trip < SpreeCmCommissioner::Base
     attr_accessor :hours, :minutes, :seconds
 
     before_validation :convert_duration_to_seconds
 
-    belongs_to :vendor, class_name: 'Spree::Vendor', inverse_of: :trips, optional: false
     belongs_to :product, class_name: 'Spree::Product', inverse_of: :trip, optional: false
     belongs_to :vehicle, class_name: 'SpreeCmCommissioner::Vehicle', optional: false
-
-    belongs_to :origin_place, class_name: 'SpreeCmCommissioner::Place', optional: false
-    belongs_to :destination_place, class_name: 'SpreeCmCommissioner::Place', optional: false
+    belongs_to :origin, class_name: 'SpreeCmCommissioner::Place', optional: true
+    belongs_to :destination, class_name: 'SpreeCmCommissioner::Place', optional: true
 
     has_many :trip_blazer_queries, as: :queryable, class_name: 'SpreeCmCommissioner::BlazerQueryable'
     has_many :blazer_queries, through: :trip_blazer_queries, source: :blazer_query, class_name: 'Blazer::Query'
@@ -17,13 +16,16 @@ module SpreeCmCommissioner
     has_many :trip_stops, class_name: 'SpreeCmCommissioner::TripStop', dependent: :destroy
     has_many :variants, through: :product
     has_many :inventory_items, through: :variants
+    belongs_to :variant, class_name: 'Spree::Variant'
+
+    belongs_to :route, class_name: 'Spree::Product'
+    belongs_to :vehicle, class_name: 'SpreeCmCommissioner::Vehicle'
+    validates :departure_time, presence: true
 
     validates :departure_time, presence: true
     validates :duration, numericality: { greater_than: 0 }
     validate :origin_and_destination_cannot_be_the_same
 
-    accepts_nested_attributes_for :trip_stops, :product, allow_destroy: true
-
     def convert_duration_to_seconds
       return if hours.blank? && minutes.blank? && seconds.blank?
 
@@ -48,7 +50,7 @@ module SpreeCmCommissioner
     private
 
     def origin_and_destination_cannot_be_the_same
-      return unless origin_place_id == destination_place_id
+      return unless origin_id == destination_id
 
       errors.add(:base, 'Origin and destination cannot be the same')
     end

data/app/models/spree_cm_commissioner/trip_connection.rb

@@ -1,7 +1,7 @@
 module SpreeCmCommissioner
-  class TripConnection < Base
-    belongs_to :from_trip, class_name: 'SpreeCmCommissioner::Trip'
-    belongs_to :to_trip, class_name: 'SpreeCmCommissioner::Trip'
+  class TripConnection < ApplicationRecord
+    belongs_to :from_trip, class_name: 'Spree::Variant'
+    belongs_to :to_trip, class_name: 'Spree::Variant'
 
     validate :both_trip_cannot_be_the_same
     before_validation :calculate_connection_time_minutes
@@ -12,8 +12,8 @@ module SpreeCmCommissioner
     def calculate_connection_time_minutes
       return if from_trip.nil? || to_trip.nil?
 
-      arrival_seconds   = from_trip.arrival_time.seconds_since_midnight
-      departure_seconds = to_trip.departure_time.seconds_since_midnight
+      arrival_seconds   = from_trip.trip.arrival_time.seconds_since_midnight
+      departure_seconds = to_trip.trip.departure_time.seconds_since_midnight
 
       layover_seconds = departure_seconds - arrival_seconds
       layover_seconds += 86_400 if layover_seconds.negative?

data/app/models/spree_cm_commissioner/trip_stop.rb

@@ -1,17 +1,36 @@
+require_dependency 'spree_cm_commissioner'
 module SpreeCmCommissioner
-  class TripStop < Base
+  class TripStop < SpreeCmCommissioner::Base
     acts_as_list column: :sequence, scope: :trip_id
     enum :stop_type, { boarding: 0, drop_off: 1 }
 
-    belongs_to :trip, class_name: 'SpreeCmCommissioner::Trip', optional: false
-    belongs_to :stop_place, class_name: 'SpreeCmCommissioner::Place', optional: false
-    belongs_to :location_place, class_name: 'SpreeCmCommissioner::Place', optional: false
+    belongs_to :trip, class_name: 'Spree::Variant'
+    belongs_to :stop, class_name: 'SpreeCmCommissioner::Place'
 
     before_validation :set_stop_name
-    validates :stop_place_id, uniqueness: { scope: :trip_id }
+    after_destroy :decrement_trip_count
+    after_commit :create_vendor_stop
+    validates :stop_id, uniqueness: { scope: :trip_id }
 
     def set_stop_name
-      self.stop_name = stop_place.name if stop_place.present?
+      self.stop_name = stop.name if stop.present?
+    end
+
+    def create_vendor_stop
+      vendor_stop = vendor.vendor_stops.where(stop_id: stop_id, stop_type: stop_type).first_or_create
+      vendor_stop.update(trip_count: vendor_stop.trip_count.to_i + 1)
+    end
+
+    private
+
+    def decrement_trip_count
+      vendor.vendor_stops.where(stop_id: stop_id, stop_type: stop_type).find_each do |vendor_stop|
+        vendor_stop.update(trip_count: [vendor_stop.trip_count - 1, 0].max)
+      end
+    end
+
+    def vendor
+      Spree::Product.find(trip&.product_id).vendor
     end
   end
 end

data/app/models/spree_cm_commissioner/user_identity_provider.rb

@@ -3,15 +3,17 @@ module SpreeCmCommissioner
     enum identity_type: { :google => 0, :apple => 1, :facebook => 2, :telegram => 3, :vattanac_bank => 4 }
 
     belongs_to :user, class_name: Spree.user_class.to_s, optional: false
+    has_one :tenant, through: :user
 
-    validates :sub, presence: true
-    validates :sub, uniqueness: { scope: :identity_type }
+    has_many :user_identity_provider_telegram_bots
+    has_many :telegram_bots, through: :user_identity_provider_telegram_bots
 
+    validates :sub, presence: true
     validates :identity_type, presence: true
     validates :identity_type, uniqueness: { scope: :user_id }
 
-    has_many :user_identity_provider_telegram_bots
-    has_many :telegram_bots, through: :user_identity_provider_telegram_bots
+    # custom validation to ensure that the sub is unique per identity_type and spree_users.tenant
+    validate :unique_sub_per_identity_type_and_tenant
 
     # sub is a telegram uid, which telegram considered a chatID if
     # user have /started with bot.
@@ -20,5 +22,25 @@ module SpreeCmCommissioner
 
       nil
     end
+
+    private
+
+    def unique_sub_per_identity_type_and_tenant
+      return if user.nil? || sub.blank? || identity_type.blank?
+
+      tenant_id = user.tenant_id
+
+      # Check for duplicates in the same tenant
+      scope = self.class
+                  .joins(:user)
+                  .where(sub: sub, identity_type: identity_type)
+                  .where(spree_users: { tenant_id: tenant_id })
+
+      scope = scope.where.not(id: id) if persisted?
+
+      return unless scope.exists?
+
+      errors.add(:sub, 'must be unique per identity_type and tenant')
+    end
   end
 end