What's Wrong With This Code?

[caseygollan]

<?php
if (isset($_POST['email'])){
if ($dbemail == $email) {
echo 'email is good'
}
if ($dbpassword == $password) {
echo 'password is good'
}} ?>

what i want it to do is
if 'email' is set in the post array
and if the database email is equal to the form email echo "email is good"
if the database password is equal to the form password echo "password is good"

but this code makes the whole page blank out!

[edshuck]

I have no comment on the first part of the code. Just do not know.

I would end the echo statement with a ;

and I would, during testing, put in some else statements to give the code someplace to go if things are not right.

[cranial-bore]

Where is the script getting $email and $password from ?
Unless you have register gloabals on (which you shouldn't) it won't get those variables automatically from the form.

You need to compare with $_POST['email'] and $_POST['password'] or copy the $_POST variables to normal variables if you prefer.

I assume you already have $dbemail and $dbpassword.

You should also be securing those variables, by adding slashes (if magic quotes is off) and removing invalid characters.

[pitcher17]

minor thing but in your code you are missing the semi-colons at the end of the two echo lines.

--just noticed that edshucks said the same thing --
never mind..

[spikeZ]

I assume you have already set $dbemail, $dbpassword as the row from your database, and set $email and $password as your posted variables?!

Also just thinking about it, do you have your error reporting on?
This code (as mentioned above) should have thrown up at leaset two parse errors.

[Daimaju]

i think $email and $password should be $_POST['email'] and $_POST['password']