fix: Catch exceptions at Circle::getInitiator during mountpoint setup

Signed-off-by: Julius Knorr <jus@bitgrid.net>

Author: juliusknorr

lib/Mount/MountProvider.php

@@ -12,6 +12,7 @@
 use OC\Files\Cache\Cache;
 use OCA\Collectives\Db\Collective;
 use OCA\Collectives\Fs\UserFolderHelper;
+use OCA\Collectives\Service\CircleException;
 use OCA\Collectives\Service\CollectiveHelper;
 use OCA\Collectives\Service\MissingDependencyException;
 use OCA\Collectives\Service\NotFoundException;
@@ -50,7 +51,7 @@ public function getFoldersForUser(IUser $user): array {
 
 		try {
 			$collectives = $this->collectiveHelper->getCollectivesForUser($user->getUID(), true, false);
-		} catch (QueryException|MissingDependencyException|NotFoundException|NotPermittedException $e) {
+		} catch (QueryException|MissingDependencyException|NotFoundException|NotPermittedException|CircleException $e) {
 			$this->log($e);
 			return $folders;
 		}

lib/Service/CircleHelper.php

@@ -285,10 +285,9 @@ public function getLevel(string $circleId, string $userId): int {
 			$member = $circle->getInitiator();
 		} catch (CircleNotFoundException $e) {
 			throw new NotFoundException($e->getMessage(), 0, $e);
-		} catch (RequestBuilderException|
-			FederatedItemException $e) {
-				throw new NotPermittedException($e->getMessage(), 0, $e);
-			}
+		} catch (RequestBuilderException|FederatedItemException $e) {
+			throw new NotPermittedException($e->getMessage(), 0, $e);
+		}
 
 		return $member->getLevel();
 	}

lib/Service/CollectiveHelper.php

@@ -22,6 +22,7 @@ public function __construct(
 	}
 
 	/**
+	 * @throws CircleException
 	 * @throws NotFoundException
 	 * @throws NotPermittedException
 	 * @throws MissingDependencyException
@@ -36,7 +37,11 @@ public function getCollectivesForUser(string $userId, bool $getLevel = true, boo
 			$cid = $c->getCircleId();
 			$circle = $circles[$cid];
 			$c->setName($circle->getSanitizedName());
-			$c->setLevel($getLevel ? $circle->getInitiator()->getLevel(): 0);
+			try {
+				$c->setLevel($getLevel ? $circle->getInitiator()->getLevel(): 0);
+			} catch (\Exception $e) {
+				throw new CircleException($e->getMessage(), 0, $e);
+			}
 			if ($getUserSettings) {
 				// TODO: merge queries for collective and user settings into one?
 				$settings = $this->collectiveUserSettingsMapper->findByCollectiveAndUser($c->getId(), $userId);
@@ -50,6 +55,7 @@ public function getCollectivesForUser(string $userId, bool $getLevel = true, boo
 	}
 
 	/**
+	 * @throws CircleException
 	 * @throws NotFoundException
 	 * @throws NotPermittedException
 	 * @throws MissingDependencyException
@@ -63,7 +69,11 @@ public function getCollectivesTrashForUser(string $userId): array {
 			$cid = $c->getCircleId();
 			$circle = $circles[$cid];
 			$c->setName($circle->getSanitizedName());
-			$c->setLevel($circle->getInitiator()->getLevel());
+			try {
+				$c->setLevel($circle->getInitiator()->getLevel());
+			} catch (\Exception $e) {
+				throw new CircleException($e->getMessage(), 0, $e);
+			}
 		}
 		return $collectives;
 	}