site: player: Fix teams not being sorted

We never actually used the order clause for sorting teams. Fix this, and make sure the default order puts more-recent teams first. Fixes: 6ff791d ("site: player: Add separate teams page") Signed-off-by: Sean Anderson <[email protected]>
Forty-Bot · Nov 24, 2023 · a110415 · a110415
1 parent e576215
commit a110415
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/trends/site/player.py b/trends/site/player.py
@@ -139,7 +139,7 @@ def get_logs(c, playerid, filters, duplicates=True, order_clause="logid DESC", l
         })
     return logs
 
-def get_teams(c, filters, order_clause="upper(rostered)", limit=10, offset=0):
+def get_teams(c, filters, order_clause="upper(rostered) DESC", limit=10, offset=0):
     inner_clauses = get_filter_clauses(filters, 'league', date_range='rostered')
     outer_clauses = get_filter_clauses(filters, 'formatid')
 
@@ -188,7 +188,7 @@ def get_teams(c, filters, order_clause="upper(rostered)", limit=10, offset=0):
            LEFT JOIN div_name USING (div_nameid)
            WHERE r = 1
                    {}
-           ORDER BY {} NULLS FIRST, lower(rostered) ASC
+           ORDER BY {}
            LIMIT %(limit)s OFFSET %(offset)s;""".format(inner_clauses, outer_clauses, order_clause),
            { 'playerid': flask.g.playerid, 'limit': limit, 'offset': offset, **filters })
     return teams.fetchall()