[wp-trac] [WordPress Trac] #50163: Perform a canonical redirect when paginated states of the front page are not found

WordPress Trac noreply at wordpress.org
Mon Sep 30 13:01:43 UTC 2024


#50163: Perform a canonical redirect when paginated states of the front page are
not found
-------------------------------------------------+-------------------------
 Reporter:  dd32                                 |       Owner:
                                                 |  hellofromTonya
     Type:  defect (bug)                         |      Status:  reopened
 Priority:  normal                               |   Milestone:  6.7
Component:  Canonical                            |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-screenshots early      |     Focuses:
  has-unit-tests has-testing-info commit         |
-------------------------------------------------+-------------------------
Changes (by david.binda):

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


Comment:

 This might be a breaking change for some plugins. I've run into an issue
 with bbpress which is setup the way there is a `[bbp-topic-index]`
 shortcode used on a page which is set as a static homepage for a blog.

 Under such setup, the pagination of the topics is broken, as the `/page/2`
 is being redirected back to `/` (means the r59091 works as expected).


 Reproduction steps:

 1. install the bbpress plugin
 1. have a page with `[bbp-topic-index]`
 1. set is a static homepage
 1. create at least 2 bbpress topics
 1. set the number of topics in the bbpress's setting screen in `Topics and
 Replies Per Page` to `1`
 1. check that the homepage shows the list of topics
 1. try to go to 2nd page of the listing

 My understanding is that bbpress is using the
 [https://bbpress.trac.wordpress.org/browser/trunk/src/bbpress.php#L269
 `paged` query] var in the
 [https://bbpress.trac.wordpress.org/browser/trunk/src/bbpress.php#L969
 rewrite rules it's creating].

 It worked well until the r59091 landed, which changed the `page` check to
 `paged` for front pages.

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


More information about the wp-trac mailing list