Parse error: syntax error, unexpected $end

Orangish

<?php  header("Cache-Control:no-cache");

if( ! $auth == "ok" )
 { header("Location:index.php ); exit (); }
?>
<html><head><title>Login</title></head><body>
Congratulations. You are logged in. 
</body></html>

What is wrong with it and why is it giving me an unexpected end thing.

laserlight

Because you did not close your string.

<?php
header("Cache-Control: no-cache");
if( ! $auth == "ok" ) {
	header("Location: index.php );
	exit ();
}
?>
<html><head><title>Login</title></head><body>
Congratulations. You are logged in.
</body></html>

Also, you should note that an absolute URI should be used, not just "index.php".

Orangish

It's still not working. I did what you did (infact i copied and pasted it)

laserlight

Also, your conditional probably is wrong.

you might use:
$auth != "ok"
instead

Incidentally, is the code snippet that you gave the whole code?

Installer

He didn't correct it for you; he only told you where your mistake is so that you can correct it yourself. Where might the string you didn't close be? Why is the syntax highlighting for "exit()" wrong? These are clues for you to use.

Orangish

That was the whole code on that page. On Index.php it is

<?php if( ($user != Me) and ($pass != test) )
	{
	setcookie("auth","ok");
	header("Location:logged.php" );  exit();
	}
?>
<html>
<head>
<title>Login</title>
</head>
<body>

<html>
<head>
<title>Login</title>
</head><body>
<form action"<?php echo($PHP_SELF); ?>" method="post">
Name <br/><input type="test" name="user"<br/><br/>
Password<br/><input type="text" name="pass"<br/><br/>
<input type="submit" value="Login">
</form></body></html>

laserlight

Let's consider a part of line 1:

($user != Me) and ($pass != test)

Read this forum's FAQ on register_globals
You should check for $POST['user'] and $POST['pass'] using [man]isset[/man] or [man]empty[/man]
After that, you should compare to the string literals '"Me"' and '"test"', not the undefined constants 'Me' and 'test'.
You should also use '&&' instead of 'and' for the comparison operator.

Later, you should use $SERVER['PHP_SELF'] instead of $PHP_SELF, e.g.
<form action="<?php echo $SERVER['PHP_SELF']; ?>" method="post">

TheDefender

One other thing in your conditional...

Unless you are dealing with TRUE and FALSE, NULL, or integers, your values should be quoted.

Also, you need the close parenthesis on the conditional.

if( ($_POST['user'] != "Me") && ($_POST['pass'] != "test") )