spree_cm_commissioner 2.1.4.pre.pre1 → 2.1.5.pre.pre1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 53fa91ddf6ab2cbf7d0fecc25664116e5e8689f212e0db8a1319d6f37ba58f69
-  data.tar.gz: 3b64416fc5ecb2b21578f1a736cbffe887f264e51433d42fb28c009b72b2259c
+  metadata.gz: d85338c01f8051cd2aff1de565907e13f401f0044b766cca7738056f1089a7d1
+  data.tar.gz: 32b15e97d40db31db7749ee684bb02f3b01bfaa44afc32bc79d1a259960fae50
 SHA512:
-  metadata.gz: '029fc4d782f0b7e61af6675999f36905e345d4406f55098728959c05cebd26b32562c29d63af8b472dc2d98f50657b627e6ba1d8880ef81968d1611f115d8c2b'
-  data.tar.gz: ca652af048f6c0a6d2efd16c9a1b59d14eccbb2d7861a8403ffa5822bb32cdc8501c92277bcd894ca58e568ea04df4fd6367cd36992816e580a28e0459b4dcd0
+  metadata.gz: 3a2d72ee902e1a625713777268aa727d2c92e25cea7a912bd799dd999d4e99f0c7ec5eeb2e5cce49575205d4e01606cf0aa4776e5844074348728ce00eaa4efd
+  data.tar.gz: 5dc5ad4f7c7ff9121503d09fa49e67903b15cc86e21cbd468494fa2119ca758712410c3bb3896a1fa6013bb7e9429b793e6dac1856a676c67caf928950a8b271

data/Gemfile.lock

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

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

@@ -4,6 +4,8 @@ module Spree
       module Tenant
         class BaseController < ::Spree::Api::V2::ResourceController
           include Spree::Api::V2::CollectionOptionsHelpers
+          rescue_from ActiveRecord::RecordInvalid, with: :handle_record_invalid
+          rescue_from ActiveRecord::RecordNotFound, with: :handle_record_not_found
 
           set_current_tenant_through_filter
           before_action :require_tenant
@@ -24,6 +26,16 @@ module Spree
           def scope
             raise 'scope should be implemented in a sub-class'
           end
+
+          private
+
+          def handle_record_invalid(exception)
+            render_error_payload(exception.record.errors, 422)
+          end
+
+          def handle_record_not_found(exception)
+            render_error_payload({ base: [exception.message] }, 404)
+          end
         end
       end
     end

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

@@ -156,7 +156,7 @@ module Spree
           end
 
           def resource_serializer
-            Spree::Api::Dependencies.storefront_cart_serializer.constantize
+            Spree::V2::Tenant::CartSerializer
           end
 
           def create_service

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

@@ -67,7 +67,7 @@ module Spree
           end
 
           def resource_serializer
-            Spree::Api::Dependencies.storefront_cart_serializer.constantize
+            Spree::V2::Tenant::CartSerializer
           end
 
           def next_service

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

@@ -0,0 +1,32 @@
+module Spree
+  module Api
+    module V2
+      module Tenant
+        class DynamicFieldOptionsController < BaseController
+          # override
+          def collection
+            return @collection if defined?(@collection)
+            return SpreeCmCommissioner::DynamicFieldOption.none if params[:dynamic_field_id].blank?
+
+            @collection = SpreeCmCommissioner::DynamicFieldOption
+                          .where(dynamic_field_id: params[:dynamic_field_id])
+                          .active
+                          .order(position: :asc)
+
+            @collection = @collection.where('value LIKE ?', "%#{params[:query]}%") if params[:query].present?
+            @collection
+          end
+
+          # override
+          def collection_serializer
+            Spree::V2::Tenant::DynamicFieldOptionSerializer
+          end
+
+          def model_class
+            SpreeCmCommissioner::DynamicFieldOption
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,38 @@
+module Spree
+  module Api
+    module V2
+      module Tenant
+        class DynamicFieldsController < BaseController
+          before_action :load_product, only: :index
+
+          # override
+          def collection
+            return @collection if defined?(@collection)
+
+            dynamic_fields = @product.dynamic_fields.includes(:dynamic_field_options)
+            if params[:phase].present?
+              requested_phase = params[:phase].to_s
+              if SpreeCmCommissioner::DynamicField.data_fill_stages.key?(requested_phase)
+                dynamic_fields = dynamic_fields.where(data_fill_stage: requested_phase)
+              end
+            end
+
+            @collection = dynamic_fields.order(:position)
+          end
+
+          private
+
+          def load_product
+            return render_error_payload({ error: 'product_id is required' }, 400) if params[:product_id].blank?
+
+            @product ||= Spree::Product.find(params[:product_id])
+          end
+
+          def collection_serializer
+            Spree::V2::Tenant::DynamicFieldSerializer
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -27,23 +27,28 @@ module Spree
           def create
             spree_authorize! :update, spree_current_order, order_token
 
