graphql-java-kickstart / graphql-java-tools

A schema-first tool for graphql-java inspired by graphql-tools for JS
https://www.graphql-java-kickstart.com/tools/
MIT License
812 stars 174 forks source link

SchemaParser#createInputObject issue creating directives #280

Closed eferreyramontes closed 5 years ago

eferreyramontes commented 5 years ago

There's a small bug regarding inputObject directives in this line guys:

https://github.com/graphql-java-kickstart/graphql-java-tools/blob/master/src/main/kotlin/com/coxautodev/graphql/tools/SchemaParser.kt#L208

Instead of:

.withDirectives(*buildDirectives(definition.directives, setOf(), Introspection.DirectiveLocation.INPUT_FIELD_DEFINITION))

you should have:

.withDirectives(*buildDirectives(inputDefinition.directives, setOf(), Introspection.DirectiveLocation.INPUT_FIELD_DEFINITION))
elielsonms commented 5 years ago

So, I'm using the 5.5.2 version on my project and I really need this tiny bug resolution on our project. We are avoiding to move to a newer version because that has a graphl-java update and will cause a lot of modifications on our project. Is It possible to generate a 5.5.3 version just including this bug resolution? Well, I just saw that there a 5.5.3 with the Graphl-Java upgraded to 12, so is it possible to have a 5.5.2.1?