Merge pull request #40363 from nextcloud/share-external-watcher

apps/files_sharing/lib/SharedStorage.php

@@ -42,9 +42,10 @@
 use OC\Files\Storage\Home;
 use OC\Files\Storage\Wrapper\PermissionsMask;
 use OC\User\NoUserException;
-use OCA\Files_External\Config\ExternalMountPoint;
+use OCA\Files_External\Config\ConfigAdapter;
 use OCP\Constants;
 use OCP\Files\Cache\ICacheEntry;
+use OCP\Files\Config\IUserMountCache;
 use OCP\Files\Folder;
 use OCP\Files\IHomeStorage;
 use OCP\Files\IRootFolder;
@@ -431,21 +432,29 @@ public function getOwner($path): string {
 	}
 
 	public function getWatcher($path = '', $storage = null): Watcher {
-		$mountManager = \OC::$server->getMountManager();
+		if ($this->watcher) {
+			return $this->watcher;
+		}
 
 		// Get node information
 		$node = $this->getShare()->getNodeCacheEntry();
 		if ($node) {
-			$mount = $mountManager->findByNumericId($node->getStorageId());
-			// If the share is originating from an external storage
-			if (count($mount) > 0 && $mount[0] instanceof ExternalMountPoint) {
-				// Propagate original storage scan
-				return parent::getWatcher($path, $storage);
+			/** @var IUserMountCache $userMountCache */
+			$userMountCache = \OC::$server->get(IUserMountCache::class);
+			$mounts = $userMountCache->getMountsForStorageId($node->getStorageId());
+			foreach ($mounts as $mount) {
+				// If the share is originating from an external storage
+				if ($mount->getMountProvider() === ConfigAdapter::class) {
+					// Propagate original storage scan
+					$this->watcher = parent::getWatcher($path, $storage);
+					return $this->watcher;
+				}
 			}
 		}
 
 		// cache updating is handled by the share source
-		return new NullWatcher();
+		$this->watcher = new NullWatcher();
+		return $this->watcher;
 	}
 
 	/**