From 5aed91c3212bc0dbb2e19f6ec8cab4d91cbb34c0 Mon Sep 17 00:00:00 2001 From: Bjoern Schiessle Date: Mon, 21 Nov 2016 09:07:50 +0100 Subject: [PATCH] small fixes Signed-off-by: Bjoern Schiessle --- settings/Application.php | 6 ++++-- settings/Controller/UsersController.php | 5 +---- settings/js/federationscopemenu.js | 4 ++-- 3 files changed, 7 insertions(+), 8 deletions(-) diff --git a/settings/Application.php b/settings/Application.php index 2f9b786d43..ca2b2b91f7 100644 --- a/settings/Application.php +++ b/settings/Application.php @@ -96,16 +96,18 @@ class Application extends App { $c->query('Logger') ); }); + $container->registerService('AppSettingsController', function(IContainer $c) { return new AppSettingsController( $c->query('AppName'), $c->query('Request'), $c->query('L10N'), $c->query('Config'), - $c->query('ICacheFactory'), $c->query('INavigationManager'), $c->query('IAppManager'), - $c->query('OcsClient') + $c->query('CategoryFetcher'), + $c->query('AppFetcher'), + \OC::$server->getL10NFactory() ); }); $container->registerService('AuthSettingsController', function(IContainer $c) { diff --git a/settings/Controller/UsersController.php b/settings/Controller/UsersController.php index 77d65877ba..58cc59f984 100644 --- a/settings/Controller/UsersController.php +++ b/settings/Controller/UsersController.php @@ -63,7 +63,7 @@ class UsersController extends Controller { private $isAdmin; /** @var IUserManager */ private $userManager; - /** @var \OC\Group\Manager */ + /** @var IGroupManager */ private $groupManager; /** @var IConfig */ private $config; @@ -671,9 +671,6 @@ class UsersController extends Controller { * @PasswordConfirmationRequired * @todo merge into saveUserSettings * - * @NoAdminRequired - * @PasswordConfirmationRequired - * * @param string $username * @param string $displayName * @return DataResponse diff --git a/settings/js/federationscopemenu.js b/settings/js/federationscopemenu.js index 36b720e8c4..066f648c10 100644 --- a/settings/js/federationscopemenu.js +++ b/settings/js/federationscopemenu.js @@ -46,8 +46,8 @@ this._scopes = [ { name: 'private', - displayName: (this.field == 'avatar' || this.field == 'displayname') ? t('core', 'Local') : t('core', 'Private'), - tooltip: (this.field == 'avatar' || this.field == 'displayname') ? t('core', 'Only visible to local users') : t('core', 'Only visible to you'), + displayName: (this.field === 'avatar' || this.field === 'displayname') ? t('core', 'Local') : t('core', 'Private'), + tooltip: (this.field === 'avatar' || this.field === 'displayname') ? t('core', 'Only visible to local users') : t('core', 'Only visible to you'), icon: OC.imagePath('core', 'actions/password'), active: false },