[RESOLVED] MySQL syntax.

Heinous

I've read no documentation on either php or MySQL (I've picked up my knowledge from pre-made scripts and know most basics), and I'm trying to make a DJ timetable script.

I've got the form all done, but when I go to submit the data into the tables, it never works. I've tried multiple ways, but never works. Here's what I have currently:

			$comments = $_POST['comments'];
			$time = $_POST['time'];
			$date = $_POST['date'];
			$show = $_POST['name'];
			$username = $logged['username'];
			$sql = 'INSERT INTO `timetable` (`id`, `dj`, `time`, `date`, `show`, `comments`) VALUES (NULL, $logged[user], $time, $date, $show, $comments);';

Help is appreciated, so thanks in advance.

devinemke

$sql = "INSERT INTO timetable SET 
dj = '" . $logged['user'] . "',
time = '" . $time . "',
date = '" . $date . "',
show = '" . $show . "',
comments = '" . $comments . "'";

why are you storing the time and date in 2 separate fields? store in a single DATETIME field.

Houdini

I'm sure there tis more code before to connect (I hope) and to select the database, and more code below but this should work also if the 'id' is a number with an auto_increment it is not necessary to even include it ineither list, or if you actually want a value of NULL to go into an 'id' field why not set a default of NULL in the table or better yet why even have it?

$sql = "INSERT INTO `timetable` (`id`, `dj`, `time`, `date`, `show`, `comments`) VALUES (NULL, '$username', '$time', '$date', '$show', '$comments')";

You need to quote your variables and since you assigned the$_POST['username'] to $username and I saw no logged['user'] anywhere is suppose you had that variable as an error also.

Heinous

devinemke wrote:
$sql = "INSERT INTO timetable SET 
dj = '" . $logged['user'] . "',
time = '" . $time . "',
date = '" . $date . "',
show = '" . $show . "',
comments = '" . $comments . "'";
why are you storing the time and date in 2 separate fields? store in a single DATETIME field.

That code didn't work either. ;S And I'm doing two seperate fields because of the way I want to show the timetable.

@ Houdini -
Your code didn't work either, heh. ;S I'm testing locally, so I might upload it to my host in a minute and check to see if it's not just my PC..

Database connection and setting of the $logged['username'] variable are at the start of the script.

<?php
ob_start();
include("config.php");

devinemke

Heinous wrote:
That code didn't work either.

what does [man]mysql_error[/man] have to say about the query?

Heinous wrote:
And I'm doing two seperate fields because of the way I want to show the timetable.

you can format a DATETIME field any way you want using mySQL's Date and Time Functions

bradgrafelman

OR store it as a Unix timestamp (MySQL's UNIX_TIMESTAMP() function) and use PHP's extremely versatile [man]date/man function.

Heinous

Not locally testing on Unix. :]

@ devinemke -

I get the following after submitting the form.

Warning: mysql_errno(): supplied argument is not a valid MySQL-Link resource in C:\XAMPP\xampp\htdocs\dj2\submit_show1.php on line 118

Warning: mysql_error(): supplied argument is not a valid MySQL-Link resource in C:\XAMPP\xampp\htdocs\dj2\submit_show1.php on line 118
: Thank You For Submitting Your Next Time Slot

edit;
removed the arguments (I think? haha) and now I get this once the form is submitted.

0: Thank You For Submitting Your Next Time Slot

bradgrafelman

Heinous wrote:
Not locally testing on Unix. :]

Doesn't matter. I use Unix timestamps in all of my PHP scripts which have been used on Unix and my Windows servers.

Also, how did you try to use those two commands? Normally, if only one connection is made to MySQL, you don't need to give it a parameter at all, you can just do something like:

mysql_query($query) or die('MySQL said: ' . mysql_error());

Heinous

Haha. Crud. I was a fooooool.

I didn't have a mysql_query thingo in there, just that code setting it as a variable. Blah, oops.

Thanks for the help guys.

bradgrafelman

😉

Don't forget to mark this thread resolved!