spree_cm_commissioner 2.5.13.pre.patch1 → 2.5.13.pre.patch3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2ca95409fb32a7b43530a6b61ffdc869564893d7c9a8abcf68e72f9db8c9362d
-  data.tar.gz: 93e7de99a30c3f58f899426132837c134ca9a6a725e799a90443b9699b91213f
+  metadata.gz: 93df32aae4991bb798b86f21c7ddfc3b71e13fc6c5a71ce4315a50eaa70ea1e6
+  data.tar.gz: 37574b773bbfc43008b28d390de0a032e9ab5db9d08d0307b10de20075fabcfe
 SHA512:
-  metadata.gz: efd4bc5698835f6eaebfa42e71e213c6cb9ace7d5a6eb5219b3143e49993ed8679145da7c0732ee16176b8428509973f6bd1ead97dcbf0ede168af042e67ae0c
-  data.tar.gz: 2ea123009aa6a3ecb2208117d35e27a8ece4010e78142b619f8c71323a6741b453bb37e8f2148f23fca7320d933923e81be037ed20baf616b43b294c7be52739
+  metadata.gz: 17d233c16e34ee73284559a5536ba042c955a1a140d45cc172645d91cf4bbda83a7025a99886a5bd61ead85d16696c75af8c3db015aba9e5b6b2bc12ca25e489
+  data.tar.gz: 64ffb26a7ad38915f38d655878eba9521b4791cdbab6d6a51d558d81a4f3a3a112fad0063ad413f858b1d9997e750f3b62abc1deccf3da704d429560ef7ced69

data/Gemfile.lock

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

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

@@ -64,7 +64,7 @@ module SpreeCmCommissioner
       elsif HIGH_FRESHNESS_CONTROLLERS.include?(controller_name)
         ENV.fetch('CACHE_REALTIME_MAX_AGE', 5.minutes.to_i).to_i
       else
-        ENV.fetch('CACHE_DEFAULT_MAX_AGE', 5.minutes.to_i).to_i
+        ENV.fetch('CACHE_DEFAULT_MAX_AGE', 0.minutes.to_i).to_i
       end
     end
 

data/app/controllers/spree/api/v2/operator/guest_json_gzips_controller.rb

@@ -8,6 +8,15 @@ module Spree
           before_action :require_spree_current_user
           before_action :require_event_user
 
+          # Override index to disable server-side caching.
+          # We always want to show the latest exports and their download status.
+          # The app will cache files on its side to avoid unnecessary server requests once files are ready.
+          def index
+            render_serialized_payload do
+              serialize_collection(paginated_collection)
+            end
+          end
+
           # POST /api/v2/operator/guest_json_gzips
           # - taxon_id=1
           # - resource_includes[]=[check_ins,check_ins.check_in_by]

data/app/helpers/spree/base_helper_decorator.rb

@@ -21,7 +21,7 @@ module Spree
     end
 
     def custom_product_line_item_url(line_item)
-      main_app.url_for("/a/#{line_item.qr_data}")
+      main_app.url_for("/a/#{line_item.internal_qr_data}")
     end
   end
 end

data/app/models/spree_cm_commissioner/line_item_decorator.rb

@@ -203,6 +203,10 @@ module SpreeCmCommissioner
       return external_qr_data if external_qr_data.present?
       return nil if order.nil?
 
+      internal_qr_data
+    end
+
+    def internal_qr_data
       "#{order.number}-#{order.token}-L#{id}"
     end
 

data/app/services/spree_cm_commissioner/integrations/base/sync_result.rb

@@ -106,7 +106,7 @@ module SpreeCmCommissioner::Integrations::Base
       #
       # @param record [Object] The record to save (defaults to the tracked record)
       def save_if_changed!(record = @record)
-        return unless record.new_record? || record.changed?
+        return unless record.new_record? || record.changed? || default_price_changed?(record)
 
         record.save!
 
@@ -117,6 +117,28 @@ module SpreeCmCommissioner::Integrations::Base
           @sync_result.increment_metric(@type, :updated)
         end
       end
+
+      private
+
+      # Check if record's default_price has changed (for Spree variants/products)
+      def default_price_changed?(record)
+        variant_price_changed?(record) || product_price_changed?(record)
+      end
+
+      def variant_price_changed?(record)
+        price = record.try(:default_price)
+        price_changed?(price)
+      end
+
+      def product_price_changed?(record)
+        master = record.try(:master)
+        price = master&.default_price
+        price_changed?(price)
+      end
+
+      def price_changed?(price)
+        price.present? && (price.changed? || price.new_record?)
+      end
     end
 
     # Increment a metric counter

data/app/services/spree_cm_commissioner/integrations/stadium_x_v1/polling/sync_matches.rb

@@ -36,8 +36,8 @@ class SpreeCmCommissioner::Integrations::StadiumXV1
 
         match_mapping = Spree::Taxon.find_or_initialize_integration_mapping(integration_id: @integration.id, external_id: external_match._id)
         match_taxon = match_mapping.internal
