#4413 closed defect (bug) (fixed)
Pagination components shouldn't link to 'page 1' state
| Reported by: | jonoaldersonwp | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | |
| Component: | Support Forums | Keywords: | seo |
| Cc: |
Description
On templates like https://wordpress.org/support/view/support-forum-no/page/2/, the link to the first page in the pagination component links to /page/1/. This should instead link to the series root.
Change History (5)
#1
@
7 years ago
Version 0, edited 7 years ago by
(next)
This ticket was mentioned in Slack in #forums by clorith. View the logs.
7 years ago
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Ultimately caused by WordPress's
paginate_links()function returning/page/1/links, and bbPress doing a very bland attempt at working around this in https://bbpress.trac.wordpress.org/browser/branches/2.5/includes/topics/template.php?marks=409#L405 by only replacing/page/1/'with'.. but core uses"for some links.This is fixed in bbPress 2.6 with bbp_paginate_links() I believe, which we should probably look at moving to instead.