Warning log when an enum value cannot be parsed and continue processing (#23447)

Mattias-Sehlstedt · web-flow · commit 6756caf68105 · 2026-04-15T15:11:16.000+08:00
diff --git a/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractTypeScriptClientCodegen.java b/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractTypeScriptClientCodegen.java
@@ -772,7 +772,11 @@ private String enumValuesToEnumTypeUnion(List<String> values, String dataType) {
     private String numericEnumValuesToEnumTypeUnion(List<Number> values) {
         List<String> stringValues = new ArrayList<>();
         for (Number value : values) {
-            stringValues.add(value.toString());
+            if (value == null) {
+                LOGGER.warn("An enum value was null. See https://github.com/swagger-api/swagger-core/issues/4223");
+            } else {
+                stringValues.add(value.toString());
+            }
         }
         return enumValuesToEnumTypeUnion(stringValues, "number");
     }
diff --git a/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/N4jsClientCodegen.java b/modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/N4jsClientCodegen.java
@@ -595,7 +595,11 @@ private String enumValuesToEnumTypeUnion(List<String> values, String dataType) {
     private String numericEnumValuesToEnumTypeUnion(List<Number> values) {
         List<String> stringValues = new ArrayList<>();
         for (Number value : values) {
-            stringValues.add(value.toString());
+            if (value == null) {
+                LOGGER.warn("An enum value was null. See https://github.com/swagger-api/swagger-core/issues/4223");
+            } else {
+                stringValues.add(value.toString());
+            }
         }
         return enumValuesToEnumTypeUnion(stringValues, "number");
     }

Original file line number	Diff line number	Diff line change
`@@ -772,7 +772,11 @@ private String enumValuesToEnumTypeUnion(List<String> values, String dataType) {`
`772`	`772`	`private String numericEnumValuesToEnumTypeUnion(List<Number> values) {`
`773`	`773`	`List<String> stringValues = new ArrayList<>();`
`774`	`774`	`for (Number value : values) {`
`775`		`- stringValues.add(value.toString());`
	`775`	`+ if (value == null) {`
	`776`	`+ LOGGER.warn("An enum value was null. See https://github.com/swagger-api/swagger-core/issues/4223");`
	`777`	`+ } else {`
	`778`	`+ stringValues.add(value.toString());`
	`779`	`+ }`
`776`	`780`	`}`
`777`	`781`	`return enumValuesToEnumTypeUnion(stringValues, "number");`
`778`	`782`	`}`
Original file line number	Diff line number	Diff line change
`@@ -595,7 +595,11 @@ private String enumValuesToEnumTypeUnion(List<String> values, String dataType) {`
`595`	`595`	`private String numericEnumValuesToEnumTypeUnion(List<Number> values) {`
`596`	`596`	`List<String> stringValues = new ArrayList<>();`
`597`	`597`	`for (Number value : values) {`
`598`		`- stringValues.add(value.toString());`
	`598`	`+ if (value == null) {`
	`599`	`+ LOGGER.warn("An enum value was null. See https://github.com/swagger-api/swagger-core/issues/4223");`
	`600`	`+ } else {`
	`601`	`+ stringValues.add(value.toString());`
	`602`	`+ }`
`599`	`603`	`}`
`600`	`604`	`return enumValuesToEnumTypeUnion(stringValues, "number");`
`601`	`605`	`}`