Opened 8 months ago
Last modified 2 months ago
#50163 accepted defect (bug)
Perform a canonical redirect when paginated states of the front page are not found
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Canonical | Keywords: | has-patch has-screenshots needs-unit-tests |
Focuses: | Cc: |
Description
As a followup to #45337, [34492], and [47727].
When example.com/ has a static front page assigned and uses <!--nextpage-->
accessing non-existing pages uses example.com/page/3/ and Canonical won't kick in.
This is because paged states of the front page use the paged
query var, not the page
query var.
Should also fix https://meta.trac.wordpress.org/ticket/5184 :)
Attachments (3)
Change History (14)
This ticket was mentioned in PR #270 on WordPress/wordpress-develop by dd32.
8 months ago
#2
@
8 months ago
- Milestone changed from Awaiting Review to 5.5
- Owner set to SergeyBiryukov
- Status changed from new to accepted
Thanks for the ticket, I'm working on this as part of #28081 :)
#3
@
6 months ago
- Milestone changed from 5.5 to 5.6
Didn't get a chance to write the tests for this in time for 5.5 RC, moving to 5.6.
#4
@
3 months ago
- Keywords needs-refresh added
Adding the refresh keyword for the PR merge conflict.
This ticket was mentioned in Slack in #core by helen. View the logs.
3 months ago
#6
@
3 months ago
- Keywords has-screenshots added; needs-refresh removed
50163.diff
refreshes the previous patch. It appears to fix the issue.
#7
@
3 months ago
Patch 50163.diff applied fine in trunk version
#8
@
2 months ago
- Milestone changed from 5.6 to 5.7
Thanks for the refresh! This still needs tests though, moving to the next release for now.
#9
@
2 months ago
Just to be sure: do we need unit tests @SergeyBiryukov ?
Because in my testing the suggested patch fixes the issue.
#10
@
2 months ago
- Keywords needs-unit-tests added
Yes, sorry I was not clear enough :) Any functional changes to the Canonical component should really be accompanied by unit tests, to avoid unexpected regressions like comment:40:ticket:12456.
See https://core.trac.wordpress.org/ticket/50163