Improve and translate pull-request

app/Action/TaskMoveColumnAssigned.php

@@ -34,8 +34,8 @@ class TaskMoveColumnAssigned extends Base
     public function getActionRequiredParameters()
     {
         return array(
-            'src_column_id' => t('From column'),
+            'src_column_id' => t('Source column'),
-            'dest_column_id' => t('To column')
+            'dest_column_id' => t('Destination column')
         );
     }
 
@@ -85,7 +85,6 @@ class TaskMoveColumnAssigned extends Base
      */
     public function hasRequiredCondition(array $data)
     {
-        return $data['column_id'] == $this->getParam('src_column_id') &&
+        return $data['column_id'] == $this->getParam('src_column_id') && $data['owner_id'];
-               $data['owner_id'];
     }
 }

app/Action/TaskMoveColumnUnAssigned.php

@@ -34,8 +34,8 @@ class TaskMoveColumnUnAssigned extends Base
     public function getActionRequiredParameters()
     {
         return array(
-            'src_column_id' => t('From column'),
+            'src_column_id' => t('Source column'),
-            'dest_column_id' => t('To column')
+            'dest_column_id' => t('Destination column')
         );
     }
 
@@ -85,7 +85,6 @@ class TaskMoveColumnUnAssigned extends Base
      */
     public function hasRequiredCondition(array $data)
     {
-        return $data['column_id'] == $this->getParam('src_column_id') &&
+        return $data['column_id'] == $this->getParam('src_column_id') && ! $data['owner_id'];
-               !$data['owner_id'];
     }
 }

app/Locale/da_DK/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/de_DE/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/es_ES/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/fi_FI/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/fr_FR/translations.php

@@ -806,4 +806,8 @@ return array(
     'Unable to remove this budget line.' => 'Impossible de supprimer cette ligne budgétaire.',
     'USD - US Dollar' => 'USD - Dollar américain',
     'Remaining' => 'Restant',
+    'Destination column' => 'Colonne de destination',
+    'Move the task to another column when assigned to a user' => 'Déplacer la tâche dans une autre colonne lorsque celle-ci est assignée à quelqu\'un',
+    'Move the task to another column when assignee is cleared' => 'Déplacer la tâche dans une autre colonne lorsque celle-ci n\'est plus assignée',
+    'Source column' => 'Colonne d\'origine',
 );

app/Locale/hu_HU/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/it_IT/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/ja_JP/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/nl_NL/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/pl_PL/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/pt_BR/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/ru_RU/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/sr_Latn_RS/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/sv_SE/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/th_TH/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/tr_TR/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Locale/zh_CN/translations.php

@@ -804,4 +804,8 @@ return array(
     // 'Unable to remove this budget line.' => '',
     // 'USD - US Dollar' => '',
     // 'Remaining' => '',
+    // 'Destination column' => '',
+    // 'Move the task to another column when assigned to a user' => '',
+    // 'Move the task to another column when assignee is cleared' => '',
+    // 'Source column' => '',
 );

app/Model/TaskPosition.php

@@ -21,6 +21,7 @@ class TaskPosition extends Base
      * @param  integer    $column_id         Column id
      * @param  integer    $position          Position (must be >= 1)
      * @param  integer    $swimlane_id       Swimlane id
+     * @param  boolean    $fire_events       Fire events
      * @return boolean
      */
     public function movePosition($project_id, $task_id, $column_id, $position, $swimlane_id = 0, $fire_events = true)
@@ -30,13 +31,14 @@ class TaskPosition extends Base
         $result = $this->calculateAndSave($project_id, $task_id, $column_id, $position, $swimlane_id);
 
         if ($result) {
-            
+
             if ($original_task['swimlane_id'] != $swimlane_id) {
                 $this->calculateAndSave($project_id, 0, $column_id, 1, $original_task['swimlane_id']);
             }
 
-            if ($fire_events)
+            if ($fire_events) {
                 $this->fireEvents($original_task, $column_id, $position, $swimlane_id);
+            }
         }
 
         return $result;