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

[WordPress Trac]

#761: Add hook to conditionally disable comment notifications
------------------------------------+-----------------------------
 Reporter:  coffee2code             |       Owner:  SergeyBiryukov
     Type:  enhancement             |      Status:  assigned
 Priority:  normal                  |   Milestone:  4.4
Component:  Comments                |     Version:  1.2.2
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:
------------------------------------+-----------------------------

Comment (by adamsilverstein):

 @thomaswm - thanks for pointing out that change, it is closely related.

 Although the new method of firing the notifications does enable better
 filtering of notifications, for example disabling them, it does not
 override the settings in the same way the hooks on this ticket.

 With the hooks here, your filter can force an email to be sent even if it
 would normally not be, or the opposite.  For example, you could have a
 plugin that always sends you an email when certain keywords are used in a
 post, even if your normal settings are no notifications.

 In any case, the new hook triggering likely means my patch needs a refresh
 :) thanks again for pointing it out.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/761#comment:28>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform