From 617dc79dafab580d173fee7a55b66c34c30b6525 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();