MyFreeCounter — Fri, 27 Jan 2006 21:05:54 +0000

also along the same lines as Greg K. Change:

mysql_query($sql);

to:

mysql_query($sql) or die("MySQL Query Error: ".mysql_error());

if there's something wrong with your query, this will tell you. Also make sure your connection variables are correct (someone already suggested that), and make sure you have added the user to the database you are trying to connect to.

Matt Pegler
MyFreeCounter Owner/Operator
http://www.MyFreeCounter.net - free hit, realtime, and click counters, as well as statistics

Greg K — Wed, 25 Jan 2006 17:13:53 +0000

Even though it is still not working with what Busy suggested, keep the formatting that he used. Without the double quotes you will get the actual string '$name','$mail','$comment' instead of the values of the variables.

Also, your or DIE(" ") statement is not that informative. Imagine if you had about 10 of these in a script, you run it and the page is blank. Unless you happen to be outputing things to the browser in between then, you will not know where it died. It is good to get into the practice of using good error messages.

For this one, I would recommend something along the lines of die('Could not connect: ' . mysql_error());

Lastly if all else seems to be working correctly, echo out the sql statement right before executing it. If there is something misformatted in it, this can help you spot it. Sometimes with all the quotes, variable names, backslashes, etc, it is easy to miss the mistake in the assignemtn line.

Until you get it working, add echo "\nSQL: $sql \n"; right before the query line.

-Greg

timjpriebe — Wed, 25 Jan 2006 16:58:04 +0000

Does dogpics.inc have the correct values for $hostname,$username, $password and $dbname ?

Chrislt — Wed, 25 Jan 2006 13:19:44 +0000

That didnt seem to work. I'm not getting any errors, everything seems to be working but for some reason the database is not getting updated. Im using godaddy hosting if that helps. The script I used came from them.

Busy — Wed, 25 Jan 2006 09:26:36 +0000

change
$sql = 'INSERT INTO `questionaire` (`Date`,`Name`,`Email`,`Comment`)
VALUES (CURDATE(), \'$name\',\'$mail\',\'$comment\');';

to
$sql = "INSERT INTO questionaire (Date,Name,Email,Comment)
VALUES (CURDATE(), '$name','$mail','$comment')";

Also if you can tell us the error you are getting it would be helpful