spree_cm_commissioner 2.3.0.pre.pre4 → 2.3.0.pre.pre5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7e7546de16834ee46678d06170bf2c9cd1f20bb41df2df1e6861b8228d5c493f
-  data.tar.gz: c455d1a19ea25846649e857e2f426e914a8e5f326782ed9fbc70f727778f516b
+  metadata.gz: acb7fbb875f336918c3f656d38400951956f6e4ae5c1cefa44ce946912334c17
+  data.tar.gz: 4ec51fea1eb1807a071805fe98d175ad631c009198ee1ea28ffdcfd396176cd2
 SHA512:
-  metadata.gz: 014ac4197bd506955cf8aff2546b459beb51aa01dadf086e162071652ecbdcdc064de97b881ed744ad6da0b0a7e90d22a19ed0e58aca903a9b8c40dc44b92be6
-  data.tar.gz: 83e89ee73435330e30591565e446494efc7c90202f5bf1b4680b6bc9ae8783cf80617cd3bcf9e0503258ed82058b72c7bb4302742ae89791bf6b1a5480aa109b
+  metadata.gz: 9fe75fa019c2603487547366cf6c9d579c33a1d51559eb7b61a243806c3a11d244522fad83f4afb17a724af9ffb03077a65a9ca313b566ced63ae4fe0b0f97e4
+  data.tar.gz: 6c1674cefe433ac9cbe1f934397eb63818d738a7c3ecbc614fd66451ade6dab719a1643ff1314d62ca79794c60829129bddb7bd191a24a69fb09bfa39f10d098

data/Gemfile.lock

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

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

@@ -8,6 +8,37 @@ module Spree
     module V2
       module Storefront
         class SeatLayoutsController < Spree::Api::V2::ResourceController
+          # GET /api/v2/storefront/seat_layouts?
+          # - variant_ids[]=[1,2]
+          # - include_details=true|false (optional, default: true)
+          def index
+            render_serialized_payload do
+              collection_serializer.new(
+                collection,
+                { include: resource_includes, params: serializer_params }
+              ).serializable_hash
+            end
+          end
+
+          # GET /api/v2/storefront/seat_layouts/:id
+          # - include_details=true|false (optional, default: true)
+          def show
+            render_serialized_payload { serialize_resource(resource) }
+          end
+
+          # override
+          def collection
+            scope = if include_details?
+                      SpreeCmCommissioner::SeatLayout.active.includes(:top_level_blocks, seat_sections: :blocks)
+                    else
+                      SpreeCmCommissioner::SeatLayout.active
+                    end
+
+            @collection = scope.joins(:blocks)
+                               .where(blocks: { variant_id: params[:variant_ids] })
+                               .distinct
+          end
+
           # override
           def resource
             SpreeCmCommissioner::SeatLayout.includes(
@@ -29,6 +60,21 @@ module Spree
           def resource_serializer
             SpreeCmCommissioner::V2::Storefront::SeatLayoutSerializer
           end
+
+          # override
+          def collection_serializer
+            SpreeCmCommissioner::V2::Storefront::SeatLayoutSerializer
+          end
+
+          # override
+          def serializer_params
+            super.merge(include_details: include_details?)
+          end
+
+          # default return true
+          def include_details?
+            params.fetch(:include_details, 'true') == 'true'
+          end
         end
       end
     end

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

@@ -13,10 +13,10 @@ module SpreeCmCommissioner
       # For certain large events, admin can disable this manually.
       store_public_metadata :allow_manual_search_for_operator, :boolean, default: true
 
-      # 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.
-      # The ID is automatically updated whenever the seat_layout is saved.
-      store_public_metadata :preload_seat_layout_id, :integer
+      # We store the preload_seat_layout_ids in public_metadata which lets
+      # us check if a seat layout exists without triggering a database query.
+      # The IDs are automatically updated whenever the seat_layout is saved.
+      store_public_metadata :preload_seat_layout_ids, :array, default: []
 
       before_validation :validate_at_least_one_check_in_flow_presence
     end

data/app/models/spree_cm_commissioner/block.rb

@@ -6,6 +6,7 @@ module SpreeCmCommissioner
       driver: 3,
       bathroom: 4,
       text: 5,
+      standing: 6,
       other: 0
     }
 
