Author: thomas.heute(a)jboss.com Date: 2008-04-03 05:16:19 -0400 (Thu, 03 Apr 2008) New Revision: 10490 Modified: modules/identity/branches/JBP_IDENTITY_BRANCH_1_0/identity/src/main/org/jboss/portal/identity/boot/IdentityServiceLoader.java Log: JBPORTAL-1730: IdentityServiceControllerImpl : unregister existing MBean service before to re register them (Applied to Branch_1_0) Modified: modules/identity/branches/JBP_IDENTITY_BRANCH_1_0/identity/src/main/org/jboss/portal/identity/boot/IdentityServiceLoader.java =================================================================== --- modules/identity/branches/JBP_IDENTITY_BRANCH_1_0/identity/src/main/org/jboss/portal/identity/boot/IdentityServiceLoader.java 2008-04-03 09:01:50 UTC (rev 10489) +++ modules/identity/branches/JBP_IDENTITY_BRANCH_1_0/identity/src/main/org/jboss/portal/identity/boot/IdentityServiceLoader.java 2008-04-03 09:16:19 UTC (rev 10490) @@ -97,9 +97,8 @@ //register as an mbean if (isRegisterMBeans() && (ds.getServiceName() != null)) { - + unregisterMBean(ds.getServiceName()); registerMBean(ds.getServiceName(), datasource); - } @@ -149,12 +148,23 @@ //register as an mbean if (isRegisterMBeans() && (module.getServiceName() != null)) { + unregisterMBean(module.getServiceName()); registerMBean(module.getServiceName(), moduleServiceObject); } } } + + /** + * Should be extended to provide mbean registration + * @param serviceName + * @param serviceObject + */ + protected void unregisterMBean(String serviceName) throws Exception + { + //does nothing + } /** * Should be extended to provide mbean registration