Skip to content

Commit ac975df

Browse files
committed
Fix javadoc ambiguity
Issue: SPR-13642
1 parent f0464e8 commit ac975df

File tree

2 files changed

+31
-19
lines changed

2 files changed

+31
-19
lines changed

spring-web/src/main/java/org/springframework/web/accept/ContentNegotiationManagerFactoryBean.java

Lines changed: 16 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -40,29 +40,34 @@
4040
*
4141
* <table>
4242
* <tr>
43-
* <td>{@link #setFavorPathExtension favorPathExtension}</td>
44-
* <td>{@link PathExtensionContentNegotiationStrategy}</td>
45-
* <td>Yes</td>
43+
* <th>Property Setter</th>
44+
* <th>Underlying Strategy</th>
45+
* <th>Default Setting</th>
46+
* </tr>
47+
* <tr>
48+
* <td>{@link #setFavorPathExtension}</td>
49+
* <td>{@link PathExtensionContentNegotiationStrategy Path Extension strategy}</td>
50+
* <td>On</td>
4651
* </tr>
4752
* <tr>
4853
* <td>{@link #setFavorParameter favorParameter}</td>
49-
* <td>{@link ParameterContentNegotiationStrategy}</td>
50-
* <td>-</td>
54+
* <td>{@link ParameterContentNegotiationStrategy Parameter strategy}</td>
55+
* <td>Off</td>
5156
* </tr>
5257
* <tr>
5358
* <td>{@link #setIgnoreAcceptHeader ignoreAcceptHeader}</td>
54-
* <td>{@link HeaderContentNegotiationStrategy}</td>
55-
* <td>Yes</td>
59+
* <td>{@link HeaderContentNegotiationStrategy Header strategy}</td>
60+
* <td>On</td>
5661
* </tr>
5762
* <tr>
5863
* <td>{@link #setDefaultContentType defaultContentType}</td>
59-
* <td>{@link FixedContentNegotiationStrategy}</td>
60-
* <td>-</td>
64+
* <td>{@link FixedContentNegotiationStrategy Fixed content strategy}</td>
65+
* <td>Not set</td>
6166
* </tr>
6267
* <tr>
6368
* <td>{@link #setDefaultContentTypeStrategy defaultContentTypeStrategy}</td>
6469
* <td>{@link ContentNegotiationStrategy}</td>
65-
* <td>-</td>
70+
* <td>Not set</td>
6671
* </tr>
6772
* </table>
6873
*
@@ -226,6 +231,7 @@ public void setDefaultContentType(MediaType contentType) {
226231
/**
227232
* Set a custom {@link ContentNegotiationStrategy} to use to determine
228233
* the content type to use when no content type is requested.
234+
* <p>By default this is not set.
229235
* @see #setDefaultContentType
230236
* @since 4.1.2
231237
*/

spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/ContentNegotiationConfigurer.java

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -36,29 +36,34 @@
3636
*
3737
* <table>
3838
* <tr>
39+
* <th>Configurer Property</th>
40+
* <th>Underlying Strategy</th>
41+
* <th>Default Setting</th>
42+
* </tr>
43+
* <tr>
3944
* <td>{@link #favorPathExtension}</td>
40-
* <td>{@link PathExtensionContentNegotiationStrategy}</td>
41-
* <td>Yes</td>
45+
* <td>{@link PathExtensionContentNegotiationStrategy Path Extension strategy}</td>
46+
* <td>On</td>
4247
* </tr>
4348
* <tr>
4449
* <td>{@link #favorParameter}</td>
45-
* <td>{@link ParameterContentNegotiationStrategy}</td>
46-
* <td>-</td>
50+
* <td>{@link ParameterContentNegotiationStrategy Parameter strategy}</td>
51+
* <td>Off</td>
4752
* </tr>
4853
* <tr>
4954
* <td>{@link #ignoreAcceptHeader}</td>
50-
* <td>{@link HeaderContentNegotiationStrategy}</td>
51-
* <td>Yes</td>
55+
* <td>{@link HeaderContentNegotiationStrategy Header strategy}</td>
56+
* <td>On</td>
5257
* </tr>
5358
* <tr>
5459
* <td>{@link #defaultContentType}</td>
55-
* <td>{@link FixedContentNegotiationStrategy}</td>
56-
* <td>-</td>
60+
* <td>{@link FixedContentNegotiationStrategy Fixed content strategy}</td>
61+
* <td>Not set</td>
5762
* </tr>
5863
* <tr>
5964
* <td>{@link #defaultContentTypeStrategy}</td>
6065
* <td>{@link ContentNegotiationStrategy}</td>
61-
* <td>-</td>
66+
* <td>Not set</td>
6267
* </tr>
6368
* </table>
6469
*
@@ -215,6 +220,7 @@ public ContentNegotiationConfigurer defaultContentType(MediaType defaultContentT
215220
/**
216221
* Set a custom {@link ContentNegotiationStrategy} to use to determine
217222
* the content type to use when no content type is requested.
223+
* <p>By default this is not set.
218224
* @see #defaultContentType
219225
* @since 4.1.2
220226
*/

0 commit comments

Comments
 (0)