You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by gi...@apache.org on 2007/11/25 20:13:55 UTC
svn commit: r598033 -
/cocoon/trunk/core/cocoon-configuration/cocoon-configuration-api/src/main/java/org/apache/cocoon/configuration/PropertyHelper.java
Author: giacomo
Date: Sun Nov 25 11:13:53 2007
New Revision: 598033
URL: http://svn.apache.org/viewvc?rev=598033&view=rev
Log:
revert last change
Modified:
cocoon/trunk/core/cocoon-configuration/cocoon-configuration-api/src/main/java/org/apache/cocoon/configuration/PropertyHelper.java
Modified: cocoon/trunk/core/cocoon-configuration/cocoon-configuration-api/src/main/java/org/apache/cocoon/configuration/PropertyHelper.java
URL: http://svn.apache.org/viewvc/cocoon/trunk/core/cocoon-configuration/cocoon-configuration-api/src/main/java/org/apache/cocoon/configuration/PropertyHelper.java?rev=598033&r1=598032&r2=598033&view=diff
==============================================================================
--- cocoon/trunk/core/cocoon-configuration/cocoon-configuration-api/src/main/java/org/apache/cocoon/configuration/PropertyHelper.java (original)
+++ cocoon/trunk/core/cocoon-configuration/cocoon-configuration-api/src/main/java/org/apache/cocoon/configuration/PropertyHelper.java Sun Nov 25 11:13:53 2007
@@ -159,10 +159,8 @@
String value = props.getProperty(key);
// replace
value = replace(value, props, settings);
- // and put back but prevent NPE because of null value ?!?!?
- // TODO: How to handle value==null situations?
- if (value != null)
- props.put(key, value);
+ // and put back
+ props.put(key, value);
}
}
}