log 5xx errors

Also, fix console.log usage.
pull/50/merge
Martin Zimmermann 11 years ago
parent a29393ee3f
commit 306d2d9f9e

@ -47,7 +47,11 @@ define(["app/lib/promise"], function(Q) {
document.cookie = cookie;
}
resolve({status: xhr.status, body: xhr.responseText});
if (xhr.status >= 500) {
reject(xhr.body);
} else {
resolve({status: xhr.status, body: xhr.responseText});
}
}
try {
@ -88,7 +92,13 @@ define(["app/lib/promise"], function(Q) {
var modify = function(id, data) {
var deferred = Q.defer();
curl("PUT", endpoint + "/id/" + id, JSON.stringify(data), function (rv) {
deferred.resolve(JSON.parse(rv.body));
if (rv.status === 403) {
deferred.reject("Not authorized to modify this comment!");
} else if (rv.status === 200) {
deferred.resolve(JSON.parse(rv.body));
} else {
deferred.reject(rv.body);
}
});
return deferred.promise;
};

@ -2,6 +2,8 @@ define(function() {
"use strict";
var stderr = function(text) { console.log(text); };
var Promise = function() {
this.success = [];
this.errors = [];
@ -12,7 +14,7 @@ define(function() {
if (onError) {
this.errors.push(onError);
} else {
this.errors.push(console.log);
this.errors.push(stderr);
}
};

Loading…
Cancel
Save