Aerex/Taskwarrior
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

refactor validation

Browse Source
This commit is contained in:
DavidBadura 2015-04-03 14:39:19 +00:00
parent c026f2b954
commit 014c986103
1 changed files with 13 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/TaskManager.php
View File

@ -44,6 +44,8 @@ class TaskManager
*/ */
public function save(Task $task) public function save(Task $task)
{ {
$this->validate($task);
if (!$task->getUuid()) { if (!$task->getUuid()) {
$this->add($task); $this->add($task);
} else { } else {
@ -209,10 +211,6 @@ class TaskManager
*/ */
private function edit(Task $task) private function edit(Task $task)
{ {
if ($task->isReccuring() && !$task->getRecurring()) {
throw new TaskwarriorException('You cannot remove the recurrence from a recurring task.');
}
$this->taskwarrior->modify( $this->taskwarrior->modify(
[ [
'description' => $task->getDescription(), 'description' => $task->getDescription(),
@ -228,6 +226,17 @@ class TaskManager
); );
} }
/**
* @param Task $task
* @throws TaskwarriorException
*/
private function validate(Task $task)
{
if ($task->isReccuring() && !$task->getRecurring()) {
throw new TaskwarriorException('You cannot remove the recurrence from a recurring task.');
}
}
/** /**
* @param Task $old * @param Task $old
* @param Task $new * @param Task $new