Don't throw on UniqueConstraintViolationException since a written value is okay enough

Signed-off-by: Joas Schilling <coding@schilljs.com>

1 files changed, 8 insertions, 2 deletions

diff --git a/lib/private/Collaboration/Resources/Manager.php b/lib/private/Collaboration/Resources/Manager.php
index 0555be5d7af..0d85db324b6 100644
--- a/lib/private/Collaboration/Resources/Manager.php
+++ b/lib/private/Collaboration/Resources/Manager.php
@@ -406,7 +406,10 @@ class Manager implements IManager {
 				'resource_id' => $query->createNamedParameter($resource->getId()),
 				'access' => $query->createNamedParameter($access),
 			]);
-		$query->execute();
+		try {
+			$query->execute();
+		} catch (UniqueConstraintViolationException $e) {
+		}
 	}
 
 	public function cacheAccessForCollection(ICollection $collection, ?IUser $user, bool $access): void {
@@ -419,7 +422,10 @@ class Manager implements IManager {
 				'collection_id' => $query->createNamedParameter($collection->getId()),
 				'access' => $query->createNamedParameter($access),
 			]);
-		$query->execute();
+		try {
+			$query->execute();
+		} catch (UniqueConstraintViolationException $e) {
+		}
 	}
 
 	public function invalidateAccessCacheForUser(?IUser $user): void {