Skip to content

Commit 80684d7

Browse files
committed
#26206 Changes requested during Code Review
1 parent 79fe786 commit 80684d7

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

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

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -128,8 +128,8 @@ protected function getIndexers(InputInterface $input)
128128
$dependentIndexers = [[]];
129129

130130
foreach ($indexers as $indexer) {
131-
array_push($relatedIndexers, $this->getRelatedIndexerIds($indexer->getId()));
132-
array_push($dependentIndexers, $this->getDependentIndexerIds($indexer->getId()));
131+
$relatedIndexers[] = $this->getRelatedIndexerIds($indexer->getId());
132+
$dependentIndexers[] = $this->getDependentIndexerIds($indexer->getId());
133133
}
134134

135135
$relatedIndexers = array_merge(...$relatedIndexers);
@@ -162,16 +162,15 @@ protected function getIndexers(InputInterface $input)
162162
* @param string $indexerId
163163
* @return array
164164
*/
165-
private function getRelatedIndexerIds(string $indexerId)
165+
private function getRelatedIndexerIds(string $indexerId): array
166166
{
167167
$relatedIndexerIds = [[]];
168168
foreach ($this->getDependencyInfoProvider()->getIndexerIdsToRunBefore($indexerId) as $relatedIndexerId) {
169-
array_push($relatedIndexerIds, [$relatedIndexerId], $this->getRelatedIndexerIds($relatedIndexerId));
169+
$relatedIndexerIds[] = [$relatedIndexerId];
170+
$relatedIndexerIds[] = $this->getRelatedIndexerIds($relatedIndexerId);
170171
}
171172

172-
$relatedIndexerIds = array_merge(...$relatedIndexerIds);
173-
174-
return array_unique($relatedIndexerIds);
173+
return array_unique(array_merge(...$relatedIndexerIds));
175174
}
176175

177176
/**
@@ -180,13 +179,14 @@ private function getRelatedIndexerIds(string $indexerId)
180179
* @param string $indexerId
181180
* @return array
182181
*/
183-
private function getDependentIndexerIds(string $indexerId)
182+
private function getDependentIndexerIds(string $indexerId): array
184183
{
185184
$dependentIndexerIds = [[]];
186185
foreach (array_keys($this->getConfig()->getIndexers()) as $id) {
187186
$dependencies = $this->getDependencyInfoProvider()->getIndexerIdsToRunBefore($id);
188187
if (array_search($indexerId, $dependencies) !== false) {
189-
array_push($dependentIndexerIds, [$id], $this->getDependentIndexerIds($id));
188+
$dependentIndexerIds[] = [$id];
189+
$dependentIndexerIds[] = $this->getDependentIndexerIds($id);
190190
}
191191
}
192192

0 commit comments

Comments
 (0)