cosmetics

Author: martinbonnin

src/main/java/graphql/execution/Execution.java

@@ -256,9 +256,9 @@ private boolean propagateErrors(CoercedVariables variables, List<Directive> dire
         Directive foundDirective = NodeUtil.findNodeByName(directives, ERROR_HANDLING_DIRECTIVE_DEFINITION.getName());
         if (foundDirective != null) {
             Map<String, Object> argumentValues = ValuesResolver.getArgumentValues(ErrorHandlingDirective.getArguments(), foundDirective.getArguments(), variables, GraphQLContext.getDefault(), Locale.getDefault());
-            Object flag = argumentValues.get("onError");
-            Assert.assertTrue(flag instanceof String, "The '%s' directive MUST have a OnError value for the 'if' argument", ERROR_HANDLING_DIRECTIVE_DEFINITION.getName());
-            return flag.equals(Enums.ON_ERROR_PROPAGATE);
+            Object argumentValue = argumentValues.get("onError");
+            Assert.assertTrue(argumentValue instanceof String, "The '%s' directive MUST have an OnError value for the 'onError' argument", ERROR_HANDLING_DIRECTIVE_DEFINITION.getName());
+            return argumentValue.equals(Enums.ON_ERROR_PROPAGATE);
         }
         return defaultValue;
     }

src/main/java/graphql/schema/idl/SchemaGenerator.java

@@ -59,7 +59,9 @@ public SchemaGenerator() {
      * Created a schema from the SDL that is has a mocked runtime.
      *
      * @param sdl the SDL to be mocked
+     *
      * @return a schema with a mocked runtime
+     *
      * @see RuntimeWiring#MOCKED_WIRING
      */
     public static GraphQLSchema createdMockedSchema(String sdl) {
@@ -73,7 +75,9 @@ public static GraphQLSchema createdMockedSchema(String sdl) {
      *
      * @param typeRegistry this can be obtained via {@link SchemaParser#parse(String)}
      * @param wiring       this can be built using {@link RuntimeWiring#newRuntimeWiring()}
+     *
      * @return an executable schema
+     *
      * @throws SchemaProblem if there are problems in assembling a schema such as missing type resolvers or no operations defined
      */
     public GraphQLSchema makeExecutableSchema(TypeDefinitionRegistry typeRegistry, RuntimeWiring wiring) throws SchemaProblem {
@@ -87,7 +91,9 @@ public GraphQLSchema makeExecutableSchema(TypeDefinitionRegistry typeRegistry, R
      * @param options      the controlling options
      * @param typeRegistry this can be obtained via {@link SchemaParser#parse(String)}
      * @param wiring       this can be built using {@link RuntimeWiring#newRuntimeWiring()}
+     *
      * @return an executable schema
+     *
      * @throws SchemaProblem if there are problems in assembling a schema such as missing type resolvers or no operations defined
      */
     public GraphQLSchema makeExecutableSchema(Options options, TypeDefinitionRegistry typeRegistry, RuntimeWiring wiring) throws SchemaProblem {