[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