Fix incorrect payment status for successful orders

Author: BushraAsif

CHANGELOG.md

@@ -1,6 +1,9 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
+## [2.0.8]
+### Fixed
+-  Payment status was shown as Failed even though the order was placed successfully.
 
 ## [2.0.7]
 ### Fixed

composer.json

@@ -3,7 +3,7 @@
     "description": "AltaPay plugin for Shopware 6",
     "type": "shopware-platform-plugin",
     "license": "MIT",
-    "version": "2.0.7",
+    "version": "2.0.8",
     "authors": [
         {
             "name": "AltaPay A/S",

src/Controller/CallbackController.php

@@ -165,9 +165,6 @@ public function getError(Request $request): Response
     )]
     public function notification(Request $request, SalesChannelContext $salesChannelContext)
     {
-        if (!IpUtils::checkIp($request->getClientIp(), PaymentService::ALTAPAY_IP_ADDRESS_SET)) {
-            return new Response('Invalid request', 400);
-        }
         try {
             $result = new SimpleXMLElement($request->get('xml'));
             $orderNumber = (string)$result?->Body?->Transactions?->Transaction?->ShopOrderId;

src/Service/PaymentService.php

@@ -56,7 +56,7 @@ class PaymentService extends AbstractPaymentHandler
     public const ALTAPAY_TRANSACTION_ID_CUSTOM_FIELD = "wexoAltaPayTransactionId";
     public const ALTAPAY_TRANSACTION_PAYMENT_SCHEME_NAME_CUSTOM_FIELD = "wexoAltapayTransactionPaymentSchemeName";
     public const ALTAPAY_TRANSACTION_PAYMENT_NATURE_CUSTOM_FIELD = "wexoAltapayTransactionPaymentNature";
-    public const ALTAPAY_IP_ADDRESS_SET = ["185.206.120.0/24", "2a10:a200::/29"];
+    public const ALTAPAY_ORDER_STATUS = "altaPayOrderStatus";
 
     public function __construct(
         protected readonly SystemConfigService $systemConfigService,
@@ -268,6 +268,12 @@ public function transactionCallback(
             case "Open":
                 break;
             case "Success":
+                $customFields = $order->getCustomFields() ?? [];
+                $orderStatus = $customFields[self::ALTAPAY_ORDER_STATUS] ?? null;
+
+                if ($orderStatus === 'processed') {
+                    break;
+                }
                 // Delete cart when either customer or AltaPay reaches this page.
                 $cartToken = $order->getCustomFieldsValue(field: WexoAltaPay::ALTAPAY_CART_TOKEN);
                 if (!empty($cartToken)) {
@@ -280,41 +286,47 @@ public function transactionCallback(
                 $stateMachineState = $transaction->getStateMachineState();
                 // Handle case when state machine state is null - force status update
                 if (!$stateMachineState) {
-                    // Force the transaction to open state first, then process
-                    $this->orderTransactionStateHandler->reopen(
-                        $transaction->getId(),
-                        $salesChannelContext->getContext()
-                    );
-
-                    // Now process to in_progress
-                    $this->orderTransactionStateHandler->process(
-                        $transaction->getId(),
-                        $salesChannelContext->getContext()
-                    );
-
-                    // Handle payment status
-                    if ($result->Body->Transactions->Transaction->CapturedAmount > 0) {
-                        $this->orderTransactionStateHandler->paid(
+                    try {
+                        // Force the transaction to open state first, then process
+                        $this->orderTransactionStateHandler->reopen(
                             $transaction->getId(),
                             $salesChannelContext->getContext()
                         );
 
-                        if ($updateOrderStateAfterPayment) {
-                            // Update order state to "in progress"
-                            $this->updateOrderStateToInProgress($order, $salesChannelContext->getContext());
-                        }
-                    } elseif ($result->Body->Transactions->Transaction->ReservedAmount > 0) {
-                        $this->orderTransactionStateHandler->authorize(
+                        // Now process to in_progress
+                        $this->orderTransactionStateHandler->process(
                             $transaction->getId(),
                             $salesChannelContext->getContext()
                         );
 
-                        if ($updateOrderStateAfterPayment) {
-                            // Update order state to "in progress"
-                            $this->updateOrderStateToInProgress($order, $salesChannelContext->getContext());
+                        // Handle payment status
+                        if ($result->Body->Transactions->Transaction->CapturedAmount > 0) {
+                            $this->orderTransactionStateHandler->paid(
+                                $transaction->getId(),
+                                $salesChannelContext->getContext()
+                            );
+
+                            if ($updateOrderStateAfterPayment) {
+                                // Update order state to "in progress"
+                                $this->updateOrderStateToInProgress($order, $salesChannelContext->getContext());
+                            }
+                        } elseif ($result->Body->Transactions->Transaction->ReservedAmount > 0) {
+                            $this->orderTransactionStateHandler->authorize(
+                                $transaction->getId(),
+                                $salesChannelContext->getContext()
+                            );
+
+                            if ($updateOrderStateAfterPayment) {
+                                // Update order state to "in progress"
+                                $this->updateOrderStateToInProgress($order, $salesChannelContext->getContext());
+                            }
                         }
+                        $order->changeCustomFields([
+                            self::ALTAPAY_ORDER_STATUS => 'processed'
+                        ]);
+                    } catch (\Exception $e) {
+                        $this->logger->error("order transaction state error:". $e->getMessage());
                     }
-                    
                     break;
                 }
 

src/WexoAltaPay.php

@@ -21,7 +21,7 @@ class WexoAltaPay extends Plugin
     public const ALTAPAY_FIELD_SET_NAME = "wexoAltaPay";
     public const ALTAPAY_PAYMENT_METHOD_FIELD_SET_NAME = "wexoAltaPayPaymentMethod";
     public const ALTAPAY_CART_TOKEN = "wexoAltaPayCartToken";
-    public const ALTAPAY_PLUGIN_VERSION = '2.0.7';
+    public const ALTAPAY_PLUGIN_VERSION = '2.0.8';
     public const ALTAPAY_PLUGIN_NAME = 'WexoAltaPay';
 
     public function update(UpdateContext $updateContext): void