-            resource = scope.new(guest_params)
+            resource = scope.new(guest_params.except(:guest_dynamic_fields_attributes))
 
-            if resource.save
-              render_serialized_payload(201) { serialize_resource(resource) }
-            else
-              render_error_payload(resource.errors, 400)
+            ActiveRecord::Base.transaction do
+              resource.save!
+              run_dynamic_fields(resource)
+              resource.save_and_move_to_next_stage
             end
+
+            render_serialized_payload(201) { serialize_resource(resource) }
           end
 
           def update
             spree_authorize! :update, spree_current_order, order_token
 
-            if resource.update(guest_params)
-              render_serialized_payload { serialize_resource(resource) }
-            else
-              render_error_payload(resource, 400)
+            ActiveRecord::Base.transaction do
+              run_dynamic_fields(resource) if guest_params[:guest_dynamic_fields_attributes]
+              resource.assign_attributes(guest_params.except(:guest_dynamic_fields_attributes))
+              resource.save!
+              resource.save_and_move_to_next_stage
             end
+
+            render_serialized_payload { serialize_resource(resource) }
           end
 
           private
@@ -78,7 +83,21 @@ module Spree
               :address,
               :other_organization,
               :expectation,
-              :upload_later
+              :upload_later,
+              guest_dynamic_fields_attributes: %i[
+                id
+                dynamic_field_id
+                dynamic_field_option_id
+                value
+                _destroy
+              ]
+            )
+          end
+
+          def run_dynamic_fields(resource)
+            SpreeCmCommissioner::GuestDynamicFieldsManager.call(
+              resource: resource,
+              dynamic_fields_attributes: guest_params[:guest_dynamic_fields_attributes]
             )
           end
         end

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

@@ -0,0 +1,41 @@
+module Spree
+  module Api
+    module V2
+      module Tenant
+        class ProductDynamicFieldsController < BaseController
+          before_action :load_product, only: :index
+
+          # override
+          def collection
+            return @collection if defined?(@collection)
+
+            product_dynamic_fields = @product.product_dynamic_fields.includes(dynamic_field: :dynamic_field_options)
+
+            if params[:phase].present?
+              requested_phase = params[:phase].to_s
+              if SpreeCmCommissioner::DynamicField.data_fill_stages.key?(requested_phase)
+                product_dynamic_fields = product_dynamic_fields.joins(:dynamic_field)
+                                                               .where(dynamic_field: { data_fill_stage: requested_phase })
+                                                               .order(:position)
+              end
+            end
+
+            @collection = product_dynamic_fields.order(:section)
+          end
+
+          private
+
+          def load_product
+            return render_error_payload({ error: 'product_id is required' }, 400) if params[:product_id].blank?
+
+            @product ||= Spree::Product.find(params[:product_id])
+          end
+
+          def collection_serializer
+            Spree::V2::Tenant::ProductDynamicFieldSerializer
+          end
+        end
+      end
+    end
+  end
+end

data/app/interactors/spree_cm_commissioner/guest_dynamic_fields_manager.rb

@@ -0,0 +1,33 @@
+module SpreeCmCommissioner
+  class GuestDynamicFieldsManager < BaseInteractor
+    delegate :resource, :dynamic_fields_attributes, to: :context
+
+    def call
+      return if dynamic_fields_attributes.blank?
+
+      dynamic_fields_attributes.each { |attr| upsert_or_destroy_field(attr) }
+    end
+
+    private
+
+    def upsert_or_destroy_field(attr)
+      if attr[:_destroy] == '1' && attr[:id].present?
+        resource.guest_dynamic_fields.find_by(id: attr[:id])&.destroy!
+        return
+      end
+
+      return if attr[:dynamic_field_id].blank?
+
+      field = attr[:id].present? ? resource.guest_dynamic_fields.find(attr[:id]) : resource.guest_dynamic_fields.build
+      field.assign_attributes(attr.except(:id, :_destroy))
+      associate_dynamic_field_option(field)
+      field.save!
+    end
+
+    def associate_dynamic_field_option(field)
+      return unless field.value.to_s.match?(/\A\d+\z/)
+
+      field.dynamic_field_option = field.dynamic_field.dynamic_field_options.find_by(id: field.value.to_i)
+    end
+  end
+end

