Jump Script Parse Error

[gawnd]

I am going crazy trying to get a simple jump script setup for affiliate links.

I keep getting the following error.
PHP Parse error: parse error, unexpected T_STRING in Line 2

on the following script:

<?PHP
if ($o == "l1") {$link = "http://www.example.com";} // Default link
if ($o == "l2") {$link = "http://www.anotherexample.com";}
header("Location: $link"); // Jump to the hiddden URL above
exit();
?>

Anyone have any idea what could be wrong here?

[Bender]

-where is $o defined?

<?
switch($o)
{
case "l2":
$redirect="http://www.link2.com";
break;
case "l1":
$redirect="http://www.link1.com";
break;
}
header("Location: ".$redirect);
?>

[gawnd]

I have my links built like this:

http://www.example.com/jumpscript.php?o=l1
http://www.example.com/jumpscript.php?o=l2

I tried the above script and got the same results. Even with a simple header redirect without variables, I'm getting parse errors. Could there be a server setting, htaccess or php.ini that I'm missing?

[Johnn]

Missing ";" at the end of line 2 and 3?

[Bender]

<?
switch($_GET['o'])
{
case "l2":
$redirect="http://www.link2.com";
break;
case "l1":
$redirect="http://www.link1.com";
break;
}
header("Location: ".$redirect);
?>

[gawnd]

bender - that worked out. thanks so much.
i had a combination of issues - one, was editing in wordpad for some reason and it was throwing in odd formatting.

once i cleared that up, it was clear that the variable wasn't being defined and your new script solved that.

thanks again.

[Bender]

glad to help whenever I can.

[Johnn]

Good job Bender.

50 $F donated to your account.

[gawnd]

if you look online the first script i posted seems to be the norm. any specific reason the variable o wasn't being defined in my version?

i'm still a copy and paste php hack, but slowly learning.

[Bender]

the $o variable wasn't defined - you expected to be automatically extracted from the $_GET variables, but this is no longer the case in newer PHP versions(security reasons)
you could use an extract($_GET) but this NOT recommended

[JRBHosting]

You could also do something like...

function pop_vals($associative_array) {
 foreach ($associative_array as $key=>$val) {
  eval('$'.$key.' = "'.$val.'";");
 }
}

pop_vals($_REQUEST);

// $o should now be defined

I do NOT recommend this, though, as someone could run...

YOURURL?o=|1&SOMEREMOTECOMMANDSHERE

Jason