fix: Order refund

two-inc · Nov 29, 2024 · 2e4fc2f · 2e4fc2f
1 parent 52cb974
commit 2e4fc2f
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/class/WC_Twoinc.php b/class/WC_Twoinc.php
@@ -1608,9 +1608,10 @@ public function process_refund($order_id, $amount = null, $reason = '')
                     sprintf(__('Order has already been fully refunded with %s.', 'twoinc-payment-gateway'), self::PRODUCT_NAME)
                 );
             }
+
             $order_refunds = $order->get_refunds();
-            // Need to loop instead of getting the last element because the last element is not always the latest refund
             $order_refund = null;
+            // Need to loop instead of getting the last element because the last element is not always the latest refund
             foreach ($order_refunds as $refund) {
                 if (!$refund->get_refunded_payment()) {
                     if (!$order_refund || $refund->get_date_created() > $order_refund->get_date_created()) {
@@ -1619,11 +1620,8 @@ public function process_refund($order_id, $amount = null, $reason = '')
                 }
             }
 
-            if (!$order_refund || !$twoinc_order_id) {
-                return new WP_Error(
-                    'invalid_twoinc_refund',
-                    sprintf(__('Could not initiate refund with %s', 'twoinc-payment-gateway'), self::PRODUCT_NAME)
-                );
+            if (!$order_refund) {
+                return false;
             }
 
             $refund_amount = $order_refund->get_amount();
@@ -1641,7 +1639,7 @@ public function process_refund($order_id, $amount = null, $reason = '')
                 "/v1/order/{$twoinc_order_id}/refund",
                 WC_Twoinc_Helper::compose_twoinc_refund(
                     $order_refund,
-                    -$amount,
+                    $amount,
                     $order->get_currency()
                 ),
                 'POST'