diff --git a/settings/Controller/AdminSettingsController.php b/settings/Controller/AdminSettingsController.php index 0114ee73d9..e2ad577024 100644 --- a/settings/Controller/AdminSettingsController.php +++ b/settings/Controller/AdminSettingsController.php @@ -38,9 +38,6 @@ use OCP\Template; class AdminSettingsController extends Controller { use CommonSettingsTrait; - /** @var INavigationManager */ - private $navigationManager; - /** * @param string $appName * @param IRequest $request @@ -65,7 +62,6 @@ class AdminSettingsController extends Controller { * @NoCSRFRequired */ public function index($section) { - $this->navigationManager->setActiveEntry('admin'); return $this->getIndexResponse('admin', $section); } diff --git a/settings/Controller/CommonSettingsTrait.php b/settings/Controller/CommonSettingsTrait.php index 82d675531a..3d314d9ed5 100644 --- a/settings/Controller/CommonSettingsTrait.php +++ b/settings/Controller/CommonSettingsTrait.php @@ -25,6 +25,7 @@ namespace OC\Settings\Controller; use OCP\AppFramework\Http\TemplateResponse; +use OCP\INavigationManager; use OCP\Settings\IManager as ISettingsManager; use OCP\Settings\IIconSection; use OCP\Settings\ISettings; @@ -33,6 +34,9 @@ trait CommonSettingsTrait { /** @var ISettingsManager */ private $settingsManager; + /** @var INavigationManager */ + private $navigationManager; + /** * @param string $currentSection * @return array @@ -116,6 +120,7 @@ trait CommonSettingsTrait { } private function getIndexResponse($type, $section) { + $this->navigationManager->setActiveEntry('settings'); $templateParams = []; $templateParams = array_merge($templateParams, $this->getNavigationParameters($type, $section)); $templateParams = array_merge($templateParams, $this->getSettings($section)); diff --git a/settings/Controller/PersonalSettingsController.php b/settings/Controller/PersonalSettingsController.php index ad4eb69c32..9ec7ce7264 100644 --- a/settings/Controller/PersonalSettingsController.php +++ b/settings/Controller/PersonalSettingsController.php @@ -34,9 +34,6 @@ use OCP\Template; class PersonalSettingsController extends Controller { use CommonSettingsTrait; - /** @var INavigationManager */ - private $navigationManager; - public function __construct( $appName, IRequest $request, @@ -57,7 +54,6 @@ class PersonalSettingsController extends Controller { * @NoSubadminRequired */ public function index($section) { - $this->navigationManager->setActiveEntry('personal'); return $this->getIndexResponse('personal', $section); }