From cc9d0f41ed94043b2373d4de3e04f229c106c5c0 Mon Sep 17 00:00:00 2001 From: Oliver Gierke Date: Mon, 8 Aug 2016 12:08:43 +0200 Subject: [PATCH] DATAREST-866 - Avoid premature initialization of RepositoryRestMvcConfiguration. Previously, RepositoryRestMvcConfiguration declared a BeanPostProcessor instance in a non-static bean factory method. That required Spring to instantiate the configuration class and keeping it in uninitialized state, which could in turn cause issues downstream. We now use a static method so that the container can obtain the bean instance without having to create an instance of the configuration class and thus deferring its initialization. --- .../rest/webmvc/config/RepositoryRestMvcConfiguration.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/spring-data-rest-webmvc/src/main/java/org/springframework/data/rest/webmvc/config/RepositoryRestMvcConfiguration.java b/spring-data-rest-webmvc/src/main/java/org/springframework/data/rest/webmvc/config/RepositoryRestMvcConfiguration.java index 1d04b567f..cb8279863 100644 --- a/spring-data-rest-webmvc/src/main/java/org/springframework/data/rest/webmvc/config/RepositoryRestMvcConfiguration.java +++ b/spring-data-rest-webmvc/src/main/java/org/springframework/data/rest/webmvc/config/RepositoryRestMvcConfiguration.java @@ -273,7 +273,8 @@ public class RepositoryRestMvcConfiguration extends HateoasAwareSpringDataWebCon } @Bean - public ProjectionDefinitionRegistar projectionDefinitionRegistrar(ObjectFactory config) { + public static ProjectionDefinitionRegistar projectionDefinitionRegistrar( + ObjectFactory config) { return new ProjectionDefinitionRegistar(config); }