php IF statement problem, probably a newbie problem

[HunteRRosE]

This is driving me crazy.

I have a mysql db I'm trying to use to verify whether an account is an admin account or not, based on a bool value of 0 (not admin) or 1 (admin).

For whatever reason, my if statement isn't working correctly. Hopefully someone can help.

Basically I cant get the admin's to go to the admin page and the members to the members page. The members can somehow get to the admin page. These are literally just test pages I've setup that tell you which page your on.

Here's part of my code:

PHP Code:

$admin = mysql_query('SELECT admin FROM users');

if ($admin == 0) {
//then redirect them to the members area
header("Location: members.php");
}
else
{
//then redirect them to the admin area
header("Location: admin.php");
} 


[proxomos]

PHP Code:

$admin = mysql_query('SELECT admin FROM users'); 

if ($admin == 0) { 
//then redirect them to the members area 
header("Location: members.php"); 
} 
else 
{ 
//then redirect them to the admin area 
header("Location: admin.php"); 
} 


I would do:

PHP Code:

$result = mysql_query("SELECT admin FROM users");
list($admin)=mysql_fetch_row($result);

switch ($admin){
case 0:
   header("Location: members.php");
   break;

case 1:
  header("Location: admin.php");
  break;
} 


Hope that helps

Proxomos

[itwasntme]

The code is going to have to specify which user to select; otherwise, you're going to get unpredictable results.

Proxomos code modified to do so would look like:

$result = mysql_query("SELECT admin FROM users WHERE id=$user_id");
list($admin)=mysql_fetch_row($result);

switch ($admin){
case 0:
   header("Location: members.php");
   break;

case 1:
  header("Location: admin.php");
  break;
}

That's if you have a unique (primary key) field in your DB called "id" and have already declared the variable $user_id according to the login procedure to correspond to the current user. You'll need to change that part to match your field name.