geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From xuhaih...@apache.org
Subject svn commit: r1072946 - /geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
Date Mon, 21 Feb 2011 12:34:05 GMT
Author: xuhaihong
Date: Mon Feb 21 12:34:05 2011
New Revision: 1072946

URL: http://svn.apache.org/viewvc?rev=1072946&view=rev
Log:
GERONIMO-5816 Now in OpenEJB, it treats sessionContext and messageContext from callback methods
as normal resource-env injections, think we should not remove those reference types in unmap
method, or those injections will be lost

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

Modified: geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java?rev=1072946&r1=1072945&r2=1072946&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
(original)
+++ geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
Mon Feb 21 12:34:05 2011
@@ -35,8 +35,6 @@ import java.util.TreeSet;
 import java.util.jar.JarFile;
 
 import javax.ejb.EntityContext;
-import javax.ejb.MessageDrivenContext;
-import javax.ejb.SessionContext;
 import javax.ejb.TimerService;
 import javax.xml.namespace.QName;
 import javax.xml.ws.WebServiceContext;
@@ -481,11 +479,7 @@ public class EjbModuleBuilder implements
             }
             for (Iterator<ResourceEnvRef> iterator = enterpriseBean.getResourceEnvRef().iterator();
iterator.hasNext();) {
                 ResourceEnvRef ref = iterator.next();
-                if (ref.getType().equals(SessionContext.class.getName())) {
-                    iterator.remove();
-                } else if (ref.getType().equals(EntityContext.class.getName())) {
-                    iterator.remove();
-                } else if (ref.getType().equals(MessageDrivenContext.class.getName())) {
+                if (ref.getType().equals(EntityContext.class.getName())) {
                     iterator.remove();
                 } else if (ref.getType().equals(TimerService.class.getName())) {
                     iterator.remove();
@@ -644,7 +638,7 @@ public class EjbModuleBuilder implements
 
         // convert the plan to xmlbeans since geronimo naming is coupled on xmlbeans objects
             OpenejbGeronimoEjbJarType geronimoOpenejb = XmlUtil.convertToXmlbeans(geronimoEjbJarType);
-            
+
            ejbModule.getVendorDD().set(geronimoOpenejb);
 
         // todo move namingBuilders.buildEnvironment() here when geronimo naming supports
it



Mime
View raw message