change filter

This commit is contained in:
DavidBadura 2015-02-06 09:49:28 +01:00
parent c8cdba7909
commit a12ea63848
2 changed files with 13 additions and 13 deletions

View File

@ -62,7 +62,7 @@ class Taskwarrior
return $this->tasks[$uuid]; return $this->tasks[$uuid];
} }
$tasks = $this->filter($uuid); $tasks = $this->filterAll($uuid);
if (count($tasks) == 0) { if (count($tasks) == 0) {
return null; return null;
@ -79,7 +79,7 @@ class Taskwarrior
* @param $filter * @param $filter
* @return Task[] * @return Task[]
*/ */
public function filter($filter = '') public function filterAll($filter = '')
{ {
$result = $this->export($filter); $result = $this->export($filter);
@ -101,9 +101,9 @@ class Taskwarrior
* @param string $filter * @param string $filter
* @return Task[] * @return Task[]
*/ */
public function filterPending($filter = '') public function filter($filter = '')
{ {
$tasks = $this->filter($filter . ' status:pending'); $tasks = $this->filterAll($filter . ' status:pending');
return $this->sort($tasks); return $this->sort($tasks);
} }

View File

@ -68,7 +68,7 @@ class TaskwarriorTest extends \PHPUnit_Framework_TestCase
$this->taskwarrior->save($task); $this->taskwarrior->save($task);
$result = $this->taskwarrior->filter($task->getUuid()); $result = $this->taskwarrior->filterAll($task->getUuid());
$this->assertSame($task, $result[0]); $this->assertSame($task, $result[0]);
} }
@ -132,7 +132,7 @@ class TaskwarriorTest extends \PHPUnit_Framework_TestCase
$this->assertEquals(Task::STATUS_PENDING, $result->getStatus()); $this->assertEquals(Task::STATUS_PENDING, $result->getStatus());
$this->assertTrue($result->isPending()); $this->assertTrue($result->isPending());
$this->assertCount(1, $this->taskwarrior->filterPending()); $this->assertCount(1, $this->taskwarrior->filter());
} }
public function testDelete() public function testDelete()
@ -143,14 +143,14 @@ class TaskwarriorTest extends \PHPUnit_Framework_TestCase
$this->assertCount(0, $this->taskwarrior->filter()); $this->assertCount(0, $this->taskwarrior->filter());
$this->taskwarrior->save($task1); $this->taskwarrior->save($task1);
$this->assertCount(1, $this->taskwarrior->filterAll());
$this->assertCount(1, $this->taskwarrior->filter()); $this->assertCount(1, $this->taskwarrior->filter());
$this->assertCount(1, $this->taskwarrior->filterPending());
$this->assertFalse($task1->isDeleted()); $this->assertFalse($task1->isDeleted());
$this->assertEquals(Task::STATUS_PENDING, $task1->getStatus()); $this->assertEquals(Task::STATUS_PENDING, $task1->getStatus());
$this->taskwarrior->delete($task1); $this->taskwarrior->delete($task1);
$this->assertCount(1, $this->taskwarrior->filter()); $this->assertCount(1, $this->taskwarrior->filterAll());
$this->assertCount(0, $this->taskwarrior->filterPending()); $this->assertCount(0, $this->taskwarrior->filter());
$this->assertTrue($task1->isDeleted()); $this->assertTrue($task1->isDeleted());
$this->assertEquals(Task::STATUS_DELETED, $task1->getStatus()); $this->assertEquals(Task::STATUS_DELETED, $task1->getStatus());
} }
@ -163,14 +163,14 @@ class TaskwarriorTest extends \PHPUnit_Framework_TestCase
$this->assertCount(0, $this->taskwarrior->filter()); $this->assertCount(0, $this->taskwarrior->filter());
$this->taskwarrior->save($task1); $this->taskwarrior->save($task1);
$this->assertCount(1, $this->taskwarrior->filterAll());
$this->assertCount(1, $this->taskwarrior->filter()); $this->assertCount(1, $this->taskwarrior->filter());
$this->assertCount(1, $this->taskwarrior->filterPending());
$this->assertFalse($task1->isCompleted()); $this->assertFalse($task1->isCompleted());
$this->assertEquals(Task::STATUS_PENDING, $task1->getStatus()); $this->assertEquals(Task::STATUS_PENDING, $task1->getStatus());
$this->taskwarrior->done($task1); $this->taskwarrior->done($task1);
$this->assertCount(1, $this->taskwarrior->filter()); $this->assertCount(1, $this->taskwarrior->filterAll());
$this->assertCount(0, $this->taskwarrior->filterPending()); $this->assertCount(0, $this->taskwarrior->filter());
$this->assertTrue($task1->isCompleted()); $this->assertTrue($task1->isCompleted());
$this->assertEquals(Task::STATUS_COMPLETED, $task1->getStatus()); $this->assertEquals(Task::STATUS_COMPLETED, $task1->getStatus());
} }
@ -264,7 +264,7 @@ class TaskwarriorTest extends \PHPUnit_Framework_TestCase
$this->assertEquals(12, $task2->getUrgency()); $this->assertEquals(12, $task2->getUrgency());
$this->assertEquals(0, $task3->getUrgency()); $this->assertEquals(0, $task3->getUrgency());
$tasks = $this->taskwarrior->filterPending(); $tasks = $this->taskwarrior->filter();
$this->assertEquals(array($task2, $task1, $task3), $tasks); $this->assertEquals(array($task2, $task1, $task3), $tasks);
} }