From 4c046f43af85d197fca40dc112687b9a69395ed5 Mon Sep 17 00:00:00 2001
From: Julien Loizelet <julienloizelet@okaeli.com>
Date: Thu, 25 Jan 2024 15:27:53 +0900
Subject: [PATCH] fix(2.3): Disable phpcs
 Magento2.Legacy.EscapeMethodsOnBlockClass for some templates

---
 .../templates/report/events-list-title.phtml  |  8 +++-
 view/adminhtml/templates/report/metrics.phtml | 42 ++++++++++---------
 2 files changed, 29 insertions(+), 21 deletions(-)

diff --git a/view/adminhtml/templates/report/events-list-title.phtml b/view/adminhtml/templates/report/events-list-title.phtml
index 9957c37..9bd1378 100644
--- a/view/adminhtml/templates/report/events-list-title.phtml
+++ b/view/adminhtml/templates/report/events-list-title.phtml
@@ -24,13 +24,17 @@
  * @author   CrowdSec team
  *
  */
+/**
+ *
+ * @phpcs:disable Magento2.Legacy.EscapeMethodsOnBlockClass
+ *
+ */
 
 /** @var \Magento\Backend\Block\Template $block */
-/** @var \Magento\Framework\Escaper $escaper */
 ?>
 
 <div>
-    <h2 class="crowdsec-engine-report-subtitle"><?= $escaper->escapeHtml(__('Detected events list'))?></h2>
+    <h2 class="crowdsec-engine-report-subtitle"><?= $block->escapeHtml(__('Detected events list'))?></h2>
 </div>
 
 
diff --git a/view/adminhtml/templates/report/metrics.phtml b/view/adminhtml/templates/report/metrics.phtml
index ba88d77..8b30f48 100644
--- a/view/adminhtml/templates/report/metrics.phtml
+++ b/view/adminhtml/templates/report/metrics.phtml
@@ -24,9 +24,13 @@
  * @author   CrowdSec team
  *
  */
+/**
+ *
+ * @phpcs:disable Magento2.Legacy.EscapeMethodsOnBlockClass
+ *
+ */
 
 /** @var \CrowdSec\Engine\Block\Adminhtml\Report\Metrics $block */
-/** @var \Magento\Framework\Escaper $escaper */
 ?>
 
 <?php
@@ -34,14 +38,14 @@
 ?>
 
 <div id="crowdsec-engine-remediation-metrics" >
-    <h2 class="crowdsec-engine-report-subtitle"><?= $escaper->escapeHtml(__('Metrics')) ?></h2>
+    <h2 class="crowdsec-engine-report-subtitle"><?= $block->escapeHtml(__('Metrics')) ?></h2>
     <div class="description">
-        <h3><?= $escaper->escapeHtml(__('Current bouncing behaviors settings')) ?></h3>
+        <h3><?= $block->escapeHtml(__('Current bouncing behaviors settings')) ?></h3>
         <ul>
             <li>&bull;
                 <?php if ($block->isLocalBanEnabled()): ?>
                     <?=
