[wp-trac] [WordPress Trac] #25699: When filtering comment notification email recipients, notifications are not sent for author comments

[WordPress Trac]

#25699: When filtering comment notification email recipients, notifications are not
sent for author comments
--------------------------+--------------------------
 Reporter:  chipbennett   |       Owner:  markjaquith
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  3.8
Component:  Comments      |     Version:  3.7
 Severity:  minor         |  Resolution:  fixed
 Keywords:  has-patch     |
--------------------------+--------------------------
Changes (by markjaquith):

 * owner:   => markjaquith
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"26367"]:
 {{{
 #!CommitTicketReference repository="" revision="26367"
 Fix `comment_notification_recipients` filter behavior so that it is still
 respected even on comments left by the post author

 The code was bailing on this-is-a-comment-on-your-own-post detection,
 ignoring additional recipients. Now:

 * Logic check is done within `wp_notify_postauthor()`
 * Logic check is overridable via `comment_notification_notify_author`
 filter (default still false)
 * The code doesn't bail on comment-on-own-post detection, but just removes
 the author from the array
 * The code instead now bails if the recipients list is empty, so
 `comment_notification_recipients` works properly

 props ethitter.
 fixes #25699
 }}}

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