Ticket #1786 (closed enhancement: fixed)

Opened 6 years ago

Last modified 6 years ago

Add hook for blank or incorrect comment field errors

Reported by: filosofo Owned by: markjaquith
Priority: normal Milestone: 2.1
Component: General Version: 1.6
Severity: normal Keywords: comments error bg|has-patch bg|commit
Cc:

Description

In this discussion ( button patch for comment post section), someone suggested adding a function to improve the way WordPress handles blank comment fields and missing required emails (In other words, do something other than just die with 'Error: please enter a valid email address,' for example).

My attached patch instead adds a hook that will allow plugin authors and theme developers to create their own custom error responses.

The default action is the existing die behavior.

Attachments

wp-comments-post.php.diff Download (961 bytes) - added by filosofo 6 years ago.
wp-comments-post.php.2.diff Download (1.5 KB) - added by filosofo 6 years ago.
A better version of the same idea, using wp_die

Change History

  • Keywords bg|has-patch bg|commit added
  • Owner changed from anonymous to markjaquith
  • Status changed from new to assigned

I like it!

comment:2   matt6 years ago

  • Milestone changed from 1.6 to 2.1

A better version of the same idea, using wp_die

  • Owner changed from markjaquith to filosofo
  • Status changed from assigned to new

I've uploaded a new patch that uses wp_die to accomplish what I was hoping to do with this back when I started this ticket.

  • Owner changed from filosofo to markjaquith

Oops, didn't mean to change the assignment just now.

comment:5   ryan6 years ago

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

(In [3984]) wp_die() for comment errors. Props filosofo. fixes #1786

Note: See TracTickets for help on using tickets.