compatibility Prestashop Validator

202-ecommerce · Jul 30, 2020 · d3f08ef · d3f08ef
1 parent 0d47e83
commit d3f08ef
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/classes/StripeWebhook.php b/classes/StripeWebhook.php
@@ -46,7 +46,7 @@ public static function create()
 
             Configuration::updateValue(Stripe_official::WEBHOOK_SIGNATURE, $webhookEndpoint->secret);
         } catch (Exception $e) {
-          ProcessLoggerHandler::logError(
+            ProcessLoggerHandler::logError(
                 'Create webhook endpoint - '.(string)$e->getMessage(),
                 null,
                 null,

diff --git a/classes/actions/ValidationOrderActions.php b/classes/actions/ValidationOrderActions.php
@@ -207,7 +207,7 @@ public function createOrder()
             && $this->conveyor['status'] == 'pending') {
             $orderStatus = Configuration::get('STRIPE_OS_SOFORT_WAITING');
             $this->conveyor['result'] = 4;
-        } else if ($this->conveyor['datas']['type'] == 'sepa_debit') {
+        } elseif ($this->conveyor['datas']['type'] == 'sepa_debit') {
             $orderStatus = Configuration::get(Stripe_official::SEPA_WAITING);
             $this->conveyor['result'] = 3;
         } else {
@@ -474,12 +474,12 @@ public function chargeWebhook()
             $order->update();
 
             $history->addWithemail();
-        } elseif ($this->conveyor['event_json']->type == 'charge.expired' || $this->conveyor['event_json']->type == 'charge.refunded') {
+        } elseif ($this->conveyor['event_json']->type == 'charge.expired'
+            || $this->conveyor['event_json']->type == 'charge.refunded') {
             $order->setCurrentState(Configuration::get('PS_OS_CANCELED'));
         } elseif ($this->conveyor['event_json']->type == 'charge.succeeded') {
             $order->setCurrentState(Configuration::get('PS_OS_PAYMENT'));
         } elseif ($this->conveyor['event_json']->type == 'charge.failed') {
-
             $order->setCurrentState(Configuration::get('PS_OS_ERROR'));
         }
 

diff --git a/controllers/front/validation.php b/controllers/front/validation.php
@@ -73,7 +73,7 @@ public function initContent()
         // Set list of actions to execute
         if (Tools::getValue('source')) {
             $handler->addActions('prepareFlowRedirect', 'updatePaymentIntent', 'createOrder', 'addTentative');
-        } else if (Tools::getValue('payment_intent')) {
+        } elseif (Tools::getValue('payment_intent')) {
             $handler->addActions('prepareFlowRedirectPaymentIntent', 'updatePaymentIntent', 'createOrder', 'addTentative');
         } else {
             $handler->addActions('prepareFlowNone', 'updatePaymentIntent', 'createOrder', 'saveCard', 'addTentative');

diff --git a/views/templates/hook/admin_content_order.tpl b/views/templates/hook/admin_content_order.tpl
@@ -25,7 +25,7 @@
 <div class="tab-pane" id="StripePayment">
     <p>
         <span><strong>{l s='Payment ID' mod='stripe_official'}</strong></span><br/>
-        <span><a href="{$stripe_dashboardUrl.charge|escape:'htmlall'}" target="blank">{$stripe_charge}</a></span>
+        <span><a href="{$stripe_dashboardUrl.charge|escape:'htmlall'}" target="blank">{$stripe_charge|escape:'htmlall':'UTF-8'}</a></span>
     </p>
     <p>
         <span><strong>{l s='Payment date' mod='stripe_official'}</strong></span><br/>