CodeCow/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2026-07-18 21:04:19 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Rename stream_list_hash -> stream_set

Browse Source 
This makes the intent more clear, since the values are all 'true'.

(imported from commit 1f1d6e206af9f2258b55bc77e7722158c2ffac98)
This commit is contained in:
Keegan McAllister 2012-10-31 18:16:51 -04:00
parent 651566ba16
commit 22dc487a72
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
zephyr/static/js/subs.js
View File

@ -2,7 +2,7 @@ var subs = (function () {
var exports = {};
var stream_list_hash = {};
var stream_set = {};
function case_insensitive_subscription_index(stream_name) {
var i;
@ -19,13 +19,13 @@ function case_insensitive_subscription_index(stream_name) {
function add_to_stream_list(stream_name) {
if (!exports.have(stream_name)) {
stream_list.push(stream_name);
stream_list_hash[stream_name.toLowerCase()] = true;
stream_set[stream_name.toLowerCase()] = true;
$('#subscriptions_table').prepend(templates.subscription({subscription: stream_name}));
}
}
function remove_from_stream_list(stream_name) {
delete stream_list_hash[stream_name.toLowerCase()];
delete stream_set[stream_name.toLowerCase()];
var removal_index = case_insensitive_subscription_index(stream_name);
if (removal_index !== -1) {
stream_list.splice(removal_index, 1);
@ -74,14 +74,14 @@ exports.subscribe_for_send = function (stream, prompt_button) {
};
exports.have = function (stream_name) {
return (stream_list_hash[stream_name.toLowerCase()] === true);
return (stream_set[stream_name.toLowerCase()] === true);
};
$(function () {
var i;
// Populate stream_list_hash with data handed over to client-side template.
// Populate stream_set with data handed over to client-side template.
for (i = 0; i < stream_list.length; i++) {
stream_list_hash[stream_list[i].toLowerCase()] = true;
stream_set[stream_list[i].toLowerCase()] = true;
}
// FIXME: It would be nice to move the UI setup into ui.js.