Skip to content

[stable31] fix(files_sharing): Hide 'Open locally' action - #56737

Merged
AndyScherzinger merged 3 commits into
stable31from
backport/56060/stable31
Nov 29, 2025
Merged

[stable31] fix(files_sharing): Hide 'Open locally' action#56737
AndyScherzinger merged 3 commits into
stable31from
backport/56060/stable31

Conversation

@backportbot

@backportbotbackportbotBot commented Nov 28, 2025

Copy link
Copy Markdown

Backport of #56060

Warning, This backport had conflicts that were resolved with the ours merge strategy and is likely incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbotbackportbotBot added bug 3. to review Waiting for reviews labels Nov 28, 2025
@backportbotbackportbotBot added this to the Nextcloud 31.0.12 milestone Nov 28, 2025
fauzanfebrianand others added 2 commits November 28, 2025 08:52
This patch ensures that the "Open locally" context menu item is not
displayed for files in a share where the "download and sync" permission
has not been granted.
This prevents user confusion, as the action would fail anyway. The fix
adds a permission check before rendering the menu item, and adds a
corresponding unit test to verify this behavior.
Resolves: #54970
Signed-off-by: Fauzan <fauzanfebriann@gmail.com>
Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
[skip ci]
@skjnldsv
skjnldsvforce-pushed the backport/56060/stable31 branch from 37d89ce to 9da57f6CompareNovember 28, 2025 08:54
@skjnldsv

Copy link
Copy Markdown
Member

/compile

@skjnldsv
skjnldsv marked this pull request as ready for review November 28, 2025 08:55
@skjnldsv
skjnldsv self-requested a review as a code ownerNovember 28, 2025 08:55
@skjnldsv
skjnldsv requested review from a team, nfebe, sorbaugh and susnux and removed request for a teamNovember 28, 2025 08:55
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@AndyScherzinger
AndyScherzinger merged commit 3cda9e3 into stable31Nov 29, 2025
128 of 130 checks passed
@AndyScherzinger
AndyScherzinger deleted the backport/56060/stable31 branch November 29, 2025 06:45
@nextcloud-botnextcloud-bot mentioned this pull request Dec 3, 2025
7 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@skjnldsv@AndyScherzinger@fauzanfebrian@nextcloud-command