Changes report #1829.

bnasslahsen · bnasslahsen · commit 3593af0b3f52 · 2022-10-14T15:05:34.000+02:00
diff --git a/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/configuration/SpringDocHints.java b/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/configuration/SpringDocHints.java
@@ -51,7 +51,7 @@
  * @author bnasslahsen
  */
 @Lazy(false)
-@ConditionalOnExpression("${springdoc.api-docs.enabled:true} and ${springdoc.enable-native-support:true}")
+@ConditionalOnExpression("${springdoc.api-docs.enabled:true} and ${springdoc.enable-native-support:false}")
 @ConditionalOnWebApplication
 @Configuration(proxyBeanMethods = false)
 @ConditionalOnBean(SpringDocConfiguration.class)
diff --git a/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/properties/SpringDocConfigProperties.java b/springdoc-openapi-starter-common/src/main/java/org/springdoc/core/properties/SpringDocConfigProperties.java
@@ -214,7 +214,7 @@ public class SpringDocConfigProperties {
 	/**
 	 * The Enable native support.
 	 */
-	private boolean enableNativeSupport = true;
+	private boolean enableNativeSupport = false;
 
 	/**
 	 * The Enable hateoas.
@@ -229,7 +229,7 @@ public class SpringDocConfigProperties {
 	/**
 	 * convert query param to form data when consumes is multipart/form-data
 	 */
-	private boolean defaultSupportFormData=true;
+	private boolean defaultSupportFormData;
 
 	/**
 	 * The Sort converter.
diff --git a/springdoc-openapi-starter-webmvc-api/src/test/java/test/org/springdoc/api/v30/app189/SpringDocApp189Test.java b/springdoc-openapi-starter-webmvc-api/src/test/java/test/org/springdoc/api/v30/app189/SpringDocApp189Test.java
@@ -25,7 +25,9 @@
 import test.org.springdoc.api.v30.AbstractSpringDocV30Test;
 
 import org.springframework.boot.autoconfigure.SpringBootApplication;
+import org.springframework.test.context.TestPropertySource;
 
+@TestPropertySource(properties = "springdoc.default-support-form-data=true")
 public class SpringDocApp189Test extends AbstractSpringDocV30Test {
 
     @SpringBootApplication
diff --git a/springdoc-openapi-starter-webmvc-api/src/test/resources/results/3.0.1/app105-3.json b/springdoc-openapi-starter-webmvc-api/src/test/resources/results/3.0.1/app105-3.json
@@ -331,6 +331,15 @@
               "type": "integer",
               "format": "int64"
             }
+          },
+          {
+            "name": "additionalMetadata",
+            "in": "query",
+            "description": "Additional data to pass to server",
+            "required": false,
+            "schema": {
+              "type": "string"
+            }
           }
         ],
         "requestBody": {
@@ -346,9 +355,6 @@
                     "type": "string",
                     "description": "file detail",
                     "format": "binary"
-                  },
-                  "additionalMetadata": {
-                    "description": "Additional data to pass to server"
                   }
                 }
               }
diff --git a/springdoc-openapi-starter-webmvc-api/src/test/resources/results/3.0.1/app2.json b/springdoc-openapi-starter-webmvc-api/src/test/resources/results/3.0.1/app2.json
@@ -655,6 +655,15 @@
               "type": "integer",
               "format": "int64"
             }
+          },
+          {
+            "name": "additionalMetadata",
+            "in": "query",
+            "description": "Additional data to pass to server",
+            "required": false,
+            "schema": {
+              "type": "string"
+            }
           }
         ],
         "requestBody": {
@@ -670,9 +679,6 @@
                     "type": "string",
                     "description": "file detail",
                     "format": "binary"
-                  },
-                  "additionalMetadata": {
-                    "description": "Additional data to pass to server"
                   }
                 }
               }
@@ -835,8 +841,8 @@
               }
             }
           },
-          "200": {
-            "description": "successful operation",
+          "404": {
+            "description": "Order not found",
             "content": {
               "application/xml": {
                 "schema": {
@@ -850,8 +856,8 @@
               }
             }
           },
-          "404": {
-            "description": "Order not found",
+          "200": {
+            "description": "successful operation",
             "content": {
               "application/xml": {
                 "schema": {
diff --git a/springdoc-openapi-starter-webmvc-api/src/test/resources/results/3.1.0/app2.json b/springdoc-openapi-starter-webmvc-api/src/test/resources/results/3.1.0/app2.json
@@ -631,6 +631,15 @@
               "type": "integer",
               "format": "int64"
             }
+          },
+          {
+            "name": "additionalMetadata",
+            "in": "query",
+            "description": "Additional data to pass to server",
+            "required": false,
+            "schema": {
+              "type": "string"
+            }
           }
         ],
         "requestBody": {
@@ -643,9 +652,6 @@
                     "type": "string",
                     "format": "binary",
                     "description": "file detail"
-                  },
-                  "additionalMetadata": {
-                    "description": "Additional data to pass to server"
                   }
                 },
                 "required": [
@@ -803,8 +809,8 @@
               }
             }
           },
-          "404": {
-            "description": "Order not found",
+          "200": {
+            "description": "successful operation",
             "content": {
               "application/xml": {
                 "schema": {
@@ -818,8 +824,8 @@
               }
             }
           },
-          "200": {
-            "description": "successful operation",
+          "404": {
+            "description": "Order not found",
             "content": {
               "application/xml": {
                 "schema": {
diff --git a/springdoc-openapi-tests/springdoc-openapi-actuator-webmvc-tests/src/test/resources/results/app68-3.json b/springdoc-openapi-tests/springdoc-openapi-actuator-webmvc-tests/src/test/resources/results/app68-3.json
@@ -331,6 +331,15 @@
               "type": "integer",
               "format": "int64"
             }
+          },
+          {
+            "name": "additionalMetadata",
+            "in": "query",
+            "description": "Additional data to pass to server",
+            "required": false,
+            "schema": {
+              "type": "string"
+            }
           }
         ],
         "requestBody": {
@@ -346,9 +355,6 @@
                     "type": "string",
                     "description": "file detail",
                     "format": "binary"
-                  },
-                  "additionalMetadata": {
-                    "description": "Additional data to pass to server"
                   }
                 }
               }
diff --git a/springdoc-openapi-tests/springdoc-openapi-javadoc-tests/src/test/resources/results/app105-3.json b/springdoc-openapi-tests/springdoc-openapi-javadoc-tests/src/test/resources/results/app105-3.json
diff --git a/springdoc-openapi-tests/springdoc-openapi-javadoc-tests/src/test/resources/results/app2.json b/springdoc-openapi-tests/springdoc-openapi-javadoc-tests/src/test/resources/results/app2.json
diff --git a/springdoc-openapi-tests/springdoc-openapi-native-tests/src/test/java/test/org/springdoc/api/app1/TestApplicationTests.java b/springdoc-openapi-tests/springdoc-openapi-native-tests/src/test/java/test/org/springdoc/api/app1/TestApplicationTests.java

Original file line number	Diff line number	Diff line change
`@@ -331,6 +331,15 @@`
`331`	`331`	`"type": "integer",`
`332`	`332`	`"format": "int64"`
`333`	`333`	`}`
	`334`	`+ },`
	`335`	`+ {`
	`336`	`+ "name": "additionalMetadata",`
	`337`	`+ "in": "query",`
	`338`	`+ "description": "Additional data to pass to server",`
	`339`	`+ "required": false,`
	`340`	`+ "schema": {`
	`341`	`+ "type": "string"`
	`342`	`+ }`
`334`	`343`	`}`
`335`	`344`	`],`
`336`	`345`	`"requestBody": {`
`@@ -346,9 +355,6 @@`
`346`	`355`	`"type": "string",`
`347`	`356`	`"description": "file detail",`
`348`	`357`	`"format": "binary"`
`349`		`- },`
`350`		`- "additionalMetadata": {`
`351`		`- "description": "Additional data to pass to server"`
`352`	`358`	`}`
`353`	`359`	`}`
`354`	`360`	`}`
Original file line number	Diff line number	Diff line change
`@@ -655,6 +655,15 @@`
`655`	`655`	`"type": "integer",`
`656`	`656`	`"format": "int64"`
`657`	`657`	`}`
	`658`	`+ },`
	`659`	`+ {`
	`660`	`+ "name": "additionalMetadata",`
	`661`	`+ "in": "query",`
	`662`	`+ "description": "Additional data to pass to server",`
	`663`	`+ "required": false,`
	`664`	`+ "schema": {`
	`665`	`+ "type": "string"`
	`666`	`+ }`
`658`	`667`	`}`
`659`	`668`	`],`
`660`	`669`	`"requestBody": {`
`@@ -670,9 +679,6 @@`
`670`	`679`	`"type": "string",`
`671`	`680`	`"description": "file detail",`
`672`	`681`	`"format": "binary"`
`673`		`- },`
`674`		`- "additionalMetadata": {`
`675`		`- "description": "Additional data to pass to server"`
`676`	`682`	`}`
`677`	`683`	`}`
`678`	`684`	`}`
`@@ -835,8 +841,8 @@`
`835`	`841`	`}`
`836`	`842`	`}`
`837`	`843`	`},`
`838`		`- "200": {`
`839`		`- "description": "successful operation",`
	`844`	`+ "404": {`
	`845`	`+ "description": "Order not found",`
`840`	`846`	`"content": {`
`841`	`847`	`"application/xml": {`
`842`	`848`	`"schema": {`
`@@ -850,8 +856,8 @@`
`850`	`856`	`}`
`851`	`857`	`}`
`852`	`858`	`},`
`853`		`- "404": {`
`854`		`- "description": "Order not found",`
	`859`	`+ "200": {`
	`860`	`+ "description": "successful operation",`
`855`	`861`	`"content": {`
`856`	`862`	`"application/xml": {`
`857`	`863`	`"schema": {`