spree_cm_commissioner 2.3.2.pre.pre6 → 2.3.2

data/app/services/spree_cm_commissioner/pricing_rules/build_params.rb

@@ -1,53 +0,0 @@
-module SpreeCmCommissioner
-  module PricingRules
-    module BuildParams
-      module_function
-
-      def permitted_group_params(params)
-        group_params = params[:spree_cm_commissioner_pricing_rule_group] || params
-
-        {
-          name: group_params[:name],
-          status: group_params[:status],
-          template_key: group_params[:template_key],
-          match_type: group_params[:match_type],
-          pricing_rules_attributes: build_rules_attributes(params[:pricing_rules]),
-          pricing_action_attributes: build_action_attributes(params[:pricing_action])
-        }
-      end
-
-      def build_rules_attributes(rules_array)
-        return [] unless rules_array
-
-        rules_array.map do |r|
-          {
-            id: r[:id],
-            type: r[:type],
-            private_metadata: r[:private_metadata],
-            _destroy: r[:_destroy]
-          }
-        end
-      end
-
-      def build_action_attributes(action_data)
-        return nil unless action_data&.dig(:calculator)
-
-        calc_data = action_data[:calculator]
-        calculator = calc_data[:type].safe_constantize&.new
-        set_preferences(calculator, calc_data[:preferences]) if calculator
-
-        { id: action_data[:id], calculator: calculator }
-      end
-
-      def set_preferences(calculator, preferences)
-        return unless preferences && calculator
-
-        prefs = preferences.respond_to?(:to_unsafe_h) ? preferences.to_unsafe_h : preferences
-        prefs.each do |key, value|
-          setter = "preferred_#{key}="
-          calculator.public_send(setter, value) if calculator.respond_to?(setter)
-        end
-      end
-    end
-  end
-end

data/app/services/spree_cm_commissioner/pricing_rules/build_template.rb

@@ -1,27 +0,0 @@
-module SpreeCmCommissioner
-  module PricingRules
-    class BuildTemplate
-      def self.call(pricing_model:, template_key:)
-        template = PRICING_RULE_GROUP_TEMPLATES[template_key.to_sym]
-        return unless template
-
-        group = pricing_model.pricing_rule_groups.build
-
-        template[:rules]&.each do |rule_data|
-          group.pricing_rules.where(type: rule_data[:type]).build(
-            type: rule_data[:type],
-            private_metadata: rule_data[:private_metadata] || {}
-          )
-        end
-
-        if template[:action]
-          calculator = template[:action][:calculator_type].constantize.new
-          BuildParams.set_preferences(calculator, template[:action][:preferences])
-          group.build_pricing_action(calculator: calculator)
-        end
-
-        group
-      end
-    end
-  end
-end

data/app/services/spree_cm_commissioner/pricing_rules/create.rb

@@ -1,10 +0,0 @@
-module SpreeCmCommissioner
-  module PricingRules
-    class Create
-      def self.call(pricing_model:, params:)
-        attrs = BuildParams.permitted_group_params(params)
-        pricing_model.pricing_rule_groups.create(attrs)
-      end
-    end
-  end
-end

data/app/services/spree_cm_commissioner/pricing_rules/update.rb

@@ -1,11 +0,0 @@
-module SpreeCmCommissioner
-  module PricingRules
-    class Update
-      def self.call(pricing_rule_group:, params:)
-        attrs = BuildParams.permitted_group_params(params)
-        pricing_rule_group.update(attrs)
-        pricing_rule_group
-      end
-    end
-  end
-end

data/app/services/spree_cm_commissioner/vendor_places/base.rb

@@ -1,24 +0,0 @@
-module SpreeCmCommissioner
-  module VendorPlaces
-    class Base
-      prepend ::Spree::ServiceModule::Base
-
-      ALLOWED_PLACE_TYPES = %w[venue stop branch location].freeze
-
-      private
-
-      def validate_place_type!(place_type)
-        return I18n.t('controller.vendor_places.place_type_required', default: 'Place type is required') if place_type.nil?
-
-        unless ALLOWED_PLACE_TYPES.include?(place_type.to_s)
-          return I18n.t(
-            'controller.vendor_places.invalid_place_type',
-            default: "Place type must be one of: #{ALLOWED_PLACE_TYPES.join(', ')}"
-          )
-        end
-
-        nil
-      end
-    end
-  end
-end

data/app/services/spree_cm_commissioner/vendor_places/bulk_create.rb

