Author: markt
Date: Wed Feb  8 21:54:07 2012
New Revision: 1242125

URL: http://svn.apache.org/viewvc?rev=1242125&view=rev
Log:
Remove dependency on ValveBase

Modified:
    tomcat/trunk/java/org/apache/catalina/core/StandardHost.java

Modified: tomcat/trunk/java/org/apache/catalina/core/StandardHost.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardHost.java?rev=1242125&r1=1242124&r2=1242125&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/core/StandardHost.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardHost.java Wed Feb  8 
21:54:07 2012
@@ -26,16 +26,18 @@ import java.util.WeakHashMap;
 import java.util.concurrent.ExecutorService;
 import java.util.regex.Pattern;
 
+import javax.management.ObjectName;
+
 import org.apache.catalina.Container;
 import org.apache.catalina.Context;
 import org.apache.catalina.Host;
+import org.apache.catalina.JmxEnabled;
 import org.apache.catalina.Lifecycle;
 import org.apache.catalina.LifecycleEvent;
 import org.apache.catalina.LifecycleException;
 import org.apache.catalina.LifecycleListener;
 import org.apache.catalina.Valve;
 import org.apache.catalina.loader.WebappClassLoader;
-import org.apache.catalina.valves.ValveBase;
 import org.apache.tomcat.util.ExceptionUtils;
 
 /**
@@ -799,15 +801,16 @@ public class StandardHost extends Contai
       *
       * @exception Exception if an MBean cannot be created or registered
       */
-     public String [] getValveNames()
-         throws Exception
-    {
+     public String [] getValveNames() throws Exception {
          Valve [] valves = this.getPipeline().getValves();
          String [] mbeanNames = new String[valves.length];
          for (int i = 0; i < valves.length; i++) {
-             if( valves[i] == null ) continue;
-             if( ((ValveBase)valves[i]).getObjectName() == null ) continue;
-             mbeanNames[i] = ((ValveBase)valves[i]).getObjectName().toString();
+             if (valves[i] instanceof JmxEnabled) {
+                 ObjectName oname = ((JmxEnabled) valves[i]).getObjectName();
+                 if (oname != null) {
+                     mbeanNames[i] = oname.toString();
+                 }
+             }
          }
 
          return mbeanNames;



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to