Opened 4 years ago
Last modified 17 months ago
#3411 new enhancement
Some reply context options are unavailable when topic is marked as spam
Reported by: | Clorith | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Component - Replies | Keywords: | |
Cc: |
Description
If you mark a topic as spam, it may have legitimate replies, and spam replies mixed in under it.
Currently, if you mark a topic as spam, the context of the replies is switched. in base bbPress, they are all trashed instantly, and you then need to restore the reply, to then be able to mark it as spam.
To be able to effectively tackle a spam-topic with replies, you therefore need to start by marking any replies as spam, and then mark the topic as spam, which is a backwards route to take.
Would it make sense to still include the spam action on trashed replies?
Attachments (1)
Change History (4)
#1
@
4 years ago
Hey @Clorith 👋
bbPress inherited some of this behavior from bbPress 1.x, because transitioning between statuses is tricky, especially with hierarchical or linked data (topics, replies, hierarchical replies, etc...)
Over the years of bbPress 2.x, we've tried to identify and introduce more support for more transitions, and I am pretty sure we have this exact case covered, but I also could be misunderstanding. I've attached a screen shot that shows the row-actions that I'm thinking that you're thinking about. 😁
#2
@
3 years ago
Sorry, completely missed your reply here.
I'm referring to the front-end actions shown under each post when looking at a topic, since you'd ideally not want any moderators to be tied to the back-end.
This appears to be caused by bbp_get_reply_admin_links()
, specifically https://bbpress.trac.wordpress.org/browser/trunk/src/includes/replies/template.php#L1799 which explicitly removes the trash
and spam
options if the topic it self is marked as spam.
Spam action on Trashed replies in 2.6