Improve error display when card payment is refused

PrestaShopCorp · Jul 31, 2023 · 83dfee1 · 83dfee1
1 parent 1b63c4d
commit 83dfee1
Show file tree

Hide file tree

Showing 3 changed files with 47 additions and 32 deletions.
diff --git a/controllers/front/validate.php b/controllers/front/validate.php
@@ -89,11 +89,6 @@ public function postProcess()
 
             $this->sendOkResponse($this->generateResponse());
         } catch (Exception $exception) {
-            $this->module->getLogger()->error('CheckoutCompletedEvent failed', [
-                'exception_class' => get_class($exception),
-                'exception_message' => $exception->getMessage(),
-                'exception_code' => $exception->getCode(),
-            ]);
             $response = $this->generateResponse();
 
             if (!empty($response)) {

diff --git a/src/PayPal/Order/CommandHandler/CapturePayPalOrderCommandHandler.php b/src/PayPal/Order/CommandHandler/CapturePayPalOrderCommandHandler.php
@@ -78,19 +78,31 @@ public function handle(CapturePayPalOrderCommand $capturePayPalOrderCommand)
         $orderPayPal = $response['body'];
         $capturePayPal = $orderPayPal['purchase_units'][0]['payments']['captures'][0];
 
+        if ($orderPayPal['status'] === PayPalOrderStatus::COMPLETED) {
+            $this->eventDispatcher->dispatch(new PayPalOrderCompletedEvent($orderPayPal['id'], $orderPayPal));
+        }
+
+        if ($capturePayPal['status'] === PayPalCaptureStatus::PENDING) {
+            $this->eventDispatcher->dispatch(new PayPalCapturePendingEvent($capturePayPal['id'], $orderPayPal['id'], $capturePayPal));
+        }
+
+        if ($capturePayPal['status'] === PayPalCaptureStatus::COMPLETED) {
+            $this->eventDispatcher->dispatch(new PayPalCaptureCompletedEvent($capturePayPal['id'], $orderPayPal['id'], $capturePayPal));
+        }
+
         if ($capturePayPal['status'] === PayPalCaptureStatus::DECLINED || $capturePayPal['status'] === PayPalCaptureStatus::FAILED) {
             $this->eventDispatcher->dispatch(new PayPalCaptureDeclinedEvent($capturePayPal['id'], $orderPayPal['id'], $capturePayPal));
         }
 
         if (
-            'DECLINED' === $capturePayPal['status']
-            && false === empty($response['body']['payment_source'])
-            && false === empty($response['body']['payment_source']['card'])
+            PayPalCaptureStatus::DECLINED === $capturePayPal['status']
+            && false === empty($orderPayPal['payment_source'])
+            && false === empty($orderPayPal['payment_source']['card'])
             && false === empty($capturePayPal['processor_response'])
         ) {
             $payPalProcessorResponse = new PayPalProcessorResponse(
-                isset($response['body']['payment_source']['card']['brand']) ? $response['body']['payment_source']['card']['brand'] : null,
-                isset($response['body']['payment_source']['card']['type']) ? $response['body']['payment_source']['card']['type'] : null,
+                isset($orderPayPal['payment_source']['card']['brand']) ? $orderPayPal['payment_source']['card']['brand'] : null,
+                isset($orderPayPal['payment_source']['card']['type']) ? $orderPayPal['payment_source']['card']['type'] : null,
                 isset($capturePayPal['processor_response']['avs_code']) ? $capturePayPal['processor_response']['avs_code'] : null,
                 isset($capturePayPal['processor_response']['cvv_code']) ? $capturePayPal['processor_response']['cvv_code'] : null,
                 isset($capturePayPal['processor_response']['response_code']) ? $capturePayPal['processor_response']['response_code'] : null
@@ -99,17 +111,5 @@ public function handle(CapturePayPalOrderCommand $capturePayPalOrderCommand)
         } elseif (PayPalCaptureStatus::DECLINED === $capturePayPal['status'] || PayPalCaptureStatus::FAILED === $capturePayPal['status']) {
             throw new PsCheckoutException('PayPal declined the capture', PsCheckoutException::PAYPAL_PAYMENT_CAPTURE_DECLINED);
         }
-
-        if ($orderPayPal['status'] === PayPalOrderStatus::COMPLETED) {
-            $this->eventDispatcher->dispatch(new PayPalOrderCompletedEvent($orderPayPal['id'], $orderPayPal));
-        }
-
-        if ($capturePayPal['status'] === PayPalCaptureStatus::PENDING) {
-            $this->eventDispatcher->dispatch(new PayPalCapturePendingEvent($capturePayPal['id'], $orderPayPal['id'], $capturePayPal));
-        }
-
-        if ($capturePayPal['status'] === PayPalCaptureStatus::COMPLETED) {
-            $this->eventDispatcher->dispatch(new PayPalCaptureCompletedEvent($capturePayPal['id'], $orderPayPal['id'], $capturePayPal));
-        }
     }
 }
diff --git a/src/PayPal/Payment/Capture/EventSubscriber/PayPalCaptureEventSubscriber.php b/src/PayPal/Payment/Capture/EventSubscriber/PayPalCaptureEventSubscriber.php
@@ -137,8 +137,12 @@ public function createOrder(PayPalCaptureEvent $event)
 
     public function createOrderPayment(PayPalCaptureCompletedEvent $event)
     {
-        /** @var GetOrderForPaymentCompletedQueryResult $order */
-        $order = $this->commandBus->handle(new GetOrderForPaymentCompletedQuery($event->getPayPalOrderId()->getValue(), $event->getPayPalCaptureId()->getValue()));
+        try {
+            /** @var GetOrderForPaymentCompletedQueryResult $order */
+            $order = $this->commandBus->handle(new GetOrderForPaymentCompletedQuery($event->getPayPalOrderId()->getValue(), $event->getPayPalCaptureId()->getValue()));
+        } catch (OrderNotFoundException $exception) {
+            return;
+        }
 
         if ($order->getOrderPaymentId()) {
             return;
@@ -158,8 +162,12 @@ public function createOrderPayment(PayPalCaptureCompletedEvent $event)
 
     public function setPaymentCompletedOrderStatus(PayPalCaptureCompletedEvent $event)
     {
-        /** @var GetOrderForPaymentCompletedQueryResult $order */
-        $order = $this->commandBus->handle(new GetOrderForPaymentCompletedQuery($event->getPayPalOrderId()->getValue(), $event->getPayPalCaptureId()->getValue()));
+        try {
+            /** @var GetOrderForPaymentCompletedQueryResult $order */
+            $order = $this->commandBus->handle(new GetOrderForPaymentCompletedQuery($event->getPayPalOrderId()->getValue(), $event->getPayPalCaptureId()->getValue()));
+        } catch (OrderNotFoundException $exception) {
+            return;
+        }
 
         if ($order->hasBeenPaid()) {
             return;
@@ -178,8 +186,12 @@ public function setPaymentCompletedOrderStatus(PayPalCaptureCompletedEvent $even
 
     public function setPaymentPendingOrderStatus(PayPalCapturePendingEvent $event)
     {
-        /** @var GetOrderForPaymentPendingQueryResult $order */
-        $order = $this->commandBus->handle(new GetOrderForPaymentPendingQuery($event->getPayPalOrderId()->getValue()));
+        try {
+            /** @var GetOrderForPaymentPendingQueryResult $order */
+            $order = $this->commandBus->handle(new GetOrderForPaymentPendingQuery($event->getPayPalOrderId()->getValue()));
+        } catch (OrderNotFoundException $exception) {
+            return;
+        }
 
         if ($order->isInPending()) {
             return;
@@ -217,8 +229,12 @@ public function setPaymentDeclinedOrderStatus(PayPalCaptureDeclinedEvent $event)
 
     public function setPaymentRefundedOrderStatus(PayPalCaptureRefundedEvent $event)
     {
-        /** @var GetOrderForPaymentRefundedQueryResult $order */
-        $order = $this->commandBus->handle(new GetOrderForPaymentRefundedQuery($event->getPayPalOrderId()->getValue()));
+        try {
+            /** @var GetOrderForPaymentRefundedQueryResult $order */
+            $order = $this->commandBus->handle(new GetOrderForPaymentRefundedQuery($event->getPayPalOrderId()->getValue()));
+        } catch (OrderNotFoundException $exception) {
+            return;
+        }
 
         if (!$order->hasBeenPaid() || $order->hasBeenTotallyRefund()) {
             return;
@@ -233,8 +249,12 @@ public function setPaymentRefundedOrderStatus(PayPalCaptureRefundedEvent $event)
 
     public function setPaymentReversedOrderStatus(PayPalCaptureReversedEvent $event)
     {
-        /** @var GetOrderForPaymentReversedQueryResult $order */
-        $order = $this->commandBus->handle(new GetOrderForPaymentReversedQuery($event->getPayPalOrderId()->getValue(), $event->getPayPalCaptureId()->getValue()));
+        try {
+            /** @var GetOrderForPaymentReversedQueryResult $order */
+            $order = $this->commandBus->handle(new GetOrderForPaymentReversedQuery($event->getPayPalOrderId()->getValue(), $event->getPayPalCaptureId()->getValue()));
+        } catch (OrderNotFoundException $exception) {
+            return;
+        }
 
         if (!$order->hasBeenPaid() || $order->hasBeenTotallyRefund()) {
             return;