Don't add HostConfig as a listener twice. It breaks the Manager app and probably...
authormarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Sun, 23 May 2010 11:32:12 +0000 (11:32 +0000)
committermarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Sun, 23 May 2010 11:32:12 +0000 (11:32 +0000)
git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@947397 13f79535-47bb-0310-9956-ffa450edef68

java/org/apache/catalina/core/StandardHost.java
test/org/apache/catalina/mbeans/TestRegistration.java

index 6e34295..1973492 100644 (file)
@@ -33,7 +33,6 @@ import org.apache.catalina.LifecycleListener;
 import org.apache.catalina.Valve;
 import org.apache.catalina.loader.WebappClassLoader;
 import org.apache.catalina.mbeans.MBeanUtils;
-import org.apache.catalina.startup.HostConfig;
 import org.apache.catalina.util.LifecycleBase;
 import org.apache.catalina.valves.ValveBase;
 
@@ -802,16 +801,6 @@ public class StandardHost extends ContainerBase implements Host {
     }
 
     @Override
-    protected void initInternal() throws LifecycleException {
-
-        super.initInternal();
-
-        HostConfig deployer = new HostConfig();
-        addLifecycleListener(deployer);                
-    }
-
-    
-    @Override
     protected String getObjectNameKeyProperties() {
 
         StringBuilder keyProperties = new StringBuilder("type=Host");
index b2c031a..99fcb6f 100644 (file)
@@ -56,7 +56,7 @@ public class TestRegistration extends TomcatBaseTest {
 
         // Verify there are some Tomcat MBeans
         onames = mbeanServer.queryNames(new ObjectName("Tomcat:*"), null);
-        assertTrue("Not enough Tomcat MBeans", onames.size() >= 21);
+        assertTrue("Not enough Tomcat MBeans", onames.size() >= 20);
 
         tomcat.stop();