@@ -1,70 +0,0 @@
-# Bulk creates vendor places of any valid type (e.g., :stop, :branch, :location) with validation and transaction support.
-#
-# Validates place_ids count and creates vendor places atomically.
-# Rolls back all changes if any creation fails.
-#
-# Usage:
-#   result = SpreeCmCommissioner::VendorPlaces::BulkCreate.call(
-#     vendor: current_vendor,
-#     place_ids: [1, 2, 3],
-#     place_type: :location, # or :stop, :branch, etc.
-#     description: "Sample description",
-#     max_count: 10,
-#     location: parent_location # required for :stop and :branch
-#   )
-#
-#   if result.success?
-#     created_vendor_places = result.value
-#   else
-#     error_message = result.error
-
-module SpreeCmCommissioner
-  module VendorPlaces
-    class BulkCreate < Base
-      def call(vendor:, place_ids:, place_type:, description: nil, max_count: 10, location: nil) # rubocop:disable Metrics/ParameterLists
-        error_message = validate_inputs!(place_ids: place_ids, place_type: place_type, max_count: max_count, location: location)
-        return failure(nil, error_message) if error_message
-
-        created_vendor_places = []
-
-        ActiveRecord::Base.transaction do
-          place_ids.each do |place_id|
-            vendor_place = build_vendor_place(
-              vendor: vendor,
-              place_id: place_id,
-              place_type: place_type,
-              description: description,
-              location: location
-            )
-
-            vendor_place.save!
-            created_vendor_places << vendor_place
-          end
-        end
-
-        success(created_vendor_places)
-      rescue ActiveRecord::RecordInvalid => e
-        failure(nil, e.record.errors.full_messages.to_sentence)
-      end
-
-      private
-
-      def validate_inputs!(place_ids:, place_type:, max_count:, location: nil)
-        validate_place_type!(place_type) ||
-          (place_ids.blank? && I18n.t('controller.vendor_places.place_ids_blank')) ||
-          (place_ids.size > max_count && I18n.t('controller.vendor_places.selection_limit', count: max_count)) ||
-          (requires_location?(place_type: place_type) && location.blank? && I18n.t('controller.vendor_places.location_required'))
-      end
-
-      def build_vendor_place(vendor:, place_id:, place_type:, description:, location: nil)
-        attrs = { vendor_id: vendor.id, place_type: place_type.to_sym, place_id: place_id, description: description }
-        attrs[:location_id] = location&.id if requires_location?(place_type: place_type)
-        SpreeCmCommissioner::VendorPlace.new(attrs)
-      end
-
-      def requires_location?(place_type:)
-        %i[stop branch].include?(place_type.to_sym)
-      end
-    end
-  end
-end

data/app/services/spree_cm_commissioner/vendor_places/create_with_google_map_data.rb

@@ -1,85 +0,0 @@
-# Creates a vendor place from Google Map base64-encoded place data.
-# Processes place data and creates both Place and VendorPlace records atomically.
-#
-# Usage:
-#   result = SpreeCmCommissioner::VendorPlaces::CreateWithGoogleMapData.call(
-#     vendor: current_vendor,
-#     place_type: 'stop',
-#     google_place_base_64_content: google_place_base_64_content,
-#     description: 'Main Warehouse',
-#     location: parent_vendor_location
-#   )
-#
-#   if result.success?
-#     vendor_place = result.value
-#   else
-#     error_message = result.error
-#   end
-
-module SpreeCmCommissioner
-  module VendorPlaces
-    class CreateWithGoogleMapData < Base
-      def call(vendor:, place_type:, google_place_base_64_content:, description: nil, location: nil)
-        error_message = validate_place_type!(place_type)
-        return failure(nil, error_message) if error_message.present?
-
-        # Only require location for stop and branch types
-        return failure(nil, I18n.t('controller.vendor_places.location_required')) if %w[stop branch].include?(place_type) && location.nil?
-
-        place = process_place(google_place_base_64_content: google_place_base_64_content)
-        return failure(nil, place) unless place.is_a?(SpreeCmCommissioner::Place)
-
-        vendor_place = create_vendor_place(vendor: vendor, place: place, place_type: place_type, description: description, location: location)
-        return failure(nil, vendor_place) if vendor_place.nil? || vendor_place.is_a?(String)
-
-        success(vendor_place)
-      end
-
-      private
-
-      def process_place(google_place_base_64_content:)
-        place = SpreeCmCommissioner::PlaceDecoder.process_place(google_place_base_64_content)
-        return I18n.t('controller.vendor_places.process_place_failed', default: 'Failed to process place') if place.nil? || !place.save
-
-        place
-      rescue StandardError => e
-        e.message
-      end
-
-      def create_vendor_place(vendor:, place:, place_type:, description:, location:)
-        error_message = nil
-        result = ActiveRecord::Base.transaction do
-          vendor_place = build_vendor_place(vendor: vendor, place: place, type: place_type, description: description, location: location)
-          unless vendor_place.save
-            error_message = vendor_place.errors.full_messages.to_sentence
-            raise ActiveRecord::Rollback
-          end
-          vendor_place
-        end
-        error_message || result
-      end
-
-      def build_vendor_place(vendor:, place:, type:, description: nil, location: nil)
-        case type
-        when 'stop'
-          vendor.stops.new(
-            place_id: place.id,
-            description: description,
-            location_id: location&.id
-          )
-        when 'branch'
-          vendor.branches.new(
-            place_id: place.id,
-            description: description,
-            location_id: location&.id
-          )
-        when 'location'
-          vendor.locations.new(
-            place_id: place.id,
-            description: description
-          )
-        end
-      end
-    end
-  end
-end

