diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2017-02-27 17:14:20 +0300 |
---|---|---|
committer | Sean McGivern <sean@mcgivern.me.uk> | 2017-02-27 17:14:20 +0300 |
commit | dfd15596a4b53ba2b7b3d0be9d5c17e2d7824c8f (patch) | |
tree | 21e843dcc9ac0f33ee34dbe897be0a482c43472d /app/assets | |
parent | ead12a90afbff0a750883583e2474ad24f83c9d8 (diff) | |
parent | 0d000d351ca587ff7a6d4a14ad3cfa693238eec0 (diff) |
Merge branch 'issue_24815' into 'master'
fix ERR_CONTENT_LENGTH_MISMATCH on task checkboxes
See merge request !8567
Diffstat (limited to 'app/assets')
-rw-r--r-- | app/assets/javascripts/task_list.js | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/app/assets/javascripts/task_list.js b/app/assets/javascripts/task_list.js index dfe24d1fb33..b1402c0a880 100644 --- a/app/assets/javascripts/task_list.js +++ b/app/assets/javascripts/task_list.js @@ -1,3 +1,4 @@ +/* global Flash */ require('vendor/task_list'); class TaskList { @@ -6,6 +7,16 @@ class TaskList { this.dataType = options.dataType; this.fieldName = options.fieldName; this.onSuccess = options.onSuccess || (() => {}); + this.onError = function showFlash(response) { + let errorMessages = ''; + + if (response.responseJSON) { + errorMessages = response.responseJSON.errors.join(' '); + } + + return new Flash(errorMessages || 'Update failed', 'alert'); + }; + this.init(); } @@ -32,6 +43,7 @@ class TaskList { url: $target.data('update-url') || $('form.js-issuable-update').attr('action'), data: patchData, success: this.onSuccess, + error: this.onError, }); } } |