Merge pull request #2743 from dpfaffenbauer/issue/2740

[PayumBundle] remove payment_state processing to confirm orders
coreshop · Oct 31, 2024 · 0d659a3 · 0d659a3
2 parents 4061015 + f0e5ed5
commit 0d659a3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/CoreShop/Bundle/PayumBundle/Action/ConfirmOrderAction.php b/src/CoreShop/Bundle/PayumBundle/Action/ConfirmOrderAction.php
@@ -37,8 +37,7 @@ public function execute($request): void
         $payment = $request->getFirstModel();
         $order = $payment->getOrder();
         if ($payment->getState() === PaymentInterface::STATE_COMPLETED ||
-            $payment->getState() === PaymentInterface::STATE_AUTHORIZED ||
-            $payment->getState() === PaymentInterface::STATE_PROCESSING
+            $payment->getState() === PaymentInterface::STATE_AUTHORIZED
         ) {
             $this->stateMachineApplier->apply($order, OrderTransitions::IDENTIFIER, OrderTransitions::TRANSITION_CONFIRM);
 

diff --git a/src/CoreShop/Bundle/PayumBundle/Extension/UpdateOrderStateExtension.php b/src/CoreShop/Bundle/PayumBundle/Extension/UpdateOrderStateExtension.php
@@ -92,8 +92,7 @@ public function onPostExecute(Context $context): void
         }
 
         if ($value === PaymentInterface::STATE_COMPLETED ||
-            $value === PaymentInterface::STATE_AUTHORIZED ||
-            $value === PaymentInterface::STATE_PROCESSING
+            $value === PaymentInterface::STATE_AUTHORIZED
         ) {
             $order = $payment->getOrder();
             $this->confirmOrderState($order);