Context Navigation

← Previous Ticket
Next Ticket →

#34357 closed defect (bug) (invalid)

Canonical redirects don't take endpoints into account

Reported by:	johnbillion	Owned by:
Priority:	normal	Milestone:
Component:	Canonical	Version:
Severity:	normal	Keywords:	needs-unit-tests reporter-feedback needs-testing
Cc:		Focuses:

Description

Steps to reproduce:

Publish a post and visit its /embed endpoint URL.
Change your permalink structure.
Visit the same URL again.

Note that the canonical redirect kicks in and redirects you to the correct permalink, but the /embed endpoint gets stripped.

It might be possible for redirect_canonical() to look at the registered rewrite endpoints and take them into consideration when performing a redirect.

Change History (3)

#1 @swissspidy
11 years ago

We had the same problem with wp_old_slug_redirect(), see #33920. Perhaps we can reuse some code from there.

#2 @rommelxcastro
10 years ago

Keywords reporter-feedback needs-testing added; needs-patch removed

@johnbillion - This seems to be fixed now, tested on WP version 4.8-alpha-39471 and works as expected.

Last edited 10 years ago by rommelxcastro (previous) (diff)

#3 @johnbillion
10 years ago

Milestone Awaiting Review
Resolution → invalid
Status new → closed

Thanks for verifying, @rommelxcastro.

It looks like this might have been a false report. This works as expected in at least the last few version (past the point this ticket was created).

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core