-                    $escaper->escapeHtml(
+                    $block->escapeHtml(
                         __('<i>Ban IP locally</i> <b>enabled</b>
 &rarr; When a Magento scenario triggers an alert, IP <b>is banned</b> locally.'),
                         ['i','a','b']
@@ -49,7 +53,7 @@
                     ?>
                 <?php else: ?>
                     <?=
-                    $escaper->escapeHtml(
+                    $block->escapeHtml(
                         __('<i>Ban IP locally</i> <b>disabled</b>
 &rarr; When a Magento scenario triggers an alert, IP <b>is not banned locally</b>.'),
                         ['i','a','b']
@@ -59,14 +63,14 @@
             <li>&bull;
                 <?php if ($block->isBanBouncingEnabled()): ?>
                     <?=
-                    $escaper->escapeHtml(
+                    $block->escapeHtml(
                         __('<i>Block banned IP</i> <b>enabled</b>
 &rarr; Access is currently <b>blocked</b> for banned IP addresses.'),
                         ['i','a','b']
                     ) ?>
                 <?php else: ?>
                     <?=
-                    $escaper->escapeHtml(
+                    $block->escapeHtml(
                         __('<i>Block banned IP</i> <b>disabled</b>
 &rarr; Access is currently <b>allowed</b> for banned IP addresses.'),
                         ['i','a','b']
@@ -74,14 +78,14 @@
                 <?php endif; ?>
             </li>
         </ul>
-        <p class="notice"><?= $escaper->escapeHtml(__(
+        <p class="notice"><?= $block->escapeHtml(__(
             'You can change these behaviors by modifying decisions settings <a href="%1">here</a>.',
             $block->getSettingsUrl() .'#crowdsec_engine_decisions-link'
         ), ['i','a','b'])
 ?></p>
     </div>
     <div class="description">
-        <h3><?= $escaper->escapeHtml(__('Decision metrics')) ?><sup>(1)</sup></h3>
+        <h3><?= $block->escapeHtml(__('Decision metrics')) ?><sup>(1)</sup></h3>
     </div>
 
     <div class="admin_data-grid-wrap">
@@ -89,39 +93,39 @@
             <thead>
                 <th class="data_grid-th">
                     <span class="data-grid-cell-content">
-                        <?= $escaper->escapeHtml(__('Origin of the decision')) ?>
+                        <?= $block->escapeHtml(__('Origin of the decision')) ?>
                     </span>
                 </th>
                 <th class="data_grid-th">
-                    <span class="data-grid-cell-content"><?= $escaper->escapeHtml(__('Count')) ?></span>
+                    <span class="data-grid-cell-content"><?= $block->escapeHtml(__('Count')) ?></span>
                 </th>
             </thead>
             <tbody>
                 <tr class="data-row">
                     <td class="origin">
-                        <?= $escaper->escapeHtml(__('CrowdSec Engine extension (Local decision)')) ?>
+                        <?= $block->escapeHtml(__('CrowdSec Engine extension (Local decision)')) ?>
                     </td>
-                    <td class="count"><?= $escaper->escapeHtml($origins[$block->getOriginCrowdsec()] ?? 0) ?></td>
+                    <td class="count"><?= $block->escapeHtml($origins[$block->getOriginCrowdsec()] ?? 0) ?></td>
                 </tr>
                 <tr class="data-row">
-                    <td class="origin"><?= $escaper->escapeHtml(__('CrowdSec Community blocklist')) ?></td>
-                    <td class="count"><?= $escaper->escapeHtml($origins[$block->getOriginCapi()] ?? 0) ?></td>
+                    <td class="origin"><?= $block->escapeHtml(__('CrowdSec Community blocklist')) ?></td>
+                    <td class="count"><?= $block->escapeHtml($origins[$block->getOriginCapi()] ?? 0) ?></td>
                 </tr>
                 <tr class="data-row">
-                    <td class="origin"><?= $escaper->escapeHtml(__('Subscribed third-party blocklists')) ?></td>
-                    <td class="count"><?= $escaper->escapeHtml($origins[$block->getOriginLists()] ?? 0) ?></td>
+                    <td class="origin"><?= $block->escapeHtml(__('Subscribed third-party blocklists')) ?></td>
+                    <td class="count"><?= $block->escapeHtml($origins[$block->getOriginLists()] ?? 0) ?></td>
                 </tr>
             </tbody>
         </table>
     </div>
     <div>
         <p><span>1. </span>
-        <?= $escaper->escapeHtml(
+        <?= $block->escapeHtml(
             __('This module stores CrowdSec decision metrics only if the <i>Block banned IP</i> setting is enabled.'),
             ['i']
         ) ?><br>
         <?=
-        $escaper->escapeHtml(
+        $block->escapeHtml(
             __('As this information is stored in cache,
                 it will be reset if you clear the CrowdSec Engine cache.')
         )?>