From 7028d9f1483d6f1e77bb0f5ebd0ecc6239e431c5 Mon Sep 17 00:00:00 2001
From: Jean-Noël Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Mon, 14 Nov 2016 15:38:45 +0000
Subject: [PATCH] AutoRefactor'ed use Map.entrySet() instead of Map.keySet() in a loop

---
 opendj-server-legacy/src/main/java/org/opends/server/tools/makeldif/TemplateEntry.java |   16 ++++++++++------
 1 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/opendj-server-legacy/src/main/java/org/opends/server/tools/makeldif/TemplateEntry.java b/opendj-server-legacy/src/main/java/org/opends/server/tools/makeldif/TemplateEntry.java
index e8941b9..464103c 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/tools/makeldif/TemplateEntry.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/tools/makeldif/TemplateEntry.java
@@ -23,6 +23,7 @@
 import java.util.ArrayList;
 import java.util.LinkedHashMap;
 import java.util.List;
+import java.util.Map;
 
 import org.forgerock.opendj.ldap.AVA;
 import org.forgerock.opendj.ldap.ByteString;
@@ -240,9 +241,10 @@
     LinkedHashMap<AttributeType, List<Attribute>> urlAttributes = new LinkedHashMap<>();
     LinkedHashMap<AttributeType, List<Attribute>> base64Attributes = new LinkedHashMap<>();
 
-    for (AttributeType t : attributes.keySet())
+    for (Map.Entry<AttributeType, ArrayList<TemplateValue>> entry : attributes.entrySet())
     {
-      ArrayList<TemplateValue> valueList = attributes.get(t);
+      AttributeType t = entry.getKey();
+      ArrayList<TemplateValue> valueList = entry.getValue();
       if (t.isObjectClass())
       {
         for (TemplateValue v : valueList)
@@ -334,11 +336,12 @@
 
 
     // Now the set of user attributes.
-    for (AttributeType attrType : userAttributes.keySet())
+    for (Map.Entry<AttributeType, List<Attribute>> entry : userAttributes.entrySet())
     {
+      AttributeType attrType = entry.getKey();
       if (exportConfig.includeAttribute(attrType))
       {
-        for (Attribute a : userAttributes.get(attrType))
+        for (Attribute a : entry.getValue())
         {
           if (a.isVirtual() && !exportConfig.includeVirtualAttributes())
           {
@@ -378,11 +381,12 @@
     // Next, the set of operational attributes.
     if (exportConfig.includeOperationalAttributes())
     {
-      for (AttributeType attrType : operationalAttributes.keySet())
+      for (Map.Entry<AttributeType, List<Attribute>> entry : operationalAttributes.entrySet())
       {
+        AttributeType attrType = entry.getKey();
         if (exportConfig.includeAttribute(attrType))
         {
-          for (Attribute a : operationalAttributes.get(attrType))
+          for (Attribute a : entry.getValue())
           {
             if (a.isVirtual() && !exportConfig.includeVirtualAttributes())
             {

--
Gitblit v1.10.0