Skip to content

Commit fd021f6

Browse files
committed
MAGETWO-89990: Multishipping: server side extension point (MAGETWO-84587)
- Fixed composer dependency version - Updated FAT assert - Updated modules dependencies - Added copyrights - Removed unused dependencies
1 parent 4461889 commit fd021f6

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed

app/code/Magento/Multishipping/Block/Checkout/Results.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,15 @@
55
*/
66
namespace Magento\Multishipping\Block\Checkout;
77

8+
use Magento\Customer\Model\Address\Config as AddressConfig;
89
use Magento\Framework\Exception\LocalizedException;
10+
use Magento\Framework\Session\SessionManagerInterface;
911
use Magento\Framework\View\Element\Template\Context;
1012
use Magento\Multishipping\Model\Checkout\Type\Multishipping;
11-
use Magento\Sales\Api\OrderRepositoryInterface;
12-
use Magento\Customer\Model\Address\Config as AddressConfig;
1313
use Magento\Quote\Model\Quote\Address as QuoteAddress;
14+
use Magento\Sales\Api\OrderRepositoryInterface;
1415
use Magento\Sales\Model\Order\Address as OrderAddress;
15-
use Magento\Framework\Session\SessionManagerInterface;
16+
use Magento\Theme\Block\Html\Title;
1617

1718
/**
1819
* Multi-shipping checkout results information
@@ -177,6 +178,7 @@ public function getAddressError(QuoteAddress $address): string
177178
*/
178179
protected function _prepareLayout(): Success
179180
{
181+
/** @var Title $pageTitle */
180182
$pageTitle = $this->getLayout()->getBlock('page.main.title');
181183
if ($pageTitle) {
182184
$title = $this->getOrderIds() ? $pageTitle->getPartlySuccessTitle() : $pageTitle->getFailedTitle();

app/code/Magento/Multishipping/composer.json

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,7 @@
1414
"magento/module-quote": "*",
1515
"magento/module-sales": "*",
1616
"magento/module-store": "*",
17-
"magento/module-tax": "*"
18-
},
19-
"suggest": {
17+
"magento/module-tax": "*",
2018
"magento/module-theme": "*"
2119
},
2220
"type": "magento2-module",

dev/tests/functional/tests/app/Magento/Multishipping/Test/Constraint/AssertMultishippingOrderSuccessPlacedMessage.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ class AssertMultishippingOrderSuccessPlacedMessage extends AbstractConstraint
2121
/**
2222
* Expected success message
2323
*/
24-
const SUCCESS_MESSAGE = 'We received your order!';
24+
const SUCCESS_MESSAGE = 'Thank you for your purchase!';
2525

2626
/**
2727
* Assert that success message is correct.

dev/tests/integration/testsuite/Magento/Multishipping/Model/Checkout/Type/MultishippingTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525

2626
/**
2727
* @magentoAppArea frontend
28+
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
2829
*/
2930
class MultishippingTest extends \PHPUnit\Framework\TestCase
3031
{

0 commit comments

Comments
 (0)