[wp-trac] [WordPress Trac] #761: Add hook to conditionally disable comment notifications [w/ patch]

WordPress Trac wp-trac at lists.automattic.com
Mon Aug 6 02:12:44 UTC 2012


#761: Add hook to conditionally disable comment notifications [w/ patch]
-------------------------+-----------------------
 Reporter:  coffee2code  |       Owner:  matt
     Type:  enhancement  |      Status:  reopened
 Priority:  normal       |   Milestone:  3.5
Component:  Comments     |     Version:  1.2.2
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+-----------------------

Comment (by scribu):

 Replying to [comment:7 johnbillion]:
 > This is an elegantly simple patch, but the main issue at hand is that
 the settings for comment notifications are inadequate. This filter should
 be taken into consideration in #7835 and #6286.

 I disagree; this filter should make sense no matter what settings are in
 place in the UI.

 Regarding the patch from coffee2code, I'm wondering why the following
 check is only done in one place:

 {{{
 && ( ! isset( $commentdata['user_id'] ) || $post->post_author !=
 $commentdata['user_id'] )
 }}}

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/761#comment:10>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list