fix: stops reprocessing parameters extracted from nested BeanParam Fixes: #3973 #4334

Author: vitormelo95

modules/swagger-jaxrs2/src/main/java/io/swagger/v3/jaxrs2/DefaultParameterExtension.java

@@ -258,36 +258,17 @@ private boolean handleAdditionalAnnotation(List<Parameter> parameters, List<Para
                         resolvedParameter.parameters;
 
                 for (Parameter p : extractedParameters) {
-                    Parameter processedParam = ParameterProcessor.applyAnnotations(
-                            p,
-                            paramType,
-                            paramAnnotations,
-                            components,
-                            classConsumes == null ? new String[0] : classConsumes.value(),
-                            methodConsumes == null ? new String[0] : methodConsumes.value(),
-                            jsonViewAnnotation,
-                            configuration);
-                    if (processedParam != null) {
-                        parameters.add(processedParam);
+                    if (p != null) {
+                        parameters.add(p);
                     }
                 }
 
                 List<Parameter> extractedFormParameters =
                         resolvedParameter.formParameters;
 
                 for (Parameter p : extractedFormParameters) {
-                    Parameter processedParam = ParameterProcessor.applyAnnotations(
-                            p,
-                            paramType,
-                            paramAnnotations,
-                            components,
-                            classConsumes == null ? new String[0] : classConsumes.value(),
-                            methodConsumes == null ? new String[0] : methodConsumes.value(),
-                            jsonViewAnnotation,
-                            openapi31,
-                            this.schemaResolution);
-                    if (processedParam != null) {
-                        formParameters.add(processedParam);
+                    if (p != null) {
+                        formParameters.add(p);
                     }
                 }
 

modules/swagger-jaxrs2/src/test/java/io/swagger/v3/jaxrs2/BeanParamTest.java

@@ -6,6 +6,9 @@
 import javax.ws.rs.GET;
 import javax.ws.rs.Path;
 
+import io.swagger.v3.jaxrs2.resources.model.NestedBeanParam;
+import io.swagger.v3.oas.models.media.IntegerSchema;
+import org.testng.Assert;
 import org.testng.annotations.Test;
 
 import io.swagger.v3.jaxrs2.resources.model.ListOfStringsBeanParam;
@@ -24,6 +27,13 @@ private static class MyBeanParamResource {
         public String getWithBeanParam(@BeanParam ListOfStringsBeanParam listOfStringsBean) {
             return "result";
         }
+
+        @GET
+        @Path("/nested-param")
+        public String getWithNestedBeanParam(@BeanParam NestedBeanParam listOfStringsBean) {
+            return "result";
+        }
+
     }
 
     @Test(description = "check array type of serialized BeanParam containing QueryParams") // tests issue #2466
@@ -39,4 +49,18 @@ public void shouldSerializeTypeParameter() {
         assertEquals(schema.getItems().getType(), "string");
     }
 
+    @Test(description = "check integer type of nested BeanParam containing a QueryParam") // tests issue #2466
+    public void shouldSerializeNestedTypeParameter() {
+        OpenAPI openApi = new Reader(new OpenAPI()).read(MyBeanParamResource.class);
+        List<Parameter> getOperationParams = openApi.getPaths().get("/nested-param").getGet().getParameters();
+        Assert.assertEquals(getOperationParams.size(), 1);
+
+        Parameter queryParam = getOperationParams.get(0);
+        Assert.assertEquals(queryParam.getName(), "queryParam");
+        Schema<?> schema = queryParam.getSchema();
+        Assert.assertEquals(schema.getClass(), IntegerSchema.class);
+        Assert.assertEquals(schema.getType(), "integer");
+        Assert.assertEquals(((IntegerSchema) schema).getDefault(), 10);
+    }
+
 }

modules/swagger-jaxrs2/src/test/java/io/swagger/v3/jaxrs2/FormParamBeanTest.java

@@ -1,43 +1,20 @@
 package io.swagger.v3.jaxrs2;
 
+import io.swagger.v3.jaxrs2.resources.model.FormParamBean;
 import io.swagger.v3.oas.models.OpenAPI;
 import io.swagger.v3.oas.models.media.MediaType;
 import io.swagger.v3.oas.models.media.Schema;
 import io.swagger.v3.oas.models.media.StringSchema;
 import io.swagger.v3.oas.models.parameters.RequestBody;
+import org.testng.Assert;
 import org.testng.annotations.Test;
 
 import javax.ws.rs.*;
 
-import static org.testng.Assert.assertEquals;
 import static org.testng.Assert.assertNotNull;
 
 public class FormParamBeanTest {
 
-    private static class FormParamBean {
-        @FormParam(value = "param1")
-        private String param1;
-
-        @FormParam(value = "param2")
-        private String param2;
-
-        public String getParam1() {
-            return param1;
-        }
-
-        public void setParam1(String param1) {
-            this.param1 = param1;
-        }
-
-        public String getParam2() {
-            return param2;
-        }
-
-        public void setParam2(String param2) {
-            this.param2 = param2;
-        }
-    }
-
     @Path("/")
     private static class MyFormBeanParamResource {
         @GET
@@ -55,9 +32,10 @@ public void shouldSerializeTypeParameter() {
         MediaType mediaType = requestBody.getContent().get("application/x-www-form-urlencoded");
         assertNotNull(mediaType);
         Schema schema = mediaType.getSchema();
-        assertEquals(schema.getProperties().size(), 2);
-        assertEquals(schema.getProperties().get("param1"), new StringSchema());
-        assertEquals(schema.getProperties().get("param2"), new StringSchema());
+        Assert.assertEquals(schema.getProperties().size(), 3);
+        Assert.assertEquals(schema.getProperties().get("param1"), new StringSchema());
+        Assert.assertEquals(schema.getProperties().get("param2"), new StringSchema());
+        Assert.assertEquals(schema.getProperties().get("param3"), new StringSchema());
     }
 
 }

modules/swagger-jaxrs2/src/test/java/io/swagger/v3/jaxrs2/resources/model/FormParamBean.java

@@ -0,0 +1,52 @@
+package io.swagger.v3.jaxrs2.resources.model;
+
+import javax.ws.rs.BeanParam;
+import javax.ws.rs.FormParam;
+
+public class FormParamBean {
+
+    @FormParam(value = "param1")
+    private String param1;
+
+    @FormParam(value = "param2")
+    private String param2;
+
+    @BeanParam
+    private NestedBeanParam nestedParams;
+
+    public String getParam1() {
+        return param1;
+    }
+
+    public void setParam1(String param1) {
+        this.param1 = param1;
+    }
+
+    public String getParam2() {
+        return param2;
+    }
+
+    public void setParam2(String param2) {
+        this.param2 = param2;
+    }
+
+    public NestedBeanParam getNestedParams() {
+        return nestedParams;
+    }
+
+    public void setNestedParams(final NestedBeanParam nestedParams) {
+        this.nestedParams = nestedParams;
+    }
+
+    private static class NestedBeanParam {
+        @FormParam(value = "param3")
+        private String param3;
+        public String getParam3() {
+            return param3;
+        }
+
+        public void setParam3(String param3) {
+            this.param3 = param3;
+        }
+    }
+}

modules/swagger-jaxrs2/src/test/java/io/swagger/v3/jaxrs2/resources/model/NestedBeanParam.java

@@ -0,0 +1,32 @@
+package io.swagger.v3.jaxrs2.resources.model;
+
+import javax.ws.rs.BeanParam;
+import javax.ws.rs.DefaultValue;
+import javax.ws.rs.QueryParam;
+
+public class NestedBeanParam {
+
+    @BeanParam
+    private NestedClass nestedParams;
+
+    public NestedClass getNestedParams() {
+        return nestedParams;
+    }
+
+    public void setNestedParams(final NestedClass nestedParams) {
+        this.nestedParams = nestedParams;
+    }
+
+    public class NestedClass {
+        @QueryParam(value = "queryParam")
+        @DefaultValue("10")
+        private Integer queryParam = 10;
+        public Integer getQueryParam() {
+            return queryParam;
+        }
+
+        public void setQueryParam(final Integer queryParam) {
+            this.queryParam = queryParam;
+        }
+    }
+}