improve error handling on input objects

joonlabs · joonlabs · commit af3939114ffa · 2021-09-08T01:30:22.000+02:00
diff --git a/src/Utilities/InputValues.php b/src/Utilities/InputValues.php
@@ -76,7 +76,12 @@ static function coerceInputValue($inputValue, GraphQLType $type, $path = [])
                     continue;
                 }
 
-                $coercedValue[$field->getName()] = self::coerceInputValue($fieldValue, $field->getType(), [$path, $field->getName(), $type->getName()]);
+                try {
+                    $coercedValue[$field->getName()] = self::coerceInputValue($fieldValue, $field->getType(), [$path, $field->getName(), $type->getName()]);
+                } catch (GraphQLError $error) {
+                    // add information about explicit field name of input object type when coersion failed.
+                    throw new GraphQLError("Coercing value of \"{$field->getName()}\" failed explicity. " . $error->getMessage());
+                }
             }
 
             // ensure every provided field is defined

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,12 @@ static function coerceInputValue($inputValue, GraphQLType $type, $path = [])`
`76`	`76`	`continue;`
`77`	`77`	`}`
`78`	`78`
`79`		`- $coercedValue[$field->getName()] = self::coerceInputValue($fieldValue, $field->getType(), [$path, $field->getName(), $type->getName()]);`
	`79`	`+ try {`
	`80`	`+ $coercedValue[$field->getName()] = self::coerceInputValue($fieldValue, $field->getType(), [$path, $field->getName(), $type->getName()]);`
	`81`	`+ } catch (GraphQLError $error) {`
	`82`	`+ // add information about explicit field name of input object type when coersion failed.`
	`83`	`+ throw new GraphQLError("Coercing value of \"{$field->getName()}\" failed explicity. " . $error->getMessage());`
	`84`	`+ }`
`80`	`85`	`}`
`81`	`86`
`82`	`87`	`// ensure every provided field is defined`