Extra comment

bbakerman · bbakerman · commit e09e6b698fd2 · 2024-12-09T21:37:00.000+11:00
diff --git a/src/main/java/graphql/normalized/VariableAccumulator.java b/src/main/java/graphql/normalized/VariableAccumulator.java
@@ -30,6 +30,8 @@ public VariableAccumulator(@Nullable VariablePredicate variablePredicate) {
     }
 
     public boolean shouldMakeVariable(ExecutableNormalizedField executableNormalizedField, QueryAppliedDirective queryAppliedDirective, String argName, NormalizedInputValue normalizedInputValue) {
+        // when a variable is used on the argument to a query directive then the queryAppliedDirective will be nonnull.
+        // otherwise it must be a field argument
         if (queryAppliedDirective != null) {
             return variablePredicate != null && variablePredicate.shouldMakeVariable(executableNormalizedField, queryAppliedDirective, argName, normalizedInputValue);
         } else {

Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,8 @@ public VariableAccumulator(@Nullable VariablePredicate variablePredicate) {`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`public boolean shouldMakeVariable(ExecutableNormalizedField executableNormalizedField, QueryAppliedDirective queryAppliedDirective, String argName, NormalizedInputValue normalizedInputValue) {`
	`33`	`+ // when a variable is used on the argument to a query directive then the queryAppliedDirective will be nonnull.`
	`34`	`+ // otherwise it must be a field argument`
`33`	`35`	`if (queryAppliedDirective != null) {`
`34`	`36`	`return variablePredicate != null && variablePredicate.shouldMakeVariable(executableNormalizedField, queryAppliedDirective, argName, normalizedInputValue);`
`35`	`37`	`} else {`