Revert "no message"

This reverts commit 6c11aeb.

Author: basvanpoppel

app/code/Magento/CatalogSearch/Model/ResourceModel/Advanced/Collection.php

@@ -6,30 +6,27 @@
 
 namespace Magento\CatalogSearch\Model\ResourceModel\Advanced;
 
-use Magento\Catalog\Model\Category;
 use Magento\Catalog\Model\Product;
-use Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitationFactory;
-use Magento\CatalogSearch\Model\ResourceModel\Advanced;
 use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\DefaultFilterStrategyApplyChecker;
 use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\DefaultFilterStrategyApplyCheckerInterface;
-use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\SearchCriteriaResolverFactory;
 use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\SearchCriteriaResolverInterface;
-use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\SearchResultApplierFactory;
 use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\SearchResultApplierInterface;
-use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\TotalRecordsResolverFactory;
+use Magento\Framework\Search\EngineResolverInterface;
 use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\TotalRecordsResolverInterface;
-use Magento\Eav\Model\Entity\Attribute\AbstractAttribute;
+use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\TotalRecordsResolverFactory;
 use Magento\Framework\Api\FilterBuilder;
+use Magento\Framework\DB\Select;
 use Magento\Framework\Api\Search\SearchCriteriaBuilder;
 use Magento\Framework\Api\Search\SearchResultFactory;
-use Magento\Framework\Api\Search\SearchResultInterface;
-use Magento\Framework\App\ObjectManager;
-use Magento\Framework\DB\Select;
 use Magento\Framework\EntityManager\MetadataPool;
 use Magento\Framework\Exception\LocalizedException;
-use Magento\Framework\Search\EngineResolverInterface;
 use Magento\Framework\Search\Request\EmptyRequestDataException;
 use Magento\Framework\Search\Request\NonExistingRequestNameException;
+use Magento\Catalog\Model\ResourceModel\Product\Collection\ProductLimitationFactory;
+use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\SearchCriteriaResolverFactory;
+use Magento\CatalogSearch\Model\ResourceModel\Fulltext\Collection\SearchResultApplierFactory;
+use Magento\Framework\App\ObjectManager;
+use Magento\Framework\Api\Search\SearchResultInterface;
 
 /**
  * Advanced search collection
@@ -109,11 +106,6 @@ class Collection extends \Magento\Catalog\Model\ResourceModel\Product\Collection
      */
     private $defaultFilterStrategyApplyChecker;
 
-    /**
-     * @var Advanced
-     */
-    private $advancedSearchResource;
-
     /**
      * Collection constructor
      *
@@ -149,7 +141,6 @@ class Collection extends \Magento\Catalog\Model\ResourceModel\Product\Collection
      * @param TotalRecordsResolverFactory|null $totalRecordsResolverFactory
      * @param EngineResolverInterface|null $engineResolver
      * @param DefaultFilterStrategyApplyCheckerInterface|null $defaultFilterStrategyApplyChecker
-     * @param Advanced|null $advancedSearchResource
      * @SuppressWarnings(PHPMD.ExcessiveParameterList)
      * @SuppressWarnings(PHPMD.UnusedFormalParameter)
      */
