[wp-trac] Re: [WordPress Trac] #4788: wp_loginout() / ampersand in URL

WordPress Trac wp-trac at lists.automattic.com
Tue Aug 21 20:44:19 GMT 2007


#4788: wp_loginout() / ampersand in URL
-----------------------+----------------------------------------------------
 Reporter:  raptorNL   |        Owner:  anonymous   
     Type:  defect     |       Status:  new         
 Priority:  low        |    Milestone:  2.4 (future)
Component:  General    |      Version:  2.2.2       
 Severity:  minor      |   Resolution:              
 Keywords:  has-patch  |  
-----------------------+----------------------------------------------------
Comment (by markjaquith):

 tmountjr, {{{&}}} should only be converted to {{{&}}} in XHTML
 contexts.  Redirects cannot accept {{{&}}} and neither can
 querystring-accepting WP functions.

 So, this needs to be fixed:

 {{{
 _e('<a href="foo.php?test=blah&foo=bar">foo</a>');
 }}}

 But not this:

 {{{
 wp_redirect('http://foo.com/?test=foo&bar=foo');
 }}}

 And not this:

 {{{
 query_posts('foo=bar&bar=foo');
 }}}

-- 
Ticket URL: <http://trac.wordpress.org/ticket/4788#comment:3>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list