Merge pull request #193 from paynl/feature/PLUG-5162

PLUG-5162 - paylink fix

Author: woutse

includes/classes/PPMFWC/Gateways.php

@@ -1,5 +1,6 @@
 <?php
 
+use PayNL\Sdk\Model\Pay\PayOrder;
 use PayNL\Sdk\Model\Pay\PayStatus;
 use PayNL\Sdk\Util\Exchange;
 
@@ -1023,6 +1024,27 @@ public static function ppmfwc_getGateWayById($payment_profile_id)
         return null;
     }
 
+    /**
+     * @param string $action
+     * @param PayOrder $payOrder
+     * @param Exchange $exchange
+     * @return void
+     * @throws Exception
+     */
+    private static function validateExchange(string $action, PayOrder $payOrder, Exchange $exchange): void
+    {
+        if ($payOrder->isPending() && $action != self::ACTION_PENDING) {
+            $exchange->setResponse(false, 'Unexpected action (' . $action . ') for order state pending.');
+
+        } elseif ($payOrder->isCancelled() && $action != self::ACTION_CANCEL) {
+            $exchange->setResponse(false, 'Unexpected action (' . $action . ') for order state cancelled.');
+
+        } elseif ($payOrder->isRefunded() && $action != self::ACTION_REFUND && $action != self::ACTION_REFUND_ADD) {
+            $exchange->setResponse(false, 'Unexpected action (' . $action . ') for order state refunded.');
+
+        }
+    }
+
     /**
      * Handles the Pay. Exchange requests
      * @phpcs:ignore Squiz.Commenting.FunctionComment.MissingReturn
@@ -1042,9 +1064,7 @@ public static function ppmfwc_onExchange()
             $payOrder = $exchange->process(PPMFWC_Helper_Config::getPayConfig());
             $payOrderId = $payOrder->getOrderId();
 
-            if ($payOrder->isPending()) {
-                $exchange->setResponse(true, 'Ignoring pending.');
-            }
+            self::validateExchange($action, $payOrder, $exchange);
 
             $order_id = $exchange->getReference();
             $methodId = $payOrder->getPaymentMethod();