From 06ec8c88556b02782c7b91a233de91eaf4a1439d Mon Sep 17 00:00:00 2001
From: jvergara <jvergara@localhost>
Date: Tue, 09 Jun 2009 12:28:05 +0000
Subject: [PATCH] Fix for issue 3550 (Consider making the control panel able to manage remote servers)

---
 opends/src/guitools/org/opends/guitools/controlpanel/browser/BrowserController.java |   13 ++++++++++---
 1 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/opends/src/guitools/org/opends/guitools/controlpanel/browser/BrowserController.java b/opends/src/guitools/org/opends/guitools/controlpanel/browser/BrowserController.java
index f0c536b..cec350c 100644
--- a/opends/src/guitools/org/opends/guitools/controlpanel/browser/BrowserController.java
+++ b/opends/src/guitools/org/opends/guitools/controlpanel/browser/BrowserController.java
@@ -36,6 +36,8 @@
 import java.util.Set;
 import java.util.SortedSet;
 import java.util.TreeSet;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import javax.naming.NameNotFoundException;
 import javax.naming.NamingException;
@@ -133,6 +135,9 @@
 
   private String filter;
 
+  private static final Logger LOG =
+    Logger.getLogger(BrowserController.class.getName());
+
   /**
    * Constructor of the BrowserController.
    * @param tree the tree that must be updated.
@@ -1540,8 +1545,9 @@
         try {
           refreshTaskDidProgress(task, oldState, newState);
         }
-        catch(Exception x) {
-          x.printStackTrace();
+        catch(Throwable t)
+        {
+          LOG.log(Level.SEVERE, "Error calling refreshTaskDidProgress: "+t, t);
         }
       }
     };
@@ -1735,7 +1741,8 @@
     if (node.getError() != null) {
       if (node.getError().getException() != null)
       {
-        node.getError().getException().printStackTrace();
+        LOG.log(Level.SEVERE, "node has error: "+node.getError().getException(),
+            node.getError().getException());
       }
       modifiers |= IconPool.MODIFIER_ERROR;
     }

--
Gitblit v1.10.0