[Jaxrs-cxf] Add bean-level cascaded beanvalidation for pojos (@Valid) fix #4738 (#7807)

* add valid for pojos #4738

* add Valid to imports for pojos #4738

* Do not generate Valid-annotation for Date-types and UUID

* * add Valid-annotation to Containers.  #4738

Collection-valued, array-valued and generally Iterable fields and properties may also be decorated with the @Valid annotation. This causes the contents of the iterator to be validated.
Quoted from: http://beanvalidation.org/1.1/spec/

* add equivalent windows-bat-script for jaxrs-cxf-petstore-server.sh

* differences on jaxrs-cxf-generated-classes introduced by other changes (not related to this PR)

bin/windows/jaxrs-cxf-petstore-server.bat

@@ -0,0 +1,10 @@
+set executable=.\modules\swagger-codegen-cli\target\swagger-codegen-cli.jar
+
+If Not Exist %executable% (
+  mvn clean package
+)
+
+REM set JAVA_OPTS=%JAVA_OPTS% -Xmx1024M -DloggerPath=conf/log4j.properties
+set ags=generate -t modules\swagger-codegen\src\main\resources\JavaJaxRS\cxf -i modules\swagger-codegen\src\test\resources\2_0\petstore-with-fake-endpoints-models-for-testing.yaml -l jaxrs-cxf -o samples\server\petstore\jaxrs-cxf -DhideGenerationTimestamp=true
+
+java %JAVA_OPTS% -jar %executable% %ags%

modules/swagger-codegen/src/main/resources/JavaJaxRS/cxf/model.mustache

