Skip to content

Commit e5b0386

Browse files
committed
Polish "Add property to configure Spring MVC default content types"
See gh-44040
1 parent d27fd31 commit e5b0386

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/servlet/WebMvcProperties.java

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -309,18 +309,18 @@ public static class Contentnegotiation {
309309
private boolean favorParameter = false;
310310

311311
/**
312-
* Map file extensions to media types for content negotiation. For instance, yml
313-
* to text/yaml.
312+
* Query parameter name to use when "favor-parameter" is enabled.
314313
*/
315-
private Map<String, MediaType> mediaTypes = new LinkedHashMap<>();
314+
private String parameterName;
316315

317316
/**
318-
* Query parameter name to use when "favor-parameter" is enabled.
317+
* Map file extensions to media types for content negotiation. For instance, yml
318+
* to text/yaml.
319319
*/
320-
private String parameterName;
320+
private Map<String, MediaType> mediaTypes = new LinkedHashMap<>();
321321

322322
/**
323-
* The default content types to be used when no specific content type is
323+
* List of default content types to be used when no specific content type is
324324
* requested.
325325
*/
326326
private List<MediaType> defaultContentTypes = new ArrayList<>();
@@ -333,14 +333,6 @@ public void setFavorParameter(boolean favorParameter) {
333333
this.favorParameter = favorParameter;
334334
}
335335

336-
public Map<String, MediaType> getMediaTypes() {
337-
return this.mediaTypes;
338-
}
339-
340-
public void setMediaTypes(Map<String, MediaType> mediaTypes) {
341-
this.mediaTypes = mediaTypes;
342-
}
343-
344336
public String getParameterName() {
345337
return this.parameterName;
346338
}
@@ -349,6 +341,14 @@ public void setParameterName(String parameterName) {
349341
this.parameterName = parameterName;
350342
}
351343

344+
public Map<String, MediaType> getMediaTypes() {
345+
return this.mediaTypes;
346+
}
347+
348+
public void setMediaTypes(Map<String, MediaType> mediaTypes) {
349+
this.mediaTypes = mediaTypes;
350+
}
351+
352352
public List<MediaType> getDefaultContentTypes() {
353353
return this.defaultContentTypes;
354354
}

spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/WebMvcAutoConfigurationTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -580,7 +580,7 @@ void customMediaTypes() {
580580
}
581581

582582
@Test
583-
void defaultContentTypes() {
583+
void customDefaultContentTypes() {
584584
this.contextRunner
585585
.withPropertyValues("spring.mvc.contentnegotiation.default-content-types:application/json,application/xml")
586586
.run((context) -> {

0 commit comments

Comments
 (0)