Author: fmeschbe
Date: Sun Jan  9 02:20:49 2011
New Revision: 1056866

URL: http://svn.apache.org/viewvc?rev=1056866&view=rev
Log:
Better handle non-String configuration properties

Modified:
    
felix/trunk/http/jetty/src/main/java/org/apache/felix/http/jetty/internal/JettyConfig.java

Modified: 
felix/trunk/http/jetty/src/main/java/org/apache/felix/http/jetty/internal/JettyConfig.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/http/jetty/src/main/java/org/apache/felix/http/jetty/internal/JettyConfig.java?rev=1056866&r1=1056865&r2=1056866&view=diff
==============================================================================
--- 
felix/trunk/http/jetty/src/main/java/org/apache/felix/http/jetty/internal/JettyConfig.java
 (original)
+++ 
felix/trunk/http/jetty/src/main/java/org/apache/felix/http/jetty/internal/JettyConfig.java
 Sun Jan  9 02:20:49 2011
@@ -181,21 +181,20 @@ public final class JettyConfig
 
     private String getProperty(Dictionary props, String name, String defValue)
     {
-        String value = (String)props.get(name);
-        if (value == null) {
+        Object value = props.get(name);
+        if (value == null)
+        {
             value = this.context.getProperty(name);
         }
 
-        return value != null ? value : defValue;
+        return value != null ? String.valueOf(value) : defValue;
     }
 
     private boolean getBooleanProperty(Dictionary props, String name, boolean 
defValue)
     {
         String value = getProperty(props, name, null);
-        if (value == null) {
-            value = this.context.getProperty(name);
-        }
-        if (value != null) {
+        if (value != null)
+        {
             return (value.equalsIgnoreCase("true") || 
value.equalsIgnoreCase("yes"));
         }
 


Reply via email to