[RESOLVED] mysql_fetch_array() argument is not a valid MySQL result resource

TacticalPenguin

I'm getting this error. It's part of a login script I'm working on. Here's where the error is. The error is line 36, which is commented.

if (isset($_POST["username"]) && isset($_POST["password"]))
{

$temp_pw = trim($_POST["password"]);
$temp_user = trim($_POST["username"]);

echo $temp_user;
echo $temp_pw;
$sql_string = 'SELECT * FROM '.$sql_db_prefix.'users WHERE username = '.$temp_user; //line 36
$result = mysql_query($sql_string);

$temp_sql_row = mysql_fetch_array($result);

if ($temp_pw == $temp_sql_row["password"])
{
	$_SESSION["loggedin"] = true;
	$_SESSION["username"] = $temp_username;

}
}

Also, if there are any better ways to do things in there, please tell me, I know a lot of app/game programming but I just recently started php.

leatherback

The error tels you that the query you present to the databae is not valid, and therefor you do not get a valid output of the query.
Solution in this case: You need to add quotes around the username.

In general:
- during debudding add or die(mysql_error()) at the and of the mysql_query statement
- Echo out the query you sent to the database

TacticalPenguin

Ok, thanks, I'll try that and see if I can get the problem solved.

leatherback

$sql_string = "SELECT * FROM ".$sql_db_prefix."users WHERE username = \"".$temp_user."\";

TacticalPenguin

Yay! It works! Thanks for the pointers!

leatherback

Happy to know you're happy

Please markt the thread resolved (See thread tools)

TacticalPenguin

Ok, will do.