FIX issue#26217 - Wrong fields selection while using fragments on GraphQL products query

Author: phoenix128

app/code/Magento/CatalogGraphQl/Model/Resolver/Products/Query/FieldSelection.php

@@ -7,7 +7,6 @@
 
 namespace Magento\CatalogGraphQl\Model\Resolver\Products\Query;
 
-use GraphQL\Language\AST\SelectionNode;
 use Magento\Framework\GraphQl\Query\FieldTranslator;
 use Magento\Framework\GraphQl\Schema\Type\ResolveInfo;
 
@@ -37,57 +36,18 @@ public function __construct(FieldTranslator $fieldTranslator)
      */
     public function getProductsFieldSelection(ResolveInfo $resolveInfo): array
     {
-        return $this->getProductFields($resolveInfo);
-    }
+        $productFields = $resolveInfo->getFieldSelection(1);
+        $sectionNames = ['items', 'product'];
 
-    /**
-     * Return field names for all requested product fields.
-     *
-     * @param ResolveInfo $info
-     * @return string[]
-     */
-    private function getProductFields(ResolveInfo $info): array
-    {
         $fieldNames = [];
-        foreach ($info->fieldNodes as $node) {
-            if ($node->name->value !== 'products' && $node->name->value !== 'variants') {
-                continue;
-            }
-            foreach ($node->selectionSet->selections as $selection) {
-                if ($selection->name->value !== 'items' && $selection->name->value !== 'product') {
-                    continue;
-                }
-                $fieldNames[] = $this->collectProductFieldNames($selection, $fieldNames);
-            }
-        }
-        if (!empty($fieldNames)) {
-            $fieldNames = array_merge(...$fieldNames);
-        }
-        return $fieldNames;
-    }
-
-    /**
-     * Collect field names for each node in selection
-     *
-     * @param SelectionNode $selection
-     * @param array $fieldNames
-     * @return array
-     */
-    private function collectProductFieldNames(SelectionNode $selection, array $fieldNames = []): array
-    {
-        foreach ($selection->selectionSet->selections as $itemSelection) {
-            if ($itemSelection->kind === 'InlineFragment') {
-                foreach ($itemSelection->selectionSet->selections as $inlineSelection) {
-                    if ($inlineSelection->kind === 'InlineFragment') {
-                        continue;
-                    }
-                    $fieldNames[] = $this->fieldTranslator->translate($inlineSelection->name->value);
+        foreach ($sectionNames as $sectionName) {
+            if (isset($productFields[$sectionName])) {
+                foreach (array_keys($productFields[$sectionName]) as $fieldName) {
+                    $fieldNames[] = $this->fieldTranslator->translate($fieldName);
                 }
-                continue;
             }
-            $fieldNames[] = $this->fieldTranslator->translate($itemSelection->name->value);
         }
 
-        return $fieldNames;
+        return array_unique($fieldNames);
     }
 }