diff --git a/spring-test/src/test/java/org/springframework/test/web/client/samples/MockMvcClientHttpRequestFactoryTests.java b/spring-test/src/test/java/org/springframework/test/web/client/samples/MockMvcClientHttpRequestFactoryTests.java index 2d49a4904c8a47d07c7b8697c7c25bec20696f38..e787a336174168f8ffb0b1aaf2d7a0035e440027 100644 --- a/spring-test/src/test/java/org/springframework/test/web/client/samples/MockMvcClientHttpRequestFactoryTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/client/samples/MockMvcClientHttpRequestFactoryTests.java @@ -38,7 +38,7 @@ import org.springframework.web.bind.annotation.ResponseBody; import org.springframework.web.client.RestTemplate; import org.springframework.web.context.WebApplicationContext; import org.springframework.web.servlet.config.annotation.EnableWebMvc; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; import static org.junit.Assert.assertEquals; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; @@ -87,7 +87,7 @@ public class MockMvcClientHttpRequestFactoryTests { @EnableWebMvc @Configuration @ComponentScan(basePackageClasses=MockMvcClientHttpRequestFactoryTests.class) - static class MyWebConfig extends WebMvcConfigurerAdapter { + static class MyWebConfig implements WebMvcConfigurer { } @Controller diff --git a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/context/AsyncControllerJavaConfigTests.java b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/context/AsyncControllerJavaConfigTests.java index 2226af293727f988a0f49d1d8a47ea2922cc0b64..f2522bca26ab654704afda754dece0a9465e3223 100644 --- a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/context/AsyncControllerJavaConfigTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/context/AsyncControllerJavaConfigTests.java @@ -42,12 +42,14 @@ import org.springframework.web.context.WebApplicationContext; import org.springframework.web.context.request.async.CallableProcessingInterceptor; import org.springframework.web.servlet.config.annotation.AsyncSupportConfigurer; import org.springframework.web.servlet.config.annotation.EnableWebMvc; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; -import static org.mockito.ArgumentMatchers.*; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; +import static org.mockito.ArgumentMatchers.any; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.asyncDispatch; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.content; import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.request; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; /** * Tests with Java configuration. @@ -100,7 +102,7 @@ public class AsyncControllerJavaConfigTests { @Configuration @EnableWebMvc - static class WebConfig extends WebMvcConfigurerAdapter { + static class WebConfig implements WebMvcConfigurer { @Override public void configureAsyncSupport(AsyncSupportConfigurer configurer) { diff --git a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/context/JavaConfigTests.java b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/context/JavaConfigTests.java index 70e49dc3332effe5701ac63cef6760e3709317ec..09ef34d1dfa31f464a1b8bef0de7800c21a4e65d 100644 --- a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/context/JavaConfigTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/context/JavaConfigTests.java @@ -21,7 +21,6 @@ import javax.servlet.ServletContext; import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; - import org.mockito.Mockito; import org.springframework.beans.factory.annotation.Autowired; @@ -44,14 +43,18 @@ import org.springframework.web.servlet.config.annotation.EnableWebMvc; import org.springframework.web.servlet.config.annotation.ResourceHandlerRegistry; import org.springframework.web.servlet.config.annotation.ViewControllerRegistry; import org.springframework.web.servlet.config.annotation.ViewResolverRegistry; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; import org.springframework.web.servlet.view.tiles3.TilesConfigurer; -import static org.junit.Assert.*; -import static org.mockito.BDDMockito.*; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; -import static org.springframework.test.web.servlet.result.MockMvcResultHandlers.*; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertSame; +import static org.junit.Assert.assertTrue; +import static org.mockito.BDDMockito.given; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; +import static org.springframework.test.web.servlet.result.MockMvcResultHandlers.print; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.content; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.forwardedUrl; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; /** * Tests with Java configuration. @@ -142,7 +145,7 @@ public class JavaConfigTests { @Configuration @EnableWebMvc - static class WebConfig extends WebMvcConfigurerAdapter { + static class WebConfig implements WebMvcConfigurer { @Autowired private RootConfig rootConfig; diff --git a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/CustomRequestAttributesRequestContextHolderTests.java b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/CustomRequestAttributesRequestContextHolderTests.java index a50125b17b04368a29eb25021afae01746de9f86..5c5fdaaf1430ba07f0c06b3f2fbfed012363bca4 100644 --- a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/CustomRequestAttributesRequestContextHolderTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/CustomRequestAttributesRequestContextHolderTests.java @@ -39,13 +39,15 @@ import org.springframework.web.context.request.ServletRequestAttributes; import org.springframework.web.context.request.ServletWebRequest; import org.springframework.web.context.support.GenericWebApplicationContext; import org.springframework.web.servlet.config.annotation.EnableWebMvc; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; -import static org.hamcrest.CoreMatchers.*; -import static org.junit.Assert.*; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; -import static org.springframework.test.web.servlet.setup.MockMvcBuilders.*; +import static org.hamcrest.CoreMatchers.instanceOf; +import static org.hamcrest.CoreMatchers.is; +import static org.hamcrest.CoreMatchers.nullValue; +import static org.junit.Assert.assertThat; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; +import static org.springframework.test.web.servlet.setup.MockMvcBuilders.webAppContextSetup; /** * Integration tests for SPR-13211 which verify that a custom mock request @@ -107,7 +109,7 @@ public class CustomRequestAttributesRequestContextHolderTests { @Configuration @EnableWebMvc - static class WebConfig extends WebMvcConfigurerAdapter { + static class WebConfig implements WebMvcConfigurer { @Bean public SingletonController singletonController() { diff --git a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/EncodedUriTests.java b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/EncodedUriTests.java index de8e576cacd3642e7f302e9562e16a36eacf0f9f..f4aab2970dbbbfbd37a44ad4f63fdc4a293e8092 100644 --- a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/EncodedUriTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/EncodedUriTests.java @@ -43,14 +43,15 @@ import org.springframework.web.bind.annotation.RequestMethod; import org.springframework.web.context.WebApplicationContext; import org.springframework.web.servlet.config.annotation.EnableWebMvc; import org.springframework.web.servlet.config.annotation.ViewResolverRegistry; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; import org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandlerMapping; import org.springframework.web.util.UriComponentsBuilder; -import static org.hamcrest.core.Is.*; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; -import static org.springframework.test.web.servlet.setup.MockMvcBuilders.*; +import static org.hamcrest.core.Is.is; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.model; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; +import static org.springframework.test.web.servlet.setup.MockMvcBuilders.webAppContextSetup; /** * Tests for SPR-11441 (MockMvc accepts an already encoded URI). @@ -83,7 +84,7 @@ public class EncodedUriTests { @Configuration @EnableWebMvc - static class WebConfig extends WebMvcConfigurerAdapter { + static class WebConfig implements WebMvcConfigurer { @Bean public MyController myController() { diff --git a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/HttpOptionsTests.java b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/HttpOptionsTests.java index 1aae207e7e2614423f6161828c2dce5b33111f02..3e6bf011178fcae20666b0a3587cb00c827ac31b 100644 --- a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/HttpOptionsTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/HttpOptionsTests.java @@ -35,12 +35,12 @@ import org.springframework.web.bind.annotation.RequestMethod; import org.springframework.web.bind.annotation.ResponseBody; import org.springframework.web.context.WebApplicationContext; import org.springframework.web.servlet.config.annotation.EnableWebMvc; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; -import static org.junit.Assert.*; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; -import static org.springframework.test.web.servlet.setup.MockMvcBuilders.*; +import static org.junit.Assert.assertEquals; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.options; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; +import static org.springframework.test.web.servlet.setup.MockMvcBuilders.webAppContextSetup; /** * Tests for SPR-10093 (support for OPTIONS requests). @@ -75,7 +75,7 @@ public class HttpOptionsTests { @Configuration @EnableWebMvc - static class WebConfig extends WebMvcConfigurerAdapter { + static class WebConfig implements WebMvcConfigurer { @Bean public MyController myController() { diff --git a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/MockMvcBuilderMethodChainTests.java b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/MockMvcBuilderMethodChainTests.java index 8900c7c6db02a6eb71259ed82f0aaaef39f11993..93c28d55ce6fb28dbd3064984a4595714e6847e7 100644 --- a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/MockMvcBuilderMethodChainTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/MockMvcBuilderMethodChainTests.java @@ -27,9 +27,9 @@ import org.springframework.test.web.servlet.setup.MockMvcBuilders; import org.springframework.web.context.WebApplicationContext; import org.springframework.web.filter.CharacterEncodingFilter; import org.springframework.web.servlet.config.annotation.EnableWebMvc; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; /** * Test for SPR-10277 (multiple method chaining when building MockMvc). @@ -55,6 +55,6 @@ public class MockMvcBuilderMethodChainTests { @Configuration @EnableWebMvc - static class WebConfig extends WebMvcConfigurerAdapter { + static class WebConfig implements WebMvcConfigurer { } } diff --git a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/RequestContextHolderTests.java b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/RequestContextHolderTests.java index acfb8c6a41048e6a7e9c023507f15f42df64d36c..0ec33e0419c1948187c379b653fc89357b71c2fe 100644 --- a/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/RequestContextHolderTests.java +++ b/spring-test/src/test/java/org/springframework/test/web/servlet/samples/spr/RequestContextHolderTests.java @@ -17,7 +17,6 @@ package org.springframework.test.web.servlet.samples.spr; import java.io.IOException; - import javax.servlet.FilterChain; import javax.servlet.ServletException; import javax.servlet.ServletRequest; @@ -48,13 +47,16 @@ import org.springframework.web.context.request.RequestContextHolder; import org.springframework.web.context.request.ServletRequestAttributes; import org.springframework.web.filter.GenericFilterBean; import org.springframework.web.servlet.config.annotation.EnableWebMvc; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; -import static org.hamcrest.CoreMatchers.*; -import static org.junit.Assert.*; -import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*; -import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*; -import static org.springframework.test.web.servlet.setup.MockMvcBuilders.*; +import static org.hamcrest.CoreMatchers.instanceOf; +import static org.hamcrest.CoreMatchers.is; +import static org.hamcrest.CoreMatchers.nullValue; +import static org.junit.Assert.assertThat; +import static org.junit.Assert.assertTrue; +import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get; +import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status; +import static org.springframework.test.web.servlet.setup.MockMvcBuilders.webAppContextSetup; /** * Integration tests for the following use cases. @@ -145,7 +147,7 @@ public class RequestContextHolderTests { @Configuration @EnableWebMvc - static class WebConfig extends WebMvcConfigurerAdapter { + static class WebConfig implements WebMvcConfigurer { @Bean public SingletonController singletonController() { diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/EnableWebMvc.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/EnableWebMvc.java index 3960aa068383fbbdbab912c966efb31228bcd01a..16f2630a4311c81264647fea9f342a12b476175b 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/EnableWebMvc.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/EnableWebMvc.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2016 the original author or authors. + * Copyright 2002-2017 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -38,14 +38,13 @@ import org.springframework.context.annotation.Import; * * *

To customize the imported configuration, implement the interface - * {@link WebMvcConfigurer} or more likely extend the empty method base class - * {@link WebMvcConfigurerAdapter} and override individual methods, e.g.: + * {@link WebMvcConfigurer} and override individual methods, e.g.: * *

  * @Configuration
  * @EnableWebMvc
  * @ComponentScan(basePackageClasses = MyConfiguration.class)
- * public class MyConfiguration extends WebMvcConfigurerAdapter {
+ * public class MyConfiguration implements WebMvcConfigurer {
  *
  * 	   @Override
  * 	   public void addFormatters(FormatterRegistry formatterRegistry) {
@@ -93,7 +92,6 @@ import org.springframework.context.annotation.Import;
  * @author Rossen Stoyanchev
  * @since 3.1
  * @see org.springframework.web.servlet.config.annotation.WebMvcConfigurer
- * @see org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter
  * @see org.springframework.web.servlet.config.annotation.WebMvcConfigurationSupport
  * @see org.springframework.web.servlet.config.annotation.DelegatingWebMvcConfiguration
  */
diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurationSupport.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurationSupport.java
index 49abd8574a96931a1d9fc075ef1be65aba6844dd..73660ce5b87a57206e37c2af4401f8882c819ce6 100644
--- a/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurationSupport.java
+++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurationSupport.java
@@ -168,7 +168,6 @@ import org.springframework.web.util.UrlPathHelper;
  * @since 3.1
  * @see EnableWebMvc
  * @see WebMvcConfigurer
- * @see WebMvcConfigurerAdapter
  */
 public class WebMvcConfigurationSupport implements ApplicationContextAware, ServletContextAware {
 
diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurer.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurer.java
index ad3c33c967376d834b81addce2f882f3fc193f35..e576c7bb591c10c40c21230975505778932917c0 100644
--- a/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurer.java
+++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurer.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2016 the original author or authors.
+ * Copyright 2002-2017 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -36,8 +36,7 @@ import org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandl
  *
  * 

{@code @EnableWebMvc}-annotated configuration classes may implement * this interface to be called back and given a chance to customize the - * default configuration. Consider extending {@link WebMvcConfigurerAdapter}, - * which provides a stub implementation of all interface methods. + * default configuration. * * @author Rossen Stoyanchev * @author Keith Donald @@ -57,17 +56,20 @@ public interface WebMvcConfigurer { * * @since 4.0.3 */ - void configurePathMatch(PathMatchConfigurer configurer); + default void configurePathMatch(PathMatchConfigurer configurer) { + } /** * Configure content negotiation options. */ - void configureContentNegotiation(ContentNegotiationConfigurer configurer); + default void configureContentNegotiation(ContentNegotiationConfigurer configurer) { + } /** * Configure asynchronous request handling options. */ - void configureAsyncSupport(AsyncSupportConfigurer configurer); + default void configureAsyncSupport(AsyncSupportConfigurer configurer) { + } /** * Configure a handler to delegate unhandled requests by forwarding to the @@ -75,13 +77,15 @@ public interface WebMvcConfigurer { * the {@link DispatcherServlet} is mapped to "/" thus overriding the * Servlet container's default handling of static resources. */ - void configureDefaultServletHandling(DefaultServletHandlerConfigurer configurer); + default void configureDefaultServletHandling(DefaultServletHandlerConfigurer configurer) { + } /** * Add {@link Converter}s and {@link Formatter}s in addition to the ones * registered by default. */ - void addFormatters(FormatterRegistry registry); + default void addFormatters(FormatterRegistry registry) { + } /** * Add Spring MVC lifecycle interceptors for pre- and post-processing of @@ -95,20 +99,23 @@ public interface WebMvcConfigurer { * {@link org.springframework.web.servlet.config.annotation.WebMvcConfigurationSupport * WebMvcConfigurationSupport} and then override {@code resourceHandlerMapping}. */ - void addInterceptors(InterceptorRegistry registry); + default void addInterceptors(InterceptorRegistry registry) { + } /** * Add handlers to serve static resources such as images, js, and, css * files from specific locations under web application root, the classpath, * and others. */ - void addResourceHandlers(ResourceHandlerRegistry registry); + default void addResourceHandlers(ResourceHandlerRegistry registry) { + } /** * Configure cross origin requests processing. * @since 4.2 */ - void addCorsMappings(CorsRegistry registry); + default void addCorsMappings(CorsRegistry registry) { + } /** * Configure simple automated controllers pre-configured with the response @@ -117,7 +124,8 @@ public interface WebMvcConfigurer { * home page, perform simple site URL redirects, return a 404 status with * HTML content, a 204 with no content, and more. */ - void addViewControllers(ViewControllerRegistry registry); + default void addViewControllers(ViewControllerRegistry registry) { + } /** * Configure view resolvers to translate String-based view names returned from @@ -125,25 +133,28 @@ public interface WebMvcConfigurer { * implementations to perform rendering with. * @since 4.1 */ - void configureViewResolvers(ViewResolverRegistry registry); + default void configureViewResolvers(ViewResolverRegistry registry) { + } /** * Add resolvers to support custom controller method argument types. *

This does not override the built-in support for resolving handler * method arguments. To customize the built-in support for argument * resolution, configure {@link RequestMappingHandlerAdapter} directly. - * @param argumentResolvers initially an empty list + * @param resolvers initially an empty list */ - void addArgumentResolvers(List argumentResolvers); + default void addArgumentResolvers(List resolvers) { + } /** * Add handlers to support custom controller method return value types. *

Using this option does not override the built-in support for handling * return values. To customize the built-in support for handling return * values, configure RequestMappingHandlerAdapter directly. - * @param returnValueHandlers initially an empty list + * @param handlers initially an empty list */ - void addReturnValueHandlers(List returnValueHandlers); + default void addReturnValueHandlers(List handlers) { + } /** * Configure the {@link HttpMessageConverter}s to use for reading or writing @@ -155,7 +166,8 @@ public interface WebMvcConfigurer { * {@link #extendMessageConverters(java.util.List)} instead. * @param converters initially an empty list of converters */ - void configureMessageConverters(List> converters); + default void configureMessageConverters(List> converters) { + } /** * A hook for extending or modifying the list of converters after it has been @@ -164,7 +176,8 @@ public interface WebMvcConfigurer { * @param converters the list of configured converters to extend. * @since 4.1.3 */ - void extendMessageConverters(List> converters); + default void extendMessageConverters(List> converters) { + } /** * Configure exception resolvers. @@ -177,21 +190,23 @@ public interface WebMvcConfigurer { *

Alternatively you can use * {@link #extendHandlerExceptionResolvers(List)} which allows you to extend * or modify the list of exception resolvers configured by default. - * @param exceptionResolvers initially an empty list + * @param resolvers initially an empty list * @see #extendHandlerExceptionResolvers(List) * @see WebMvcConfigurationSupport#addDefaultHandlerExceptionResolvers(List) */ - void configureHandlerExceptionResolvers(List exceptionResolvers); + default void configureHandlerExceptionResolvers(List resolvers) { + } /** * Extending or modify the list of exception resolvers configured by default. * This can be useful for inserting a custom exception resolver without * interfering with default ones. - * @param exceptionResolvers the list of configured resolvers to extend + * @param resolvers the list of configured resolvers to extend * @since 4.3 * @see WebMvcConfigurationSupport#addDefaultHandlerExceptionResolvers(List) */ - void extendHandlerExceptionResolvers(List exceptionResolvers); + default void extendHandlerExceptionResolvers(List resolvers) { + } /** * Provide a custom {@link Validator} instead of the one created by default. @@ -199,13 +214,17 @@ public interface WebMvcConfigurer { * {@link org.springframework.validation.beanvalidation.OptionalValidatorFactoryBean}. * Leave the return value as {@code null} to keep the default. */ - Validator getValidator(); + default Validator getValidator() { + return null; + } /** * 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(); + default MessageCodesResolver getMessageCodesResolver() { + return null; + } } diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurerAdapter.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurerAdapter.java index 3d1c28845e30ed3341f82d0de9195946feea6423..0270046e605ad0ec114ea91a43572b72fb8c6603 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurerAdapter.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurerAdapter.java @@ -32,7 +32,11 @@ import org.springframework.web.servlet.HandlerExceptionResolver; * * @author Rossen Stoyanchev * @since 3.1 + * @deprecated as of 5.0 {@link WebMvcConfigurer} has default methods (made + * possible by a Java 8 baseline) and can be implemented directly without the + * need for this adapter */ +@Deprecated public abstract class WebMvcConfigurerAdapter implements WebMvcConfigurer { /** diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/resource/AppCacheManifestTransformer.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/resource/AppCacheManifestTransformer.java index 6b26266542daa28f3801d7380386951f445932de..bd075950dae6b75eb7faf9c87fe46044898d9b7d 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/resource/AppCacheManifestTransformer.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/resource/AppCacheManifestTransformer.java @@ -55,7 +55,7 @@ import org.springframework.util.StringUtils; *

In order to serve manifest files with the proper {@code "text/manifest"} content type, * it is required to configure it with * {@code contentNegotiationConfigurer.mediaType("appcache", MediaType.valueOf("text/manifest")} - * in a {@code WebMvcConfigurerAdapter}. + * in a {@code WebMvcConfigurer}. * * @author Brian Clozel * @since 4.1 diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/DelegatingWebMvcConfigurationTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/DelegatingWebMvcConfigurationTests.java index 4263263a9b274a09275e5e7538962a54d0c63190..c2f67a323b40025bec3e06b59ce45ac565c6b672 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/DelegatingWebMvcConfigurationTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/DelegatingWebMvcConfigurationTests.java @@ -46,8 +46,13 @@ import org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandl import org.springframework.web.servlet.mvc.support.DefaultHandlerExceptionResolver; import org.springframework.web.util.UrlPathHelper; -import static org.junit.Assert.*; -import static org.mockito.BDDMockito.*; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertSame; +import static org.junit.Assert.assertTrue; +import static org.mockito.BDDMockito.given; +import static org.mockito.BDDMockito.mock; +import static org.mockito.BDDMockito.verify; /** * A test fixture for {@link DelegatingWebMvcConfiguration} tests. @@ -118,7 +123,7 @@ public class DelegatingWebMvcConfigurationTests { final HttpMessageConverter customConverter = mock(HttpMessageConverter.class); final StringHttpMessageConverter stringConverter = new StringHttpMessageConverter(); List configurers = new ArrayList<>(); - configurers.add(new WebMvcConfigurerAdapter() { + configurers.add(new WebMvcConfigurer() { @Override public void configureMessageConverters(List> converters) { converters.add(stringConverter); @@ -176,7 +181,7 @@ public class DelegatingWebMvcConfigurationTests { @Test public void configureExceptionResolvers() throws Exception { List configurers = new ArrayList<>(); - configurers.add(new WebMvcConfigurerAdapter() { + configurers.add(new WebMvcConfigurer() { @Override public void configureHandlerExceptionResolvers(List exceptionResolvers) { exceptionResolvers.add(new DefaultHandlerExceptionResolver()); @@ -195,7 +200,7 @@ public class DelegatingWebMvcConfigurationTests { final UrlPathHelper pathHelper = mock(UrlPathHelper.class); List configurers = new ArrayList<>(); - configurers.add(new WebMvcConfigurerAdapter() { + configurers.add(new WebMvcConfigurer() { @Override public void configurePathMatch(PathMatchConfigurer configurer) { configurer.setUseRegisteredSuffixPatternMatch(true) diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/ViewResolutionIntegrationTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/ViewResolutionIntegrationTests.java index 902e3f285ebbe7a06c8743f3d739fbc6113cfc72..c8611b8efdcca3776ed8e46a32c595f33bc83bb1 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/ViewResolutionIntegrationTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/ViewResolutionIntegrationTests.java @@ -40,7 +40,7 @@ import org.springframework.web.servlet.view.freemarker.FreeMarkerViewResolver; import org.springframework.web.servlet.view.groovy.GroovyMarkupConfigurer; import org.springframework.web.servlet.view.tiles3.TilesConfigurer; -import static org.junit.Assert.*; +import static org.junit.Assert.assertEquals; /** * Integration tests for view resolution with {@code @EnableWebMvc}. @@ -128,7 +128,7 @@ public class ViewResolutionIntegrationTests { } @EnableWebMvc - static abstract class AbstractWebConfig extends WebMvcConfigurerAdapter { + static abstract class AbstractWebConfig implements WebMvcConfigurer { @Bean public SampleController sampleController() { diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurationSupportTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurationSupportTests.java index 947a048bacd6b91c395fc4abd2d54c069828e37a..e1a53aba14ea67882151cfee9ba3ec90f0f7f0a5 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurationSupportTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/config/annotation/WebMvcConfigurationSupportTests.java @@ -18,7 +18,6 @@ package org.springframework.web.servlet.config.annotation; import java.util.List; import java.util.Locale; - import javax.servlet.http.HttpServletRequest; import com.fasterxml.jackson.databind.ObjectMapper; @@ -363,7 +362,7 @@ public class WebMvcConfigurationSupportTests { @EnableWebMvc @Configuration - static class CustomViewResolverOrderConfig extends WebMvcConfigurerAdapter { + static class CustomViewResolverOrderConfig implements WebMvcConfigurer { @Override public void configureViewResolvers(ViewResolverRegistry registry) { @@ -374,7 +373,7 @@ public class WebMvcConfigurationSupportTests { @EnableWebMvc @Configuration - static class CustomArgumentResolverConfig extends WebMvcConfigurerAdapter { + static class CustomArgumentResolverConfig implements WebMvcConfigurer { @Override public void addArgumentResolvers(List resolvers) { diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilderTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilderTests.java index 2536dce8f89f13e11a9cd0760595f9597cabfdf9..5327686c3142e88041d19d80f82208053ef48f07 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilderTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilderTests.java @@ -50,13 +50,21 @@ import org.springframework.web.context.request.ServletRequestAttributes; import org.springframework.web.context.support.AnnotationConfigWebApplicationContext; import org.springframework.web.servlet.DispatcherServlet; import org.springframework.web.servlet.config.annotation.EnableWebMvc; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; import org.springframework.web.util.UriComponents; import org.springframework.web.util.UriComponentsBuilder; -import static org.hamcrest.Matchers.*; -import static org.junit.Assert.*; -import static org.springframework.web.servlet.mvc.method.annotation.MvcUriComponentsBuilder.*; +import static org.hamcrest.Matchers.contains; +import static org.hamcrest.Matchers.containsInAnyOrder; +import static org.hamcrest.Matchers.endsWith; +import static org.hamcrest.Matchers.is; +import static org.hamcrest.Matchers.startsWith; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertThat; +import static org.springframework.web.servlet.mvc.method.annotation.MvcUriComponentsBuilder.fromController; +import static org.springframework.web.servlet.mvc.method.annotation.MvcUriComponentsBuilder.fromMethodCall; +import static org.springframework.web.servlet.mvc.method.annotation.MvcUriComponentsBuilder.fromMethodName; +import static org.springframework.web.servlet.mvc.method.annotation.MvcUriComponentsBuilder.on; /** * Unit tests for {@link org.springframework.web.servlet.mvc.method.annotation.MvcUriComponentsBuilder}. @@ -500,7 +508,7 @@ public class MvcUriComponentsBuilderTests { @EnableWebMvc - static class WebConfig extends WebMvcConfigurerAdapter { + static class WebConfig implements WebMvcConfigurer { @Bean public PersonsAddressesController controller() { diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/RequestPartIntegrationTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/RequestPartIntegrationTests.java index 5813a6d561953fad5e7000a993755c5561f097bb..15b7da41e5eecae97879ada543fca53e183db3ed 100644 --- a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/RequestPartIntegrationTests.java +++ b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/RequestPartIntegrationTests.java @@ -65,10 +65,10 @@ import org.springframework.web.multipart.commons.CommonsMultipartResolver; import org.springframework.web.multipart.support.StandardServletMultipartResolver; import org.springframework.web.servlet.DispatcherServlet; import org.springframework.web.servlet.config.annotation.EnableWebMvc; -import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter; +import org.springframework.web.servlet.config.annotation.WebMvcConfigurer; -import static org.junit.Assert.*; -import static org.springframework.web.bind.annotation.RequestMethod.*; +import static org.junit.Assert.assertEquals; +import static org.springframework.web.bind.annotation.RequestMethod.POST; /** * Test access to parts of a multipart request with {@link RequestPart}. @@ -198,7 +198,7 @@ public class RequestPartIntegrationTests { @Configuration @EnableWebMvc - static class RequestPartTestConfig extends WebMvcConfigurerAdapter { + static class RequestPartTestConfig implements WebMvcConfigurer { @Bean public RequestPartTestController controller() { diff --git a/src/docs/asciidoc/web/web-cors.adoc b/src/docs/asciidoc/web/web-cors.adoc index d32a095f16ba400b0a15282dff5b762a8e24f96b..33b566fd4ddf09044c0af28082a4f82a2cbe4d52 100644 --- a/src/docs/asciidoc/web/web-cors.adoc +++ b/src/docs/asciidoc/web/web-cors.adoc @@ -125,7 +125,7 @@ Enabling CORS for the whole application is as simple as: ---- @Configuration @EnableWebMvc -public class WebConfig extends WebMvcConfigurerAdapter { +public class WebConfig implements WebMvcConfigurer { @Override public void addCorsMappings(CorsRegistry registry) { @@ -142,7 +142,7 @@ specific path pattern: ---- @Configuration @EnableWebMvc -public class WebConfig extends WebMvcConfigurerAdapter { +public class WebConfig implements WebMvcConfigurer { @Override public void addCorsMappings(CorsRegistry registry) { diff --git a/src/docs/asciidoc/web/web-mvc.adoc b/src/docs/asciidoc/web/web-mvc.adoc index daabe4862ddd23bdbfcdcb9875ac9e3dc164c35e..d891070dfe1a248c7c21aa9d77c97d5c57a3d5c2 100644 --- a/src/docs/asciidoc/web/web-mvc.adoc +++ b/src/docs/asciidoc/web/web-mvc.adoc @@ -4522,7 +4522,7 @@ a `CacheControl` instance, which supports more specific directives: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void addResourceHandlers(ResourceHandlerRegistry registry) { @@ -4935,7 +4935,7 @@ It also automatically registers the following well-known modules if they are det [[mvc-config-customize]] === Customizing the Provided Configuration To customize the default configuration in Java you simply implement the -`WebMvcConfigurer` interface or more likely extend the class `WebMvcConfigurerAdapter` +`WebMvcConfigurer` interface or more likely extend the class `WebMvcConfigurer` and override the methods you need: [source,java,indent=0] @@ -4943,7 +4943,7 @@ and override the methods you need: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { // Override configuration methods... @@ -4970,7 +4970,7 @@ register custom formatters and converters, override the `addFormatters` method: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void addFormatters(FormatterRegistry registry) { @@ -5064,7 +5064,7 @@ Alternatively you can configure your own global `Validator` instance: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public Validator getValidator(); { @@ -5129,7 +5129,7 @@ An example of registering interceptors in Java: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void addInterceptors(InterceptorRegistry registry) { @@ -5183,7 +5183,7 @@ Below is an example of customizing content negotiation options through the MVC J ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void configureContentNegotiation(ContentNegotiationConfigurer configurer) { @@ -5242,7 +5242,7 @@ An example of forwarding a request for `"/"` to a view called `"home"` in Java: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void addViewControllers(ViewControllerRegistry registry) { @@ -5274,7 +5274,7 @@ JSON rendering: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void configureViewResolvers(ViewResolverRegistry registry) { @@ -5331,7 +5331,7 @@ In Java config simply add the respective "Configurer" bean: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void configureViewResolvers(ViewResolverRegistry registry) { @@ -5370,7 +5370,7 @@ to serve resource requests with a URL pattern of `/resources/{asterisk}{asterisk ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void addResourceHandlers(ResourceHandlerRegistry registry) { @@ -5396,7 +5396,7 @@ browser cache and a reduction in HTTP requests made by the browser: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void addResourceHandlers(ResourceHandlerRegistry registry) { @@ -5430,7 +5430,7 @@ serving of resources from both the web application root and from a known path of ---- @EnableWebMvc @Configuration - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void addResourceHandlers(ResourceHandlerRegistry registry) { @@ -5477,7 +5477,7 @@ Java config example; ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void addResourceHandlers(ResourceHandlerRegistry registry) { @@ -5544,7 +5544,7 @@ To enable the feature using the default setup use: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void configureDefaultServletHandling(DefaultServletHandlerConfigurer configurer) { @@ -5576,7 +5576,7 @@ then the default Servlet's name must be explicitly provided as in the following ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void configureDefaultServletHandling(DefaultServletHandlerConfigurer configurer) { @@ -5609,7 +5609,7 @@ Below is an example in Java config: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void configurePathMatch(PathMatchConfigurer configurer) { @@ -5658,9 +5658,9 @@ And the same in XML, use the `` element: === Message Converters Customization of `HttpMessageConverter` can be achieved in Java config by overriding -{api-spring-framework}/web/servlet/config/annotation/WebMvcConfigurerAdapter.html#configureMessageConverters-java.util.List-[`configureMessageConverters()`] +{api-spring-framework}/web/servlet/config/annotation/WebMvcConfigurer.html#configureMessageConverters-java.util.List-[`configureMessageConverters()`] if you want to replace the default converters created by Spring MVC, or by overriding -{api-spring-framework}/web/servlet/config/annotation/WebMvcConfigurerAdapter.html#extendMessageConverters-java.util.List-[`extendMessageConverters()`] +{api-spring-framework}/web/servlet/config/annotation/WebMvcConfigurer.html#extendMessageConverters-java.util.List-[`extendMessageConverters()`] if you just want to customize them or add additional converters to the default ones. Below is an example that adds Jackson JSON and XML converters with a customized @@ -5671,7 +5671,7 @@ Below is an example that adds Jackson JSON and XML converters with a customized ---- @Configuration @EnableWebMvc - public class WebConfiguration extends WebMvcConfigurerAdapter { + public class WebConfiguration implements WebMvcConfigurer { @Override public void configureMessageConverters(List> converters) { @@ -5779,7 +5779,7 @@ or a single `@EnableWebMvc` annotated class, since they both register the same u beans. Modifying beans in this way does not prevent you from using any of the higher-level -constructs shown earlier in this section. `WebMvcConfigurerAdapter` subclasses and +constructs shown earlier in this section. `WebMvcConfigurer` subclasses and `WebMvcConfigurer` implementations are still being used. ==== diff --git a/src/docs/asciidoc/web/web-view.adoc b/src/docs/asciidoc/web/web-view.adoc index a52bb9352c36412c0522edecf09ea0178f7a5763..9ea984103f769d56243195f4f1b79c6fa0fd70bb 100644 --- a/src/docs/asciidoc/web/web-view.adoc +++ b/src/docs/asciidoc/web/web-view.adoc @@ -47,7 +47,7 @@ Configuring the Groovy Markup Template Engine is quite easy: ---- @Configuration @EnableWebMvc - public class WebConfig extends WebMvcConfigurerAdapter { + public class WebConfig implements WebMvcConfigurer { @Override public void configureViewResolvers(ViewResolverRegistry registry) { @@ -1385,7 +1385,7 @@ provided with Java 8+, you should declare the following configuration: ---- @Configuration @EnableWebMvc - public class MustacheConfig extends WebMvcConfigurerAdapter { + public class MustacheConfig implements WebMvcConfigurer { @Override public void configureViewResolvers(ViewResolverRegistry registry) { @@ -1472,7 +1472,7 @@ browser facilities not available in the server-side script engine. ---- @Configuration @EnableWebMvc - public class MustacheConfig extends WebMvcConfigurerAdapter { + public class MustacheConfig implements WebMvcConfigurer { @Override public void configureViewResolvers(ViewResolverRegistry registry) { @@ -1746,7 +1746,7 @@ regular MVC annotation configuration. @EnableWebMvc @ComponentScan @Configuration -public class WebConfig extends WebMvcConfigurerAdapter { +public class WebConfig implements WebMvcConfigurer { @Bean public XsltViewResolver xsltViewResolver() {