probs passing variables with get

gw360

Hi,

I have been lurking here for a while and have picked up enough to PHP-enable my site - thanks!

But I have run into a problem I can't solve, and I hope to get some help. I have visitors arriving at a page with a URL like this:

http://mysite.com/zipcheck.php?m=101

On that page, I have a form that queries the user for their ZIP code, then I use GET to call a script that looks like this:

<?php
if ($zip > 99499){$h = '1614';}
else if ($zip > 98799){$h = '2350';}
else if ($zip > 98599){$h = '2019';}
else if ($zip > 97999){$h = '2350';}
else if ($zip > 96999){$h = '2019';}
//snip//
else if ($zip > 199){$h = '504';}
header ("Location: http://mysite.com/pricing.php?m=$m&h=$h")
?>

The problem is, when the next page, pricing.php is loaded, only the $h variable appears correctly in the URL. The value of $m that the user arrived with is lost. So what I am getting is:

http://mysite.com/pricing.php?m=&h=2019

Instead of what I need:

http://mysite.com/pricing.php?m=101&h=2019

I'm stumped. Any help would be appreciated.

thanks,
Gino

TheDefender

Sounds as though your register globals is off in your php.ini file. Instead of calling $zip, call it using $_GET['zip'] instead. This page in the manual describes the use of predefined variables in the superglobal arrays. It also talks about register globals.

gw360

Thanks, Defender.

Unfortunately, that went right over my head. Could you give me an example of how I should change my code? I checked the manual page you referred me to, but I don't really understand the material.

thanks,
Gino

gw360

Ok. I'm not trying to be dense, but how do I define $m?

thanks, I really appreciate the help 🙂

TheDefender

OK, you hve this page:
http://mysite.com/zipcheck.php?m=101
Then on the page you want to use that m variable to pass it to the header right?

Well somewhere in your script you need to define it from the url string to pass to the header.
Add this line to your code before calling the header().

$m = $_GET['m'];

The $m variable won't be available globally if the register globals is off in your php.ini. Instead you have to call it from the superglobal $_GET array.

gw360

Ok - thanks.

I've added the $m = $_GET['m']; statement to my script.

The same problem still persists...

TheDefender

<?php 
if ($zip > 99499){$h = '1614';} 
else if ($zip > 98799){$h = '2350';} 
else if ($zip > 98599){$h = '2019';} 
else if ($zip > 97999){$h = '2350';} 
else if ($zip > 96999){$h = '2019';} 
//snip// 
else if ($zip > 199){$h = '504';} 
$m = $_GET['m']; //You have already added this
echo $m;  //Add this to make sure it's defined right

//header ("Location: http://mysite.com/pricing.php?m=$m&h=$h") 
?>

Comment out the header line for now to keep from getting the "headers already sent" error for now.

gw360

Ok, I made that change. I had to change my form method to "post" to test this. But the output page is not echoing $m

I will PM you with the URL...