-        match_name = "#{external_match.home_name} vs #{external_match.away_name}"
-        match_permalink = generate_match_permalink(match_name)
+        match_name = "#{external_match.home_name} vs #{external_match.away_name} - #{external_match.date} #{external_match.time}"
+        match_permalink = match_taxon.persisted? ? match_taxon.permalink : generate_match_permalink(match_name)
 
         # Track match sync and create event taxon directly under events/
         @sync_result.track(:match, match_taxon) do |tracker|

data/app/services/spree_cm_commissioner/integrations/stadium_x_v1/polling/sync_zones.rb

@@ -13,12 +13,14 @@ class SpreeCmCommissioner::Integrations::StadiumXV1
         match_mapping = Spree::Taxon.find_integration_mapping(integration_id: @integration.id, external_id: external_match_id)
         raise SpreeCmCommissioner::Integrations::SyncError, "Match not found: #{external_match_id}" if match_mapping.blank?
 
-        synced_zone_mappings = sync_zones!(match_mapping)
-        cleanup_stale_mappings!(match_mapping, synced_zone_mappings)
+        match_section = match_mapping.internal.children.first_or_create! { |s| s.name = 'Zones' }
+
+        synced_zone_mappings = sync_zones!(match_mapping, match_section)
+        cleanup_stale_mappings!(match_mapping, synced_zone_mappings, match_section)
       end
 
       # Sync zones for a specific club and match
-      def sync_zones!(match_mapping)
+      def sync_zones!(match_mapping, match_section)
         external_club_id = match_mapping.external_payload&.dig('club_id')
         raise SpreeCmCommissioner::Integrations::SyncError, 'club_id is required' if external_club_id.blank?
 
@@ -35,9 +37,8 @@ class SpreeCmCommissioner::Integrations::StadiumXV1
           zone.set_match_id(match_mapping.external_id)
         end
 
-        taxon_match_section = match_mapping.internal.children.first_or_create! { |s| s.name = 'Zones' }
         synced_zone_mappings = external_zones.map do |external_zone|
-          sync_zone!(external_zone, match_mapping.internal, taxon_match_section)
+          sync_zone!(external_zone, match_mapping.internal, match_section)
         end
 
         CmAppLogger.log(
@@ -50,7 +51,7 @@ class SpreeCmCommissioner::Integrations::StadiumXV1
 
       private
 
-      def sync_zone!(external_zone, internal_match, taxon_match_section)
+      def sync_zone!(external_zone, internal_match, match_section)
         Spree::Product.transaction do
           product_mapping = Spree::Product.find_or_initialize_integration_mapping(integration_id: @integration.id, external_id: external_zone._id)
           product = product_mapping.internal
@@ -66,7 +67,7 @@ class SpreeCmCommissioner::Integrations::StadiumXV1
               vendor: @integration.vendor,
               product_type: :ecommerce,
               event: internal_match,
-              taxons: [taxon_match_section],
+              taxons: [match_section],
               name: external_zone.title.strip,
               price: external_zone.online_price,
               available_on: external_zone.start_datetime,
@@ -78,6 +79,9 @@ class SpreeCmCommissioner::Integrations::StadiumXV1
             tracker.save_if_changed!(product)
           end
 
+          classification = product.classifications.find_by(taxon_id: match_section.id)
+          classification&.update!(visible: true)
+
           product_mapping.mark_as_active!(external_payload: external_zone.to_h)
 
           ensure_variant_with_zone_option_value!(product, external_zone, zone_option_type, color_option_type)
@@ -190,7 +194,7 @@ class SpreeCmCommissioner::Integrations::StadiumXV1
       # Archive stale zone mappings that are no longer in the external API
       # Finds all active zone mappings for this integration and archives any
       # that weren't included in the latest API response.
-      def cleanup_stale_mappings!(match_mapping, synced_zone_mappings)
+      def cleanup_stale_mappings!(match_mapping, synced_zone_mappings, match_section)
         synced_external_zone_ids = synced_zone_mappings.map(&:external_id)
 
         # Find all active zone mappings for this integration and match
@@ -207,6 +211,12 @@ class SpreeCmCommissioner::Integrations::StadiumXV1
           mapping.internal.discontinue!
 
           mapping.mark_as_archived!
+          classification = Spree::Classification.find_or_initialize_by(
+            product_id: mapping.internal_id,
+            taxon_id: match_section.id
+          )
+          classification.update!(visible: false)
+
           @sync_result.increment_metric(:zone, :discontinued)
         end
       end

data/lib/spree_cm_commissioner/version.rb

@@ -1,5 +1,5 @@
 module SpreeCmCommissioner
-  VERSION = '2.5.13-patch1'.freeze
+  VERSION = '2.5.13-patch3'.freeze
 
   module_function
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spree_cm_commissioner
 version: !ruby/object:Gem::Version
-  version: 2.5.13.pre.patch1
+  version: 2.5.13.pre.patch3
 platform: ruby
 authors:
 - You
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-02-24 00:00:00.000000000 Z
+date: 2026-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree