From 9748e704763be89c57cad0a65fe74dce65fbcf84 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Fri, 07 Nov 2014 10:44:24 +0000
Subject: [PATCH] Code cleanup for the tools

---
 opendj3-server-dev/src/guitools/org/opends/guitools/controlpanel/util/ConfigReader.java |   21 +++++++++------------
 1 files changed, 9 insertions(+), 12 deletions(-)

diff --git a/opendj3-server-dev/src/guitools/org/opends/guitools/controlpanel/util/ConfigReader.java b/opendj3-server-dev/src/guitools/org/opends/guitools/controlpanel/util/ConfigReader.java
index b1504c7..b55bbeb 100644
--- a/opendj3-server-dev/src/guitools/org/opends/guitools/controlpanel/util/ConfigReader.java
+++ b/opendj3-server-dev/src/guitools/org/opends/guitools/controlpanel/util/ConfigReader.java
@@ -24,11 +24,8 @@
  *      Copyright 2008-2009 Sun Microsystems, Inc.
  *      Portions Copyright 2014 ForgeRock AS
  */
-
 package org.opends.guitools.controlpanel.util;
 
-import static org.opends.messages.AdminToolMessages.*;
-
 import java.io.File;
 import java.net.InetAddress;
 import java.util.ArrayList;
@@ -39,13 +36,12 @@
 
 import org.forgerock.i18n.LocalizableMessage;
 import org.forgerock.i18n.slf4j.LocalizedLogger;
-
+import org.forgerock.opendj.config.server.ConfigException;
 import org.opends.guitools.controlpanel.datamodel.BackendDescriptor;
 import org.opends.guitools.controlpanel.datamodel.ConnectionHandlerDescriptor;
 import org.opends.guitools.controlpanel.datamodel.VLVSortOrder;
 import org.opends.guitools.controlpanel.task.OfflineUpdateException;
 import org.opends.server.admin.std.meta.AdministrationConnectorCfgDefn;
-import org.forgerock.opendj.config.server.ConfigException;
 import org.opends.server.core.DirectoryServer;
 import org.opends.server.tools.tasks.TaskEntry;
 import org.opends.server.types.DN;
@@ -55,10 +51,11 @@
 import org.opends.server.types.OpenDsException;
 import org.opends.server.types.Schema;
 
+import static org.opends.messages.AdminToolMessages.*;
+
 /**
  * An abstract class providing some common interface for the class that read
  * the configuration (and if the server is running, the monitoring information).
- *
  */
 public abstract class ConfigReader
 {
@@ -322,19 +319,19 @@
     if (s != null)
     {
       String[] attrNames = s.split(" ");
-      for (int i=0; i<attrNames.length; i++)
+      for (String attrName : attrNames)
       {
-        if (attrNames[i].startsWith("+"))
+        if (attrName.startsWith("+"))
         {
-          sortOrder.add(new VLVSortOrder(attrNames[i].substring(1), true));
+          sortOrder.add(new VLVSortOrder(attrName.substring(1), true));
         }
-        else if (attrNames[i].startsWith("-"))
+        else if (attrName.startsWith("-"))
         {
-          sortOrder.add(new VLVSortOrder(attrNames[i].substring(1), false));
+          sortOrder.add(new VLVSortOrder(attrName.substring(1), false));
         }
         else
         {
-          sortOrder.add(new VLVSortOrder(attrNames[i], true));
+          sortOrder.add(new VLVSortOrder(attrName, true));
         }
       }
     }

--
Gitblit v1.10.0