fix: created a patch from 3258 and 3592 PRs to fix acl issues while l…

…isting group folders

Signed-off-by: yemkareems <[email protected]>

lib/ACL/ACLManager.php

@@ -28,6 +28,7 @@
 use OCP\Constants;
 use OCP\Files\IRootFolder;
 use OCP\IUser;
+use RuntimeException;
 
 class ACLManager {
 	private CappedMemoryCache $ruleCache;
@@ -108,6 +109,10 @@ private function getRelevantPaths(string $path): array {
 			$groupFolderId = (int)$groupFolderId;
 			/* Remove the date part */
 			$separatorPos = strrpos($rootTrashedItemName, '.d');
+			if ($separatorPos === false) {
+				throw new RuntimeException('Invalid trash item name ' . $rootTrashedItemName);
+			}
+
 			$rootTrashedItemDate = (int)substr($rootTrashedItemName, $separatorPos + 2);
 			$rootTrashedItemName = substr($rootTrashedItemName, 0, $separatorPos);
 		}
@@ -116,6 +121,7 @@ private function getRelevantPaths(string $path): array {
 			$path = dirname($path);
 			if ($fromTrashbin && ($path === '__groupfolders/trash')) {
 				/* We are in trash and hit the root folder, continue looking for ACLs on parent folders in original location */
+				/** @psalm-suppress PossiblyUndefinedVariable Variables are defined above */
 				$trashItemRow = $this->trashManager->getTrashItemByFileName($groupFolderId, $rootTrashedItemName, $rootTrashedItemDate);
 				$path = dirname('__groupfolders/' . $groupFolderId . '/' . $trashItemRow['original_location']);
 				$fromTrashbin = false;

lib/Trash/TrashBackend.php

@@ -260,8 +260,13 @@ private function userHasAccessToPath(
 		string $path,
 		int $permission = Constants::PERMISSION_READ
 	): bool {
-		$activePermissions = $this->aclManagerFactory->getACLManager($user)
-			->getACLPermissionsForPath($path);
+		try {
+			$activePermissions = $this->aclManagerFactory->getACLManager($user)
+				->getACLPermissionsForPath($path);
+		} catch (\Exception $e) {
+			$this->logger->warning("Failed to get permissions for {$path}", ['exception' => $e]);
+			return false;
+		}
 		return (bool)($activePermissions & $permission);
 	}