Changeset 48133 for trunk/src/wp-includes/comment-template.php

Timestamp:

06/23/2020 05:22:39 AM (4 years ago)

Author:

whyisjake

Message:

Comments: Ensure the proper comment count and pages for unapproved comments.

Previiously, unapproved comments can alter the comment count, returning incorrect page numbers.

Fixes #8973.

Props GregMulhauser, dd32, ryan, mrmist, hakre, solarissmoke, billerickson, ericlewis, SergeyBiryukov, chriscct7, dossy, lukecavanagh, renggo888, jdorner, matjack1, pento, audrasjb, imath, davidbaumwald, whyisjake.

File:

• trunk/src/wp-includes/comment-template.php (modified) (4 diffs)

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

@@ -1331 +1331 @@
  *
  * @since 1.5.0
+ * @since 5.5.0 Removed the need to use the $user_ID global.
  *
  * @global WP_Query   $wp_query         WordPress Query object.
@@ -1338 +1339 @@
  * @global WP_Comment $comment          Global comment object.
  * @global string     $user_login
- * @global int        $user_ID
  * @global string     $user_identity
  * @global bool       $overridden_cpage
@@ -1348 +1348 @@
  */
 function comments_template( $file = '/comments.php', $separate_comments = false ) {
-    global $wp_query, $withcomments, $post, $wpdb, $id, $comment, $user_login, $user_ID, $user_identity, $overridden_cpage;
+    global $wp_query, $withcomments, $post, $wpdb, $id, $comment, $user_login, $user_identity, $overridden_cpage;
 
     if ( ! ( is_single() || is_page() || $withcomments ) || empty( $post ) ) {
@@ -1397 +1397 @@
     }
 
-    if ( $user_ID ) {
-        $comment_args['include_unapproved'] = array( $user_ID );
-    } else {
-        $unapproved_email = wp_get_unapproved_comment_author_email();
-
-        if ( $unapproved_email ) {
-            $comment_args['include_unapproved'] = array( $unapproved_email );
-        }
+    $include_unapproved = wp_get_include_unapproved_comments_argument();
+    if ( $include_unapproved ) {
+        $comment_args['include_unapproved'] = $include_unapproved;
     }