Properly handle test mode for existing payments/refunds (#8)

Author: raphaelm

pretix_payone/payment.py

@@ -274,8 +274,7 @@ def payment_control_render(self, request, payment) -> str:
         }
         return template.render(ctx)
 
-    @property
-    def _default_params(self):
+    def _default_params(self, testmode):
         from pretix import __version__
 
         from pretix_payone import __version__ as pluginver
@@ -286,7 +285,7 @@ def _default_params(self):
             "portalid": self.settings.portalid,
             "key": hashlib.md5(self.settings.key.encode()).hexdigest(),
             "api_version": "3.11",
-            "mode": "test" if self.event.testmode else "live",
+            "mode": "test" if testmode else "live",
             "encoding": "UTF-8",
             "integrator_name": "rami.io GmbH",
             "integrator_version": pluginver,
@@ -312,7 +311,7 @@ def execute_refund(self, refund: OrderRefund):
             )[:81],
             "transaction_param": f"{self.event.slug}-{refund.full_id}",
         }
-        data = dict(**refund_params, **self._default_params)
+        data = dict(**refund_params, **self._default_params(refund.order.testmode))
         try:
             req = requests.post(
                 "https://api.pay1.de/post-gateway/",
@@ -502,7 +501,8 @@ def _get_payment_params(self, request, payment):
 
     def execute_payment(self, request: HttpRequest, payment: OrderPayment):
         data = dict(
-            **self._get_payment_params(request, payment), **self._default_params
+            **self._get_payment_params(request, payment),
+            **self._default_params(payment.order.testmode),
         )
         try:
             req = requests.post(