a small bugfix
[mir.git] / source / mir / config / MirPropertiesConfiguration.java
index 50196e8..359ad46 100755 (executable)
  */
 package mir.config;
 
-import multex.Exc;
-import multex.Failure;
-
-import org.apache.commons.configuration.PropertiesConfiguration;
-
 import java.io.File;
 import java.io.FileNotFoundException;
 import java.io.IOException;
-
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
-
 import javax.servlet.ServletContext;
 
+import multex.Exc;
+import multex.Failure;
+
+import org.apache.commons.collections.ExtendedProperties;
+
 
 /**
  * @author idefix
  */
-public class MirPropertiesConfiguration extends PropertiesConfiguration {
+public class MirPropertiesConfiguration extends ExtendedProperties {
   private static MirPropertiesConfiguration instance;
   private static ServletContext context;
   private static String contextPath;
 
+  //if one of these properties is not present a new
+  //property is added with its default value;
+  private static NeededProperty[] neededWithValue =
+  {
+    new NeededProperty("Producer.DocRoot", ""),
+    new NeededProperty("Producer.ImageRoot", ""),
+    new NeededProperty("Producer.Image.Path", ""),
+    new NeededProperty("Producer.Media.Path", ""),
+    new NeededProperty("Producer.RealMedia.Path", ""),
+    new NeededProperty("Producer.Image.IconPath", "")
+  };
+
   /**
    * Constructor for MirPropertiesConfiguration.
    */
   private MirPropertiesConfiguration(ServletContext ctx, String ctxPath)
     throws IOException {
-    super(ctx.getRealPath("/WEB-INF/etc/") + "/config.properties",
-      ctx.getRealPath("/WEB-INF/etc/") + "/default.properties");
-    this.addProperty("Home", ctx.getRealPath("/WEB-INF/") + "/");
-    this.addProperty("RootUri", ctxPath);
+    //loading the defaults-config
+    super(ctx.getRealPath("/WEB-INF/") + "/default.properties");
+    //loading the user-config
+    ExtendedProperties userConfig = 
+       new ExtendedProperties(ctx.getRealPath("/WEB-INF/etc/") + "/config.properties");
+    //merging them to one config while overriding the defaults
+    this.combine(userConfig);
+    addProperty("Home", ctx.getRealPath("/WEB-INF/") + "/");
+    checkMissing();
   }
 
   public static synchronized MirPropertiesConfiguration instance()
     throws PropertiesConfigExc {
     if (instance == null) {
-      if ((context == null) || (contextPath == null) ||
-          (contextPath.length() == 0)) {
+      if (context == null) {
         throw new MirPropertiesConfiguration.PropertiesConfigExc(
           "Context was not set");
       }
@@ -93,33 +107,40 @@ public class MirPropertiesConfiguration extends PropertiesConfiguration {
   }
 
   /**
-   * Sets the contextPath.
-   * @param contextPath The contextPath to set
+   * Returns the context.
+   * @return ServletContext
    */
-  public static void setContextPath(String contextPath) {
-    MirPropertiesConfiguration.contextPath = contextPath;
+  public static ServletContext getContext() {
+    return context;
   }
 
+  /**
+   * Returns all properties in a Map
+   * @return Map
+   */
   public Map allSettings() {
     Iterator iterator = this.getKeys();
     Map returnMap = new HashMap();
-
     while (iterator.hasNext()) {
       String key = (String) iterator.next();
-      returnMap.put(key, this.getProperty(key));
+      Object o = this.getProperty(key);
+
+      if (o == null) {
+        o = new Object();
+      }
+
+      returnMap.put(key, o);
     }
 
     return returnMap;
   }
 
   /**
-   * Returns the context.
-   * @return ServletContext
+   * Returns a String-property concatenated with the home-dir of the
+   * installation
+   * @param key
+   * @return String
    */
-  public static ServletContext getContext() {
-    return context;
-  }
-
   public String getStringWithHome(String key) {
     String returnString = getString(key);
 
@@ -130,6 +151,17 @@ public class MirPropertiesConfiguration extends PropertiesConfiguration {
     return getString("Home") + returnString;
   }
 
+  /**
+   * Checks if one property is missing and adds a default value
+   */
+  private void checkMissing() {
+    for (int i = 0; i < neededWithValue.length; i++) {
+      if (super.getProperty(neededWithValue[i].getKey()) == null) {
+        addProperty(neededWithValue[i].getKey(), neededWithValue[i].getValue());
+      }
+    }
+  }
+
   public File getFile(String key) throws FileNotFoundException {
     String path = getStringWithHome(key);
     File returnFile = new File(path);
@@ -142,16 +174,49 @@ public class MirPropertiesConfiguration extends PropertiesConfiguration {
   }
 
   /**
+   * @return the vlaue of this property as String
+   * @param key the key of this property
    * @see org.apache.commons.configuration.Configuration#getString(java.lang.String)
    */
   public String getString(String key) {
-    if (super.getString(key) == null) {
+       return getString(key, "");
+  }
+  
+  
+  /** 
+   * @return the value of this property as String
+   * @param key the key of the property
+   * @param defaultValue the default value of this property if it is null
+   * @see org.apache.commons.collections.ExtendedProperties#getString(java.lang.String, java.lang.String)
+   */
+  public String getString(String key, String defaultValue) {
+               Object object = getProperty(key);       
+               if(object == null){
+                       if (defaultValue == null) {
+                               return new String();
+                       }
+                 return defaultValue;
+               }                       
+               if (object instanceof String) {
+                       return (String)object;
+               }
+               return object.toString();
+  }
+
+  /**
+   * Returns a property according to the given key
+   * @param key the key of the property
+   * @return the value of the property as Object, if no property available it returns a empty String
+   * @see org.apache.commons.configuration.Configuration#getString(java.lang.String)
+   */
+  public Object getProperty(String key) {
+    if (super.getProperty(key) == null) {
       return new String();
     }
 
-    return super.getString(key);
+    return super.getProperty(key);
   }
-
+  
   /**
    * @author idefix
    */
@@ -177,4 +242,26 @@ public class MirPropertiesConfiguration extends PropertiesConfiguration {
       super(msg, cause);
     }
   }
+
+  /**
+   * A Class for properties to be checked
+   * @author idefix
+   */
+  private static class NeededProperty {
+    private String _key;
+    private String _value;
+
+    public NeededProperty(String key, String value) {
+      _key = key;
+      _value = value;
+    }
+
+    public String getKey() {
+      return _key;
+    }
+
+    public String getValue() {
+      return _value;
+    }
+  }
 }