Browse Source

fix(payments-plugin): Improve Mollie ignore order states (#2670)

Martijn 1 year ago
parent
commit
f02fc563cd
1 changed files with 10 additions and 1 deletions
  1. 10 1
      packages/payments-plugin/src/mollie/mollie.service.ts

+ 10 - 1
packages/payments-plugin/src/mollie/mollie.service.ts

@@ -12,6 +12,7 @@ import {
     Logger,
     Order,
     OrderService,
+    OrderState,
     OrderStateTransitionError,
     PaymentMethod,
     PaymentMethodService,
@@ -226,7 +227,15 @@ export class MollieService {
                 `Unable to find order ${mollieOrder.orderNumber}, unable to process Mollie order ${mollieOrder.id}`,
             );
         }
-        if (order.state === 'PaymentSettled' || order.state === 'Shipped' || order.state === 'Delivered') {
+        const statesThatRequireAction: OrderState[] = [
+            'AddingItems',
+            'ArrangingPayment',
+            'ArrangingAdditionalPayment',
+            'PaymentAuthorized',
+            'Draft',
+        ];
+        if (!statesThatRequireAction.includes(order.state)) {
+            // If order is not in one of these states, we don't need to handle the Mollie webhook
             Logger.info(
                 `Order ${order.code} is already '${order.state}', no need for handling Mollie status '${mollieOrder.status}'`,
                 loggerCtx,