You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jb...@apache.org on 2006/07/31 22:50:15 UTC
svn commit: r427255 -
/geronimo/branches/1.1/modules/system/src/java/org/apache/geronimo/system/configuration/GBeanOverride.java
Author: jbohn
Date: Mon Jul 31 13:50:14 2006
New Revision: 427255
URL: http://svn.apache.org/viewvc?rev=427255&view=rev
Log:
GERONIMO-2241 prevent duplicate attributes with both values and null in config.xml
Modified:
geronimo/branches/1.1/modules/system/src/java/org/apache/geronimo/system/configuration/GBeanOverride.java
Modified: geronimo/branches/1.1/modules/system/src/java/org/apache/geronimo/system/configuration/GBeanOverride.java
URL: http://svn.apache.org/viewvc/geronimo/branches/1.1/modules/system/src/java/org/apache/geronimo/system/configuration/GBeanOverride.java?rev=427255&r1=427254&r2=427255&view=diff
==============================================================================
--- geronimo/branches/1.1/modules/system/src/java/org/apache/geronimo/system/configuration/GBeanOverride.java (original)
+++ geronimo/branches/1.1/modules/system/src/java/org/apache/geronimo/system/configuration/GBeanOverride.java Mon Jul 31 13:50:14 2006
@@ -388,6 +388,9 @@
setNullAttribute(name);
}
else {
+ if (getNullAttribute(name)) {
+ nullAttributes.remove(name);
+ }
if(name.toLowerCase().indexOf("password") > -1) {
value = EncryptionManager.encrypt(value);
}