diff options
author | Frederic Guillot <fred@kanboard.net> | 2015-02-07 21:44:51 -0500 |
---|---|---|
committer | Frederic Guillot <fred@kanboard.net> | 2015-02-07 21:44:51 -0500 |
commit | c9cc82eec898654846c3c7e409269166cc5dba7c (patch) | |
tree | 4ddc6479ff5447627f07a40f37024420e2bd0a71 /app/Model/Subtask.php | |
parent | 2d890cbc712371f17ba4bbceb02af3c5ba04e6da (diff) |
Update subtask spent based on the time tracking
Diffstat (limited to 'app/Model/Subtask.php')
-rw-r--r-- | app/Model/Subtask.php | 39 |
1 files changed, 1 insertions, 38 deletions
diff --git a/app/Model/Subtask.php b/app/Model/Subtask.php index 1d5ed566..048594bd 100644 --- a/app/Model/Subtask.php +++ b/app/Model/Subtask.php @@ -176,9 +176,6 @@ class Subtask extends Base $subtask_id = $this->persist(self::TABLE, $values); if ($subtask_id) { - - $this->updateTaskTimeTracking($values['task_id']); - $this->container['dispatcher']->dispatch( self::EVENT_CREATE, new SubtaskEvent(array('id' => $subtask_id) + $values) @@ -202,10 +199,6 @@ class Subtask extends Base if ($result) { - if (isset($values['task_id'])) { - $this->updateTaskTimeTracking($values['task_id']); - } - $this->container['dispatcher']->dispatch( self::EVENT_UPDATE, new SubtaskEvent($values) @@ -231,6 +224,7 @@ class Subtask extends Base $values = array( 'id' => $subtask['id'], 'status' => ($subtask['status'] + 1) % 3, + 'task_id' => $subtask['task_id'], ); return $this->update($values); @@ -268,37 +262,6 @@ class Subtask extends Base } /** - * Update task time tracking based on subtasks time tracking - * - * @access public - * @param integer $task_id Task id - * @return bool - */ - public function updateTaskTimeTracking($task_id) - { - $result = $this->db - ->table(self::TABLE) - ->eq('task_id', $task_id) - ->columns( - 'SUM(time_spent) AS total_spent', - 'SUM(time_estimated) AS total_estimated' - ) - ->findOne(); - - if (empty($result['total_spent']) && empty($result['total_estimated'])) { - return true; - } - - return $this->db - ->table(Task::TABLE) - ->eq('id', $task_id) - ->update(array( - 'time_spent' => $result['total_spent'], - 'time_estimated' => $result['total_estimated'], - )); - } - - /** * Remove * * @access public |