Skip to content

Commit 751faa0

Browse files
mickenordinbackportbot[bot]
authored andcommitted
fix(ocm-sharing): The owner needs to be set for sharing to work
The specification says that the display name is optional and can thus be empty, and in fact it is from oCIS and CERNBox shares. The correct thing to set is the required opaque id from the remote provider, the `owner` which will allways be there. Signed-off-by: Micke Nordin <kano@sunet.se>
1 parent 55228fe commit 751faa0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

apps/federatedfilesharing/lib/OCM/CloudFederationProviderFiles.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ public function shareReceived(ICloudFederationShare $share) {
108108

109109
$token = $share->getShareSecret();
110110
$name = $share->getResourceName();
111-
$owner = $share->getOwnerDisplayName();
111+
$owner = $share->getOwnerDisplayName() ?: $share->getOwner();
112112
$sharedBy = $share->getSharedByDisplayName();
113113
$shareWith = $share->getShareWith();
114114
$remoteId = $share->getProviderId();

0 commit comments

Comments
 (0)