From b0a00d248d3f6ae91627490e7a177d9ad5184331 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Mon, 16 Feb 2015 15:59:49 +0000
Subject: [PATCH] AutoRefactor: booleans

---
 opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/NodeRefresher.java |   19 ++++---------------
 1 files changed, 4 insertions(+), 15 deletions(-)

diff --git a/opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/NodeRefresher.java b/opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/NodeRefresher.java
index 674288f..b2ffd1e 100644
--- a/opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/NodeRefresher.java
+++ b/opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/browser/NodeRefresher.java
@@ -761,31 +761,20 @@
   // is listed in in the hacker.
   // Note: *usable* means *usable for detecting children presence*.
   private boolean isNumSubOrdinatesUsable() throws NamingException {
-    boolean result;
     SearchResult entry = getDisplayedEntry();
     boolean hasSubOrdinates = BrowserController.getHasSubOrdinates(entry);
     if (!hasSubOrdinates) { // We must check
       LDAPURL url = getDisplayedUrl();
-      if (controller.getNumSubordinateHacker().contains(url)) {
-        // The numSubOrdinate we have is unreliable.
-        result = false;
-//        System.out.println("numSubOrdinates of " + url +
-//                           " is not reliable");
-      }
-      else {
-        result = true;
-      }
+      return !controller.getNumSubordinateHacker().contains(url);
     }
-    else { // Other values are usable
-      result = true;
-    }
-    return result;
+    // Other values are usable
+    return true;
   }
 
 
 
   /**
-   * Searchs for the children.
+   * Searches for the children.
    * @throws SearchAbandonException if an error occurs.
    */
   private void runSearchChildren() throws SearchAbandonException {

--
Gitblit v1.10.0