geronimo-scm mailing list archives

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());