WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Last modified 7 years ago

#20388 new defect (bug)

?cpage=N URLs do not have canonical redirection

Reported by: markjaquith Owned by:
Priority: normal Milestone:
Component: Canonical Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

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

Attachments (1)

20388-tests.patch​ (740 bytes ) - added by boonebgorges 12 years ago.

Download all attachments as: .zip

Change History (6)

#1 @cklosows
14 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 14 years ago by cklosows (next)

#3 @wonderboymusic
13 years 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.

#4 @wonderboymusic
13 years 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.

@boonebgorges
12 years ago

#5 @chriscct7
11 years ago

  • Keywords has-patch added
Note: See TracTickets for help on using tickets.