Skip to content

Conversation

@theoholl
Copy link
Contributor

@theoholl theoholl commented Jan 4, 2026

Related to: #3534

Summary

On main (and maybe other branches/versions), the undelete action is broken. The web endpoint returns a 403 Permission denied error.

This is because the PermissionService::getBoard() method is called without the allowDeleted parameter (it defaults to false), and, thus, throws a DoesNotExistException error. This error is then caught in PermissionService::getPermissions(), setting both $owner = false and $acls = []. This results in all permissions being false.

This pull request adds an optional allowDeleted parameter to relevant functions.

Screencast

Screencast.Fix.mp4

@theoholl theoholl changed the title When undeleting a board (and force-deleting), pass the allowDeleted parameter through to BoardMapper::find() to find the relevant board. [main] Fix: Board cannot be undeleted Jan 4, 2026
@grnd-alt
Copy link
Member

grnd-alt commented Jan 5, 2026

thanks for your contribution, the code changes look fine, and work as expected 🔥
The test runs seem to be unrelated!

… parameter through to BoardMapper::find() to find the relevant board.

Signed-off-by: Theo <36564257+theoholl@users.noreply.github.com>
@luka-nextcloud luka-nextcloud merged commit af36557 into nextcloud:main Jan 7, 2026
24 of 39 checks passed
@github-actions
Copy link
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

@theoholl theoholl deleted the fix-undelete-board branch January 27, 2026 16:37
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.

Boards cannot be undeleted

3 participants