Provide PhabricatorSavedQuery to renderResultsList()
Summary: This allows the SavedQuery to modify what the result list looks like (e.g., include display flags and similar). Test Plan: Looked at some ApplicationSearch apps. Reviewers: btrahan Reviewed By: btrahan CC: aran Maniphest Tasks: T2625 Differential Revision: https://secure.phabricator.com/D6346
This commit is contained in:
@@ -22,7 +22,9 @@ final class PhabricatorConduitListController
|
|||||||
return $this->delegateToController($controller);
|
return $this->delegateToController($controller);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function renderResultsList(array $methods) {
|
public function renderResultsList(
|
||||||
|
array $methods,
|
||||||
|
PhabricatorSavedQuery $query) {
|
||||||
assert_instances_of($methods, 'ConduitAPIMethod');
|
assert_instances_of($methods, 'ConduitAPIMethod');
|
||||||
|
|
||||||
$viewer = $this->getRequest()->getUser();
|
$viewer = $this->getRequest()->getUser();
|
||||||
|
|||||||
@@ -23,7 +23,9 @@ final class DivinerAtomListController extends DivinerController
|
|||||||
return $this->delegateToController($controller);
|
return $this->delegateToController($controller);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function renderResultsList(array $symbols) {
|
public function renderResultsList(
|
||||||
|
array $symbols,
|
||||||
|
PhabricatorSavedQuery $query) {
|
||||||
return $this->renderAtomList($symbols);
|
return $this->renderAtomList($symbols);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -23,7 +23,10 @@ final class PhabricatorFileListController extends PhabricatorFileController
|
|||||||
return $this->delegateToController($controller);
|
return $this->delegateToController($controller);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function renderResultsList(array $files) {
|
public function renderResultsList(
|
||||||
|
array $files,
|
||||||
|
PhabricatorSavedQuery $query) {
|
||||||
|
|
||||||
assert_instances_of($files, 'PhabricatorFile');
|
assert_instances_of($files, 'PhabricatorFile');
|
||||||
|
|
||||||
$request = $this->getRequest();
|
$request = $this->getRequest();
|
||||||
|
|||||||
@@ -23,7 +23,10 @@ final class PhabricatorMacroListController extends PhabricatorMacroController
|
|||||||
return $this->delegateToController($controller);
|
return $this->delegateToController($controller);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function renderResultsList(array $macros) {
|
public function renderResultsList(
|
||||||
|
array $macros,
|
||||||
|
PhabricatorSavedQuery $query) {
|
||||||
|
|
||||||
assert_instances_of($macros, 'PhabricatorFileImageMacro');
|
assert_instances_of($macros, 'PhabricatorFileImageMacro');
|
||||||
$viewer = $this->getRequest()->getUser();
|
$viewer = $this->getRequest()->getUser();
|
||||||
|
|
||||||
|
|||||||
@@ -23,7 +23,9 @@ final class PhabricatorPasteListController extends PhabricatorPasteController
|
|||||||
return $this->delegateToController($controller);
|
return $this->delegateToController($controller);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function renderResultsList(array $pastes) {
|
public function renderResultsList(
|
||||||
|
array $pastes,
|
||||||
|
PhabricatorSavedQuery $query) {
|
||||||
assert_instances_of($pastes, 'PhabricatorPaste');
|
assert_instances_of($pastes, 'PhabricatorPaste');
|
||||||
|
|
||||||
$user = $this->getRequest()->getUser();
|
$user = $this->getRequest()->getUser();
|
||||||
|
|||||||
@@ -27,7 +27,10 @@ final class PhabricatorPeopleListController extends PhabricatorPeopleController
|
|||||||
return $this->delegateToController($controller);
|
return $this->delegateToController($controller);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function renderResultsList(array $users) {
|
public function renderResultsList(
|
||||||
|
array $users,
|
||||||
|
PhabricatorSavedQuery $query) {
|
||||||
|
|
||||||
assert_instances_of($users, 'PhabricatorUser');
|
assert_instances_of($users, 'PhabricatorUser');
|
||||||
|
|
||||||
$request = $this->getRequest();
|
$request = $this->getRequest();
|
||||||
|
|||||||
@@ -177,7 +177,7 @@ final class PhabricatorApplicationSearchController
|
|||||||
$objects = $query->setViewer($request->getUser())
|
$objects = $query->setViewer($request->getUser())
|
||||||
->executeWithCursorPager($pager);
|
->executeWithCursorPager($pager);
|
||||||
|
|
||||||
$list = $parent->renderResultsList($objects);
|
$list = $parent->renderResultsList($objects, $saved_query);
|
||||||
|
|
||||||
$nav->appendChild($list);
|
$nav->appendChild($list);
|
||||||
|
|
||||||
|
|||||||
@@ -2,6 +2,8 @@
|
|||||||
|
|
||||||
interface PhabricatorApplicationSearchResultsControllerInterface {
|
interface PhabricatorApplicationSearchResultsControllerInterface {
|
||||||
|
|
||||||
public function renderResultsList(array $items);
|
public function renderResultsList(
|
||||||
|
array $items,
|
||||||
|
PhabricatorSavedQuery $query);
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user