Skip to:
Content

bbPress.org

Opened 4 years ago

Last modified 15 months ago

#3411 new enhancement

Some reply context options are unavailable when topic is marked as spam

Reported by: clorith's profile 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)

jjj-sc-2021-01-03 at 02.25.47@2x.png (81.4 KB) - added by johnjamesjacoby 4 years ago.
Spam action on Trashed replies in 2.6

Download all attachments as: .zip

Change History (4)

@johnjamesjacoby
4 years ago

Spam action on Trashed replies in 2.6

#1 @johnjamesjacoby
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 @Clorith
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.

This ticket was mentioned in Slack in #forums by thelmachido. View the logs.


15 months ago

Note: See TracTickets for help on using tickets.