diff --git a/app/code/Magento/CatalogImportExport/Test/Unit/Model/Import/Product/Type/OptionTest.php b/app/code/Magento/CatalogImportExport/Test/Unit/Model/Import/Product/Type/OptionTest.php index c4c81329267f4887789e3aa2fd562873205a9c22..1d141e036a2772e9d151dc050f1f9563d93948e8 100644 --- a/app/code/Magento/CatalogImportExport/Test/Unit/Model/Import/Product/Type/OptionTest.php +++ b/app/code/Magento/CatalogImportExport/Test/Unit/Model/Import/Product/Type/OptionTest.php @@ -289,8 +289,7 @@ class OptionTest extends \Magento\ImportExport\Test\Unit\Model\Import\AbstractIm ]; $modelClassName = \Magento\CatalogImportExport\Model\Import\Product\Option::class; - $class = new \ReflectionClass($modelClassName); - $this->model = $class->newInstanceArgs($modelClassArgs); + $this->model = new $modelClassName(...$modelClassArgs); // Create model mock with rewritten _getMultiRowFormat method to support test data with the old format. $this->modelMock = $this->getMockBuilder($modelClassName) ->setConstructorArgs($modelClassArgs) diff --git a/lib/internal/Magento/Framework/Filter/Input.php b/lib/internal/Magento/Framework/Filter/Input.php index 3008700b096426413f18beeacbfdd2d7ee7269ee..83c67cddf4f2ae7dadca9e46a96bb83ba2790b0e 100644 --- a/lib/internal/Magento/Framework/Filter/Input.php +++ b/lib/internal/Magento/Framework/Filter/Input.php @@ -342,16 +342,13 @@ class Input implements \Zend_Filter_Interface { $filter = $filterData['zend']; if (is_string($filter)) { - $class = new \ReflectionClass('Zend_Filter_' . $filter); - if ($class->implementsInterface('Zend_Filter_Interface')) { - if (isset($filterData['args']) && $class->hasMethod('__construct')) { - $filter = $class->newInstanceArgs($filterData['args']); - } else { - $filter = $class->newInstance(); - } - } else { - throw new \Exception('Filter is not instance of \Zend_Filter_Interface'); - } + $filterClassName = 'Zend_Filter_' . ucfirst($filter); + $filterClassOptions = isset($filterData['args']) ? $filterData['args'] : []; + $filter = new $filterClassName(...$filterClassOptions); + } + + if (!$filter instanceof \Zend_Filter_Interface) { + throw new \Exception('Filter is not instance of \Zend_Filter_Interface'); } return $filter; } diff --git a/setup/src/Magento/Setup/Test/Unit/Module/I18n/Dictionary/PhraseTest.php b/setup/src/Magento/Setup/Test/Unit/Module/I18n/Dictionary/PhraseTest.php index b04d02b5084b751b8f259ee170cdff7adfe3407f..4e4833e8617a88b55ef6d046f361ddd1c5255b2d 100644 --- a/setup/src/Magento/Setup/Test/Unit/Module/I18n/Dictionary/PhraseTest.php +++ b/setup/src/Magento/Setup/Test/Unit/Module/I18n/Dictionary/PhraseTest.php @@ -17,8 +17,7 @@ class PhraseTest extends \PHPUnit_Framework_TestCase */ public function testPhraseCreation($constructArguments, $getter, $result) { - $reflectionClass = new \ReflectionClass('Magento\Setup\Module\I18n\Dictionary\Phrase'); - $phrase = $reflectionClass->newInstanceArgs($constructArguments); + $phrase = new Phrase(...$constructArguments); $this->assertEquals($result, $phrase->{$getter}()); } @@ -58,8 +57,7 @@ class PhraseTest extends \PHPUnit_Framework_TestCase { $this->setExpectedException('DomainException', $message); - $reflectionClass = new \ReflectionClass('Magento\Setup\Module\I18n\Dictionary\Phrase'); - $reflectionClass->newInstanceArgs($constructArguments); + new Phrase(...$constructArguments); } /**