WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 5 months ago

#20388 new defect (bug)

?cpage=N URLs do not have canonical redirection

Reported by: markjaquith Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Canonical Keywords:
Focuses: Cc:

Description

?cpage=N URLs aren't redirected to their pretty URL counterparts. They should be.

Change History (4)

comment:1 cklosows2 years ago

  • Cc cklosowski@… added

Tested on trunk build (3.4-beta4-20838) and this appears to be working.

localhost/wordpress/hello-world?cpage=2 -> localhost/wordpress/hello-world/comment-page-2/ (via a 301 redirect)

Last edited 2 years ago by cklosows (previous) (diff)

comment:3 wonderboymusic5 months ago

In 26091:

Set the page_comments option to true in Tests_Canonical to properly check canonical redirection for paged comments. By doing so, one assertion tied to #20388 no longer needs to be.

See #20388, 25913.

comment:4 wonderboymusic5 months ago

The redirection works when the page_comments option is true. However, the other 2 assertions that check for redirection of out of range comment pages still fail.

Note: See TracTickets for help on using tickets.