data/app/interactors/spree_cm_commissioner/sms.rb

@@ -8,8 +8,7 @@ module  SpreeCmCommissioner
         create_sms_log(sms_options)
         create_message(sms_options)
       rescue StandardError => e
-        context.sms_log.error = e.message
-        context.sms_log.save
+        context.sms_log.update(error: e.message, status: :failed)
         context.fail!(message: e.message)
 
         log_error_to_telegram(e.message)
@@ -29,7 +28,7 @@ module  SpreeCmCommissioner
     # sms adapter can decide whether to update additional sms log fields
     def update_sms_log(attributes)
       attributes = attributes.slice(:external_ref)
-      context.sms_log.update(attributes)
+      context.sms_log.update(attributes.merge(status: :sent))
     end
 
     def create_sms_log(sms_options)

data/app/models/spree_cm_commissioner/guest.rb

@@ -40,7 +40,7 @@ module SpreeCmCommissioner
 
     has_many :state_changes, as: :stateful, class_name: 'Spree::StateChange'
     has_many :guest_dynamic_fields, dependent: :destroy, class_name: 'SpreeCmCommissioner::GuestDynamicField'
-    accepts_nested_attributes_for :guest_dynamic_fields, allow_destroy: true
+    accepts_nested_attributes_for :guest_dynamic_fields, allow_destroy: true, reject_if: :reject_guest_dynamic_field?
 
     belongs_to :event, class_name: 'Spree::Taxon'
 
@@ -397,6 +397,11 @@ module SpreeCmCommissioner
 
       required_fields.reject { |field| filled_ids.include?(field.id) }
     end
+
+    def reject_guest_dynamic_field?(attributes)
+      # Reject if dynamic_field_id is blank or if _destroy is true
+      attributes[:dynamic_field_id].blank? || ActiveModel::Type::Boolean.new.cast(attributes[:_destroy])
+    end
   end
 end
 # rubocop:enable Metrics/ClassLength

data/app/models/spree_cm_commissioner/guest_dynamic_field.rb

@@ -46,6 +46,10 @@ module SpreeCmCommissioner
       return if dynamic_field.blank? || dynamic_field_option.blank?
       return if dynamic_field_option.dynamic_field_id == dynamic_field_id
 
+      # Only validate if both dynamic_field and dynamic_field_option are persisted
+      # This prevents validation errors during creation when associations might not be fully loaded
+      return if dynamic_field.new_record? || dynamic_field_option.new_record?
+
       errors.add(:dynamic_field_option, 'must belong to the same dynamic field')
     end
 

data/app/models/spree_cm_commissioner/sms_log.rb

@@ -2,5 +2,7 @@ require_dependency 'spree_cm_commissioner'
 
 module SpreeCmCommissioner
   class SmsLog < ApplicationRecord
+    enum send_type: { otp: 0 }
+    enum status: { queued: 0, sent: 1, failed: 2 }
   end
 end

data/app/models/spree_cm_commissioner/taxon_decorator.rb

@@ -155,6 +155,10 @@ module SpreeCmCommissioner
       ::SpreeCmCommissioner::EventLineItemsDateSyncerJob.perform_later(id)
     end
 
+    def seat_layout
+      nil
+    end
+
     private
 
     def normalize_custom_redirect_url

data/app/serializers/spree/v2/tenant/cart_serializer.rb

@@ -1,7 +1,30 @@
 module Spree
   module V2
     module Tenant
-      class CartSerializer < Spree::V2::Storefront::CartSerializer
+      class CartSerializer < BaseSerializer
+        attributes :number, :item_total, :total, :ship_total, :adjustment_total, :created_at,
+                   :updated_at, :completed_at, :included_tax_total, :additional_tax_total, :display_additional_tax_total,
+                   :display_included_tax_total, :tax_total, :currency, :state, :token, :email,
+                   :display_item_total, :display_ship_total, :display_adjustment_total, :display_tax_total,
+                   :promo_total, :display_promo_total, :item_count, :special_instructions, :display_total,
+                   :pre_tax_item_amount, :display_pre_tax_item_amount, :pre_tax_total, :display_pre_tax_total,
+                   :shipment_state, :payment_state, :public_metadata,
+                   :phone_number, :intel_phone_number, :country_code, :request_state, :channel
+
+        belongs_to :user, serializer: Spree::V2::Tenant::UserSerializer
+        belongs_to :billing_address,
+                   id_method_name: :bill_address_id,
+                   serializer: :address,
+                   record_type: :address
+
+        has_many :promotions, object_method_name: :valid_promotions, id_method_name: :valid_promotion_ids,
+                              serializer: Spree::V2::Storefront::PromotionSerializer
+
+        has_many :variants, serializer: Spree::V2::Tenant::VariantSerializer
+        has_many :products, serializer: Spree::V2::Tenant::ProductSerializer
+        has_many :line_items, serializer: Spree::V2::Tenant::LineItemSerializer
+        has_many :payments, serializer: Spree::V2::Storefront::PaymentSerializer
+        has_many :guests, serializer: Spree::V2::Tenant::GuestSerializer
       end
     end
   end

