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

Extract notifications.reify_message_id().

Browse Source 
This removes the last need for a message_id_changed event.
This commit is contained in:
Steve Howell
2017-07-19 08:39:28 -04:00
committed by Tim Abbott
parent 3f06f28ad7
commit 7caf8edadc
2 changed files with 17 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
static/js/echo.js
View File

@@ -203,8 +203,7 @@ exports.reify_message_id = function reify_message_id(local_id, server_id) {
var opts = {old_id: local_id, new_id: server_id};
message_store.reify_message_id(opts);
$(document).trigger($.Event('message_id_changed', opts));
notifications.reify_message_id(opts);
};
exports.process_from_server = function process_from_server(messages) {

11
static/js/notifications.js
View File

@@ -619,9 +619,11 @@ $(function () {
window.bridge = bridge;
}
$(document).on('message_id_changed', function (event) {
var old_id = event.old_id;
var new_id = event.new_id;
});
exports.reify_message_id = function (opts) {
var old_id = opts.old_id;
var new_id = opts.new_id;
// If a message ID that we're currently storing (as a link) has changed,
// update that link as well
@@ -633,8 +635,7 @@ $(function () {
elem.data('msgid', new_id);
}
});
});
});
};
exports.register_click_handlers = function () {
$('#out-of-view-notification').on('click', '.compose_notification_narrow_by_subject', function (e) {