Skip to content

Commit 677f66e

Browse files
committed
Merge remote-tracking branch 'magento-mpi/MC-21459' into MPI-PR
2 parents 63e1804 + 0a308f0 commit 677f66e

File tree

6 files changed

+43
-28
lines changed

6 files changed

+43
-28
lines changed

app/code/Magento/OfflinePayments/view/adminhtml/templates/info/checkmo.phtml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,9 @@
77
/**
88
* @var $block \Magento\OfflinePayments\Block\Info\Checkmo
99
*/
10+
$paymentTitle = $block->getMethod()->getConfigData('title', $block->getInfo()->getOrder()->getStoreId());
1011
?>
11-
<?= $block->escapeHtml($block->getMethod()->getTitle()) ?>
12+
<?= $block->escapeHtml($paymentTitle) ?>
1213
<?php if ($block->getInfo()->getAdditionalInformation()) : ?>
1314
<?php if ($block->getPayableTo()) : ?>
1415
<br /><?= $block->escapeHtml(__('Make Check payable to: %1', $block->getPayableTo())) ?>

app/code/Magento/OfflinePayments/view/adminhtml/templates/info/pdf/checkmo.phtml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,9 @@
77
/**
88
* @var $block \Magento\OfflinePayments\Block\Info\Checkmo
99
*/
10+
$paymentTitle = $block->getMethod()->getConfigData('title', $block->getInfo()->getOrder()->getStoreId());
1011
?>
11-
<?= $block->escapeHtml($block->getMethod()->getTitle()) ?>
12+
<?= $block->escapeHtml($paymentTitle) ?>
1213
{{pdf_row_separator}}
1314
<?php if ($block->getInfo()->getAdditionalInformation()) : ?>
1415
{{pdf_row_separator}}

app/code/Magento/OfflinePayments/view/adminhtml/templates/info/purchaseorder.phtml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,9 @@
66
/**
77
* @var $block \Magento\OfflinePayments\Block\Info\Purchaseorder
88
*/
9+
$paymentTitle = $block->getMethod()->getConfigData('title', $block->getInfo()->getOrder()->getStoreId());
910
?>
10-
<div class="order-payment-method-name"><?= $block->escapeHtml($block->getMethod()->getTitle()) ?></div>
11+
<div class="order-payment-method-name"><?= $block->escapeHtml($paymentTitle) ?></div>
1112
<table class="data-table admin__table-secondary">
1213
<tr>
1314
<th><?= $block->escapeHtml(__('Purchase Order Number')) ?>:</th>

app/code/Magento/Payment/view/adminhtml/templates/info/default.phtml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,9 @@
99
* @see \Magento\Payment\Block\Info
1010
*/
1111
$specificInfo = $block->getSpecificInformation();
12+
$paymentTitle = $block->getMethod()->getConfigData('title', $block->getInfo()->getOrder()->getStoreId());
1213
?>
13-
<?= $block->escapeHtml($block->getMethod()->getTitle()) ?>
14+
<?= $block->escapeHtml($paymentTitle) ?>
1415

1516
<?php if ($specificInfo) : ?>
1617
<table class="data-table admin__table-secondary">

app/code/Magento/Payment/view/adminhtml/templates/info/pdf/default.phtml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,9 @@
88
* @see \Magento\Payment\Block\Info
99
* @var \Magento\Payment\Block\Info $block
1010
*/
11+
$paymentTitle = $block->getMethod()->getConfigData('title', $block->getInfo()->getOrder()->getStoreId());
1112
?>
12-
<?= $block->escapeHtml($block->getMethod()->getTitle()) ?>{{pdf_row_separator}}
13+
<?= $block->escapeHtml($paymentTitle) ?>{{pdf_row_separator}}
1314

1415
<?php if ($specificInfo = $block->getSpecificInformation()) : ?>
1516
<?php foreach ($specificInfo as $label => $value) : ?>

dev/tests/integration/testsuite/Magento/Payment/Block/InfoTest.php

Lines changed: 33 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -5,47 +5,57 @@
55
*/
66
namespace Magento\Payment\Block;
77

