[RESOLVED] can you tell me whats wrong?

[filn]

ive been working on this all day and i cant figure out whats causing this 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 ') VALUES (75, 100, 500, 20, 'demo5', MD5(' at line 12

INSERT INTO plogger_config ( compression, max_thumbnail_size, max_display_size, thumb_num, admin_username, admin_password, date_format, feed_title, gallery_name, gallery_address, gallery_phone,) VALUES (75, 100, 500, 20, 'demo5', MD5('demo5'), 'n.j.Y', 'Plogger Photo Feed', 'NAME', 'ADDRESS', 'PHONE NUMBER')

here is the bit of code that inserts the data:

$query = "INSERT INTO `".$TABLE_PREFIX."config` 
			( `compression`,
			  `max_thumbnail_size`,
		  	`max_display_size`,
		  	`thumb_num`,
		  	`admin_username`,
		  	`admin_password`,
		  	`date_format`,
			`feed_title`,
		   `gallery_name`,
		   `gallery_address`,
		   `gallery_phone`,)
		   VALUES
		   (75,
			  100,
		    500,
		    20,
		    '".mysql_escape_string($_POST["username"])."',
		    MD5('".mysql_escape_string($_POST["password"])."'),
		    'n.j.Y',
		    'Plogger Photo Feed',
		    '".mysql_escape_string($_POST["gallery_name"])."',
			 '".mysql_escape_string($_POST["gallery_address"])."',
			 '".mysql_escape_string($_POST["gallery_phone"])."')";

[theelectricwiz]

You have an extra comma - gallery_phone,) VALUE -> gallery_phone) VALUE

[filn]

theelectricwiz wrote:

You have an extra comma - gallery_phone,) VALUE -> gallery_phone) VALUE

thank you thank you thank you!

[filn]

ok, i screwed it up again, same section, I added some new fields, I thought I did it right, but it doesnt look like it. Do I have too many or a missing , or ' or something?

$query = "INSERT INTO `".$TABLE_PREFIX."config` 
			( `compression`,
			`max_thumbnail_size`,
		   	`max_display_size`,
		   	`thumb_num`,
		   	`admin_username`,
		   	`admin_password`,
		   	`date_format`,
		   	`feed_title`,
		   	`gallery_name`,
		   	`gallery_address`,
		   	`gallery_phone`,
		   	`gallery_forums`,
		   	`gallery_reviews`)
		   	VALUES (75,100, 500,20,'".mysql_escape_string($_POST["username"])."', MD5('".mysql_escape_string($_POST["password"])."'),
		    'n.j.Y',
		    'Plogger Photo Feed',
		     '".mysql_escape_string($_POST["gallery_name"])."',
			 '".mysql_escape_string($_POST["gallery_address"])."',
			 '".mysql_escape_string($_POST["gallery_phone"])."',
			 '".mysql_escape_string($_POST["gallery_forums"])."',
			 '".mysql_escape_string($_POST["gallery_reviews"])."'";

thanks everybody for your help!

[theelectricwiz]

filn wrote:

'".mysql_escape_string($_POST["gallery_reviews"])."'"; 

I think it is just a closing ) for the values section of the query at the the end of this line that you have missed 🙂 :

'".mysql_escape_string($_POST["gallery_reviews"])."')";