implements getAccessList

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
nextcloud · Jun 13, 2024 · 37b6279 · 37b6279
1 parent 8938fcb
commit 37b6279
Show file tree

Hide file tree

Showing 4 changed files with 73 additions and 5 deletions.
diff --git a/lib/Db/ShareWrapperRequest.php b/lib/Db/ShareWrapperRequest.php
@@ -275,6 +275,30 @@ public function getSharesByFileId(int $fileId, bool $getData = false): array {
 	}
 
 
+	/**
+	 * returns all share, related to a list of fileids.
+	 * if $getData is true, will return details about the recipient circle.
+	 *
+	 * @param array $fileIds
+	 * @param bool $getData
+	 *
+	 * @return ShareWrapper[]
+	 * @throws RequestBuilderException
+	 */
+	public function getSharesByFileIds(array $fileIds, bool $getData = false): array {
+		$qb = $this->getShareSelectSql();
+		$qb->limitToFileSourceArray($fileIds);
+
+		if ($getData) {
+			$qb->setOptions([CoreQueryBuilder::SHARE], ['getData' => $getData]);
+			$qb->leftJoinCircle(CoreQueryBuilder::SHARE, null, 'share_with');
+			$qb->limitNull('parent', false);
+		}
+
+		return $this->getItemsFromRequest($qb);
+	}
+
+
 	/**
 	 * @param FederatedUser $federatedUser
 	 * @param int $nodeId

diff --git a/lib/Service/ShareWrapperService.php b/lib/Service/ShareWrapperService.php
@@ -193,6 +193,17 @@ public function getSharesByFileId(int $fileId, bool $getData = false): array {
 		return $this->shareWrapperRequest->getSharesByFileId($fileId, $getData);
 	}
 
+	/**
+	 * @param array $fileIds
+	 * @param bool $getData
+	 *
+	 * @return ShareWrapper[]
+	 * @throws RequestBuilderException
+	 */
+	public function getSharesByFileIds(array $fileIds, bool $getData = false): array {
+		return $this->shareWrapperRequest->getSharesByFileIds($fileIds, $getData);
+	}
+
 	/**
 	 * @param string $token
 	 * @param FederatedUser|null $federatedUser

diff --git a/lib/ShareByCircleProvider.php b/lib/ShareByCircleProvider.php
@@ -58,6 +58,7 @@
 use OCA\Circles\FederatedItems\Files\FileShare;
 use OCA\Circles\FederatedItems\Files\FileUnshare;
 use OCA\Circles\Model\Federated\FederatedEvent;
+use OCA\Circles\Model\Member;
 use OCA\Circles\Model\Probes\CircleProbe;
 use OCA\Circles\Model\Probes\DataProbe;
 use OCA\Circles\Model\ShareWrapper;
@@ -654,7 +655,43 @@ public function userDeletedFromGroup($uid, $gid): void {
 	 * @return array
 	 */
 	public function getAccessList($nodes, $currentAccess): array {
-		return [];
+		$ids = [];
+		foreach ($nodes as $node) {
+			$ids[] = $node->getId();
+		}
+
+		$users = $remote = $mails = [];
+		$shares = $this->shareWrapperService->getSharesByFileIds($ids, true);
+
+		foreach($shares as $share) {
+			$circle = $share->getCircle();
+			foreach ($circle->getInheritedMembers() as $member) {
+				switch ($member->getUserType()) {
+					case Member::TYPE_USER:
+						if ($member->isLocal()) {
+							if (!in_array($member->getUserId(), $users)) {
+								$users[] = $member->getUserId();
+							}
+						} else {
+							if (!in_array($member->getUserId(), $remote)) {
+								$remote[] = $member->getUserid() . '@' . $member->getInstance();
+							}
+						}
+						break;
+					case Member::TYPE_MAIL:
+						if (!in_array($member->getUserId(), $mails)) {
+							$mails[] = $member->getUserId();
+						}
+						break;
+				}
+			}
+		}
+
+		return [
+			'users' => $users,
+			'remote' => $remote,
+			'email' => $mails
+		];
 	}
 
 

diff --git a/lib/Tools/Db/ExtendedQueryBuilder.php b/lib/Tools/Db/ExtendedQueryBuilder.php
@@ -528,10 +528,6 @@ public function exprLimitArray(
 		string $alias = '',
 		bool $cs = true
 	): ICompositeExpression {
-		if ($this->getType() === DBALQueryBuilder::SELECT) {
-			$field = (($alias === '') ? $this->getDefaultSelectAlias() : $alias) . '.' . $field;
-		}
-
 		$andX = $this->expr()->andX();
 		foreach ($values as $value) {
 			if (is_integer($value)) {