Author: dblevins
Date: Fri Jun  4 23:15:45 2010
New Revision: 951613

URL: http://svn.apache.org/viewvc?rev=951613&view=rev
Log:
Reverting java: prefix change previously merged from trunk

Modified:
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/InjectionProcessor.java
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/BaseContext.java
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/cmp/jpa/JpaCmpEngine.java
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/managed/ManagedContainer.java
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/mdb/MdbInstanceFactory.java
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/singleton/SingletonInstanceManager.java
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateful/StatefulContainer.java
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateless/StatelessInstanceManager.java
    
openejb/branches/openejb-3.1.x/container/openejb-core/src/test/java/org/apache/openejb/config/BusinessInterfacesTest.java
   (props changed)

Modified: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/InjectionProcessor.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/InjectionProcessor.java?rev=951613&r1=951612&r2=951613&view=diff
==============================================================================
--- 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/InjectionProcessor.java
 (original)
+++ 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/InjectionProcessor.java
 Fri Jun  4 23:15:45 2010
@@ -198,13 +198,8 @@ public class InjectionProcessor<T> {
     public static Context unwrap(Context context) {
         if (context == null) return null;
         try {
-            context = (Context) context.lookup("comp/env/");
+            context = (Context) context.lookup("java:comp/env/");
         } catch (NamingException notAnIssue) {
-            //TODO figure out which clause should work and remove the other 
one.
-            try {
-                context = (Context) context.lookup("java:comp/env/");
-            } catch (NamingException notAnIssue2) {
-            }
         }
         
         return context;

Modified: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/BaseContext.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/BaseContext.java?rev=951613&r1=951612&r2=951613&view=diff
==============================================================================
--- 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/BaseContext.java
 (original)
+++ 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/BaseContext.java
 Fri Jun  4 23:15:45 2010
@@ -119,7 +119,7 @@ public abstract class BaseContext implem
         DeploymentInfo deploymentInfo = threadContext.getDeploymentInfo();
         Context jndiEnc = deploymentInfo.getJndiEnc();
         try {
-            jndiEnc = (Context) jndiEnc.lookup("comp/env");
+            jndiEnc = (Context) jndiEnc.lookup("java:comp/env");
             return jndiEnc.lookup(name);
         } catch (NamingException e) {
             throw new IllegalArgumentException(e);

Modified: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/cmp/jpa/JpaCmpEngine.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/cmp/jpa/JpaCmpEngine.java?rev=951613&r1=951612&r2=951613&view=diff
==============================================================================
--- 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/cmp/jpa/JpaCmpEngine.java
 (original)
+++ 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/cmp/jpa/JpaCmpEngine.java
 Fri Jun  4 23:15:45 2010
@@ -91,7 +91,7 @@ public class JpaCmpEngine implements Cmp
     private EntityManager getEntityManager(CoreDeploymentInfo deploymentInfo) {
         EntityManager entityManager = null;
         try {
-            entityManager = (EntityManager) 
deploymentInfo.getJndiEnc().lookup("comp/env/" + 
CMP_PERSISTENCE_CONTEXT_REF_NAME);
+            entityManager = (EntityManager) 
deploymentInfo.getJndiEnc().lookup("java:comp/env/" + 
CMP_PERSISTENCE_CONTEXT_REF_NAME);
         } catch (NamingException ignored) {
             //TODO see OPENEJB-1259 temporary hack until geronimo jndi 
integration works better
             try {

Modified: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/managed/ManagedContainer.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/managed/ManagedContainer.java?rev=951613&r1=951612&r2=951613&view=diff
==============================================================================
--- 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/managed/ManagedContainer.java
 (original)
+++ 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/managed/ManagedContainer.java
 Fri Jun  4 23:15:45 2010
@@ -579,11 +579,11 @@ public class ManagedContainer implements
             SessionContext sessionContext;
             synchronized (this) {
                 try {
-                    sessionContext = (SessionContext) 
ctx.lookup("comp/EJBContext");
+                    sessionContext = (SessionContext) 
ctx.lookup("java:comp/EJBContext");
                 } catch (NamingException e1) {
                     ManagedUserTransaction userTransaction = new 
ManagedUserTransaction(new EjbUserTransaction(), entityManagerRegistry);
                     sessionContext = new ManagedContext(securityService, 
userTransaction);
-                    ctx.bind("comp/EJBContext", sessionContext);
+                    ctx.bind("java:comp/EJBContext", sessionContext);
                 }
             }
 

Modified: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/mdb/MdbInstanceFactory.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/mdb/MdbInstanceFactory.java?rev=951613&r1=951612&r2=951613&view=diff
==============================================================================
--- 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/mdb/MdbInstanceFactory.java
 (original)
+++ 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/mdb/MdbInstanceFactory.java
 Fri Jun  4 23:15:45 2010
@@ -182,10 +182,10 @@ public class MdbInstanceFactory {
             MdbContext mdbContext;
             synchronized(this) {
                 try {
-                    mdbContext = (MdbContext) ctx.lookup("comp/EJBContext");
+                    mdbContext = (MdbContext) 
ctx.lookup("java:comp/EJBContext");
                 } catch (NamingException e) {
                     mdbContext = new MdbContext(securityService);
-                    ctx.bind("comp/EJBContext",mdbContext);
+                    ctx.bind("java:comp/EJBContext",mdbContext);
                 }
             }
 
@@ -257,7 +257,7 @@ public class MdbInstanceFactory {
             if (!injection.getTarget().isAssignableFrom(clazz)) continue;
             try {
                 String jndiName = injection.getJndiName();
-                Object object = context.lookup("comp/env/" + jndiName);
+                Object object = context.lookup("java:comp/env/" + jndiName);
                 if (object instanceof String) {
                     String string = (String) object;
                     // Pass it in raw so it could be potentially converted to

Modified: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/singleton/SingletonInstanceManager.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/singleton/SingletonInstanceManager.java?rev=951613&r1=951612&r2=951613&view=diff
==============================================================================
--- 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/singleton/SingletonInstanceManager.java
 (original)
+++ 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/singleton/SingletonInstanceManager.java
 Fri Jun  4 23:15:45 2010
@@ -143,11 +143,11 @@ public class SingletonInstanceManager {
             // This is a fix for GERONIMO-3444
             synchronized(this){
                 try {
-                    sessionContext = (SessionContext) 
ctx.lookup("comp/EJBContext");
+                    sessionContext = (SessionContext) 
ctx.lookup("java:comp/EJBContext");
                 } catch (NamingException e1) {
                     sessionContext = createSessionContext();
                     // TODO: This should work
-                    ctx.bind("comp/EJBContext", sessionContext);
+                    ctx.bind("java:comp/EJBContext", sessionContext);
                 }
             }
 
@@ -160,11 +160,11 @@ public class SingletonInstanceManager {
             // This is a fix for GERONIMO-3444
             synchronized(this){
                 try {
-                    ctx.lookup("comp/WebServiceContext");
+                    ctx.lookup("java:comp/WebServiceContext");
                 } catch (NamingException e) {
                     WebServiceContext wsContext;
                     wsContext = new EjbWsContext(sessionContext);
-                    ctx.bind("comp/WebServiceContext", wsContext);
+                    ctx.bind("java:comp/WebServiceContext", wsContext);
                 }
             }
 
@@ -262,7 +262,7 @@ public class SingletonInstanceManager {
             if (!injection.getTarget().isAssignableFrom(clazz)) continue;
             try {
                 String jndiName = injection.getJndiName();
-                Object object = context.lookup("comp/env/" + jndiName);
+                Object object = context.lookup("java:comp/env/" + jndiName);
                 String prefix;
                 if (usePrefix) {
                     prefix = injection.getTarget().getName() + "/";

Modified: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateful/StatefulContainer.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateful/StatefulContainer.java?rev=951613&r1=951612&r2=951613&view=diff
==============================================================================
--- 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateful/StatefulContainer.java
 (original)
+++ 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateful/StatefulContainer.java
 Fri Jun  4 23:15:45 2010
@@ -590,11 +590,11 @@ public class StatefulContainer implement
             SessionContext sessionContext;
             synchronized (this) {
                 try {
-                    sessionContext = (SessionContext) 
ctx.lookup("comp/EJBContext");
+                    sessionContext = (SessionContext) 
ctx.lookup("java:comp/EJBContext");
                 } catch (NamingException e1) {
                     StatefulUserTransaction userTransaction = new 
StatefulUserTransaction(new EjbUserTransaction(), entityManagerRegistry);
                     sessionContext = new StatefulContext(securityService, 
userTransaction);
-                    ctx.bind("comp/EJBContext", sessionContext);
+                    ctx.bind("java:comp/EJBContext", sessionContext);
                 }
             }
 

Modified: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateless/StatelessInstanceManager.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateless/StatelessInstanceManager.java?rev=951613&r1=951612&r2=951613&view=diff
==============================================================================
--- 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateless/StatelessInstanceManager.java
 (original)
+++ 
openejb/branches/openejb-3.1.x/container/openejb-core/src/main/java/org/apache/openejb/core/stateless/StatelessInstanceManager.java
 Fri Jun  4 23:15:45 2010
@@ -178,11 +178,11 @@ public class StatelessInstanceManager {
             // This is a fix for GERONIMO-3444
             synchronized(this){
                 try {
-                    sessionContext = (SessionContext) 
ctx.lookup("comp/EJBContext");
+                    sessionContext = (SessionContext) 
ctx.lookup("java:comp/EJBContext");
                 } catch (NamingException e1) {
                     sessionContext = createSessionContext(deploymentInfo);
                     // TODO: This should work
-                    ctx.bind("comp/EJBContext", sessionContext);
+                    ctx.bind("java:comp/EJBContext", sessionContext);
                 }
             }
             if (SessionBean.class.isAssignableFrom(beanClass) || 
hasSetSessionContext(beanClass)) {
@@ -194,10 +194,10 @@ public class StatelessInstanceManager {
             // This is a fix for GERONIMO-3444
             synchronized(this){
                 try {
-                    ctx.lookup("comp/WebServiceContext");
+                    ctx.lookup("java:comp/WebServiceContext");
                 } catch (NamingException e) {
                     WebServiceContext wsContext = new 
EjbWsContext(sessionContext);
-                    ctx.bind("comp/WebServiceContext", wsContext);
+                    ctx.bind("java:comp/WebServiceContext", wsContext);
                 }
             }
 
@@ -284,7 +284,7 @@ public class StatelessInstanceManager {
             if (!injection.getTarget().isAssignableFrom(clazz)) continue;
             try {
                 String jndiName = injection.getJndiName();
-                Object object = context.lookup("comp/env/" + jndiName);
+                Object object = context.lookup("java:comp/env/" + jndiName);
                 String prefix;
                 if (usePrefix) {
                     prefix = injection.getTarget().getName() + "/";

Propchange: 
openejb/branches/openejb-3.1.x/container/openejb-core/src/test/java/org/apache/openejb/config/BusinessInterfacesTest.java
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Jun  4 23:15:45 2010
@@ -1,2 +1,2 @@
 
/openejb/branches/openejb-3.1.1/container/openejb-core/src/test/java/org/apache/openejb/config/UberInterfaceTest.java:779593
-/openejb/trunk/openejb3/container/openejb-core/src/test/java/org/apache/openejb/config/BusinessInterfacesTest.java:943472,943862,943965,944757,945989,946399,946485,946489,946705,946792,946814,946861,946863-946864,947010,947017,947042,948022,948241,948243,948548,949014,949233,950391,950801,951611
+/openejb/trunk/openejb3/container/openejb-core/src/test/java/org/apache/openejb/config/BusinessInterfacesTest.java:943472,943862,943965,944757,945989,946399,946485,946489,946705,946792,946814,946861,946863-946864,947010,947017,947042,948022,948241,948548,949014,949233,950391,950801,951611


Reply via email to