Merge pull request #234 from openHPI/fix_jquery

Migrate jQuery function use for ajax in case of success or failure
This commit is contained in:
MrSerth
2018-11-26 12:35:19 +01:00
committed by GitHub
4 changed files with 5 additions and 5 deletions

View File

@ -436,7 +436,7 @@ configureEditors: function () {
url: $('#editor').data('errors-url')
});
jqxhr.always(this.hideSpinner);
jqxhr.success(this.renderHint);
jqxhr.done(this.renderHint);
},
toggleButtonStates: function () {

View File

@ -83,7 +83,7 @@ CodeOceanEditorRequestForComments = {
$.flash.success({text: $('#askForCommentsButton').data('message-success')});
// trigger a run
this.runSubmission.call(this, submission);
}.bind(this)).error(this.ajaxError.bind(this));
}.bind(this)).fail(this.ajaxError.bind(this));
};
this.createSubmission($('#requestComments'), null, createRequestForComments.bind(this));

View File

@ -106,7 +106,7 @@ CodeOceanEditorSubmissions = {
this.ajax({
method: 'GET',
url: $('#start-over').data('url')
}).success(function(response) {
}).done(function(response) {
this.hideSpinner();
_.each(this.editors, function(editor) {
var file_id = $(editor.container).data('file-id');

View File

@ -8,9 +8,9 @@ $(document).on('turbolinks:load', function() {
dataType: 'json',
error_template_attribute_id: $('#add-attribute').find('select').val()
}
}).success(function () {
}).done(function () {
location.reload();
}).error(function (error) {
}).fail(function (error) {
$.flash.danger({text: error.statusText});
});
});