Date Conversion to MySql doesn't work

natesdad

Can someone please tell me why this date conversion doesn't work? I thought it would. You don't have to fix it unless you want to. I don't mind fixing it myself but I do not understand why it doesn't work. It kinda works if I put in $date[0] or $date [1], etc. Any thoughts? The rest of the page works fine. I didn't show all of it. This is the only part that is not working.

Thanks
Stephen

<?php

if ($submit):

list($m, $d, $y) = explode("/", $date[$y]);

! NOTE ! - It works if I put in $date[0] - it will update all rows with the value entered in the date form on Row #1
If I put $date[1] - it will update all rows with the value entered in Row #2, etc.

$sql = "INSERT INTO tblData (EID, CID, SID, Date, Time, Description) VALUES";

for ($y=0; $y<count($cid); $y++)
{
if ($y == 0)
$sql .= "('$id', '$cid[$y]','$sid[$y]','$y-$m-$d','$time[$y]','$desc[$y]')";

else
$sql .= ", ('$id', '$cid[$y]','$sid[$y]','$y-$m-$d','$time[$y]','$desc[$y]')";

}
?>

<TR>
<TD width= 10></TD>
<TD><input type="text" name="cid[]" .../></TD>
<TD><selinput type="text" name="sid[]" .../></TD>>
<TD><input type="text" name="date[]" .../></TD>
<TD><input type="text" name="time[]" .../></TD>
<TD><input type="text" name="desc[]" .../></TD>
</TR>

<TR>
<TD width= 10></TD>
<TD><input type="text" name="cid[]" .../></TD>
<TD><selinput type="text" name="sid[]" .../></TD>>
<TD><input type="text" name="date[]" .../></TD>
<TD><input type="text" name="time[]" .../></TD>
<TD><input type="text" name="desc[]" .../></TD>
</TR>

etc.

etc.
<... submit >.... etc.

natesdad

it should read:
list($m, $d, $yr) = explode("/", $date[$y]);

and

for ($y=0; $y<count($cid); $y++)
{
if ($y == 0)
$sql .= "('$id', '$cid[$y]','$sid[$y]','$yr-$m-$d','$time[$y]','$desc[$y]')";

else
$sql .= ", ('$id', '$cid[$y]','$sid[$y]','$yr-$m-$d','$time[$y]','$desc[$y]')";

}

I do not have a variable conflict/usage issue. That was my fault when I typed the post.
Sorry about the confusion.

tekky

what the resulting SQL statement you get when it doesnt work?

natesdad

nothing gets inserted in the database.

I was thinking, maybe I need a for loop for the list statement.

something like:
for ($x=0; $x<count($cid); $x++)
{
if ($x == 0)

list($m, $d, $yr) = explode("/", $date[$x]);

}

tekky

Not the question that I asked

You are building an Sql ($sql) statement in your script, echo it out and paste what it is when you make the query, 9/10 times the problem is your SQL statement you are sending isnt what you expect... once we know what it is producing, we can trouble shoot why your SQL is coming out wrong

Originally posted by natesdad
nothing gets inserted in the database.

I was thinking, maybe I need a for loop for the list statement.

something like:
for ($x=0; $x<count($cid); $x++)
{
if ($x == 0)

list($m, $d, $yr) = explode("/", $date[$x]);

}

natesdad

Thank you for your help. I figured out my problem and learned some things:
1. The reason it wasn't working is my variable $y wasn't defined until the for loop. Where if says "for ( $y=0 ..."

Then my "list" statement needed to be put in my for loop.

Then I changed "...count($cid)" to "...count($date)" because that is really the variable in question (or some reason :bemused: ).

So it works and here is the way i have it:

for ($y=0; $y<count($date); $y++)
{
list($m, $d, $yr) = explode("/", $date[$y]);
if ($y == 0)
$sql .= "('$id', '$cid[$y]','$sid[$y]','$yr-$m-$d','$time[$y]','$desc[$y]')";

else
$sql .= ", ('$id', '$cid[$y]','$sid[$y]','$yr-$m-$d','$time[$y]','$desc[$y]')";

}

Thank you again,
Stephen