spree_vpago 2.2.2.pre.pre38 → 2.2.2.pre.pre40

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6f928f176e0a9faa7c833f8a84357ef1b47b54e90317a746572e82990ec2fb70
-  data.tar.gz: 2575d2d9ed15e63f58efa5ab80efe48a82f21cadef0b42e0277d59b64adfe98e
+  metadata.gz: ba1b0a5520634444abf62c6b2bd65ed6660be8bbad0f53ae16a3eb359cabe260
+  data.tar.gz: 56fc2911ed815ec53799f6d0a6456d64798e513de758830e34c79af6b871df16
 SHA512:
-  metadata.gz: 1c783db7c54b8692924880f17c19e589e7b0ed097031fd93778e701364e5520c341e02ae1c2bf957ae4c014450d9411bfee35a14e73c27327ce6418e2d903c42
-  data.tar.gz: aa2a7c19225b123ef6e6c1dd1531cef9402fe67c7fbf04da3aa035ab54f46a5214501fbcbb4bf2393f84a602ba813fdf4cbe60910a1aa0937f126c9619a58572
+  metadata.gz: b29505ce698611342dc6eb6ab1a787cb8ec7a427244a25db1cd476b17431b0425603fd6b8b8cb30ba5262155e5a701d910f5a4bb182815bc64b6daf151fa5f6f
+  data.tar.gz: fd7291a06bb337ea5be06da2d0ef7cd2bd0affec96c73057d9d34eceda56b44ec7954347bf1b2858dcd3aa207bca9b5a72db66e101e62c6179b7e1b17038ea91

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    spree_vpago (2.2.2.pre.pre38)
+    spree_vpago (2.2.2.pre.pre40)
       faraday
       google-cloud-firestore
       spree_api (>= 4.5)

data/app/controllers/spree/vpago_payments_controller.rb

@@ -16,6 +16,8 @@ module Spree
       return redirect_to @payment.processing_url, allow_other_host: true unless @payment.checkout?
 
       @order = @payment.order
+
+      Rails.logger.info("[Vpago][#{@payment.number}] Showing checkout page for order #{@order.number}")
     end
 
     # GET
@@ -25,7 +27,7 @@ module Spree
 
       @order = @payment.order
 
-      Rails.logger.info("[Vpago] Showing processing page for payment #{@payment.number} for order #{@order.number}")
+      Rails.logger.info("[Vpago][#{@payment.number}] Showing processing page for order #{@order.number}")
     end
 
     # GET
@@ -36,7 +38,7 @@ module Spree
       @order = @payment.order
       raise CanCan::AccessDenied unless @order.completed?
 
-      Rails.logger.info("[Vpago] Showing success page for payment #{@payment.number} for order #{@order.number}")
+      Rails.logger.info("[Vpago][#{@payment.number}] Showing success page for order #{@order.number}")
     end
 
     # GET
@@ -48,9 +50,9 @@ module Spree
       return render json: { status: :failed }, status: :ok if @payment.failed?
 
       if @payment.payment_method.support_check_transaction_api?
-        Rails.logger.info("[Vpago] Checking transaction for payment #{@payment.number}")
+        Rails.logger.info("[Vpago][#{@payment.number}] Checking transaction for payment #{@payment.number}")
         checker = @payment.payment_method.check_transaction(@payment)
-        Rails.logger.info("[Vpago] Check transaction result for payment #{@payment.number} with success: #{checker.success?}, failed: #{checker.try(:failed?)}")
+        Rails.logger.info("[Vpago][#{@payment.number}] Check transaction result for payment #{@payment.number} with success: #{checker.success?}, failed: #{checker.try(:failed?)}")
 
         if checker.success?
           render json: { status: :success }, status: :ok
@@ -60,43 +62,46 @@ module Spree
           render json: { status: :pending }, status: :ok
         end
       else
+        Rails.logger.info("[Vpago][#{@payment.number}] Payment method does not support check transaction API")
         render json: { status: :pending }, status: :ok
       end
     end
 
     # POST
     def process_payment
-      Rails.logger.info("[Vpago] Received payment notification: method=#{request.method}, params=#{params.to_unsafe_h}")
       return render json: { status: :ok }, status: :ok if request.method != 'POST'
 
       return_params = sanitize_return_params
       @payment = Vpago::PaymentFinder.new(return_params).find_and_verify
 
+      # log process payment requested
+      Rails.logger.info("[Vpago][#{@payment&.number}] Process payment requested with params: #{return_params}")
+
       if @payment.nil?
         Rails.logger.error("[Vpago] Payment not found for params: #{return_params}")
         return render_not_found
       end
 
