SQL with PHP, problem

DenKain · #1 (**permalink**) 10-13-06, 04:55 PM

I am trying to implement a secure login/session for my website following this guid: http://tech.hostforadollar.com/index...k=verart&aid=3

I have to write my own "add a user form" because the author asumes we all know how. Well I have created one that in throry should work but does not work in implementation. I have two parts to my "add a user" form. The first is an HTML page that uses the post method to get my information to a PHP page that inserts the info into the database. When I try this I get an error on the PHP page: Parse error: parse error, unexpected $ in /home/content/s/a/t/user/html/adduseraction.php on line 14

I can't seem to find the unexpected $.

Here is the HTML page:

Code:

<html> 
<head><title>Login</title></head> 
<body> 

<form action="adduseraction.php" method="Post"> 
Email Address:<br /> 
<input type="Text" name="psEmail" /> 
<br /> 
Password:<br /> 
<input type="password" name="psPassword" /> 
<br /> 
<input type="submit" value="Add" /> 

</form> 

</body> 
</html>

Here is the PHP processing page (keep in mind I took out the server address, username and password to show it to you):

Code:

<?PHP
// Check if the information has been filled in 
if($psEmail == '' || $psPassword == '') { 
// No login information 
echo "blank entrys"; 
} else { 
// Authenticate user 
$hDB = mysql_connect('server', 'username', 'password'); 
mysql_select_db('database'); 
// Insert a row of information into the table
mysql_query("INSERT INTO tblUsers 
(iUser, sEmail, sPassword, sGUID, sData) VALUES(Null, '$psEmail', password('$psPassword'), Null, Null) ") 
or die(mysql_error());
?>

I just can't figure out whats wrong.

mab · #2 (**permalink**) 10-13-06, 05:08 PM

There is no closing "}" to match the "} else {" opening one.

DenKain · #3 (**permalink**) 10-13-06, 05:17 PM

...wow...there are some days I feel like an idiot. Anyway that solved that problem but the processing PHP insists that the feilds are blank. Any ideas?

mab · #4 (**permalink**) 10-13-06, 05:29 PM

If we assume that register_globals are off (which is not a bad assumption as php.net recommends that new code should not be written using them and they will disappear in future versions of PHP), then you need to setup your variables in the processing page like this -

PHP Code:


			
$psEmail = $_POST['psEmail'];

$psPassword = $_POST['psPassword'];

DenKain · #5 (**permalink**) 10-13-06, 06:01 PM

Thank you so much for helping me. I have a lot of tweaking to do with this (Logouts, time expiration, security levels, etc) and I have a feeling I may be asking for help again some time.

End User · #6 (**permalink**) 10-14-06, 09:51 AM

Never mind.............