fix: avoid recomputing list of mounts by provider [skip ci]

Signed-off-by: Salvatore Martire <[email protected]>

Author: salmart-dev

lib/private/Files/Mount/Manager.php

@@ -20,6 +20,7 @@
 class Manager implements IMountManager {
 	/** @var array<string, IMountPoint> */
 	private array $mounts = [];
+	private array $mountsByProvider = [];
 	private bool $areMountsSorted = false;
 	/** @var list<string>|null $mountKeys */
 	private ?array $mountKeys = null;
@@ -36,7 +37,10 @@ public function __construct(SetupManagerFactory $setupManagerFactory) {
 	}
 
 	public function addMount(IMountPoint $mount): void {
-		$this->mounts[$mount->getMountPoint()] = $mount;
+		$mountPoint = $mount->getMountPoint();
+		$this->mounts[$mountPoint] = $mount;
+		$this->mountsByProvider[$mountPoint] ??= [];
+		$this->mountsByProvider[$mount->getMountProvider()][$mountPoint] = $mount;
 		$this->pathCache->clear();
 		$this->inPathCache->clear();
 		$this->areMountsSorted = false;
@@ -167,6 +171,7 @@ private function binarySearch(array $sortedArray, array $sortedKeys, string $pre
 
 	public function clear(): void {
 		$this->mounts = [];
+		$this->mountsByProvider = [];
 		$this->pathCache->clear();
 		$this->inPathCache->clear();
 	}
@@ -241,9 +246,12 @@ public function getMountsByMountProvider(string $path, array $mountProviders) {
 		if (in_array('', $mountProviders)) {
 			return $this->mounts;
 		} else {
-			return array_filter($this->mounts, function ($mount) use ($mountProviders) {
-				return in_array($mount->getMountProvider(), $mountProviders);
-			});
+			$mounts = [];
+			foreach ($mountProviders as $mountProvider) {
+				$mounts[] = $this->mountsByProvider[$mountProvider];
+			}
+
+			return array_merge(...$mounts);
 		}
 	}