ci(dev-deps): bump psalm from 6.5 to 6.20.1

lib/Activity/ChangeSet.php

@@ -53,6 +53,7 @@ public function getAfter() {
 		return $this->after;
 	}
 
+	#[\Override]
 	public function jsonSerialize(): array {
 		return [
 			'before' => $this->getBefore(),

lib/Activity/DeckProvider.php

@@ -65,6 +65,7 @@ public function __construct(IURLGenerator $urlGenerator, ActivityManager $activi
 	 * @throws UnknownActivityException Should be thrown if your provider does not know this event
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function parse($language, IEvent $event, ?IEvent $previousEvent = null): IEvent {
 		if ($event->getApp() !== 'deck') {
 			throw new UnknownActivityException();

lib/Activity/Filter.php

@@ -26,6 +26,7 @@ public function __construct(
 	 * @return string Lowercase a-z and underscore only identifier
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getIdentifier(): string {
 		return 'deck';
 	}
@@ -34,6 +35,7 @@ public function getIdentifier(): string {
 	 * @return string A translated string
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getName(): string {
 		return $this->l10n->t('Deck');
 	}
@@ -44,6 +46,7 @@ public function getName(): string {
 	 *             priority values. It is required to return a value between 0 and 100.
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getPriority(): int {
 		return 90;
 	}
@@ -52,6 +55,7 @@ public function getPriority(): int {
 	 * @return string Full URL to an icon, empty string when none is given
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getIcon(): string {
 		return $this->urlGenerator->imagePath('deck', 'deck-dark.svg');
 	}
@@ -61,6 +65,7 @@ public function getIcon(): string {
 	 * @return string[] An array of allowed apps from which activities should be displayed
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function filterTypes($types): array {
 		return array_merge($types, ['deck_comment']);
 	}
@@ -69,6 +74,7 @@ public function filterTypes($types): array {
 	 * @return string[] An array of allowed apps from which activities should be displayed
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function allowedApps(): array {
 		return ['deck'];
 	}

lib/Activity/SettingBase.php

@@ -22,10 +22,12 @@ public function __construct(IL10N $l) {
 		$this->l = $l;
 	}
 
+	#[\Override]
 	public function getGroupIdentifier() {
 		return 'deck';
 	}
 
+	#[\Override]
 	public function getGroupName() {
 		return $this->l->t('Deck');
 	}
@@ -34,6 +36,7 @@ public function getGroupName() {
 	 * @return string Lowercase a-z and underscore only identifier
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getIdentifier(): string {
 		return 'deck';
 	}
@@ -42,6 +45,7 @@ public function getIdentifier(): string {
 	 * @return string A translated string
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getName(): string {
 		return $this->l->t('Changes in the <strong>Deck app</strong>');
 	}
@@ -52,6 +56,7 @@ public function getName(): string {
 	 *             priority values. It is required to return a value between 0 and 100.
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getPriority(): int {
 		return 90;
 	}
@@ -60,6 +65,7 @@ public function getPriority(): int {
 	 * @return bool True when the option can be changed for the stream
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function canChangeStream(): bool {
 		return true;
 	}
@@ -68,6 +74,7 @@ public function canChangeStream(): bool {
 	 * @return bool True when the option can be changed for the stream
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function isDefaultEnabledStream(): bool {
 		return true;
 	}
@@ -76,6 +83,7 @@ public function isDefaultEnabledStream(): bool {
 	 * @return bool True when the option can be changed for the mail
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function canChangeMail(): bool {
 		return true;
 	}
@@ -84,6 +92,7 @@ public function canChangeMail(): bool {
 	 * @return bool True when the option can be changed for the stream
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function isDefaultEnabledMail(): bool {
 		return false;
 	}

lib/Activity/SettingChanges.php

@@ -12,6 +12,7 @@ class SettingChanges extends SettingBase {
 	 * @return string Lowercase a-z and underscore only identifier
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getIdentifier(): string {
 		return 'deck';
 	}
@@ -20,6 +21,7 @@ public function getIdentifier(): string {
 	 * @return string A translated string
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getName(): string {
 		return $this->l->t('A <strong>board, list or card</strong> was changed');
 	}
@@ -30,6 +32,7 @@ public function getName(): string {
 	 *             priority values. It is required to return a value between 0 and 100.
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getPriority(): int {
 		return 90;
 	}
@@ -38,6 +41,7 @@ public function getPriority(): int {
 	 * @return bool True when the option can be changed for the stream
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function canChangeStream(): bool {
 		return true;
 	}
@@ -46,6 +50,7 @@ public function canChangeStream(): bool {
 	 * @return bool True when the option can be changed for the stream
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function isDefaultEnabledStream(): bool {
 		return true;
 	}
@@ -54,6 +59,7 @@ public function isDefaultEnabledStream(): bool {
 	 * @return bool True when the option can be changed for the mail
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function canChangeMail(): bool {
 		return true;
 	}
@@ -62,6 +68,7 @@ public function canChangeMail(): bool {
 	 * @return bool True when the option can be changed for the stream
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function isDefaultEnabledMail(): bool {
 		return false;
 	}

lib/Activity/SettingComment.php

@@ -16,6 +16,7 @@ class SettingComment extends SettingBase {
 	 * @return string Lowercase a-z and underscore only identifier
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getIdentifier(): string {
 		return 'deck_comment';
 	}
@@ -24,6 +25,7 @@ public function getIdentifier(): string {
 	 * @return string A translated string
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getName(): string {
 		return $this->l->t('A <strong>comment</strong> was created on a card');
 	}
@@ -32,6 +34,7 @@ public function getName(): string {
 	 * @return bool True when the option can be changed for the stream
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function canChangeStream(): bool {
 		return false;
 	}

lib/Activity/SettingDescription.php

@@ -13,6 +13,7 @@ class SettingDescription extends SettingBase {
 	 * @return string Lowercase a-z and underscore only identifier
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getIdentifier(): string {
 		return 'deck_card_description';
 	}
@@ -21,6 +22,7 @@ public function getIdentifier(): string {
 	 * @return string A translated string
 	 * @since 11.0.0
 	 */
+	#[\Override]
 	public function getName(): string {
 		return $this->l->t('A <strong>card description</strong> has been changed');
 	}

lib/AppInfo/Application.php

@@ -106,6 +106,7 @@ public function __construct(array $urlParams = []) {
 		});
 	}
 
+	#[\Override]
 	public function boot(IBootContext $context): void {
 		$context->injectFn($this->registerCommentsEntity(...));
 		$context->injectFn($this->registerCollaborationResources(...));
@@ -126,6 +127,7 @@ public function boot(IBootContext $context): void {
 		$context->injectFn([$this, 'registerCloudFederationProviderManager']);
 	}
 
+	#[\Override]
 	public function register(IRegistrationContext $context): void {
 		if ((@include_once __DIR__ . '/../../vendor/autoload.php') === false) {
 			throw new Exception('Cannot include autoload. Did you run install dependencies using composer?');

lib/BadRequestException.php

@@ -14,6 +14,7 @@ public function __construct($message) {
 		parent::__construct($message);
 	}
 
+	#[\Override]
 	public function getStatus() {
 		return HTTP::STATUS_BAD_REQUEST;
 	}

lib/Capabilities.php

@@ -26,6 +26,7 @@ public function __construct(IAppManager $appManager, PermissionService $permissi
 	 * @return array{deck: array{version: string, canCreateBoards: bool, supportsDoneColumn:true, apiVersions: array<string>}}
 	 * @since 8.2.0
 	 */
+	#[\Override]
 	public function getCapabilities() {
 		return [
 			'deck' => [

lib/Collaboration/Resources/ResourceProvider.php

@@ -42,6 +42,7 @@ public function __construct(BoardMapper $boardMapper, PermissionService $permiss
 	 * @return string
 	 * @since 15.0.0
 	 */
+	#[\Override]
 	public function getType(): string {
 		return self::RESOURCE_TYPE;
 	}
@@ -55,6 +56,7 @@ public function getType(): string {
 	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
 	 * @since 16.0.0
 	 */
+	#[\Override]
 	public function getResourceRichObject(IResource $resource): array {
 		$board = $this->getBoard($resource);
 		$link = $this->urlGenerator->linkToRoute('deck.page.indexBoard', ['boardId' => $resource->getId()]);
@@ -76,6 +78,7 @@ public function getResourceRichObject(IResource $resource): array {
 	 * @return bool
 	 * @since 16.0.0
 	 */
+	#[\Override]
 	public function canAccessResource(IResource $resource, ?IUser $user): bool {
 		if ($resource->getType() !== self::RESOURCE_TYPE || !$user instanceof IUser) {
 			return false;

lib/Collaboration/Resources/ResourceProviderCard.php

@@ -45,6 +45,7 @@ public function __construct(CardMapper $cardMapper, BoardMapper $boardMapper, Pe
 	 * @return string
 	 * @since 15.0.0
 	 */
+	#[\Override]
 	public function getType(): string {
 		return self::RESOURCE_TYPE;
 	}
@@ -56,6 +57,7 @@ public function getType(): string {
 	 * @return array
 	 * @since 16.0.0
 	 */
+	#[\Override]
 	public function getResourceRichObject(IResource $resource): array {
 		try {
 			$card = $this->cardMapper->find($resource->getId());
@@ -88,6 +90,7 @@ public function getResourceRichObject(IResource $resource): array {
 	 * @return bool
 	 * @since 16.0.0
 	 */
+	#[\Override]
 	public function canAccessResource(IResource $resource, ?IUser $user): bool {
 		if (!$user instanceof IUser || $resource->getType() !== self::RESOURCE_TYPE) {
 			return false;

lib/Command/BoardImport.php

@@ -24,6 +24,7 @@ public function __construct(
 	/**
 	 * @return void
 	 */
+	#[\Override]
 	protected function configure() {
 		$allowedSystems = $this->boardImportCommandService->getAllowedImportSystems();
 		$names = array_map(function ($name) {
@@ -67,6 +68,7 @@ protected function configure() {
 	 *
 	 * @return int
 	 */
+	#[\Override]
 	protected function execute(InputInterface $input, OutputInterface $output): int {
 		$this
 			->boardImportCommandService

lib/Command/CalendarToggle.php

@@ -24,6 +24,7 @@ public function __construct(IUserManager $userManager, IConfig $config) {
 		$this->config = $config;
 	}
 
+	#[\Override]
 	protected function configure() {
 		$this
 			->setName('deck:calendar-toggle')
@@ -42,6 +43,7 @@ protected function configure() {
 			);
 	}
 
+	#[\Override]
 	protected function execute(InputInterface $input, OutputInterface $output): int {
 		$enable = $input->getOption('on');
 		$disable = $input->getOption('off');

lib/Command/TransferOwnership.php

@@ -33,6 +33,7 @@ public function __construct(BoardService $boardService, BoardMapper $boardMapper
 		$this->questionHelper = $questionHelper;
 	}
 
+	#[\Override]
 	protected function configure() {
 		$this
 			->setName('deck:transfer-ownership')
@@ -61,6 +62,7 @@ protected function configure() {
 		;
 	}
 
+	#[\Override]
 	protected function execute(InputInterface $input, OutputInterface $output): int {
 		$owner = $input->getArgument('owner');
 		$newOwner = $input->getArgument('newOwner');

lib/Command/UserExport.php

@@ -34,6 +34,7 @@ public function __construct(
 		parent::__construct();
 	}
 
+	#[\Override]
 	protected function configure() {
 		$this
 			->setName('deck:export')
@@ -50,6 +51,7 @@ protected function configure() {
 	/**
 	 * @throws Exception
 	 */
+	#[\Override]
 	protected function execute(InputInterface $input, OutputInterface $output): int {
 		$userId = $input->getArgument('user-id');
 		$legacyFormat = $input->getOption('legacy-format');

lib/Cron/CardDescriptionActivity.php

@@ -29,6 +29,7 @@ public function __construct(ITimeFactory $time, ActivityManager $activityManager
 	 * @param $argument
 	 * @SuppressWarnings(PHPMD.UnusedFormalParameter)
 	 */
+	#[\Override]
 	public function run($argument) {
 		$cards = $this->cardMapper->findUnexposedDescriptionChances();
 		foreach ($cards as $card) {

lib/Cron/DeleteCron.php

@@ -64,6 +64,7 @@ public function __construct(
 	 * @param $argument
 	 * @SuppressWarnings(PHPMD.UnusedFormalParameter)
 	 */
+	#[\Override]
 	protected function run($argument) {
 		$timeLimit = time() - $this->configService->getTrashRetention();
 

lib/Cron/ScheduledNotifications.php

@@ -30,6 +30,7 @@ public function __construct(
 	 * @param $argument
 	 * @SuppressWarnings(PHPMD.UnusedFormalParameter)
 	 */
+	#[\Override]
 	public function run($argument) {
 		// Notify board owner and card creator about overdue cards
 		$cards = $this->cardMapper->findOverdue();