From ed2a5a9a34a73804f8349f891e7602661d7cb70a Mon Sep 17 00:00:00 2001 From: epriestley Date: Thu, 22 Jan 2015 16:10:08 -0800 Subject: [PATCH] Fix PhabricatorWorkerTriggerQuery method visibility 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 --- .../daemon/workers/query/PhabricatorWorkerTriggerQuery.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php b/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php index 2524a2e217..679a5236a7 100644 --- a/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php +++ b/src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php @@ -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');