paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-15 03:11:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

subs: Fix set_muted parameter order.

Browse Source 
The status_element parameter is optional, and the other caller in
stream_popover.js does not provide it.  This fixes a regression in
commit e6a66063a9 (#15868).

Signed-off-by: Anders Kaseorg <anders@zulip.com>
This commit is contained in:
Anders Kaseorg
2020-07-23 00:10:06 -07:00
committed by Tim Abbott
parent 4a7eb47c36
commit e123f8f723
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
static/js/stream_edit.js
View File

@@ -413,7 +413,7 @@ function stream_is_muted_changed(e) {
const sub_settings = exports.settings_for_sub(sub);
const notification_checkboxes = sub_settings.find(".sub_notification_setting");
subs.set_muted(sub, `#stream_change_property_status${sub.stream_id}`, e.target.checked);
subs.set_muted(sub, e.target.checked, `#stream_change_property_status${sub.stream_id}`);
sub_settings.find(".mute-note").toggleClass("hide-mute-note", !sub.is_muted);
notification_checkboxes.toggleClass("muted-sub", sub.is_muted);
notification_checkboxes.find("input[type='checkbox']").prop("disabled", sub.is_muted);

2
static/js/subs.js
View File

@@ -95,7 +95,7 @@ exports.active_stream = function () {
}
};
exports.set_muted = function (sub, status_element, is_muted) {
exports.set_muted = function (sub, is_muted, status_element) {
stream_muting.update_is_muted(sub, is_muted);
stream_edit.set_stream_property(sub, "is_muted", sub.is_muted, status_element);
};