Merge pull request #17 from dystcz/fix/change-order-of-execution-duri…

…ng-authorization Fix: Create transaction before updating order status
dystcz · Jan 16, 2025 · 8e1662a · 8e1662a
2 parents d111e44 + a8bab05
commit 8e1662a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/api/src/Domain/Payments/PaymentTypes/OfflinePaymentType.php b/packages/api/src/Domain/Payments/PaymentTypes/OfflinePaymentType.php
@@ -44,14 +44,14 @@ public function authorize(string $paymentType = 'offline'): PaymentAuthorize
 
         $status = $this->data['authorized'] ?? null;
 
+        $this->createCaptureTransaction($paymentType);
+
         $this->order->update([
             'status' => $status ?? $this->config['authorized'] ?? 'payment-received',
             'meta' => $meta,
             'placed_at' => Carbon::now(),
         ]);
 
-        $this->createCaptureTransaction($paymentType);
-
         $authorization = new PaymentAuthorize(
             success: true,
             orderId: $this->order->id,