NYCPHP Meetup

NYPHP.org

[nycphp-talk] Code review

csnyder chsnyder at gmail.com
Mon Jun 6 10:17:35 EDT 2005


On 6/5/05, Paul Reinheimer <preinheimer at gmail.com> wrote:

> 1. In several places you echo out links to the user agent, but encode the
> ampersand, I don't think you need to do this, since you are using it as
> a seperator, not a charecter (or such is my understanding).

> href="/anp/auth_man.php?event=doQueryLogin&amp;loginAgent=[...]"

Using &amp; as a separator in output URLs is required if the output is
XHTML, because an ampersand by itself is not an allowed entity. So the
above snippet is fine.

-- 
Chris Snyder
http://chxo.com/



More information about the talk mailing list