Skip to content

Commit 310dd4b

Browse files
committed
Polish gh-910
* expand conditional for generating required-args constructor to check requiredFields.size < 256 even if fields is not
1 parent 1bc91ba commit 310dd4b

File tree

1 file changed

+7
-7
lines changed
  • graphql-dgs-codegen-core/src/main/kotlin/com/netflix/graphql/dgs/codegen/generators/java

1 file changed

+7
-7
lines changed

graphql-dgs-codegen-core/src/main/kotlin/com/netflix/graphql/dgs/codegen/generators/java/DataTypeGenerator.kt

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -459,16 +459,16 @@ abstract class BaseDataTypeGenerator(
459459
addGetterAndSetter(it, javaType)
460460
}
461461

462-
val requiredFields = fields.filter { !it.nullable }
463-
464462
if (config.generateJSpecifyAnnotations) {
465-
val allFieldsNullable = requiredFields.isEmpty()
466-
addDefaultConstructor(javaType, allFieldsNullable)
463+
val requiredFields = fields.filter { !it.nullable }
464+
addDefaultConstructor(javaType, requiredFields.isEmpty())
467465

468-
if (config.javaGenerateAllConstructor && fields.isNotEmpty() && fields.size < 256) {
469-
addParameterizedConstructor(fields, javaType)
466+
if (config.javaGenerateAllConstructor) {
467+
if (fields.isNotEmpty() && fields.size < 256) {
468+
addParameterizedConstructor(fields, javaType)
469+
}
470470

471-
if (requiredFields.isNotEmpty() && requiredFields.size < fields.size) {
471+
if (requiredFields.isNotEmpty() && requiredFields.size < fields.size && requiredFields.size < 256) {
472472
addParameterizedConstructor(requiredFields, javaType)
473473
}
474474
}

0 commit comments

Comments
 (0)