refactor: declare getMount() and getMountsIn() at IRootFolder

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

lib/private/Files/Cache/QuerySearchHelper.php

@@ -199,12 +199,8 @@ class QuerySearchHelper {
 	 */
 	public function getCachesAndMountPointsForSearch(IRootFolder $root, string $path, bool $limitToHome = false): array {
 		$rootLength = strlen($path);
-		$storage = null;
-		if (method_exists($root, 'getMount')) {
-			/** @var IMountPoint $mount */
-			$mount = $root->getMount($path);
-			$storage = $mount->getStorage();
-		}
+		$mount = $root->getMount($path);
+		$storage = $mount->getStorage();
 		if ($storage === null) {
 			return [];
 		}
@@ -221,8 +217,7 @@ class QuerySearchHelper {
 		/** @var IMountPoint[] $mountByMountPoint */
 		$mountByMountPoint = ['' => $mount];
 
-		if (!$limitToHome && method_exists($root, 'getMountsIn')) {
-			/** @var IMountPoint[] $mounts */
+		if (!$limitToHome) {
 			$mounts = $root->getMountsIn($path);
 			foreach ($mounts as $mount) {
 				$storage = $mount->getStorage();

lib/private/Files/Node/Folder.php

@@ -330,12 +330,8 @@ class Folder extends Node implements \OCP\Files\Folder {
 	 * @return array
 	 */
 	protected function getByIdInRootMount(int $id): array {
-		$storage = null;
-		if (\method_exists($this->root, 'getMount')) {
-			/** @var IMountPoint $mount */
-			$mount = $this->root->getMount('');
-			$storage = $mount->getStorage();
-		}
+		$mount = $this->root->getMount('');
+		$storage = $mount->getStorage();
 		$cacheEntry = $storage?->getCache($this->path)->get($id);
 		if (!$cacheEntry) {
 			return [];

lib/private/Files/Node/LazyFolder.php

@@ -29,6 +29,7 @@ namespace OC\Files\Node;
 use OC\Files\Utils\PathHelper;
 use OCP\Files\Folder;
 use OCP\Constants;
+use OCP\Files\Mount\IMountPoint;
 
 /**
  * Class LazyFolder
@@ -111,14 +112,14 @@ class LazyFolder implements Folder {
 	/**
 	 * @inheritDoc
 	 */
-	public function getMount($mountPoint) {
+	public function getMount(string $mountPoint): IMountPoint {
 		return $this->__call(__FUNCTION__, func_get_args());
 	}
 
 	/**
-	 * @inheritDoc
+	 * @return IMountPoint[]
 	 */
-	public function getMountsIn($mountPoint) {
+	public function getMountsIn(string $mountPoint): array {
 		return $this->__call(__FUNCTION__, func_get_args());
 	}
 

lib/private/Files/Node/Root.php

@@ -154,11 +154,7 @@ class Root extends Folder implements IRootFolder {
 		$this->mountManager->addMount($mount);
 	}
 
-	/**
-	 * @param string $mountPoint
-	 * @return \OC\Files\Mount\MountPoint
-	 */
-	public function getMount($mountPoint) {
+	public function getMount(string $mountPoint): IMountPoint {
 		return $this->mountManager->find($mountPoint);
 	}
 
@@ -166,7 +162,7 @@ class Root extends Folder implements IRootFolder {
 	 * @param string $mountPoint
 	 * @return \OC\Files\Mount\MountPoint[]
 	 */
-	public function getMountsIn($mountPoint) {
+	public function getMountsIn(string $mountPoint): array {
 		return $this->mountManager->findIn($mountPoint);
 	}
 

lib/public/Files/IRootFolder.php

@@ -26,6 +26,7 @@ namespace OCP\Files;
 
 use OC\Hooks\Emitter;
 use OC\User\NoUserException;
+use OCP\Files\Mount\IMountPoint;
 
 /**
  * Interface IRootFolder
@@ -55,4 +56,16 @@ interface IRootFolder extends Folder, Emitter {
 	 * @since 24.0.0
 	 */
 	public function getByIdInPath(int $id, string $path);
+
+	/**
+	 * @return IMountPoint[]
+	 *
+	 * @since 28.0.0
+	 */
+	public function getMountsIn(string $mountPoint): array;
+
+	/**
+	 * @since 28.0.0
+	 */
+	public function getMount(string $mountPoint): IMountPoint;
 }