geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jsis...@apache.org
Subject svn commit: r380604 - /geronimo/branches/1.0/modules/j2ee-schema/src/java/org/apache/geronimo/schema/SchemaConversionUtils.java
Date Fri, 24 Feb 2006 06:38:13 GMT
Author: jsisson
Date: Thu Feb 23 22:38:11 2006
New Revision: 380604

URL: http://svn.apache.org/viewcvs?rev=380604&view=rev
Log:
GERONIMO-1649 - Fix Invalid deployment descriptor error when deploying an EJB 2.0 MDB (see
associated test case in rev 380600) - Actually fix it this time (previous commit had actual
fix commented out to reproduce problem).

Modified:
    geronimo/branches/1.0/modules/j2ee-schema/src/java/org/apache/geronimo/schema/SchemaConversionUtils.java

Modified: geronimo/branches/1.0/modules/j2ee-schema/src/java/org/apache/geronimo/schema/SchemaConversionUtils.java
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.0/modules/j2ee-schema/src/java/org/apache/geronimo/schema/SchemaConversionUtils.java?rev=380604&r1=380603&r2=380604&view=diff
==============================================================================
--- geronimo/branches/1.0/modules/j2ee-schema/src/java/org/apache/geronimo/schema/SchemaConversionUtils.java
(original)
+++ geronimo/branches/1.0/modules/j2ee-schema/src/java/org/apache/geronimo/schema/SchemaConversionUtils.java
Thu Feb 23 22:38:11 2006
@@ -529,10 +529,9 @@
                         }
                     } else {
                         //cursor pointing at activation-config
-                        //cursor.toNextSibling();
+                        cursor.toNextSibling();
                         //cursor should now be at first element in JNDIEnvironmentRefsGroup
                     }
-                    cursor.toNextSibling();
                     convertToJNDIEnvironmentRefsGroup(cursor, moveable);
                 }
                 cursor.pop();



Mime
View raw message