Remove double coercion from QueryTraversers

dondonz · dondonz · commit 5a3a0e23ecad · 2022-05-25T12:06:58.000+10:00
diff --git a/src/main/java/graphql/analysis/MaxQueryComplexityInstrumentation.java b/src/main/java/graphql/analysis/MaxQueryComplexityInstrumentation.java
@@ -147,7 +147,7 @@ QueryTraverser newQueryTraverser(ExecutionContext executionContext) {
                 .schema(executionContext.getGraphQLSchema())
                 .document(executionContext.getDocument())
                 .operationName(executionContext.getExecutionInput().getOperationName())
-                .variables(executionContext.getVariables())
+                .coercedVariables(executionContext.getCoercedVariables())
                 .build();
     }
 
diff --git a/src/main/java/graphql/analysis/MaxQueryDepthInstrumentation.java b/src/main/java/graphql/analysis/MaxQueryDepthInstrumentation.java
@@ -84,7 +84,7 @@ QueryTraverser newQueryTraverser(ExecutionContext executionContext) {
                 .schema(executionContext.getGraphQLSchema())
                 .document(executionContext.getDocument())
                 .operationName(executionContext.getExecutionInput().getOperationName())
-                .variables(executionContext.getVariables())
+                .coercedVariables(executionContext.getCoercedVariables())
                 .build();
     }
 
diff --git a/src/main/java/graphql/execution/instrumentation/fieldvalidation/FieldValidationSupport.java b/src/main/java/graphql/execution/instrumentation/fieldvalidation/FieldValidationSupport.java
@@ -33,7 +33,7 @@ static List<GraphQLError> validateFieldsAndArguments(FieldValidation fieldValida
                 .schema(executionContext.getGraphQLSchema())
                 .document(executionContext.getDocument())
                 .operationName(executionContext.getOperationDefinition().getName())
-                .variables(executionContext.getVariables())
+                .coercedVariables(executionContext.getCoercedVariables())
                 .build();
 
         queryTraverser.visitPreOrder(new QueryVisitorStub() {

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ QueryTraverser newQueryTraverser(ExecutionContext executionContext) {`
`147`	`147`	`.schema(executionContext.getGraphQLSchema())`
`148`	`148`	`.document(executionContext.getDocument())`
`149`	`149`	`.operationName(executionContext.getExecutionInput().getOperationName())`
`150`		`- .variables(executionContext.getVariables())`
	`150`	`+ .coercedVariables(executionContext.getCoercedVariables())`
`151`	`151`	`.build();`
`152`	`152`	`}`
`153`	`153`
Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,7 @@ QueryTraverser newQueryTraverser(ExecutionContext executionContext) {`
`84`	`84`	`.schema(executionContext.getGraphQLSchema())`
`85`	`85`	`.document(executionContext.getDocument())`
`86`	`86`	`.operationName(executionContext.getExecutionInput().getOperationName())`
`87`		`- .variables(executionContext.getVariables())`
	`87`	`+ .coercedVariables(executionContext.getCoercedVariables())`
`88`	`88`	`.build();`
`89`	`89`	`}`
`90`	`90`