data/app/serializers/spree/v2/tenant/dynamic_field_option_serializer.rb

@@ -0,0 +1,9 @@
+module Spree
+  module V2
+    module Tenant
+      class DynamicFieldOptionSerializer < BaseSerializer
+        attributes :id, :value, :dynamic_field_id, :status, :position
+      end
+    end
+  end
+end

data/app/serializers/spree/v2/tenant/dynamic_field_serializer.rb

@@ -0,0 +1,11 @@
+module Spree
+  module V2
+    module Tenant
+      class DynamicFieldSerializer < BaseSerializer
+        attributes :id, :label, :data_type, :vendor_id, :multiple_select, :position, :data_fill_stage
+
+        has_many :dynamic_field_options, serializer: Spree::V2::Tenant::DynamicFieldOptionSerializer
+      end
+    end
+  end
+end

data/app/serializers/spree/v2/tenant/guest_dynamic_field_serializer.rb

@@ -0,0 +1,12 @@
+module Spree
+  module V2
+    module Tenant
+      class GuestDynamicFieldSerializer < BaseSerializer
+        attributes :id, :value, :guest_id, :dynamic_field_id, :dynamic_field_option_id, :created_at, :updated_at
+
+        belongs_to :dynamic_field, serializer: Spree::V2::Tenant::DynamicFieldSerializer
+        belongs_to :dynamic_field_option, serializer: Spree::V2::Tenant::DynamicFieldOptionSerializer
+      end
+    end
+  end
+end

data/app/serializers/spree/v2/tenant/guest_serializer.rb

@@ -13,7 +13,9 @@ module Spree
 
         belongs_to :occupation, serializer: Spree::V2::Tenant::TaxonSerializer
         belongs_to :nationality, serializer: Spree::V2::Tenant::TaxonSerializer
+        belongs_to :line_item, serializer: Spree::V2::Tenant::LineItemSerializer
         has_one :id_card, serializer: Spree::V2::Tenant::IdCardSerializer
+        has_many :guest_dynamic_fields, serializer: Spree::V2::Tenant::GuestDynamicFieldSerializer
 
         # allowed_checkout updates frequently
         cache_options store: nil

data/app/serializers/spree/v2/tenant/product_dynamic_field_serializer.rb

@@ -0,0 +1,11 @@
+module Spree
+  module V2
+    module Tenant
+      class ProductDynamicFieldSerializer < BaseSerializer
+        attributes :id, :product_id, :dynamic_field_id, :section
+
+        belongs_to :dynamic_field, serializer: Spree::V2::Tenant::DynamicFieldSerializer
+      end
+    end
+  end
+end

data/app/serializers/spree/v2/tenant/product_serializer.rb

@@ -5,6 +5,9 @@ module Spree
         # This serializer extends the Storefront ProductSerializer
         # to include any additional attributes or relationships needed
         # for the Tenant context.
+
+        # Overrided dynamic fields relationship of the parent serializer(Storefront).
+        has_many :dynamic_fields, serializer: Spree::V2::Tenant::DynamicFieldSerializer
       end
     end
   end

data/config/routes.rb

@@ -507,6 +507,10 @@ Spree::Core::Engine.add_routes do
           resources :id_cards, only: %i[create update destroy]
         end
 
+        resources :dynamic_fields, only: :index
+        resources :product_dynamic_fields, only: :index
+        resources :dynamic_field_options, only: :index
+
         resource :checkout, controller: :checkout, only: %i[update] do
           patch  :next
           patch  :advance

data/db/migrate/20250912084944_add_send_type_to_sms_log.rb

@@ -0,0 +1,9 @@
+class AddSendTypeToSmsLog < ActiveRecord::Migration[7.0]
+  def change
+      add_column :cm_sms_logs, :send_type, :integer, default: 0, null: false, if_not_exists: true
+      add_index :cm_sms_logs, :from_number, if_not_exists: true
+      if column_exists?(:cm_sms_logs, :status)
+        change_column_null :cm_sms_logs, :status, false, 0
+      end
+  end
+end

