Re: Syntax checker? Character replacing

[haliphax <haliphax@xxxxxxxxx>]

On Fri, Mar 6, 2009 at 2:05 PM, Terion Miller <webdev.terion@xxxxxxxxx> wrote:
> I have this and think maybe something is off, because if there is an amp (&)
> in the location then it only displays a comma , and nothing else:
>
> if (isset($_SERVER['QUERY_STRING'])) {$Page .= ($_SERVER['QUERY_STRING']?
> '?'. str_replace("&","&amp;",$_SERVER['QUERY_STRING']) : '');}
>
>
> is that wrong?

It looks alright to me, but I don't see the need for the in-line
conditional statement since you've already run isset() on
$_SERVER['QUERY_STRING']. Simplify your code:

if(isset($_SERVER['QUERY_STRING']))
	$Page .= str_replace('&', '&amp;', $_SERVER['QUERY_STRING']);

Does that do the trick?


-- 
// Todd

-- 
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php