[wp-trac] [WordPress Trac] #37696: WP_Comment_Query loses sql_clauses with Object Cache

[WordPress Trac]

#37696: WP_Comment_Query loses sql_clauses with Object Cache
--------------------------+---------------------------
 Reporter:  Rarst         |       Owner:  boonebgorges
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  4.6.1
Component:  Comments      |     Version:  4.6
 Severity:  normal        |  Resolution:  fixed
 Keywords:  fixed-major   |     Focuses:
--------------------------+---------------------------
Changes (by jeremyfelt):

 * status:  reopened => closed
 * resolution:   => fixed


Comment:

 In [changeset:"38497"]:
 {{{
 #!CommitTicketReference repository="" revision="38497"
 Comments: Don't do direct SQL query when fetching decendants.

 The SQL query was built using the clauses compiled when querying for
 top-level comments. But in cases where the top-level comment query
 results are already in the cache, the SQL clauses are not built, and
 so are unavailable for `fill_descendants()`. Instead, we call
 `get_comments()`, using modified versions of the parameters passed
 to the main `WP_Comment_Query` class.

 Merge of [38446] to the 4.6 branch.

 Props boonebgorges, Akeif, Rarst for testing.
 Fixes #37696.
 }}}

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