fix(graphql/@builder): InputObject will check the convertibility …

…before name conflict.
LastDragon-ru · Dec 27, 2022 · 9d7fd62 · 9d7fd62
1 parent 1af0503
commit 9d7fd62
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/packages/graphql/src/Builder/Types/InputObject.php b/packages/graphql/src/Builder/Types/InputObject.php
@@ -86,18 +86,18 @@ public function getTypeDefinitionNode(
             : $node->fields;
 
         foreach ($fields as $field) {
+            // Convertable?
+            if (!$this->isConvertable($manipulator, $field)) {
+                continue;
+            }
+
             // Name should be unique (may conflict with Type's operators)
             $fieldName = $manipulator->getNodeName($field);
 
             if (isset($definition->fields[$fieldName])) {
                 throw new TypeDefinitionFieldAlreadyDefined($fieldName);
             }
 
-            // Convertable?
-            if (!$this->isConvertable($manipulator, $field)) {
-                continue;
-            }
-
             // Determine type
             $fieldType     = $manipulator->getNodeTypeName($field);
             $fieldNullable = $manipulator->isNullable($field);