Warning: mysql_num_rows(): supplied argument...

[Mutley]

I've fixed this error sooo many times, it must be the early morning today!

 

Anybody see what is causing this? The SQL when printed runs fine in phpmyadmin.

 

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource

 

$sql="SELECT `tid`, `title`, `start_date`, `posts` FROM `ipb_tablestopics` WHERE `forum_id` = '$set_newsarea' ORDER BY `tid` DESC LIMIT $set_newslimit";
$res=mysql_query($sql);
print $sql;
if(mysql_num_rows($res)!=0) {
while(list($tid, $title, $start_date, $posts) = mysql_fetch_row($res)){


print $title.'<br />';
print $start_date.'<br />';
print $posts.'<br />';

$resBody=mysql_query("SELECT `post` FROM `ipb_tableposts` WHERE `topic_id` = '$tid' AND `new_topic` = '1' LIMIT 1");
while($topic=mysql_fetch_array($resBody)){
print $topic["post"];
}
print '<hr />';


}
}else{
echo $e_norows;
}

 

Many thanks.

[shadiadiph]

can't  see anything must be something wrong in the

 

$sql="SELECT `tid`, `title`, `start_date`, `posts` FROM `ipb_tablestopics` WHERE `forum_id` = '$set_newsarea' ORDER BY `tid` DESC LIMIT $set_newslimit";

 

but it does appear you are asking it do do something with num_rows if it is empty maybe its not empty

 

maybe this !=0 should be !==0 ??

[Mutley]

Didn't work either. I've used the same method a dozen times. 

[shadiadiph]

have you tried to print_r $res; or  print $set_newsarea jus to what they are returning

[Mutley]

Doesn't print anything.

 

But the query is correct, it does work.

[shadiadiph]

 

$sql="SELECT `tid`, `title`, `start_date`, `posts`, `forum_id` FROM `ipb_tablestopics` WHERE `forum_id` = '$set_newsarea' ORDER BY `tid` DESC LIMIT $set_newslimit";

 

`forum_id` is missing

[Mutley]

Still hasn't fixed it, I'm pretty sure you don't need to select fields for the WHERE statement to work.

 

Thanks for the help so far.

[shadiadiph]

maybe try it without DESC LIMIT $set_newslimit

 

i reckon you must have a typo on fields or variables somewhere then in that statement

[Mutley]

That didn't work either.

 

As I've said the variables in the SQL work fine, it prints the correct SQL that works in phpmyadmin - and does bring up results.

 

So you are probably right it's a variable problem but I've tried just doing `tid` and it didn't work either.  ???

[shadiadiph]

i still don't get it you seem to be asking that if the value is empty then print the value which would be empty ??

[Mutley]

The value isn't empty (or shouldn't be!)

 

If the query returns 0 rows, it prints an error message else, it displays the rows.

 

 

[Potatis]

Instead of your "if" why not use while?

 

while (mysql_num_rows($res)!=0) {

 

//do things

 

}

[Mutley]

Same error...

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource

[Potatis]

Do any of the column names in your $sql line have the wrong spelling? Also, they are case sensitive, are they named the same in the database?

[Mutley]

Yes they are correct, as I've said it executes fine in phpmyadmin.

[PFMaBiSmAd]

Why not just get php/mysql to tell you why the query failed? -

$res=mysql_query($sql) or die('Query failed: ' . mysql_error());

[Potatis]

Yes something is not working with the query.  ???

[Ken2k7]

Instead of your "if" why not use while?

 

while (mysql_num_rows($res)!=0) {

 

//do things

 

}

Not to put a stab at you helping others, but I'm sure that would cause an infinite loop.

[Potatis]

Good. That's why I asked the question, why not? Now I know why.