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

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

Author: ALRubinger
Date: 2009-01-14 22:03:03 -0500 (Wed, 14 Jan 2009)
New Revision: 82890

Modified:
   projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/service/ServiceContainer.java
Log:
[EJBTHREE-1530][EJBTHREE-655] Hack in some backwards-compatible changes so that no modifications are required of JBossASKernel in AS, keeping Plugin backwards-compatible with AS 5.0.0.GA

Modified: projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/service/ServiceContainer.java
===================================================================
--- projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/service/ServiceContainer.java	2009-01-15 01:01:46 UTC (rev 82889)
+++ projects/ejb3/trunk/core/src/main/java/org/jboss/ejb3/service/ServiceContainer.java	2009-01-15 03:03:03 UTC (rev 82890)
@@ -754,7 +754,32 @@
 
             delegate = new ServiceMBeanDelegate(mbeanServer, this, intf, delegateObjectName);
             
-            getDeployment().getKernelAbstraction().installMBean(delegateObjectName, newPolicy, delegate);
+            /*
+             * 
+             * This section is in place to replace the KernelAbstraction.installMBean
+             * method which will be removed JBossASKernel (AS/trunk/ejb3) for 5.0.1.
+             * 
+             * Here to be backwards-compatible with JBossAS 5.0.0.GA
+             * 
+             * http://www.jboss.com/index.html?module=bb&op=viewtopic&t=148497
+             * 
+             */
+            
+            // The old/deprecated access
+            //getDeployment().getKernelAbstraction().installMBean(delegateObjectName, newPolicy, delegate);
+            
+            // Register w/ MBean Server
+            mbeanServer.registerMBean(delegate, delegateObjectName);
+            
+            // Install into MC
+            getDeployment().getKernelAbstraction().install(delegateObjectName.getCanonicalName(), newPolicy, null, delegate);
+
+            /*
+             * 
+             * End backwards-compatible replacement for:
+             * getDeployment().getKernelAbstraction().installMBean
+             * 
+             */
          }
          // XMBeans
          else
@@ -784,7 +809,28 @@
    {
       if (delegate != null)
       {
-         getDeployment().getKernelAbstraction().uninstallMBean(delegateObjectName);
+         /*
+          * 
+          * This section is in place to replace the KernelAbstraction.uninstallMBean
+          * method which will be removed JBossASKernel (AS/trunk/ejb3) for 5.0.1.
+          * 
+          * Here to be backwards-compatible with JBossAS 5.0.0.GA
+          * 
+          * http://www.jboss.com/index.html?module=bb&op=viewtopic&t=148497
+          * 
+          */
+         
+         //getDeployment().getKernelAbstraction().uninstallMBean(delegateObjectName);
+         
+         mbeanServer.unregisterMBean(delegateObjectName);
+         
+         
+         /*
+          * 
+          * End backwards-compatible replacement for:
+          * getDeployment().getKernelAbstraction().uninstallMBean
+          * 
+          */
       }
    }