spree_cm_commissioner 2.8.3.pre.pre8 → 2.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d301125d9cf803ac60cae2e7d479c356fd1e5ae5a08c23d90efb9b810ffd7851
-  data.tar.gz: d4ba06e44669773b337bc3a899edbc7172e81e5a3d1be3f3979bd7c62c0bbf5c
+  metadata.gz: 0774f6d0c4fad7ae396f14326fd6d0965164b74d89f355c3966bc8d1fd9563f9
+  data.tar.gz: 002d84c9f1273b54f01d5a25bdecc5f3b475a12e07b32d8e1846c19107fe776f
 SHA512:
-  metadata.gz: f9a79752c7c06baaefe87e3b4d0200f9dabb6c71ee5b47448b08e3449c84754571dfab78a312000e9cb07674ee03f05405cbe054a12f25ccc371bb24f7ed2312
-  data.tar.gz: 4adc20d83e93b6b1be8c23dd590a023b07e90edc955b3c64c6cf2522b5bba2a0f432039bc55afeca4f16979c23d3dd7f4f7d40f23bd40ff1c9551f2ebf158cdd
+  metadata.gz: 260d54db1f9dd63dbfef02edf2944d02e2c582972b8903815462e438be9cbfb6f5561a7b4a4b64c2cda4af18908c8c7a91958213c06764d1f7ee4d8e2c16dec3
+  data.tar.gz: 003d6ecd547b0215851266f06e823f2f7c5e7a8da35f5b4a8051784855a4d095d088b79f700dcadca8820d561d7a9989a3ee0d9eb7940943c3c5777cc7a4742c

data/.github/workflows/test_and_build_gem.yml

@@ -164,8 +164,7 @@ jobs:
         env:
           DATABASE_URL: postgres://myuser:mypassword@localhost:5432/test_db
         run: |
-          # Using --no-threads to bypass a known Ruby 3.2.0 GC segmentation fault in CI.
-          bundle exec brakeman --no-threads --no-exit-on-warn
+          bundle exec brakeman --no-exit-on-warn
 
   test:
     needs: [setup]

data/.gitignore

@@ -34,9 +34,3 @@ vendor/bundle/
 # Cursor
 .cursor
 dump.rdb
-# Added by code-review-graph
-.code-review-graph/
-.cursorrules
-.claude
-CLAUDE.md
-AGENTS.md

data/Gemfile.lock

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

data/app/controllers/concerns/spree_cm_commissioner/content_cachable.rb

@@ -16,7 +16,6 @@ module SpreeCmCommissioner
       GuestCardClassesController
       SeatLayoutController
       CmsPagesController
-      AdvertisementsController
     ].freeze
 
     # Priority 2: Semi-Static (1 hour)
@@ -26,7 +25,6 @@ module SpreeCmCommissioner
       HomepageDataController
       PopularRoutesController
       RoutePlacesController
-      ShowsController
     ].freeze
 
     # Priority 3: Moderate Freshness (30 minutes)

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/homepage_section_controller.rb

@@ -51,10 +51,7 @@ module Spree
       def permitted_resource_params
         segment_value = helpers.calculate_segment_value(params[:spree_cm_commissioner_homepage_section])
 
-        params
-          .require(:spree_cm_commissioner_homepage_section)
-          .permit(:title, :description, :active, :tenant_id, :preview)
-          .merge(segment: segment_value)
+        params.require(:spree_cm_commissioner_homepage_section).permit(:title, :description, :active, :tenant_id).merge(segment: segment_value)
       end
 
       def load_tenants

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/admin/taxons_controller_decorator.rb

@@ -3,7 +3,6 @@ module Spree
     module TaxonsControllerDecorator
       def self.prepended(base)
         base.before_action :build_assets, only: %i[create update]
-        base.before_action :normalize_sti_taxon_params, only: %i[create update]
       end
 
       # override
