Make reload save options required explicit arguments.

This commit is contained in:
Tim Abbott
2015-11-30 08:47:19 -08:00
parent 12bff0441c
commit 05acd510c0
4 changed files with 20 additions and 3 deletions

View File

@@ -35,7 +35,10 @@ function call(args, idempotent) {
if (xhr.status === 403) {
try {
if ($.parseJSON(xhr.responseText).msg.indexOf("CSRF Error:") !== -1) {
reload.initiate({immediate: true});
reload.initiate({immediate: true,
save_pointer: true,
save_narrow: true,
save_compose: true});
}
} catch (ex) {
blueslip.error('Unexpected 403 response from server',

View File

@@ -366,7 +366,11 @@ function send_message_ajax(request, success, error) {
error: function (xhr, error_type) {
if (error_type !== 'timeout' && reload.is_pending()) {
// The error might be due to the server changing
reload.initiate({immediate: true, send_after_reload: true});
reload.initiate({immediate: true,
save_pointer: true,
save_narrow: true,
save_compose: true,
send_after_reload: true});
return;
}

View File

@@ -199,6 +199,12 @@ exports.initiate = function (options) {
send_after_reload: false
});
if (options.save_pointer === undefined ||
options.save_narrow === undefined ||
options.save_compose === undefined) {
blueslip.error("reload.initiate() called without explicit save options.");
}
if (options.immediate) {
do_reload_app(options.send_after_reload,
options.save_pointer,

View File

@@ -67,7 +67,11 @@ function get_events_success(events) {
new_pointer = event.pointer;
break;
case 'restart':
reload.initiate({message: "The application has been updated; reloading!"});
reload.initiate({save_pointer: true,
save_narrow: true,
save_compose: true,
message: "The application has been updated; reloading!"
});
break;
case 'update_message':
messages_to_update.push(event);