[TASK] Changed logic so only reindexList function is called

lewisvoncken · lewisvoncken · commit 1a253909338a · 2017-08-22T12:41:10.000Z
diff --git a/app/code/Magento/Catalog/Model/Category.php b/app/code/Magento/Catalog/Model/Category.php
@@ -1094,10 +1094,11 @@ public function reindex()
         if ($this->flatState->isFlatEnabled()) {
             $flatIndexer = $this->indexerRegistry->get(Indexer\Category\Flat\State::INDEXER_ID);
             if (!$flatIndexer->isScheduled()) {
-                $flatIndexer->reindexRow($this->getId());
+                $idsList = [$this->getId()];
                 if ($this->dataHasChangedFor('url_key')) {
-                    $flatIndexer->reindexList(explode(',', $this->getAllChildren()));
+                    $idsList = array_merge($idsList, explode(',', $this->getAllChildren()));
                 }
+                $flatIndexer->reindexList($idsList);
             }
         }
         $productIndexer = $this->indexerRegistry->get(Indexer\Category\Product::INDEXER_ID);

Original file line number	Diff line number	Diff line change
`@@ -1094,10 +1094,11 @@ public function reindex()`
`1094`	`1094`	`if ($this->flatState->isFlatEnabled()) {`
`1095`	`1095`	`$flatIndexer = $this->indexerRegistry->get(Indexer\Category\Flat\State::INDEXER_ID);`
`1096`	`1096`	`if (!$flatIndexer->isScheduled()) {`
`1097`		`- $flatIndexer->reindexRow($this->getId());`
	`1097`	`+ $idsList = [$this->getId()];`
`1098`	`1098`	`if ($this->dataHasChangedFor('url_key')) {`
`1099`		`- $flatIndexer->reindexList(explode(',', $this->getAllChildren()));`
	`1099`	`+ $idsList = array_merge($idsList, explode(',', $this->getAllChildren()));`
`1100`	`1100`	`}`
	`1101`	`+ $flatIndexer->reindexList($idsList);`
`1101`	`1102`	`}`
`1102`	`1103`	`}`
`1103`	`1104`	`$productIndexer = $this->indexerRegistry->get(Indexer\Category\Product::INDEXER_ID);`