Refactor PaymentNotFoundException constructor parameters

damms005 · Jul 24, 2024 · 5ef8c1a · 5ef8c1a
1 parent 5f225c4
commit 5ef8c1a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/Exceptions/PaymentNotFoundException.php b/src/Exceptions/PaymentNotFoundException.php
@@ -6,4 +6,12 @@
 
 class PaymentNotFoundException extends Exception
 {
+    public function __construct(
+        public \Illuminate\Http\Request $webhookRequest,
+        ?string $message = null,
+    ) {
+        $message = $message ?? 'Payment not found in Paystack\'s charge.success event. Payload: ' . json_encode($webhookRequest->all());
+
+        parent::__construct($message);
+    }
 }
diff --git a/src/Webhooks/Paystack/ChargeSuccess.php b/src/Webhooks/Paystack/ChargeSuccess.php
@@ -30,7 +30,7 @@ public function handle(Request $webhookRequest): Payment
             ->first();
 
         if (!$payment) {
-            throw new PaymentNotFoundException(get_class(app(PaymentHandlerInterface::class)) . ' - Payment not found in Paystack\'s charge.success event. Payload: ' . json_encode($webhookRequest->all()));
+            throw new PaymentNotFoundException($webhookRequest, get_class(app(PaymentHandlerInterface::class)) . ' - Payment not found in Paystack\'s charge.success event. Payload: ' . json_encode($webhookRequest->all()));
         }
 
         $metadata = [...$payment->metadata ?? []];