You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2006/12/12 19:45:58 UTC

svn commit: r486273 - /geronimo/server/trunk/modules/geronimo-naming-builder/src/main/java/org/apache/geronimo/naming/deployment/AbstractNamingBuilder.java

Author: djencks
Date: Tue Dec 12 10:45:57 2006
New Revision: 486273

URL: http://svn.apache.org/viewvc?view=rev&rev=486273
Log:
GERONIMO-2652 fix for mysterious xmlbeans validation problems fixed by copying even elements that are already the correct type

Modified:
    geronimo/server/trunk/modules/geronimo-naming-builder/src/main/java/org/apache/geronimo/naming/deployment/AbstractNamingBuilder.java

Modified: geronimo/server/trunk/modules/geronimo-naming-builder/src/main/java/org/apache/geronimo/naming/deployment/AbstractNamingBuilder.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-naming-builder/src/main/java/org/apache/geronimo/naming/deployment/AbstractNamingBuilder.java?view=diff&rev=486273&r1=486272&r2=486273
==============================================================================
--- geronimo/server/trunk/modules/geronimo-naming-builder/src/main/java/org/apache/geronimo/naming/deployment/AbstractNamingBuilder.java (original)
+++ geronimo/server/trunk/modules/geronimo-naming-builder/src/main/java/org/apache/geronimo/naming/deployment/AbstractNamingBuilder.java Tue Dec 12 10:45:57 2006
@@ -128,9 +128,8 @@
         //bizarre ArrayStoreException if xmlObjects is loaded by the wrong classloader
         XmlObject[] converted = new XmlObject[xmlObjects.length];
         for (int i = 0; i < xmlObjects.length; i++) {
-            XmlObject xmlObject = xmlObjects[i];
+            XmlObject xmlObject = xmlObjects[i].copy();
             if (xmlObject.schemaType() != type) {
-                xmlObject = xmlObject.copy();
                 converter.convertElement(xmlObject);
                 converted[i] = xmlObject.changeType(type);
             } else {