Merge remote-tracking branch 'origin/AC-7920' into Arrows_2.4.7_Backlog_Delivery

Author: rimple-saini

app/code/Magento/CatalogGraphQl/DataProvider/Product/LayeredNavigation/AttributeOptionProvider.php

@@ -63,6 +63,7 @@ public function getOptions(array $optionIds, ?int $storeId, array $attributeCode
                     'attribute_id' => 'a.attribute_id',
                     'attribute_code' => 'a.attribute_code',
                     'attribute_label' => 'a.frontend_label',
+                    'attribute_type' => 'a.frontend_input',
                     'position' => 'attribute_configuration.position'
                 ]
             )
@@ -137,6 +138,7 @@ private function formatResult(Select $select): array
                     'attribute_code' => $option['attribute_code'],
                     'attribute_label' => $option['attribute_store_label']
                         ? $option['attribute_store_label'] : $option['attribute_label'],
+                    'attribute_type' => $option['attribute_type'],
                     'position' => $option['position'],
                     'options' => [],
                 ];

app/code/Magento/CatalogGraphQl/DataProvider/Product/LayeredNavigation/Builder/Attribute.php

@@ -13,6 +13,7 @@
 use Magento\Framework\Api\Search\AggregationValueInterface;
 use Magento\Framework\Api\Search\BucketInterface;
 use Magento\CatalogGraphQl\DataProvider\Product\LayeredNavigation\Formatter\LayerFormatter;
+use Magento\Config\Model\Config\Source\Yesno;
 
 /**
  * @inheritdoc
@@ -49,18 +50,26 @@ class Attribute implements LayerBuilderInterface
         self::CATEGORY_BUCKET
     ];
 
+    /**
+     * @var Yesno
+     */
+    private Yesno $YesNo;
+
     /**
      * @param AttributeOptionProvider $attributeOptionProvider
      * @param LayerFormatter $layerFormatter
+     * @param Yesno $YesNo
      * @param array $bucketNameFilter
      */
     public function __construct(
         AttributeOptionProvider $attributeOptionProvider,
         LayerFormatter $layerFormatter,
+        Yesno $YesNo,
         $bucketNameFilter = []
     ) {
         $this->attributeOptionProvider = $attributeOptionProvider;
         $this->layerFormatter = $layerFormatter;
+        $this->YesNo = $YesNo;
         $this->bucketNameFilter = \array_merge($this->bucketNameFilter, $bucketNameFilter);
     }
 
@@ -87,7 +96,11 @@ public function build(AggregationInterface $aggregation, ?int $storeId): array
                 isset($attribute['position']) ? $attribute['position'] : null
             );
 
-            $options = $this->getSortedOptions($bucket, isset($attribute['options']) ? $attribute['options'] : []);
+            $options = $this->getSortedOptions(
+                $bucket,
+                isset($attribute['options']) ? $attribute['options'] : [],
+                ($attribute['attribute_type']) ? $attribute['attribute_type']: ''
+            );
             foreach ($options as $option) {
                 $result[$bucketName]['options'][] = $this->layerFormatter->buildItem(
                     $option['label'],
@@ -168,9 +181,11 @@ function (AggregationValueInterface $value) {
      *
      * @param BucketInterface $bucket
      * @param array $optionLabels
+     * @param string $attributeType
      * @return array
+     * @SuppressWarnings(PHPMD.UnusedLocalVariable)
      */
-    private function getSortedOptions(BucketInterface $bucket, array $optionLabels): array
+    private function getSortedOptions(BucketInterface $bucket, array $optionLabels, string $attributeType): array
     {
         /**
          * Option labels array has been sorted
@@ -179,7 +194,16 @@ private function getSortedOptions(BucketInterface $bucket, array $optionLabels):
         foreach ($bucket->getValues() as $value) {
             $metrics = $value->getMetrics();
             $optionValue = $metrics['value'];
-            $optionLabel = $optionLabels[$optionValue] ?? $optionValue;
+            if (isset($optionLabels[$optionValue])) {
+                $optionLabel = $optionLabels[$optionValue];
+            } else {
+                if ($attributeType === 'boolean') {
+                    $yesNoOptions = $this->YesNo->toArray();
+                    $optionLabel = $yesNoOptions[$optionValue];
+                } else {
+                    $optionLabel =  $optionValue;
+                }
+            }
             $options[$optionValue] = $metrics + ['label' => $optionLabel];
         }
 
@@ -188,7 +212,7 @@ private function getSortedOptions(BucketInterface $bucket, array $optionLabels):
          */
         foreach ($options as $optionId => $option) {
             if (!is_array($options[$optionId])) {
-               unset($options[$optionId]);
+                unset($options[$optionId]);
             }
         }
 

app/code/Magento/CatalogGraphQl/composer.json

@@ -14,6 +14,7 @@
         "magento/module-catalog-search": "*",
         "magento/framework": "*",
         "magento/module-graph-ql": "*",
+        "magento/module-config": "*",
         "magento/module-advanced-search": "*"
     },
     "suggest": {

app/code/Magento/CatalogGraphQl/etc/module.xml

@@ -13,6 +13,7 @@
             <module name="Magento_Store"/>
             <module name="Magento_Eav"/>
             <module name="Magento_GraphQl"/>
+            <module name="Magento_Config"/>
             <module name="Magento_StoreGraphQl"/>
             <module name="Magento_EavGraphQl"/>
         </sequence>

dev/tests/api-functional/testsuite/Magento/GraphQl/Catalog/ProductSearchAggregationsTest.php

@@ -37,11 +37,11 @@ function ($a) {
         $booleanAggregation = reset($booleanAggregation);
         $this->assertEquals('Boolean Attribute', $booleanAggregation['label']);
         $this->assertEquals('boolean_attribute', $booleanAggregation['attribute_code']);
-        $this->assertContainsEquals(['label' => '1', 'value' => '1', 'count' => '3'], $booleanAggregation['options']);
+        $this->assertContainsEquals(['label' => 'Yes', 'value' => '1', 'count' => '3'], $booleanAggregation['options']);
 
         $this->assertEquals(2, $booleanAggregation['count']);
         $this->assertCount(2, $booleanAggregation['options']);
-        $this->assertContainsEquals(['label' => '0', 'value' => '0', 'count' => '2'], $booleanAggregation['options']);
+        $this->assertContainsEquals(['label' => 'No', 'value' => '0', 'count' => '2'], $booleanAggregation['options']);
     }
 
     /**