OF-368 - fixed user-summary.jsp which now returns range=3 instead of -1 for 3...

OF-368 - fixed user-summary.jsp which now returns range=3 instead of -1 for 3 users. fixed DefaultUserProvider.java which did return all users instead of start-range (introduced during removal of duplicate code)

git-svn-id: http://svn.igniterealtime.org/svn/repos/openfire/trunk@11767 b35dd754-fafc-0310-a699-88a17e54d16e

src/java/org/jivesoftware/openfire/user/DefaultUserProvider.java

@@ -290,7 +290,7 @@ public class DefaultUserProvider implements UserProvider {
     }
 
     public Collection<User> getUsers(int startIndex, int numResults) {
-        Collection<String> usernames = getUsernames(0, Integer.MAX_VALUE);
+        Collection<String> usernames = getUsernames(startIndex, numResults);
         return new UserCollection(usernames.toArray(new String[usernames.size()]));
     }
 

src/web/user-summary.jsp

@@ -101,7 +101,7 @@
 
     <% for (int aRANGE_PRESETS : RANGE_PRESETS) { %>
 
-    <option value="<%= aRANGE_PRESETS %>"
+    <option value="<%  if (aRANGE_PRESETS > 0) { %><%= aRANGE_PRESETS %><%  }else{ %><%= userCount %><%}%>"
             <%= (aRANGE_PRESETS == range ? "selected" : "") %>><%  if (aRANGE_PRESETS > 0) { %><%= aRANGE_PRESETS %><%  }else{ %><%= userCount %><%}%>
     </option>