mysql_num_rows

seby

i cannot figure out why this returns this error:
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource

even if i manually set a value for news_id.
am i overlooking something?

$results = mysql_query("SELECT * FROM comments WHERE news_id ='".$mcontent[0]."'"); 
             $total = mysql_num_rows($results);

Kinsbane

Originally posted by seby
i cannot figure out why this returns this error:
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource

even if i manually set a value for news_id.
am i overlooking something?
$results = mysql_query("SELECT * FROM comments WHERE news_id='".$mcontent[0]."'"); 
             $total = mysql_num_rows($results);
[/B]

Try the above code.

Also try mysql_db_query.

drag0n

That error means your query didnt produce any results..

seby

well i found my problem, or part of it.. It seems whenever i include this code with other includes that have other queries it freaks out.. Anyone see anything wrong with this small function? The problem is somewhere in here becuase it doesn't happen when i remove this function..

function template($templatesname) 
{
	global $templatescache;
	$c=mysql_connect('host','user','pass') or die ( "Could not connet to MySQL" );
	   mysql_select_db(seby_content) or die ( "Could not connect to database" );

// check if template has already been loaded 
if ($templatescache[$templatesname]!="") { 
// return cached version 
$templates = $templatescache[$templatesname]; 
} else {

$query = mysql_query("SELECT template FROM site_template WHERE title='$templatesname'");
if (mysql_num_rows($query)!=0) {
$templates = mysql_result($query,"template");
#replace \" with \\" ... this will be needed later in the script 
$templates = str_replace("\"","\\\"",$templates);
// cache template
$templatescache[$name] = $templates;

} else { 
	$templates=""; 
	}
} 

return $templates; 
mysql_close($c);
}

sarahk

try:

<?php
function template($templatesname)  

{ 
  global $templatescache; 
  $c=mysql_connect('host','user','pass') or die ( 'Could not connet to MySQL' ); 
  mysql_select_db(seby_content) or die ( 'Could not connect to database' ); 

  // check if template has already been loaded 
  $templates = $templatescache[$templatesname];
  if ($templatescache[$templatesname] == '')   

  {
    $query = mysql_query("SELECT `template` FROM `site_template` WHERE `title`='{$templatesname}'"); 
    if ($row = mysql_fetch_array($query))
    { 
      $templates = $row['template']; 
      //replace \" with \" ... this will be needed later in the script 
      $templates = str_replace('\"','\\"',$templates); 
      // cache template 
      $templatescache[$name] = $templates; 
    } 
    else $templates=''; 
  }  

  return $templates; 
  mysql_close($c); 
}//function template($templatesname)  

?>

I've mucked around abit with the way you process the query.