[jboss-cvs] JBossAS SVN: r73320 - trunk/ejb3/src/main/org/jboss/ejb3/deployers.

[jboss-cvs-commits at lists.jboss.org]

Author: scott.stark at jboss.org
Date: 2008-05-12 21:25:55 -0400 (Mon, 12 May 2008)
New Revision: 73320

Modified:
   trunk/ejb3/src/main/org/jboss/ejb3/deployers/Ejb3JBoss5Deployment.java
Log:
Move the warning about missing ContainerDependencyMetaData until where its actually needed.

Modified: trunk/ejb3/src/main/org/jboss/ejb3/deployers/Ejb3JBoss5Deployment.java
===================================================================
--- trunk/ejb3/src/main/org/jboss/ejb3/deployers/Ejb3JBoss5Deployment.java	2008-05-12 23:12:23 UTC (rev 73319)
+++ trunk/ejb3/src/main/org/jboss/ejb3/deployers/Ejb3JBoss5Deployment.java	2008-05-13 01:25:55 UTC (rev 73320)
@@ -67,8 +67,6 @@
       this.mbeanServer = mbeanServer;
       org.jboss.deployers.structure.spi.DeploymentUnit topUnit = jbossUnit.getTopLevel();
       endpoints = (Map<String, ContainerDependencyMetaData>) topUnit.getAttachment(MappedReferenceMetaDataResolverDeployer.ENDPOINT_MAP_KEY);
-      if(endpoints == null)
-         log.warn(jbossUnit+" has no ContainerDependencyMetaData attachment");
    }
 
    protected PolicyConfiguration createPolicyConfiguration() throws Exception
@@ -94,13 +92,17 @@
       EJBContainer ejbContainer = (EJBContainer) container;
       JBossEnterpriseBeanMetaData beanMD = ejbContainer.getXml();
 
-      EjbDeploymentSummary defaultsInfo = getUnitSummary(this.getDeploymentUnit(), beanMD);
       ContainerDependencyMetaData cdmd = null;
       if(endpoints != null)
       {
          String ejbKey = "ejb/" + jbossUnit.getRelativePath() + "#" + container.getEjbName();
          cdmd = endpoints.get(ejbKey);
       }
+      else
+      {
+         log.warn(jbossUnit+" has no ContainerDependencyMetaData attachment");
+      }
+
       if(cdmd != null)
       {
          for(String jndiName : cdmd.getJndiNames())