Browse Source

@EnableMBeanExport supports empty placeholders as well

Issue: SPR-11105
pull/421/merge
Juergen Hoeller 12 years ago
parent
commit
ef3a3b03ce
  1. 12
      spring-context/src/main/java/org/springframework/context/annotation/MBeanExportConfiguration.java

12
spring-context/src/main/java/org/springframework/context/annotation/MBeanExportConfiguration.java

@ -89,20 +89,20 @@ public class MBeanExportConfiguration implements ImportAware, EnvironmentAware,
private void setupDomain(AnnotationMBeanExporter exporter) { private void setupDomain(AnnotationMBeanExporter exporter) {
String defaultDomain = this.attributes.getString("defaultDomain"); String defaultDomain = this.attributes.getString("defaultDomain");
if (defaultDomain != null && this.environment != null) {
defaultDomain = this.environment.resolvePlaceholders(defaultDomain);
}
if (StringUtils.hasText(defaultDomain)) { if (StringUtils.hasText(defaultDomain)) {
if (this.environment != null) {
defaultDomain = this.environment.resolvePlaceholders(defaultDomain);
}
exporter.setDefaultDomain(defaultDomain); exporter.setDefaultDomain(defaultDomain);
} }
} }
private void setupServer(AnnotationMBeanExporter exporter) { private void setupServer(AnnotationMBeanExporter exporter) {
String server = this.attributes.getString("server"); String server = this.attributes.getString("server");
if (server != null && this.environment != null) {
server = this.environment.resolvePlaceholders(server);
}
if (StringUtils.hasText(server)) { if (StringUtils.hasText(server)) {
if (this.environment != null) {
server = this.environment.resolvePlaceholders(server);
}
exporter.setServer(this.beanFactory.getBean(server, MBeanServer.class)); exporter.setServer(this.beanFactory.getBean(server, MBeanServer.class));
} }
else { else {

Loading…
Cancel
Save