data/app/services/spree_cm_commissioner/vendor_places/update.rb

@@ -1,56 +0,0 @@
-# Updates all vendor places matching the same vendor, place_id, and location_id.
-# Only updates description and location_id. Rolls back if any update fails.
-#
-# Usage:
-#   result = SpreeCmCommissioner::VendorPlaces::Update.call(
-#     vendor: current_vendor,
-#     vendor_place: existing_vendor_place,
-#     location_id: new_location_id,
-#     description: 'Updated description'
-#     )
-#
-#   if result.success?
-#     updated_place = result.value
-#   else
-#     error_message = result.error
-#   end
-module SpreeCmCommissioner
-  module VendorPlaces
-    class Update < Base
-      def call(vendor:, vendor_place:, description: nil, location_id: nil)
-        if vendor_place.blank?
-          return failure(
-            nil,
-            I18n.t('controller.vendor_places.vendor_place_not_found', default: 'Vendor place not found')
-          )
-        end
-
-        ActiveRecord::Base.transaction do
-          related_places = SpreeCmCommissioner::VendorPlace.where(
-            vendor_id: vendor.id,
-            place_id: vendor_place.place_id,
-            location_id: vendor_place.location_id
-          )
-
-          update_attrs = {}
-          update_attrs[:description] = description unless description.nil?
-          update_attrs[:location_id] = location_id unless location_id.nil?
-
-          related_places.each do |vp|
-            unless vp.update(update_attrs)
-              error_message = vp.errors.full_messages.to_sentence
-              raise ActiveRecord::Rollback, error_message
-            end
-          end
-        end
-
-        vendor_place.reload
-        success(vendor_place)
-      rescue ActiveRecord::RecordInvalid => e
-        failure(nil, e.record.errors.full_messages.to_sentence)
-      rescue ActiveRecord::Rollback => e
-        failure(nil, e.message)
-      end
-    end
-  end
-end

data/db/migrate/20250915075830_create_spree_cm_commissioner_pricing_models.rb

@@ -1,17 +0,0 @@
-class CreateSpreeCmCommissionerPricingModels < ActiveRecord::Migration[7.0]
-  def change
-    create_table :cm_pricing_models, if_not_exists: true do |t|
-      t.string  :name, null: false
-      t.text    :description
-      t.integer :status, default: 0, null: false
-      t.integer :position, null: false, default: 0
-
-      t.references :vendor, null: false, foreign_key: { to_table: :spree_vendors }
-
-      t.timestamps
-    end
-
-    add_index :cm_pricing_models, :position unless index_exists?(:cm_pricing_models, :position)
-    add_index :cm_pricing_models, :vendor_id unless index_exists?(:cm_pricing_models, :vendor_id)
-  end
-end

data/db/migrate/20250915080733_create_spree_cm_commissioner_pricing_model_variants.rb

@@ -1,14 +0,0 @@
-class CreateSpreeCmCommissionerPricingModelVariants < ActiveRecord::Migration[7.0]
-  def change
-    create_table :cm_pricing_model_variants, if_not_exists: true do |t|
-      t.references :pricing_model, null: false, foreign_key: { to_table: :cm_pricing_models }
-      t.references :variant, null: false, foreign_key: { to_table: :spree_variants }
-      t.references :product, null: true, foreign_key: { to_table: :spree_products }
-
-      t.timestamps
-    end
-
-    add_index :cm_pricing_model_variants, [:pricing_model_id, :variant_id], name: 'idx_pm_variants_on_model_variant'
-    add_index :cm_pricing_model_variants, [:pricing_model_id, :product_id], name: 'idx_pm_variants_on_model_product'
-  end
-end

