Skip to:
Content

bbPress.org

Opened 3 months ago

Last modified 3 months ago

#3395 new enhancement

Akismet spam cleanup functionality

Reported by: stephdau Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Extend - Akismet Keywords:
Cc:

Description

This was first proposed on Slack here.

In the WP AK plugin, we have a scheduled function Akismet::delete_old_comments_meta() to delete old akismet_as_submitted meta records, because they are pretty large, and not useful at all after a while.

Since then, @johnjamesjacoby suggested:

We could go as far as moving it out of post meta and into a custom post type with a post_parent relationship to the topic/reply. That would allow for some longer data retention, without the surplus of metadata keys.

I'm attaching a patch with what I had started before JJJ add his comment, since I have it, but leaving any fancier concept to the bbPress team to bounce around.

Note that my patch does not actually have the scheduling of said cleanup, as I wasn't sure where it would be best to trigger that in bbPress. See:

/**
 * The above will need to be scheduled with something like:
 * if ( ! wp_next_scheduled( 'akismet_scheduled_delete' ) ) {
 * 	wp_schedule_event( time(), 'daily', 'akismet_scheduled_delete' );
 * }
 */

Also note that the patch in wholly untasted, since I didn't get to keep going. :)

Attachments (1)

3395.diff (5.9 KB) - added by stephdau 3 months ago.

Download all attachments as: .zip

Change History (2)

@stephdau
3 months ago

#1 @stephdau
3 months ago

Another note: I will be off on a sabbatical in Sept/Oct/Nov, so I'll tag my fellow Akismet devs on this ticket, so they can keep up with it, and your team.

Ping @cfinke and @procifer. Tag! ;p

Last edited 3 months ago by stephdau (previous) (diff)
Note: See TracTickets for help on using tickets.