Opened 5 weeks ago
Last modified 5 weeks 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. :)
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