[RESOLVED] MySQL result resource troubles

[AndrewGurn]

Ahoy there.

I'm having some trouble with returning a MySQL result from a class.

The error:
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in ...blah blah blah

This goes three files deep, so I'll start at the top.

This is the script that is initally called:

//Load the Login class
require_once("phpClasses/Login.php");

//Instantiate a Login object
$Login = new Login;

//ERROR OCCURS HERE, so we step into that..
$Login->loginUser($_GET['username'],$_GET['pass']);

Both of those GETs are what I expect them to be.

Here is the loginUser() function in the Login object. DataConnection is an object that is loaded in the Login.php class file.

$DataConnection = new DataConnection;
$db = $DataConnection->openConnection();

//When I run this query manually, it returns the row that I want
$query = "Select password, id from userTable where uname = '".$username."'";

//This function just does a mysql_query and retuns the result
$result = $DataConnection->selectQuery($query);

//ERROR HAPPENS HERE, so we step in..
$numRows = $DataConnection->getNumberOfRows($result);

And here is the DataConnection class's getNumberOfRows() function:

//ERROR HAPPENS HERE:
	function getNumberOfRows($resultSet)
	{
		$numRows = mysql_num_rows($resultSet);
		return $numRows;
	}

I'm really lost on this.

Anyhelp wpuld be great. Thanks!

[NogDog]

Sounds like your query if failing. Try something like the following to see if you get any useful debug data:

$result = $DataConnection->selectQuery($query) or die("Query failed ($query): " . mysql_error());

[AndrewGurn]

Thanks for your reply, NogDog.. I occasionally get frustrated to the point where I forget how to debug.

I pulled out the loginUser() function from my Login class and put it into a seperate test script:

require_once("phpClasses/DataConnection.php");

function loginUser($username, $password)
{
		$DataConnection = new DataConnection;
		$db = $DataConnection->openConnection();
		$query = "Select password, id from userTable where uname = '".$username."'";
		$result = $DataConnection->selectQuery($query) or die("Query failed ($query): " . mysql_error());
		echo($result);
	/*	$numRows = $DataConnection->getNumberOfRows($result);
		if($numRows)
		{
			$i = 0;
			while ($i < $numRows) 
			{
				$i = $i+1;
				$rows = mysql_fetch_array($results);
				$dbPassword = $rows[0];			
				$id = $rows[1];
			}	
		}		
	*/
}

loginUser("Andrew","myPassword");

The query doesn't die, and the result of the echo is "Resource id #3".. which seems to imply that it's a result resource.

Here is $DataConnection->selectQuery() again for reference:

function selectQuery($query)
	{
		$results = mysql_query($query)  or die("Query failed : " . mysql_error());
		return $results;
	}

Is it ok that it is also has a die in this block to? I figure that it would throw the error up to the top step, but I could be wrong.

[AndrewGurn]

Forget it. Typo. 🙁 :xbones:

At least it wasn't anything wrong my logic 🙂