@@ -51,24 +50,6 @@ module Spree
         @taxon.build_home_banner(attachment: permitted_resource_params.delete(:home_banner)) if permitted_resource_params[:home_banner]
         @taxon.build_video_banner(attachment: permitted_resource_params.delete(:video_banner)) if permitted_resource_params[:video_banner]
       end
-
-      # Some admin forms submit STI records under their class key
-      # (e.g. :spree_cm_commissioner_show) while Spree update reads :taxon.
-      # Merge both into :taxon so all edited fields are persisted.
-      def normalize_sti_taxon_params
-        return unless @taxon
-
-        sti_key = @taxon.class.model_name.param_key.to_sym
-        sti_params = params[sti_key]
-        return unless sti_params.is_a?(::ActionController::Parameters)
-
-        base_params = params[:taxon]
-        base_hash = base_params.is_a?(::ActionController::Parameters) ? base_params.to_unsafe_h : {}
-        sti_hash = sti_params.to_unsafe_h
-        merged_params = ::ActionController::Parameters.new(base_hash.merge(sti_hash))
-
-        params[:taxon] = merged_params
-      end
     end
   end
 end

data/app/controllers/spree/api/v2/storefront/homepage_sections_controller.rb

@@ -19,7 +19,6 @@ module Spree
             @collection ||= model_class.filter_by_segment(params[:homepage_id] || :general)
                                        .active
                                        .where(tenant_id: nil)
-                                       .where(preview: false)
                                        .order(position: :asc)
           end
         end

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

@@ -16,10 +16,6 @@ module Spree
             set_current_tenant(@tenant)
           end
 
-          def current_vendor
-            @current_vendor ||= @tenant.active_vendor
-          end
-
           def render_serialized_payload(status = 200)
             render json: yield, status: status, content_type: content_type
           end

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

@@ -5,7 +5,6 @@ module Spree
         class HomepageSectionsController < BaseController
           def collection
             @collection ||= scope.filter_by_segment(params[:homepage_id] || :general)
-                                 .where(preview: false)
                                  .active
                                  .order(position: :asc)
           end

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

@@ -21,7 +21,7 @@ module Spree
 
           def scope
             MultiTenant.with(@tenant) do
-              model_class.where(tenant_id: MultiTenant.current_tenant_id, preview: false)
+              model_class.where(tenant_id: MultiTenant.current_tenant_id)
             end
           end
 

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

@@ -31,7 +31,7 @@ module Spree
 
           def scope
             MultiTenant.with(@tenant) do
-              model_class.where(preview: false)
+              model_class
             end
           end
 

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/controllers/spree_cm_commissioner/admin/products_controller_decorator.rb

@@ -6,7 +6,6 @@ module SpreeCmCommissioner
         base.around_action :set_writing_role, only: %i[index]
         base.after_action :set_tenant_after_update, only: %i[update]
         base.before_action :merge_industry_taxons_into_taxons, only: [:update]
-        base.before_action :normalize_sti_product_params, only: %i[create update]
       end
 
       # Override
@@ -52,24 +51,6 @@ module SpreeCmCommissioner
 
       protected
 
-      # Some admin forms submit STI records under their class key while
-      # Spree update reads :product. Merge both into :product so all edited
-      # fields are persisted for STI product subclasses.
-      def normalize_sti_product_params
-        return unless @product
-
-        sti_key = @product.class.model_name.param_key.to_sym
-        sti_params = params[sti_key]
-        return unless sti_params.is_a?(::ActionController::Parameters)
-
-        base_params = params[:product]
-        base_hash = base_params.is_a?(::ActionController::Parameters) ? base_params.to_unsafe_h : {}
-        sti_hash = sti_params.to_unsafe_h
-        merged_params = ::ActionController::Parameters.new(base_hash.merge(sti_hash))
-
-        params[:product] = merged_params
-      end
-
       def merge_industry_taxons_into_taxons
         return if params[:product].blank?
 

data/app/finders/spree_cm_commissioner/events/find_matches.rb

