clanstyles Posted June 28, 2007 Share Posted June 28, 2007 I can't figure out whats wrong witht his query. It looks fine to me but somebody else please...have a look mysql_query("INSERT INTO `serverInfo` ( `id` , `serverid` , `size` , `filename` , `timestamp` ) VALUES('null', '1', '".$size."', '".$dir."', '".$date."'") or die(mysql_error()); Anything stupidly wrong here? the id,serverid, size, filename,timestamp thsoe are all right. Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1 Quote Link to comment Share on other sites More sharing options...
pocobueno1388 Posted June 28, 2007 Share Posted June 28, 2007 Try it without concating the variables. mysql_query("INSERT INTO `serverInfo` ( `id` , `serverid` , `size` , `filename` , `timestamp` ) VALUES('null', '1', '$size', '$dir', '$date')") or die(mysql_error()); Try using mysql_real_escape_string() on all of those variables before insert as well. Quote Link to comment Share on other sites More sharing options...
clanstyles Posted June 28, 2007 Author Share Posted June 28, 2007 of course i did that lol but I will try that function Quote Link to comment Share on other sites More sharing options...
Wildbug Posted June 28, 2007 Share Posted June 28, 2007 You're missing the closing parenthesis for the VALUES(... portion. EDIT: I was wrong the first time. Quote Link to comment Share on other sites More sharing options...
Wildbug Posted June 28, 2007 Share Posted June 28, 2007 It's useful to assemble the query in a variable before passing it to mysql_query(). That way you can print it out along with the error in a die() function and see things that will be more obvious than in the often cluttered, complex string concatenations we use to build queries. Quote Link to comment 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.