@@ -4,6 +4,7 @@ package {{package}};
 {{/imports}}
 {{#useBeanValidation}}
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 {{/useBeanValidation}}
 
 {{#models}}

modules/swagger-codegen/src/main/resources/JavaJaxRS/cxf/pojo.mustache

@@ -31,10 +31,11 @@ public class {{classname}} {{#parent}}extends {{{parent}}}{{/parent}} {
 {{#withXml}}
   @XmlElement(name="{{baseName}}"{{#required}}, required = {{required}}{{/required}})
 {{/withXml}}
-  @ApiModelProperty({{#example}}example = "{{{example}}}", {{/example}}{{#required}}required = {{required}}, {{/required}}value = "{{{description}}}")
+  @ApiModelProperty({{#example}}example = "{{{example}}}", {{/example}}{{#required}}required = {{required}}, {{/required}}value = "{{{description}}}"){{^isPrimitiveType}}{{^isDate}}{{^isDateTime}}{{^isString}}{{^isFile}}{{#useBeanValidation}}
+  @Valid{{/useBeanValidation}}{{/isFile}}{{/isString}}{{/isDateTime}}{{/isDate}}{{/isPrimitiveType}}
 {{#description}}
  /**
-   * {{{description}}}  
+   * {{{description}}}
   **/
 {{/description}}
 {{#isContainer}}

samples/server/petstore/jaxrs-cxf/.swagger-codegen/VERSION

@@ -1 +1 @@
-2.3.0-SNAPSHOT
+2.4.0-SNAPSHOT

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/api/FakeClassnameTags123Api.java

@@ -30,6 +30,8 @@ import javax.validation.Valid;
 public interface FakeClassnameTags123Api  {
 
     /**
+     * To test class name in snake case
+     *
      * To test class name in snake case
      *
      */

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/api/StoreApi.java

@@ -73,7 +73,7 @@ public interface StoreApi  {
         @ApiResponse(code = 200, message = "successful operation", response = Order.class),
         @ApiResponse(code = 400, message = "Invalid ID supplied"),
         @ApiResponse(code = 404, message = "Order not found") })
-    public Order getOrderById(@PathParam("order_id") @Min(1) @Max(5) Long orderId);
+    public Order getOrderById(@PathParam("order_id") @Min(1L) @Max(5L) Long orderId);
 
     /**
      * Place an order for a pet

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/AdditionalPropertiesClass.java

@@ -4,6 +4,7 @@ import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -21,6 +22,7 @@ public class AdditionalPropertiesClass  {
   private Map<String, String> mapProperty = null;
 
   @ApiModelProperty(value = "")
+  @Valid
   private Map<String, Map<String, String>> mapOfMapProperty = null;
  /**
    * Get mapProperty

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Animal.java

@@ -3,6 +3,7 @@ package io.swagger.model;
 import com.fasterxml.jackson.annotation.JsonSubTypes;
 import com.fasterxml.jackson.annotation.JsonTypeInfo;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/AnimalFarm.java

@@ -4,6 +4,7 @@ import io.swagger.model.Animal;
 import java.util.ArrayList;
 import java.util.List;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/ArrayOfArrayOfNumberOnly.java

@@ -4,6 +4,7 @@ import java.math.BigDecimal;
 import java.util.ArrayList;
 import java.util.List;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -18,6 +19,7 @@ import com.fasterxml.jackson.annotation.JsonProperty;
 public class ArrayOfArrayOfNumberOnly  {
   
   @ApiModelProperty(value = "")
+  @Valid
   private List<List<BigDecimal>> arrayArrayNumber = null;
  /**
    * Get arrayArrayNumber

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/ArrayOfNumberOnly.java

@@ -4,6 +4,7 @@ import java.math.BigDecimal;
 import java.util.ArrayList;
 import java.util.List;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -18,6 +19,7 @@ import com.fasterxml.jackson.annotation.JsonProperty;
 public class ArrayOfNumberOnly  {
   
   @ApiModelProperty(value = "")
+  @Valid
   private List<BigDecimal> arrayNumber = null;
  /**
    * Get arrayNumber

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/ArrayTest.java

@@ -4,6 +4,7 @@ import io.swagger.model.ReadOnlyFirst;
 import java.util.ArrayList;
 import java.util.List;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -21,9 +22,11 @@ public class ArrayTest  {
   private List<String> arrayOfString = null;
 
   @ApiModelProperty(value = "")
+  @Valid
   private List<List<Long>> arrayArrayOfInteger = null;
 
   @ApiModelProperty(value = "")
+  @Valid
   private List<List<ReadOnlyFirst>> arrayArrayOfModel = null;
  /**
    * Get arrayOfString

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Capitalization.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -31,7 +32,7 @@ public class Capitalization  {
 
   @ApiModelProperty(value = "Name of the pet ")
  /**
-   * Name of the pet   
+   * Name of the pet 
   **/
   private String ATT_NAME = null;
  /**

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Cat.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import io.swagger.model.Animal;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Category.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/ClassModel.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import io.swagger.annotations.ApiModel;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Client.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Dog.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import io.swagger.model.Animal;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/EnumArrays.java

@@ -3,6 +3,7 @@ package io.swagger.model;
 import java.util.ArrayList;
 import java.util.List;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/EnumClass.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonValue;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/EnumTest.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import io.swagger.model.OuterEnum;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -52,6 +53,42 @@ public enum EnumStringEnum {
   private EnumStringEnum enumString = null;
 
 
+@XmlType(name="EnumStringRequiredEnum")
+@XmlEnum(String.class)
+public enum EnumStringRequiredEnum {
+
+@XmlEnumValue("UPPER") UPPER(String.valueOf("UPPER")), @XmlEnumValue("lower") LOWER(String.valueOf("lower")), @XmlEnumValue("") EMPTY(String.valueOf(""));
+
+
+    private String value;
+
+    EnumStringRequiredEnum (String v) {
+        value = v;
+    }
+
+    public String value() {
+        return value;
+    }
+
+    @Override
+    public String toString() {
+        return String.valueOf(value);
+    }
+
+    public static EnumStringRequiredEnum fromValue(String v) {
+        for (EnumStringRequiredEnum b : EnumStringRequiredEnum.values()) {
+            if (String.valueOf(b.value).equals(v)) {
+                return b;
+            }
+        }
+        return null;
+    }
+}
+
+  @ApiModelProperty(required = true, value = "")
+  private EnumStringRequiredEnum enumStringRequired = null;
+
+
 @XmlType(name="EnumIntegerEnum")
 @XmlEnum(Integer.class)
 public enum EnumIntegerEnum {
@@ -124,6 +161,7 @@ public enum EnumNumberEnum {
   private EnumNumberEnum enumNumber = null;
 
   @ApiModelProperty(value = "")
+  @Valid
   private OuterEnum outerEnum = null;
  /**
    * Get enumString
@@ -146,6 +184,28 @@ public enum EnumNumberEnum {
     return this;
   }
 
+ /**
+   * Get enumStringRequired
+   * @return enumStringRequired
+  **/
+  @JsonProperty("enum_string_required")
+  @NotNull
+  public String getEnumStringRequired() {
+    if (enumStringRequired == null) {
+      return null;
+    }
+    return enumStringRequired.value();
+  }
+
+  public void setEnumStringRequired(EnumStringRequiredEnum enumStringRequired) {
+    this.enumStringRequired = enumStringRequired;
+  }
+
+  public EnumTest enumStringRequired(EnumStringRequiredEnum enumStringRequired) {
+    this.enumStringRequired = enumStringRequired;
+    return this;
+  }
+
  /**
    * Get enumInteger
    * @return enumInteger
@@ -213,6 +273,7 @@ public enum EnumNumberEnum {
     sb.append("class EnumTest {\n");
     
     sb.append("    enumString: ").append(toIndentedString(enumString)).append("\n");
+    sb.append("    enumStringRequired: ").append(toIndentedString(enumStringRequired)).append("\n");
     sb.append("    enumInteger: ").append(toIndentedString(enumInteger)).append("\n");
     sb.append("    enumNumber: ").append(toIndentedString(enumNumber)).append("\n");
     sb.append("    outerEnum: ").append(toIndentedString(outerEnum)).append("\n");

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/FormatTest.java

@@ -5,6 +5,7 @@ import java.util.Date;
 import java.util.UUID;
 import org.joda.time.LocalDate;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -28,6 +29,7 @@ public class FormatTest  {
   private Long int64 = null;
 
   @ApiModelProperty(required = true, value = "")
+  @Valid
   private BigDecimal number = null;
 
   @ApiModelProperty(value = "")

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/HasOnlyReadOnly.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/MapTest.java

@@ -4,6 +4,7 @@ import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -18,6 +19,7 @@ import com.fasterxml.jackson.annotation.JsonProperty;
 public class MapTest  {
   
   @ApiModelProperty(value = "")
+  @Valid
   private Map<String, Map<String, String>> mapMapOfString = null;
 
 

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/MixedPropertiesAndAdditionalPropertiesClass.java

@@ -7,6 +7,7 @@ import java.util.List;
 import java.util.Map;
 import java.util.UUID;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -27,6 +28,7 @@ public class MixedPropertiesAndAdditionalPropertiesClass  {
   private Date dateTime = null;
 
   @ApiModelProperty(value = "")
+  @Valid
   private Map<String, Animal> map = null;
  /**
    * Get uuid

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Model200Response.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import io.swagger.annotations.ApiModel;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/ModelApiResponse.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/ModelReturn.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import io.swagger.annotations.ApiModel;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Name.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import io.swagger.annotations.ApiModel;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/NumberOnly.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import java.math.BigDecimal;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -16,6 +17,7 @@ import com.fasterxml.jackson.annotation.JsonProperty;
 public class NumberOnly  {
   
   @ApiModelProperty(value = "")
+  @Valid
   private BigDecimal justNumber = null;
  /**
    * Get justNumber

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Order.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import java.util.Date;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -62,7 +63,7 @@ public enum StatusEnum {
 
   @ApiModelProperty(value = "Order Status")
  /**
-   * Order Status  
+   * Order Status
   **/
   private StatusEnum status = null;
 

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/OuterComposite.java

@@ -2,6 +2,7 @@ package io.swagger.model;
 
 import java.math.BigDecimal;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -16,6 +17,7 @@ import com.fasterxml.jackson.annotation.JsonProperty;
 public class OuterComposite  {
   
   @ApiModelProperty(value = "")
+  @Valid
   private BigDecimal myNumber = null;
 
   @ApiModelProperty(value = "")

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/OuterEnum.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonValue;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Pet.java

@@ -5,6 +5,7 @@ import io.swagger.model.Tag;
 import java.util.ArrayList;
 import java.util.List;
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -22,6 +23,7 @@ public class Pet  {
   private Long id = null;
 
   @ApiModelProperty(value = "")
+  @Valid
   private Category category = null;
 
   @ApiModelProperty(example = "doggie", required = true, value = "")
@@ -31,6 +33,7 @@ public class Pet  {
   private List<String> photoUrls = new ArrayList<String>();
 
   @ApiModelProperty(value = "")
+  @Valid
   private List<Tag> tags = null;
 
 
@@ -68,7 +71,7 @@ public enum StatusEnum {
 
   @ApiModelProperty(value = "pet status in the store")
  /**
-   * pet status in the store  
+   * pet status in the store
   **/
   private StatusEnum status = null;
  /**

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/ReadOnlyFirst.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/SpecialModelName.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/Tag.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;

samples/server/petstore/jaxrs-cxf/src/gen/java/io/swagger/model/User.java

@@ -1,6 +1,7 @@
 package io.swagger.model;
 
 import javax.validation.constraints.*;
+import javax.validation.Valid;
 
 import io.swagger.annotations.ApiModelProperty;
 import javax.xml.bind.annotation.XmlElement;
@@ -37,7 +38,7 @@ public class User  {
 
   @ApiModelProperty(value = "User Status")
  /**
-   * User Status  
+   * User Status
   **/
   private Integer userStatus = null;
  /**

samples/server/petstore/jaxrs-cxf/src/main/java/io/swagger/api/impl/FakeClassnameTags123ApiServiceImpl.java

@@ -24,6 +24,8 @@ import io.swagger.annotations.Api;
  */
 public class FakeClassnameTags123ApiServiceImpl implements FakeClassnameTags123Api {
     /**
+     * To test class name in snake case
+     *
      * To test class name in snake case
      *
      */