From 343e86a138648644870209708760c1e1f36805f7 Mon Sep 17 00:00:00 2001 From: Frederic Guillot Date: Thu, 23 Feb 2017 17:30:27 -0500 Subject: Automatic action to change task color when due date is expired (PR #3062) --- ChangeLog | 1 + app/Action/TaskAssignColorOnDueDate.php | 100 +++++++++++++++++++++ app/ServiceProvider/ActionProvider.php | 2 + .../units/Action/TaskAssignColorOnDueDateTest.php | 37 ++++++++ 4 files changed, 140 insertions(+) create mode 100644 app/Action/TaskAssignColorOnDueDate.php create mode 100644 tests/units/Action/TaskAssignColorOnDueDateTest.php diff --git a/ChangeLog b/ChangeLog index 30b6d0e2..10c41b2c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -7,6 +7,7 @@ New features: * Send tasks by email * Add Reply-To header to emails sent from Kanboard * Upload Sqlite database from user interface +* Automatic action to change task color when due date is expired Improvements: diff --git a/app/Action/TaskAssignColorOnDueDate.php b/app/Action/TaskAssignColorOnDueDate.php new file mode 100644 index 00000000..06b70a18 --- /dev/null +++ b/app/Action/TaskAssignColorOnDueDate.php @@ -0,0 +1,100 @@ + t('Color'), + ); + } + + /** + * Get all tasks + * + * @access public + * @return array + */ + + public function getEventRequiredParameters() + { + return array('tasks'); + } + + /** + * Execute the action (change the task color) + * + * @access public + * @param array $data Event data dictionary + * @return bool True if the action was executed or false when not executed + */ + public function doAction(array $data) + { + $results = array(); + + foreach ($data['tasks'] as $task) { + if ($task['date_due'] <= time() && $task['date_due'] > 0) { + $values = array( + 'id' => $task['id'], + 'color_id' => $this->getParam('color_id'), + ); + $results[] = $this->taskModificationModel->update($values, false); + } + } + + return in_array(true, $results, true); + } + + /** + * Check if the event data meet the action condition + * + * @access public + * @param array $data Event data dictionary + * @return bool + */ + public function hasRequiredCondition(array $data) + { + return count($data['tasks']) > 0; + } +} diff --git a/app/ServiceProvider/ActionProvider.php b/app/ServiceProvider/ActionProvider.php index 81f2b39e..a7e8040e 100644 --- a/app/ServiceProvider/ActionProvider.php +++ b/app/ServiceProvider/ActionProvider.php @@ -4,6 +4,7 @@ namespace Kanboard\ServiceProvider; use Pimple\Container; use Pimple\ServiceProviderInterface; +use Kanboard\Action\TaskAssignColorOnDueDate; use Kanboard\Action\TaskAssignColorPriority; use Kanboard\Action\TaskAssignDueDateOnCreation; use Kanboard\Action\TaskMoveColumnClosed; @@ -92,6 +93,7 @@ class ActionProvider implements ServiceProviderInterface $container['actionManager']->register(new TaskAssignDueDateOnCreation($container)); $container['actionManager']->register(new TaskAssignColorSwimlane($container)); $container['actionManager']->register(new TaskAssignPrioritySwimlane($container)); + $container['actionManager']->register(new TaskAssignColorOnDueDate($container)); return $container; } diff --git a/tests/units/Action/TaskAssignColorOnDueDateTest.php b/tests/units/Action/TaskAssignColorOnDueDateTest.php new file mode 100644 index 00000000..4bb87c06 --- /dev/null +++ b/tests/units/Action/TaskAssignColorOnDueDateTest.php @@ -0,0 +1,37 @@ +container); + $taskCreationModel = new TaskCreationModel($this->container); + $taskFinderModel = new TaskFinderModel($this->container); + + $this->assertEquals(1, $projectModel->create(array('name' => 'test1'))); + $this->assertEquals(1, $taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'date_due' => strtotime('-1 day')))); + $this->assertEquals(2, $taskCreationModel->create(array('project_id' => 1, 'title' => 'test'))); + + $tasks = $taskFinderModel->getAll(1); + $event = new TaskListEvent(array('tasks' => $tasks, 'project_id' => 1)); + + $action = new TaskAssignColorOnDueDate($this->container); + $action->setProjectId(1); + $action->setParam('color_id', 'red'); + + $this->assertTrue($action->execute($event, TaskModel::EVENT_DAILY_CRONJOB)); + + $tasks = $taskFinderModel->getAll(1); + $this->assertEquals('red', $tasks[0]['color_id']); + $this->assertEquals('yellow', $tasks[1]['color_id']); + } +} -- cgit v1.2.3