Skip to content

Commit 1dbd60c

Browse files
committed
AC-15140 : When Gift message enabled on Order level but user does not enter any data and place order, then still From Name and To Name in Admin showing customer first and last name.
1 parent 4801c4a commit 1dbd60c

File tree

1 file changed

+33
-20
lines changed

1 file changed

+33
-20
lines changed

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

Lines changed: 33 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,55 +1,68 @@
11
<?php
22
/**
3-
* Copyright © Magento, Inc. All rights reserved.
4-
* See COPYING.txt for license details.
3+
* Copyright 2011 Adobe
4+
* All Rights Reserved.
55
*/
66

77
/** @var $block \Magento\Sales\Block\Adminhtml\Order\View\Giftmessage */
88
?>
9-
<?php if ($block->canDisplayGiftmessage()) : ?>
9+
<?php if ($block->canDisplayGiftmessage()): ?>
1010
<?php $_required = $block->getMessage()->getMessage() != '' ?>
11-
<div id="<?= $block->escapeHtmlAttr($block->getHtmlId()) ?>" class="admin__page-section-content giftmessage-whole-order-container">
12-
<form class="entry-edit form-inline" id="<?= $block->escapeHtmlAttr($block->getFieldId('form')) ?>" action="<?= $block->escapeUrl($block->getSaveUrl()) ?>">
11+
<div id="<?= $escaper->escapeHtmlAttr($block->getHtmlId()) ?>"
12+
class="admin__page-section-content giftmessage-whole-order-container">
13+
<form class="entry-edit form-inline"
14+
id="<?= $escaper->escapeHtmlAttr($block->getFieldId('form')) ?>"
15+
action="<?= $escaper->escapeUrl($block->getSaveUrl()) ?>">
1316
<fieldset class="admin__fieldset">
14-
<legend class="admin__legend"><span><?= $block->escapeHtml(__('Gift Message for the Entire Order')) ?></span></legend>
17+
<legend class="admin__legend">
18+
<span><?= $escaper->escapeHtml(__('Gift Message for the Entire Order')) ?></span>
19+
</legend>
1520
<br/>
1621

17-
<input type="hidden" id="<?= $block->escapeHtmlAttr($block->getFieldId('type')) ?>"
18-
name="<?= $block->escapeHtmlAttr($block->getFieldName('type')) ?>"
22+
<input type="hidden" id="<?= $escaper->escapeHtmlAttr($block->getFieldId('type')) ?>"
23+
name="<?= $escaper->escapeHtmlAttr($block->getFieldName('type')) ?>"
1924
value="order"/>
2025

2126
<div class="admin__field field-from-name<?= $_required ? ' required' : '' ?>">
2227
<label class="admin__field-label"
23-
for="<?= $block->escapeHtmlAttr($block->getFieldId('sender')) ?>"><span><?= $block->escapeHtml(__('From Name')) ?></span></label>
28+
for="<?= $escaper->escapeHtmlAttr($block->getFieldId('sender')) ?>">
29+
<span><?= $escaper->escapeHtml(__('From Name')) ?></span>
30+
</label>
2431

2532
<div class="admin__field-control">
2633
<input class="admin__control-text <?= $_required ? 'required-entry' : '' ?>" type="text"
27-
id="<?= $block->escapeHtmlAttr($block->getFieldId('sender')) ?>"
28-
name="<?= $block->escapeHtmlAttr($block->getFieldName('sender')) ?>"
29-
value="<?= $_required ? $block->escapeHtml($block->getMessage()->getSender()):'' ?>"/>
34+
id="<?= $escaper->escapeHtmlAttr($block->getFieldId('sender')) ?>"
35+
name="<?= $escaper->escapeHtmlAttr($block->getFieldName('sender')) ?>"
36+
value="<?= $_required ? $escaper->escapeHtml($block->getMessage()->getSender()):'' ?>"/>
3037
</div>
3138
</div>
3239

3340
<div class="admin__field field-to-name<?= $_required ? ' required' : '' ?>">
3441
<label class="admin__field-label"
35-
for="<?= $block->escapeHtmlAttr($block->getFieldId('recipient')) ?>"><span><?= $block->escapeHtml(__('To Name')) ?></span></label>
42+
for="<?= $escaper->escapeHtmlAttr($block->getFieldId('recipient')) ?>">
43+
<span><?= $escaper->escapeHtml(__('To Name')) ?></span>
44+
</label>
3645

3746
<div class="admin__field-control">
3847
<input class="admin__control-text <?= $_required ? 'required-entry' : '' ?>" type="text"
39-
id="<?= $block->escapeHtmlAttr($block->getFieldId('recipient')) ?>"
40-
name="<?= $block->escapeHtmlAttr($block->getFieldName('recipient')) ?>"
41-
value="<?= $_required ? $block->escapeHtml($block->getMessage()->getRecipient()):'' ?>"/>
48+
id="<?= $escaper->escapeHtmlAttr($block->getFieldId('recipient')) ?>"
49+
name="<?= $escaper->escapeHtmlAttr($block->getFieldName('recipient')) ?>"
50+
value="<?= $_required ?
51+
$escaper->escapeHtml($block->getMessage()->getRecipient()):'' ?>"
52+
/>
4253
</div>
4354
</div>
4455
<div class="admin__field field-gift-message">
4556
<label class="admin__field-label"
46-
for="<?= $block->escapeHtmlAttr($block->getFieldId('message')) ?>"><span><?= $block->escapeHtml(__('Gift Message')) ?></span></label>
57+
for="<?= $escaper->escapeHtmlAttr($block->getFieldId('message')) ?>">
58+
<span><?= $escaper->escapeHtml(__('Gift Message')) ?></span>
59+
</label>
4760
<div class="admin__field-control">
48-
<textarea id="<?= $block->escapeHtmlAttr($block->getFieldId('message')) ?>"
49-
name="<?= $block->escapeHtmlAttr($block->getFieldName('message')) ?>"
61+
<textarea id="<?= $escaper->escapeHtmlAttr($block->getFieldId('message')) ?>"
62+
name="<?= $escaper->escapeHtmlAttr($block->getFieldName('message')) ?>"
5063
class="admin__control-textarea"
5164
rows="2"
52-
cols="15"><?= $block->escapeHtml($block->getMessage()->getMessage()) ?></textarea>
65+
cols="15"><?= $escaper->escapeHtml($block->getMessage()->getMessage()) ?></textarea>
5366
</div>
5467
</div>
5568

0 commit comments

Comments
 (0)