fixup! Fix SQL statement and provider method call

Author: juliusknorr

lib/private/Collaboration/Resources/Manager.php

@@ -85,9 +85,9 @@ public function getCollectionForUser(int $id, ?IUser $user): ICollection {
 		$query->select('*')
 			->from(self::TABLE_COLLECTIONS, 'c')
 			->leftJoin(
-				'a', self::TABLE_ACCESS_CACHE, 'a',
+				'c', self::TABLE_ACCESS_CACHE, 'a',
 				$query->expr()->andX(
-					$query->expr()->eq('c.id', 'a.resource_id'),
+					$query->expr()->eq('c.id', 'a.collection_id'),
 					$query->expr()->eq('a.user_id', $query->createNamedParameter($userId, IQueryBuilder::PARAM_STR))
 				)
 			)
@@ -103,10 +103,10 @@ public function getCollectionForUser(int $id, ?IUser $user): ICollection {
 		$access = $row['access'] === null ? null : (bool) $row['access'];
 		if ($user instanceof IUser) {
 			$access = [$user->getUID() => $access];
-			return new Collection($this, $this->connection, (int) $row['id'], (string) $row['name'], $access, null);
+			return new Collection($this, $this->connection, (int) $row['id'], (string) $row['name'], $user, null);
 		}
 
-		return new Collection($this, $this->connection, (int) $row['id'], (string) $row['name'], [], $access);
+		return new Collection($this, $this->connection, (int) $row['id'], (string) $row['name'], $user, $access);
 	}
 
 	/**
@@ -122,16 +122,16 @@ public function searchCollections(IUser $user, string $filter, int $limit = 50,
 		$userId = $user instanceof IUser ? $user->getUID() : '';
 
 		$query->select('c.*', 'a.access')
-			->from(self::TABLE_COLLECTIONS)
+			->from(self::TABLE_COLLECTIONS, 'c')
 			->leftJoin(
-				'r', self::TABLE_ACCESS_CACHE, 'a',
+				'c', self::TABLE_ACCESS_CACHE, 'a',
 				$query->expr()->andX(
-					$query->expr()->eq('c.id', 'a.resource_id'),
+					$query->expr()->eq('c.id', 'a.collection_id'),
 					$query->expr()->eq('a.user_id', $query->createNamedParameter($userId, IQueryBuilder::PARAM_STR))
 				)
 			)
 			->where($query->expr()->iLike('c.name', $query->createNamedParameter($filter, IQueryBuilder::PARAM_STR)))
-			->andWhere($query->expr()->neq('a.access', $query->createNamedParameter(0, IQueryBuilder::PARAM_INT)))
+			->andWhere($query->expr()->eq('a.access', $query->createNamedParameter(1, IQueryBuilder::PARAM_INT)))
 			->orderBy('c.id')
 			->setMaxResults($limit)
 			->setFirstResult($start);
@@ -236,7 +236,7 @@ public function getResourcesByCollectionForUser(ICollection $collection, ?IUser
 			->leftJoin(
 				'r', self::TABLE_ACCESS_CACHE, 'a',
 				$query->expr()->andX(
-					$query->expr()->eq('r.id', 'a.resource_id'),
+					$query->expr()->eq('r.resource_id', 'a.resource_id'),
 					$query->expr()->eq('a.user_id', $query->createNamedParameter($userId, IQueryBuilder::PARAM_STR))
 				)
 			)