ENGCOM-5745: Resolve undefined variable in ReviewPayment class #24286

 - Merge Pull Request #24286 from edenduong/magento2:2.3-bugfix/resolve_undefined_variable_reviewpayment
 - Merged commits:
   1. d66f6c3

Author: magento-engcom-team

app/code/Magento/Sales/Controller/Adminhtml/Order/ReviewPayment.php

@@ -3,11 +3,18 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+
+declare(strict_types=1);
+
 namespace Magento\Sales\Controller\Adminhtml\Order;
 
 use Magento\Backend\App\Action;
+use Magento\Framework\App\Action\HttpGetActionInterface as HttpGetActionInterface;
 
-class ReviewPayment extends \Magento\Sales\Controller\Adminhtml\Order
+/**
+ * Class \Magento\Sales\Controller\Adminhtml\Order\ReviewPayment
+ */
+class ReviewPayment extends \Magento\Sales\Controller\Adminhtml\Order implements HttpGetActionInterface
 {
     /**
      * Authorization level of a basic admin session
@@ -21,7 +28,7 @@ class ReviewPayment extends \Magento\Sales\Controller\Adminhtml\Order
      *
      * Either denies or approves a payment that is in "review" state
      *
-     * @return \Magento\Backend\Model\View\Result\Redirect
+     * @return \Magento\Framework\Controller\Result\Redirect
      */
     public function execute()
     {
@@ -50,21 +57,23 @@ public function execute()
                         }
                         break;
                     default:
-                        throw new \Exception(sprintf('Action "%s" is not supported.', $action));
+                        throw new \Magento\Framework\Exception\NotFoundException(
+                            __('Action "%1" is not supported.', $action)
+                        );
                 }
                 $this->orderRepository->save($order);
                 $this->messageManager->addSuccessMessage($message);
+                $resultRedirect->setPath('sales/order/view', ['order_id' => $order->getEntityId()]);
             } else {
                 $resultRedirect->setPath('sales/*/');
                 return $resultRedirect;
             }
+            // phpcs:ignore Magento2.Exceptions.ThrowCatch
         } catch (\Magento\Framework\Exception\LocalizedException $e) {
-            $this->messageManager->addErrorMessage($e->getMessage());
-        } catch (\Exception $e) {
             $this->messageManager->addErrorMessage(__('We can\'t update the payment right now.'));
             $this->logger->critical($e);
+            $resultRedirect->setPath('sales/*/');
         }
-        $resultRedirect->setPath('sales/order/view', ['order_id' => $order->getEntityId()]);
         return $resultRedirect;
     }
 }