Changeset 36381 for trunk/src/wp-includes/class-wp-comment-query.php

Timestamp:

01/22/2016 08:19:49 PM (9 years ago)

Author:

boonebgorges

Message:

Allow comment query results to be limited to comments with comment_post_ID = 0.

Previously, this was not possible due to an overly broad empty() check.

Passing null, false, or '' to 'post_id', or omitting 'post_id'
altogether, will continue to return comments regardless of comment_post_ID,
as before. Passing 0 or '0' will limit results to comments with no
associated post.

Props danielbachhuber.
Fixes #35090.

File:

• trunk/src/wp-includes/class-wp-comment-query.php (modified) (3 diffs)

trunk/src/wp-includes/class-wp-comment-query.php

@@ -215 +215 @@
      *     @type int          $post_ID                   Currently unused.
      *     @type int          $post_id                   Limit results to those affiliated with a given post ID.
-     *                                                   Default 0.
+     *                                                   Default null.
      *     @type array        $post__in                  Array of post IDs to include affiliated comments for.
      *                                                   Default empty.
@@ -277 +277 @@
             'post_author__not_in' => '',
             'post_ID' => '',
-            'post_id' => 0,
+            'post_id' => null,
             'post__in' => '',
             'post__not_in' => '',
@@ -646 +646 @@
         }
 
-        $post_id = absint( $this->query_vars['post_id'] );
-        if ( ! empty( $post_id ) ) {
-            $this->sql_clauses['where']['post_id'] = $wpdb->prepare( 'comment_post_ID = %d', $post_id );
+        if ( strlen( $this->query_vars['post_id'] ) ) {
+            $this->sql_clauses['where']['post_id'] = $wpdb->prepare( 'comment_post_ID = %d', $this->query_vars['post_id'] );
         }