Issue with recovering posts to un-deleted state when permanently deleting

zogstrip · 23 maart 2026 om 16:24

Thanks for the report This will be fixed by

github.com/discourse/discourse

FIX: Prevent failed permanent delete from recovering soft-deleted post (#38793)

main ← fix/permanently-delete-undoes-soft-delete

merged 06:35PM - 23 Mar 26 UTC

ZogStriP

+63 -17

When an admin soft-deletes a post then tries to permanently delete it before the… 5-minute timer expires, the server correctly returns a 403. However, the client-side error handler calls `undoDeleteState()` which clears `deleted_at` and restores the post to a non-deleted state in the UI — even though it remains deleted on the server. The admin then has to delete it again and wait another 5 minutes. This happens because `destroy()` always calls `setDeletedState()` before the AJAX request, overwriting `oldCooked` even when the post is already deleted. When the request fails, `undoDeleteState()` uses that stale `oldCooked` to revert the post to a non-deleted state. The fix skips `setDeletedState()` in `destroy()` when `force_destroy` is set (the post is already deleted), and makes `undoDeleteState()` accept opts so it can bail early for `force_destroy` — centralizing the guard instead of repeating it at each call site. https://meta.discourse.org/t/398938

Topic		Antwoorden	Weergaven
Permanently delete not functioning Bug permanently-delete	6	679	23 januari 2023
Permanent deletion bug Bug permanently-delete	17	1388	30 juli 2022
Final deletion does not work correctly Bug permanently-delete	6	663	21 november 2023
Error during permanently deleting a post of a deleted user Support permanently-delete	3	96	23 september 2024
Introducing permanently delete post functionality Announcements new-feature , permanently-delete	5	9386	22 april 2026

Issue with recovering posts to un-deleted state when permanently deleting

Gerelateerde topics