Merge pull request #1013 from edx/jkarni/fix/import-warnings

Jkarni/fix/import warnings
This commit is contained in:
jkarni
2013-09-23 13:15:55 -07:00

View File

@@ -77,16 +77,12 @@ $('#fileupload').fileupload({
e.preventDefault();
submitBtn.hide();
data.submit().complete(function(result, textStatus, xhr) {
if (result.status != 200) {
alert('${_("Your import has failed.")}\n\n' + JSON.parse(result.responseText)["ErrMsg"]);
if (xhr.status != 200) {
var errMsg = result.hasOwnProperty("ErrMsg") ? result["ErrMsg"] : "" ;
alert('${_("Your import has failed.")}\n\n' + errMsg);
submitBtn.show();
bar.hide();
} else {
if (result.responseText["ImportStatus"] == 1) {
bar.hide();
statusBlock.show();
}
}
}
});
});
} else {
@@ -102,13 +98,17 @@ $('#fileupload').fileupload({
},
done: function(e, data){
bar.hide();
window.onbeforeunload = null;
alert('${_("Your import was successful.")}');
window.location = '${successful_import_redirect_url}';
},
start: function(e) {
window.onbeforeunload = function() {
return '${_("Your import is in progress; navigating away will abort it.")}';
}
},
sequentialUploads: true
});
})();