Show error messages in dialog box to move tasks

app/Controller/TaskMovePositionController.php

@@ -36,7 +36,7 @@ class TaskMovePositionController extends BaseController
         $values = $this->request->getJson();
 
         if (! $this->helper->projectRole->canMoveTask($task['project_id'], $task['column_id'], $values['column_id'])) {
-            throw new AccessForbiddenException(e("You don't have the permission to move this task"));
+            throw new AccessForbiddenException(e('You are not allowed to move this task.'));
         }
 
         $result = $this->taskPositionModel->movePosition(

app/Locale/bs_BA/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/cs_CZ/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/da_DK/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/de_DE/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/el_GR/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/es_ES/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/fi_FI/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/fr_FR/translations.php

@@ -1278,4 +1278,5 @@ return array(
     'This field is required' => 'Ce champ est requis',
     'Moving a task is not permitted' => 'Déplaçer une tâche n\'est pas autorisé',
     'This value must be in the range %d to %d' => 'Cette valeur doit être définie entre %d et %d',
+    'You are not allowed to move this task.' => 'Vous n\'êtes pas autorisé à déplacer cette tâche.',
 );

app/Locale/hu_HU/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/id_ID/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/it_IT/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/ja_JP/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/ko_KR/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/my_MY/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/nb_NO/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/nl_NL/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/pl_PL/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/pt_BR/translations.php

@@ -1277,4 +1277,5 @@ return array(
     'This field is required' => 'Este campo é obrigatório',
     'Moving a task is not permitted' => 'Mover uma tarefa não é permitido',
     'This value must be in the range %d to %d' => 'Este valor precisa estar no intervalo %d até %d',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/pt_PT/translations.php

@@ -1277,4 +1277,5 @@ return array(
     'This field is required' => 'Este campo é obrigatório',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/ru_RU/translations.php

@@ -1277,4 +1277,5 @@ return array(
     'This field is required' => 'Заполните это поле',
     'Moving a task is not permitted' => 'Перемещение задачи не разрешено',
     'This value must be in the range %d to %d' => 'Значение должно находиться в диапазоне от %d до %d',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/sr_Latn_RS/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/sv_SE/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/th_TH/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/tr_TR/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Locale/zh_CN/translations.php

@@ -1277,4 +1277,5 @@ return array(
     // 'This field is required' => '',
     // 'Moving a task is not permitted' => '',
     // 'This value must be in the range %d to %d' => '',
+    // 'You are not allowed to move this task.' => '',
 );

app/Template/task_move_position/show.php

@@ -28,6 +28,10 @@
         <label><input type="radio" value="after" v-model="positionChoice"><?= t('Insert after this task') ?></label>
     </div>
 
+    <div v-if="errorMessage">
+        <div class="alert alert-error">{{ errorMessage }}</div>
+    </div>
+
     <submit-cancel
         label-button="<?= t('Save') ?>"
         label-or="<?= t('or') ?>"

assets/js/components/submit-cancel.js

@@ -26,5 +26,10 @@ Vue.component('submit-cancel', {
         onCancel: function () {
             _KB.get('Popover').close();
         }
+    },
+    events: {
+        'submitCancelled': function() {
+            this.loading = false;
+        }
     }
 });

assets/js/components/task-move-position.js

@@ -8,13 +8,15 @@ Vue.component('task-move-position', {
             position: 1,
             columns: [],
             tasks: [],
-            positionChoice: 'before'
+            positionChoice: 'before',
+            errorMessage: ''
         }
     },
     ready: function () {
         this.columns = this.board[0].columns;
         this.columnId = this.columns[0].id;
         this.tasks = this.columns[0].tasks;
+        this.errorMessage = '';
     },
     methods: {
         onChangeSwimlane: function () {
@@ -50,6 +52,8 @@ Vue.component('task-move-position', {
             });
         },
         onSubmit: function () {
+            var self = this;
+
             if (this.positionChoice == 'after') {
                 this.position++;
             }
@@ -65,8 +69,16 @@ Vue.component('task-move-position', {
                     "swimlane_id": this.swimlaneId,
                     "position": this.position
                 }),
-                complete: function() {
+                statusCode: {
-                    window.location.reload(true);
+                    200: function() {
+                        window.location.reload(true);
+                    },
+                    403: function(jqXHR) {
+                        var response = JSON.parse(jqXHR.responseText);
+                        self.errorMessage = response.message;
+
+                        self.$broadcast('submitCancelled');
+                    }
                 }
             });
         }