Author: rob.stryker(a)jboss.com Date: 2007-11-11 14:54:41 -0500 (Sun, 11 Nov 2007) New Revision: 4836 Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/extensions/jmx/JMXModel.java Log: JBIDE-1293 JMXSafeRunner now more safe (and won't log an error unless the server is IServer.STATE_STARTED) - edit, had != instead of ==. Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/extensions/jmx/JMXModel.java =================================================================== --- trunk/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/extensions/jmx/JMXModel.java 2007-11-11 19:33:24 UTC (rev 4835) +++ trunk/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/extensions/jmx/JMXModel.java 2007-11-11 19:54:41 UTC (rev 4836) @@ -447,9 +447,8 @@ r.run(connection); } } catch (Exception e) { - // if the server isn't started (or began shutting - // down during the op),don't log the error. - if( s.getServerState() != IServer.STATE_STARTED ) { + // only if the server *IS* started should we log the error + if( s.getServerState() == IServer.STATE_STARTED ) { JBossServerCorePlugin.getDefault().getLog().log( new Status(IStatus.ERROR, JBossServerCorePlugin.PLUGIN_ID, "Error while running JMX-safe code", e));