fixup: small fixes

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

Author: salmart-dev

apps/files_sharing/lib/External/MountProvider.php

@@ -18,6 +18,7 @@
 use OCP\IUser;
 use OCP\Server;
 use OCP\Share\IShare;
+use function count;
 use function str_starts_with;
 use function strlen;
 use function substr;
@@ -78,7 +79,7 @@ public function getMountsFromMountPoints(
 		array $mountProviderArgs,
 		IStorageFactory $loader,
 	): array {
-		if (empty($mountsInfo)) {
+		if (\empty($mountProviderArgs)) {
 			return [];
 		}
 
@@ -93,7 +94,7 @@ public function getMountsFromMountPoints(
 		$uniqueMountOwnerIds = array_keys($uniqueMountOwnerIds);
 
 		// make sure the MPs belong to the same user
-		if (count($uniqueMountOwnerIds) !== 1) {
+		if ($user === null || count($uniqueMountOwnerIds) !== 1) {
 			// question: what kind of exception to throw in here?
 			throw new LogicException();
 		}

apps/files_sharing/lib/MountProvider.php

@@ -286,11 +286,10 @@ public function getMountsFromMountPoints(
 		array $mountProviderArgs,
 		IStorageFactory $loader,
 	): array {
-		/**
-		 * If path and count of mount info match, we can use the getsharedwith
-		 * If we get a path passed which doesn't match the MP in the infos we
-		 * get, it means we are loading data for a collection.
-		 */
+		if (empty($mountProviderArgs)) {
+			return [];
+		}
+
 		$uniqueMountOwnerIds = [];
 		$uniqueRootIds = [];
 		$user = null;
@@ -305,7 +304,7 @@ public function getMountsFromMountPoints(
 		$uniqueRootIds = array_keys($uniqueRootIds);
 
 		// make sure the MPs belong to the same user
-		if (count($uniqueMountOwnerIds) !== 1) {
+		if ($user === null || count($uniqueMountOwnerIds) !== 1) {
 			// question: what kind of exception to throw in here?
 			throw new LogicException();
 		}