[wp-trac] [WordPress Trac] #33587: Notification Functions Should be Replaced by Hooks

WordPress Trac noreply at wordpress.org
Fri Sep 4 20:53:55 UTC 2015


#33587: Notification Functions Should be Replaced by Hooks
-------------------------+------------------------------
 Reporter:  dshanske     |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Mail         |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |     Focuses:
-------------------------+------------------------------

Comment (by dshanske):

 Well, you are right that it is probably better to put the hook further up
 in the process.

 The part of the guts of the functions I do think should be ripped out is
 the part that generates the comment information to be placed in the
 notification. This is duplicative between wp_notify_postauthor and
 wp_notify_moderator and would be better served if the same code did both.
 Also...if you note pfefferle's comment, the part you might want to enhance
 the display of.

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


More information about the wp-trac mailing list