Skip to:
Content

bbPress.org

Opened 6 years ago

Closed 3 years ago

Last modified 7 months ago

#2246 closed enhancement (fixed)

Add 'mark as spam' to bulk actions in Topics/Replies admin

Reported by: DJPaul Owned by:
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.2.3
Component: General - Administration Keywords: has-patch
Cc: wordpress@…, jmdodd@…

Description

It'd be very handy from a workflow perspective if bbPress had a "mark as spam" option in the bulk actions dropdown inside the Topics/Replies wp-admin screens.

Currently there are only "edit" and "move to trash" options.

Attachments (2)

2246.diff (8.8 KB) - added by jmdodd 3 years ago.
First pass at adding bulk spam/unspam functionality to topics and replies in wp-admin.
2246.2.diff (10.4 KB) - added by netweb 3 years ago.

Download all attachments as: .zip

Change History (23)

#2 @alex-ye
6 years ago

  • Version set to 2.2.3

#3 @johnjamesjacoby
6 years ago

  • Milestone changed from Awaiting Review to 2.4

Makes sense, and not difficult to do.

#4 follow-up: @MZAWeb
6 years ago

  • Cc wordpress@… added

Actually, as we're not doing our own WP_List_Table child for our admin lists, but instead hooking into the standard WP_Posts_List_Table, it's not easy to add bulk actions in a non-hacky way. @johnjamesjacoby have you considered moving our tables to WP_List_Table childs?

#5 @MZAWeb
6 years ago

by "not easy" I meant "impossible". Only way I could find is via jQuery, which is insane.

#6 in reply to: ↑ 4 @johnjamesjacoby
6 years ago

  • Milestone changed from 2.4 to 2.5

Replying to MZAWeb:

Actually, as we're not doing our own WP_List_Table child for our admin lists, but instead hooking into the standard WP_Posts_List_Table, it's not easy to add bulk actions in a non-hacky way. @johnjamesjacoby have you considered moving our tables to WP_List_Table childs?

Yes, and it's possible we'll need to do this at some point, to avoid trying to hack these things in.

Moving to 2.5.

#7 @MZAWEb
6 years ago

I'd love to do that, if no one beats me to it. Will start asap.

#8 follow-up: @MZAWeb
6 years ago

Core hardcodes which WP_List_Table child is used in every instance (see function _get_list_table )

Until this is resolved http://core.trac.wordpress.org/ticket/18449 doesn't make much sense to do this ticket, because we'll need to do some ugly hacks to make it work.

#9 in reply to: ↑ 8 @johnjamesjacoby
6 years ago

  • Milestone changed from 2.5 to 2.6

Thanks for keeping an eye on this. Agree there's little value in doing more work than should be needed.

Also, 2.5 is going to focus on performance improvements, so moving to 2.6.

#10 @johnjamesjacoby
6 years ago

  • Milestone changed from 2.6 to 2.7

Bump to 2.7.

#11 @johnjamesjacoby
5 years ago

  • Milestone changed from 2.7 to 2.8

Bumping all 2.7 to 2.8 milestone.

#12 @Ipstenu
3 years ago

Any traction on this? It makes managing large sites like WordPress.org almost impossible.

This ticket was mentioned in Slack in #meta by ipstenu. View the logs.


3 years ago

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


3 years ago

#15 @Kenshino
3 years ago

I'm weeping now while trying to clear spam from un-managed forums on rosetta sites

@jmdodd
3 years ago

First pass at adding bulk spam/unspam functionality to topics and replies in wp-admin.

#16 @jmdodd
3 years ago

  • Cc jmdodd@… added
  • Keywords has-patch added

@netweb
3 years ago

#17 @netweb
3 years ago

  • Milestone changed from 2.8 to 2.6

See also:

I've made a couple of tweaks to 2246.diff in 2246.2.diff the patch, doc tweaks, and only add the filters for WP >= 4.7, #WP16031 has been reopened with some changes requested.

Going to commit 2246.2.diff and leave this ticket open for further follow up.

#18 @netweb
3 years ago

In 6101:

General - Administration: Introduce bulk actions to spam and unspam topics and replies in wp-admin

This Friday #yolo changeset comes care of the upcoming WordPress 4.7 release via #WP16031 / wp:changeset:38647

Props jmdodd.
See #2246.

#19 @netweb
3 years ago

In 6102:

General - Administration: Add i18n text domain missed in [6101]

See #2246.

#20 @johnjamesjacoby
3 years ago

  • Resolution set to fixed
  • Status changed from new to closed

Looks like this is in. Anything left to do can reference this, or open new ticket if they are major new bugs.

#21 @johnjamesjacoby
7 months ago

Assigning all closed & unassigned tickets in the 2.6 milestone to myself.

Note: See TracTickets for help on using tickets.