Make WordPress Core

Opened 3 years ago

Last modified 22 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:


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

Attachments (1)

20388-tests.patch (740 bytes) - added by boonebgorges 10 months ago.

Download all attachments as: .zip

Change History (5)

comment:1 @cklosows3 years ago

  • Cc cklosowski@… added

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

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

Version 0, edited 3 years ago by cklosows (next)

comment:3 @wonderboymusic22 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 @wonderboymusic22 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.