From f7dcd1d71bcae2c02aed743b9650fbf802e552fe Mon Sep 17 00:00:00 2001 From: Micke Nordin Date: Fri, 17 Oct 2025 13:43:22 +0200 Subject: [PATCH] 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 --- .../lib/OCM/CloudFederationProviderFiles.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/federatedfilesharing/lib/OCM/CloudFederationProviderFiles.php b/apps/federatedfilesharing/lib/OCM/CloudFederationProviderFiles.php index 1ce639532e8..461a4b1948a 100644 --- a/apps/federatedfilesharing/lib/OCM/CloudFederationProviderFiles.php +++ b/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();