Changeset 34252 for trunk/src/wp-includes/comment-functions.php

Timestamp:

09/16/2015 10:25:07 PM (9 years ago)

Author:

boonebgorges

Message:

Improve consistency of comment notification callback signatures.

Both wp_new_comment_notify_moderator() and wp_new_comment_notify_postauthor()
now accept a single argument: $comment_ID.

Props SergeyBiryukov.
Fixes #33587.

File:

• trunk/src/wp-includes/comment-functions.php (modified) (1 diff)

trunk/src/wp-includes/comment-functions.php

@@ -1647 +1647 @@
  * @since 4.4.0
  *
- * @param int $comment_ID       ID of the comment.
- * @param int $comment_approved Whether the comment is approved.
+ * @param int $comment_ID ID of the comment.
  * @return bool True on success, false on failure.
  */
-function wp_new_comment_notify_moderator( $comment_ID, $comment_approved ) {
+function wp_new_comment_notify_moderator( $comment_ID ) {
+    $comment = get_comment( $comment_ID );
+
     // Only send notifications for pending comments.
-    if ( '0' != $comment_approved ) {
+    if ( '0' != $comment->comment_approved ) {
         return false;
     }