[SOLVED] Unexpected T_VARIABLE

[Akenatehm]

Hey Guys,

 

This script is apart of a login script. It shows up this error:

 

Parse error: syntax error, unexpected T_VARIABLE in /home/kaurlcom/public_html/relayadmin/scripts/login.php on line 8

 

Here's the code:

 

<?php
// we must never forget to start the session
session_start();
$errorMessage = '';
if (isset($_POST['username']) && isset($_POST['password'])) {
   include 'connect.php'

   $username = $_POST['username'];
   $password = $_POST['password'];

   // check if the user id and password combination exist in database
   $sql = "SELECT username
           FROM users
           WHERE username = '$username' 
                 AND password = password('$password')";

   $result = mysql_query($sql) 
             or die('Query failed. ' . mysql_error()); 

   if (mysql_num_rows($result) == 1) {
      // the user id and password match, 
      // set the session
      $_SESSION['db_is_logged_in'] = true;

      // after login we move to the main page
      header('Location: home.php');
      exit;
   } else {
      $errorMessage = 'Sorry, wrong Username or Password';
   }

}
?>

 

Help would be greatly appreciated!

[cooldude832]

include needs to be closed with a ;

 

more preferred method is to write

 

<?php
include('connect.php');
?>

[Akenatehm]

Ugh. Such an obvious one. Damn. Sorry and Thanks.

[cooldude832]

I should make a list of things but a general note of unexpected variable generally means something isn't closed so check the lines above for that one.

[Akenatehm]

Ok. Now the page just goes blank and it looks like the session doesnt get created at all.

[cooldude832]

Obviously because you have no output on the page you are only setting the variable for the error message

 

in your else portion near the end add

echo  $errorMessage;

 

 

and odds are you will see something

[Akenatehm]

It now giving the $errorMessage "Invalid Username or Password" but the details I am using are in the database.

[cooldude832]

add into the else statement

 

echo "<br />".$sql>"<br />";

[Akenatehm]

Ok here's the reply: It has a 1 on the end.

 

Sorry, wrong Username or Password1

[cooldude832]

something with the variable $sql is causing the issue

[Akenatehm]

Any idea what?

 

 

[Akenatehm]

Ok, I think i fixed it.

 

I changed to this:

 

   $sql = "SELECT username
           FROM `users`
           WHERE username = '$username' 
                 AND password = '$password'";

[cooldude832]

What ever works you have to read your logic to realize how its working and if you think the credentials are correct and the output says else then odds are your method to checking said credentials are flawed

[Akenatehm]

It's all working fine now.