[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