[jboss-cvs] JBossAS SVN: r76553 - in projects/metadata/trunk/src: test/java/org/jboss/test/metadata/ejb and 1 other directory.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Fri Aug 1 01:52:44 EDT 2008


Author: ALRubinger
Date: 2008-08-01 01:52:44 -0400 (Fri, 01 Aug 2008)
New Revision: 76553

Modified:
   projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/jndipolicy/plugins/BasicJndiBindingPolicy.java
   projects/metadata/trunk/src/test/java/org/jboss/test/metadata/ejb/ResolveJndiNameDecoratorUnitTestCase.java
Log:
[JBMETA-86] Avoid JNDI Collisions in the case jndiName is equal to deploymentScopeBaseName in BasicJndiBindingPolicy

Modified: projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/jndipolicy/plugins/BasicJndiBindingPolicy.java
===================================================================
--- projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/jndipolicy/plugins/BasicJndiBindingPolicy.java	2008-08-01 05:47:00 UTC (rev 76552)
+++ projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/jndipolicy/plugins/BasicJndiBindingPolicy.java	2008-08-01 05:52:44 UTC (rev 76553)
@@ -279,13 +279,22 @@
          
          /*
           * Adjust deploymentScopeBaseName in the case it's equal 
-          * to mappedName 
+          * to mappedName or jndiName
           * 
           * JBMETA-83
+          * JBMETA-86
           */
          String mappedName = summary.getBeanMD().getMappedName();
-         if(deploymentScopeBaseName.equals(mappedName))
+         if (md.isSession())
          {
+            if (mappedName == null || mappedName.trim().length() == 0)
+            {
+               JBossSessionBeanMetaData smd = (JBossSessionBeanMetaData) md;
+               mappedName = smd.getJndiName();
+            }
+         }
+         if (deploymentScopeBaseName.equals(mappedName))
+         {
             log.warn("The EAR name, which is used as a base, is equal to the mappedName for EJB \""
                   + summary.getBeanMD().getEjbName()
                   + "\", to avoid JNDI Naming Conflict appending a '_' to the base JNDI name. [JBMETA-83]");

Modified: projects/metadata/trunk/src/test/java/org/jboss/test/metadata/ejb/ResolveJndiNameDecoratorUnitTestCase.java
===================================================================
--- projects/metadata/trunk/src/test/java/org/jboss/test/metadata/ejb/ResolveJndiNameDecoratorUnitTestCase.java	2008-08-01 05:47:00 UTC (rev 76552)
+++ projects/metadata/trunk/src/test/java/org/jboss/test/metadata/ejb/ResolveJndiNameDecoratorUnitTestCase.java	2008-08-01 05:52:44 UTC (rev 76553)
@@ -126,7 +126,7 @@
    }
    
    /**
-    * JBMETA-83
+    * JBMETA-86
     * 
     * When deploymentScopeBaseName is equal to the jndiName, 
     * the base JNDI name becomes bound and is therefore an




More information about the jboss-cvs-commits mailing list