Skip to content

Commit 4f6f01f

Browse files
committed
MC-16618: Eliminate @escapeNotVerified in Sales-related Modules
- Admin static issues
1 parent eee9485 commit 4f6f01f

File tree

5 files changed

+10
-8
lines changed

5 files changed

+10
-8
lines changed

app/code/Magento/Sales/view/adminhtml/templates/order/create/js.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6-
?>
6+
?>
77
<script>
88
require([
99
"prototype",

app/code/Magento/Sales/view/adminhtml/templates/order/details.phtml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ $_order = $block->getOrder() ?>
4444
<td align="center" valign="top" style="padding:3px 9px"><?= (int) $_item->getQtyOrdered()*1 ?></td>
4545
<td align="right" valign="top" style="padding:3px 9px"><?= /* @noEscape */ $_order->formatPrice($_item->getRowTotal()) ?></td>
4646
</tr>
47-
<?php endforeach ?>
47+
<?php endforeach; ?>
4848
</tbody>
4949

5050
<tfoot>
@@ -57,7 +57,7 @@ $_order = $block->getOrder() ?>
5757
<br /><?= $block->escapeHtml(__('Message:')) ?><br /> <?= $block->escapeHtml($_giftMessage->getMessage()) ?>
5858
</td>
5959
</tr>
60-
<?php endif; ?>
60+
<?php endif; ?>
6161
<tr>
6262
<td colspan="2" align="right" style="padding:3px 9px"><?= $block->escapeHtml(__('Subtotal')) ?></td>
6363
<td align="right" style="padding:3px 9px"><?= /* @noEscape */ $_order->formatPrice($_order->getSubtotal()) ?></td>

app/code/Magento/Sales/view/adminhtml/templates/order/totalbar.phtml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,13 @@
77
// @deprecated
88
$totals = $block->getTotals();
99
?>
10-
<?php if ($totals && count($totals) > 0) : ?>
10+
<?php if ($totals && !empty($totals)) : ?>
1111
<table class="items-to-invoice">
1212
<tr>
13-
<?php foreach ($totals as $total): ?>
14-
<td <?php if ($total['grand']): ?>class="grand-total"<?php endif; ?>>
13+
<?php foreach ($totals as $total) : ?>
14+
<td <?php if ($total['grand']) :
15+
?>class="grand-total"<?php
16+
endif; ?>>
1517
<?= $block->escapeHtml($total['label']) ?><br />
1618
<?= $block->escapeHtml($total['value']) ?>
1719
</td>

app/code/Magento/Sales/view/adminhtml/templates/order/totals/paid.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* See COPYING.txt for license details.
55
*/
66
?>
7-
<?php if ($block->getCanDisplayTotalPaid()): ?>
7+
<?php if ($block->getCanDisplayTotalPaid()) : ?>
88
<tr>
99
<td class="label"><strong><?= $block->escapeHtml(__('Total Paid')) ?></strong></td>
1010
<td class="emph"><?= /* @noEscape */ $block->displayPriceAttribute('total_paid', true) ?>

app/code/Magento/Sales/view/adminhtml/templates/rss/order/grid/link.phtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,6 @@
66

77
/** @var $block \Magento\Sales\Block\Adminhtml\Rss\Order\Grid\Link */
88
?>
9-
<?php if ($block->isRssAllowed() && $block->getLink()): ?>
9+
<?php if ($block->isRssAllowed() && $block->getLink()) : ?>
1010
<a href="<?= $block->escapeUrl($block->getLink()) ?>" class="link-feed"><?= $block->escapeHtml($block->getLabel()) ?></a>
1111
<?php endif; ?>

0 commit comments

Comments
 (0)