fix: Fix navigation manager to be in line with server

lib/FilteredNavigationManager.php

@@ -69,4 +69,23 @@ public function setActiveEntry($appId): void {
 	public function setUnreadCounter(string $id, int $unreadCounter): void {
 		$this->navigationManager->setUnreadCounter($id, $unreadCounter);
 	}
+
+	public function get(string $id): ?array {
+		return $this->navigationManager->get($id);
+	}
+
+	public function getDefaultEntryIdForUser(?IUser $user = null, bool $withFallbacks = true): string {
+		return $this->navigationManager->getDefaultEntryIdForUser($user, $withFallbacks);
+	}
+
+	public function getDefaultEntryIds(bool $withFallbacks = true): array {
+		return $this->navigationManager->getDefaultEntryIds($withFallbacks);
+	}
+
+	/**
+	 * @psalm-suppress MethodSignatureMismatch
+	 */
+	public function setDefaultEntryIds(array $ids): void {
+		$this->navigationManager->setDefaultEntryIds($ids);
+	}
 }

lib/RestrictionManager.php

@@ -100,7 +100,7 @@ public function setupRestrictions(): void {
 			}
 
 			/** @var NavigationManager $navManager */
-			$navManager = $this->server->getNavigationManager();
+			$navManager = \OCP\Server::get(INavigationManager::class);
 
 			$this->server->registerService(INavigationManager::class, function () use ($navManager) {
 				return new FilteredNavigationManager($this->userSession->getUser(), $navManager, $this->whitelist);

lib/Storage/DirMask.php

@@ -22,6 +22,7 @@
 namespace OCA\Guests\Storage;
 
 use OC\Files\Storage\Wrapper\PermissionsMask;
+use OCP\Files\Cache\ICache;
 
 /**
  * While PermissionMask can mask a whole storage this  can
@@ -100,9 +101,6 @@ public function getPermissions($path): int {
 		}
 	}
 
-	/**
-	 * @psalm-suppress ParamNameMismatch
-	 */
 	public function rename($source, $target): bool {
 		if (!$this->isUpdatable($source)) {
 			return false;
@@ -123,9 +121,6 @@ public function rename($source, $target): bool {
 		return false;
 	}
 
-	/**
-	 * @psalm-suppress ParamNameMismatch
-	 */
 	public function copy($source, $target): bool {
 		if (!$this->isReadable($source)) {
 			return false;
@@ -162,10 +157,7 @@ public function mkdir($path): bool {
 		}
 	}
 
-	/**
-	 * @psalm-suppress MethodSignatureMustProvideReturnType
-	 */
-	public function rmdir($path) {
+	public function rmdir($path): bool {
 		if ($this->checkPath($path)) {
 			return parent::rmdir($path);
 		} else {
@@ -181,20 +173,14 @@ public function unlink($path): bool {
 		}
 	}
 
-	/**
-	 * @psalm-suppress MethodSignatureMustProvideReturnType
-	 */
-	public function file_put_contents($path, $data) {
+	public function file_put_contents($path, $data): int|float|false {
 		if ($this->checkPath($path)) {
 			return parent::file_put_contents($path, $data);
 		} else {
 			return $this->storage->file_put_contents($path, $data);
 		}
 	}
 
-	/**
-	 * @psalm-suppress MethodSignatureMustProvideReturnType
-	 */
 	public function fopen($path, $mode) {
 		if ($this->checkPath($path)) {
 			return parent::fopen($path, $mode);
@@ -203,10 +189,7 @@ public function fopen($path, $mode) {
 		}
 	}
 
-	/**
-	 * @psalm-suppress MethodSignatureMustProvideReturnType
-	 */
-	public function getCache($path = '', $storage = null) {
+	public function getCache($path = '', $storage = null): ICache {
 		if (!$storage) {
 			$storage = $this;
 		}