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>
pull/55827/head
Micke Nordin 10 months ago committed by Micke Nordin
parent 9290922a72
commit f7dcd1d71b
  1. 2
      apps/federatedfilesharing/lib/OCM/CloudFederationProviderFiles.php

@ -108,7 +108,7 @@ class CloudFederationProviderFiles implements ISignedCloudFederationProvider {
$token = $share->getShareSecret();
$name = $share->getResourceName();
$owner = $share->getOwnerDisplayName();
$owner = $share->getOwnerDisplayName() ?: $share->getOwner();
$sharedBy = $share->getSharedByDisplayName();
$shareWith = $share->getShareWith();
$remoteId = $share->getProviderId();

Loading…
Cancel
Save