spree_cm_commissioner 2.8.3.pre6 → 2.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 03105af0ab1e480deee42cf481634c850da1a67603e5b443353d4aad31017181
-  data.tar.gz: 1add556af176fac0f8ac9be335406537e857c438dacd38a3a0769fa841a60e99
+  metadata.gz: 0774f6d0c4fad7ae396f14326fd6d0965164b74d89f355c3966bc8d1fd9563f9
+  data.tar.gz: 002d84c9f1273b54f01d5a25bdecc5f3b475a12e07b32d8e1846c19107fe776f
 SHA512:
-  metadata.gz: d6e85ae08be39fc2a66791c42a049ae3be212fa16af34bc0edaa7366c1b4b22305a381029f0d76a0f03807e7bc1e53ed82c35d8dd5b88f05cb806dc39f520492
-  data.tar.gz: fc23fd4348e6b70720c5e803b278f07e9c6303e37c3ea1080b254a511da02fd75c30426bbfe9107b925efa8280f6b10b5d4d84edc42d5e38b9643cc4f3190375
+  metadata.gz: 260d54db1f9dd63dbfef02edf2944d02e2c582972b8903815462e438be9cbfb6f5561a7b4a4b64c2cda4af18908c8c7a91958213c06764d1f7ee4d8e2c16dec3
+  data.tar.gz: 003d6ecd547b0215851266f06e823f2f7c5e7a8da35f5b4a8051784855a4d095d088b79f700dcadca8820d561d7a9989a3ee0d9eb7940943c3c5777cc7a4742c

data/Gemfile.lock

@@ -34,7 +34,7 @@ GIT
 PATH
   remote: .
   specs:
-    spree_cm_commissioner (2.8.3.pre6)
+    spree_cm_commissioner (2.8.3)
       activerecord-multi-tenant
       activerecord_json_validator (~> 2.1, >= 2.1.3)
       aws-sdk-cloudfront

data/app/controllers/spree/admin/guests_controller.rb

@@ -55,7 +55,7 @@ module Spree
         if result.success?
           flash[:success] = "Guest #{@guest.full_name} has been checked in."
         else
-          flash[:error] = result.error.value.to_s.titleize
+          flash[:error] = result.message.to_s.titleize
         end
 
         redirect_to collection_url

data/app/controllers/spree/admin/stock_managements_controller.rb

@@ -92,7 +92,7 @@ module Spree
                                                                                                 .call
                                                                                                 .index_by(&:inventory_item_id)
         @reserved_stocks = Spree::LineItem
-                           .affecting_inventory
+                           .complete
                            .where(variant_id: @variants.pluck(:id))
                            .group('spree_line_items.variant_id')
                            .sum(:quantity)

data/app/controllers/spree/api/v2/tenant/trips_controller.rb

@@ -25,17 +25,6 @@ module Spree
           def resource_serializer
             Spree::V2::Tenant::TripSerializer
           end
-
-          # override
-          def serializer_params
-            super.merge(
-              include_details: include_details?
-            )
-          end
-
-          def include_details?
-            params.fetch(:include_details, 'true') == 'true'
-          end
         end
       end
     end

data/app/controllers/spree_cm_commissioner/admin/orders_controller_decorator.rb

@@ -40,13 +40,9 @@ module SpreeCmCommissioner
 
       # override
       def cancel
-        if @order.guests_in_transfer?
-          flash[:error] = Spree.t('admin.orders.cannot_cancel_guest_transfer')
-        else
-          internal_note = params.dig(:order, :internal_note)
-          @order.canceled_by(try_spree_current_user, cancellation_reason: internal_note)
-          flash[:success] = Spree.t(:order_canceled)
-        end
+        internal_note = params.dig(:order, :internal_note)
+        @order.canceled_by(try_spree_current_user, cancellation_reason: internal_note)
+        flash[:success] = Spree.t(:order_canceled)
         redirect_back fallback_location: spree.edit_admin_order_url(@order)
       end
 

data/app/finders/spree_cm_commissioner/accommodations/find_variant.rb

@@ -15,8 +15,8 @@ module SpreeCmCommissioner
           .joins(:inventory_items)
           .where(vendor_id: vendor_id)
           .where(inventory_items: { inventory_date: stay_dates })
