Skip to content

Commit 5ebc7e2

Browse files
committed
Move more constants to CodegenConstants
1 parent 5f0bfe1 commit 5ebc7e2

5 files changed

Lines changed: 8 additions & 17 deletions

File tree

modules/openapi-generator/src/main/java/org/openapitools/codegen/CodegenConstants.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -501,5 +501,9 @@ public static enum ENUM_PROPERTY_NAMING_TYPE {camelCase, PascalCase, snake_case,
501501
public static final String X_NULLABLE = "x-nullable";
502502

503503
public static final String X_ONE_OF_JSON_CREATOR = "x-oneof-jsonCreator";
504-
public static final String X_ONE_OF_UNWRAPPED = "x-oneOfunwrapper";
504+
public static final String X_ONE_OF_UNWRAPPED = "x-oneOfunwrapped";
505+
public static final String JACKSON2_PACKAGE = "com.fasterxml.jackson";
506+
public static final String JACKSON3_PACKAGE = "tools.jackson";
507+
public static final String JACKSON_PACKAGE = "jacksonPackage";
508+
505509
}

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractJavaCodegen.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2866,7 +2866,7 @@ protected void addOneOfMixinSupport(ModelsMap obj, CodegenModel cm) {
28662866
if (!isUseJackson3()) {
28672867
obj.getImports().add(Map.of("import", "com.fasterxml.jackson.core.JsonProcessingException"));
28682868
}
2869-
obj.getImports().add(Map.of("import", importMapping.get("JsonNode")));
2869+
obj.getImports().add(Map.of("import", (isUseJackson3()? JACKSON3_PACKAGE:JACKSON2_PACKAGE) + ".databind.JsonNode"));
28702870
obj.getImports().add(Map.of("import", configPackage + ".JacksonMixinConfig"));
28712871

28722872
}

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractKotlinCodegen.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@
4646
import java.util.stream.Collectors;
4747
import java.util.stream.Stream;
4848

49+
import static org.openapitools.codegen.CodegenConstants.*;
4950
import static org.openapitools.codegen.utils.CamelizeOption.LOWERCASE_FIRST_LETTER;
5051
import static org.openapitools.codegen.utils.StringUtils.*;
5152

@@ -58,9 +59,6 @@ public abstract class AbstractKotlinCodegen extends DefaultCodegen implements Co
5859
public static final String JAVAX_PACKAGE = "javaxPackage";
5960
public static final String USE_JAKARTA_EE = "useJakartaEe";
6061
public static final String USE_JACKSON_3 = "useJackson3";
61-
public static final String JACKSON2_PACKAGE = "com.fasterxml.jackson";
62-
public static final String JACKSON3_PACKAGE = "tools.jackson";
63-
public static final String JACKSON_PACKAGE = "jacksonPackage";
6462
public static final String SCHEMA_IMPLEMENTS = "schemaImplements";
6563
public static final String SCHEMA_IMPLEMENTS_FIELDS = "schemaImplementsFields";
6664
public static final String X_KOTLIN_IMPLEMENTS_SKIP = "xKotlinImplementsSkip";

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/JavaClientCodegen.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -127,10 +127,6 @@ public class JavaClientCodegen extends AbstractJavaCodegen
127127
public static final String SERIALIZATION_LIBRARY_JSONB = "jsonb";
128128

129129
public static final String USE_SPRING_BOOT4 = "useSpringBoot4";
130-
private static final String JACKSON2_PACKAGE = "com.fasterxml.jackson";
131-
private static final String JACKSON3_PACKAGE = "tools.jackson";
132-
private static final String JACKSON_PACKAGE = "jacksonPackage";
133-
134130
public static final String GENERATE_CLIENT_AS_BEAN = "generateClientAsBean";
135131

136132
protected String gradleWrapperPackage = "gradle.wrapper";
@@ -1322,13 +1318,11 @@ public void setCaseInsensitiveResponseHeaders(final Boolean caseInsensitiveRespo
13221318

13231319
protected void applyJackson2Package() {
13241320
writePropertyBack(JACKSON_PACKAGE, JACKSON2_PACKAGE);
1325-
importMapping.put("JsonNode", JACKSON2_PACKAGE + ".databind.JsonNode");
13261321
importMapping.put("JsonMapper", JACKSON2_PACKAGE + ".databind.json.JsonMapper");
13271322
}
13281323

13291324
protected void applyJackson3Package() {
13301325
writePropertyBack(JACKSON_PACKAGE, JACKSON3_PACKAGE);
1331-
importMapping.put("JsonNode", JACKSON3_PACKAGE + ".databind.JsonNode");
13321326
importMapping.put("JsonMapper", JACKSON3_PACKAGE + ".databind.json.JsonMapper");
13331327
}
13341328

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/SpringCodegen.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,7 @@
5454
import java.util.stream.Collectors;
5555

5656
import static org.apache.commons.lang3.StringUtils.isNotEmpty;
57-
import static org.openapitools.codegen.CodegenConstants.USE_DEDUCTION_FOR_ONE_OF_INTERFACES;
58-
import static org.openapitools.codegen.CodegenConstants.USE_DEDUCTION_FOR_ONE_OF_INTERFACES_DESC;
57+
import static org.openapitools.codegen.CodegenConstants.*;
5958
import static org.openapitools.codegen.utils.CamelizeOption.LOWERCASE_FIRST_LETTER;
6059
import static org.openapitools.codegen.utils.StringUtils.camelize;
6160

@@ -114,9 +113,6 @@ public class SpringCodegen extends AbstractJavaCodegen
114113
public static final String USE_SPRING_BUILT_IN_VALIDATION = "useSpringBuiltInValidation";
115114
public static final String SPRING_API_VERSION = "springApiVersion";
116115
public static final String USE_JACKSON_3 = "useJackson3";
117-
public static final String JACKSON2_PACKAGE = "com.fasterxml.jackson";
118-
public static final String JACKSON3_PACKAGE = "tools.jackson";
119-
public static final String JACKSON_PACKAGE = "jacksonPackage";
120116
public static final String ADDITIONAL_NOT_NULL_ANNOTATIONS = "additionalNotNullAnnotations";
121117
public static final String AUTO_X_SPRING_PAGINATED = "autoXSpringPaginated";
122118
public static final String GENERATE_SORT_VALIDATION = "generateSortValidation";
@@ -607,7 +603,6 @@ public void processOpts() {
607603
importMapping.put("DateTimeFormat", "org.springframework.format.annotation.DateTimeFormat");
608604
importMapping.put("ParameterObject", "org.springdoc.api.annotations.ParameterObject");
609605
String jacksonPackage = (String)additionalProperties.get("jacksonPackage");
610-
importMapping.put("JsonNode", jacksonPackage + ".databind.JsonNode");
611606
importMapping.put("JsonMapper", jacksonPackage + ".databind.json.JsonMapper");
612607

613608
if (isUseSpringBoot3() || isUseSpringBoot4()) {

0 commit comments

Comments
 (0)