From 1c57956bf6f7dd20832c148925826e57f4c9caa3 Mon Sep 17 00:00:00 2001 From: Tim Abbott Date: Tue, 23 Apr 2013 13:53:11 -0400 Subject: [PATCH] Avoid an unnecessary load_old_messages call on page load. Previously, our check for whether we needed to call load_old_messages a second time on page load to get up to the present caused us to basically always do such a call. (imported from commit b599041e8c0853b4c8c9ab2def6679142302523e) --- zephyr/static/js/zephyr.js | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/zephyr/static/js/zephyr.js b/zephyr/static/js/zephyr.js index 871f3c73b0..e51d513c87 100644 --- a/zephyr/static/js/zephyr.js +++ b/zephyr/static/js/zephyr.js @@ -907,16 +907,18 @@ $(function () { } // catch the user up - if (messages.length !== 1) { + if (messages.length !== 0) { var latest_id = messages[messages.length-1].id; - load_old_messages({ - anchor: latest_id, - num_before: 0, - num_after: 400, - msg_list: home_msg_list, - cont: load_more - }); - return; + if (latest_id < page_params.max_message_id) { + load_old_messages({ + anchor: latest_id, + num_before: 0, + num_after: 400, + msg_list: home_msg_list, + cont: load_more + }); + return; + } } // now start subscribing to updates get_updates();