ENGCOM-5639: #3993; Fix same level categories (position)re-index on category move. #23820

sidolov · web-flow · commit 76803918977b · 2019-08-22T10:32:24.000-05:00
diff --git a/app/code/Magento/Catalog/Model/Category.php b/app/code/Magento/Catalog/Model/Category.php
@@ -448,7 +448,9 @@ public function move($parentId, $afterCategoryId)
         if ($this->flatState->isFlatEnabled()) {
             $flatIndexer = $this->indexerRegistry->get(Indexer\Category\Flat\State::INDEXER_ID);
             if (!$flatIndexer->isScheduled()) {
-                $flatIndexer->reindexList([$this->getId(), $oldParentId, $parentId]);
+                $sameLevelCategories = explode(',', $this->getParentCategory()->getChildren());
+                $list = array_unique(array_merge($sameLevelCategories, [$this->getId(), $oldParentId, $parentId]));
+                $flatIndexer->reindexList($list);
             }
         }
         $productIndexer = $this->indexerRegistry->get(Indexer\Category\Product::INDEXER_ID);

Original file line number	Diff line number	Diff line change
`@@ -448,7 +448,9 @@ public function move($parentId, $afterCategoryId)`
`448`	`448`	`if ($this->flatState->isFlatEnabled()) {`
`449`	`449`	`$flatIndexer = $this->indexerRegistry->get(Indexer\Category\Flat\State::INDEXER_ID);`
`450`	`450`	`if (!$flatIndexer->isScheduled()) {`
`451`		`- $flatIndexer->reindexList([$this->getId(), $oldParentId, $parentId]);`
	`451`	`+ $sameLevelCategories = explode(',', $this->getParentCategory()->getChildren());`
	`452`	`+ $list = array_unique(array_merge($sameLevelCategories, [$this->getId(), $oldParentId, $parentId]));`
	`453`	`+ $flatIndexer->reindexList($list);`
`452`	`454`	`}`
`453`	`455`	`}`
`454`	`456`	`$productIndexer = $this->indexerRegistry->get(Indexer\Category\Product::INDEXER_ID);`