@@ -7,7 +7,6 @@ module SpreeCmCommissioner
       def execute
         scope.joins(:integration_mappings).where(
           kind: :event,
-          preview: false,
           integration_mappings: { status: :active }
         ).distinct
       end

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/helpers/spree_cm_commissioner/admin/homepage_segment_helper.rb

@@ -15,8 +15,6 @@ module SpreeCmCommissioner
           'badge badge-success text-uppercase'
         when :football
           'badge badge-dark text-uppercase'
-        when :show
-          'badge badge-danger text-uppercase'
         else
           'badge'
         end

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

@@ -8,8 +8,7 @@ module SpreeCmCommissioner
       tour: 0b00100,
       accommodation: 0b01000,
       things_to_do: 0b10000,
-      football: 0b100000,
-      show: 0b1000000
+      football: 0b100000
     }.freeze
 
     BIT_SEGMENT.each do |segment, bit_value|

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/option_type_attr_type.rb

@@ -45,8 +45,7 @@ module SpreeCmCommissioner
       'color' => 'color',
       'ticket-type' => 'string',
       'seat-type' => 'string',
-      'intercity-taxi' => 'string',
-      'vote-package' => 'integer'
+      'intercity-taxi' => 'string'
     }.freeze
 
     included do

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

@@ -26,10 +26,8 @@ module SpreeCmCommissioner
 
       state_machine.after_transition to: :canceled, do: :precalculate_conversion
       state_machine.after_transition to: :canceled, do: :restock_inventory!
-      state_machine.after_transition to: :canceled, do: :de_allocate_resources
 
       state_machine.after_transition to: :complete, do: :increment_route_fulfilled_order_count
-      state_machine.after_transition to: :complete, do: :allocate_resources
 
       scope :accepted, -> { where(request_state: 'accepted') }
 
@@ -330,13 +328,5 @@ module SpreeCmCommissioner
       factory = OrderTelegramMessageFactory.new(title: title, order: self)
       TelegramNotificationSenderJob.perform_later(chat_id: chat_id, message: factory.message, parse_mode: factory.parse_mode)
     end
-
-    def allocate_resources
-      SpreeCmCommissioner::VotingCreditAllocationJob.perform_later(order_id: id)
-    end
-
-    def de_allocate_resources
-      SpreeCmCommissioner::VotingCreditDeAllocationJob.perform_later(order_id: id)
-    end
   end
 end

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

@@ -5,7 +5,7 @@ module SpreeCmCommissioner
   module ProductType
     extend ActiveSupport::Concern
 
-    PRODUCT_TYPES = %i[accommodation service ecommerce transit show_episode vote_package advertisement].freeze
+    PRODUCT_TYPES = %i[accommodation service ecommerce transit].freeze
     PERMANENT_STOCK_PRODUCT_TYPES = %w[accommodation service transit].freeze
     PRE_INVENTORY_DAYS = { 'transit' => 90, 'accommodation' => 365, 'service' => 30 }.freeze
 

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

@@ -3,9 +3,7 @@ module SpreeCmCommissioner
     extend ActiveSupport::Concern
 
     included do
-      enum kind: { category: 0, cms: 1, event: 2, occupation: 3, nationality: 4, organization: 5, transit: 6, industry: 7, agency_category: 8,
-                   ads: 9
-}
+      enum kind: { category: 0, cms: 1, event: 2, occupation: 3, nationality: 4, organization: 5, transit: 6, industry: 7, agency_category: 8 }
     end
   end
 end

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/import.rb

@@ -3,7 +3,7 @@ module SpreeCmCommissioner
     extend FriendlyId
 
     enum :status, { :queue => 0, :progress => 1, :done => 2, :failed => 3 }
