graphql-java · andimarek · Jul 30, 2018 · Jul 30, 2018
diff --git a/src/main/java/graphql/language/Argument.java b/src/main/java/graphql/language/Argument.java
@@ -13,8 +13,8 @@
 @PublicApi
 public class Argument extends AbstractNode<Argument> implements NamedNode<Argument> {
 
-    private final String name;
-    private final Value value;
+    private String name;
+    private Value value;
 
     @Internal
     protected Argument(String name, Value value, SourceLocation sourceLocation, List<Comment> comments) {
@@ -42,6 +42,14 @@ public Value getValue() {
         return value;
     }
 
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public void setValue(Value value) {
+        this.value = value;
+    }
+
     @Override
     public List<Node> getChildren() {
         List<Node> result = new ArrayList<>();

diff --git a/src/main/java/graphql/language/ArrayValue.java b/src/main/java/graphql/language/ArrayValue.java
@@ -32,7 +32,7 @@ public ArrayValue(List<Value> values) {
     }
 
     public List<Value> getValues() {
-        return new ArrayList<>(values);
+        return values;
     }
 
     @Override

diff --git a/src/main/java/graphql/language/Document.java b/src/main/java/graphql/language/Document.java
@@ -29,7 +29,7 @@ public Document(List<Definition> definitions) {
     }
 
     public List<Definition> getDefinitions() {
-        return new ArrayList<>(definitions);
+        return definitions;
     }
 
 

diff --git a/src/main/java/graphql/language/EnumValueDefinition.java b/src/main/java/graphql/language/EnumValueDefinition.java
@@ -2,13 +2,15 @@
 
 
 import graphql.Internal;
+import graphql.PublicApi;
 import graphql.util.TraversalControl;
 import graphql.util.TraverserContext;
 
 import java.util.ArrayList;
 import java.util.List;
 import java.util.function.Consumer;
 
+@PublicApi
 public class EnumValueDefinition extends AbstractNode<EnumValueDefinition> implements DirectivesContainer<EnumValueDefinition> {
     private final String name;
     private final Description description;
@@ -17,10 +19,10 @@ public class EnumValueDefinition extends AbstractNode<EnumValueDefinition> imple
 
     @Internal
     protected EnumValueDefinition(String name,
-                                List<Directive> directives,
-                                Description description,
-                                SourceLocation sourceLocation,
-                                List<Comment> comments) {
+                                  List<Directive> directives,
+                                  Description description,
+                                  SourceLocation sourceLocation,
+                                  List<Comment> comments) {
         super(sourceLocation, comments);
         this.name = name;
         this.description = description;

diff --git a/src/main/java/graphql/language/Field.java b/src/main/java/graphql/language/Field.java
@@ -17,20 +17,20 @@
 @PublicApi
 public class Field extends AbstractNode<Field> implements Selection<Field>, SelectionSetContainer<Field>, DirectivesContainer<Field> {
 
-    private final String name;
-    private final String alias;
-    private final List<Argument> arguments;
-    private final List<Directive> directives;
-    private final SelectionSet selectionSet;
+    private String name;
+    private String alias;
+    private List<Argument> arguments;
+    private List<Directive> directives;
+    private SelectionSet selectionSet;
 
     @Internal
     protected Field(String name,
-                  String alias,
-                  List<Argument> arguments,
-                  List<Directive> directives,
-                  SelectionSet selectionSet,
-                  SourceLocation sourceLocation,
-                  List<Comment> comments) {
+                    String alias,
+                    List<Argument> arguments,
+                    List<Directive> directives,
+                    SelectionSet selectionSet,
+                    SourceLocation sourceLocation,
+                    List<Comment> comments) {
         super(sourceLocation, comments);
         this.name = name;
         this.alias = alias;
@@ -49,7 +49,6 @@ public Field(String name) {
 
     /**
      * alternative to using a Builder for convenience
-     *
      */
     public Field(String name, List<Argument> arguments) {
         this(name, null, arguments, new ArrayList<>(), null, null, new ArrayList<>());
@@ -74,7 +73,9 @@ public List<Node> getChildren() {
         List<Node> result = new ArrayList<>();
         result.addAll(arguments);
         result.addAll(directives);
-        if (selectionSet != null) result.add(selectionSet);
+        if (selectionSet != null) {
+            result.add(selectionSet);
+        }
         return result;
     }
 
@@ -89,7 +90,15 @@ public String getAlias() {
     }
 
     public List<Argument> getArguments() {
-        return new ArrayList<>(arguments);
+        return arguments;
+    }
+
+    public void setArguments(List<Argument> arguments) {
+        this.arguments = arguments;
+    }
+
+    public void setDirectives(List<Directive> directives) {
+        this.directives = directives;
     }
 
     @Override
@@ -102,10 +111,26 @@ public SelectionSet getSelectionSet() {
         return selectionSet;
     }
 
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public void setAlias(String alias) {
+        this.alias = alias;
+    }
+
+    public void setSelectionSet(SelectionSet selectionSet) {
+        this.selectionSet = selectionSet;
+    }
+
     @Override
     public boolean isEqualTo(Node o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
+        if (this == o) {
+            return true;
+        }
+        if (o == null || getClass() != o.getClass()) {
+            return false;
+        }
 
         Field that = (Field) o;
 

diff --git a/src/main/java/graphql/language/FieldDefinition.java b/src/main/java/graphql/language/FieldDefinition.java
@@ -12,8 +12,8 @@
 
 @PublicApi
 public class FieldDefinition extends AbstractNode<FieldDefinition> implements DirectivesContainer<FieldDefinition> {
-    private final String name;
-    private final Type type;
+    private String name;
+    private Type type;
     private final Description description;
     private final List<InputValueDefinition> inputValueDefinitions;
     private final List<Directive> directives;
@@ -53,7 +53,7 @@ public Description getDescription() {
     }
 
     public List<InputValueDefinition> getInputValueDefinitions() {
-        return new ArrayList<>(inputValueDefinitions);
+        return inputValueDefinitions;
     }
 
     @Override
@@ -92,6 +92,14 @@ public FieldDefinition deepCopy() {
         );
     }
 
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public void setType(Type type) {
+        this.type = type;
+    }
+
     @Override
     public String toString() {
         return "FieldDefinition{" +

diff --git a/src/main/java/graphql/language/InputObjectTypeDefinition.java b/src/main/java/graphql/language/InputObjectTypeDefinition.java
@@ -38,7 +38,7 @@ public List<Directive> getDirectives() {
     }
 
     public List<InputValueDefinition> getInputValueDefinitions() {
-        return new ArrayList<>(inputValueDefinitions);
+        return inputValueDefinitions;
     }
 
     @Override

diff --git a/src/main/java/graphql/language/ObjectTypeDefinition.java b/src/main/java/graphql/language/ObjectTypeDefinition.java
@@ -51,7 +51,7 @@ public List<Directive> getDirectives() {
     }
 
     public List<FieldDefinition> getFieldDefinitions() {
-        return new ArrayList<>(fieldDefinitions);
+        return fieldDefinitions;
     }
 
     @Override

diff --git a/src/main/java/graphql/language/OperationDefinition.java b/src/main/java/graphql/language/OperationDefinition.java
@@ -19,10 +19,10 @@ public enum Operation {
 
     private final String name;
 
-    private final Operation operation;
+    private Operation operation;
     private final List<VariableDefinition> variableDefinitions;
     private final List<Directive> directives;
-    private final SelectionSet selectionSet;
+    private SelectionSet selectionSet;
 
     @Internal
     protected OperationDefinition(String name,
@@ -67,13 +67,21 @@ public Operation getOperation() {
     }
 
     public List<VariableDefinition> getVariableDefinitions() {
-        return new ArrayList<>(variableDefinitions);
+        return variableDefinitions;
     }
 
     public List<Directive> getDirectives() {
         return new ArrayList<>(directives);
     }
 
+    public void setOperation(Operation operation) {
+        this.operation = operation;
+    }
+
+    public void setSelectionSet(SelectionSet selectionSet) {
+        this.selectionSet = selectionSet;
+    }
+
     @Override
     public SelectionSet getSelectionSet() {
         return selectionSet;

diff --git a/src/main/java/graphql/language/SchemaDefinition.java b/src/main/java/graphql/language/SchemaDefinition.java
@@ -43,7 +43,7 @@ public Directive getDirective(String directiveName) {
 
 
     public List<OperationTypeDefinition> getOperationTypeDefinitions() {
-        return new ArrayList<>(operationTypeDefinitions);
+        return operationTypeDefinitions;
     }
 
     @Override

diff --git a/src/main/java/graphql/language/SelectionSet.java b/src/main/java/graphql/language/SelectionSet.java
@@ -29,7 +29,7 @@ public SelectionSet(List<Selection> selections) {
     }
 
     public List<Selection> getSelections() {
-        return new ArrayList<>(selections);
+        return selections;
     }
-Original file line number
+Diff line change
@@ Expand Up / @@ -29,7 +29,7 @@ public Document(List<Definition> definitions) { @@
         }
         public List<Definition> getDefinitions() {
-            return new ArrayList<>(definitions);
+            return definitions;
         }
@@ Expand Down @@