Skip to content

feat: show the source name of the renamed received share - #65158

Merged
susnux merged 1 commit into
masterfrom
fix/share-rename
Oct 14, 2026
Merged

susnux merged 1 commit into
masterfrom
fix/share-rename

Conversation

@kristian-zendato

Copy link
Copy Markdown
Contributor

Summary

When someone renames a received share, the sharing sidebar still needs to show the name it had on the source. This adds that name as original_displayname on the share API and as the nc:original-displayname WebDAV property, only for the share root and only when the names differ.
The sidebar keeps "Shared with you by {owner}" and appends "originally as {name}" in that case. Public WebDAV does not expose the property.

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@kristian-zendato
kristian-zendato requested review from a team and provokateurin as code owners October 6, 2026 06:57
@kristian-zendato
kristian-zendato requested review from CarlSchwan, leftybournes, skjnldsv, sorbaugh and susnux and removed request for a team October 6, 2026 06:57
Comment thread apps/files_sharing/src/components/SharingEntrySimple.vue Outdated
Comment thread apps/files_sharing/src/utils/SharedWithMe.js Outdated
Comment thread apps/dav/lib/Connector/Sabre/FilesPlugin.php
Comment thread apps/files_sharing/src/views/SharingTab.vue Outdated
Comment thread apps/files_sharing/src/views/SharingTab.vue Outdated
Comment thread apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
Comment thread apps/files_sharing/tests/OriginalDisplayNameTest.php Outdated
@susnux susnux changed the title fix: show the source name of the renamed received share feat: show the source name of the renamed received share Oct 13, 2026
@susnux

susnux commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Tests look related?

Signed-off-by: kristian-zendato <kristian.zendato@nextcloud.com>
@kristian-zendato

Copy link
Copy Markdown
Contributor Author

Tests look related?

Related tests are fixed.

@susnux
susnux merged commit e7c103a into master Oct 14, 2026
178 of 181 checks passed
@susnux
susnux deleted the fix/share-rename branch October 14, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants