[wp-trac] [WordPress Trac] #15775: Redirect Name Requests Correctly with Default Permalinks
WordPress Trac
wp-trac at lists.automattic.com
Sat Dec 11 10:21:09 UTC 2010
#15775: Redirect Name Requests Correctly with Default Permalinks
--------------------------+-------------------------------------------------
Reporter: filosofo | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.1
Component: Canonical | Version: 3.1
Severity: normal | Keywords: has-patch
--------------------------+-------------------------------------------------
Suppose a post has the slug
{{{
an-exciting-post-title
}}}
but someone requests
{{{
/?name=an-exciting-post
}}}
Then `redirect_guess_404_permalink()` correctly guesses the permalink for
that post, but the incorrect, partial `name` argument is retained:
{{{
/?p=123&name=an-exciting-post
}}}
and so the posts query fails.
Instead, when `redirect_guess_404_permalink()` returns a permalink that
doesn't have a `name` argument, the old, incorrect `name` argument should
be removed from the final redirect URL.
Note that in this patch I am using the fix appearing in the patch on
#15774, because this patch uses some of the work done there. Applying
this patch will also fix #15774.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/15775>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list