spree_cm_commissioner 1.8.6 → 1.8.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8cd5a0a2e3fc958b2a2e6bf0fd4dcd5773be9b512038dd183613b9586162c725
-  data.tar.gz: 3d37e405da973e87aa03bd9177dd3474d720106e8f44ac9524156623aa586d4a
+  metadata.gz: 687ff4cf4e03ffe157a851660a8d0a82f90463ab2f7e2050e8d25e0792377989
+  data.tar.gz: 78794baf30a3bf2bcf033cc1ca921c422c273d2a3eb50ce40481c7424cd07950
 SHA512:
-  metadata.gz: f16265bfb68e1ed87973e1b5afc987d99362e627f7a0072f56e10445712c2516b841a3a8285ee8f20961cbcf3eddfcfbacbd7829156b834875126b16834684fe
-  data.tar.gz: 7bde1b69e06d5e56ab2b505784f44e75ecf930ca381f19a758c851dfc7da721b9e24895da8b51553ff0ac613f5de1b0a6a24f27066917e768728bfdc062e27f1
+  metadata.gz: 7b1884005a4e95691a3f71c6d2a128c821cd32fb41a29bb7af72f4be5a2a3734a8c81137a29d4a3a258832118c446ce8b93656122ebeeb8dc8e985edfca400ad
+  data.tar.gz: 5c5c8d8dd8f1a75402b034f9e158554cf00aa3e1d7a25f16c983c71a3867b4d4312f882e0863a30b463df82e205a59daa19b432f84e67a34c6adcab484e38786

data/Gemfile.lock

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

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

@@ -0,0 +1,28 @@
+module SpreeCmCommissioner
+  module OrderConcern
+    # override
+    def spree_current_order
+      @spree_current_order ||= find_spree_current_order
+      return nil if @spree_current_order.blank?
+
+      # Spree doesn't validate this by default (might be a bug).
+      # Temporary fix to ensure the order's user ID matches the logged-in user.
+      if @spree_current_order.user_id.present? && spree_current_user.present? && @spree_current_order.user_id != spree_current_user.id
+        raise CanCan::AccessDenied
+      end
+
+      @spree_current_order
+    end
+
+    # override
+    def find_spree_current_order
+      SpreeCmCommissioner::Orders::Find.new.execute(
+        store: current_store,
+        user: spree_current_user,
+        currency: current_currency,
+        token: order_token,
+        state: params[:state]
+      )
+    end
+  end
+end

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

@@ -0,0 +1,58 @@
+module Spree
+  module Api
+    module V2
+      module Storefront
+        class OrderHistoriesController < ::Spree::Api::V2::ResourceController
+          # GET /api/v2/storefront/order_histories
+          # For user - no params needed
+          # For guest - pass: { "order_tokens[]": [1, 2, 3] }
+          def index
+            render_serialized_payload do
+              serialize_collection(paginated_collection)
+            end
+          end
+
+          # PATCH /api/v2/storefront/order_histories/:id/archive
+          def archive
+            order_token = params[:id]
+            order = Spree::Order.not_archived.find_by!(token: order_token)
+
+            spree_authorize! :update, order, order_token
+            raise CanCan::AccessDenied unless order.payment?
+
+            if order.archive
+              render_serialized_payload { serialize_resource(order) }
+            else
+              render_error_payload(order.errors.full_messages.to_sentence)
+            end
+          end
+
+          private
+
+          def collection
+            if spree_current_user.present?
+              spree_current_user.orders.payment.not_archived
+            else
+              order_tokens = Array(params[:order_tokens])
+              return Spree::Order.none if order_tokens.empty?
+
+              Spree::Order.payment.not_archived.without_user.where(token: order_tokens)
+            end
+          end
+
+          def collection_serializer
+            Spree::V2::Storefront::CartSerializer
+          end
+
+          def resource_serializer
+            Spree::V2::Storefront::CartSerializer
+          end
+
+          def model_class
+            Spree::Order
+          end
+        end
+      end
+    end
+  end
+end

data/app/controllers/spree/api/v2/storefront/queue_cart/line_items_controller.rb

@@ -6,7 +6,6 @@ module Spree
           class LineItemsController < CartController
             before_action :ensure_order, only: :create
             before_action :load_variant, only: :create
-            before_action :ensure_cart_exist, only: :create
 
             # override
             def create

data/app/controllers/spree_cm_commissioner/api/v2/storefront/cart_controller_decorator.rb

@@ -4,42 +4,17 @@ module SpreeCmCommissioner
       module Storefront
         module CartControllerDecorator
           def self.prepended(base)
-            base.before_action :ensure_cart_exist, only: :add_item
+            base.include SpreeCmCommissioner::OrderConcern
           end
 
-          # one usecase where this neccessary is when order.state is 'complete', but complated_at & payment_state is null,
-          # this cause app to be stuck at book page because cart is not considered cart or completed.
-          #
-          # solution is to restart it back to cart.
+          # Restart the checkout flow to bring the order back to the cart view
           def restart_checkout_flow
             spree_authorize! :update, spree_current_order, order_token
 
