refactor: Some PHP8+ optimizations

lib/Activity/Provider.php

@@ -17,15 +17,10 @@
 
 class Provider implements IProvider {
 
-	/** @var L10nFactory */
-	private $l10n;
-
-	/** @var IURLGenerator */
-	private $urlGenerator;
-
-	public function __construct(L10nFactory $l10n, IURLGenerator $urlGenerator) {
-		$this->urlGenerator = $urlGenerator;
-		$this->l10n = $l10n;
+	public function __construct(
+		private L10nFactory $l10n,
+		private IURLGenerator $urlGenerator,
+	) {
 	}
 
 	public function parse($language, IEvent $event, ?IEvent $previousEvent = null): IEvent {

lib/Activity/Setting.php

@@ -14,11 +14,9 @@
 
 class Setting implements ISetting {
 
-	/** @var IL10N */
-	private $l10n;
-
-	public function __construct(IL10N $l10n) {
-		$this->l10n = $l10n;
+	public function __construct(
+		private IL10N $l10n,
+	) {
 	}
 
 	public function canChangeMail(): bool {

lib/Command/CleanUp.php

@@ -19,25 +19,13 @@
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 class CleanUp extends Command {
-	/** @var IDBConnection */
-	private $db;
-
-	/** @var IUserManager */
-	private $userManager;
-
-	/** @var TotpSecretMapper */
-	private $totpSecretMapper;
 
 	public function __construct(
-		IDBConnection $db,
-		IUserManager $userManager,
-		TotpSecretMapper $totpSecretMapper,
+		private IDBConnection $db,
+		private IUserManager $userManager,
+		private TotpSecretMapper $totpSecretMapper,
 	) {
 		parent::__construct();
-
-		$this->db = $db;
-		$this->userManager = $userManager;
-		$this->totpSecretMapper = $totpSecretMapper;
 	}
 
 	protected function configure(): void {

lib/Controller/SettingsController.php

@@ -21,28 +21,22 @@
 
 class SettingsController extends ALoginSetupController {
 
-	/** @var ITotp */
-	private $totp;
-
-	/** @var IUserSession */
-	private $userSession;
-
-	/** @var Defaults */
-	private $defaults;
-
-	public function __construct(string $appName, IRequest $request, IUserSession $userSession, ITotp $totp, Defaults $defaults) {
+	public function __construct(
+		string $appName,
+		IRequest $request,
+		private IUserSession $userSession,
+		private ITotp $totp,
+		private Defaults $defaults,
+	) {
 		parent::__construct($appName, $request);
-		$this->userSession = $userSession;
-		$this->totp = $totp;
-		$this->defaults = $defaults;
 	}
 
 	/**
 	 * @NoAdminRequired
 	 * @return JSONResponse
 	 */
 	public function state(): JSONResponse {
-		$user = $this->userSession->getUser();
+		$user = $this->userSession?->getUser();
 		if (is_null($user)) {
 			throw new \Exception('user not available');
 		}
@@ -59,7 +53,7 @@ public function state(): JSONResponse {
 	 * @param string|null $code for verification
 	 */
 	public function enable(int $state, ?string $code = null): JSONResponse {
-		$user = $this->userSession->getUser();
+		$user = $this->userSession?->getUser();
 		if (is_null($user)) {
 			throw new \Exception('user not available');
 		}
@@ -100,7 +94,7 @@ public function enable(int $state, ?string $code = null): JSONResponse {
 	 */
 	private function getSecretName(): string {
 		$productName = $this->defaults->getName();
-		$user = $this->userSession->getUser();
+		$user = $this->userSession?->getUser();
 		if ($user === null) {
 			throw new RuntimeException('No user in this context');
 		}

lib/Db/TotpSecretMapper.php

@@ -39,7 +39,7 @@ public function getSecret(IUser $user): TotpSecret {
 			->from('twofactor_totp_secrets')
 			->where($qb->expr()->eq('user_id', $qb->createNamedParameter($user->getUID())));
 		/** @var Statement $result */
-		$result = $qb->execute();
+		$result = $qb->executeQuery();
 
 		$row = $result->fetch();
 		$result->closeCursor();

lib/Event/StateChanged.php

@@ -14,29 +14,17 @@
 
 class StateChanged extends Event {
 
-	/** @var IUser */
-	private $user;
-
-	/** @var bool */
-	private $enabled;
-
-	public function __construct(IUser $user, bool $enabled) {
+	public function __construct(
+		private IUser $user,
+		private bool $enabled,
+	) {
 		parent::__construct();
-
-		$this->user = $user;
-		$this->enabled = $enabled;
 	}
 
-	/**
-	 * @return IUser
-	 */
 	public function getUser(): IUser {
 		return $this->user;
 	}
 
-	/**
-	 * @return bool
-	 */
 	public function isEnabled(): bool {
 		return $this->enabled;
 	}

lib/Listener/StateChangeActivity.php

@@ -20,11 +20,9 @@
  */
 class StateChangeActivity implements IEventListener {
 
-	/** @var ActivityManager */
-	private $activityManager;
-
-	public function __construct(ActivityManager $activityManager) {
-		$this->activityManager = $activityManager;
+	public function __construct(
+		private ActivityManager $activityManager,
+	) {
 	}
 
 	public function handle(Event $event): void {

lib/Listener/StateChangeRegistryUpdater.php

@@ -20,15 +20,10 @@
  */
 class StateChangeRegistryUpdater implements IEventListener {
 
-	/** @var IRegistry */
-	private $registry;
-
-	/** @var TotpProvider */
-	private $provider;
-
-	public function __construct(IRegistry $registry, TotpProvider $provider) {
-		$this->registry = $registry;
-		$this->provider = $provider;
+	public function __construct(
+		private IRegistry $registry,
+		private TotpProvider $provider,
+	) {
 	}
 
 	public function handle(Event $event): void {

lib/Listener/UserDeleted.php

@@ -21,23 +21,18 @@
  */
 class UserDeleted implements IEventListener {
 
-	/** @var TotpSecretMapper */
-	private $totpSecretMapper;
-
-	/** @var LoggerInterface */
-	private $logger;
-
-	public function __construct(TotpSecretMapper $totpSecretMapper, LoggerInterface $logger) {
-		$this->totpSecretMapper = $totpSecretMapper;
-		$this->logger = $logger;
+	public function __construct(
+		private TotpSecretMapper $totpSecretMapper,
+		private LoggerInterface $logger,
+	) {
 	}
 
 	public function handle(Event $event): void {
 		if ($event instanceof UserDeletedEvent) {
 			try {
-				$this->totpSecretMapper->deleteSecretByUserId($event->getUser()->getUID());
+				$this->totpSecretMapper->deleteSecretByUserId($event->getUser()?->getUID());
 			} catch (Exception $e) {
-				$this->logger->warning($e->getMessage(), ['uid' => $event->getUser()->getUID()]);
+				$this->logger->warning($e->getMessage(), ['uid' => $event->getUser()?->getUID()]);
 			}
 		}
 	}

lib/Provider/AtLoginProvider.php

@@ -16,11 +16,9 @@
 
 class AtLoginProvider implements ILoginSetupProvider {
 
-	/** @var IURLGenerator */
-	private $urlGenerator;
-
-	public function __construct(IURLGenerator $urlGenerator) {
-		$this->urlGenerator = $urlGenerator;
+	public function __construct(
+		private IURLGenerator $urlGenerator,
+	) {
 	}
 
 	public function getBody(): Template {

lib/Provider/TotpProvider.php

@@ -28,31 +28,13 @@
 
 class TotpProvider implements IProvider, IProvidesIcons, IProvidesPersonalSettings, IDeactivatableByAdmin, IActivatableAtLogin {
 
-	/** @var ITotp */
-	private $totp;
-
-	/** @var IL10N */
-	private $l10n;
-
-	/** @var IAppContainer */
-	private $container;
-
-	/** @var IInitialState */
-	private $initialState;
-
-	/** @var IURLGenerator */
-	private $urlGenerator;
-
-	public function __construct(ITotp $totp,
-		IL10N $l10n,
-		IAppContainer $container,
-		IInitialState $initialStateService,
-		IURLGenerator $urlGenerator) {
-		$this->totp = $totp;
-		$this->l10n = $l10n;
-		$this->container = $container;
-		$this->initialState = $initialStateService;
-		$this->urlGenerator = $urlGenerator;
+	public function __construct(
+		private ITotp $totp,
+		private IL10N $l10n,
+		private IAppContainer $container,
+		private IInitialState $initialStateService,
+		private IURLGenerator $urlGenerator,
+	) {
 	}
 
 	/**

lib/Service/Totp.php

@@ -25,26 +25,12 @@
 
 class Totp implements ITotp {
 
-	/** @var TotpSecretMapper */
-	private $secretMapper;
-
-	/** @var ICrypto */
-	private $crypto;
-
-	/** @var IEventDispatcher */
-	private $eventDispatcher;
-
-	/** @var ISecureRandom */
-	private $random;
-
-	public function __construct(TotpSecretMapper $secretMapper,
-		ICrypto $crypto,
-		IEventDispatcher $eventDispatcher,
-		ISecureRandom $random) {
-		$this->secretMapper = $secretMapper;
-		$this->crypto = $crypto;
-		$this->eventDispatcher = $eventDispatcher;
-		$this->random = $random;
+	public function __construct(
+		private TotpSecretMapper $secretMapper,
+		private ICrypto $crypto,
+		private IEventDispatcher $eventDispatcher,
+		private ISecureRandom $random,
+	) {
 	}
 
 	public function hasSecret(IUser $user): bool {