[RESOLVED] Sessions Won't stay registered

[lmayer4]

Afternoon,

I've used sessions before with good results but can't seem to make this work.

I have a form with 2 fields : DIVISIONCD and Type. On the form at the top of the page I have:

// start the session
session_start();
header("Cache-control: private");

On the second page I have again at the top of the page:

// start the session
session_start();
header("Cache-control: private"); //IE 6 Fix

// Get the user's input from the form
$DIVISIONCD = $_POST['DIVISIONCD'];
$Type = $_POST['Type'];
// Create a new Session Value 
session_register('DIVISIONCD');
session_register('Type');

// Register the input with the value
$_SESSION['DIVISIONCD'] = $DIVISIONCD;
$_SESSION['Type'] = $Type;

the first query runs fine :

.....
from dbo.vwTFW_Statistics
where DIVISIONCD = '".urldecode($_SESSION['DIVISIONCD'])."'
and TYPE = '".urldecode($_SESSION['Type'])."'";

The second query which also uses the

.....
from dbo.vwTFW_Statistics
where DIVISIONCD = '".urldecode($_SESSION['DIVISIONCD'])."'
and TYPE = '".urldecode($_SESSION['Type'])."'";

drops the session variable. No where on the page do I destroy it or unregister it. There must be a reason I just can't seem to find it anywhere.

Any thoughts or suggestions would be most welcome.

Thanks

Laura

[kburger]

I'd get rid of the session_register() lines. From the manual:

If you want your script to work regardless of register_globals, you need to instead use the $SESSION array as $SESSION entries are automatically registered. If your script uses session_register(), it will not work in environments where the PHP directive register_globals is disabled.

If you are using $_SESSION (or $HTTP_SESSION_VARS), do not use session_register(), session_is_registered(), and session_unregister().

[lmayer4]

I removed the "session_register" lines and I still get the same thing. Very strange.

Laura

[kburger]

I guess it's time to take a look at your entire code.

[toplay]

I agree that we really need to see more of your code. Like I don't understand if the second query you talk about is in the same script or another?

It could be lots of different things causing this (especially when you consider all the various php.ini session related settings too).

Put aside your script for a moment and try a small simple session test scripts to see if sessions work on your system:
set.php script:

<?PHP
session_start();
$_SESSION['test'] = 'Hello World!';
header('Location: read.php?' . SID);
exit;
?>

read.php script:

<?PHP
session_start();
echo isSet($_SESSION['test']) ? $_SESSION['test'] : 'Not Set';
?>

Put both scripts in the same web directory. Run set.php. It will then redirect to read.php. If it works, then you should see 'Hello World!' displayed. If the session data isn't being saved, you'll see the 'Not Set' message instead. If you see the session name and ID in the URL, then it means session cookies aren't being created (or allowed).

So, report back to us what message you saw after running this test and if the session name and ID was in the URL or not.

[lmayer4]

Thanks for that. I did run it and did get Hello World.

There are two queries making up a table. The first one gets the rows the second one makes the totals of those rows:

beginning of table page:

// start the session
session_start();
header("Cache-control: private"); //IE 6 Fix

echo "<strong>Step 2 - Register Session </strong><br />";

// Get the user's input from the form
$Division = $_POST['DIVISIONCD'];
$Type = $_POST['Type'];

// Register the input with the value
$_SESSION['DIVISIONCD'] = $DIVISIONCD;
$_SESSION['Type'] = $Type;

query 1:

/* connect to db */
include ("TFWConnect.php");

//load it all into the associative array
$sql = "select DIVISIONCD,MODEL,DESCRIP,DTargetQty,RTargetQty,OnOrder,Received,
Active,NRW,PreDisposal,Disposed, Returned,
(OnOrder+Received+Active+NRW+PreDisposal+Disposed+Returned) as GroupTotal
from dbo.vwTFW_Statistics
where DIVISIONCD = '".urldecode($_SESSION['DIVISIONCD'])."'
and TYPE = '".urldecode($_SESSION['Type'])."'";
$result = mssql_query($sql);

table data blah blah

second query:

$Totsql = "select 
           SUM(OnOrder)AS TotOnOrder, 
	   SUM(Received) as TotRecieved, 
	   SUM(Active) as TotActive, 
	   SUM(NRW)as TotNRW, 
	   SUM(PreDisposal)as TotPreDisposal, 
	   SUM(Disposed)as TotDisposed, 
	   SUM(Returned)as TotReturned ,			sum(OnOrder+Received+Active+NRW+PreDisposal+Disposed+Returned) as TotGroup
	from dbo.vwTFW_Statistics
	where DIVISIONCD = '".urldecode($_SESSION['DIVISIONCD'])."'
	and TYPE = '".urldecode($_SESSION['Type'])."'";
    $Totresult = mssql_query($Totsql);

This is where it trips up and drops the variable. If I make it REQUEST instead of SESSION it works. It just seems to forget it ever saw the session variables.

Thanks for taking a look

Laura

[kburger]

// Get the user's input from the form
$Division = $POST['DIVISIONCD'];
$Type = $POST['Type'];

// Register the input with the value
$SESSION['DIVISIONCD'] = $DIVISIONCD;
$SESSION['Type'] = $Type;

shouldn't it either be

// Get the user's input from the form
$DIVISIONCD = $_POST['DIVISIONCD'];
$Type = $_POST['Type'];

// Register the input with the value
$_SESSION['DIVISIONCD'] = $DIVISIONCD;
$_SESSION['Type'] = $Type;

or

// Get the user's input from the form
$Division = $_POST['DIVISIONCD'];
$Type = $_POST['Type'];

// Register the input with the value
$_SESSION['DIVISIONCD'] = $Division;
$_SESSION['Type'] = $Type;

[toplay]

You didn't say if you saw the session name and ID in the URL during the test. Did you?

Is the code last posted all in one script?

Start putting displays (echo/print) at strategic places within your code to help yourself debug this. Display the queries ($sql and $Totsql).

Are you checking for any possible MySQL errors after the queries? I don't see any (you chopped off the code right there).

FYI - You can select and do the sum() in one query. There's really no need for two!

[toplay]

Yes kburger, you're right.

In lmayer4's original (first post) he/she did use:
$DIVISIONCD = $_POST['DIVISIONCD'];

So, it's changed!

[lmayer4]

Yes I have been testing, trying to figure it out. I GOT IT!!!!

Thanks for all the help.

I changed some things. As kburger recommended I took out the session_register and then I changed the where clauses to

where DIVISIONCD = '".$Division."'
and TYPE = '".$Type."'";

instead of

where DIVISIONCD = '".urldecode($_SESSION['DIVISIONCD'])."'
and TYPE = '".urldecode($_SESSION['Type'])."'";

and it worked just how its supposed to. Thanks again for all the help.

Laura

[kburger]

Glad to hear. Please mark this thread resolved.