Skip to content

Commit f35fb2b

Browse files
mrtuvnmrtuvn
committed
Cleanup duplicate html class
Fix test fails fix static tests Fix static fails update update update
1 parent bb176b8 commit f35fb2b

File tree

2 files changed

+38
-24
lines changed
  • app/code/Magento/Sales/view/frontend/templates/order/print
  • dev/tests/integration/testsuite/Magento/Sales/Block/Order/PrintOrder

2 files changed

+38
-24
lines changed

‎app/code/Magento/Sales/view/frontend/templates/order/print/shipment.phtml

Lines changed: 37 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -3,26 +3,36 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6+
7+
/**
8+
* @var \Magento\Sales\Block\Order\PrintOrder\Shipment $block
9+
* @var \Magento\Framework\Escaper $escaper
10+
*/
611
?>
7-
<?php /* @var $block \Magento\Sales\Block\Order\PrintOrder\Shipment */ ?>
812
<?php $order = $block->getOrder(); ?>
9-
<?php if (!$block->getObjectData($order, 'is_virtual')) : ?>
10-
<?php foreach ($block->getShipmentsCollection() as $shipment) : ?>
13+
<?php if (!$block->getObjectData($order, 'is_virtual')): ?>
14+
<?php foreach ($block->getShipmentsCollection() as $shipment): ?>
1115
<div class="order-details-items shipments">
1216
<div class="order-title">
13-
<strong><?= $block->escapeHtml(__('Shipment #%1', $block->getObjectData($shipment, 'increment_id'))) ?></strong>
17+
<strong><?= $escaper->escapeHtml(
18+
__(
19+
'Shipment #%1',
20+
$block->getObjectData($shipment, 'increment_id')
21+
)
22+
) ?></strong>
1423
</div>
1524
<div class="table-wrapper order-items-shipment">
16-
<table class="data table table-order-items shipment" id="my-shipment-table-<?= (int) $block->getObjectData($shipment, 'id') ?>">
17-
<caption class="table-caption"><?= $block->escapeHtml(__('Items Invoiced')) ?></caption>
25+
<table class="data table table-order-items shipment"
26+
id="my-shipment-table-<?= (int)$block->getObjectData($shipment, 'id') ?>">
27+
<caption class="table-caption"><?= $escaper->escapeHtml(__('Items Invoiced')) ?></caption>
1828
<thead>
19-
<tr>
20-
<th class="col name"><?= $block->escapeHtml(__('Product Name')) ?></th>
21-
<th class="col sku"><?= $block->escapeHtml(__('SKU')) ?></th>
22-
<th class="col price"><?= $block->escapeHtml(__('Qty Shipped')) ?></th>
23-
</tr>
29+
<tr>
30+
<th class="col name"><?= $escaper->escapeHtml(__('Product Name')) ?></th>
31+
<th class="col sku"><?= $escaper->escapeHtml(__('SKU')) ?></th>
32+
<th class="col price"><?= $escaper->escapeHtml(__('Qty Shipped')) ?></th>
33+
</tr>
2434
</thead>
25-
<?php foreach ($block->getShipmentItems($shipment) as $item) : ?>
35+
<?php foreach ($block->getShipmentItems($shipment) as $item): ?>
2636
<tbody>
2737
<?= $block->getItemHtml($item) ?>
2838
</tbody>
@@ -31,12 +41,12 @@
3141
</div>
3242
<div class="block block-order-details-view">
3343
<div class="block-title">
34-
<strong><?= $block->escapeHtml(__('Order Information')) ?></strong>
44+
<strong><?= $escaper->escapeHtml(__('Order Information')) ?></strong>
3545
</div>
3646
<div class="block-content">
3747
<div class="box box-order-shipping-address">
3848
<div class="box-title">
39-
<strong><?= $block->escapeHtml(__('Shipping Address')) ?></strong>
49+
<strong><?= $escaper->escapeHtml(__('Shipping Address')) ?></strong>
4050
</div>
4151
<div class="box-content">
4252
<address><?= $block->getShipmentAddressFormattedHtml($shipment) ?></address>
@@ -45,25 +55,29 @@
4555

4656
<div class="box box-order-shipping-method">
4757
<div class="box-title">
48-
<strong><?= $block->escapeHtml(__('Shipping Method')) ?></strong>
58+
<strong><?= $escaper->escapeHtml(__('Shipping Method')) ?></strong>
4959
</div>
5060
<div class="box-content">
51-
<?= $block->escapeHtml($block->getObjectData($order, 'shipping_description')) ?>
61+
<?= $escaper->escapeHtml($block->getObjectData($order, 'shipping_description')) ?>
5262
<?php $tracks = $block->getShipmentTracks($shipment);
53-
if ($tracks) : ?>
63+
if ($tracks): ?>
5464
<dl class="order-tracking">
55-
<?php foreach ($tracks as $track) : ?>
56-
<dt class="tracking-title"><?= $block->escapeHtml($block->getObjectData($track, 'title')) ?></dt>
57-
<dd class="tracking-content"><?= $block->escapeHtml($block->getObjectData($track, 'number')) ?></dd>
65+
<?php foreach ($tracks as $track): ?>
66+
<dt class="tracking-title">
67+
<?= $escaper->escapeHtml($block->getObjectData($track, 'title')) ?>
68+
</dt>
69+
<dd class="tracking-content">
70+
<?= $escaper->escapeHtml($block->getObjectData($track, 'number')) ?>
71+
</dd>
5872
<?php endforeach; ?>
5973
</dl>
6074
<?php endif; ?>
6175
</div>
6276
</div>
6377

64-
<div class="box box-order-billing-method">
78+
<div class="box box-order-billing-address">
6579
<div class="box-title">
66-
<strong><?= $block->escapeHtml(__('Billing Address')) ?></strong>
80+
<strong><?= $escaper->escapeHtml(__('Billing Address')) ?></strong>
6781
</div>
6882
<div class="box-content">
6983
<address><?= $block->getBillingAddressFormattedHtml($order) ?></address>
@@ -72,7 +86,7 @@
7286

7387
<div class="box box-order-billing-method">
7488
<div class="box-title">
75-
<strong><?= $block->escapeHtml(__('Payment Method')) ?></strong>
89+
<strong><?= $escaper->escapeHtml(__('Payment Method')) ?></strong>
7690
</div>
7791
<div class="box-content">
7892
<?= $block->getPaymentInfoHtml() ?>

‎dev/tests/integration/testsuite/Magento/Sales/Block/Order/PrintOrder/ShipmentTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ private function assertOrderInformation(OrderInterface $order, string $html): vo
151151
foreach ([$order->getShippingAddress(), $order->getBillingAddress()] as $address) {
152152
$addressBoxXpath = ($address->getAddressType() == 'shipping')
153153
? "//div[contains(@class, 'box-order-shipping-address')]//address[contains(., '%s')]"
154-
: "//div[contains(@class, 'box-order-billing-method')]//address[contains(., '%s')]";
154+
: "//div[contains(@class, 'box-order-billing-address')]//address[contains(., '%s')]";
155155
$this->assertEquals(
156156
1,
157157
Xpath::getElementsCountForXpath(sprintf($addressBoxXpath, $address->getName()), $html),

0 commit comments

Comments
 (0)