Browse Source

Polishing

pull/1489/head
Juergen Hoeller 7 years ago
parent
commit
ca0983cd85
  1. 3
      spring-context/src/test/java/org/springframework/context/annotation/ConfigurationClassWithConditionTests.java
  2. 9
      spring-web/src/main/java/org/springframework/web/util/WebUtils.java

3
spring-context/src/test/java/org/springframework/context/annotation/ConfigurationClassWithConditionTests.java

@ -290,6 +290,7 @@ public class ConfigurationClassWithConditionTests { @@ -290,6 +290,7 @@ public class ConfigurationClassWithConditionTests {
@Never
@Import({ConfigurationNotCreated.class, RegistrarNotCreated.class, ImportSelectorNotCreated.class})
static class ImportsNotCreated {
static {
if (true) throw new RuntimeException();
}
@ -297,12 +298,14 @@ public class ConfigurationClassWithConditionTests { @@ -297,12 +298,14 @@ public class ConfigurationClassWithConditionTests {
@Configuration
static class ConfigurationNotCreated {
static {
if (true) throw new RuntimeException();
}
}
static class RegistrarNotCreated implements ImportBeanDefinitionRegistrar {
static {
if (true) throw new RuntimeException();
}

9
spring-web/src/main/java/org/springframework/web/util/WebUtils.java

@ -154,16 +154,15 @@ public abstract class WebUtils { @@ -154,16 +154,15 @@ public abstract class WebUtils {
String root = servletContext.getRealPath("/");
if (root == null) {
throw new IllegalStateException(
"Cannot set web app root system property when WAR file is not expanded");
"Cannot set web app root system property when WAR file is not expanded");
}
String param = servletContext.getInitParameter(WEB_APP_ROOT_KEY_PARAM);
String key = (param != null ? param : DEFAULT_WEB_APP_ROOT_KEY);
String oldValue = System.getProperty(key);
if (oldValue != null && !StringUtils.pathEquals(oldValue, root)) {
throw new IllegalStateException(
"Web app root system property already set to different value: '" +
key + "' = [" + oldValue + "] instead of [" + root + "] - " +
"Choose unique values for the 'webAppRootKey' context-param in your web.xml files!");
throw new IllegalStateException("Web app root system property already set to different value: '" +
key + "' = [" + oldValue + "] instead of [" + root + "] - " +
"Choose unique values for the 'webAppRootKey' context-param in your web.xml files!");
}
System.setProperty(key, root);
servletContext.log("Set web app root system property: '" + key + "' = [" + root + "]");

Loading…
Cancel
Save