mrpastewart/platform
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Pass argumentName, not fieldName, to type validations.

Browse source
This commit is contained in:
Tobe O 2019-08-14 12:07:45 -04:00
parent 34cf2065fb
commit 2bf39a81e1
2 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
graphql_server/CHANGELOG.md
View file

@ -3,6 +3,7 @@
* Now that variables are `InputValueContext` descendants, handle them the * Now that variables are `InputValueContext` descendants, handle them the
same way as other values in `coerceArgumentValues`. TLDR - Removed same way as other values in `coerceArgumentValues`. TLDR - Removed
now-obsolete, variable-specific logic in `coerceArgumentValues`. now-obsolete, variable-specific logic in `coerceArgumentValues`.
* Pass `argumentName`, not `fieldName`, to type validations.
# 1.0.1 # 1.0.1
* Fix a bug where `globalVariables` were not being properly passed * Fix a bug where `globalVariables` were not being properly passed

2
graphql_server/lib/graphql_server.dart
View file

@ -416,7 +416,7 @@ class GraphQL {
} else { } else {
try { try {
var validation = argumentType.validate( var validation = argumentType.validate(
fieldName, argumentValue.value.computeValue(variableValues)); argumentName, argumentValue.value.computeValue(variableValues));
if (!validation.successful) { if (!validation.successful) {
var errors = <GraphQLExceptionError>[ var errors = <GraphQLExceptionError>[