From d577c73e46de23742bc069d0a444f0bedeca5abc Mon Sep 17 00:00:00 2001
From: Frederic Guillot <fred@kanboard.net>
Date: Sat, 13 Jun 2015 22:03:12 -0400
Subject: Add new automatic action: Move task to another column when the
 category is changed

---
 app/Model/Action.php           |  1 +
 app/Model/TaskModification.php | 25 ++++++++++++++++++++++---
 2 files changed, 23 insertions(+), 3 deletions(-)

(limited to 'app/Model')

diff --git a/app/Model/Action.php b/app/Model/Action.php
index 3e8aa091..c3bfe017 100644
--- a/app/Model/Action.php
+++ b/app/Model/Action.php
@@ -57,6 +57,7 @@ class Action extends Base
             'TaskAssignUser' => t('Change the assignee based on an external username'),
             'TaskAssignCategoryLabel' => t('Change the category based on an external label'),
             'TaskUpdateStartDate' => t('Automatically update the start date'),
+            'TaskMoveColumnCategoryChange' => t('Move the task to another column when the category is changed'),
         );
 
         asort($values);
diff --git a/app/Model/TaskModification.php b/app/Model/TaskModification.php
index 677fcd60..4691ce81 100644
--- a/app/Model/TaskModification.php
+++ b/app/Model/TaskModification.php
@@ -42,13 +42,19 @@ class TaskModification extends Base
      */
     public function fireEvents(array $task, array $new_values)
     {
+        $events = array();
         $event_data = array_merge($task, $new_values, array('task_id' => $task['id']));
 
-        if (isset($new_values['owner_id']) && $task['owner_id'] != $new_values['owner_id']) {
-            $events = array(Task::EVENT_ASSIGNEE_CHANGE);
+        // Values changed
+        $event_data['changes'] = array_diff_assoc($new_values, $task);
+        unset($event_data['changes']['date_modification']);
+
+        if ($this->isFieldModified('owner_id', $event_data['changes'])) {
+            $events[] = Task::EVENT_ASSIGNEE_CHANGE;
         }
         else {
-            $events = array(Task::EVENT_CREATE_UPDATE, Task::EVENT_UPDATE);
+            $events[] = Task::EVENT_CREATE_UPDATE;
+            $events[] = Task::EVENT_UPDATE;
         }
 
         foreach ($events as $event) {
@@ -56,6 +62,19 @@ class TaskModification extends Base
         }
     }
 
+    /**
+     * Return true if the field is the only modified value
+     *
+     * @access public
+     * @param  string  $field
+     * @param  array   $changes
+     * @return boolean
+     */
+    public function isFieldModified($field, array $changes)
+    {
+        return isset($changes[$field]) && count($changes) === 1;
+    }
+
     /**
      * Prepare data before task modification
      *
-- 
cgit v1.2.3