You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ga...@apache.org on 2007/10/30 03:02:46 UTC

svn commit: r589936 - /geronimo/server/trunk/modules/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbRefBuilder.java

Author: gawor
Date: Mon Oct 29 19:02:46 2007
New Revision: 589936

URL: http://svn.apache.org/viewvc?rev=589936&view=rev
Log:
with recent KeyedCollection OpenEJB changes we must set the reference name before it is added to the collection since the underlying collection uses the reference name as a key in the map

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

Modified: geronimo/server/trunk/modules/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbRefBuilder.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/modules/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbRefBuilder.java?rev=589936&r1=589935&r2=589936&view=diff
==============================================================================
--- geronimo/server/trunk/modules/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbRefBuilder.java (original)
+++ geronimo/server/trunk/modules/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbRefBuilder.java Mon Oct 29 19:02:46 2007
@@ -209,10 +209,10 @@
             }
             // create the ejb-ref
             EjbRef ref = new EjbRef();
-            jndiConsumer.getEjbRef().add(ref);
-
             // ejb-ref-name
             ref.setEjbRefName(refName);
+            
+            jndiConsumer.getEjbRef().add(ref);
 
             // ejb-ref-type
             String refType = getStringValue(xmlbeansRef.getEjbRefType());
@@ -283,10 +283,10 @@
 
             // create the ejb-ref
             EjbLocalRef ref = new EjbLocalRef();
-            jndiConsumer.getEjbLocalRef().add(ref);
-
             // ejb-ref-name
             ref.setEjbRefName(refName);
+            
+            jndiConsumer.getEjbLocalRef().add(ref);
 
             // ejb-ref-type
             String refType = getStringValue(xmlbeansRef.getEjbRefType());