-    enum :import_type, { :new_order => 0, :existing_order => 1, :aba_payment_reference => 2, :contestant_import => 3 }
+    enum :import_type, { :new_order => 0, :existing_order => 1, :aba_payment_reference => 2 }
     has_one_attached :imported_file
     friendly_id :name, use: :slugged
 

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
@@ -241,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(
@@ -355,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/product_decorator.rb

@@ -14,8 +14,6 @@ module SpreeCmCommissioner
 
       base.delegate :is_open_dated, :is_open_dated?, to: :trip, allow_nil: true
 
-      base.has_many :voting_sessions, class_name: 'SpreeCmCommissioner::VotingSession', foreign_key: :episode_id, dependent: :destroy
-
       base.has_many :variant_kind_option_types, -> { where(kind: :variant).order(:position) },
                     through: :product_option_types, source: :option_type
 
@@ -57,45 +55,8 @@ module SpreeCmCommissioner
 
       base.belongs_to :event, class_name: 'Spree::Taxon', optional: true
 
-      base.has_many :preview_roles, class_name: 'SpreeCmCommissioner::PreviewRole', as: :previewable
       base.has_many :industry_classifications, -> { joins(:taxon).where(spree_taxons: { kind: :industry }) }, class_name: 'Spree::Classification'
       base.has_many :industry_taxons, through: :industry_classifications, source: :taxon
-      base.scope :visible_to, lambda { |user|
-        publicly_available = where(status: :active, preview: false)
-
-        if user
-          # Resolved through associations — avoids hardcoding class names.
-          taxon_type = reflect_on_association(:taxons).klass.polymorphic_name
-
-          # IDs of products that have been explicitly assigned their own PreviewRole.
-          products_with_own_roles = SpreeCmCommissioner::PreviewRole
-                                    .where(previewable_type: polymorphic_name)
-                                    .select(:previewable_id)
-
-          # Path A: product has its own PreviewRole → user must hold that role directly.
-          # Taxon membership is irrelevant for these products.
-          via_product_role = where(status: :active, preview: true)
-                             .where(id: products_with_own_roles)
-                             .where(id: user.preview_roles.where(previewable_type: polymorphic_name).select(:previewable_id))
-
-          # Path B: product has no own PreviewRole → inherit access from its event or taxons.
-          # Accessible if the user holds a preview role for the product's event_id or any
-          # taxon the product belongs to via the classifications join table.
-          allowed_taxon_ids = user.preview_roles.where(previewable_type: taxon_type).select(:previewable_id)
-          preview_no_own = where(status: :active, preview: true).where.not(id: products_with_own_roles)
-
-          via_event_id = preview_no_own.where(event_id: allowed_taxon_ids)
-          via_taxon_join = preview_no_own.joins(:taxons).where(spree_taxons: { id: allowed_taxon_ids })
-
-          publicly_available
-            .or(via_product_role)
-            .or(where(id: via_event_id.select(:id)))
-            .or(where(id: via_taxon_join.select(:id)))
-        else
-          publicly_available
-        end
-      }
-
       base.scope :min_price, lambda { |vendor|
         joins(:prices_including_master)
           .where(vendor_id: vendor.id, product_type: vendor.primary_product_type)
@@ -108,12 +69,6 @@ module SpreeCmCommissioner
           .maximum('spree_prices.price').to_f
       }
       base.scope :subscribable, -> { where(subscribable: 1) }
-      base.scope :advertisements, lambda {
-        advertisement
-          .available
-          .where('spree_products.available_on IS NULL OR spree_products.available_on <= CURRENT_TIMESTAMP')
-          .where('spree_products.discontinue_on IS NULL OR spree_products.discontinue_on > CURRENT_TIMESTAMP')
-      }
 
       base.before_validation :set_event_id
 
@@ -127,11 +82,6 @@ module SpreeCmCommissioner
       base.store_public_metadata :open_dated_validity_days, :integer, default: 90
       base.store_public_metadata :enable_inventory_hold, :boolean, default: false
 
-      base.store_private_metadata :advertise_weight, :integer, default: 1
-      base.store_private_metadata :video_url, :string
-      base.store_private_metadata :skip_after_seconds, :integer, default: 5
-      base.store_private_metadata :duration_seconds, :integer, default: 30
-
       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?
 

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? }