[wp-trac] [WordPress Trac] #8071: Refrain from querying all the comments on a post when paged

[WordPress Trac]

#8071: Refrain from querying all the comments on a post when paged
--------------------------------------+---------------------------
 Reporter:  markjaquith               |       Owner:  boonebgorges
     Type:  enhancement               |      Status:  closed
 Priority:  normal                    |   Milestone:  4.4
Component:  Comments                  |     Version:  2.7
 Severity:  normal                    |  Resolution:  fixed
 Keywords:  dev-feedback needs-patch  |     Focuses:
--------------------------------------+---------------------------

Comment (by boonebgorges):

 In [changeset:"34729"]:
 {{{
 #!CommitTicketReference repository="" revision="34729"
 Simplify pagination logic in `comments_template()`.

 [34561] "fixed" the problem of newest-first comments showing fewer than
 'per_page' comments on the post permalink when the total number of
 comments
 was not divisible by 'per_page'. See #29462. But this fix caused numerous
 other problems. First, comment pages reported by `get_page_of_comment()`
 (which expects comment pages to be filled oldest-first) were no longer
 correct.
 Second, and more seriously, the new logic caused comments to be shifted
 between pages, making their permalinks non-permanent.

 The current changeset reverts the changed behavior. In order to preserve
 the
 performance improvements introduced in [34561], an additional query must
 be
 performed when 'default_comments_page=newest' and 'cpage=0' (ie, you're
 viewing
 the post permalink). A nice side effect of this revert is that we no
 longer
 need the hacks required to determine proper comment pagination, introduced
 in
 [34561].

 See #8071. See #34073.
 }}}

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