-            if spree_current_order.completed_at.present?
-              render_error_payload('Cart already completed!')
-            else
-              spree_current_order.restart_checkout_flow
-              spree_current_order.update_with_updater!
+            spree_current_order.restart_checkout_flow
+            spree_current_order.update_with_updater!
 
-              render_serialized_payload { serialized_current_order }
-            end
-          end
-
-          # we required only user can create cart
-          # in case there is no cart, and user try to add item to cart,
-          # it should create one instead of raising error.
-          def ensure_cart_exist
-            spree_authorize! :create, Spree::Order
-            return if spree_current_order.present?
-
-            create_cart_params = {
-              user: spree_current_user,
-              store: current_store,
-              currency: current_currency,
-              public_metadata: add_item_params[:public_metadata],
-              private_metadata: add_item_params[:private_metadata]
-            }
-
-            @spree_current_order ||= create_service.call(create_cart_params).value
+            render_serialized_payload { serialized_current_order }
           end
         end
       end

data/app/controllers/spree_cm_commissioner/api/v2/storefront/checkout_controller_decorator.rb

@@ -4,9 +4,7 @@ module SpreeCmCommissioner
       module Storefront
         module CheckoutControllerDecorator
           def self.prepended(base)
-            # spree_vpago gem update payment state in #payment_redirect
-            # GET /payment_redirect
-            base.around_action :set_writing_role, only: %i[payment_redirect]
+            base.include SpreeCmCommissioner::OrderConcern
           end
         end
       end

data/app/finders/spree_cm_commissioner/orders/find.rb

@@ -0,0 +1,34 @@
+# Custom replacement for Spree::Orders::FindCurrent with the following improvements:
+# - Supports filtering by state (e.g., to retrieve only 'cart' or 'address' orders for the page cart)
+# - Allows retrieving the order in any state (not just incomplete), which solves the issue of 404s after payment
+module SpreeCmCommissioner
+  module Orders
+    class Find
+      def execute(store:, user:, currency:, token: nil, state: nil)
+        params = { store_id: store.id, currency: currency }
+        params[:state] = state if state.present?
+
+        return find_by_token(params, token) if token.present?
+        return find_by_user(params, user) if user.present?
+
+        nil
+      end
+
+      def find_by_token(params, token)
+        params[:token] = token
+        scope.find_by(params)
+      end
+
+      def find_by_user(params, user)
+        params[:user_id] = user.id
+        scope.order(created_at: :desc).find_by(params)
+      end
+
+      private
+
+      def scope
+        Spree::Order.not_archived.not_canceled
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/order_decorator.rb

@@ -7,6 +7,10 @@ module SpreeCmCommissioner
       base.scope :subscription, -> { where.not(subscription_id: nil) }
       base.scope :paid, -> { where(payment_state: :paid) }
       base.scope :complete_or_canceled, -> { complete.or(where(state: 'canceled')) }
