]> source.dussan.org Git - vaadin-framework.git/commitdiff
Restore legacyPropertyToString value parsing logic (#11970)
authorLeif Åstrand <leif@vaadin.com>
Wed, 12 Jun 2013 08:11:08 +0000 (11:11 +0300)
committerLeif Åstrand <leif@vaadin.com>
Wed, 12 Jun 2013 08:11:08 +0000 (11:11 +0300)
Change-Id: If9d46577e4d59999f35fcf6478d06652ac80468a

server/src/com/vaadin/server/DefaultDeploymentConfiguration.java

index a55c3231f3f5e373221e796de8821e7d12d7af76..0a8096e84ccb84f1f59b033d00c3c480ca0cd101 100644 (file)
@@ -86,17 +86,25 @@ public class DefaultDeploymentConfiguration implements DeploymentConfiguration {
     }
 
     private void checkLegacyPropertyToString() {
+        // Verify that the default value has not changed without also
+        // updating logic here
+        assert DEFAULT_LEGACY_PROPERTY_TO_STRING.toString().equals("warning");
+
         String param = getApplicationOrSystemProperty(
-                Constants.SERVLET_PARAMETER_LEGACY_PROPERTY_TOSTRING,
-                DEFAULT_LEGACY_PROPERTY_TO_STRING.name().toLowerCase());
+                Constants.SERVLET_PARAMETER_LEGACY_PROPERTY_TOSTRING, "warning");
 
-        try {
-            legacyPropertyToStringMode = LegacyProperyToStringMode
-                    .valueOf(param.toUpperCase());
-        } catch (IllegalArgumentException e) {
-            getLogger().log(Level.WARNING,
-                    Constants.WARNING_UNKNOWN_LEGACY_PROPERTY_TOSTRING_VALUE,
-                    param);
+        if ("true".equals(param)) {
+            legacyPropertyToStringMode = LegacyProperyToStringMode.ENABLED;
+        } else if ("false".equals(param)) {
+            legacyPropertyToStringMode = LegacyProperyToStringMode.DISABLED;
+        } else {
+            if (!"warning".equals(param)) {
+                getLogger()
+                        .log(Level.WARNING,
+                                Constants.WARNING_UNKNOWN_LEGACY_PROPERTY_TOSTRING_VALUE,
+                                param);
+
+            }
             legacyPropertyToStringMode = DEFAULT_LEGACY_PROPERTY_TO_STRING;
         }
     }