data/db/migrate/20251003082144_create_spree_cm_commissioner_pricing_rule_groups.rb

@@ -1,16 +0,0 @@
-class CreateSpreeCmCommissionerPricingRuleGroups < ActiveRecord::Migration[7.0]
-  def change
-    create_table :cm_pricing_rule_groups, if_not_exists: true do |t|
-      t.references :pricing_model, null: false, foreign_key: { to_table: :cm_pricing_models }
-      t.string  :name
-      t.integer :position, default: 0
-      t.integer :status, default: 0, null: false
-      t.jsonb :public_metadata
-      t.jsonb :private_metadata
-      t.timestamps
-    end
-
-    add_index :cm_pricing_rule_groups, :pricing_model_id, name: 'idx_rule_groups_on_model'
-    add_index :cm_pricing_rule_groups, [:pricing_model_id, :position], name: 'idx_rule_groups_on_model_position'
-  end
-end

data/db/migrate/20251003093408_create_spree_cm_commissioner_pricing_rule.rb

@@ -1,19 +0,0 @@
-class CreateSpreeCmCommissionerPricingRule < ActiveRecord::Migration[7.0]
-  def change
-    create_table :cm_pricing_rules, if_not_exists: true do |t|
-      t.references :pricing_rule_group, null: false, foreign_key: { to_table: :cm_pricing_rule_groups }
-      t.references :product, null: true, foreign_key: { to_table: :spree_products }
-      t.references :variant, null: true, foreign_key: { to_table: :spree_variants }
-      t.string :type, null: false
-      t.jsonb :public_metadata
-      t.jsonb :private_metadata
-      t.integer :position, null: false, default: 0
-
-      t.timestamps
-    end
-
-    add_index :cm_pricing_rules, :position, name: 'idx_pricing_rules_on_position'
-    add_index :cm_pricing_rules, [:pricing_rule_group_id, :product_id], name: 'idx_pricing_rules_on_group_product'
-    add_index :cm_pricing_rules, [:pricing_rule_group_id, :variant_id], name: 'idx_pricing_rules_on_group_variant'
-  end
-end

data/db/migrate/20251003093643_create_spree_cm_commissioner_pricing_action.rb

@@ -1,14 +0,0 @@
-class CreateSpreeCmCommissionerPricingAction < ActiveRecord::Migration[7.0]
-  def change
-    create_table :cm_pricing_actions, if_not_exists: true do |t|
-      t.references :pricing_rule_group, null: false, foreign_key: { to_table: :cm_pricing_rule_groups }
-      t.references :product, null: true, foreign_key: { to_table: :spree_products }
-      t.references :variant, null: true, foreign_key: { to_table: :spree_variants }
-
-      t.timestamps
-    end
-
-    add_index :cm_pricing_actions, [:pricing_rule_group_id, :product_id], name: 'idx_pricing_actions_on_group_product'
-    add_index :cm_pricing_actions, [:pricing_rule_group_id, :variant_id], name: 'idx_pricing_actions_on_group_variant'
-  end
-end

data/db/migrate/20251119093000_add_tenant_id_to_cm_vendor_places.rb

@@ -1,8 +0,0 @@
-class AddTenantIdToCmVendorPlaces < ActiveRecord::Migration[7.0]
-  def change
-    add_column :cm_vendor_places, :tenant_id, :integer, if_not_exists: true
-    # Why: VendorPlace#before_save sets tenant_id from vendor; tenant APIs (TripPlacesController) and TripQuery
-    # join on vendor_places scoped by tenant, so this speeds tenant-scoped lookups
-    add_index :cm_vendor_places, :tenant_id, if_not_exists: true
-  end
-end

data/lib/spree_cm_commissioner/intercity_taxi/distance.rb

