Make sure the super share also has a valid type

Signed-off-by: Joas Schilling <[email protected]>
nextcloud · Oct 31, 2019 · 9fdda51 · 9fdda51
1 parent c34c38c
commit 9fdda51
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/apps/files_sharing/lib/MountProvider.php b/apps/files_sharing/lib/MountProvider.php
@@ -96,9 +96,10 @@ public function getMountsForUser(IUser $user, IStorageFactory $storageFactory) {
 				/** @var \OCP\Share\IShare $parentShare */
 				$parentShare = $share[0];
 
-				if (($parentShare->getShareType() === IShare::TYPE_GROUP ||
-					$parentShare->getShareType() === IShare::TYPE_USERGROUP ||
-					$parentShare->getShareType() === IShare::TYPE_USER) && $parentShare->getStatus() !== IShare::STATUS_ACCEPTED) {
+				if ($parentShare->getStatus() !== IShare::STATUS_ACCEPTED &&
+					($parentShare->getShareType() === IShare::TYPE_GROUP ||
+						$parentShare->getShareType() === IShare::TYPE_USERGROUP ||
+						$parentShare->getShareType() === IShare::TYPE_USER)) {
 					continue;
 				}
 
@@ -192,6 +193,7 @@ private function buildSuperShares(array $allShares, \OCP\IUser $user) {
 			$superShare->setId($shares[0]->getId())
 				->setShareOwner($shares[0]->getShareOwner())
 				->setNodeId($shares[0]->getNodeId())
+				->setShareType($shares[0]->getShareType())
 				->setTarget($shares[0]->getTarget());
 
 			// use most permissive permissions