[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