Revert lazy use of userId in ConfigService

Signed-off-by: Julius Härtl <jus@bitgrid.net>
nextcloud · May 13, 2022 · ce7da62 · ce7da62
1 parent 3a783a7
commit ce7da62
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/lib/Service/ConfigService.php b/lib/Service/ConfigService.php
@@ -32,6 +32,7 @@
 use OCP\IConfig;
 use OCP\IGroup;
 use OCP\IGroupManager;
+use OCP\IUserSession;
 
 class ConfigService {
 	public const SETTING_BOARD_NOTIFICATION_DUE_OFF = 'off';
@@ -40,20 +41,25 @@ class ConfigService {
 	public const SETTING_BOARD_NOTIFICATION_DUE_DEFAULT = self::SETTING_BOARD_NOTIFICATION_DUE_ASSIGNED;
 
 	private IConfig $config;
-	private ?string $userId;
+	private ?string $userId = null;
 	private IGroupManager $groupManager;
 
 	public function __construct(
 		IConfig $config,
-		IGroupManager $groupManager,
-		?string $userId
+		IGroupManager $groupManager
 	) {
 		$this->groupManager = $groupManager;
 		$this->config = $config;
-		$this->userId = $userId;
 	}
 
 	public function getUserId(): ?string {
+		if (!$this->userId) {
+			// We cannot use DI for the userId or UserSession as the ConfigService
+			// is initiated too early before the session is actually loaded
+			$user = \OCP\Server::get(IUserSession::class)->getUser();
+			$this->userId = $user ? $user->getUID() : null;
+		}
+
 		return $this->userId;
 	}
 

diff --git a/tests/integration/app/AppTest.php b/tests/integration/app/AppTest.php
@@ -34,7 +34,7 @@ class AppTest extends TestCase {
 
 	public function setUp(): void {
 		parent::setUp();
-		$this->app = new \OCA\Deck\AppInfo\Application();
+		$this->app = \OCP\Server::get(\OCA\Deck\AppInfo\Application::class);
 		$this->container = $this->app->getContainer();
 	}