@@ -27,17 +28,25 @@ module SpreeCmCommissioner
     before_validation :assign_layout_from_section, if: -> { seat_layout.nil? && seat_section.present? }
 
     def label_required?
-      seatable? || %w[other text].include?(block_type.to_s)
+      sellable? || %w[other text].include?(block_type.to_s)
     end
 
     def self.seatable?(block_type)
       block_type.to_s.in?(%w[sleeping_seat seat])
     end
 
+    def self.sellable?(block_type)
+      seatable?(block_type) || block_type.to_s == 'standing'
+    end
+
     def seatable?
       self.class.seatable?(block_type)
     end
 
+    def sellable?
+      self.class.sellable?(block_type)
+    end
+
     def assign_layout_from_section
       self.seat_layout = seat_section.seat_layout
     end

data/app/models/spree_cm_commissioner/guest.rb

@@ -80,6 +80,20 @@ module SpreeCmCommissioner
       ]
     end
 
+    # Only allow holding a block if it represents a seat.
+    # Other block types, like standing, don’t need a hold.
+    # Use the variant's inventory item to track availability instead.
+    def can_hold_block?
+      block.present? && block.seatable?
+    end
+
+    # Only allow holding a block if it represents a seat.
+    # Other block types, like standing, don’t need a hold.
+    # Use the variant's inventory item to track availability instead.
+    def can_reserve_block?
+      block.present? && block.seatable?
+    end
+
     # no validation for each field as we allow user to save data to model partially.
     def allowed_checkout?
       kyc_fields.all? { |field| allowed_checkout_for?(field) }

data/app/models/spree_cm_commissioner/seat_layout.rb

@@ -27,13 +27,19 @@ module SpreeCmCommissioner
 
     private
 
-    # layoutable (taxon, vehicle) must have preload_seat_layout_id attribute,
-    # either as a column or as a key in a JSONB column (e.g., metadata)
     def sync_seat_layout_id_to_layoutable!
-      if destroyed?
-        layoutable.update!(preload_seat_layout_id: nil)
-      else
-        layoutable.update!(preload_seat_layout_id: id)
+      if layoutable.respond_to?(:seat_layouts)
+        # for layoutable (eg. event) that has many seat layouts, we update preload_seat_layout_ids attribute.
+        # either as a column or as a key in a JSONB column (e.g., metadata)
+        layoutable.update!(preload_seat_layout_ids: layoutable.seat_layouts.pluck(:id))
+      elsif layoutable.respond_to?(:seat_layout)
+        # layoutable (eg. taxon, vehicle) must have preload_seat_layout_id attribute,
+        # either as a column or as a key in a JSONB column (e.g., metadata)
+        if destroyed?
+          layoutable.update!(preload_seat_layout_id: nil)
+        else
+          layoutable.update!(preload_seat_layout_id: id)
+        end
       end
     end
   end

data/app/models/spree_cm_commissioner/seats/blocks_holder.rb

@@ -24,6 +24,9 @@ module SpreeCmCommissioner
 
         ActiveRecord::Base.transaction do
           guests_with_blocks.each do |guest|
+            next unless guest.can_hold_block?
+
+            # here i want to set if guest can hold? (it mean should it hold block & follow inventory logics with count instead)
             inventory_items.each do |inventory_item|
               reserved_blocks << hold_specific_block!(inventory_item, guest)
             end

data/app/models/spree_cm_commissioner/seats/blocks_reserver.rb

@@ -19,6 +19,8 @@ module SpreeCmCommissioner
 
         ActiveRecord::Base.transaction do
           guests_with_blocks.each do |guest|
