From 617b0a95d23bcb48a21b8c5bc3cfb9f7a2de1501 Mon Sep 17 00:00:00 2001 From: Steve Howell Date: Wed, 23 Oct 2013 13:12:40 -0400 Subject: [PATCH] Short circuit MIT case earlier in update_users(). This can be squashed with the prior commit, which inlined this MIT check into update_users() while trying to avoid other complications. After inlining the code, it's clear that we don't need to call the sort_users() line of code for MIT users, so I moved the MIT guard condition up. (imported from commit fa5b52e14964ad595b34d40ce6c8450ea93726c5) --- static/js/activity.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/static/js/activity.js b/static/js/activity.js index e6adbd73eb..e0890f2bbf 100644 --- a/static/js/activity.js +++ b/static/js/activity.js @@ -82,12 +82,12 @@ function focus_lost() { } function update_users() { - var users = sort_users(Object.keys(presence_info), presence_info); - if (page_params.domain === 'mit.edu') { return; // MIT realm doesn't have a presence list } + var users = sort_users(Object.keys(presence_info), presence_info); + var my_info = { name: page_params.fullname, email: page_params.email,