Skip to content

Commit 6024bed

Browse files
bnasslahsenbnasslahsen
committed
Changes report #1793 
1 parent f34b9be commit 6024bed

File tree

7 files changed

+269
-2
lines changed

7 files changed

+269
-2
lines changed

‎springdoc-openapi-starter-common/src/main/java/org/springdoc/core/properties/SpringDocConfigProperties.java‎

Lines changed: 26 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -226,11 +226,34 @@ public class SpringDocConfigProperties {
226226
*/
227227
private boolean enableDataRest = true;
228228

229+
/**
230+
* convert query param to form data when consumes is multipart/form-data
231+
*/
232+
private boolean defaultSupportFormData=true;
233+
229234
/**
230235
* The Sort converter.
231236
*/
232237
private SortConverter sortConverter = new SortConverter();
233238

239+
/**
240+
* Is default support form data boolean.
241+
*
242+
* @return the boolean
243+
*/
244+
public boolean isDefaultSupportFormData() {
245+
return defaultSupportFormData;
246+
}
247+
248+
/**
249+
* Sets default support form data.
250+
*
251+
* @param defaultSupportFormData the default support form data
252+
*/
253+
public void setDefaultSupportFormData(boolean defaultSupportFormData) {
254+
this.defaultSupportFormData = defaultSupportFormData;
255+
}
256+
234257
/**
235258
* Is default flat param object boolean.
236259
*
@@ -1080,7 +1103,7 @@ public void setEnabled(boolean enabled) {
10801103

10811104
/**
10821105
* The type Sort converter.
1083-
* @author daniel-shuy
1106+
* @author daniel -shuy
10841107
*/
10851108
public static class SortConverter {
10861109

@@ -1107,6 +1130,7 @@ public void setEnabled(boolean enabled) {
11071130
this.enabled = enabled;
11081131
}
11091132
}
1133+
11101134
/**
11111135
* The type Webjars.
11121136
* @author bnasslahsen
@@ -1600,4 +1624,5 @@ public void setDisplayName(String displayName) {
16001624
this.displayName = displayName;
16011625
}
16021626
}
1627+
16031628
}

‎springdoc-openapi-starter-common/src/main/java/org/springdoc/core/service/AbstractRequestService.java‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,11 @@ public abstract class AbstractRequestService {
177177
*/
178178
private final boolean defaultFlatParamObject;
179179

180+
/**
181+
* The Default support form data.
182+
*/
183+
private boolean defaultSupportFormData;
184+
180185
/**
181186
* Instantiates a new Abstract request builder.
182187
*
@@ -197,6 +202,7 @@ protected AbstractRequestService(GenericParameterService parameterBuilder, Reque
197202
this.parameterCustomizers = parameterCustomizers;
198203
this.localSpringDocParameterNameDiscoverer = localSpringDocParameterNameDiscoverer;
199204
this.defaultFlatParamObject = parameterBuilder.getPropertyResolverUtils().getSpringDocConfigProperties().isDefaultFlatParamObject();
205+
this.defaultSupportFormData = parameterBuilder.getPropertyResolverUtils().getSpringDocConfigProperties().isDefaultSupportFormData();
200206
}
201207

202208
/**
@@ -325,7 +331,7 @@ else if (!RequestMethod.GET.equals(requestMethod)) {
325331
LinkedHashMap<String, Parameter> map = getParameterLinkedHashMap(components, methodAttributes, operationParameters, parametersDocMap);
326332
RequestBody requestBody = requestBodyInfo.getRequestBody();
327333
// support form-data
328-
if (requestBody != null
334+
if (defaultSupportFormData && requestBody != null
329335
&& requestBody.getContent() != null
330336
&& requestBody.getContent().containsKey(org.springframework.http.MediaType.MULTIPART_FORM_DATA_VALUE)) {
331337
io.swagger.v3.oas.models.media.Schema<?> mergedSchema = requestBodyInfo.getMergedSchema();

‎springdoc-openapi-starter-webmvc-api/src/test/java/test/org/springdoc/api/v30/app194/ArticleApi.java‎

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
package test.org.springdoc.api.v30.app194;
2+
3+
import java.util.LinkedList;
4+
import java.util.List;
5+
6+
import io.swagger.v3.oas.annotations.tags.Tag;
7+
import org.springdoc.core.annotations.ParameterObject;
8+
9+
import org.springframework.http.MediaType;
10+
import org.springframework.web.bind.annotation.GetMapping;
11+
import org.springframework.web.bind.annotation.PostMapping;
12+
import org.springframework.web.bind.annotation.RequestPart;
13+
import org.springframework.web.bind.annotation.RestController;
14+
import org.springframework.web.multipart.MultipartFile;
15+
16+
@Tag(name = "Article Api")
17+
@RestController("/article")
18+
public class ArticleApi {
19+
20+
@GetMapping("query")
21+
public List<ArticleDto> query(@ParameterObject ArticleQueryCondition condition) {
22+
return new LinkedList<>();
23+
}
24+
25+
@PostMapping(value = "create", consumes = MediaType.MULTIPART_FORM_DATA_VALUE)
26+
public ArticleDto create(@ParameterObject ArticleDto dto, @RequestPart MultipartFile file) {
27+
return new ArticleDto();
28+
}
29+
}

‎springdoc-openapi-starter-webmvc-api/src/test/java/test/org/springdoc/api/v30/app194/ArticleDto.java‎

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
package test.org.springdoc.api.v30.app194;
2+
3+
import io.swagger.v3.oas.annotations.media.Schema;
4+
5+
@Schema
6+
public class ArticleDto {
7+
@Schema(description = "title")
8+
private String title;
9+
10+
@Schema(description = "content")
11+
private String content;
12+
13+
public String getTitle() {
14+
return title;
15+
}
16+
17+
public void setTitle(String title) {
18+
this.title = title;
19+
}
20+
21+
public String getContent() {
22+
return content;
23+
}
24+
25+
public void setContent(String content) {
26+
this.content = content;
27+
}
28+
}

‎springdoc-openapi-starter-webmvc-api/src/test/java/test/org/springdoc/api/v30/app194/ArticleQueryCondition.java‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
package test.org.springdoc.api.v30.app194;
2+
3+
import io.swagger.v3.oas.annotations.media.Schema;
4+
@Schema
5+
public class ArticleQueryCondition {
6+
@Schema(description = "title")
7+
private String title;
8+
9+
public String getTitle() {
10+
return title;
11+
}
12+
13+
public void setTitle(String title) {
14+
this.title = title;
15+
}
16+
}

‎springdoc-openapi-starter-webmvc-api/src/test/java/test/org/springdoc/api/v30/app194/SpringDocApp194Test.java‎

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
/*
2+
*
3+
* *
4+
* * *
5+
* * * * Copyright 2019-2022 the original author or authors.
6+
* * * *
7+
* * * * Licensed under the Apache License, Version 2.0 (the "License");
8+
* * * * you may not use this file except in compliance with the License.
9+
* * * * You may obtain a copy of the License at
10+
* * * *
11+
* * * * https://www.apache.org/licenses/LICENSE-2.0
12+
* * * *
13+
* * * * Unless required by applicable law or agreed to in writing, software
14+
* * * * distributed under the License is distributed on an "AS IS" BASIS,
15+
* * * * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
16+
* * * * See the License for the specific language governing permissions and
17+
* * * * limitations under the License.
18+
* * *
19+
* *
20+
*
21+
*/
22+
23+
package test.org.springdoc.api.v30.app194;
24+
25+
import test.org.springdoc.api.v30.AbstractSpringDocV30Test;
26+
27+
import org.springframework.boot.autoconfigure.SpringBootApplication;
28+
import org.springframework.test.context.TestPropertySource;
29+
30+
@TestPropertySource(properties = "springdoc.default-support-form-data=false")
31+
public class SpringDocApp194Test extends AbstractSpringDocV30Test {
32+
33+
@SpringBootApplication
34+
static class SpringDocTestApp {}
35+
36+
}

‎springdoc-openapi-starter-webmvc-api/src/test/resources/results/3.0.1/app194.json‎

Lines changed: 127 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,127 @@
1+
{
2+
"openapi": "3.0.1",
3+
"info": {
4+
"title": "OpenAPI definition",
5+
"version": "v0"
6+
},
7+
"servers": [
8+
{
9+
"url": "http://localhost",
10+
"description": "Generated server url"
11+
}
12+
],
13+
"paths": {
14+
"/create": {
15+
"post": {
16+
"tags": [
17+
"Article Api"
18+
],
19+
"operationId": "create",
20+
"parameters": [
21+
{
22+
"name": "title",
23+
"in": "query",
24+
"description": "title",
25+
"required": false,
26+
"schema": {
27+
"type": "string",
28+
"description": "title"
29+
}
30+
},
31+
{
32+
"name": "content",
33+
"in": "query",
34+
"description": "content",
35+
"required": false,
36+
"schema": {
37+
"type": "string",
38+
"description": "content"
39+
}
40+
}
41+
],
42+
"requestBody": {
43+
"content": {
44+
"multipart/form-data": {
45+
"schema": {
46+
"required": [
47+
"file"
48+
],
49+
"type": "object",
50+
"properties": {
51+
"file": {
52+
"type": "string",
53+
"format": "binary"
54+
}
55+
}
56+
}
57+
}
58+
}
59+
},
60+
"responses": {
61+
"200": {
62+
"description": "OK",
63+
"content": {
64+
"*/*": {
65+
"schema": {
66+
"$ref": "#/components/schemas/ArticleDto"
67+
}
68+
}
69+
}
70+
}
71+
}
72+
}
73+
},
74+
"/query": {
75+
"get": {
76+
"tags": [
77+
"Article Api"
78+
],
79+
"operationId": "query",
80+
"parameters": [
81+
{
82+
"name": "title",
83+
"in": "query",
84+
"description": "title",
85+
"required": false,
86+
"schema": {
87+
"type": "string",
88+
"description": "title"
89+
}
90+
}
91+
],
92+
"responses": {
93+
"200": {
94+
"description": "OK",
95+
"content": {
96+
"*/*": {
97+
"schema": {
98+
"type": "array",
99+
"items": {
100+
"$ref": "#/components/schemas/ArticleDto"
101+
}
102+
}
103+
}
104+
}
105+
}
106+
}
107+
}
108+
}
109+
},
110+
"components": {
111+
"schemas": {
112+
"ArticleDto": {
113+
"type": "object",
114+
"properties": {
115+
"title": {
116+
"type": "string",
117+
"description": "title"
118+
},
119+
"content": {
120+
"type": "string",
121+
"description": "content"
122+
}
123+
}
124+
}
125+
}
126+
}
127+
}

0 commit comments

Comments
 (0)