+            next unless guest.can_reserve_block?
+
             inventory_items.each do |inventory_item|
               reserve_specific_block!(inventory_item, guest)
             end

data/app/models/spree_cm_commissioner/taxon_decorator.rb

@@ -21,7 +21,7 @@ module SpreeCmCommissioner
       base.has_many :check_ins, as: :checkinable, class_name: 'SpreeCmCommissioner::CheckIn', dependent: :nullify
       base.has_many :customer_taxons, class_name: 'SpreeCmCommissioner::CustomerTaxon'
 
-      base.has_one :seat_layout, as: :layoutable, class_name: 'SpreeCmCommissioner::SeatLayout', dependent: :destroy
+      base.has_many :seat_layouts, as: :layoutable, class_name: 'SpreeCmCommissioner::SeatLayout', dependent: :destroy
       base.has_one :category_icon, as: :viewable, dependent: :destroy, class_name: 'SpreeCmCommissioner::TaxonCategoryIcon'
 
       base.has_one :web_banner, as: :viewable, dependent: :destroy, class_name: 'SpreeCmCommissioner::TaxonWebBanner'

data/app/serializers/spree/v2/storefront/taxon_serializer_decorator.rb

@@ -18,7 +18,7 @@ module Spree
                           :purchasable_on, :vendor_id, :available_on, :hide_video_banner
 
           # To get full seat layout details, call the seat_layouts API with this ID.
-          base.attribute :seat_layout_id, &:preload_seat_layout_id
+          base.attribute :seat_layout_ids, &:preload_seat_layout_ids
 
           base.attribute :purchasable_on_app do |taxon|
             taxon.purchasable_on == 'app' || taxon.purchasable_on == 'both'

data/app/serializers/spree_cm_commissioner/v2/storefront/block_serializer.rb

@@ -5,6 +5,7 @@ module SpreeCmCommissioner
         attributes :label, :width, :height, :x, :y, :rotation, :block_type, :variant_id
 
         attribute :seatable, &:seatable?
+        attribute :sellable, &:sellable?
       end
     end
   end

data/app/serializers/spree_cm_commissioner/v2/storefront/seat_layout_serializer.rb

@@ -4,8 +4,11 @@ module SpreeCmCommissioner
       class SeatLayoutSerializer < BaseSerializer
         attributes :name, :status, :width, :height
 
-        has_many :seat_sections, serializer: SpreeCmCommissioner::V2::Storefront::SeatSectionSerializer
-        has_many :top_level_blocks, serializer: SpreeCmCommissioner::V2::Storefront::BlockSerializer
+        has_many :seat_sections, serializer: SpreeCmCommissioner::V2::Storefront::SeatSectionSerializer,
+                                 if: proc { |_, params| params && params[:include_details] == true }
+
+        has_many :top_level_blocks, serializer: SpreeCmCommissioner::V2::Storefront::BlockSerializer,
+                                    if: proc { |_, params| params && params[:include_details] == true }
       end
     end
   end

data/config/routes.rb

@@ -674,7 +674,7 @@ Spree::Core::Engine.add_routes do
         post :user_order_transfer, to: 'user_order_transfer#create'
         resources :anonymous_orders, path: 'o', only: %i[show]
 
-        resources :seat_layouts, only: %i[show]
+        resources :seat_layouts, only: %i[show index]
         resources :inventory_items, only: %i[index]
         namespace :transit do
           resources :draft_orders, only: %i[create]

data/lib/spree_cm_commissioner/version.rb

@@ -1,5 +1,5 @@
 module SpreeCmCommissioner
-  VERSION = '2.3.0-pre4'.freeze
+  VERSION = '2.3.0-pre5'.freeze
 
   module_function
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spree_cm_commissioner
 version: !ruby/object:Gem::Version
-  version: 2.3.0.pre.pre4
+  version: 2.3.0.pre.pre5
 platform: ruby
 authors:
 - You