Skip to content

Commit 1188397

Browse files
authored
Merge pull request #1168 from nextcloud/chore/noid/fix-deprecated-oc_user-useBackend
Replace deprecated OC_User::useBackend with OCP\IUserManager::registerBackend
2 parents 0b6b2b1 + 2d875de commit 1188397

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

lib/AppInfo/Application.php

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010

1111
use Exception;
1212
use OC_App;
13-
use OC_User;
1413
use OCA\Files\Event\LoadAdditionalScriptsEvent;
1514
use OCA\UserOIDC\Db\ProviderMapper;
1615
use OCA\UserOIDC\Event\ExchangedTokenRequestedEvent;
@@ -53,11 +52,8 @@ public function register(IRegistrationContext $context): void {
5352

5453
/* Register our own user backend */
5554
$this->backend = $this->getContainer()->get(Backend::class);
56-
// this was done before but OC_User::useBackend calls OC::$server->getUserManager()->registerBackend anyway
57-
// so the backend was registered twice, leading to wrong user count (double)
58-
// $userManager->registerBackend($this->backend);
59-
// TODO check if it can be replaced by $userManager->registerBackend($this->backend); in our case
60-
OC_User::useBackend($this->backend);
55+
// see https://docs.nextcloud.com/server/latest/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_32.html#id3
56+
$userManager->registerBackend($this->backend);
6157

6258
$context->registerEventListener(LoadAdditionalScriptsEvent::class, TimezoneHandlingListener::class);
6359
$context->registerEventListener(ExchangedTokenRequestedEvent::class, ExchangedTokenRequestedListener::class);

0 commit comments

Comments
 (0)