Skip to content

Commit d4e24e2

Browse files
committed
MC-42795: GraphQl products query layered navigation filters return incorrect child categories list
1 parent 61a904b commit d4e24e2

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

app/code/Magento/CatalogGraphQl/DataProvider/Product/LayeredNavigation/Builder/Aggregations/IncludeSubcategoriesOnly.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ class IncludeSubcategoriesOnly
2323
/**
2424
* @var string
2525
*/
26-
private $categoryBucket = 'category_bucket';
26+
private const CATEGORY_BUCKET = 'category_bucket';
2727

2828
/**
2929
* @var string
3030
*/
31-
private $bucketsName = 'buckets';
31+
private const BUCKETS_NAME = 'buckets';
3232

3333
/**
3434
* @var AggregationFactory
@@ -95,7 +95,7 @@ public function filter(AggregationInterface $aggregation, ?int $storeId): Aggreg
9595
return $aggregation;
9696
}
9797
$buckets = $aggregation->getBuckets();
98-
$categoryBucket = $buckets[$this->categoryBucket] ?? null;
98+
$categoryBucket = $buckets[self::CATEGORY_BUCKET] ?? null;
9999
if ($categoryBucket === null || empty($categoryBucket->getValues())) {
100100
return $aggregation;
101101
}
@@ -107,12 +107,12 @@ public function filter(AggregationInterface $aggregation, ?int $storeId): Aggreg
107107
);
108108
$categoryBucketResolved = $this->bucketFactory->create(
109109
[
110-
'name' => $this->categoryBucket,
110+
'name' => self::CATEGORY_BUCKET,
111111
'values' => $bucketValuesFiltered
112112
]
113113
);
114-
$buckets[$this->categoryBucket] = $categoryBucketResolved;
115-
return $this->aggregationFactory->create([$this->bucketsName => $buckets]);
114+
$buckets[self::CATEGORY_BUCKET] = $categoryBucketResolved;
115+
return $this->aggregationFactory->create([self::BUCKETS_NAME => $buckets]);
116116
}
117117

118118
/**

0 commit comments

Comments
 (0)