Dedupe DIRECTORY w/ Directory tab in directory header
Summary: the tab is a bit silly right next to DIRECTORY Test Plan: viewed phabricator with an admin account * looks good on load * clicked Categories and Items; looked good viewed phabricator with a non-admin account * looks good on load * nothing else to click in the header Reviewers: epriestley Reviewed By: epriestley CC: aran, epriestley Differential Revision: 1131
This commit is contained in:
		| @@ -30,15 +30,8 @@ abstract class PhabricatorDirectoryController extends PhabricatorController { | |||||||
|     $page->setBaseURI('/'); |     $page->setBaseURI('/'); | ||||||
|     $page->setTitle(idx($data, 'title')); |     $page->setTitle(idx($data, 'title')); | ||||||
|  |  | ||||||
|     $tabs = array( |  | ||||||
|       'directory' => array( |  | ||||||
|         'href' => '/', |  | ||||||
|         'name' => 'Directory', |  | ||||||
|       ), |  | ||||||
|     ); |  | ||||||
|  |  | ||||||
|     if ($this->getRequest()->getUser()->getIsAdmin()) { |     if ($this->getRequest()->getUser()->getIsAdmin()) { | ||||||
|       $tabs += array( |       $tabs = array( | ||||||
|         'categories' => array( |         'categories' => array( | ||||||
|           'href' => '/directory/category/', |           'href' => '/directory/category/', | ||||||
|           'name' => 'Categories', |           'name' => 'Categories', | ||||||
| @@ -48,6 +41,8 @@ abstract class PhabricatorDirectoryController extends PhabricatorController { | |||||||
|           'name' => 'Items', |           'name' => 'Items', | ||||||
|         ), |         ), | ||||||
|       ); |       ); | ||||||
|  |     } else { | ||||||
|  |       $tabs = array(); | ||||||
|     } |     } | ||||||
|  |  | ||||||
|     $page->setTabs( |     $page->setTabs( | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Bob Trahan
					Bob Trahan