Author: felixk
Date: Fri Sep 23 09:25:52 2011
New Revision: 1174607

URL: http://svn.apache.org/viewvc?rev=1174607&view=rev
Log:
Use interface instaed of specific implementation (HUPA-77)

Modified:
    
james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
    
james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainView.java

Modified: 
james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
URL: 
http://svn.apache.org/viewvc/james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java?rev=1174607&r1=1174606&r2=1174607&view=diff
==============================================================================
--- 
james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
 (original)
+++ 
james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
 Fri Sep 23 09:25:52 2011
@@ -22,6 +22,7 @@ package org.apache.hupa.client.mvp;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Comparator;
+import java.util.List;
 
 import net.customware.gwt.presenter.client.EventBus;
 import net.customware.gwt.presenter.client.widget.WidgetContainerDisplay;
@@ -95,7 +96,7 @@ public class MainPresenter extends Widge
       
         public HasSelectionHandlers<TreeItem> getTree();
 
-        public void bindTreeItems(ArrayList<IMAPTreeItem> treeList);
+        public void bindTreeItems(List<IMAPTreeItem> treeList);
 
         public HasClickHandlers getRenameClick();
 
@@ -169,8 +170,8 @@ public class MainPresenter extends Widge
      * @param list
      * @return
      */
-    private ArrayList<IMAPTreeItem> createTreeNodes(ArrayList<IMAPFolder> 
list) {
-        ArrayList<IMAPTreeItem> tList = new ArrayList<IMAPTreeItem>();
+    private List<IMAPTreeItem> createTreeNodes(List<IMAPFolder> list) {
+        List<IMAPTreeItem> tList = new ArrayList<IMAPTreeItem>();
 
         for (IMAPFolder iFolder : list) {
 
@@ -197,12 +198,10 @@ public class MainPresenter extends Widge
             });
             record.setUserObject(iFolder);
 
-            ArrayList<IMAPFolder> childFolders = iFolder.getChildIMAPFolders();
-            if (childFolders != null && childFolders.isEmpty() == false) {
-                ArrayList<IMAPTreeItem> items = createTreeNodes(childFolders);
-                for (IMAPTreeItem item : items) {
-                    record.addItem(item);
-                }
+            List<IMAPFolder> childFolders = iFolder.getChildIMAPFolders();
+            List<IMAPTreeItem> items = createTreeNodes(childFolders);
+            for (IMAPTreeItem item : items) {
+                record.addItem(item);
             }
 
             // Store the INBOX as starting point after first loading

Modified: 
james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainView.java
URL: 
http://svn.apache.org/viewvc/james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainView.java?rev=1174607&r1=1174606&r2=1174607&view=diff
==============================================================================
--- 
james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainView.java 
(original)
+++ 
james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainView.java 
Fri Sep 23 09:25:52 2011
@@ -179,7 +179,7 @@ public class MainView extends Composite 
      * (non-Javadoc)
      * @see 
org.apache.hupa.client.mvp.MainPresenter.Display#bindTreeItems(java.util.ArrayList)
      */
-    public void bindTreeItems(ArrayList<IMAPTreeItem> treeList) {
+    public void bindTreeItems(List<IMAPTreeItem> treeList) {
         folderTree.clear();
         for (DropController dropController : dropControllerList) {
             controller.unregisterDropController(dropController);



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to