Skip to content

Commit 71f4408

Browse files
committed
fixup: do not mark provider as setup when set up by path is used
Signed-off-by: Salvatore Martire <[email protected]>
1 parent 0e45483 commit 71f4408

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

lib/private/Files/SetupManager.php

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -719,14 +719,12 @@ public function getMountsFromProviders(
719719
$mounts = [];
720720
$userUid = $user->getUID();
721721
$this->setupUserMountProviders[$userUid] ??= [];
722-
$setupProviders = &$this->setupUserMountProviders[$userUid];
723722
/** @var class-string<IMountProvider> $providerClass */
724723
foreach ($mountInfosByProvider as $providerClass => $mountsInfos) {
725-
if (in_array($providerClass, $setupProviders)) {
724+
if (in_array($providerClass, $this->setupUserMountProviders[$userUid])) {
726725
continue; // skip already setup providers
727726
}
728727

729-
$setupProviders[] = $providerClass;
730728
if (is_a($providerClass, IPartialMountProvider::class, true)) {
731729
// mount provider capable of returning mount-points specific to
732730
// this path
@@ -738,6 +736,7 @@ public function getMountsFromProviders(
738736
$rootsMetadataByProvider[$providerClass]
739737
);
740738
} elseif (is_a($providerClass, IMountProvider::class, true)) {
739+
$this->setupUserMountProviders[$userUid] = $providerClass;
741740
// old-style provider, get the mounts for the whole provider
742741
$mounts[] = $this->mountProviderCollection
743742
->getUserMountsForProviderClasses($user, [$providerClass]);

0 commit comments

Comments
 (0)