From 287e6b744a8de1f3437ec5c31c863fbf627c4cf7 Mon Sep 17 00:00:00 2001
From: Colin McDonald <cmcdonald.main@gmail.com>
Date: Thu, 18 Aug 2016 14:26:36 -0400
Subject: [PATCH] Sort GET /staff output by the time each grant was created

---
 src/main/java/net/frozenorb/apiv3/route/users/GETStaff.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/main/java/net/frozenorb/apiv3/route/users/GETStaff.java b/src/main/java/net/frozenorb/apiv3/route/users/GETStaff.java
index d64a87d..7b739d9 100644
--- a/src/main/java/net/frozenorb/apiv3/route/users/GETStaff.java
+++ b/src/main/java/net/frozenorb/apiv3/route/users/GETStaff.java
@@ -21,7 +21,7 @@ public final class GETStaff implements Handler<RoutingContext> {
             }
         });
 
-        Map<String, Set<User>> result = new TreeMap<>((a, b) -> {
+        Map<String, List<User>> result = new TreeMap<>((a, b) -> {
             Rank aRank = staffRanks.get(a);
             Rank bRank = staffRanks.get(b);
 
@@ -29,6 +29,7 @@ public final class GETStaff implements Handler<RoutingContext> {
         });
 
         List<Grant> staffGrants = SyncUtils.runBlocking(v -> Grant.findByRank(staffRanks.values(), v));
+        staffGrants.sort((a, b) -> a.getAddedAt().compareTo(b.getAddedAt()));
 
         for (Grant staffGrant : staffGrants) {
             if (staffGrant.isActive()) {
@@ -36,7 +37,7 @@ public final class GETStaff implements Handler<RoutingContext> {
                 Rank rank = staffRanks.get(staffGrant.getRank());
 
                 if (!result.containsKey(rank.getId())) {
-                    result.put(rank.getId(), new HashSet<>());
+                    result.put(rank.getId(), new ArrayList<>());
                 }
 
                 result.get(rank.getId()).add(user);