+      base.scope :payment, -> { incomplete.where(state: 'payment') }
+      base.scope :archived, -> { where.not(archived_at: nil) }
+      base.scope :not_archived, -> { where(archived_at: nil) }
+      base.scope :without_user, -> { where(user_id: nil) }
 
       base.scope :filter_by_match_user_contact, lambda { |user|
         complete.where(
@@ -101,6 +105,10 @@ module SpreeCmCommissioner
       self.class.unscoped.where(id: self).update_all(changes) # rubocop:disable Rails/SkipsModelValidations
     end
 
+    def archive
+      update(archived_at: Time.current)
+    end
+
     # overrided
     # avoid raise error when source_id is nil.
     # https://github.com/channainfo/commissioner/pull/843

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

@@ -3,8 +3,15 @@ module Spree
     module Storefront
       module CartSerializerDecorator
         def self.prepended(base)
-          base.attributes :phone_number, :intel_phone_number, :country_code, :request_state, :qr_data,
+          base.attributes :phone_number, :intel_phone_number, :country_code, :request_state,
                           :channel
+
+          base.attribute :qr_data do |order|
+            order.qr_data if order.completed?
+          end
+
+          # override to return all payments instead of only valid_payments
+          base.has_many :payments
         end
       end
     end

data/config/routes.rb

@@ -518,6 +518,13 @@ Spree::Core::Engine.add_routes do
 
         resources :wished_items
         resources :user_promotion
+
+        resources :order_histories, only: %i[index] do
+          member do
+            patch :archive
+          end
+        end
+
         resources :order_promotions
         resources :guest_card_classes
 

data/db/migrate/20250403040036_install_blazer.rb

@@ -0,0 +1,47 @@
+class InstallBlazer < ActiveRecord::Migration[7.0]
+  def change
+    create_table :blazer_queries, if_not_exists: true do |t|
+      t.references :creator
+      t.string :name
+      t.text :description
+      t.text :statement
+      t.string :data_source
+      t.string :status
+      t.timestamps null: false
+    end
+
+    create_table :blazer_audits, if_not_exists: true do |t|
+      t.references :user
+      t.references :query
+      t.text :statement
+      t.string :data_source
+      t.datetime :created_at
+    end
+
+    create_table :blazer_dashboards, if_not_exists: true do |t|
+      t.references :creator
+      t.string :name
+      t.timestamps null: false
+    end
+
+    create_table :blazer_dashboard_queries, if_not_exists: true do |t|
+      t.references :dashboard
+      t.references :query
+      t.integer :position
+      t.timestamps null: false
+    end
+
+    create_table :blazer_checks, if_not_exists: true do |t|
+      t.references :creator
+      t.references :query
+      t.string :state
+      t.string :schedule
+      t.text :emails
+      t.text :slack_channels
+      t.string :check_type
+      t.text :message
+      t.datetime :last_run_at
+      t.timestamps null: false
+    end
+  end
+end

data/db/migrate/{20250327094626_create_spree_cm_commissioner_taxon_blazer_query.rb → 20250403083001_create_spree_cm_commissioner_taxon_blazer_query.rb}

@@ -1,6 +1,6 @@
 class CreateSpreeCmCommissionerTaxonBlazerQuery < ActiveRecord::Migration[7.0]
   def change
-    create_table :cm_taxon_blazer_queries,if_not_exists: true  do |t|
+    create_table :cm_taxon_blazer_queries, if_not_exists: true  do |t|
       t.references :taxon, foreign_key: { to_table: :spree_taxons }, null: false, index: true
       t.references :blazer_query,  foreign_key: { to_table: :blazer_queries }, null: false, index: true
       t.timestamps

data/db/migrate/20250404050117_add_archived_at_to_spree_orders.rb

@@ -0,0 +1,5 @@
+class AddArchivedAtToSpreeOrders < ActiveRecord::Migration[7.0]
+  def change
+    add_column :spree_orders, :archived_at, :datetime, if_not_exists: true
+  end
+end

data/lib/spree_cm_commissioner/version.rb

@@ -1,5 +1,5 @@
 module SpreeCmCommissioner
-  VERSION = '1.8.6'.freeze
+  VERSION = '1.8.8'.freeze
 
   module_function
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spree_cm_commissioner
 version: !ruby/object:Gem::Version
-  version: 1.8.6
+  version: 1.8.8
 platform: ruby
 authors:
 - You
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-04-01 00:00:00.000000000 Z
+date: 2025-04-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree
@@ -665,6 +665,7 @@ files:
 - app/controllers/concerns/spree_cm_commissioner/content_cachable.rb
 - app/controllers/concerns/spree_cm_commissioner/events/role_authorization.rb
 - app/controllers/concerns/spree_cm_commissioner/exception_notificable.rb
+- app/controllers/concerns/spree_cm_commissioner/order_concern.rb
 - app/controllers/concerns/spree_cm_commissioner/transit/taxon_bitwise.rb
 - app/controllers/concerns/spree_cm_commissioner/waiting_room_authorization.rb
 - app/controllers/spree/admin/account_deletions_controller.rb
@@ -786,6 +787,7 @@ files:
 - app/controllers/spree/api/v2/storefront/line_items_controller.rb
 - app/controllers/spree/api/v2/storefront/nearby_places_controller.rb
 - app/controllers/spree/api/v2/storefront/notifications_controller.rb
+- app/controllers/spree/api/v2/storefront/order_histories_controller.rb
 - app/controllers/spree/api/v2/storefront/order_products_taxons_controller.rb
 - app/controllers/spree/api/v2/storefront/order_promotions_controller.rb
 - app/controllers/spree/api/v2/storefront/order_qrs_controller.rb
@@ -920,6 +922,7 @@ files:
 - app/factory/spree_cm_commissioner/order_telegram_message_factory.rb
 - app/factory/spree_cm_commissioner/telegram_message_factory.rb
 - app/finders/spree_cm_commissioner/line_items/find_by_variant_decorator.rb
+- app/finders/spree_cm_commissioner/orders/find.rb
 - app/finders/spree_cm_commissioner/orders/find_by_state.rb
 - app/finders/spree_cm_commissioner/payment_methods/group_by_bank.rb
 - app/helpers/spree/admin/base_helper_decorator.rb
@@ -2296,10 +2299,12 @@ files:
 - db/migrate/20250314013434_add_available_on_to_spree_taxons.rb
 - db/migrate/20250321041406_create_trip_connections.rb
 - db/migrate/20250327074327_add_hide_video_banner_to_spree_taxon.rb
-- db/migrate/20250327094626_create_spree_cm_commissioner_taxon_blazer_query.rb
 - db/migrate/20250328072717_add_description_to_spree_roles.rb
 - db/migrate/20250328072841_add_vendor_id_to_spree_roles.rb
 - db/migrate/20250328072947_remove_unique_constraint_from_name_in_spree_roles.rb
+- db/migrate/20250403040036_install_blazer.rb
+- db/migrate/20250403083001_create_spree_cm_commissioner_taxon_blazer_query.rb
+- db/migrate/20250404050117_add_archived_at_to_spree_orders.rb
 - docker-compose.yml
 - docs/option_types/attr_types.md
 - docs/private_key.pem