tgxworld
(Alan Tan)
Março 21, 2016, 3:39am
1
https://meta.discourse.org/t/vote-for-your-external-batterys-color/41347
Repro steps:
Create a new user
Create a new topic with the user
Using an admin user, click flag -> it's spam -> delete spammer
4 curtidas
Weird that has always worked before.
1 curtida
GlitchMr
(Konrad Borowski)
Março 21, 2016, 7:56am
3
Apparently posts intentionally aren’t deleted, and are supposed to be seen by staff (as I think from seeing an exception when deleted_user is null). I decided to send a patch:
master ← sugar700:allow-seeing-deleted-users-posts
merged 09:24AM - 21 Mar 16 UTC
Before this change, as reported on https://meta.discourse.org/t/topic-not-delete… d-after-deleting-spammer/41348:

After this change:

Apparently, Discourse intentionally doesn't remove permanently posts by spammers, but makes them invisible from non-staff. However, the code responsible for displaying posts couldn't handle the case of null user without throwing an exception.
4 curtidas
Nice @neil can you review that Canada Monday?