Skip to content

[stable28] fix(files): Do not escape file names in the file picker - #44416

Merged
susnux merged 3 commits into
stable28from
backport/44407/stable28
Apr 17, 2024
Merged

[stable28] fix(files): Do not escape file names in the file picker#44416
susnux merged 3 commits into
stable28from
backport/44407/stable28

Conversation

@backportbot

@backportbotbackportbotBot commented Mar 22, 2024

Copy link
Copy Markdown

Backport of #44407

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Remove all the empty commits

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

@backportbotbackportbotBot added this to the Nextcloud 28.0.4 milestone Mar 22, 2024
@susnux
susnuxforce-pushed the backport/44407/stable28 branch from d203b33 to 21f5812CompareMarch 22, 2024 16:16
@susnux
susnux marked this pull request as ready for review March 22, 2024 16:16
@susnux
susnuxforce-pushed the backport/44407/stable28 branch from 21f5812 to 4ba63e5CompareMarch 22, 2024 17:37
@skjnldsvskjnldsv mentioned this pull request Mar 28, 2024
@AltahrimAltahrim mentioned this pull request Apr 17, 2024
5 tasks
@Altahrim

Copy link
Copy Markdown
Collaborator

/compile amend/

The text is already escaped by Vue, so we should not escape or sanitize the filename.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the backport/44407/stable28 branch from 318fece to 150e1d0CompareApril 17, 2024 10:42
@susnux
susnux enabled auto-merge April 17, 2024 10:42
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the backport/44407/stable28 branch from 150e1d0 to 721521fCompareApril 17, 2024 12:19
@susnux
susnux merged commit 8c87769 into stable28Apr 17, 2024
@susnux
susnux deleted the backport/44407/stable28 branch April 17, 2024 14:13
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Altahrim@susnux@Pytal@skjnldsv