checking for synthetic fields in more places

Enigmatis · Jul 9, 2019 · e860593 · e860593
1 parent f9a83d0
commit e860593
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/main/java/graphql/annotations/processor/directives/DirectiveCreator.java b/src/main/java/graphql/annotations/processor/directives/DirectiveCreator.java
@@ -50,8 +50,8 @@ public GraphQLDirective getDirective(Class<?> annotatedClass) {
 
     private void buildArguments(GraphQLDirective.Builder builder, Class<?> annotatedClass) {
         Arrays.stream(annotatedClass.getDeclaredFields())
-                .filter(x -> !x.isSynthetic())
-                .forEach(x -> builder.argument(directiveArgumentCreator.getArgument(x, annotatedClass)));
+                .filter(field -> !field.isSynthetic())
+                .forEach(field -> builder.argument(directiveArgumentCreator.getArgument(field, annotatedClass)));
     }
 
     private Introspection.DirectiveLocation[] getValidLocations(Class<?> annotatedClass) {

diff --git a/src/main/java/graphql/annotations/processor/util/ObjectUtil.java b/src/main/java/graphql/annotations/processor/util/ObjectUtil.java
@@ -18,7 +18,7 @@
 import java.util.Map;
 import java.util.TreeMap;
 
-public  class ObjectUtil {
+public class ObjectUtil {
 
     public static Map<String, Field> getAllFields(Class c) {
         Map<String, Field> fields;
@@ -30,7 +30,9 @@ public static Map<String, Field> getAllFields(Class c) {
         }
 
         for (Field f : c.getDeclaredFields()) {
-            fields.put(f.getName(), f);
+            if (!f.isSynthetic()) {
+                fields.put(f.getName(), f);
+            }
         }
 
         return fields;