Merge pull request #398 from deedee/fix_GetGroupMember

avoid infinite loop for duplicate input gid

Author: skyhit

src/java/main/com/topcoder/direct/services/view/action/contest/launch/GetGroupMemberAction.java

@@ -146,6 +146,7 @@ private Set<Long> getGroupMembers() throws Exception{
         boolean finished = false;
         while (!finished) {
             ListIterator<Long> iter = gids.listIterator();
+            finished = true;
             while (iter.hasNext()) {
                 Long gid = iter.next();
                 if (!gidProcessed.contains(gid)) {
@@ -156,23 +157,18 @@ private Set<Long> getGroupMembers() throws Exception{
                             if (gm.isGroup()) {
                                 if (!gids.contains(gm.getMemberId())) {
                                     iter.add(gm.getMemberId());
+                                    finished = false;
                                 }
                                 logger.info(" inner group: " + gm.getMemberId());
                             } else {
                                 members.add(gm.getMemberId());
                             }
                         }
                     }
-
                     gidProcessed.add(gid);
                 }
             }
-
-            if (gids.size() == gidProcessed.size()) {
-                finished = true;
-            }
         }
-
         return members;
     }