Merge remote-tracking branch 'origin/MC-19331' into 2.2.10-develop-pr112

Author: zakdma

app/code/Magento/CatalogInventory/Model/ResourceModel/Indexer/Stock/DefaultStock.php

@@ -11,6 +11,7 @@
 use Magento\Framework\DB\Adapter\AdapterInterface;
 use Magento\CatalogInventory\Api\StockConfigurationInterface;
 use Magento\CatalogInventory\Model\Indexer\Stock\Action\Full;
+use Magento\Catalog\Model\Product\Attribute\Source\Status as ProductStatus;
 
 /**
  * CatalogInventory Default Stock Status Indexer Resource Model
@@ -225,6 +226,8 @@ protected function _getStockStatusSelect($entityIds = null, $usePrimaryTable = f
     {
         $connection = $this->getConnection();
         $qtyExpr = $connection->getCheckSql('cisi.qty > 0', 'cisi.qty', 0);
+        $metadata = $this->getMetadataPool()->getMetadata(\Magento\Catalog\Api\Data\ProductInterface::class);
+        $linkField = $metadata->getLinkField();
 
         $select = $connection->select()->from(
             ['e' => $this->getTable('catalog_product_entity')],
@@ -238,6 +241,12 @@ protected function _getStockStatusSelect($entityIds = null, $usePrimaryTable = f
             ['cisi' => $this->getTable('cataloginventory_stock_item')],
             'cisi.stock_id = cis.stock_id AND cisi.product_id = e.entity_id',
             []
+        )->joinInner(
+            ['mcpei' => $this->getTable('catalog_product_entity_int')],
+            'e.' . $linkField . ' = mcpei.' . $linkField
+            . ' AND mcpei.attribute_id = ' . $this->_getAttribute('status')->getId()
+            . ' AND mcpei.value = ' . ProductStatus::STATUS_ENABLED,
+            []
         )->columns(
             ['qty' => $qtyExpr]
         )->where(
@@ -308,8 +317,7 @@ protected function _updateIndex($entityIds)
     }
 
     /**
-     * Delete records by their ids from index table.
-     *
+     * Delete records by their ids from index table
      * Used to clean table before re-indexation
      *
      * @param array $ids
@@ -354,8 +362,6 @@ public function getIdxTable($table = null)
     }
 
     /**
-     * Get status expression
-     *
      * @param AdapterInterface $connection
      * @param bool $isAggregate
      * @return mixed
@@ -381,8 +387,6 @@ protected function getStatusExpression(AdapterInterface $connection, $isAggregat
     }
 
     /**
-     * Get stock configuration
-     *
      * @return StockConfigurationInterface
      *
      * @deprecated 100.1.0
@@ -398,8 +402,6 @@ protected function getStockConfiguration()
     }
 
     /**
-     * Get query processor composite
-     *
      * @return QueryProcessorComposite
      */
     private function getQueryProcessorComposite()

dev/tests/integration/testsuite/Magento/Catalog/_files/products_with_layered_navigation_attribute.php

@@ -5,42 +5,24 @@
  */
 namespace Magento\CatalogInventory\Model\Indexer\Stock\Action;
 
-use Magento\TestFramework\Helper\Bootstrap;
-use Magento\Framework\ObjectManagerInterface;
-use Magento\CatalogInventory\Model\Indexer\Stock\Processor;
-use Magento\Catalog\Model\CategoryFactory;
-use Magento\Catalog\Block\Product\ListProduct;
-use Magento\Catalog\Model\ResourceModel\Product\CollectionFactory;
-use Magento\Catalog\Model\Product;
-use PHPUnit\Framework\TestCase;
-
 /**
  * Full reindex Test
  */
-class FullTest extends TestCase
+class FullTest extends \PHPUnit\Framework\TestCase
 {
     /**
-     * @var ObjectManagerInterface
-     */
-    private $objectManager;
-
-    /**
-     * @var Processor
+     * @var \Magento\CatalogInventory\Model\Indexer\Stock\Processor
      */
     protected $_processor;
 
-    /**
-     * @inheritdoc
-     */
     protected function setUp()
     {
-        $this->objectManager = Bootstrap::getObjectManager();
-        $this->_processor = $this->objectManager->get(Processor::class);
+        $this->_processor = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->get(
+            \Magento\CatalogInventory\Model\Indexer\Stock\Processor::class
+        );
     }
 
     /**
-     * Reindex all
-     *
      * @magentoDbIsolation disabled
      * @magentoAppIsolation enabled
      * @magentoDataFixture Magento/Catalog/_files/product_simple.php
@@ -49,9 +31,13 @@ public function testReindexAll()
     {
         $this->_processor->reindexAll();
 
-        $categoryFactory = $this->objectManager->get(CategoryFactory::class);
-        /** @var ListProduct $listProduct */
-        $listProduct = $this->objectManager->get(ListProduct::class);
+        $categoryFactory = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->get(
+            \Magento\Catalog\Model\CategoryFactory::class
+        );
+        /** @var \Magento\Catalog\Block\Product\ListProduct $listProduct */
+        $listProduct = \Magento\TestFramework\Helper\Bootstrap::getObjectManager()->get(
+            \Magento\Catalog\Block\Product\ListProduct::class
+        );
 
         $category = $categoryFactory->create()->load(2);
         $layer = $listProduct->getLayer();
@@ -75,37 +61,4 @@ public function testReindexAll()
             $this->assertEquals(100, $product->getQty());
         }
     }
-
-    /**
-     * Reindex with disabled product
-     *
-     * @return void
-     * @magentoDbIsolation disabled
-     * @magentoAppIsolation enabled
-     * @magentoDataFixture Magento/Catalog/_files/products_with_layered_navigation_attribute.php
-     */
-    public function testReindexAllWithDisabledProduct()
-    {
-        $productCollectionFactory = $this->objectManager->get(CollectionFactory::class);
-        $productCollection = $productCollectionFactory
-            ->create()
-            ->addAttributeToSelect('*')
-            ->addAttributeToFilter('sku', ['eq' => 'simple3'])
-            ->addAttributeToSort('created_at', 'DESC')
-            ->joinField(
-                'stock_status',
-                'cataloginventory_stock_status',
-                'stock_status',
-                'product_id=entity_id',
-                '{{table}}.stock_id=1',
-                'left'
-            )->load();
-
-        $this->assertCount(1, $productCollection);
-
-        /** @var Product $product */
-        foreach ($productCollection as $product) {
-            $this->assertEquals(1, $product->getData('stock_status'));
-        }
-    }
 }