fixup!: Don't filter out not requested shares

Signed-off-by: Louis Chmn <[email protected]>

Author: artonge

apps/files_sharing/lib/External/MountProvider.php

@@ -95,7 +95,7 @@ public function getMountsForPath(
 
 		if (!$forChildren) {
 			// override path with mount point when fetching without children
-			$path = $mountProviderArgs[0]->mountInfo->getMountPoint(); // TODO: not sure what this is doing, is this getting the parent path?
+			$path = $mountProviderArgs[0]->mountInfo->getMountPoint();
 		}
 
 		// remove /uid/files as the target is stored without
@@ -127,14 +127,6 @@ public function getMountsForPath(
 			$row['manager'] = $this;
 			$row['token'] = $row['share_token'];
 			$mount = $this->getMount($user, $row, $loader);
-
-			$isRequestedMount = array_any($mountProviderArgs, function (IMountProviderArgs $arg) use ($mount) {
-				return $arg->mountInfo->getMountPoint() === $mount->getMountPoint();
-			});
-			if (!$isRequestedMount) {
-				continue; // TODO: does it means something is off? because we have it in oc_share_external but not in oc_mounts
-			}
-
 			$mounts[$mount->getMountPoint()] = $mount;
 		}
 		$result->closeCursor();

apps/files_sharing/lib/MountProvider.php

@@ -406,16 +406,6 @@ public function getMountsForPath(
 		$shares = $this->filterShares($shares, $userId);
 		$superShares = $this->buildSuperShares($shares, $user);
 
-		// remove unrequested shares
-		$superShares = \array_filter($superShares, function (array $shares) use ($mountProviderArgs) {
-			$superShare = $shares[0];
-
-			return array_any($mountProviderArgs,
-				function (IMountProviderArgs $mountProviderArg) use ($superShare) {
-					return $superShare->getNodeId() === $mountProviderArg->mountInfo->getRootId();
-				});
-		});
-
 		return $this->getMountsFromSuperShares($userId,
 			$superShares,
 			$loader,