[wp-trac] [WordPress Trac] #33717: Send Notification Email When a Comment is Approved From Moderation

WordPress Trac noreply at wordpress.org
Sun Sep 20 11:11:24 UTC 2015


#33717: Send Notification Email When a Comment is Approved From Moderation
-----------------------------+-------------------------
 Reporter:  jeffr0           |       Owner:  swissspidy
     Type:  feature request  |      Status:  assigned
 Priority:  normal           |   Milestone:  4.4
Component:  Comments         |     Version:
 Severity:  normal           |  Resolution:
 Keywords:  has-patch        |     Focuses:
-----------------------------+-------------------------

Comment (by swissspidy):

 Replying to [comment:32 obenland]:
 > `Notify a comment author when his comment gets approved.` - Let's use a
 gender-neutral `their`.

 Done in the latest patch.

 > We usually don't add an empty line between `@param` and `@return`
 (unfortunately).

 I know, but my IDE unfortunately always adds an empty line, even though I
 configured all sniffs :/

 > The meta key `_wp_notification_sent` seems a bit generic.

 Renamed it to `_wp_commenter_notification_sent`.

 > I also wonder if we should accept both a comment ID and a WP_Comment
 object to be more in line with `wp_notify_postauthor()`.

 Good idea. That way the function can also be hooked to other actions where
 only an ID is passed. Added now.

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


More information about the wp-trac mailing list