displaying the subgroups in ascending order

fixes #1628

Author: tulinkry

src/org/opensolaris/opengrok/configuration/Groups.java

@@ -329,9 +329,10 @@ private static void deleteGroup(Set<Group> groups, String groupname) {
     private static boolean treeTraverseGroups(Set<Group> groups, Walker f) {
         LinkedList<Group> stack = new LinkedList<>();
         for (Group g : groups) {
+            // the flag here represents the group's depth in the group tree
             g.setFlag(0);
             if (g.getParent() == null) {
-                stack.add(g);
+                stack.addLast(g);
             }
         }
 
@@ -343,10 +344,9 @@ private static boolean treeTraverseGroups(Set<Group> groups, Walker f) {
                 return true;
             }
 
-            for (Group x : g.getSubgroups()) {
-                x.setFlag(g.getFlag() + 1);
-                stack.addFirst(x);
-            }
+            g.getSubgroups().forEach((x) -> x.setFlag(g.getFlag() + 1));
+            // add all the subgroups respecting the sorted order
+            stack.addAll(0, g.getSubgroups());
         }
         return false;
     }

web/repos.jspf

@@ -21,11 +21,12 @@ CDDL HEADER END
 Copyright (c) 2015, 2017, Oracle and/or its affiliates. All rights reserved.
 
 --%>
+<%@page import="java.util.TreeSet"%>
+<%@page import="java.util.Iterator"%>
 <%@page import="org.opensolaris.opengrok.web.Scripts"%>
 <%@page import="org.json.simple.JSONArray"%>
 <%@page import="org.opensolaris.opengrok.configuration.messages.Message"%>
 <%@page import="java.util.SortedSet"%>
-<%@page import="java.util.SortedSet"%>
 <%@page import="java.util.Set"%>
 <%@page import="java.text.ParseException"%>
 <%@page import="java.util.ArrayList"%>
@@ -80,6 +81,7 @@ Copyright (c) 2015, 2017, Oracle and/or its affiliates. All rights reserved.
             LinkedList<Group> stack = new LinkedList<>();
             for ( Group x : groups ) {
                 if (x.getParent() == null && (pHelper.hasAllowedSubgroup(x) || cfg.isAllowed(x))) {
+                    // the flag here represents the state of the group - open/close
                     x.setFlag(0);
                     stack.addLast(x);
                 }
@@ -97,14 +99,19 @@ Copyright (c) 2015, 2017, Oracle and/or its affiliates. All rights reserved.
                 }
 
                 stack.element().setFlag(1);
-                
-                for (Group x : pHelper.getSubgroups(group)) {
+
+                Set<Group> subgroups = new TreeSet<>(pHelper.getSubgroups(group));
+                for (Iterator<Group> it = subgroups.iterator(); it.hasNext();) {
+                    Group x = it.next();
                     if (cfg.isAllowed(x) || pHelper.hasAllowedSubgroup(x)) {
                         x.setFlag(0);
-                        stack.addFirst(x);
+                    } else {
+                        it.remove();
                     }
                 }
-                
+                // add all the subgroups to the beginning respecting the order
+                stack.addAll(0, subgroups);
+
                 %><div class="panel">
                     <div class="panel-heading-accordion">