Code is being weird

V0oD0o

Hi there, i have this function:

	//Function to check username/password
	function checkUser($username, $password, $color, $room){

		$result = mysql_query("SELECT password FROM girls WHERE username = '". $username ."'");
		$array = mysql_fetch_array($result);

		if ($array["password"] == $password){

			$username = "<font color=". $color ."><b>". $username ."</b></font>";

			session_register("room");

			session_register("username");

			mysql_query ("UPDATE girls SET lastAccess = NOW()") or DIE ("Unable to update NOW");

			//return $username;
			return true;

		}else{

			return false;

		}
}

But it seems to be completey bypassing this line:

$username = "<font color=". $color ."><b>". $username ."</b></font>";

I know the rest of the function is OK as it registers both sessions username and room.

Anyone have any idea why?

Cheers,

Mike

piersk

Try:

return $username;

instead of

return true;

. If you return true, you are basically emptying the variable $username.

V0oD0o

Thank piersk

Although I'm not fussed about the variable $username outside of the function, it's the session that I am conerned about.

The odd thing is, is that it puts the $username in to the session, but not the username with the HTML around it.

tomhath

I believe you can only register global variables, it might be having a problem trying to register a parameter to the function.

Atom

VooDoo,

If you have a recent version of php running try using the $_SESSION array. Something like this should work for you:

$_SESSION["username"] = "<font color=\"$color\"><b>$username</b></font>";

If you use this then you must access it the same way (i.e.: $temp = $SESSION["username"]) and session_register is no longer needed. Also variables do not have to be global to session them, especially if you use the predefined $SESSION array.

Atom

V0oD0o

I think I've found the problem.

For some reason, session_register("var") doesn't work inside a funtion, but $_SESSION["var"] = $var; does.

Thanks for your help!