Opened 13 years ago
Closed 9 years ago
#18762 closed defect (bug) (invalid)
Pagination does not work correctly on walker in reverse order.
Reported by: | gburtini | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 3.2 |
Component: | Comments | Keywords: | needs-testing |
Focuses: | Cc: |
Description
Steps to reproduce:
- Go to Discussion and change Comments should be displayed with the comments at the top of each page to "newer" (that is, set comment_order='desc').
- Make sure your theme is paginating comments. Make sure you have more than the page limit of comments.
- Note that the first page does not actually contain the newest comments (they're off by a page, or perhaps they're just sorted in one way for the pagination and a different way for the inner page sort).
If you check out the $start, $oldstart and $end variables, the way it was written before the patch, $start gets set to 11 (page limit + 1) on page = 1, the way it is written after the patch, start is 0, as it should be.
All I've done is swapped the order of $start and $end being updated. This fixes the starting point. This needs to be tested in more detail before actually deploying it, but there is definitely a pretty serious bug here.
Attachments (1)
Change History (8)
This ticket was mentioned in Slack in #core-comments by rachelbaker. View the logs.
9 years ago
#5
@
9 years ago
- Keywords needs-testing added; needs-refresh removed
Has anyone reproduced this issue lately? The logic surrounding comment pagination was rewritten in 4.4, and it's possible that this bug was fixed as a part of that work.
Patch to fix strange sort order.