@@ -185,8 +176,7 @@ public function __construct(
         SearchResultApplierFactory $searchResultApplierFactory = null,
         TotalRecordsResolverFactory $totalRecordsResolverFactory = null,
         EngineResolverInterface $engineResolver = null,
-        DefaultFilterStrategyApplyCheckerInterface $defaultFilterStrategyApplyChecker = null,
-        Advanced $advancedSearchResource = null
+        DefaultFilterStrategyApplyCheckerInterface $defaultFilterStrategyApplyChecker = null
     ) {
         $this->searchRequestName = $searchRequestName;
         if ($searchResultFactory === null) {
@@ -203,8 +193,6 @@ public function __construct(
             ->get(EngineResolverInterface::class);
         $this->defaultFilterStrategyApplyChecker = $defaultFilterStrategyApplyChecker ?: ObjectManager::getInstance()
             ->get(DefaultFilterStrategyApplyChecker::class);
-        $this->advancedSearchResource = $advancedSearchResource ?: ObjectManager::getInstance()
-            ->get(Advanced::class);
         parent::__construct(
             $entityFactory,
             $logger,
@@ -270,14 +258,14 @@ public function setOrder($attribute, $dir = Select::SQL_DESC)
      */
     public function addCategoryFilter(\Magento\Catalog\Model\Category $category)
     {
-        $this->setAttributeFilterData(Category::ENTITY, 'category_ids', $category->getId());
         /**
          * This changes need in backward compatible reasons for support dynamic improved algorithm
          * for price aggregation process.
          */
         if ($this->defaultFilterStrategyApplyChecker->isApplicable()) {
             parent::addCategoryFilter($category);
         } else {
+            $this->addFieldToFilter('category_ids', $category->getId());
             $this->_productLimitationPrice();
         }
 
@@ -290,13 +278,14 @@ public function addCategoryFilter(\Magento\Catalog\Model\Category $category)
      */
     public function setVisibility($visibility)
     {
-        $this->setAttributeFilterData(Product::ENTITY, 'visibility', $visibility);
         /**
          * This changes need in backward compatible reasons for support dynamic improved algorithm
          * for price aggregation process.
          */
         if ($this->defaultFilterStrategyApplyChecker->isApplicable()) {
             parent::setVisibility($visibility);
+        } else {
+            $this->addFieldToFilter('visibility', $visibility);
         }
 
         return $this;
@@ -317,25 +306,6 @@ private function setSearchOrder($field, $direction)
         $this->searchOrders[$field] = $direction;
     }
 
-    /**
-     * Prepare attribute data to filter.
-     *
-     * @param string $entityType
-     * @param string $attributeCode
-     * @param mixed $condition
-     * @return $this
-     */
-    private function setAttributeFilterData(string $entityType, string $attributeCode, $condition): self
-    {
-        /** @var AbstractAttribute $attribute */
-        $attribute = $this->_eavConfig->getAttribute($entityType, $attributeCode);
-        $table = $attribute->getBackend()->getTable();
-        $condition = $this->advancedSearchResource->prepareCondition($attribute, $condition);
-        $this->addFieldsToFilter([$table => [$attributeCode => $condition]]);
-
-        return $this;
-    }
-
     /**
      * @inheritdoc
      */
@@ -407,7 +377,7 @@ public function _loadEntities($printQuery = false, $logQuery = false)
             $query = $this->getSelect();
             $rows = $this->_fetchAll($query);
         } catch (\Exception $e) {
-            $this->printLogQuery(false, true, $query ?? null);
+            $this->printLogQuery(false, true, $query);
             throw $e;
         }
 

app/code/Magento/CatalogSearch/etc/search_request.xml

@@ -66,7 +66,6 @@
                 <queryReference clause="should" ref="sku_query"/>
                 <queryReference clause="should" ref="price_query"/>
                 <queryReference clause="should" ref="category_query"/>
-                <queryReference clause="must" ref="visibility_query"/>
             </query>
             <query name="sku_query" xsi:type="filteredQuery">
                 <filterReference clause="must" ref="sku_query_filter"/>
@@ -77,15 +76,11 @@
             <query name="category_query" xsi:type="filteredQuery">
                 <filterReference clause="must" ref="category_filter"/>
             </query>
-            <query name="visibility_query" xsi:type="filteredQuery">
-                <filterReference clause="must" ref="visibility_filter"/>
-            </query>
         </queries>
         <filters>
             <filter xsi:type="wildcardFilter" name="sku_query_filter" field="sku" value="$sku$"/>
             <filter xsi:type="rangeFilter" name="price_query_filter" field="price" from="$price.from$" to="$price.to$"/>
             <filter xsi:type="termFilter" name="category_filter" field="category_ids" value="$category_ids$"/>
-            <filter xsi:type="termFilter" name="visibility_filter" field="visibility" value="$visibility$"/>
         </filters>
         <from>0</from>
         <size>10000</size>

app/code/Magento/Elasticsearch/Model/Advanced/ProductCollectionPrepareStrategy.php

@@ -5,11 +5,9 @@
  */
 namespace Magento\Elasticsearch\Model\Advanced;
 
-use Magento\Catalog\Model\Config;
-use Magento\Catalog\Model\Product\Visibility;
 use Magento\Catalog\Model\ResourceModel\Product\Collection;
+use Magento\Catalog\Model\Config;
 use Magento\CatalogSearch\Model\Advanced\ProductCollectionPrepareStrategyInterface;
-use Magento\Framework\App\ObjectManager;
 
 /**
  * Strategy interface for preparing product collection.
@@ -21,22 +19,13 @@ class ProductCollectionPrepareStrategy implements ProductCollectionPrepareStrate
      */
     private $catalogConfig;
 
-    /**
-     * @var Visibility
-     */
-    private $catalogProductVisibility;
-
     /**
      * @param Config $catalogConfig
-     * @param Visibility|null $catalogProductVisibility
      */
     public function __construct(
-        Config $catalogConfig,
-        Visibility $catalogProductVisibility = null
+        Config $catalogConfig
     ) {
         $this->catalogConfig = $catalogConfig;
-        $this->catalogProductVisibility = $catalogProductVisibility
-            ?? ObjectManager::getInstance()->get(Visibility::class);
     }
 
     /**
@@ -47,7 +36,6 @@ public function prepare(Collection $collection)
         $collection
             ->addAttributeToSelect($this->catalogConfig->getProductAttributes())
             ->addMinimalPrice()
-            ->addTaxPercents()
-            ->setVisibility($this->catalogProductVisibility->getVisibleInSearchIds());
+            ->addTaxPercents();
     }
 }

app/code/Magento/Indexer/Console/Command/IndexerReindexCommand.php

@@ -74,7 +74,7 @@ protected function configure()
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-        $returnValue = Cli::RETURN_SUCCESS;
+        $returnValue = Cli::RETURN_FAILURE;
         foreach ($this->getIndexers($input) as $indexer) {
             try {
                 $this->validateIndexerStatus($indexer);
@@ -97,15 +97,14 @@ protected function execute(InputInterface $input, OutputInterface $output)
                 $output->writeln(
                     __('has been rebuilt successfully in %time', ['time' => gmdate('H:i:s', $resultTime)])
                 );
+                $returnValue = Cli::RETURN_SUCCESS;
             } catch (LocalizedException $e) {
                 $output->writeln(__('exception: %message', ['message' => $e->getMessage()]));
-                $returnValue = Cli::RETURN_FAILURE;
             } catch (\Exception $e) {
                 $output->writeln('process unknown error:');
                 $output->writeln($e->getMessage());
 
                 $output->writeln($e->getTraceAsString(), OutputInterface::VERBOSITY_DEBUG);
-                $returnValue = Cli::RETURN_FAILURE;
             }
         }
 

dev/tests/integration/testsuite/Magento/Elasticsearch/Model/CatalogSearch/AdvancedTest.php

@@ -7,11 +7,9 @@
 
 namespace Magento\Indexer\Console\Command;
 
-use Magento\Framework\Console\Cli;
 use Magento\Framework\ObjectManagerInterface;
 use Magento\TestFramework\Helper\Bootstrap;
 use PHPUnit\Framework\MockObject\MockObject as Mock;
-use PHPUnit\Framework\TestCase;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
@@ -21,7 +19,7 @@
  * @magentoDbIsolation disabled
  * @magentoAppIsolation enabled
  */
-class IndexerReindexCommandTest extends TestCase
+class IndexerReindexCommandTest extends \PHPUnit\Framework\TestCase
 {
     /**
      * @var ObjectManagerInterface
@@ -58,23 +56,14 @@ protected function setUp(): void
 
     /**
      * @magentoDataFixture Magento/Store/_files/second_store_group_with_second_website.php
-     * @return void
      */
-    public function testReindexAll(): void
+    public function testReindexAll()
     {
         $status = $this->command->run($this->inputMock, $this->outputMock);
-        $this->assertEquals(Cli::RETURN_SUCCESS, $status, 'Index wasn\'t success');
-    }
-
-    /**
-     * Check that 'indexer:reindex' command return right code.
-     *
-     * @magentoDataFixture Magento/Indexer/_files/wrong_config_data.php
-     * @return void
-     */
-    public function testReindexAllWhenSomethingIsWrong(): void
-    {
-        $status = $this->command->run($this->inputMock, $this->outputMock);
-        $this->assertEquals(Cli::RETURN_FAILURE, $status, 'Index didn\'t return failure code');
+        $this->assertEquals(
+            \Magento\Framework\Console\Cli::RETURN_SUCCESS,
+            $status,
+            'Index wasn\'t success'
+        );
     }
 }