[RESOLVED] Getting data from one page to another

[Muppet9010]

Hi i am trying to get inputted responses from one page to another.

Input Page is shown below. This records and verifys the data fine:

if(isset($_POST['date'])) {	
	if($_POST['date'] == "confirmed") {
		//set medalhistrorydate_month & medalhistrorydate_year
		if(isset($_POST['medalhistorydate_month']))
			$medalhistorydatemonth = $_POST['medalhistorydate_month'];
		else
			$medalhistorydatemonth = "0";

	if(isset($_POST['medalhistorydate_year'])) {
		$medalhistorydateyear = $_POST['medalhistorydate_year'];
		if($medalhistorydateyear < 1900 || $medalhistorydateyear > 2100)
			$medalhistorydateyear = "0000";
	}
	else
		$medalhistorydateyear = "0000";

	//set medalhistrorydate_continue to the end confirmation screen response
	if($medalhistorydatemonth != "0") {
		if($medalhistorydateyear != "0000") {				
			MuppetRedirectResult(GetAndReplaceLangStr($content['LN_CALCMEDALHISTORYACTION'],$medalhistorydatemonth,$medalhistorydateyear), "parser.php?year=" . $medalhistorydateyear . "&month=" . $medalhistorydatemonth . "&op=calcmedalhistoryonly", 3);
		else
			MuppetRedirectResult(GetAndReplaceLangStr($content['LN_CALCMEDALHISTORYYEARERROR'],$medalhistorydateyear), "calcmedalhistorydate.php", 3);
	}
	else	
		MuppetRedirectResult(GetAndReplaceLangStr($content['LN_CALCMEDALHISTORYMONTHERROR'],$medalhistorydatemonth), "calcmedalhistorydate.php", 3);
}
}

The function MuppetRedirectResult displays a message in a temproary webpage and then moves the webpage onto the specified one in the 2nd function input, the third is a delay timer.

It is the part where it sends it to the parser.php file that it is having issues. The address in the address bar of the parser.php is correct (parser.php?op=calcmedalhistoryonly&month=5&year=2007). But the below code in parser.php can only pickup the 'op' varibale and not the year or month (i have tried other words incase these where locked in php as special):

if ( isset($_GET['op']) )
	$parseroperation = DB_RemoveBadChars($_GET['op']);
else
	$parseroperation = "";

if ( isset($_GET['op']) )
{
if($parseroperation == 'calcmedalhistoryonly') {
			//Added by Muppet - create the medal history bits
			if(isset($_GET['month']))
				$medalhistorydatemonth = $_GET['month'];
			else
				$medalhistorydatemonth = "0";
			if(isset($_GET['year']))
				$medalhistorydateyear = $_GET['year'];
			else
				$medalhistorydateyear = "0000";
			if($medalhistorydatemonth != "0" && $medalhistorydateyear != "0000")
				CreateAllMedalHistory(-1, $medalhistorydateyear, $medalhistorydatemonth);
			else if($medalhistorydatemonth == "0")
				DieWithErrorMsg("Error, bad month: " . $medalhistorydateyear . " : " . $medalhistorydatemonth);
			else
				DieWithErrorMsg("Error, bad year: " . $medalhistorydateyear . " : " . $medalhistorydatemonth);
		}
}

The above code reconises the 'op' but fails on the other two and so displays in the built in error message a year of "0000" and a month of "0" (my failback values).

Anyone have any idea, I am trying to add this into existing code.

[devinemke]

your URL string is:

parser.php?op=calcmedalhistoryonly&month=5&year=2007

but in your code you are referencing $GET['medalhistorydate_month'] and $GET['medalhistorydate_year'] instead of $GET['month'] and $GET['year']

[Muppet9010]

sorry thats a typo in what i copied into here. have spent so long fiderling with bits of this and trying global variables incase that would work (small C++ experience so this concept of pages is very wierd for me).
corrected the above

[SyWill]

hey hows it goin muppet,

ok so i know your already gung ho for get variables apparently but just a thought might want to switch over to session type setup MUCH easier to maintain and to control data flow back and forth. excellent resource is http://www.tizag.com/phpT/phpsessions.php basically you add session_start(); to top of any php page you need to pass the variables to and then set the variables by using $SESSION['CreativeName'] = "Creative Data;
then shift them off to next page and preform your checks using $SESSION['CreativeName'], anyway after all that long windedness your code here is 100% correct only thing i can think of is maybe check your DB_RemoveBadChars function call to see if its changing the values somewhere because isset will still trigger true because the variable is sent but the second part will fail if its altered in that db_remove call just a thought.

[Muppet9010]

Thanks SyWill, sessions have done the trick 😃

Incase its of any use to anyone else:

session_start();

if(isset($_POST['date'])) {	
	if($_POST['date'] == "confirmed") {
		//set medalhistrorydate_month & medalhistrorydate_year
		if(isset($_POST['medalhistorydate_month']))
			$_SESSION['medalhistorydatemonth'] = $_POST['medalhistorydate_month'];
		else
			$_SESSION['medalhistorydatemonth'] = "0";

	if(isset($_POST['medalhistorydate_year'])) {
		$_SESSION['medalhistorydateyear'] = $_POST['medalhistorydate_year'];
		if($_SESSION['medalhistorydateyear'] < 1900 || $_SESSION['medalhistorydateyear'] > 2100)
			$_SESSION['medalhistorydateyear'] = "0000";
	}
	else
		$_SESSION['medalhistorydateyear'] = "0000";

	//set medalhistrorydate_continue to the end confirmation screen response
	if($_SESSION['medalhistorydatemonth'] != "0") {
		if($_SESSION['medalhistorydateyear'] != "0000")			
			MuppetRedirectResult(GetAndReplaceLangStr($content['LN_CALCMEDALHISTORYACTION'],$_SESSION['medalhistorydatemonth'],$_SESSION['medalhistorydateyear']), "parser.php?op=calcmedalhistoryonly", 3);
		else
			MuppetRedirectResult(GetAndReplaceLangStr($content['LN_CALCMEDALHISTORYYEARERROR'],$_SESSION['medalhistorydateyear']), "calcmedalhistorydate.php", 3);
	}
	else	
		MuppetRedirectResult(GetAndReplaceLangStr($content['LN_CALCMEDALHISTORYMONTHERROR'],$_SESSION['medalhistorydatemonth']), "calcmedalhistorydate.php", 3);
}
}

session_start();

if ( isset($_GET['op']) )
    $parseroperation = DB_RemoveBadChars($_GET['op']);
else
    $parseroperation = "";

if ( isset($_GET['op']) )
{ 
if($parseroperation == 'calcmedalhistoryonly') {
			//Added by Muppet - create the medal history bits
			if(!isset($_SESSION['medalhistorydatemonth']) || $_SESSION['medalhistorydatemonth'] == "0")
				$dateerror = "month";
			if(!isset($_SESSION['medalhistorydateyear']) || $_SESSION['medalhistorydateyear'] == "0000")
				$dateerror = "year";
			if(!isset($dateerror))
				CreateAllMedalHistory(-1, $_SESSION['medalhistorydateyear'], $_SESSION['medalhistorydatemonth']);
			else if($dateerror == "month")
				DieWithErrorMsg("Error, bad month: " . $_SESSION['medalhistorydateyear'] . " : " . $_SESSION['medalhistorydatemonth']);
			else
				DieWithErrorMsg("Error, bad year: " . $_SESSION['medalhistorydateyear'] . " : " . $_SESSION['medalhistorydatemonth']);
			unset($_SESSION['medalhistorydateyear']); 
			unset($_SESSION['medalhistorydatemonth']);
		}
}