Skip to content

Commit e4ab50b

Browse files
refactor(dav): clarify public share user selection
Signed-off-by: Josh <josh.t.richards@gmail.com>
1 parent 075d9c5 commit e4ab50b

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
@@ -124,11 +124,10 @@ function (\Sabre\DAV\Server $server) use (
124124
Filesystem::logWarningWhenAddingStorageWrapper($previousLog);
125125

126126
$rootFolder = Server::get(IRootFolder::class);
127-
$sharedBy = $share->getSharedBy();
128-
if ($share->getShareType() === \OCP\Share\IShare::TYPE_REMOTE) {
129-
$sharedBy = $share->getShareOwner();
130-
}
131-
$userFolder = $rootFolder->getUserFolder($sharedBy);
127+
$userId = $share->getShareType() === \OCP\Share\IShare::TYPE_REMOTE
128+
? $share->getShareOwner()
129+
: $share->getSharedBy();
130+
$userFolder = $rootFolder->getUserFolder($userId);
132131
$node = $userFolder->getFirstNodeById($fileId);
133132
if (!$node) {
134133
throw new \Sabre\DAV\Exception\NotFound();

0 commit comments

Comments
 (0)