[wp-trac] [WordPress Trac] #35175: Page parameter no longer works in wp_list_comments

WordPress Trac noreply at wordpress.org
Sun Jan 3 02:06:06 UTC 2016


#35175: Page parameter no longer works in wp_list_comments
--------------------------+---------------------------
 Reporter:  smerriman     |       Owner:  boonebgorges
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  4.4.1
Component:  Comments      |     Version:  4.4
 Severity:  normal        |  Resolution:  fixed
 Keywords:  fixed-major   |     Focuses:
--------------------------+---------------------------
Changes (by boonebgorges):

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


Comment:

 In [changeset:"36158"]:
 {{{
 #!CommitTicketReference repository="" revision="36158"
 Ensure that non-default pagination values work in `wp_list_comments()`.

 Prior to 4.4, it was possible to pass 'page' and 'per_page' values to
 `wp_list_comments()` that do not match the corresponding global query
 vars.
 This ability was lost in 4.4 with the refactor of how
 `comments_template()`
 queries for comments; when the main comment query started fetching only
 the
 comments that ought to appear on a page, instead of all of a post's
 comments,
 it became impossible for the comment walker to select comments
 corresponding to
 custom pagination parameters. See #8071.

 We restore the previous behavior by (a) detecting when a 'page' or
 'per_page'
 parameter has been passed to `wp_list_comments()` that does not match the
 corresponding query vars (so that the desired comments will not be found
 in
 `$wp_query`), and if so, then (b) querying for all of the post's comments
 and
 passing them to the comment walker for pagination, as was the case before
 4.4.

 Merges [36157] to the 4.4 branch.

 Props boonebgorges, smerriman.
 Fixes #35175.
 }}}

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


More information about the wp-trac mailing list