Increase visibility of PhabricatorController::buildApplicationMenu methods
				
					
				
			Summary: Ref T6822. This method needs to be `public` because it is called from `PhabricatorApplicationSearchController::buildApplicationMenu()`. Test Plan: I wouldn't expect //increasing// method visibility to break anything. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin, epriestley Maniphest Tasks: T6822 Differential Revision: https://secure.phabricator.com/D11416
This commit is contained in:
		| @@ -18,7 +18,7 @@ abstract class PhabricatorAuditController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView()->getMenu(); |     return $this->buildSideNavView()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -16,7 +16,7 @@ abstract class PhabricatorAuthProviderConfigController | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView($for_app = true)->getMenu(); |     return $this->buildSideNavView($for_app = true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -425,7 +425,7 @@ abstract class PhabricatorController extends AphrontController { | |||||||
|       array_filter($phids)); |       array_filter($phids)); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return null; |     return null; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -20,7 +20,7 @@ abstract class PhabricatorCalendarController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView()->getMenu(); |     return $this->buildSideNavView()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -20,7 +20,7 @@ abstract class PhabricatorConduitController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView()->getMenu(); |     return $this->buildSideNavView()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -26,7 +26,7 @@ abstract class PhabricatorConfigController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(null, true)->getMenu(); |     return $this->buildSideNavView(null, true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -4,7 +4,7 @@ abstract class ConpherenceController extends PhabricatorController { | |||||||
|  |  | ||||||
|   private $conpherences; |   private $conpherences; | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     $nav = new PHUIListView(); |     $nav = new PHUIListView(); | ||||||
|  |  | ||||||
|     $nav->newLink( |     $nav->newLink( | ||||||
|   | |||||||
| @@ -21,7 +21,7 @@ abstract class PhabricatorCountdownController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView($for_app = true)->getMenu(); |     return $this->buildSideNavView($for_app = true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -29,7 +29,7 @@ abstract class DifferentialController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -7,7 +7,7 @@ abstract class DivinerController extends PhabricatorController { | |||||||
|     return AphrontSideNavFilterView::newFromMenu($menu); |     return AphrontSideNavFilterView::newFromMenu($menu); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildMenu(); |     return $this->buildMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -4,7 +4,7 @@ abstract class DrydockController extends PhabricatorController { | |||||||
|  |  | ||||||
|   public abstract function buildSideNavView(); |   public abstract function buildSideNavView(); | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView()->getMenu(); |     return $this->buildSideNavView()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -37,7 +37,7 @@ abstract class PhabricatorFeedController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView()->getMenu(); |     return $this->buildSideNavView()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -7,7 +7,7 @@ abstract class PhabricatorFileController extends PhabricatorController { | |||||||
|     return AphrontSideNavFilterView::newFromMenu($menu); |     return AphrontSideNavFilterView::newFromMenu($menu); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildMenu($for_devices = true); |     return $this->buildMenu($for_devices = true); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -39,7 +39,7 @@ final class HarbormasterBuildableListController extends HarbormasterController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -40,7 +40,7 @@ final class HarbormasterPlanListController extends HarbormasterPlanController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -16,7 +16,7 @@ abstract class HeraldController extends PhabricatorController { | |||||||
|     return $response->setContent($page->render()); |     return $response->setContent($page->render()); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -22,7 +22,7 @@ abstract class LegalpadController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNav(true)->getMenu(); |     return $this->buildSideNav(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -20,7 +20,7 @@ abstract class PhabricatorMacroController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView($for_app = true)->getMenu(); |     return $this->buildSideNavView($for_app = true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -21,7 +21,7 @@ abstract class PhabricatorMailingListsController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -17,7 +17,7 @@ abstract class PhabricatorApplicationsController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -58,7 +58,7 @@ abstract class PhabricatorOwnersController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView()->getMenu(); |     return $this->buildSideNavView()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -21,7 +21,7 @@ abstract class PassphraseController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -21,7 +21,7 @@ abstract class PhabricatorPasteController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -28,7 +28,7 @@ abstract class PhabricatorPeopleController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView()->getMenu(); |     return $this->buildSideNavView()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -103,7 +103,7 @@ abstract class PhameController extends PhabricatorController { | |||||||
|     return $stories; |     return $stories; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->renderSideNavFilterView()->getMenu(); |     return $this->renderSideNavFilterView()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -33,7 +33,7 @@ abstract class PholioController extends PhabricatorController { | |||||||
|     return $crumbs; |     return $crumbs; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -22,7 +22,7 @@ abstract class PhrictionController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -34,7 +34,7 @@ abstract class ReleephController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -355,7 +355,7 @@ final class PhabricatorApplicationSearchController | |||||||
|       )); |       )); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->getDelegatingController()->buildApplicationMenu(); |     return $this->getDelegatingController()->buildApplicationMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -151,7 +151,7 @@ final class PhabricatorSettingsMainController | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     $panels = $this->buildPanels(); |     $panels = $this->buildPanels(); | ||||||
|     return $this->renderSideNav($panels)->getMenu(); |     return $this->renderSideNav($panels)->getMenu(); | ||||||
|   } |   } | ||||||
|   | |||||||
| @@ -22,7 +22,7 @@ abstract class PhabricatorSlowvoteController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNavView(true)->getMenu(); |     return $this->buildSideNavView(true)->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
| @@ -13,7 +13,7 @@ abstract class PhabricatorTokenController extends PhabricatorController { | |||||||
|     return $nav; |     return $nav; | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   protected function buildApplicationMenu() { |   public function buildApplicationMenu() { | ||||||
|     return $this->buildSideNav()->getMenu(); |     return $this->buildSideNav()->getMenu(); | ||||||
|   } |   } | ||||||
|  |  | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Joshua Spence
					Joshua Spence