[wp-trac] Re: [WordPress Trac] #7835: Admin comment notification
setting is misleading, and behaviour is undesirable
WordPress Trac
wp-trac at lists.automattic.com
Wed Oct 22 20:50:25 GMT 2008
#7835: Admin comment notification setting is misleading, and behaviour is
undesirable
---------------------------+------------------------------------------------
Reporter: simonwheatley | Owner: anonymous
Type: defect | Status: new
Priority: normal | Milestone: 2.8
Component: General | Version: 2.6.1
Severity: normal | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Comment (by SimonK):
Actually, this setting is already used for e-mails to individual users, so
the quick fix wouldn't solve this problem.
The admin doesn't gets mailed any longer with every comment, this setting
affects the e-mails the ''post author'' gets, which means it should be a
per user setting. Here's the code, found in wp-includes/comment.php in
line 738.
{{{
if ( get_option('comments_notify') &&
$commentdata['comment_approved'] && $post->post_author !=
$commentdata['user_ID'] )
wp_notify_postauthor($comment_ID,
$commentdata['comment_type']);
}
}}}
Notice how it gets a ''global'' option, but triggers the
wp_notify_postauthor function. This is really confusing, hope it can be
fixed. This setting should either be used for comment notifications to the
admin, or the setting should be moved to the user profile, so that each
user can decide whether or not to receive e-mails of new comments. In this
case a plugin can take over e-mailing the blog admin of ''each'' comment.
--
Ticket URL: <http://trac.wordpress.org/ticket/7835#comment:1>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list