@@ -1,38 +0,0 @@
-module SpreeCmCommissioner
-  module IntercityTaxi
-    class Distance
-      attr_accessor :distance_km, :ordered_points, :estimated_time_minutes, :base_km,
-                    :detour_pickup_km, :detour_dropoff_km, :extra_pickup_km, :extra_dropoff_km
-
-      def initialize(options = {})
-        options = options.stringify_keys if options.is_a?(Hash)
-
-        @distance_km = options['distance_km']&.to_f
-        @ordered_points = options['ordered_points']
-        @estimated_time_minutes = options['estimated_time_minutes']&.to_i
-        @base_km = options['base_km']&.to_f
-        @detour_pickup_km = options['detour_pickup_km']&.to_f
-        @detour_dropoff_km = options['detour_dropoff_km']&.to_f
-        @extra_pickup_km = options['extra_pickup_km']&.to_f
-        @extra_dropoff_km = options['extra_dropoff_km']&.to_f
-      end
-
-      def self.from_hash(hash)
-        new(hash || {})
-      end
-
-      def to_h
-        {
-          'distance_km' => @distance_km,
-          'ordered_points' => @ordered_points,
-          'estimated_time_minutes' => @estimated_time_minutes,
-          'base_km' => @base_km,
-          'detour_pickup_km' => @detour_pickup_km,
-          'detour_dropoff_km' => @detour_dropoff_km,
-          'extra_pickup_km' => @extra_pickup_km,
-          'extra_dropoff_km' => @extra_dropoff_km
-        }.compact
-      end
-    end
-  end
-end

data/lib/spree_cm_commissioner/intercity_taxi/map_place.rb

@@ -1,31 +0,0 @@
-module SpreeCmCommissioner
-  module IntercityTaxi
-    class MapPlace
-      attr_accessor :place_name, :lat, :lng, :oob_confirmed
-
-      def initialize(options = {})
-        options = options.stringify_keys if options.is_a?(Hash)
-
-        @place_name = options['place_name']
-        @lat = options['lat']&.to_f
-        @lng = options['lng']&.to_f
-
-        # Normalize truthy/falsey values ("true", "1", 1, etc.) to a real boolean.
-        @oob_confirmed = ActiveModel::Type::Boolean.new.cast(options['oob_confirmed'])
-      end
-
-      def self.from_hash(hash)
-        new(hash || {})
-      end
-
-      def to_h
-        {
-          'place_name' => @place_name,
-          'lat' => @lat,
-          'lng' => @lng,
-          'oob_confirmed' => @oob_confirmed
-        }.compact
-      end
-    end
-  end
-end

data/lib/spree_cm_commissioner/test_helper/factories/pricing_action_factory.rb

@@ -1,5 +0,0 @@
-FactoryBot.define do
-  factory :pricing_action, class: 'SpreeCmCommissioner::PricingAction' do
-    association :pricing_rule_group, factory: :pricing_rule_group
-  end
-end

data/lib/spree_cm_commissioner/test_helper/factories/pricing_model_factory.rb

@@ -1,8 +0,0 @@
-FactoryBot.define do
-  factory :cm_pricing_model, class: "SpreeCmCommissioner::PricingModel" do
-    association :vendor, factory: :cm_vendor
-    sequence(:name) { |n| "Pricing Model #{n}" }
-    description { "Test pricing model description" }
-    status { 'active' }
-  end
-end

data/lib/spree_cm_commissioner/test_helper/factories/pricing_rule_factory.rb

@@ -1,17 +0,0 @@
-FactoryBot.define do
-  factory :pricing_rule_nationality, class: 'SpreeCmCommissioner::PricingRules::Nationality' do
-    association :pricing_rule_group, factory: :pricing_rule_group
-    rule_type { 'must' }
-    nationalities { ["Cambodian"] }
-  end
-
-  factory :pricing_rule_extra_pick_up_distance, class: 'SpreeCmCommissioner::PricingRules::ExtraPickUpDistance' do
-    association :pricing_rule_group, factory: :pricing_rule_group
-    min_distance { 5 }
-  end
-
-  factory :pricing_rule_extra_drop_off_distance, class: 'SpreeCmCommissioner::PricingRules::ExtraDropOffDistance' do
-    association :pricing_rule_group, factory: :pricing_rule_group
-    min_distance { 5 }
-  end
-end

data/lib/spree_cm_commissioner/test_helper/factories/pricing_rule_group_factory.rb

@@ -1,8 +0,0 @@
-FactoryBot.define do
-  factory :pricing_rule_group, class: 'SpreeCmCommissioner::PricingRuleGroup' do
-    association :pricing_model, factory: :cm_pricing_model
-    sequence(:name) { |n| "Rule Group #{n}" }
-    match_type { 'all' }
-    position { 0 }
-  end
-end