Merge pull request #3332 from nextcloud/backport/acl-no-trash-item/27

nextcloud · Oct 14, 2024 · dfeb43c · dfeb43c
2 parents f643647 + 6d467d0
commit dfeb43c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/lib/ACL/ACLManager.php b/lib/ACL/ACLManager.php
@@ -235,4 +235,8 @@ public function getPermissionsForTree(string $path): int {
 			return $permissions & $denyMask;
 		}, Constants::PERMISSION_ALL);
 	}
+
+	public function preloadRulesForFolder(string $path): void {
+		$this->ruleManager->getRulesForFilesByParent($this->user, $this->getRootStorageId(), $path);
+	}
 }
diff --git a/lib/Trash/TrashBackend.php b/lib/Trash/TrashBackend.php
@@ -95,7 +95,11 @@ public function listTrashFolder(ITrashItem $trashItem): array {
 			return [];
 		}
 		$content = $folder->getDirectoryListing();
-		return array_map(function (Node $node) use ($trashItem, $user) {
+		$this->aclManagerFactory->getACLManager($user)->preloadRulesForFolder($trashItem->getPath());
+		return array_values(array_filter(array_map(function (Node $node) use ($trashItem, $user) {
+			if (!$this->userHasAccessToPath($user, $trashItem->getPath() . '/' . $node->getName())) {
+				return null;
+			}
 			return new GroupTrashItem(
 				$this,
 				$trashItem->getOriginalLocation() . '/' . $node->getName(),
@@ -105,7 +109,7 @@ public function listTrashFolder(ITrashItem $trashItem): array {
 				$user,
 				$trashItem->getGroupFolderMountPoint()
 			);
-		}, $content);
+		}, $content)));
 	}
 
 	/**
@@ -330,6 +334,7 @@ private function getTrashForFolders(IUser $user, array $folders): array {
 			$mountPoint = $folder['mount_point'];
 			$trashFolder = $this->getTrashFolder($folderId);
 			$content = $trashFolder->getDirectoryListing();
+			$this->aclManagerFactory->getACLManager($user)->preloadRulesForFolder($trashFolder->getPath());
 			foreach ($content as $item) {
 				/** @var \OC\Files\Node\Node $item */
 				$pathParts = pathinfo($item->getName());