Skip to content

Commit a066291

Browse files
committed
MC-37594: Unexpected "Dashboard" page loading in the Admin panel
1 parent 57c527e commit a066291

File tree

1 file changed

+27
-18
lines changed

1 file changed

+27
-18
lines changed

app/code/Magento/Backend/view/adminhtml/templates/widget/tabshoriz.phtml

Lines changed: 27 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -4,45 +4,53 @@
44
* See COPYING.txt for license details.
55
*/
66

7-
/** @var \Magento\Framework\View\Helper\SecureHtmlRenderer $secureRenderer */
7+
use Magento\Framework\Escaper;
8+
use Magento\Framework\View\Helper\SecureHtmlRenderer;
9+
10+
/** @var SecureHtmlRenderer $secureRenderer */
11+
/** @var Escaper $escaper */
12+
$blockId = $block->getId();
813
?>
914
<!-- <?php if ($block->getTitle()): ?>
10-
<h3><?= $block->escapeHtml($block->getTitle()) ?></h3>
15+
<h3><?= $escaper->escapeHtml($block->getTitle()) ?></h3>
1116
<?php endif ?> -->
1217
<?php if (!empty($tabs)): ?>
13-
<div id="<?= $block->escapeHtmlAttr($block->getId()) ?>">
18+
<div id="<?= $escaper->escapeHtmlAttr($blockId) ?>" class="hidden">
1419
<ul class="tabs-horiz">
1520
<?php foreach ($tabs as $_tab): ?>
21+
<?php $tabId = $block->getTabId($_tab) ?>
1622
<?php $_tabClass = 'tab-item-link ' . $block->getTabClass($_tab) . ' ' .
1723
(preg_match('/\s?ajax\s?/', $_tab->getClass()) ? 'notloaded' : '') ?>
1824
<?php $_tabType = (!preg_match('/\s?ajax\s?/', $_tabClass) && $block->getTabUrl($_tab) != '#') ? 'link' : '' ?>
1925
<?php $_tabHref = $block->getTabUrl($_tab) == '#' ?
20-
'#' . $block->getTabId($_tab) . '_content' :
26+
'#' . $tabId . '_content' :
2127
$block->getTabUrl($_tab) ?>
2228
<li>
23-
<a href="<?= $block->escapeUrl($_tabHref) ?>"
24-
id="<?= $block->escapeHtmlAttr($block->getTabId($_tab)) ?>"
25-
title="<?= $block->escapeHtmlAttr($block->getTabTitle($_tab)) ?>"
26-
class="<?= $block->escapeHtmlAttr($_tabClass) ?>"
27-
data-tab-type="<?= $block->escapeHtmlAttr($_tabType) ?>">
29+
<a href="<?= $escaper->escapeUrl($_tabHref) ?>"
30+
id="<?= $escaper->escapeHtmlAttr($tabId) ?>"
31+
title="<?= $escaper->escapeHtmlAttr($block->getTabTitle($_tab)) ?>"
32+
class="<?= $escaper->escapeHtmlAttr($_tabClass) ?>"
33+
data-tab-type="<?= $escaper->escapeHtmlAttr($_tabType) ?>">
2834
<span>
2935
<span class="changed"
30-
title="<?= $block->escapeHtmlAttr(__('The information in this tab has been changed.')) ?>"></span>
36+
title="<?= $escaper->escapeHtmlAttr(__(
37+
'The information in this tab has been changed.'
38+
)) ?>"></span>
3139
<span class="error"
32-
title="<?= $block->escapeHtmlAttr(__(
40+
title="<?= $escaper->escapeHtmlAttr(__(
3341
'This tab contains invalid data. Please resolve this before saving.'
3442
)) ?>"></span>
3543
<span class="loader"
36-
title="<?= $block->escapeHtmlAttr(__('Loading...')) ?>"></span>
37-
<?= $block->escapeHtml($block->getTabLabel($_tab)) ?>
44+
title="<?= $escaper->escapeHtmlAttr(__('Loading...')) ?>"></span>
45+
<?= $escaper->escapeHtml($block->getTabLabel($_tab)) ?>
3846
</span>
3947
</a>
40-
<div id="<?= $block->escapeHtmlAttr($block->getTabId($_tab)) ?>_content">
48+
<div id="<?= $escaper->escapeHtmlAttr($tabId) ?>_content">
4149
<?= /* @noEscape */ $block->getTabContent($_tab) ?>
4250
</div>
4351
<?= /* @noEscape */ $secureRenderer->renderStyleAsTag(
4452
'display:none',
45-
'#' . $block->escapeJs($block->getTabId($_tab)) . '_content'
53+
'#' . $escaper->escapeJs($tabId) . '_content'
4654
); ?>
4755
</li>
4856
<?php endforeach; ?>
@@ -51,11 +59,12 @@
5159
<?php $scriptString = <<<script
5260
require(["jquery","mage/backend/tabs"], function($){
5361
$(function() {
54-
$('#{$block->getId()}').tabs({
55-
active: '{$block->getActiveTabId()}',
56-
destination: '#{$block->getDestElementId()}',
62+
$('#{$escaper->escapeJs($blockId)}').tabs({
63+
active: '{$escaper->escapeJs($block->getActiveTabId())}',
64+
destination: '#{$escaper->escapeJs($block->getDestElementId())}',
5765
shadowTabs: {$block->getAllShadowTabs()}
5866
});
67+
$('#{$escaper->escapeJs($blockId)}').removeClass('hidden');
5968
});
6069
});
6170
script;

0 commit comments

Comments
 (0)