8+
use Magento\Framework\View\Element\Text;
9+
use Magento\Framework\View\LayoutInterface;
10+
use Magento\OfflinePayments\Model\Banktransfer;
11+
use Magento\OfflinePayments\Model\Checkmo;
12+
use Magento\Payment\Block\Info as BlockInfo;
13+
use Magento\Payment\Block\Info\Instructions;
14+
use Magento\Payment\Model\Info;
15+
use Magento\Sales\Model\Order;
16+
use Magento\TestFramework\Helper\Bootstrap;
17+
18+
/**
19+
* Class InfoTest
20+
*/
821
class InfoTest extends \PHPUnit\Framework\TestCase
922
{
1023
/**
24+
* Tests payment info block.
25+
*
1126
* @magentoConfigFixture current_store payment/banktransfer/title Bank Method Title
1227
* @magentoConfigFixture current_store payment/checkmo/title Checkmo Title Of The Method
1328
* @magentoAppArea adminhtml
1429
*/
1530
public function testGetChildPdfAsArray()
1631
{
1732
/** @var $layout \Magento\Framework\View\Layout */
18-
$layout = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->get(
19-
\Magento\Framework\View\LayoutInterface::class
20-
);
21-
$block = $layout->createBlock(\Magento\Payment\Block\Info::class, 'block');
33+
$layout = Bootstrap::getObjectManager()->get(LayoutInterface::class);
34+
$block = $layout->createBlock(BlockInfo::class, 'block');
2235

23-
/** @var $paymentInfoBank \Magento\Payment\Model\Info */
24-
$paymentInfoBank = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->create(
25-
\Magento\Payment\Model\Info::class
26-
);
27-
$paymentInfoBank->setMethodInstance(
28-
\Magento\TestFramework\Helper\Bootstrap::getObjectManager()->create(
29-
\Magento\OfflinePayments\Model\Banktransfer::class
30-
)
36+
/** @var $paymentInfoBank Info */
37+
$paymentInfoBank = Bootstrap::getObjectManager()->create(
38+
Info::class
3139
);
32-
/** @var $childBank \Magento\Payment\Block\Info\Instructions */
33-
$childBank = $layout->addBlock(\Magento\Payment\Block\Info\Instructions::class, 'child.one', 'block');
40+
$order = Bootstrap::getObjectManager()->create(Order::class);
41+
$banktransferPayment = Bootstrap::getObjectManager()->create(Banktransfer::class);
42+
$paymentInfoBank->setMethodInstance($banktransferPayment);
43+
$paymentInfoBank->setOrder($order);
44+
/** @var $childBank Instructions */
45+
$childBank = $layout->addBlock(Instructions::class, 'child.one', 'block');
3446
$childBank->setInfo($paymentInfoBank);
3547

3648
$nonExpectedHtml = 'non-expected html';
37-
$childHtml = $layout->addBlock(\Magento\Framework\View\Element\Text::class, 'child.html', 'block');
49+
$childHtml = $layout->addBlock(Text::class, 'child.html', 'block');
3850
$childHtml->setText($nonExpectedHtml);
3951

40-
/** @var $paymentInfoCheckmo \Magento\Payment\Model\Info */
41-
$paymentInfoCheckmo = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->create(
42-
\Magento\Payment\Model\Info::class
43-
);
44-
$paymentInfoCheckmo->setMethodInstance(
45-
\Magento\TestFramework\Helper\Bootstrap::getObjectManager()->create(
46-
\Magento\OfflinePayments\Model\Checkmo::class
47-
)
52+
/** @var $paymentInfoCheckmo Info */
53+
$paymentInfoCheckmo = Bootstrap::getObjectManager()->create(
54+
Info::class
4855
);
56+
$checkmoPayment = Bootstrap::getObjectManager()->create(Checkmo::class);
57+
$paymentInfoCheckmo->setMethodInstance($checkmoPayment);
58+
$paymentInfoCheckmo->setOrder($order);
4959
/** @var $childCheckmo \Magento\OfflinePayments\Block\Info\Checkmo */
5060
$childCheckmo = $layout->addBlock(
5161
\Magento\OfflinePayments\Block\Info\Checkmo::class,

0 commit comments

Comments
 (0)