-      Rails.logger.info("[Vpago] Payment found: #{@payment&.number}, order: #{@payment&.order&.number}")
+      Rails.logger.info("[Vpago][#{@payment.number}] Payment found: #{@payment&.number}, order: #{@payment&.order&.number}")
 
       # for ABA reviewing mode, we can disable pushback from bank, and only process it from our app UI instead.
       # This will give ABA team to know that we don't rely on just pushback and have fallback to process payment.
       if @payment.payment_method.type_payway_v2? && @payment.payment_method.reviewing_mode? && request_from_external_server?
-        Rails.logger.info("[Vpago] Received payment notification from bank in reviewing mode, skipping processing: #{params}")
+        Rails.logger.info("[Vpago][#{@payment.number}] Received payment notification from bank in reviewing mode, skipping processing")
         return render json: { status: :ok }, status: :ok
       end
 
       unless @payment.order.paid?
-        Rails.logger.info("[Vpago] Enqueuing payment processor job for payment #{@payment.number}: #{params}")
+        Rails.logger.info("[Vpago][#{@payment.number}] Enqueuing payment processor job for payment #{@payment.number}")
         Vpago::PaymentProcessorJob.perform_later(
           payment_number: @payment.number
         )
       end
 
-      Rails.logger.info("[Vpago] Successfully enqueued payment processor job for payment #{@payment.number}: #{params}")
+      Rails.logger.info("[Vpago][#{@payment.number}] Successfully enqueued payment processor job for payment #{@payment.number}")
       render json: { status: :ok }, status: :ok
     rescue StandardError => e
-      Rails.logger.error("[Vpago] Failed to enqueue payment processor job: #{params} #{e.message}")
+      Rails.logger.error("[Vpago][#{@payment.number}] Failed to enqueue payment processor job for payment #{@payment.number}: #{e.message}")
       render json: { status: :internal_server_error, message: 'Failed to enqueue payment processor job' }, status: :internal_server_error
     end
 
@@ -108,11 +113,11 @@ module Spree
       return render_not_found unless @payment
 
       Vpago::PaymentProcessorJob.perform_later(payment_number: @payment.number) unless @payment.order.paid?
-      Rails.logger.info("[Vpago] Successfully enqueued payment processor job for payment #{@payment.number}: #{params}")
+      Rails.logger.info("[Vpago][#{@payment.number}] Successfully enqueued payment processor job for payment #{@payment.number}")
 
       render json: { status: { code: '000001', message: 'success' }, data: nil }, status: :ok
     rescue StandardError => e
-      Rails.logger.error("[Vpago] Failed to enqueue payment processor job: #{params} #{e.message}")
+      Rails.logger.error("[Vpago][#{@payment.number}] Failed to enqueue payment processor job for payment #{@payment.number}: #{e.message}")
       render json: { status: :internal_server_error, message: 'Failed to enqueue payment processor job' }, status: :internal_server_error
     end
 

data/app/models/vpago/payment_decorator.rb

@@ -33,6 +33,8 @@ module Vpago
       end
     end
 
+    # Stores preloaded payout IDs in private_metadata to avoid N+1 queries
+    # when checking whether payouts exist for this payment.
     def preload_payout_ids=(ids)
       self.private_metadata ||= {}
       self.private_metadata['preload_payout_ids'] = ids
@@ -115,4 +117,4 @@ module Vpago
   end
 end
 
-Spree::Payment.prepend(Vpago::PaymentDecorator) unless Spree::Payment.included_modules.include?(Vpago::PaymentDecorator)
+Spree::Payment.prepend(Vpago::PaymentDecorator) unless Spree::Payment.include?(Vpago::PaymentDecorator)

data/app/views/spree/vpago_payments/forms/spree/gateway/_payway_v2.html.erb

@@ -72,7 +72,7 @@
 
       shouldShowRedirectContainer = !shouldOpenCheckoutUrl && shouldOpenAbaPayDeeplink;
       if(shouldShowRedirectContainer){
-        showRedirectContainer("Open ABA Mobile", () => openDeeplinkUrl(data.abapay_deeplink));
+        showRedirectContainer("Open ABA Mobile", () => window.location.href = data.abapay_deeplink);
       }
     })
     .catch(function(error) {

data/lib/spree_vpago/version.rb

@@ -1,7 +1,7 @@
 module SpreeVpago
   module_function
 
-  VERSION = '2.2.2-pre38'.freeze
+  VERSION = '2.2.2-pre40'.freeze
 
   def version
     Gem::Version.new VERSION

data/lib/vpago/payway_v2/payment_request_updater.rb

@@ -56,7 +56,7 @@ module Vpago
       end
 
       def items_eligible?
-        @payment&.order&.line_items&.all?(&:sufficient_stock?) == true
+        @payment&.order&.line_items&.all?(&:sufficient_stock?) == true # rubocop:disable Style/SafeNavigationChainLength
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spree_vpago
 version: !ruby/object:Gem::Version
-  version: 2.2.2.pre.pre38
+  version: 2.2.2.pre.pre40
 platform: ruby
 authors:
 - You
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-03-23 00:00:00.000000000 Z
+date: 2026-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday