Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#7531 closed enhancement (invalid)

Enhancement for Akismet

Reported by: ozh Owned by: matt
Milestone: Priority: normal
Severity: normal Version:
Component: Optimization Keywords: akismet, has-patch, spam
Focuses: Cc:


(not even sure this is the proper place to do so...)

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:)

Attachments (1)

akismet_die_on_spam.diff (1.5 KB) - added by ozh 7 years ago.
Die on spam

Download all attachments as: .zip

Change History (6)

@ozh7 years ago

Die on spam

comment:1 @Viper007Bond7 years ago

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

"(not even sure this is the proper place to do so...) "

It's not. This is: http://plugins.trac.wordpress.org/

comment:2 @ozh7 years ago

  • Resolution invalid deleted
  • Status changed from closed to reopened

Tickets for Hello Dolly have been raised, patched and fixed here so I think Akismet is a similar edge case since it's bundled.

comment:3 @Viper007Bond7 years ago

Hello Dolly is a part of WordPress though, Akismet is not. It's an external: http://trac.wordpress.org/browser/trunk/wp-content/plugins (even note the external URL)

All changes/patches are made via the plugins Trac: http://plugins.trac.wordpress.org/log/akismet

Further proof: #6946 #6565 #5549 #4548 #3818 #3393

Need I keep going?

comment:4 @ozh7 years ago

  • Resolution set to invalid
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.