Invalid query: You have an error in your SQL syntax

[jeff5656]

I get "Invalid query: 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 15"

 

Here is the entire code (not long):

I cant seem to find the error!

<?php
include "connectdb.php";

$check1_date = date("Y-m-d", strtotime($_POST['check1_date']));
$check2_date = date("Y-m-d", strtotime($_POST['check2_date']));
$check3_date = date("Y-m-d", strtotime($_POST['check3_date']));
$tp_date = date("Y-m-d", strtotime($_POST['tp_date']));

$sql = "UPDATE tx SET
patient = '" . $_POST['patient'] . "',
   	mrn = '" . $_POST['mrn'] . "',
   	check1 = '" . $_POST['check1'] . "',
   	check2 = '" . $_POST['check2'] . "',
   	check3 = '" . $_POST['check3'] . "',
   	comments = '" . $_POST['comments'] . "',
check1_date = '$check1_date',
check2_date = '$check2_date',
check3_date = '$check3_date',
signoff_status = '" . $_POST['signoff_status'] . "',
tp_date = '$tp_date'

WHERE id_incr = ".$_POST['id_incr']."";
if (isset($sql) && !empty($sql)) {
$result = mysql_query($sql) or die ("Invalid query: " . mysql_error());

}

header("Location: displayactive.php");

?>

[9three]

In your if statement do this:

 

echo $sql;

 

Does that query look correct?

[ldougherty]

Try updating your query as follows..

 

$sql = "UPDATE tx SET patient = $_POST['patient'], mrn = $_POST['mrn'], check1 = $_POST['check1'], check2 = $_POST['check2'], check3 = $_POST['check3'], comments = $_POST['comments'], check1_date = '$check1_date', check2_date = '$check2_date', check3_date = '$check3_date', signoff_status = $_POST['signoff_status'], tp_date = '$tp_date' WHERE id_incr = $_POST['id_incr']";

[jeff5656]

Thanks.  The problem was I forgot to bring id_incr over from the form as a hidden POST.  The echo of sql helped me figure that out.  Thanks

[jeff5656]

To ldougherty: that is a much cleaner code thanks!