You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by xu...@apache.org on 2011/02/21 13:34:05 UTC
svn commit: r1072946 -
/geronimo/server/trunk/plugins/openejb/geronimo-openejb-builder/src/main/java/org/apache/geronimo/openejb/deployment/EjbModuleBuilder.java
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