diff options
author | Roeland Jago Douma <roeland@famdouma.nl> | 2020-05-06 22:39:49 +0300 |
---|---|---|
committer | Julius Härtl <jus@bitgrid.net> | 2020-05-26 12:28:42 +0300 |
commit | 04b02572c32445a57bf694151fe86b7ed82349f0 (patch) | |
tree | 4d6249b200c83f14f14467e293dc51e5427542ec | |
parent | 4b7b102761007c1cdf10d5f9420b16a8bab81fbf (diff) |
Have share permissions on link shares if it is enabled
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
-rw-r--r-- | apps/files_sharing/lib/Controller/ShareAPIController.php | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/apps/files_sharing/lib/Controller/ShareAPIController.php b/apps/files_sharing/lib/Controller/ShareAPIController.php index 4172b31f9a1..63d004ade5d 100644 --- a/apps/files_sharing/lib/Controller/ShareAPIController.php +++ b/apps/files_sharing/lib/Controller/ShareAPIController.php @@ -497,10 +497,12 @@ class ShareAPIController extends OCSController { } else { $permissions = Constants::PERMISSION_READ; } + // TODO: It might make sense to have a dedicated setting to allow/deny converting link shares into federated ones - if ($this->shareManager->outgoingServer2ServerSharesAllowed()) { + if (($permissions & Constants::PERMISSION_READ) && $this->shareManager->outgoingServer2ServerSharesAllowed()) { $permissions |= Constants::PERMISSION_SHARE; } + $share->setPermissions($permissions); // Set password @@ -1033,6 +1035,11 @@ class ShareAPIController extends OCSController { } if ($newPermissions !== null) { + // TODO: It might make sense to have a dedicated setting to allow/deny converting link shares into federated ones + if (($newPermissions & Constants::PERMISSION_READ) && $this->shareManager->outgoingServer2ServerSharesAllowed()) { + $newPermissions |= Constants::PERMISSION_SHARE; + } + $share->setPermissions($newPermissions); $permissions = $newPermissions; } |