revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-20 04:20:55 +01:00
Code Issues Releases Wiki Activity

Fix PhabricatorWorkerTriggerQuery method visibility

Browse source 
Summary: I got these wrong and the test didn't trigger for some reason that I haven't looked into.

Test Plan: `arc unit --everything`

Reviewers: hach-que, btrahan

Reviewed By: btrahan

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D11453
This commit is contained in:
epriestley 2015-01-22 16:10:08 -08:00
parent adf209e655
commit ed2a5a9a34
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php
View file

@ -67,13 +67,13 @@ final class PhabricatorWorkerTriggerQuery
return $this;
}
public function nextPage(array $page) {
protected function nextPage(array $page) {
// NOTE: We don't implement paging because we don't currently ever need
// it and paging ORDER_EXCUTION is a hassle.
throw new PhutilMethodNotImplementedException();
}
public function loadPage() {
protected function loadPage() {
$task_table = new PhabricatorWorkerTrigger();
$conn_r = $task_table->establishConnection('r');