Skip to content

Commit d3b6289

Browse files
committed
Make sure the super share also has a valid type
Signed-off-by: Joas Schilling <coding@schilljs.com>
1 parent 6d9afca commit d3b6289

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

apps/files_sharing/lib/MountProvider.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -96,9 +96,10 @@ public function getMountsForUser(IUser $user, IStorageFactory $storageFactory) {
9696
/** @var \OCP\Share\IShare $parentShare */
9797
$parentShare = $share[0];
9898

99-
if (($parentShare->getShareType() === IShare::TYPE_GROUP ||
100-
$parentShare->getShareType() === IShare::TYPE_USERGROUP ||
101-
$parentShare->getShareType() === IShare::TYPE_USER) && $parentShare->getStatus() !== IShare::STATUS_ACCEPTED) {
99+
if ($parentShare->getStatus() !== IShare::STATUS_ACCEPTED &&
100+
($parentShare->getShareType() === IShare::TYPE_GROUP ||
101+
$parentShare->getShareType() === IShare::TYPE_USERGROUP ||
102+
$parentShare->getShareType() === IShare::TYPE_USER)) {
102103
continue;
103104
}
104105

@@ -192,6 +193,7 @@ private function buildSuperShares(array $allShares, \OCP\IUser $user) {
192193
$superShare->setId($shares[0]->getId())
193194
->setShareOwner($shares[0]->getShareOwner())
194195
->setNodeId($shares[0]->getNodeId())
196+
->setShareType($shares[0]->getShareType())
195197
->setTarget($shares[0]->getTarget());
196198

197199
// use most permissive permissions

0 commit comments

Comments
 (0)