Issue #440 add listeners before connecting and fix populateGroups()

Change-Id: Ief028f153954484666d01f52b91da9c97490ef94

Former-commit-id: ea146f69c6fca818589587d37aace2afc75e7147

cave/com.raytheon.uf.viz.collaboration.ui/src/com/raytheon/uf/viz/collaboration/ui/CollaborationGroupView.java

@@ -608,19 +608,13 @@ public class CollaborationGroupView extends ViewPart implements IPartListener {
     private void populateGroups() {
         CollaborationDataManager manager = CollaborationDataManager
                 .getInstance();
-
-        // go through and clear out everything above the groups (my user, the
-        // sessions)
-        List<Object> obs = new ArrayList<Object>();
-        obs.addAll(topLevel.getObjects());
-        manager.getCollaborationConnection().getRosterManager();
-
         CollaborationUtils.readAliases();
-        refreshEntry(
+        for (Object ob : manager.getCollaborationConnection()
-                manager.getCollaborationConnection().getContactsManager()
+                .getRosterManager().getRoster().getItems()) {
-                        .getUsersMap()
+            if (ob instanceof IRosterGroup) {
-                        .get(manager.getCollaborationConnection().getUser()),
+                addGroup((IRosterGroup) ob);
-                manager.getCollaborationConnection());
+            }
+        }
     }
 
     private void fillStatusMenu(Menu menu) {
@@ -1295,17 +1289,13 @@ public class CollaborationGroupView extends ViewPart implements IPartListener {
         }
     }
 
-    private void refreshGroup(IRosterGroup group) {
-
-    }
-
     /**
      * Adds users to groups if necessary
      * 
      * @param entry
      * @param connection
      */
-    private synchronized void refreshEntry(IRosterEntry entry,
+    private void refreshEntry(IRosterEntry entry,
             CollaborationConnection connection) {
         List<IRosterGroup> groups = new ArrayList<IRosterGroup>();
         for (Object ob : connection.getRosterManager().getRoster().getItems()) {
@@ -1313,6 +1303,7 @@ public class CollaborationGroupView extends ViewPart implements IPartListener {
                 groups.add((IRosterGroup) ob);
             }
         }
+
         // looping through my groups
         for (final IRosterGroup group : groups) {
             // looping through the other entries groups
@@ -1322,34 +1313,33 @@ public class CollaborationGroupView extends ViewPart implements IPartListener {
                 // so both users have it (you are showing it, and they
                 // are part of it)
                 if (group.getName().equals(otherGroup.getName())) {
+                    addGroup(otherGroup);
+                }
+            }
+        }
+    }
+
+    private synchronized void addGroup(IRosterGroup group) {
         boolean created = false;
-                    List<Object> obs = new ArrayList<Object>();
+        for (Object topOb : topLevel.getObjects()) {
-                    obs.addAll(topLevel.getObjects());
-                    for (Object topOb : obs) {
             if (topOb instanceof IRosterGroup) {
                 IRosterGroup topGroup = (IRosterGroup) topOb;
                 if (topGroup.getName().equals(group.getName())) {
-                                System.out.println("Created is true : "
+                    System.out.println("Created is true : " + group.getName()
-                                        + group.getName() + " / "
+                            + " / " + topGroup.getName());
-                                        + topGroup.getName());
                     created = true;
                     break;
                 } else {
-                                System.out.println("Created is false : "
+                    System.out.println("Created is false : " + group.getName()
-                                        + group.getName() + " / "
+                            + " / " + topGroup.getName());
-                                        + topGroup.getName());
                 }
             }
         }
         if (!created) {
-                        System.out.println("creating group : "
+            System.out.println("creating group : " + group.getName());
-                                + group.getName());
             topLevel.addObject(group);
         }
     }
-            }
-        }
-    }
 
     // Does nothing, but necessary due to ViewPart
     @Override

cave/com.raytheon.uf.viz.collaboration/src/com/raytheon/uf/viz/collaboration/comm/provider/session/CollaborationConnection.java

@@ -176,6 +176,12 @@ public class CollaborationConnection implements IEventPublisher {
                                 + event.getClass().getName());
                     }
                 });
+
+                // add the listeners before we connect so we don't potentially
+                // miss something
+                presenceAdapter = Tools.getPresenceContainerAdapter(container,
+                        IPresenceContainerAdapter.class);
+                this.setupInternalConnectionListeners();
             }
 
         } catch (ContainerCreateException cce) {
@@ -202,7 +208,6 @@ public class CollaborationConnection implements IEventPublisher {
 
         setupAccountManager();
 
-        setupInternalConnectionListeners();
         setupInternalVenueInvitationListener();
         setupP2PComm(presenceAdapter);
         getPeerToPeerSession();
@@ -252,11 +257,10 @@ public class CollaborationConnection implements IEventPublisher {
 
             // Now connect
             ID targetID = createID(account);
-            container.connect(targetID, ConnectContextFactory
-                    .createPasswordConnectContext(password));
-
             presenceAdapter = Tools.getPresenceContainerAdapter(container,
                     IPresenceContainerAdapter.class);
+            container.connect(targetID, ConnectContextFactory
+                    .createPasswordConnectContext(password));
         }
     }