diff --git a/app/code/Magento/Shipping/Model/Config/Source/Allmethods.php b/app/code/Magento/Shipping/Model/Config/Source/Allmethods.php index bafb9ed49cf30..f64c24856eba5 100644 --- a/app/code/Magento/Shipping/Model/Config/Source/Allmethods.php +++ b/app/code/Magento/Shipping/Model/Config/Source/Allmethods.php @@ -5,6 +5,9 @@ */ namespace Magento\Shipping\Model\Config\Source; +/** + * @inheritdoc + */ class Allmethods implements \Magento\Framework\Option\ArrayInterface { /** @@ -33,6 +36,7 @@ public function __construct( /** * Return array of carriers. + * * If $isActiveOnlyFlag is set to true, will return only active carriers * * @param bool $isActiveOnlyFlag @@ -59,7 +63,7 @@ public function toOptionArray($isActiveOnlyFlag = false) /** Check it $carrierMethods array was well formed */ if (!$methodCode) { - continue; + continue; } $methods[$carrierCode]['value'][] = [ 'value' => $carrierCode . '_' . $methodCode, diff --git a/app/code/Magento/Shipping/Test/Unit/Model/Config/Source/AllmethodsTest.php b/app/code/Magento/Shipping/Test/Unit/Model/Config/Source/AllmethodsTest.php index b3810b6042eee..985cc0e53bad5 100644 --- a/app/code/Magento/Shipping/Test/Unit/Model/Config/Source/AllmethodsTest.php +++ b/app/code/Magento/Shipping/Test/Unit/Model/Config/Source/AllmethodsTest.php @@ -3,7 +3,7 @@ * Copyright © Magento, Inc. All rights reserved. * See COPYING.txt for license details. */ -namespace Magento\Shipping\Test\Unit\Model; +namespace Magento\Shipping\Test\Unit\Model\Config\Source; use Magento\Framework\App\Config\ScopeConfigInterface; use Magento\Shipping\Model\Carrier\AbstractCarrierInterface; @@ -14,7 +14,7 @@ /** * Tests for Allmethods Class */ -class InfoTest extends \PHPUnit\Framework\TestCase +class AllmethodsTest extends \PHPUnit\Framework\TestCase { /** * @var ScopeConfigInterface|MockObject $scopeConfig