fix: fix some psalm issues

Signed-off-by: Marcel Klehr <mklehr@gmx.net>

Author: marcelklehr

lib/Dav/Faces/FacePhoto.php

@@ -123,6 +123,9 @@ public function hasPreview(): bool {
 
 	public function isFavorite(): bool {
 		$tagger = $this->tagManager->load('files');
+		if ($tagger === null) {
+			return false;
+		}
 		$tags = $tagger->getTagsForObjects([$this->getFile()->getId()]);
 
 		if ($tags === false || empty($tags)) {

lib/Migration/Version011000002Date20260129094821.php

@@ -20,7 +20,7 @@ public function __construct(
 	) {
 	}
 
-	public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options) {
+	public function postSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {
 		foreach (SettingsService::LAZY_SETTINGS as $settingsKey) {
 			if ($this->appConfig->hasAppKey($settingsKey, lazy: false)) {
 				$value = $this->appConfig->getAppValueString($settingsKey);

lib/Service/TagManager.php

@@ -73,13 +73,13 @@ public function assignTags(int $fileId, array $tags): void {
 		}, $tags);
 		$tags[] = $this->getProcessedTag()->getId();
 		/** @var array<string, string[]> $tagsByFile */
-		$tagsByFile = $this->objectMapper->getTagIdsForObjects([$fileId], 'files');
+		$tagsByFile = $this->objectMapper->getTagIdsForObjects([(string)$fileId], 'files');
 		$oldTags = $tagsByFile[(string) $fileId];
-		$this->objectMapper->assignTags((string)$fileId, 'files', array_unique(array_merge($tags, $oldTags)));
+		$this->objectMapper->assignTags((string)$fileId, 'files', array_values(array_unique(array_merge($tags, $oldTags))));
 	}
 
 	/**
-	 * @param array $fileIds
+	 * @param list<string> $fileIds
 	 * @return array<array-key, array<array-key,ISystemTag>>
 	 */
 	public function getTagsForFiles(array $fileIds): array {