[embjopr-commits] EMBJOPR SVN: r243 - trunk/core/src/main/java/org/jboss/on/embedded/ui.

[embjopr-commits at lists.jboss.org]

Author: ips
Date: 2009-03-19 22:44:02 -0400 (Thu, 19 Mar 2009)
New Revision: 243

Modified:
   trunk/core/src/main/java/org/jboss/on/embedded/ui/NavigationAction.java
Log:
turn down some noisy logging; remove redundant imports


Modified: trunk/core/src/main/java/org/jboss/on/embedded/ui/NavigationAction.java
===================================================================
--- trunk/core/src/main/java/org/jboss/on/embedded/ui/NavigationAction.java	2009-03-18 16:26:04 UTC (rev 242)
+++ trunk/core/src/main/java/org/jboss/on/embedded/ui/NavigationAction.java	2009-03-20 02:44:02 UTC (rev 243)
@@ -30,6 +30,19 @@
 
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
+import org.rhq.core.domain.configuration.definition.ConfigurationDefinition;
+import org.rhq.core.domain.measurement.Availability;
+import org.rhq.core.domain.measurement.AvailabilityType;
+import org.rhq.core.domain.resource.Resource;
+import org.rhq.core.domain.resource.ResourceCreationDataType;
+import org.rhq.core.domain.resource.ResourceType;
+import org.richfaces.component.UITree;
+
+import org.jboss.seam.ScopeType;
+import org.jboss.seam.annotations.In;
+import org.jboss.seam.annotations.Name;
+import org.jboss.seam.annotations.Scope;
+
 import org.jboss.on.embedded.Constants;
 import org.jboss.on.embedded.manager.ResourceManager;
 import org.jboss.on.embedded.manager.ResourceManagerFactory;
@@ -39,22 +52,6 @@
 import org.jboss.on.embedded.ui.nav.PlatformResourceTreeNode;
 import org.jboss.on.embedded.ui.nav.ResourceTypeTreeNode;
 import org.jboss.on.embedded.ui.nav.TreeNodeWithResource;
-import org.jboss.seam.ScopeType;
-import org.jboss.seam.core.Events;
-import org.jboss.seam.annotations.Factory;
-import org.jboss.seam.annotations.In;
-import org.jboss.seam.annotations.Name;
-import org.jboss.seam.annotations.Observer;
-import org.jboss.seam.annotations.Out;
-import org.jboss.seam.annotations.Scope;
-import org.rhq.core.domain.configuration.definition.ConfigurationDefinition;
-import org.rhq.core.domain.measurement.Availability;
-import org.rhq.core.domain.measurement.AvailabilityType;
-import org.rhq.core.domain.resource.Resource;
-import org.rhq.core.domain.resource.ResourceCreationDataType;
-import org.rhq.core.domain.resource.ResourceType;
-import org.richfaces.component.UITree;
-import org.richfaces.component.html.HtmlTree;
 
 /**
  * NavigationAction creates the navigation objects. 
@@ -63,6 +60,7 @@
  * @author Charles Crouch
  * @author Ian Springer
  */
+// TODO (ips): Remove obsolete commented-out code.
 // TODO: Go through this class and try to ensure the minimum amount of things are public.
 //       Most actions should be dealing with CommonActionUtil for getting hold of resources etc.
 @Name("navigationAction")
@@ -387,7 +385,8 @@
     // TODO could consider renaming this getCurrentNavigationNode()
     public JONTreeNode getSelectedNode()
     {
-        log.debug("Finding node for path [" + getCurrentPath() + "]...");
+        if (log.isTraceEnabled())
+            log.trace("Finding node for path [" + getCurrentPath() + "]...");
         JONTreeNode selectedNode = getRootNode().findNode(getCurrentPath());
         ResourceManager resourceManager = ResourceManagerFactory.resourceManager();
         if (selectedNode == null) {