Changeset 36353 for branches/4.4/src/wp-includes/comment-template.php

Timestamp:

01/20/2016 04:32:22 AM (9 years ago)

Author:

dd32

Message:

Comments: In comments_template(), don't run hierarchical queries if comment threading is disabled.

When hierarchical=true, WP_Comment_Query will always fetch comments according to the comment hierarchy, even if 'thread_comments' is disabled for the site.
This can cause problems when comment threading is disabled after threaded comments have been recorded on the site; comments will no longer be returned in a strictly chronological order.
We address the issue by refraining from querying hierarchically when comment threading is disabled.

Merges [36226] to the 4.4 branch.
Props jmdodd.
Fixes #35378.

Location:

branches/4.4

Files:

• . (modified) (1 prop)
• src/wp-includes/comment-template.php (modified) (2 diffs)

branches/4.4/src/wp-includes/comment-template.php

@@ -1286 +1286 @@
         'status'  => 'approve',
         'post_id' => $post->ID,
-        'hierarchical' => 'threaded',
         'no_found_rows' => false,
         'update_comment_meta_cache' => false, // We lazy-load comment meta for performance.
     );
+
+    if ( get_option('thread_comments') ) {
+        $comment_args['hierarchical'] = 'threaded';
+    } else {
+        $comment_args['hierarchical'] = false;
+    }
 
     if ( $user_ID ) {
@@ -1336 +1341 @@
 
     // Trees must be flattened before they're passed to the walker.
-    $comments_flat = array();
-    foreach ( $_comments as $_comment ) {
-        $comments_flat[]  = $_comment;
-        $comment_children = $_comment->get_children( array(
-            'format' => 'flat',
-            'status' => $comment_args['status'],
-            'orderby' => $comment_args['orderby']
-        ) );
-
-        foreach ( $comment_children as $comment_child ) {
-            $comments_flat[] = $comment_child;
+    if ( $comment_args['hierarchical'] ) {
+        $comments_flat = array();
+        foreach ( $_comments as $_comment ) {
+            $comments_flat[]  = $_comment;
+            $comment_children = $_comment->get_children( array(
+                'format' => 'flat',
+                'status' => $comment_args['status'],
+                'orderby' => $comment_args['orderby']
+            ) );
+
+            foreach ( $comment_children as $comment_child ) {
+                $comments_flat[] = $comment_child;
+            }
         }
+    } else {
+        $comments_flat = $_comments;
     }