Skip to content

Commit 9602a92

Browse files
author
Igor Melnikov
committed
MAGETWO-67871: Requesting autogenerated classes that are not in constructor cause fatal errors in production mode
- refactor
1 parent b599abd commit 9602a92

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

app/code/Magento/CatalogSearch/Model/Adapter/Mysql/Aggregation/DataProvider.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ public function __construct(
8989
$this->scopeResolver = $scopeResolver;
9090
$this->customerSession = $customerSession;
9191
$this->indexerFrontendResource = $indexerFrontendResource ?: ObjectManager::getInstance()->get(
92-
Magento\Catalog\Model\ResourceModel\Product\Indexer\Price\FrontendResource::class
92+
\Magento\Catalog\Model\ResourceModel\Product\Indexer\Price\FrontendResource::class
9393
);
9494
$this->indexerStockFrontendResource = $indexerStockFrontendResource ?: ObjectManager::getInstance()
9595
->get(\Magento\CatalogInventory\Model\ResourceModel\Indexer\Stock\FrontendResource::class);

dev/tests/static/testsuite/Magento/Test/Legacy/AutogeneratedClassNotInConstructorTest.php

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,13 +43,14 @@ protected function setUp()
4343

4444
public function testAutogeneratedClassesRequestedInConstructor()
4545
{
46-
$fileTypes = Files::INCLUDE_APP_CODE | Files::INCLUDE_LIBS | Files::INCLUDE_SETUP;
47-
$changedFiles = ChangedFiles::getPhpFiles(__DIR__ . '/../_files/changed_files*', $fileTypes);
46+
$fileTypes = Files::INCLUDE_APP_CODE | Files::INCLUDE_LIBS | Files::INCLUDE_SETUP | Files::AS_DATA_SET;
47+
$changedFiles = ChangedFiles::getPhpFiles(__DIR__ . '/_files/changed_files*', $fileTypes);
4848
$phpFiles = $this->fileUtilities->getPhpFiles($fileTypes);
4949

5050
$existingClasses = [];
5151
$classesCreatedByObjectManager = [];
52-
foreach ($changedFiles as $filePath) {
52+
foreach ($phpFiles as $file) {
53+
$filePath = $file[0];
5354
$fileContent = file_get_contents($filePath);
5455
$className = $this->classNameExtractor->getNameWithNamespace($fileContent);
5556
if ($className) {
@@ -59,7 +60,8 @@ public function testAutogeneratedClassesRequestedInConstructor()
5960
);
6061
}
6162
}
62-
foreach ($phpFiles as $filePath) {
63+
foreach ($changedFiles as $file) {
64+
$filePath = $file[0];
6365
$fileContent = file_get_contents($filePath);
6466
$className = $this->classNameExtractor->getNameWithNamespace($fileContent);
6567
if ($className) {

0 commit comments

Comments
 (0)