Skip to content

Commit 36524ea

Browse files
Migrate settigns to the PSR container
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
1 parent 89fe541 commit 36524ea

1 file changed

Lines changed: 21 additions & 12 deletions

File tree

apps/settings/lib/AppInfo/Application.php

Lines changed: 21 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -51,12 +51,13 @@
5151
use OCP\AppFramework\Bootstrap\IBootstrap;
5252
use OCP\AppFramework\Bootstrap\IRegistrationContext;
5353
use OCP\Defaults;
54-
use OCP\IContainer;
5554
use OCP\IGroup;
5655
use OCP\ILogger;
56+
use OCP\IServerContainer;
5757
use OCP\IUser;
5858
use OCP\Settings\IManager;
5959
use OCP\Util;
60+
use Psr\Container\ContainerInterface;
6061
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
6162
use Symfony\Component\EventDispatcher\GenericEvent;
6263

@@ -83,30 +84,38 @@ public function register(IRegistrationContext $context): void {
8384
return \OC_User::isAdminUser(\OC_User::getUser());
8485
});
8586
/** FIXME: Remove once OC_SubAdmin is non-static and mockable */
86-
$context->registerService('isSubAdmin', function (IContainer $c) {
87+
$context->registerService('isSubAdmin', function () {
8788
$userObject = \OC::$server->getUserSession()->getUser();
8889
$isSubAdmin = false;
8990
if ($userObject !== null) {
9091
$isSubAdmin = \OC::$server->getGroupManager()->getSubAdmin()->isSubAdmin($userObject);
9192
}
9293
return $isSubAdmin;
9394
});
94-
$context->registerService('userCertificateManager', function (IContainer $c) {
95-
return $c->query('ServerContainer')->getCertificateManager();
95+
$context->registerService('userCertificateManager', function (ContainerInterface $c) {
96+
/** @var IServerContainer $serverContainer */
97+
$serverContainer = $c->get(IServerContainer::class);
98+
return $serverContainer->getCertificateManager();
9699
}, false);
97-
$context->registerService('systemCertificateManager', function (IContainer $c) {
98-
return $c->query('ServerContainer')->getCertificateManager(null);
100+
$context->registerService('systemCertificateManager', function (ContainerInterface $c) {
101+
/** @var IServerContainer $serverContainer */
102+
$serverContainer = $c->query('ServerContainer');
103+
return $serverContainer->getCertificateManager(null);
99104
}, false);
100-
$context->registerService(IProvider::class, function (IContainer $c) {
101-
return $c->query('ServerContainer')->query(IProvider::class);
105+
$context->registerService(IProvider::class, function (ContainerInterface $c) {
106+
/** @var IServerContainer $serverContainer */
107+
$serverContainer = $c->query(IServerContainer::class);
108+
return $serverContainer->query(IProvider::class);
102109
});
103-
$context->registerService(IManager::class, function (IContainer $c) {
104-
return $c->query('ServerContainer')->getSettingsManager();
110+
$context->registerService(IManager::class, function (ContainerInterface $c) {
111+
/** @var IServerContainer $serverContainer */
112+
$serverContainer = $c->query(IServerContainer::class);
113+
return $serverContainer->getSettingsManager();
105114
});
106115

107-
$context->registerService(NewUserMailHelper::class, function (IContainer $c) {
116+
$context->registerService(NewUserMailHelper::class, function (ContainerInterface $c) {
108117
/** @var Server $server */
109-
$server = $c->query('ServerContainer');
118+
$server = $c->query(IServerContainer::class);
110119
/** @var Defaults $defaults */
111120
$defaults = $server->query(Defaults::class);
112121

0 commit comments

Comments
 (0)