[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