Polishing

(cherry picked from commit 96f1a0e)

Author: jhoeller

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

@@ -164,10 +164,10 @@ public class WebMvcConfigurationSupport implements ApplicationContextAware, Serv
 
 	private List<Object> interceptors;
 
-	private ContentNegotiationManager contentNegotiationManager;
-
 	private PathMatchConfigurer pathMatchConfigurer;
 
+	private ContentNegotiationManager contentNegotiationManager;
+
 	private List<HttpMessageConverter<?>> messageConverters;
 
 
@@ -186,6 +186,7 @@ public void setServletContext(ServletContext servletContext) {
 		this.servletContext = servletContext;
 	}
 
+
 	/**
 	 * Return a {@link RequestMappingHandlerMapping} ordered at 0 for mapping
 	 * requests to annotated controllers.
@@ -207,11 +208,13 @@ public RequestMappingHandlerMapping requestMappingHandlerMapping() {
 		if (configurer.isUseTrailingSlashMatch() != null) {
 			handlerMapping.setUseTrailingSlashMatch(configurer.isUseTrailingSlashMatch());
 		}
-		if (configurer.getPathMatcher() != null) {
-			handlerMapping.setPathMatcher(configurer.getPathMatcher());
+		UrlPathHelper pathHelper = configurer.getUrlPathHelper();
+		if (pathHelper != null) {
+			handlerMapping.setUrlPathHelper(pathHelper);
 		}
-		if (configurer.getUrlPathHelper() != null) {
-			handlerMapping.setUrlPathHelper(configurer.getUrlPathHelper());
+		PathMatcher pathMatcher = configurer.getPathMatcher();
+		if (pathMatcher != null) {
+			handlerMapping.setPathMatcher(pathMatcher);
 		}
 
 		return handlerMapping;
@@ -240,48 +243,6 @@ protected final Object[] getInterceptors() {
 	protected void addInterceptors(InterceptorRegistry registry) {
 	}
 
-	/**
-	 * Return a {@link ContentNegotiationManager} instance to use to determine
-	 * requested {@linkplain MediaType media types} in a given request.
-	 */
-	@Bean
-	public ContentNegotiationManager mvcContentNegotiationManager() {
-		if (this.contentNegotiationManager == null) {
-			ContentNegotiationConfigurer configurer = new ContentNegotiationConfigurer(this.servletContext);
-			configurer.mediaTypes(getDefaultMediaTypes());
-			configureContentNegotiation(configurer);
-			try {
-				this.contentNegotiationManager = configurer.getContentNegotiationManager();
-			}
-			catch (Exception ex) {
-				throw new BeanInitializationException("Could not create ContentNegotiationManager", ex);
-			}
-		}
-		return this.contentNegotiationManager;
-	}
-
-	protected Map<String, MediaType> getDefaultMediaTypes() {
-		Map<String, MediaType> map = new HashMap<String, MediaType>();
-		if (romePresent) {
-			map.put("atom", MediaType.APPLICATION_ATOM_XML);
-			map.put("rss", MediaType.valueOf("application/rss+xml"));
-		}
-		if (jaxb2Present) {
-			map.put("xml", MediaType.APPLICATION_XML);
-		}
-		if (jackson2Present || jacksonPresent) {
-			map.put("json", MediaType.APPLICATION_JSON);
-		}
-		return map;
-	}
-
-	/**
-	 * Override this method to configure content negotiation.
-	 * @see DefaultServletHandlerConfigurer
-	 */
-	protected void configureContentNegotiation(ContentNegotiationConfigurer configurer) {
-	}
-
 	/**
 	 * Callback for building the {@link PathMatchConfigurer}.
 	 * Delegates to {@link #configurePathMatch}.
@@ -312,12 +273,8 @@ public void configurePathMatch(PathMatchConfigurer configurer) {
 	 */
 	@Bean
 	public PathMatcher mvcPathMatcher() {
-		if (getPathMatchConfigurer().getPathMatcher() != null) {
-			return getPathMatchConfigurer().getPathMatcher();
-		}
-		else {
-			return new AntPathMatcher();
-		}
+		PathMatcher pathMatcher = getPathMatchConfigurer().getPathMatcher();
+		return (pathMatcher != null ? pathMatcher : new AntPathMatcher());
 	}
 
 	/**
@@ -329,12 +286,50 @@ public PathMatcher mvcPathMatcher() {
 	 */
 	@Bean
 	public UrlPathHelper mvcUrlPathHelper() {
-		if (getPathMatchConfigurer().getUrlPathHelper() != null) {
-			return getPathMatchConfigurer().getUrlPathHelper();
+		UrlPathHelper pathHelper = getPathMatchConfigurer().getUrlPathHelper();
+		return (pathHelper != null ? pathHelper : new UrlPathHelper());
+	}
+
+	/**
+	 * Return a {@link ContentNegotiationManager} instance to use to determine
+	 * requested {@linkplain MediaType media types} in a given request.
+	 */
+	@Bean
+	public ContentNegotiationManager mvcContentNegotiationManager() {
+		if (this.contentNegotiationManager == null) {
+			ContentNegotiationConfigurer configurer = new ContentNegotiationConfigurer(this.servletContext);
+			configurer.mediaTypes(getDefaultMediaTypes());
+			configureContentNegotiation(configurer);
+			try {
+				this.contentNegotiationManager = configurer.getContentNegotiationManager();
+			}
+			catch (Exception ex) {
+				throw new BeanInitializationException("Could not create ContentNegotiationManager", ex);
+			}
 		}
-		else {
-			return new UrlPathHelper();
+		return this.contentNegotiationManager;
+	}
+
+	protected Map<String, MediaType> getDefaultMediaTypes() {
+		Map<String, MediaType> map = new HashMap<String, MediaType>(4);
+		if (romePresent) {
+			map.put("atom", MediaType.APPLICATION_ATOM_XML);
+			map.put("rss", MediaType.valueOf("application/rss+xml"));
+		}
+		if (jaxb2Present) {
+			map.put("xml", MediaType.APPLICATION_XML);
 		}
+		if (jackson2Present || jacksonPresent) {
+			map.put("json", MediaType.APPLICATION_JSON);
+		}
+		return map;
+	}
+
+	/**
+	 * Override this method to configure content negotiation.
+	 * @see DefaultServletHandlerConfigurer
+	 */
+	protected void configureContentNegotiation(ContentNegotiationConfigurer configurer) {
 	}
 
 	/**
@@ -450,7 +445,6 @@ public RequestMappingHandlerAdapter requestMappingHandlerAdapter() {
 
 		AsyncSupportConfigurer configurer = new AsyncSupportConfigurer();
 		configureAsyncSupport(configurer);
-
 		if (configurer.getTaskExecutor() != null) {
 			adapter.setTaskExecutor(configurer.getTaskExecutor());
 		}
@@ -475,6 +469,20 @@ protected ConfigurableWebBindingInitializer getConfigurableWebBindingInitializer
 		return initializer;
 	}
 
+	/**
+	 * Override this method to provide a custom {@link MessageCodesResolver}.
+	 */
+	protected MessageCodesResolver getMessageCodesResolver() {
+		return null;
+	}
+
+	/**
+	 * Override this method to configure asynchronous request processing options.
+	 * @see AsyncSupportConfigurer
+	 */
+	protected void configureAsyncSupport(AsyncSupportConfigurer configurer) {
+	}
+
 	/**
 	 * Return a {@link FormattingConversionService} for use with annotated
 	 * controller methods and the {@code spring:eval} JSP tag.
@@ -487,6 +495,12 @@ public FormattingConversionService mvcConversionService() {
 		return conversionService;
 	}
 
+	/**
+	 * Override this method to add custom {@link Converter}s and {@link Formatter}s.
+	 */
+	protected void addFormatters(FormatterRegistry registry) {
+	}
+
 	/**
 	 * Return a global {@link Validator} instance for example for validating
 	 * {@code @ModelAttribute} and {@code @RequestBody} method arguments.
@@ -511,7 +525,7 @@ public Validator mvcValidator() {
 				catch (LinkageError ex) {
 					throw new BeanInitializationException("Could not load default validator class", ex);
 				}
-				validator = (Validator) BeanUtils.instantiate(clazz);
+				validator = (Validator) BeanUtils.instantiateClass(clazz);
 			}
 			else {
 				validator = new NoOpValidator();
@@ -527,13 +541,6 @@ protected Validator getValidator() {
 		return null;
 	}
 
-	/**
-	 * Override this method to provide a custom {@link MessageCodesResolver}.
-	 */
-	protected MessageCodesResolver getMessageCodesResolver() {
-		return null;
-	}
-
 	/**
 	 * Add custom {@link HandlerMethodArgumentResolver}s to use in addition to
 	 * the ones registered by default.
@@ -625,19 +632,6 @@ else if (jacksonPresent) {
 		}
 	}
 
-	/**
-	 * Override this method to add custom {@link Converter}s and {@link Formatter}s.
-	 */
-	protected void addFormatters(FormatterRegistry registry) {
-	}
-
-	/**
-	 * Override this method to configure asynchronous request processing options.
-	 * @see AsyncSupportConfigurer
-	 */
-	public void configureAsyncSupport(AsyncSupportConfigurer configurer) {
-	}
-
 	/**
 	 * Returns a {@link HttpRequestHandlerAdapter} for processing requests
 	 * with {@link HttpRequestHandler}s.
@@ -670,11 +664,9 @@ public SimpleControllerHandlerAdapter simpleControllerHandlerAdapter() {
 	public HandlerExceptionResolver handlerExceptionResolver() {
 		List<HandlerExceptionResolver> exceptionResolvers = new ArrayList<HandlerExceptionResolver>();
 		configureHandlerExceptionResolvers(exceptionResolvers);
-
 		if (exceptionResolvers.isEmpty()) {
 			addDefaultHandlerExceptionResolvers(exceptionResolvers);
 		}
-
 		HandlerExceptionResolverComposite composite = new HandlerExceptionResolverComposite();
 		composite.setOrder(0);
 		composite.setExceptionResolvers(exceptionResolvers);

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

@@ -61,14 +61,6 @@ public interface WebMvcConfigurer {
 	 */
 	void configureMessageConverters(List<HttpMessageConverter<?>> converters);
 
-	/**
-	 * Provide a custom {@link Validator} instead of the one created by default.
-	 * The default implementation, assuming JSR-303 is on the classpath, is:
-	 * {@link org.springframework.validation.beanvalidation.LocalValidatorFactoryBean}.
-	 * Leave the return value as {@code null} to keep the default.
-	 */
-	Validator getValidator();
-
 	/**
 	 * Configure content negotiation options.
 	 */
@@ -79,7 +71,6 @@ public interface WebMvcConfigurer {
 	 */
 	void configureAsyncSupport(AsyncSupportConfigurer configurer);
 
-
 	/**
 	 * Helps with configuring HandlerMappings path matching options such as trailing slash match,
 	 * suffix registration, path matcher and path helper.
@@ -126,13 +117,6 @@ public interface WebMvcConfigurer {
 	 */
 	void addInterceptors(InterceptorRegistry registry);
 
-	/**
-	 * Provide a custom {@link MessageCodesResolver} for building message codes
-	 * from data binding and validation error codes. Leave the return value as
-	 * {@code null} to keep the default.
-	 */
-	MessageCodesResolver getMessageCodesResolver();
-
 	/**
 	 * Add view controllers to create a direct mapping between a URL path and
 	 * view name without the need for a controller in between.
@@ -154,4 +138,19 @@ public interface WebMvcConfigurer {
 	 */
 	void configureDefaultServletHandling(DefaultServletHandlerConfigurer configurer);
 
+	/**
+	 * Provide a custom {@link Validator} instead of the one created by default.
+	 * The default implementation, assuming JSR-303 is on the classpath, is:
+	 * {@link org.springframework.validation.beanvalidation.LocalValidatorFactoryBean}.
+	 * Leave the return value as {@code null} to keep the default.
+	 */
+	Validator getValidator();
+
+	/**
+	 * Provide a custom {@link MessageCodesResolver} for building message codes
+	 * from data binding and validation error codes. Leave the return value as
+	 * {@code null} to keep the default.
+	 */
+	MessageCodesResolver getMessageCodesResolver();
+
 }

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

@@ -28,7 +28,7 @@
 
 /**
  * An implementation of {@link WebMvcConfigurer} with empty methods allowing
- * sub-classes to override only the methods they're interested in.
+ * subclasses to override only the methods they're interested in.
  *
  * @author Rossen Stoyanchev
  * @since 3.1
@@ -49,14 +49,6 @@ public void addFormatters(FormatterRegistry registry) {
 	public void configureMessageConverters(List<HttpMessageConverter<?>> converters) {
 	}
 
-	/**
-	 * {@inheritDoc}
-	 * <p>This implementation returns {@code null}
-	 */
-	public Validator getValidator() {
-		return null;
-	}
-
 	/**
 	 * {@inheritDoc}
 	 * <p>This implementation is empty.
@@ -103,36 +95,44 @@ public void configureHandlerExceptionResolvers(List<HandlerExceptionResolver> ex
 	 * {@inheritDoc}
 	 * <p>This implementation is empty.
 	 */
-	public MessageCodesResolver getMessageCodesResolver() {
-		return null;
+	public void addInterceptors(InterceptorRegistry registry) {
 	}
 
 	/**
 	 * {@inheritDoc}
 	 * <p>This implementation is empty.
 	 */
-	public void addInterceptors(InterceptorRegistry registry) {
+	public void addViewControllers(ViewControllerRegistry registry) {
 	}
 
 	/**
 	 * {@inheritDoc}
 	 * <p>This implementation is empty.
 	 */
-	public void addViewControllers(ViewControllerRegistry registry) {
+	public void addResourceHandlers(ResourceHandlerRegistry registry) {
 	}
 
 	/**
 	 * {@inheritDoc}
 	 * <p>This implementation is empty.
 	 */
-	public void addResourceHandlers(ResourceHandlerRegistry registry) {
+	public void configureDefaultServletHandling(DefaultServletHandlerConfigurer configurer) {
 	}
 
 	/**
 	 * {@inheritDoc}
-	 * <p>This implementation is empty.
+	 * <p>This implementation returns {@code null}.
 	 */
-	public void configureDefaultServletHandling(DefaultServletHandlerConfigurer configurer) {
+	public Validator getValidator() {
+		return null;
+	}
+
+	/**
+	 * {@inheritDoc}
+	 * <p>This implementation returns {@code null}.
+	 */
+	public MessageCodesResolver getMessageCodesResolver() {
+		return null;
 	}
 
 }