Do not require variables to be non null but initialize empty (fix #168)

oliemansm · oliemansm · commit bf6e8fe27719 · 2019-04-21T08:15:12.000+02:00
diff --git a/src/main/java/graphql/servlet/internal/GraphQLRequest.java b/src/main/java/graphql/servlet/internal/GraphQLRequest.java
@@ -5,7 +5,6 @@
 
 import java.util.HashMap;
 import java.util.Map;
-import java.util.Objects;
 
 /**
  * @author Andrew Potter
@@ -41,7 +40,9 @@ public Map<String, Object> getVariables() {
     }
 
     public void setVariables(Map<String, Object> variables) {
-        this.variables = Objects.requireNonNull(variables);
+        if (variables != null) {
+            this.variables = variables;
+        }
     }
 
     public String getOperationName() {

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,6 @@`
`5`	`5`
`6`	`6`	`import java.util.HashMap;`
`7`	`7`	`import java.util.Map;`
`8`		`-import java.util.Objects;`
`9`	`8`
`10`	`9`	`/**`
`11`	`10`	`* @author Andrew Potter`
`@@ -41,7 +40,9 @@ public Map<String, Object> getVariables() {`
`41`	`40`	`}`
`42`	`41`
`43`	`42`	`public void setVariables(Map<String, Object> variables) {`
`44`		`- this.variables = Objects.requireNonNull(variables);`
	`43`	`+ if (variables != null) {`
	`44`	`+ this.variables = variables;`
	`45`	`+ }`
`45`	`46`	`}`
`46`	`47`
`47`	`48`	`public String getOperationName() {`