eRott Posted January 6, 2008 Share Posted January 6, 2008 It's the strangest error. When ever I use the code: $query = "SELECT $fld_email FROM $tbl_name WHERE $fld_email = '$email'"; $result = mysql_query($query, $conn); return (mysql_num_rows($result) > 0); I always get a syntax error: Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /home/~/public_html/~/~/FILE.php on line 100 However, if i choose to substitute the variables for their direct values, it seems to work: $query = "SELECT email FROM clients WHERE email = '$email'"; $result = mysql_query($query, $conn); return (mysql_num_rows($result) > 0); The variable's look like: $tbl_name = "clients"; $fld_email = "email"; I've tried single quoting the variable names (in the sql query), as well as changing the names of the variables. It's not really a problem, however, I am curious as to why. So, any ideas? Quote Link to comment https://forums.phpfreaks.com/topic/84726-solved-small-sql-syntax-problem/ Share on other sites More sharing options...
Barand Posted January 6, 2008 Share Posted January 6, 2008 First step is always something like $result = mysql_query($query, $conn) or die (mysql_error() . "<pre>$query</pre>"); Quote Link to comment https://forums.phpfreaks.com/topic/84726-solved-small-sql-syntax-problem/#findComment-431775 Share on other sites More sharing options...
GingerRobot Posted January 6, 2008 Share Posted January 6, 2008 No doubt you'll find one or both of the variables are undefined. Quote Link to comment https://forums.phpfreaks.com/topic/84726-solved-small-sql-syntax-problem/#findComment-431778 Share on other sites More sharing options...
eRott Posted January 6, 2008 Author Share Posted January 6, 2008 So it would appear. Thanks for your help guys . I simply forgot to declare them as global variables along with my connection variable. Woops . global $conn, $tbl_name, $fld_email; Thank you once again. Quote Link to comment https://forums.phpfreaks.com/topic/84726-solved-small-sql-syntax-problem/#findComment-431782 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.