From 6449366d6494c2df374464786081052069df6057 Mon Sep 17 00:00:00 2001 From: Clara Dantas Date: Tue, 9 Jun 2020 19:52:34 -0300 Subject: [PATCH] settings_users: Remove unnecessary sort. The populate_users function doesn't need to sort the list of active and non-active users, because the list_render is called specifying to sort users by their full_name. Author: Clara Moraes Dantas --- static/js/settings_users.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/static/js/settings_users.js b/static/js/settings_users.js index 41ad5d6faf..14d8e4ce59 100644 --- a/static/js/settings_users.js +++ b/static/js/settings_users.js @@ -137,11 +137,9 @@ function failed_listing_users() { function populate_users() { const active_user_ids = people.get_active_human_ids(); let active_users = active_user_ids.map(user_id => people.get_by_user_id(user_id)); - active_users = _.sortBy(active_users, 'full_name'); const deactivated_user_ids = people.get_non_active_human_ids(); let deactivated_users = deactivated_user_ids.map(user_id => people.get_by_user_id(user_id)); - deactivated_users = _.sortBy(deactivated_users, 'full_name'); if (active_user_ids.length === 0 && deactivated_user_ids.length === 0) { failed_listing_users();