[Moonbat]

To start, here is my code

<?php
session_start();
function CheckCredentials($username, $password){
	$query = mysql_query("SELECT * FROM blog_users WHERE rank = 'Administrator' LIMIT 1");
	while ($result = mysql_fetch_array($query)) {
		if ($username != $result['username'] || $password != $result['password']) {
			echo "NOT LOGGED IN!";
		} else {
			echo "Hai";
		}
	}
}
CheckCredentials($_SESSION['username'], $_SESSION['password']);
?>

The error I get is:

Quote

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in (PATHTOMYSCRIPT) on line 5

I really don't understand why I'm getting the error. If it helps, here is the structure of blog_users:

id 	username 	password 	rank
----------------------------------------------------------
1 	Admin 	21232f297a57a5a74389 	Administrator

I also have another table in my blog_db but that's named blog_entries and is only for entries.

I'd appreciate it if someone could help me

[mocker]

The query looks fine, but it doesn't look like you connect to mysql and select the database before running it. mysql_query assumes you've already opened a database connection unless you specify a db handle.
Use mysql_connect(host, user,password), then mysql_select_db(database_name) before running mysql_connect.

As a rule of thumb you should always check to make sure your query went through, if it doesn't then $query will not be defined, so a simple

if(!($query)){
	   print "MySQL query failed. Reason: ".mysql_error();
}

This post has been edited by mocker: 22 August 2008 - 02:38 PM

[JBrace1990]

or you could just add or die(mysql_error()) to the end....

but yes, you never connect to a database...

Also, I want to suggest that you use smallint for rank, that way you can do things like: if($rank > 2){echo "Admin(3) or Mod(2)";}

[Moonbat]

Oh wow, I feel stupid. I do have a connection to a database, but I didn't include the file holding the connection before using the function. Thanks guys

[JBrace1990]

lol, don't worry... it happens to all of us sooner or later =P