Enhancement for Akismet

[Ozh]

I have an enhancement proposal for Akismet: when catching a spam, don't return to comment page afterwards but instead clearly show the spammer that the comment was rejected.

Basically, when catching spam:

add_filter('comment_post_redirect', 'akismet_reject_screen');

function akismet_reject_screen() {
	header('HTTP/1.1 403 Forbidden');
	wp_die(__('Sorry, your comment was considered as spam and rejected'));
}

I don't see the point in wasting bandwidth, CPU and DB queries to show a page to someone who has been identified as a spammer :)

(other minor "enhancement" included: Matt's URL updated:)

[Ozh]

Die on spam

[tellyworth]

I'd suggest we should do the opposite:

​http://core.trac.wordpress.org/ticket/8968

[tellyworth]

The new patch will:

• redirect rather than die if a comment is discarded

• put the comment in the trash rather than discard it entirely when the akismet_discard_month conditional is satisfied (option is set and the post hasn't been updated in more than 30 days)

For WP versions older than 2.9 (ie without trash support) it should keep the old behaviour.

[josephscott]

(In [274580]) - When deleting spam comments on old posts, use the trash status.

Fixes #902 props tellyworth

• Catch up on change log items

[tellyworth]

I reverted [274580] because it was causing confusion (users saw spam going to trash but didn't know why).

Postponing till 2.5. If we can use comment_meta to record why a comment is put in the trash it should eliminate that confusion.

[tellyworth]

removing the akismet-2.5 tag for now, till we decide how best to handle this

[tellyworth]

For 2.5 here's what we'll do:

1. Re-word the checkbox label to make the meaning of the akismet_discard_month setting clearer

2. Replace the die() with a redirect.

Then re-visit this later, 2.6 perhaps.

[tellyworth]

Checkbox label was reworded in [292719]

[xknown]

The mentioned changes were introduced in r319990