Fix the logic gap where useOptional=true without patchOnly=true appears to do nothing

eirikb · eirikb · commit 347938e2625a · 2025-12-30T09:55:58.000+01:00
diff --git a/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractDartCodegen.java b/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractDartCodegen.java
@@ -628,7 +628,7 @@ public ModelsMap postProcessModels(ModelsMap objs) {
                     shouldUseOptional = patchRequestSchemas.contains(model.classname);
                 } else {
                     Boolean schemaUseOptional = (Boolean) model.vendorExtensions.get("x-use-optional");
-                    shouldUseOptional = schemaUseOptional != null && schemaUseOptional;
+                    shouldUseOptional = schemaUseOptional == null || schemaUseOptional;
                 }
 
                 if (shouldUseOptional) {

Original file line number	Diff line number	Diff line change
`@@ -628,7 +628,7 @@ public ModelsMap postProcessModels(ModelsMap objs) {`
`628`	`628`	`shouldUseOptional = patchRequestSchemas.contains(model.classname);`
`629`	`629`	`} else {`
`630`	`630`	`Boolean schemaUseOptional = (Boolean) model.vendorExtensions.get("x-use-optional");`
`631`		`- shouldUseOptional = schemaUseOptional != null && schemaUseOptional;`
	`631`	`+ shouldUseOptional = schemaUseOptional == null \|\| schemaUseOptional;`
`632`	`632`	`}`
`633`	`633`
`634`	`634`	`if (shouldUseOptional) {`