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

XNAT-2429 Fixed issue where initializer still loaded plugins into

application context, moving it back into root. Updated
gradle-git-version plugin.
parent b9d864fd
No related branches found
No related tags found
No related merge requests found
...@@ -41,7 +41,7 @@ buildscript { ...@@ -41,7 +41,7 @@ buildscript {
dependencies { dependencies {
classpath "com.bmuschko:gradle-cargo-plugin:2.2.2" classpath "com.bmuschko:gradle-cargo-plugin:2.2.2"
classpath "com.bmuschko:gradle-tomcat-plugin:2.2.4" classpath "com.bmuschko:gradle-tomcat-plugin:2.2.4"
classpath "gradle.plugin.com.palantir.gradle.gitversion:gradle-git-version:0.5.1" classpath "gradle.plugin.com.palantir.gradle.gitversion:gradle-git-version:0.5.2"
} }
} }
......
...@@ -69,15 +69,15 @@ public class XnatWebAppInitializer extends AbstractAnnotationConfigDispatcherSer ...@@ -69,15 +69,15 @@ public class XnatWebAppInitializer extends AbstractAnnotationConfigDispatcherSer
@Override @Override
protected Class<?>[] getRootConfigClasses() { protected Class<?>[] getRootConfigClasses() {
return new Class<?>[]{RootConfig.class}; final List<Class<?>> configClasses = new ArrayList<>();
configClasses.add(RootConfig.class);
configClasses.addAll(getPluginConfigs());
return configClasses.toArray(new Class[configClasses.size()]);
} }
@Override @Override
protected Class<?>[] getServletConfigClasses() { protected Class<?>[] getServletConfigClasses() {
final List<Class<?>> configClasses = new ArrayList<>(); return new Class[0];
// configClasses.add(ApplicationConfig.class);
configClasses.addAll(getPluginConfigs());
return configClasses.toArray(new Class[configClasses.size()]);
} }
@Override @Override
...@@ -117,6 +117,9 @@ public class XnatWebAppInitializer extends AbstractAnnotationConfigDispatcherSer ...@@ -117,6 +117,9 @@ public class XnatWebAppInitializer extends AbstractAnnotationConfigDispatcherSer
final List<Class<?>> configs = new ArrayList<>(); final List<Class<?>> configs = new ArrayList<>();
try { try {
for (final XnatPluginBean plugin : XnatPluginBean.findAllXnatPluginBeans()) { for (final XnatPluginBean plugin : XnatPluginBean.findAllXnatPluginBeans()) {
if (_log.isInfoEnabled()) {
_log.info("Found plugin {} {}: {}", plugin.getId(), plugin.getName(), plugin.getDescription());
}
configs.add(Class.forName(plugin.getPluginClass())); configs.add(Class.forName(plugin.getPluginClass()));
} }
} catch (IOException e) { } catch (IOException e) {
...@@ -125,6 +128,10 @@ public class XnatWebAppInitializer extends AbstractAnnotationConfigDispatcherSer ...@@ -125,6 +128,10 @@ public class XnatWebAppInitializer extends AbstractAnnotationConfigDispatcherSer
_log.error("Did not find a class specified in a plugin definition.", e); _log.error("Did not find a class specified in a plugin definition.", e);
} }
if (_log.isInfoEnabled()) {
_log.info("Found a total of {} plugins", configs.size());
}
return configs; return configs;
} }
......
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