From bbba2cdd721035dc2920456dd3a715fc6bc33255 Mon Sep 17 00:00:00 2001
From: Juan David Arboleda <juan.arboleda@globalqss.com>
Date: Mon, 17 Dec 2012 18:04:33 -0500
Subject: [PATCH] IDEMPIERE-364 Improve Grid Layout / Refactor customizegrid

---
 .../org/adempiere/webui/panel/CustomizeGridViewPanel.java   | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/panel/CustomizeGridViewPanel.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/panel/CustomizeGridViewPanel.java
index 40cc1ebc76..adf062fdd3 100644
--- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/panel/CustomizeGridViewPanel.java
+++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/panel/CustomizeGridViewPanel.java
@@ -408,13 +408,11 @@ public class CustomizeGridViewPanel extends Panel
 	 */
 	void migrateValueWithinYesList (int endIndex, List<ListElement> selObjects)
 	{
-		int length = selObjects.size();
 		int iniIndex =0;
 		Arrays.sort(selObjects.toArray());	
 		ListElement endObject = (ListElement)yesModel.getElementAt(endIndex);
-
-		for (int i = 0; i < length; i++) {
-			iniIndex = yesModel.indexOf(selObjects.get(i));
+		for (ListElement selected : selObjects) {
+			iniIndex = yesModel.indexOf(selected);
 			ListElement selObject = (ListElement)yesModel.getElementAt(iniIndex);
 			yesModel.removeElement(selObject);
 			endIndex = yesModel.indexOf(endObject);