data/lib/spree_cm_commissioner/version.rb

@@ -1,5 +1,5 @@
 module SpreeCmCommissioner
-  VERSION = '2.1.4-pre1'.freeze
+  VERSION = '2.1.5-pre1'.freeze
 
   module_function
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spree_cm_commissioner
 version: !ruby/object:Gem::Version
-  version: 2.1.4.pre.pre1
+  version: 2.1.5.pre.pre1
 platform: ruby
 authors:
 - You
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-09-12 00:00:00.000000000 Z
+date: 2025-09-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree
@@ -976,6 +976,8 @@ files:
 - app/controllers/spree/api/v2/tenant/checkout_controller.rb
 - app/controllers/spree/api/v2/tenant/cms_pages_controller.rb
 - app/controllers/spree/api/v2/tenant/customer_notifications_controller.rb
+- app/controllers/spree/api/v2/tenant/dynamic_field_options_controller.rb
+- app/controllers/spree/api/v2/tenant/dynamic_fields_controller.rb
 - app/controllers/spree/api/v2/tenant/guests_controller.rb
 - app/controllers/spree/api/v2/tenant/homepage_sections_controller.rb
 - app/controllers/spree/api/v2/tenant/id_cards_controller.rb
@@ -984,6 +986,7 @@ files:
 - app/controllers/spree/api/v2/tenant/order_histories_controller.rb
 - app/controllers/spree/api/v2/tenant/pin_code_checkers_controller.rb
 - app/controllers/spree/api/v2/tenant/pin_code_generators_controller.rb
+- app/controllers/spree/api/v2/tenant/product_dynamic_fields_controller.rb
 - app/controllers/spree/api/v2/tenant/products_controller.rb
 - app/controllers/spree/api/v2/tenant/profile_images_controller.rb
 - app/controllers/spree/api/v2/tenant/reset_passwords_controller.rb
@@ -1142,6 +1145,7 @@ files:
 - app/interactors/spree_cm_commissioner/firebase_email_fetcher_cron_executor.rb
 - app/interactors/spree_cm_commissioner/firebase_id_token_provider.rb
 - app/interactors/spree_cm_commissioner/google_places_fetcher.rb
+- app/interactors/spree_cm_commissioner/guest_dynamic_fields_manager.rb
 - app/interactors/spree_cm_commissioner/guest_id_card_manager.rb
 - app/interactors/spree_cm_commissioner/guest_seat_updater.rb
 - app/interactors/spree_cm_commissioner/host_matcher.rb
@@ -1741,7 +1745,10 @@ files:
 - app/serializers/spree/v2/tenant/cart_serializer.rb
 - app/serializers/spree/v2/tenant/customer_notification_serializer.rb
 - app/serializers/spree/v2/tenant/digital_link_serializer.rb
+- app/serializers/spree/v2/tenant/dynamic_field_option_serializer.rb
+- app/serializers/spree/v2/tenant/dynamic_field_serializer.rb
 - app/serializers/spree/v2/tenant/guest_card_class_serializer.rb
+- app/serializers/spree/v2/tenant/guest_dynamic_field_serializer.rb
 - app/serializers/spree/v2/tenant/guest_serializer.rb
 - app/serializers/spree/v2/tenant/homepage_section_relatable_serializer.rb
 - app/serializers/spree/v2/tenant/homepage_section_serializer.rb
@@ -1754,6 +1761,7 @@ files:
 - app/serializers/spree/v2/tenant/order_serializer.rb
 - app/serializers/spree/v2/tenant/payment_method_group_serializer.rb
 - app/serializers/spree/v2/tenant/payment_method_serializer.rb
+- app/serializers/spree/v2/tenant/product_dynamic_field_serializer.rb
 - app/serializers/spree/v2/tenant/product_serializer.rb
 - app/serializers/spree/v2/tenant/reset_password_serializer.rb
 - app/serializers/spree/v2/tenant/role_serializer.rb
@@ -2743,6 +2751,7 @@ files:
 - db/migrate/20250911100835_add_variant_id_to_cm_blocks.rb
 - db/migrate/20250911174607_add_metadata_columns_to_cm_vehicles.rb
 - db/migrate/20250911174653_add_metadata_columns_to_cm_trips.rb
+- db/migrate/20250912084944_add_send_type_to_sms_log.rb
 - docker-compose.yml
 - docs/option_types/attr_types.md
 - docs/private_key.pem