Skip to content
Snippets Groups Projects
Commit c5f008aa authored by Rick Herrick's avatar Rick Herrick
Browse files

XNAT-4382 Moved HTTP message converters to separate bean instantiations so...

XNAT-4382 Moved HTTP message converters to separate bean instantiations so they can be picked up by plugins. Moved plugin configuration instantiation to servlet mapping.
parent b59b3952
No related branches found
No related tags found
No related merge requests found
...@@ -37,10 +37,7 @@ import springfox.documentation.spring.web.plugins.Docket; ...@@ -37,10 +37,7 @@ import springfox.documentation.spring.web.plugins.Docket;
import springfox.documentation.swagger2.annotations.EnableSwagger2; import springfox.documentation.swagger2.annotations.EnableSwagger2;
import javax.xml.bind.Marshaller; import javax.xml.bind.Marshaller;
import java.util.HashMap; import java.util.*;
import java.util.List;
import java.util.Locale;
import java.util.Map;
@Configuration @Configuration
@EnableWebMvc @EnableWebMvc
...@@ -61,9 +58,24 @@ public class WebConfig extends WebMvcConfigurerAdapter { ...@@ -61,9 +58,24 @@ public class WebConfig extends WebMvcConfigurerAdapter {
@Override @Override
public void configureMessageConverters(final List<HttpMessageConverter<?>> converters) { public void configureMessageConverters(final List<HttpMessageConverter<?>> converters) {
converters.add(new MappingJackson2HttpMessageConverter(_objectMapperBuilder.build())); converters.add(mappingJackson2HttpMessageConverter());
converters.add(new MarshallingHttpMessageConverter(_marshaller, _marshaller)); converters.add(marshallingHttpMessageConverter());
converters.add(new StringHttpMessageConverter()); converters.add(stringHttpMessageConverter());
}
@Bean
public HttpMessageConverter<?> mappingJackson2HttpMessageConverter() {
return new MappingJackson2HttpMessageConverter(_objectMapperBuilder.build());
}
@Bean
public HttpMessageConverter<?> marshallingHttpMessageConverter() {
return new MarshallingHttpMessageConverter(_marshaller, _marshaller);
}
@Bean
public HttpMessageConverter<?> stringHttpMessageConverter() {
return new StringHttpMessageConverter();
} }
@Bean @Bean
......
...@@ -69,15 +69,14 @@ public class XnatWebAppInitializer extends AbstractAnnotationConfigDispatcherSer ...@@ -69,15 +69,14 @@ public class XnatWebAppInitializer extends AbstractAnnotationConfigDispatcherSer
@Override @Override
protected Class<?>[] getRootConfigClasses() { protected Class<?>[] getRootConfigClasses() {
final List<Class<?>> configClasses = new ArrayList<>(); return new Class<?>[] { RootConfig.class };
configClasses.add(RootConfig.class);
configClasses.addAll(getPluginConfigs());
return configClasses.toArray(new Class[configClasses.size()]);
} }
@Override @Override
protected Class<?>[] getServletConfigClasses() { protected Class<?>[] getServletConfigClasses() {
return new Class[0]; final List<Class<?>> configClasses = new ArrayList<>();
configClasses.addAll(getPluginConfigs());
return configClasses.toArray(new Class[configClasses.size()]);
} }
@Override @Override
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment