Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Oct 29, 2025

Summary

Its deprecated since Nextcloud 18 and nowadays we have much more complex logic for this.
So the proper replacement is to use validateFilename from @nextcloud/files package.

Checklist

@susnux susnux added this to the Nextcloud 33 milestone Oct 29, 2025
@susnux susnux requested a review from a team as a code owner October 29, 2025 22:22
@susnux susnux added 3. to review Waiting for reviews technical debt pending documentation This pull request needs an associated documentation update ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Oct 29, 2025
@susnux susnux requested review from artonge, sorbaugh and szaimen and removed request for a team October 29, 2025 22:22
@susnux susnux self-assigned this Oct 29, 2025
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@susnux susnux force-pushed the chore/remove-oc-blacklist branch from 8f6ea6a to 4255cee Compare October 31, 2025 13:27
@susnux
Copy link
Contributor Author

susnux commented Oct 31, 2025

/compile

@susnux
Copy link
Contributor Author

susnux commented Oct 31, 2025

Documentation: nextcloud/documentation#13828

@susnux susnux removed the pending documentation This pull request needs an associated documentation update label Oct 31, 2025
@susnux susnux force-pushed the chore/remove-oc-blacklist branch from d4e6881 to 6cc81ff Compare November 1, 2025 11:46
@susnux
Copy link
Contributor Author

susnux commented Nov 1, 2025

/compile amend

@nextcloud-command nextcloud-command force-pushed the chore/remove-oc-blacklist branch from 6cc81ff to bb8db46 Compare November 1, 2025 12:24
Its deprecated since Nextcloud 18 and nowadays we have much more complex
logic for this.
So the proper replacement is to use `validateFilename` from
`@nextcloud/files` package.

Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the chore/remove-oc-blacklist branch from bb8db46 to f52e838 Compare November 1, 2025 12:52
@susnux susnux merged commit 9f40d4c into master Nov 1, 2025
135 of 137 checks passed
@susnux susnux deleted the chore/remove-oc-blacklist branch November 1, 2025 13:39
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants