From 369550e85994584d1dbffe856921e66ec526505f Mon Sep 17 00:00:00 2001
From: sergey <sergey.nezbritskiy@gmail.com>
Date: Sat, 16 Jan 2021 15:07:18 +0200
Subject: [PATCH] Magento_Directory: Avoid using deprecated escape* methods
 from AbstractBlock

---
 app/code/Magento/Directory/Block/Data.php         |  2 +-
 .../templates/js/optional_zip_countries.phtml     | 10 ++++------
 .../view/frontend/templates/currency.phtml        | 15 ++++++++-------
 .../view/frontend/templates/currency/switch.phtml |  9 ++++++---
 4 files changed, 19 insertions(+), 17 deletions(-)

diff --git a/app/code/Magento/Directory/Block/Data.php b/app/code/Magento/Directory/Block/Data.php
index 0424b824e4454..b62f4c3de0818 100644
--- a/app/code/Magento/Directory/Block/Data.php
+++ b/app/code/Magento/Directory/Block/Data.php
@@ -142,7 +142,7 @@ public function getCountryHtmlSelect($defValue = null, $name = 'country_id', $id
         )->setId(
             $id
         )->setTitle(
-            $this->escapeHtmlAttr(__($title))
+            $this->_escaper->escapeHtmlAttr(__($title))
         )->setValue(
             $defValue
         )->setOptions(
diff --git a/app/code/Magento/Directory/view/adminhtml/templates/js/optional_zip_countries.phtml b/app/code/Magento/Directory/view/adminhtml/templates/js/optional_zip_countries.phtml
index 4037dc19bde33..68f48a08232e1 100644
--- a/app/code/Magento/Directory/view/adminhtml/templates/js/optional_zip_countries.phtml
+++ b/app/code/Magento/Directory/view/adminhtml/templates/js/optional_zip_countries.phtml
@@ -8,15 +8,13 @@
  * JS block for including Countries with Optional Zip
  *
  * @see \Magento\Backend\Block\Template
- */
-
-/**
+ *
  * @var \Magento\Backend\Block\Template $block
+ * @var \Magento\Framework\Escaper $escaper
  * @var \Magento\Framework\View\Helper\SecureHtmlRenderer $secureRenderer
+ * @var \Magento\Directory\Helper\Data $directoryHelper
  */
-?>
-<?php
-/** @var \Magento\Directory\Helper\Data $directoryHelper */
+
 $directoryHelper = $block->getData('directoryHelper');
 $countriesWithOptionalZip = /* @noEscape */ $directoryHelper->getCountriesWithOptionalZip(true);
 $scriptString = <<<script
diff --git a/app/code/Magento/Directory/view/frontend/templates/currency.phtml b/app/code/Magento/Directory/view/frontend/templates/currency.phtml
index 15cf9435bc5d1..b4566077258ed 100644
--- a/app/code/Magento/Directory/view/frontend/templates/currency.phtml
+++ b/app/code/Magento/Directory/view/frontend/templates/currency.phtml
@@ -8,6 +8,7 @@
  * Currency switcher
  *
  * @var \Magento\Directory\Block\Currency $block
+ * @var \Magento\Framework\Escaper $escaper
  */
 
 ?>
@@ -15,23 +16,23 @@
     <?php $currencies = $block->getCurrencies(); ?>
     <?php $currentCurrencyCode = $block->getCurrentCurrencyCode(); ?>
     <?php $id = $block->getIdModifier() ? '-' . $block->getIdModifier() : '' ?>
-    <div class="switcher currency switcher-currency" id="switcher-currency<?= $block->escapeHtmlAttr($id) ?>">
-        <strong class="label switcher-label"><span><?= $block->escapeHtml(__('Currency')) ?></span></strong>
+    <div class="switcher currency switcher-currency" id="switcher-currency<?= $escaper->escapeHtmlAttr($id) ?>">
+        <strong class="label switcher-label"><span><?= $escaper->escapeHtml(__('Currency')) ?></span></strong>
         <div class="actions dropdown options switcher-options">
             <div class="action toggle switcher-trigger"
-                 id="switcher-currency-trigger<?= $block->escapeHtmlAttr($id) ?>"
+                 id="switcher-currency-trigger<?= $escaper->escapeHtmlAttr($id) ?>"
                  data-mage-init='{"dropdown":{}}'
                  data-toggle="dropdown"
                  data-trigger-keypress-button="true">
-                <strong class="language-<?= $block->escapeHtml($block->getCurrentCurrencyCode()) ?>">
-                    <span><?= $block->escapeHtml($currentCurrencyCode) ?> - <?= $currencies[$currentCurrencyCode] ? $block->escapeHtml($currencies[$currentCurrencyCode]) : '' ?></span>
+                <strong class="language-<?= $escaper->escapeHtml($block->getCurrentCurrencyCode()) ?>">
+                    <span><?= $escaper->escapeHtml($currentCurrencyCode) ?> - <?= $currencies[$currentCurrencyCode] ? $escaper->escapeHtml($currencies[$currentCurrencyCode]) : '' ?></span>
                 </strong>
             </div>
             <ul class="dropdown switcher-dropdown" data-target="dropdown">
                 <?php foreach ($currencies as $_code => $_name) : ?>
                     <?php if ($_code != $currentCurrencyCode) : ?>
-                        <li class="currency-<?= $block->escapeHtmlAttr($_code) ?> switcher-option">
-                            <a href="#" data-post='<?= /* @noEscape */ $block->getSwitchCurrencyPostData($_code) ?>'><?= $block->escapeHtml($_code) ?> - <?= $block->escapeHtml($_name) ?></a>
+                        <li class="currency-<?= $escaper->escapeHtmlAttr($_code) ?> switcher-option">
+                            <a href="#" data-post='<?= /* @noEscape */ $block->getSwitchCurrencyPostData($_code) ?>'><?= $escaper->escapeHtml($_code) ?> - <?= $escaper->escapeHtml($_name) ?></a>
                         </li>
                     <?php endif; ?>
                 <?php endforeach; ?>
diff --git a/app/code/Magento/Directory/view/frontend/templates/currency/switch.phtml b/app/code/Magento/Directory/view/frontend/templates/currency/switch.phtml
index 50995dbd1d06d..4a4971784a5a1 100644
--- a/app/code/Magento/Directory/view/frontend/templates/currency/switch.phtml
+++ b/app/code/Magento/Directory/view/frontend/templates/currency/switch.phtml
@@ -4,8 +4,11 @@
  * See COPYING.txt for license details.
  */
 
-/** @var \Magento\Framework\View\Element\Template $block */
+/**
+ * @var \Magento\Framework\View\Element\Template $block
+ * @var \Magento\Framework\Escaper $escaper
+ */
 
 ?>
-<p><?= $block->escapeHtml(__('Your current currency is: %1.', $currency->getCode())) ?></p>
-<p><a href="<?= $block->escapeUrl($block->getBaseUrl()) ?>" class="action continue"><span><?= $block->escapeHtml(__('Continue')) ?></span></a></p>
+<p><?= $escaper->escapeHtml(__('Your current currency is: %1.', $currency->getCode())) ?></p>
+<p><a href="<?= $escaper->escapeUrl($block->getBaseUrl()) ?>" class="action continue"><span><?= $escaper->escapeHtml(__('Continue')) ?></span></a></p>