Merge pull request #1130 from swagger-api/Fyro-Ing-issues/1054

Fyro ing issues/1054

Author: HugoMario

src/main/java/io/swagger/codegen/v3/generators/handlebars/java/JavaHelper.java

@@ -79,20 +79,26 @@ public CharSequence getModelImports(Map<String, Object> templateData, Options op
         boolean withXml = Boolean.valueOf(String.valueOf(templateData.get("withXml")));
         boolean parcelableModel = Boolean.valueOf(String.valueOf(templateData.get("parcelableModel")));
         boolean useBeanValidation = Boolean.valueOf(String.valueOf(templateData.get("useBeanValidation")));
+        boolean jakarta = Boolean.valueOf(String.valueOf(templateData.get("jakarta")));
         if (serializableMode) {
             builder.append("import java.io.Serializable;\n");
         }
         if (jackson && withXml) {
             builder.append("import com.fasterxml.jackson.dataformat.xml.annotation.*;\n");
         }
-        if (withXml) {
-            builder.append("import javax.xml.bind.annotation.*;\n");
+        if (withXml && jakarta) {
+            builder.append("import jakarta.xml.bind.annotation.*;\n");
+        } else if (withXml) {
+            builder.append("import jakarta.xml.bind.annotation.*;\n");
         }
         if (parcelableModel) {
             builder.append("import android.os.Parcelable;\n");
             builder.append("import android.os.Parcel;\n");
         }
-        if (useBeanValidation) {
+        if (useBeanValidation && jakarta) {
+            builder.append("import jakarta.validation.constraints.*;\n");
+            builder.append("import jakarta.validation.Valid;\n");
+        } else if (useBeanValidation) {
             builder.append("import javax.validation.constraints.*;\n");
             builder.append("import javax.validation.Valid;\n");
         }

src/main/java/io/swagger/codegen/v3/generators/java/AbstractJavaCodegen.java

@@ -1,5 +1,10 @@
 package io.swagger.codegen.v3.generators.java;
 
+import static io.swagger.codegen.v3.CodegenConstants.HAS_ENUMS_EXT_NAME;
+import static io.swagger.codegen.v3.CodegenConstants.IS_ENUM_EXT_NAME;
+import static io.swagger.codegen.v3.generators.features.NotNullAnnotationFeatures.NOT_NULL_JACKSON_ANNOTATION;
+import static io.swagger.codegen.v3.generators.handlebars.ExtensionHelper.getBooleanValue;
+
 import com.github.jknack.handlebars.Handlebars;
 import io.swagger.codegen.v3.CliOption;
 import io.swagger.codegen.v3.CodegenArgument;
@@ -26,11 +31,6 @@
 import io.swagger.v3.oas.models.parameters.RequestBody;
 import io.swagger.v3.oas.models.responses.ApiResponse;
 import io.swagger.v3.parser.util.SchemaTypeUtil;
-import org.apache.commons.lang3.BooleanUtils;
-import org.apache.commons.lang3.StringUtils;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import java.io.File;
 import java.util.Arrays;
 import java.util.HashMap;
@@ -42,11 +42,10 @@
 import java.util.Objects;
 import java.util.Optional;
 import java.util.regex.Pattern;
-
-import static io.swagger.codegen.v3.CodegenConstants.HAS_ENUMS_EXT_NAME;
-import static io.swagger.codegen.v3.CodegenConstants.IS_ENUM_EXT_NAME;
-import static io.swagger.codegen.v3.generators.features.NotNullAnnotationFeatures.NOT_NULL_JACKSON_ANNOTATION;
-import static io.swagger.codegen.v3.generators.handlebars.ExtensionHelper.getBooleanValue;
+import org.apache.commons.lang3.BooleanUtils;
+import org.apache.commons.lang3.StringUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public abstract class AbstractJavaCodegen extends DefaultCodegenConfig {
     private static Logger LOGGER = LoggerFactory.getLogger(AbstractJavaCodegen.class);
@@ -62,6 +61,8 @@ public abstract class AbstractJavaCodegen extends DefaultCodegenConfig {
 
     public static final String WIREMOCK_OPTION = "wiremock";
 
+    public static final String JAKARTA = "jakarta";
+
     protected String dateLibrary = "threetenbp";
     protected boolean java8Mode = false;
     protected boolean java11Mode = false;
@@ -93,6 +94,7 @@ public abstract class AbstractJavaCodegen extends DefaultCodegenConfig {
     protected String apiDocPath = "docs/";
     protected String modelDocPath = "docs/";
     protected boolean supportJava6= false;
+    protected boolean jakarta = false;
     private NotNullAnnotationFeatures notNullOption;
 
     public AbstractJavaCodegen() {
@@ -194,6 +196,15 @@ public AbstractJavaCodegen() {
         cliOptions.add(CliOption.newBoolean(CHECK_DUPLICATED_MODEL_NAME, "Check if there are duplicated model names (ignoring case)"));
 
         cliOptions.add(CliOption.newBoolean(WIREMOCK_OPTION, "Use wiremock to generate endpoint calls to mock on generated tests."));
+
+        cliOptions.add(CliOption.newBoolean(JAKARTA, "Use Jakarta EE (package jakarta.*) instead of Java EE (javax.*)"));
+
+        CliOption jeeSpec = CliOption.newBoolean(JAKARTA, "Use Jakarta EE (package jakarta.*) instead of Java EE (javax.*)");
+        Map<String, String> jeeSpecModeOptions = new HashMap<String, String>();
+        jeeSpecModeOptions.put("true", "Use Jakarta EE (package jakarta.*)");
+        jeeSpecModeOptions.put("false", "Use Java EE (javax.*)");
+        jeeSpec.setEnum(jeeSpecModeOptions);
+        cliOptions.add(jeeSpec);
     }
 
     @Override
@@ -490,6 +501,11 @@ public void processOpts() {
         } else if (dateLibrary.equals("legacy")) {
             additionalProperties.put("legacyDates", true);
         }
+
+        if (additionalProperties.containsKey(JAKARTA)) {
+            setJakarta(Boolean.parseBoolean(String.valueOf(additionalProperties.get(JAKARTA))));
+            additionalProperties.put(JAKARTA, jakarta);
+        }
     }
 
     private void sanitizeConfig() {
@@ -1564,6 +1580,10 @@ public void setJava11Mode(boolean java11Mode) {
         this.java11Mode = java11Mode;
     }
 
+    public void setJakarta(boolean jakarta) {
+        this.jakarta = jakarta;
+    }
+
     @Override
     public String escapeQuotationMark(String input) {
         // remove " to avoid code injection

src/main/java/io/swagger/codegen/v3/generators/java/JavaJAXRSCXFCDIServerCodegen.java

@@ -5,8 +5,6 @@
 import io.swagger.codegen.v3.CodegenProperty;
 import io.swagger.codegen.v3.SupportingFile;
 import io.swagger.codegen.v3.generators.features.BeanValidationFeatures;
-import org.apache.commons.lang3.StringUtils;
-
 import java.io.File;
 
 /**
@@ -48,7 +46,14 @@ public void processOpts() {
 
         super.processOpts();
 
-        importMapping.put("Valid", "javax.validation.Valid");
+        if (additionalProperties.containsKey(JAKARTA)) {
+            setJakarta(convertPropertyToBoolean(JAKARTA));
+        }
+        if (jakarta) {
+            importMapping.put("Valid", "jakarta.validation.Valid");
+        } else {
+            importMapping.put("Valid", "javax.validation.Valid");
+        }
 
         // Three API templates to support CDI injection
         apiTemplateFiles.put("apiService.mustache", ".java");

src/main/resources/handlebars/Java/ApiClient.mustache

@@ -29,8 +29,14 @@ import com.sun.jersey.api.client.WebResource.Builder;
 import com.sun.jersey.multipart.FormDataMultiPart;
 import com.sun.jersey.multipart.file.FileDataBodyPart;
 
+{{#jakarta}}
+import jakarta.ws.rs.core.Response.Status.Family;
+import jakarta.ws.rs.core.MediaType;
+{{/jakarta}}
+{{^jakarta}}
 import javax.ws.rs.core.Response.Status.Family;
 import javax.ws.rs.core.MediaType;
+{{/jakarta}}
 
 import java.util.Collection;
 import java.util.Collections;

src/main/resources/handlebars/Java/BeanValidationException.mustache

@@ -1,9 +1,14 @@
 package {{invokerPackage}};
 
 import java.util.Set;
-
+{{#jakarta}}
+import jakarta.validation.ConstraintViolation;
+import jakarta.validation.ValidationException;
+{{/jakarta}}
+{{^jakarta}}
 import javax.validation.ConstraintViolation;
 import javax.validation.ValidationException;
+{{/jakarta}}
 
 public class BeanValidationException extends ValidationException {
     /**

src/main/resources/handlebars/Java/build.gradle.java11.mustache

@@ -13,7 +13,7 @@ repositories {
 ext {
     swagger_annotations_version = "{{#useOas2}}1.5.24{{/useOas2}}{{^useOas2}}2.0.0{{/useOas2}}"
     jackson_version = "{{^threetenbp}}2.11.4{{/threetenbp}}{{#threetenbp}}2.6.4{{/threetenbp}}"
-    jersey_version = "1.19.4"
+    jersey_version = "{{#jakarta}}3.1.0{{/jakarta}}{{^jakarta}}1.19.4{{/jakarta}}"
     jodatime_version = "2.10.5"
     junit_version = "4.13.1"
 }
@@ -38,7 +38,7 @@ dependencies {
     {{#threetenbp}}
     implementation "com.github.joschi.jackson:jackson-datatype-threetenbp:$jackson_version"
     {{/threetenbp}}
-    implementation 'com.sun.xml.ws:jaxws-rt:2.3.3'
+    implementation 'com.sun.xml.ws:jaxws-rt:{{#jakarta}}4.0.0{{/jakarta}}{{^jakarta}}2.3.3{{/jakarta}}'
     testImplementation "junit:junit:$junit_version"
     {{#wiremock}}
     testImplementation "com.github.tomakehurst:wiremock:2.27.2"
@@ -73,4 +73,4 @@ publishing {
 
 tasks.withType(JavaCompile) {
     options.encoding = 'UTF-8'
-}
+}

src/main/resources/handlebars/Java/build.gradle.mustache

@@ -54,7 +54,7 @@ if(hasProperty('target') && target == 'android') {
         }
 
         dependencies {
-            provided 'javax.annotation:jsr250-api:1.0'
+            provided 'jakarta.annotation:jakarta.annotation-api:{{#jakarta}}2.1.1{{/jakarta}}{{^jakarta}}1.3.5{{/jakarta}}'
         }
     }
 
@@ -113,7 +113,7 @@ ext {
     swagger_annotations_version = "2.0.0"
     {{/useOas2}}
     jackson_version = "2.10.1"
-    jersey_version = "1.19.4"
+    jersey_version = "{{#jakarta}}3.1.0{{/jakarta}}{{^jakarta}}1.19.4{{/jakarta}}"
     jodatime_version = "2.9.9"
     junit_version = "4.12"
 }

src/main/resources/handlebars/Java/generatedAnnotation.mustache

@@ -1 +1 @@
-{{^hideGenerationTimestamp}}@javax.annotation.Generated(value = "{{generatorClass}}", date = "{{generatedDate}}"){{/hideGenerationTimestamp}}
+{{^hideGenerationTimestamp}}@{{#jakarta}}jakarta{{/jakarta}}{{^jakarta}}javax{{/jakarta}}.annotation.Generated(value = "{{generatorClass}}", date = "{{generatedDate}}"){{/hideGenerationTimestamp}}

src/main/resources/handlebars/Java/libraries/feign/build.gradle.java11.mustache

@@ -50,7 +50,7 @@ dependencies {
     }
     implementation "org.json:json:20180130"
     implementation "com.brsanthu:migbase64:2.2"
-    implementation "com.sun.xml.ws:jaxws-rt:2.3.3"
+    implementation "com.sun.xml.ws:jaxws-rt:{{#jakarta}}4.0.0{{/jakarta}}{{^jakarta}}2.3.3{{/jakarta}}"
     testImplementation "junit:junit:$junit_version"
     testImplementation "com.squareup.okhttp3:mockwebserver:3.6.0"
     testImplementation "org.assertj:assertj-core:1.7.1"
@@ -87,4 +87,4 @@ publishing {
 
 tasks.withType(JavaCompile) {
     options.encoding = 'UTF-8'
-}
+}

src/main/resources/handlebars/Java/libraries/feign/build.gradle.mustache

@@ -54,7 +54,7 @@ if(hasProperty('target') && target == 'android') {
         }
 
         dependencies {
-            provided 'javax.annotation:jsr250-api:1.0'
+            provided 'jakarta.annotation:jakarta.annotation-api:{{#jakarta}}2.1.1{{/jakarta}}{{^jakarta}}1.3.5{{/jakarta}}'
         }
     }