github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-26 23:06:19 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Remove deprecated Discourse.ajax methods.

Browse Source
This commit is contained in:
Robin Ward 2015-02-05 11:07:15 -05:00
parent c3874b6ec9
commit 68c2eb9a5b
1 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
app/assets/javascripts/discourse/mixins/ajax.js
View File

@ -34,22 +34,19 @@ Discourse.Ajax = Em.Mixin.create({
}
if (args.success) {
Ember.Logger.error("DEPRECATION: Discourse.ajax should use promises, received 'success' callback");
throw "Discourse.ajax should use promises, received 'success' callback";
}
if (args.error) {
Ember.Logger.error("DEPRECATION: Discourse.ajax should use promises, received 'error' callback");
throw "DEPRECATION: Discourse.ajax should use promises, received 'error' callback";
}
var performAjax = function(resolve, reject) {
var oldSuccess = args.success;
args.success = function(xhr) {
Ember.run(null, resolve, xhr);
if (oldSuccess) oldSuccess(xhr);
};
var oldError = args.error;
args.error = function(xhr, textStatus) {
// note: for bad CSRF we don't loop an extra request right away.
// this allows us to eliminate the possibility of having a loop.
if (xhr.status === 403 && xhr.responseText === "['BAD CSRF']") {
@ -63,10 +60,7 @@ Discourse.Ajax = Em.Mixin.create({
xhr.jqTextStatus = textStatus;
xhr.requestedUrl = url;
// TODO is this sequence correct? we are calling catch defined externally before
// the error that was defined inline, it should probably be in reverse
Ember.run(null, reject, xhr);
if (oldError) oldError(xhr);
};
// We default to JSON on GET. If we don't, sometimes if the server doesn't return the proper header