Author: dblevins
Date: Tue May 26 01:22:33 2009
New Revision: 778545

URL: http://svn.apache.org/viewvc?rev=778545&view=rev
Log:
Small tweak to prevent logging some Jndi names twice

Modified:
    
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/JndiBuilder.java

Modified: 
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/JndiBuilder.java
URL: 
http://svn.apache.org/viewvc/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/JndiBuilder.java?rev=778545&r1=778544&r2=778545&view=diff
==============================================================================
--- 
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/JndiBuilder.java
 (original)
+++ 
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/JndiBuilder.java
 Tue May 26 01:22:33 2009
@@ -401,14 +401,16 @@
                 context.bind(name, ref);
                 bindings.add(name);
 
-                beanInfo.jndiNames.add(externalName);
+                if (!beanInfo.jndiNames.contains(externalName)) {
+                    beanInfo.jndiNames.add(externalName);
 
-                JndiNameInfo nameInfo = new JndiNameInfo();
-                nameInfo.intrface = intrface.getName();
-                nameInfo.name = externalName;
-                beanInfo.jndiNamess.add(nameInfo);
+                    JndiNameInfo nameInfo = new JndiNameInfo();
+                    nameInfo.intrface = intrface.getName();
+                    nameInfo.name = externalName;
+                    beanInfo.jndiNamess.add(nameInfo);
 
-                logger.info("Jndi(name=" + externalName +") --> 
Ejb(deployment-id="+beanInfo.ejbDeploymentId+")");
+                    logger.info("Jndi(name=" + externalName +") --> 
Ejb(deployment-id="+beanInfo.ejbDeploymentId+")");
+                }
             } catch (NameAlreadyBoundException e) {
                 DeploymentInfo deployment = findNameOwner(name);
                 if (deployment != null){


Reply via email to