paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-09 16:37:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

subs: Remove unnecessary data-is-private variable.

Browse Source
This commit is contained in:
Tim Abbott
2017-02-25 18:32:43 -08:00
parent f20993787c
commit bd03bb76fd
2 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
static/js/subs.js
View File

@@ -632,14 +632,13 @@ function change_stream_privacy(e) {
e.stopPropagation();
var stream_id = $(e.target).data("stream-id");
var is_private = $(e.target).data("is-private");
var sub = stream_data.get_sub_by_id(stream_id);
$("#subscriptions-status").hide();
var data = {
stream_name: sub.name,
// toggle the privacy setting
is_private: !is_private,
is_private: !sub.invite_only,
};
channel.patch({
@@ -647,12 +646,10 @@ function change_stream_privacy(e) {
data: data,
success: function () {
sub = stream_data.get_sub_by_id(stream_id);
var stream_settings = settings_for_sub(sub);
var sub_row = $(".stream-row[data-stream-id='" + stream_id + "']");
// save new privacy settings.
stream_settings.find(".change-stream-privacy").expectOne().data("is-private", !is_private);
sub.invite_only = !is_private;
sub.invite_only = !sub.invite_only;
redraw_privacy_related_stuff(sub_row, sub);
$("#stream_privacy_modal").remove();

2
static/templates/subscription_stream_privacy_modal.handlebars
View File

@@ -22,7 +22,7 @@
<div class="modal-footer">
<button class="btn btn-default close-privacy-modal" aria-hidden="true">{{t "Cancel" }}</button>
<button class="btn{{#if is_private}} btn-primary{{else}} btn-danger{{/if}}" id="change-stream-privacy-button"
tabindex="-1" data-stream-id="{{stream_id}}" data-is-private="{{is_private}}">
tabindex="-1" data-stream-id="{{stream_id}}">
{{#if is_private}}
{{t "Make Stream Public"}}
{{else}}