The same goes for refunds of course

OneGov · Feb 11, 2025 · 58b8030 · 58b8030
1 parent 0fd8e79
commit 58b8030
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/src/onegov/pay/models/payment_providers/worldline_saferpay.py b/src/onegov/pay/models/payment_providers/worldline_saferpay.py
@@ -340,12 +340,13 @@ def refund(self, tx: SaferpayTransaction) -> SaferpayTransaction | None:
             )
             self.raise_for_status(res)
             refund_tx = SaferpayTransaction.model_validate_json(res.content)
-            try:
-                # try to capture but if we can't don't stress about it
-                # they can try to manually capture it
-                self.capture(refund_tx, 'REFUND')
-            except Exception:
-                log.exception('Failed to capture Saferpay refund')
+            if refund_tx.status == 'AUTHORIZED':
+                try:
+                    # try to capture but if we can't don't stress about it
+                    # they can try to manually capture it
+                    self.capture(refund_tx, 'REFUND')
+                except Exception:
+                    log.exception('Failed to capture Saferpay refund')
             return refund_tx
         elif tx.status == 'CANCELED':
             # transaction is already canceled