Skip to content

Commit f7dcd1d

Browse files
committed
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 <[email protected]>
1 parent 9290922 commit f7dcd1d

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)