[jboss-cvs] JBossAS SVN: r79355 - projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/session.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Fri Oct 10 18:51:20 EDT 2008


Author: ALRubinger
Date: 2008-10-10 18:51:20 -0400 (Fri, 10 Oct 2008)
New Revision: 79355

Modified:
   projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/session/SessionSpecContainer.java
Log:
[EJBTHREE-1524] Remove (now unnecessary and outdated) hack to add binding metadata, obbsoleted by the completed JBoss50Creator

Modified: projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/session/SessionSpecContainer.java
===================================================================
--- projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/session/SessionSpecContainer.java	2008-10-10 19:47:10 UTC (rev 79354)
+++ projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/session/SessionSpecContainer.java	2008-10-10 22:51:20 UTC (rev 79355)
@@ -623,77 +623,6 @@
 
       super.lockedStart();
 
-      //TODO
-      /*
-       * Temporary Hack Alert
-       * 
-       * Populate JBoss-specific metadata until this is done by
-       * AnnotationMetaDataDeployer and MergedJBossMetaDataDeployer
-       * 
-       * http://jira.jboss.com/jira/browse/JBMETA-45
-       * http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4157770
-       */
-      log.warn("Populating JBoss-specific annotation metadata manually until done by deployers: " + this);
-
-      // Obtain annotations 
-      RemoteBindings remoteBindings = this.getAnnotation(RemoteBindings.class);
-      RemoteBinding remoteBinding = this.getAnnotation(RemoteBinding.class);
-      RemoteHomeBinding remoteHomeBinding = this.getAnnotation(RemoteHomeBinding.class);
-      LocalHomeBinding localHomeBinding = this.getAnnotation(LocalHomeBinding.class);
-      //      LocalBinding localBinding = this.getAnnotation(LocalBinding.class); // < No LocalBindingMetaData?
-
-      // Create a Set to hold RemoteBindings
-      Set<RemoteBinding> remoteBindingsSet = new HashSet<RemoteBinding>();
-
-      // Populate Set with Remote Bindings
-      if (remoteBindings != null)
-      {
-         for (RemoteBinding binding : remoteBindings.value())
-         {
-            remoteBindingsSet.add(binding);
-         }
-      }
-      if (remoteBinding != null)
-      {
-         remoteBindingsSet.add(remoteBinding);
-      }
-
-      // Ensure remote bindings metadata is not null
-      List<RemoteBindingMetaData> rbmd = this.getMetaData().getRemoteBindings();
-      if (rbmd == null || rbmd.size() == 0)
-      {
-         rbmd = new ArrayList<RemoteBindingMetaData>();
-         this.getMetaData().setRemoteBindings(rbmd);
-      }
-
-      // For each remote binding, populate metadata
-      for (RemoteBinding binding : remoteBindingsSet)
-      {
-         RemoteBindingMetaData md = new RemoteBindingMetaData();
-         md.setClientBindUrl(binding.clientBindUrl());
-         md.setInterceptorStack(binding.interceptorStack());
-         md.setJndiName(binding.jndiBinding());
-         md.setProxyFactory(binding.factory());
-         //TODO binding.invokerName?
-         rbmd.add(md);
-      }
-
-      // Populate metadata for @RemoteHomeBinding
-      if (remoteHomeBinding != null)
-      {
-         this.getMetaData().setHomeJndiName(remoteHomeBinding.jndiBinding());
-      }
-
-      // Populate metadata for @LocalHomeBinding
-      if (localHomeBinding != null)
-      {
-         this.getMetaData().setLocalHomeJndiName(localHomeBinding.jndiBinding());
-      }
-
-      /*
-       * End Temporary Hack
-       */
-
       // Obtain registrar
       JndiSessionRegistrarBase registrar = this.getJndiRegistrar();
 




More information about the jboss-cvs-commits mailing list