Skip to content

Commit 4a795a9

Browse files
joshtrichardsbackportbot[bot]
authored andcommitted
refactor(dav): clarify public share user selection
Signed-off-by: Josh <josh.t.richards@gmail.com>
1 parent 6fe342d commit 4a795a9

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

apps/dav/appinfo/v1/publicwebdav.php

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -132,11 +132,10 @@ function (\Sabre\DAV\Server $server) use (
132132
Filesystem::logWarningWhenAddingStorageWrapper($previousLog);
133133

134134
$rootFolder = Server::get(IRootFolder::class);
135-
$sharedBy = $share->getSharedBy();
136-
if ($share->getShareType() === \OCP\Share\IShare::TYPE_REMOTE) {
137-
$sharedBy = $share->getShareOwner();
138-
}
139-
$userFolder = $rootFolder->getUserFolder($sharedBy);
135+
$userId = $share->getShareType() === \OCP\Share\IShare::TYPE_REMOTE
136+
? $share->getShareOwner()
137+
: $share->getSharedBy();
138+
$userFolder = $rootFolder->getUserFolder($userId);
140139
$node = $userFolder->getFirstNodeById($fileId);
141140
if (!$node) {
142141
throw new \Sabre\DAV\Exception\NotFound();

0 commit comments

Comments
 (0)