Skip to content

Commit 1bae573

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 3882e52 commit 1bae573

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->getOwner();
112112
$sharedBy = $share->getSharedByDisplayName();
113113
$shareWith = $share->getShareWith();
114114
$remoteId = $share->getProviderId();

0 commit comments

Comments
 (0)