You are viewing a plain text version of this content. The canonical link for it is here.
Posted to svn@forrest.apache.org by th...@apache.org on 2006/08/03 17:21:04 UTC

svn commit: r428422 - /forrest/trunk/main/java/org/apache/forrest/conf/ForrestConfModule.java

Author: thorsten
Date: Thu Aug  3 08:21:03 2006
New Revision: 428422

URL: http://svn.apache.org/viewvc?rev=428422&view=rev
Log:
Enhancing error message, giving instructions how to fix the problem.

Modified:
    forrest/trunk/main/java/org/apache/forrest/conf/ForrestConfModule.java

Modified: forrest/trunk/main/java/org/apache/forrest/conf/ForrestConfModule.java
URL: http://svn.apache.org/viewvc/forrest/trunk/main/java/org/apache/forrest/conf/ForrestConfModule.java?rev=428422&r1=428421&r2=428422&view=diff
==============================================================================
--- forrest/trunk/main/java/org/apache/forrest/conf/ForrestConfModule.java (original)
+++ forrest/trunk/main/java/org/apache/forrest/conf/ForrestConfModule.java Thu Aug  3 08:21:03 2006
@@ -74,7 +74,19 @@
             original = "(not defined in forrest.xconf)";
             attributeValue = filteringProperties.getProperty(name);
             if (attributeValue == null) {
-                throw new ConfigurationException("Unable to get attribute value for " + name);
+                String error = "Unable to get attribute value for "
+                    + name
+                    + "\n"
+                    + "Please make sure you defined "
+                    + name
+                    + " in forrest.properties.xml either in $PROJECT_HOME "
+                    + "or in the default.forrest.properties.xml of the plugin "
+                    + "that is requesting this property."
+                    + "\n" 
+                    + "If you see this message, most of the time you spotted a plugin bug "
+                    + "(forget to define the default property). Please report it to our mailing list.";
+            throw new ConfigurationException(
+                    error);
             }
         }