Merge pull request #310 from vojtapol/294-handle-optional-parameters

oliemansm · web-flow · commit a03f31ace425 · 2019-09-25T09:55:25.000+02:00
294 handle optional parameters
diff --git a/src/main/kotlin/com/coxautodev/graphql/tools/MethodFieldResolver.kt b/src/main/kotlin/com/coxautodev/graphql/tools/MethodFieldResolver.kt
@@ -81,7 +81,7 @@ internal class MethodFieldResolver(field: FieldDefinition, search: FieldResolver
                     null
                 }
 
-                if (value == null && isOptional) {
+                if (value == null && isOptional && environment.containsArgument(definition.name)) {
                     return@add Optional.empty<Any>()
                 }
 

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ internal class MethodFieldResolver(field: FieldDefinition, search: FieldResolver`
`81`	`81`	`null`
`82`	`82`	`}`
`83`	`83`
`84`		`- if (value == null && isOptional) {`
	`84`	`+ if (value == null && isOptional && environment.containsArgument(definition.name)) {`
`85`	`85`	`return@add Optional.empty<Any>()`
`86`	`86`	`}`
`87`	`87`