-          .where('CAST(spree_variants.public_metadata->\'cm_options\'->>\'number-of-adults\' AS INTEGER) +
-                  CAST(spree_variants.public_metadata->\'cm_options\'->>\'number-of-kids\' AS INTEGER) >= ?', number_of_guests
+          .where('CAST(public_metadata->\'cm_options\'->>\'number-of-adults\' AS INTEGER) +
+                  CAST(public_metadata->\'cm_options\'->>\'number-of-kids\' AS INTEGER) >= ?', number_of_guests
           )
           .where('inventory_items.quantity_available > 0')
           .distinct

data/app/helpers/spree_cm_commissioner/admin/guest_helper.rb

@@ -61,16 +61,6 @@ module SpreeCmCommissioner
           'badge-info'
         end
       end
-
-      def guest_state_badge(guest)
-        badge_class = case guest.state
-                      when 'active' then 'badge-success'
-                      when 'locked' then 'badge-warning'
-                      when 'transferred' then 'badge-secondary'
-                      else 'badge-info'
-                      end
-        content_tag(:span, guest.state.humanize, class: "badge #{badge_class}")
-      end
     end
   end
 end

data/app/models/concerns/spree_cm_commissioner/line_items_filter_scope.rb

@@ -10,11 +10,6 @@ module SpreeCmCommissioner
       scope :accepted, -> { joins(:order).merge(Spree::Order.accepted) }
       scope :paid, -> { joins(:order).merge(Spree::Order.paid) }
       scope :with_bib_prefix, -> { joins(:option_types).where(option_types: { name: 'bib-prefix' }) }
-      scope :exclude_ticket_transfer_orders, -> { joins(:order).merge(Spree::Order.not_ticket_transfer) }
-
-      # Centralizes the rule so that adjustments (e.g. excluding new order kinds in the
-      # future) only need to be made here. Use this anywhere inventory is computed.
-      scope :affecting_inventory, -> { complete.exclude_ticket_transfer_orders }
 
       scope :filter_by_event, lambda { |event|
         case event

data/app/models/concerns/spree_cm_commissioner/vehicle_kind.rb

@@ -2,7 +2,7 @@ module SpreeCmCommissioner
   module VehicleKind
     extend ActiveSupport::Concern
 
-    VEHICLE_KINDS = %i[suv sedan minivan van sleeping_bus luxury_van air_bus bus ferry boat].freeze
+    VEHICLE_KINDS = %i[suv sedan minivan van sleeping_bus luxury_van air_bus bus ferry].freeze
 
     included do
       enum kind: VEHICLE_KINDS if table_exists? && column_names.include?('kind')

data/app/models/spree_cm_commissioner/guest.rb

@@ -12,7 +12,6 @@ module SpreeCmCommissioner
     enum gender: { :other => 0, :male => 1, :female => 2 }
     enum age_group: { :adult => 0, :child => 1 }
     enum nationality_group: { :local => 0, :foreigner => 1 }
-    enum state: { active: 0, locked: 1, transferred: 2 }
     enum social_contact_platform: {
       :other => 0,
       :telegram => 1,
@@ -47,7 +46,6 @@ module SpreeCmCommissioner
     has_many :state_changes, as: :stateful, class_name: 'Spree::StateChange'
     has_many :guest_dynamic_fields, dependent: :destroy, class_name: 'SpreeCmCommissioner::GuestDynamicField'
     has_many :check_ins, class_name: 'SpreeCmCommissioner::CheckIn'
-    has_many :ticket_transfers, foreign_key: 'from_guest_id', class_name: 'SpreeCmCommissioner::TicketTransfer'
 
     accepts_nested_attributes_for :guest_dynamic_fields, allow_destroy: true, reject_if: :reject_guest_dynamic_field?
 
@@ -192,10 +190,6 @@ module SpreeCmCommissioner
       [first_name, last_name].compact_blank.join(' ')
     end
 
-    def display_name_with_bib
-      [full_name, formatted_bib_number, seat_number].compact.join(' | ')
-    end
-
     def generate_svg_qr
       qrcode = RQRCode::QRCode.new(qr_data)
       qrcode.as_svg(
@@ -515,20 +509,6 @@ module SpreeCmCommissioner
       saved_change_to_data_fill_stage_phase? ||
         (guest_dynamic_fields.any? && (guest_dynamic_fields.any?(&:saved_changes?) || guest_dynamic_fields.any?(&:destroyed?)))
     end
-
-    # Ticket Transfer validations
-    def allows_transfer?
-      line_item&.product&.allow_transfer?
-    end
-
-    def allows_gift_transfer?
-      line_item&.product&.allow_gift_transfer?
-    end
-
-    def event_started?
-      event_from_date = event&.from_date
-      event_from_date.present? && event_from_date <= Time.current
-    end
   end
 end
 # rubocop:enable Metrics/ClassLength

data/app/models/spree_cm_commissioner/integration.rb

@@ -25,36 +25,5 @@ module SpreeCmCommissioner
     def unstock_external_inventory!(_order, _line_items)
       raise NotImplementedError, 'Subclasses must implement the unstock_external_inventory! method'
     end
-
-    def sync_item_availability!(_options = {})
-      raise NotImplementedError, 'Subclasses must implement the sync_item_availability! method'
-    end
-
-    def hold_inventory!(_order, _line_items)
-      raise NotImplementedError, 'Subclasses must implement the hold_inventory! method'
-    end
-
-    # Integrations that manage external inventory reservations should override this to return true
-    def supports_external_inventory_holding?
-      false
-    end
-
-    # Integrations that support automatically releasing external inventory
-    # after reservation expiration or order cancellation should override this to return true
-    def supports_auto_release_inventory?
-      false
-    end
-
-    def release_inventory!(_order, _line_items)
-      raise NotImplementedError, 'Subclasses must implement the release_inventory! method'
-    end
-
-    def external_inventory_items(_order)
-      raise NotImplementedError, 'Subclasses must implement the external_inventory_items method'
-    end
-
-    def external_inventory_item_status(_order, _options)
-      raise NotImplementedError, 'Subclasses must implement the external_inventory_item_status method'
-    end
   end
 end

data/app/models/spree_cm_commissioner/integrations/stadium_x_v1.rb

@@ -5,7 +5,10 @@ class SpreeCmCommissioner::Integrations::StadiumXV1 < SpreeCmCommissioner::Integ
 
   # override
   def sync_manager
-    SpreeCmCommissioner::Integrations::StadiumXV1::SyncManager.new(integration: self)
+    SpreeCmCommissioner::Integrations::StadiumXV1::SyncManager.new(
+      integration: self,
+      client: client
+    )
   end
 
   # override
@@ -15,6 +18,7 @@ class SpreeCmCommissioner::Integrations::StadiumXV1 < SpreeCmCommissioner::Integ
       order: order,
       line_items: line_items
     )
+
     raise SpreeCmCommissioner::Integrations::SyncError, result.error unless result.success?
   end
 
@@ -23,9 +27,11 @@ class SpreeCmCommissioner::Integrations::StadiumXV1 < SpreeCmCommissioner::Integ
     raise SpreeCmCommissioner::Integrations::SyncError, 'Ticket cannot be cancelled'
   end
 
-  def sync_item_availability!(_options = {}); end
-
   def client
-    @client ||= SpreeCmCommissioner::Integrations::StadiumXV1::ExternalClient::Client.new(integration: self)
+    SpreeCmCommissioner::Integrations::StadiumXV1::ExternalClient::Client.new(
+      public_key: public_key,
+      private_key: private_key,
+      base_url: base_url
+    )
   end
 end

data/app/models/spree_cm_commissioner/inventory_item.rb

@@ -1,15 +1,9 @@
 module SpreeCmCommissioner
   class InventoryItem < Base
     include SpreeCmCommissioner::ProductType
-    include SpreeCmCommissioner::StoreMetadata
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
 
     MAX_DISPLAY_STOCK = 20
 
-    store_public_metadata :last_synced_at, :datetime
-    store_public_metadata :original_price, :integer
-    store_public_metadata :currency, :string
-
     # Association
     belongs_to :variant, class_name: 'Spree::Variant', optional: false
 

data/app/models/spree_cm_commissioner/line_item_decorator.rb

@@ -119,9 +119,6 @@ module SpreeCmCommissioner
 
     # override to calculate price per date for permanent stock
     def update_price
-      # Skip price updates for ticket transfer orders to preserve custom transfer pricing
-      return if order&.ticket_transfer?
-
       base_price = variant.price_in(order.currency)
       self.price = if permanent_stock?
                      date_range.map do |date|
@@ -259,11 +256,7 @@ module SpreeCmCommissioner
     end
 
     # override
-    # Ticket transfer line items reuse the seller's already-committed inventory,
-    # so stock checks must not block the receiver's checkout when the event is sold out.
     def sufficient_stock?
-      return true if order&.ticket_transfer?
-
       SpreeCmCommissioner::Stock::LineItemAvailabilityChecker.new(self).can_supply?(quantity)
     end
 

data/app/models/spree_cm_commissioner/notification.rb

@@ -10,16 +10,7 @@ module SpreeCmCommissioner
 
     scope :user_notifications, lambda {
       where(
-        type: %w[
-          order_complete_notification
-          customer_notification
-          guest_dynamic_field_notification
-          ticket_transfer_received_notification
-          ticket_transfer_accepted_notification
-          ticket_transfer_completed_notification
-          ticket_transfer_rejected_notification
-          ticket_transfer_expired_notification
-        ]
+        type: %w[order_complete_notification customer_notification guest_dynamic_field_notification]
       )
     }
 

data/app/models/spree_cm_commissioner/order_decorator.rb

@@ -1,11 +1,5 @@
 module SpreeCmCommissioner
   module OrderDecorator
-    SPREE_CHANNEL = 'spree'.freeze
-    GOOGLE_FORM_CHANNEL = 'google_form'.freeze
-    TELEGRAM_CHANNEL = 'telegram'.freeze
-    TICKET_TRANSFER_CHANNEL = 'ticket_transfer'.freeze
-    ALLOWED_CHANNEL_PREFIXES = [SPREE_CHANNEL, GOOGLE_FORM_CHANNEL, TELEGRAM_CHANNEL, TICKET_TRANSFER_CHANNEL].freeze
-
     def self.prepended(base) # rubocop:disable Metrics/MethodLength
       base.include SpreeCmCommissioner::StoreMetadata
       base.include SpreeCmCommissioner::PhoneNumberSanitizer
@@ -14,7 +8,6 @@ module SpreeCmCommissioner
       base.include SpreeCmCommissioner::OrderStateMachine
       base.include SpreeCmCommissioner::RouteOrderCountable
       base.include SpreeCmCommissioner::OrderScopes
-      base.include SpreeCmCommissioner::TicketTransferable
 
       base.before_create :link_by_phone_number
       base.before_create :associate_customer
@@ -50,7 +43,6 @@ module SpreeCmCommissioner
       base.has_many :reserved_blocks, through: :guests, class_name: 'SpreeCmCommissioner::ReservedBlock'
       base.has_many :guest_card_classes, class_name: 'SpreeCmCommissioner::GuestCardClass', through: :variants
       base.has_many :product_completion_steps, class_name: 'SpreeCmCommissioner::ProductCompletionStep', through: :line_items
-      base.has_many :audit_events, class_name: 'SpreeCmCommissioner::AuditEvent', as: :auditable
 
       base.delegate :customer, to: :user, allow_nil: true
 
@@ -242,10 +234,6 @@ module SpreeCmCommissioner
       complete? && need_confirmation? == false
     end
 
-    def guests_in_transfer?
-      guests.exists?(state: %i[locked transferred])
-    end
-
     def generate_svg_qr
       qrcode = RQRCode::QRCode.new(qr_data)
       qrcode.as_svg(
@@ -356,9 +344,10 @@ module SpreeCmCommissioner
     end
 
     def validate_channel_prefix
-      return if ALLOWED_CHANNEL_PREFIXES.any? { |prefix| channel&.start_with?(prefix) }
+      allowed_prefixes = %w[spree google_form telegram]
+      return if allowed_prefixes.any? { |prefix| channel&.start_with?(prefix) }
 
-      errors.add(:channel, "must start with one of the following: #{ALLOWED_CHANNEL_PREFIXES.join(', ')}")
+      errors.add(:channel, "must start with one of the following: #{allowed_prefixes.join(', ')}")
     end
 
     def set_tenant_id

data/app/models/spree_cm_commissioner/payment_decorator.rb

@@ -32,14 +32,6 @@ module SpreeCmCommissioner
       payment_method.is_a?(Spree::PaymentMethod::Check)
     end
 
-    # @override Spree::Payment#after_completed
-    # Called inside the state machine transition to :completed (within the transaction),
-    # ensuring Complete is atomic with the payment completion.
-    def after_completed
-      super
-      complete_ticket_transfer if ticket_transfer_completable?
-    end
-
     def can_void?
       state.in? %i[pending processing completed checkout]
     end
@@ -55,17 +47,6 @@ module SpreeCmCommissioner
     def lock_holds_for_payment
       order.lock_order_holds_for_payment
     end
-
-    def ticket_transfer_completable?
-      completed? && order.ticket_transfer? && order.complete? && order.payment_fulfilled?
-    end
-
-    def complete_ticket_transfer
-      ticket_transfer = order.ticket_transfer
-      return if ticket_transfer.blank? || !ticket_transfer.accepted?
-
-      SpreeCmCommissioner::TicketTransfers::Complete.call(ticket_transfer: ticket_transfer)
-    end
   end
 end
 

data/app/models/spree_cm_commissioner/payment_method_decorator.rb

@@ -5,14 +5,6 @@ module SpreeCmCommissioner
     def self.prepended(base)
       base.const_set(:DISPLAY, DISPLAY)
 
-      # Expose a dedicated class helper so services can call Spree::PaymentMethod.ticket_transfer
-      # instead of duplicating lookup/creation logic.
-      base.define_singleton_method(:ticket_transfer) do
-        Spree::PaymentMethod::Check.available_on_back_end.find_or_create_by!(name: 'Ticket Transfer') do |method|
-          method.stores = [Spree::Store.default]
-        end
-      end
-
       base.scope :available_on_frontend_for_early_adopter, -> { active.where(display_on: %i[both front_end frontend_for_early_adopter]) }
 
       # override

data/app/models/spree_cm_commissioner/place.rb

@@ -1,7 +1,5 @@
 module SpreeCmCommissioner
   class Place < Base
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
-
     acts_as_nested_set
 
     validates :reference, presence: true, if: :validate_reference?

data/app/models/spree_cm_commissioner/reserved_block.rb

@@ -13,8 +13,6 @@ module SpreeCmCommissioner
       canceled: 2
     }
 
-    enum gender: { :other => 0, :male => 1, :female => 2 }
-
     validates :status, presence: true
     validates :expired_at, presence: true, if: -> { on_hold? }
 

data/app/models/spree_cm_commissioner/route.rb

@@ -1,7 +1,6 @@
 module SpreeCmCommissioner
   class Route < Base
     include SpreeCmCommissioner::RouteType
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
     include SpreeCmCommissioner::StoreMetadata
     include SpreeCmCommissioner::ServiceOrigin
 

data/app/models/spree_cm_commissioner/seat_layout.rb

@@ -1,7 +1,5 @@
 module SpreeCmCommissioner
   class SeatLayout < Base
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
-
     has_many :seat_sections, class_name: 'SpreeCmCommissioner::SeatSection', dependent: :destroy
 
     has_many :blocks, class_name: 'SpreeCmCommissioner::Block'

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

@@ -3,10 +3,6 @@ module SpreeCmCommissioner
     module AvailabilityValidatorDecorator
       # override
       def item_available?(line_item, quantity)
-        # Ticket-transfer orders move existing inventory between guests rather than
-        # purchasing new stock, so they must not be gated by the live stock check.
-        return true if line_item.order&.ticket_transfer?
-
         SpreeCmCommissioner::Stock::LineItemAvailabilityChecker.new(line_item)
                                                                .can_supply?(quantity)
       end

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

@@ -32,9 +32,6 @@ module SpreeCmCommissioner
       end
 
       def sufficient_stock_for?(line_item, cached_inventory_items)
-        # Ticket transfer orders move an already-owned stock between users,
-        # so they must skip stock checks even when the event is sold out.
-        return true if order.ticket_transfer?
         return false unless line_item.variant.available?
         return true unless line_item.variant.should_track_inventory?
         return true if line_item.variant.backorderable?

data/app/models/spree_cm_commissioner/stock_item_decorator.rb

@@ -15,10 +15,6 @@ module SpreeCmCommissioner
     private
 
     def create_inventory_items
-      # Skip automatic inventory generation when variant has manual_generate_inventory_items set to true
-      # Use case: For variants where inventory items need to be manually created/managed (e.g., external integrations)
-      return if variant.manual_generate_inventory_items?
-
       SpreeCmCommissioner::InventoryItems::GenerateInventoryItemsJob.perform_later(variant_id: variant.id)
     end
 

data/app/models/spree_cm_commissioner/trip.rb

@@ -2,7 +2,6 @@ module SpreeCmCommissioner
   class Trip < Base
     include SpreeCmCommissioner::StoreMetadata
     include SpreeCmCommissioner::RouteType
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
     include SpreeCmCommissioner::ServiceOrigin
 
     attr_accessor :hours, :minutes, :seconds
@@ -22,17 +21,12 @@ module SpreeCmCommissioner
 
     # before create, duplicate seat layout from vehicle_type if present
     before_validation :duplicate_seat_layout_from_vehicle, if: -> { seat_layout.nil? && vehicle_type&.seat_layout.present? }
-
     after_create :increment_route_metric_trip_count
-    after_create :increment_vendor_route_metric_trip_count
-
     before_destroy :decrement_route_metric_trip_count
-    before_destroy :decrement_vendor_route_metric_trip_count
-
     after_commit :update_route_price_range, on: %i[create update]
 
     belongs_to :vendor, class_name: 'Spree::Vendor', inverse_of: :trips, optional: false
-    belongs_to :product, class_name: 'Spree::Product', inverse_of: :trip, optional: true
+    belongs_to :product, class_name: 'Spree::Product', inverse_of: :trip, optional: false
 
     belongs_to :service_origin, class_name: 'Spree::Taxon', optional: true
 
@@ -168,8 +162,6 @@ module SpreeCmCommissioner
     end
 
     def increment_route_metric_trip_count
-      return if route_type.nil?
-
       SpreeCmCommissioner::RouteMetrics::IncreaseTripCountJob.perform_later(
         origin_place_id: origin_place_id,
         destination_place_id: destination_place_id,
@@ -178,8 +170,6 @@ module SpreeCmCommissioner
     end
 
     def decrement_route_metric_trip_count
-      return if route_type.nil?
-
       SpreeCmCommissioner::RouteMetrics::DecreaseTripCountJob.perform_later(
         origin_place_id: origin_place_id,
         destination_place_id: destination_place_id,
@@ -187,28 +177,6 @@ module SpreeCmCommissioner
       )
     end
 
-    def increment_vendor_route_metric_trip_count
-      return if vendor_id.nil? || route_type.nil?
-
-      SpreeCmCommissioner::VendorRouteMetrics::IncreaseTripCountJob.perform_later(
-        vendor_id: vendor_id,
-        origin_place_id: origin_place_id,
-        destination_place_id: destination_place_id,
-        route_type: route_type
-      )
-    end
-
-    def decrement_vendor_route_metric_trip_count
-      return if vendor_id.nil? || route_type.nil?
-
-      SpreeCmCommissioner::VendorRouteMetrics::DecreaseTripCountJob.perform_later(
-        vendor_id: vendor_id,
-        origin_place_id: origin_place_id,
-        destination_place_id: destination_place_id,
-        route_type: route_type
-      )
-    end
-
     def update_route_price_range
       return unless route_id.present? && product_id.present?
 

data/app/models/spree_cm_commissioner/trip_stop.rb

@@ -1,7 +1,5 @@
 module SpreeCmCommissioner
   class TripStop < Base
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
-
     acts_as_list column: :sequence, scope: :trip_id
 
     belongs_to :trip, class_name: 'SpreeCmCommissioner::Trip', optional: false

data/app/models/spree_cm_commissioner/user_decorator.rb

@@ -159,7 +159,6 @@ module SpreeCmCommissioner
       return first_name if first_name.present?
       return last_name if last_name.present?
       return email if email.present?
-      return login if login.present?
 
       phone_number
     end

data/app/models/spree_cm_commissioner/variant_decorator.rb

@@ -6,7 +6,6 @@ module SpreeCmCommissioner
       base.include SpreeCmCommissioner::VariantOptionsConcern
       base.include SpreeCmCommissioner::KycBitwise
       base.include SpreeCmCommissioner::Integrations::IntegrationMappable
-      base.include SpreeCmCommissioner::VariantMetadata
 
       base.after_commit :update_vendor_price
       base.validate     :validate_option_types

data/app/models/spree_cm_commissioner/vehicle.rb

@@ -1,7 +1,6 @@
 module SpreeCmCommissioner
   class Vehicle < Base
     include SpreeCmCommissioner::VehicleKind
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
 
     belongs_to :vendor, class_name: 'Spree::Vendor'
     belongs_to :vehicle_type, class_name: 'SpreeCmCommissioner::VehicleType'
@@ -14,8 +13,6 @@ module SpreeCmCommissioner
     validates :name, presence: true
     validates :license_plate, uniqueness: true, allow_blank: true
 
-    has_one :seat_layout, through: :vehicle_type
-
     self.whitelisted_ransackable_attributes = %w[name license_plate kind]
 
     def display_name

data/app/models/spree_cm_commissioner/vehicle_type.rb

@@ -2,7 +2,6 @@ module SpreeCmCommissioner
   class VehicleType < Base
     include SpreeCmCommissioner::StoreMetadata
     include SpreeCmCommissioner::VehicleKind
-    include SpreeCmCommissioner::Integrations::IntegrationMappable
 
     # This model has no seat_layout column (polymorphic association), so we store the preload_seat_layout_id ID in public_metadata.
     # This lets us check if a seat layout exists without triggering a database query.

data/app/models/spree_cm_commissioner/vendor_decorator.rb

@@ -30,7 +30,6 @@ module SpreeCmCommissioner
       base.has_many :nearby_places, -> { order(position: :asc) }, class_name: 'SpreeCmCommissioner::VendorPlace', dependent: :destroy
 
       base.has_many :stock_items, through: :variants, class_name: 'Spree::StockItem'
-      base.has_many :currency_rates, class_name: 'SpreeCmCommissioner::CurrencyRate', dependent: :destroy
 
       base.has_many :taxon_vendors, class_name: 'SpreeCmCommissioner::TaxonVendor'
       base.has_many :taxons, through: :taxon_vendors
@@ -98,7 +97,6 @@ module SpreeCmCommissioner
       base.has_many :boarding_trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop', source: :boarding_trip_stops
       base.has_many :drop_off_trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop', source: :drop_off_trip_stops
       base.has_many :pricing_models, class_name: 'SpreeCmCommissioner::PricingModel', foreign_key: :vendor_id, dependent: :destroy
-      base.has_many :orders, -> { distinct }, through: :variants, source: :orders
 
       base.has_many :agency_categories, -> { where(kind: :agency_category) }, class_name: 'Spree::Taxon', foreign_key: :vendor_id
 
@@ -241,21 +239,6 @@ module SpreeCmCommissioner
     end
 
     delegate :present?, to: :tenant, prefix: true
-
-    # Returns the exchange rate between two currencies
-    # If no direct rate exists, calculates the inverse rate from the reverse pair
-    # Returns 1.0 for identical currencies, nil if no rate is found
-    def exchange_rate(from_currency, to_currency)
-      return 1.0 if from_currency == to_currency
-
-      pair = currency_rates.by_pair(from_currency, to_currency).first
-      return pair.rate if pair.present?
-
-      reverse_pair = currency_rates.by_pair(to_currency, from_currency).first
-      return (1.0 / reverse_pair.rate).round(6) if reverse_pair.present?
-
-      nil
-    end
   end
 end