[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