Author: vanto
Date: Wed Dec  2 10:42:29 2009
New Revision: 886109

URL: http://svn.apache.org/viewvc?rev=886109&view=rev
Log:
ODE-721: make sure that property key created only once.

Modified:
    
ode/branches/APACHE_ODE_1.X/dao-jpa/src/main/java/org/apache/ode/dao/jpa/XmlDataDAOImpl.java

Modified: 
ode/branches/APACHE_ODE_1.X/dao-jpa/src/main/java/org/apache/ode/dao/jpa/XmlDataDAOImpl.java
URL: 
http://svn.apache.org/viewvc/ode/branches/APACHE_ODE_1.X/dao-jpa/src/main/java/org/apache/ode/dao/jpa/XmlDataDAOImpl.java?rev=886109&r1=886108&r2=886109&view=diff
==============================================================================
--- 
ode/branches/APACHE_ODE_1.X/dao-jpa/src/main/java/org/apache/ode/dao/jpa/XmlDataDAOImpl.java
 (original)
+++ 
ode/branches/APACHE_ODE_1.X/dao-jpa/src/main/java/org/apache/ode/dao/jpa/XmlDataDAOImpl.java
 Wed Dec  2 10:42:29 2009
@@ -123,6 +123,13 @@
         return null;
        }
 
+    private XmlDataProperty getPropertyObject(String propertyName) {
+        for (XmlDataProperty prop : _props) {
+            if (prop.getPropertyKey().equals(propertyName)) return prop;
+        }
+        return null;
+    }
+
        public ScopeDAO getScopeDAO() {
                return _scope;
        }
@@ -147,7 +154,12 @@
        }
 
        public void setProperty(String pname, String pvalue) {
-        _props.add(new XmlDataProperty(pname, pvalue, this));
+        XmlDataProperty prop = getPropertyObject(pname);
+        if (prop == null) {
+            _props.add(new XmlDataProperty(pname, pvalue, this));
+        } else {
+            prop.setPropertyValue(pvalue);
+        }
        }
 
 }


Reply via email to