Problem!

[georgebates]

I have another problem, when i run this code i get this error "Parse error: syntax error, unexpected T_ELSE in /home/a1408362/public_html/admin.php on line 41". I can't see why in the code below i get this.

 

Heres the code "

<?php 


//checks cookies to make sure they are logged in 
if(isset($_COOKIE['ID_my_site'])) 
{ 
}
$username = $_COOKIE['ID_my_site']; 

if ($username <> "administrator");


{ header("Location: login.php"); 
}

else {}

?>

[Alex]

You shouldn't have a semicolon at the end of this line:

 

if ($username <> "administrator");

[georgebates]

Ok thanks, That worked but now i have same error in another file. Error "Parse error: syntax error, unexpected T_ELSE in /home/a1408362/public_html/members.php on line 48"

 

Code:

<?php

//checks cookies to make sure they are logged in
if(isset($_COOKIE['ID_my_site']))
{
}
$username = $_COOKIE['ID_my_site'];
$pass = $_COOKIE['Key_my_site'];
$check = mysql_query("SELECT * FROM users WHERE username = '$username'")or die(mysql_error());
while($info = mysql_fetch_array( $check ))
{
//if the cookie has the wrong password, they are taken to the login page
if ($pass != $info['password'])
{ header("Location: login.php");
}

//if user is administrator then go to admin view

if ($username == "administrator")
{
  header("Location: admin.php");
}

//otherwise show admin page

else
{
}



//if the cookie does not exist, they are taken to the login screen
else{   
header("Location: login.php");
}
}
?>

 

[Alex]

Is that the entire code? The error says line 48 but there's only 37 lines of code there.

[georgebates]

i didn't upload the part of the code connecting to my database.

 

I think the problem is in this part:

//otherwise show admin page

else
{
}

 

 

 

[Alex]

You should really consider intending your code. It makes it much easier to read and find potential errors.

 

<?php

//checks cookies to make sure they are logged in
if(isset($_COOKIE['ID_my_site']))
{

}
$username = $_COOKIE['ID_my_site'];
$pass = $_COOKIE['Key_my_site'];
$check = mysql_query("SELECT * FROM users WHERE username = '$username'")or die(mysql_error());
while($info = mysql_fetch_array( $check ))
{
//if the cookie has the wrong password, they are taken to the login page
if ($pass != $info['password'])
{ 
	header("Location: login.php");
}
//if user is administrator then go to admin view
if ($username == "administrator")
{
	header("Location: admin.php");
}
//otherwise show admin page
else
{

}
//if the cookie does not exist, they are taken to the login screen
else
{   
	header("Location: login.php");
}
}
?>

 

Looking at it that way the errors are extremely clear. It seems to me like you mismatched some if statement, I think you want something more like this:

 

<?php
//checks cookies to make sure they are logged in
if(isset($_COOKIE['ID_my_site']))
{
$username = $_COOKIE['ID_my_site'];
$pass = $_COOKIE['Key_my_site'];
$check = mysql_query("SELECT * FROM users WHERE username = '$username'")or die(mysql_error());
while($info = mysql_fetch_array( $check ))
{
	//if the cookie has the wrong password, they are taken to the login page
	if ($pass != $info['password'])
	{ 
		header("Location: login.php");
	}
	//if user is administrator then go to admin view
	if ($username == "administrator")
	{
		header("Location: admin.php");
	}
	//otherwise show admin page
	else
	{

	}
	//if the cookie does not exist, they are taken to the login screen
}
}
else
{   
header("Location: login.php");
}
?>