##// END OF EJS Templates
Use promising_ajax in config API
Thomas Kluyver -
Show More
@@ -35,45 +35,28 b' function($, utils) {'
35 35 };
36 36
37 37 ConfigSection.prototype.load = function() {
38 var p = new Promise(function(resolve, reject) {
39 $.ajax(this.api_url(), {
40 cache : false,
41 type : "GET",
42 dataType : "json",
43 success: function(data, status, jqXHR) {
44 this.data = data;
45 this._load_done();
46 resolve(data);
47 },
48 error: function(jqXHR, status, error) {
49 // Should never happen; mark as loaded so things don't keep
50 // waiting.
51 this._load_done();
52 utils.log_ajax_error(jqXHR, status, error);
53 reject(utils.wrap_ajax_error(jqXHR, status, error));
54 }
55 });
38 return utils.promising_ajax(this.api_url(), {
39 cache: false,
40 type: "GET",
41 dataType: "json",
42 }).then(function(data) {
43 this.data = data;
44 this._load_done();
45 return data;
56 46 });
57 47 };
58 48
59 49 ConfigSection.prototype.update = function(newdata) {
60 return new Promise(function(resolve, reject) {
61 $.ajax(this.api_url(), {
62 processData: false,
63 type : "PATCH",
64 data: JSON.stringify(newdata),
65 dataType : "json",
66 contentType: 'application/json',
67 success: function(data, status, jqXHR) {
68 this.data = data;
69 this._load_done();
70 resolve(data);
71 },
72 error: function(jqXHR, status, error) {
73 utils.log_ajax_error(jqXHR, status, error);
74 reject(utils.wrap_ajax_error(jqXHR, status, error));
75 }
76 });
50 return utils.promising_ajax(this.api_url(), {
51 processData: false,
52 type : "PATCH",
53 data: JSON.stringify(newdata),
54 dataType : "json",
55 contentType: 'application/json',
56 }).then(function(data) {
57 this.data = data;
58 this._load_done();
59 return data;
77 60 });
78 61 };
79 62
General Comments 0
You need to be logged in to leave comments. Login now