From a65941eadb3ddcec93132006c885106a6aa4cb9b Mon Sep 17 00:00:00 2001 From: Frederic Guillot Date: Mon, 19 Sep 2016 21:28:11 -0400 Subject: Add new automatic action to close task based on date moved --- ChangeLog | 4 +- app/Action/TaskCloseNoActivityColumn.php | 2 +- app/Action/TaskCloseNotMovedColumn.php | 96 ++++++++++++++++++++++ app/ServiceProvider/ActionProvider.php | 6 +- tests/units/Action/TaskCloseNotMovedColumnTest.php | 49 +++++++++++ 5 files changed, 153 insertions(+), 4 deletions(-) create mode 100644 app/Action/TaskCloseNotMovedColumn.php create mode 100644 tests/units/Action/TaskCloseNotMovedColumnTest.php diff --git a/ChangeLog b/ChangeLog index 1c3d8fa2..76b02a1a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,10 +1,12 @@ -Version 1.0.34 +Version 1.0.34 (unreleased) -------------- New features: * Custom project roles with configurable restrictions * Duplicate a task to multiple projects during creation +* New automatic action: + - Close a task in a specific column when not moved during a given period Version 1.0.33 -------------- diff --git a/app/Action/TaskCloseNoActivityColumn.php b/app/Action/TaskCloseNoActivityColumn.php index b2ee5224..e97aaac6 100644 --- a/app/Action/TaskCloseNoActivityColumn.php +++ b/app/Action/TaskCloseNoActivityColumn.php @@ -5,7 +5,7 @@ namespace Kanboard\Action; use Kanboard\Model\TaskModel; /** - * Close automatically a task after inactive and in an defined column + * Close automatically a task after inactive and in a defined column * * @package Kanboard\Action * @author Frederic Guillot diff --git a/app/Action/TaskCloseNotMovedColumn.php b/app/Action/TaskCloseNotMovedColumn.php new file mode 100644 index 00000000..cee5256b --- /dev/null +++ b/app/Action/TaskCloseNotMovedColumn.php @@ -0,0 +1,96 @@ + t('Duration in days'), + 'column_id' => t('Column') + ); + } + + /** + * Get the required parameter for the event + * + * @access public + * @return string[] + */ + public function getEventRequiredParameters() + { + return array('tasks'); + } + + /** + * Execute the action (close the task) + * + * @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(); + $max = $this->getParam('duration') * 86400; + + foreach ($data['tasks'] as $task) { + $duration = time() - $task['date_moved']; + + if ($duration > $max && $task['column_id'] == $this->getParam('column_id')) { + $results[] = $this->taskStatusModel->close($task['id']); + } + } + + 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 d46562b7..81f2b39e 100644 --- a/app/ServiceProvider/ActionProvider.php +++ b/app/ServiceProvider/ActionProvider.php @@ -2,12 +2,12 @@ namespace Kanboard\ServiceProvider; +use Pimple\Container; +use Pimple\ServiceProviderInterface; use Kanboard\Action\TaskAssignColorPriority; use Kanboard\Action\TaskAssignDueDateOnCreation; use Kanboard\Action\TaskMoveColumnClosed; use Kanboard\Action\TaskMoveColumnNotMovedPeriod; -use Pimple\Container; -use Pimple\ServiceProviderInterface; use Kanboard\Core\Action\ActionManager; use Kanboard\Action\CommentCreation; use Kanboard\Action\CommentCreationMoveTaskColumn; @@ -36,6 +36,7 @@ use Kanboard\Action\TaskOpen; use Kanboard\Action\TaskUpdateStartDate; use Kanboard\Action\TaskCloseNoActivity; use Kanboard\Action\TaskCloseNoActivityColumn; +use Kanboard\Action\TaskCloseNotMovedColumn; use Kanboard\Action\TaskAssignColorSwimlane; use Kanboard\Action\TaskAssignPrioritySwimlane; @@ -75,6 +76,7 @@ class ActionProvider implements ServiceProviderInterface $container['actionManager']->register(new TaskCloseColumn($container)); $container['actionManager']->register(new TaskCloseNoActivity($container)); $container['actionManager']->register(new TaskCloseNoActivityColumn($container)); + $container['actionManager']->register(new TaskCloseNotMovedColumn($container)); $container['actionManager']->register(new TaskCreation($container)); $container['actionManager']->register(new TaskDuplicateAnotherProject($container)); $container['actionManager']->register(new TaskEmail($container)); diff --git a/tests/units/Action/TaskCloseNotMovedColumnTest.php b/tests/units/Action/TaskCloseNotMovedColumnTest.php new file mode 100644 index 00000000..63aedee6 --- /dev/null +++ b/tests/units/Action/TaskCloseNotMovedColumnTest.php @@ -0,0 +1,49 @@ +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'))); + $this->assertEquals(2, $taskCreationModel->create(array('project_id' => 1, 'title' => 'test'))); + $this->assertEquals(3, $taskCreationModel->create(array('project_id' => 1, 'title' => 'test', 'column_id' => 2))); + + $this->container['db']->table(TaskModel::TABLE)->in('id', array(1, 3))->update(array('date_moved' => strtotime('-10days'))); + + $tasks = $taskFinderModel->getAll(1); + $event = new TaskListEvent(array('tasks' => $tasks, 'project_id' => 1)); + + $action = new TaskCloseNotMovedColumn($this->container); + $action->setProjectId(1); + $action->setParam('duration', 2); + $action->setParam('column_id', 2); + + $this->assertTrue($action->execute($event, TaskModel::EVENT_DAILY_CRONJOB)); + + $task = $taskFinderModel->getById(1); + $this->assertNotEmpty($task); + $this->assertEquals(1, $task['is_active']); + + $task = $taskFinderModel->getById(2); + $this->assertNotEmpty($task); + $this->assertEquals(1, $task['is_active']); + + $task = $taskFinderModel->getById(3); + $this->assertNotEmpty($task); + $this->assertEquals(0, $task['is_active']); + } +} -- cgit v1.2.3