WordPress.org

Make WordPress Core

Opened 11 years ago

Last modified 2 years ago

#13041 reviewing defect (bug)

Canonical redirection will redirect a non-exitant page to a post.

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

Description

Following on from #12601

Canonical redirection will kick in for ​http://localhost/wordpress-commit/apage/something and redirect it to ​http://localhost/wordpress-commit/2010/02/13/something-else/

#12601 limited redirect_guess_404_permalink() to only searching in the same post_type if one was provided, for posts and pages however, which are builtin, these vars are not set.

I'm attaching a patch which will prevent a Page -> Post redirection, but not the other way around. - this patch was originally added to the other ticket.

Attachments (2)

12601.2.diff​ (963 bytes) - added by dd32 11 years ago.
13041.diff​ (4.4 KB) - added by wonderboymusic 6 years ago.

Download all attachments as: .zip

Change History (7)

@dd32
11 years ago

#1 @hakre
11 years ago

Patch still applies clean. It's in wp-includes/

#2 @nacin
11 years ago

  • Milestone changed from Awaiting Triage to Future Release

#3 @chriscct7
7 years ago

  • Keywords needs-refresh added; has-patch needs-testing removed

@wonderboymusic
6 years ago

#4 @wonderboymusic
6 years ago

  • Keywords has-patch added; needs-refresh removed
  • Owner changed from markjaquith to dd32
  • Status changed from new to reviewing

13041.diff​ stops /something/ (a broken page) from redirecting to /2015/09/18/something-else/ - requires a little bit of guessing what is happening. This breaks 2 unit tests, which is expected as it fixes the issue described in the ticket.

#5 @wonderboymusic
6 years ago

#19693 was marked as a duplicate.

Note: See TracTickets for help on using tickets.