Each Record has own checkbox

SFDonovan

I have a list of college courses on a page. Click on a course and you go to the course page where it listed a number of Sessions (Classes) each having its own row where I need each session (row) to have a checkbox. The checkbox is a status to show if the session has been completed.

echo "<form action='".$_server['php_self']."' method='post'>";

if($TLstatus=='1'){
	echo" <td width='10%' align='center'><input type=\"checkbox\" name=\"status[]\" value=\"checked\"></td>";
	}else{
	echo" <td width='10%' align='center'><input type=\"checkbox\" name=\"status[]\"></td>";
	}

I then have:

echo "<input type='hidden' name='Session_ID' value='$Session_ID'>\n";
	echo "<td colspan=\"7\" align=\"center\"><input type=\"Submit\" name=\"Update\" value=\"Update\"></td>";

In the beginning I test to see if the form was posted:

if(isset($_POST["Update"]) && ""!=$_POST["Update"]) {
$Session_ID = $_POST['Session_ID'];

foreach($_POST['status'] as $value){ 
$setstatus = $db->sql_query("UPDATE ".$prefix."_tl_session SET status =$value WHERE Session_ID = '$Session_ID'");
}
}

Not working yet.

Am I in the ballpark or have I missed something?

I have status as datatype set('0', '1')

sneakyimp

Try checking for errors resulting from your query. That could be the problem. Also, "not working yet" is not very specific now is it?

SFDonovan

Well the page just refreshes and the checkboxes are not checked.

I updated the code and added error checking but nothing happened.

if(isset($_POST["Update"]) && ""!=$_POST["Update"]) {
$Session_ID = $_POST['Session_ID'];
foreach($_POST['status'] as $value){ 
$setstatus = $db->sql_query("UPDATE ".$prefix."_tl_session SET status ='$value' WHERE Session_ID = '$Session_ID'");
if (!$setstatus) {
		echo("<p>Error performing query: " . mysql_error() . "</p>");
		exit();   

		}
}
}

sneakyimp

In that case, it would appear that $TLstatus is never equal to 1.

SFDonovan

The values for status never get written to the table.

I have my opening form tags.

echo "<form action='".$_server['php_self']."' method='post'>";

a bunch of stuff in between then the checkboxes to display the status.

if($TLstatus=='1'){
	echo" <td width='10%' align='center'><input type=\"checkbox\" name=\"status[]\" value=\"checked\"></td>";
	}else{
	echo" <td width='10%' align='center'><input type=\"checkbox\" name=\"status[]\"></td>";
	}		

<\form>

Then the submit button.

echo "<input type='hidden' name='Session_ID' value='$Session_ID'>\n";
	echo "<td colspan=\"7\" align=\"center\"><input type=\"Submit\" name=\"Update\" value=\"Update\"></td>";
	echo "</tr>\n";

I check for the _POST value when the page refreshes. If so I update the table.

if(isset($_POST["Update"]) && ""!=$_POST["Update"]) {
$Session_ID = $_POST['Session_ID'];
foreach($_POST['status'] as $value){ 
$setstatus = $db->sql_query("UPDATE ".$prefix."_tl_session SET status ='$value' WHERE Session_ID = '$Session_ID'");
if (!$setstatus) {
		echo("<p>Error performing query: " . mysql_error() . "</p>");
		exit();   

		}
}
}

SFDonovan

Now that I have thought about this some more I realize I need two things for each record. The Session_ID and the value of the checkbox (status)

I put this in:

if($TLstatus=='1'){	
	echo" <td width='10%' align='center'><input type=\"checkbox\" name=\"status[{$Session_ID}]\" value=\"checked\"></td>";
	}else{
	echo" <td width='10%' align='center'><input type=\"checkbox\" name=\"status[{$Session_ID}]\"></td>";
	}

So then I could do this:

foreach($_POST['status'] as $Key => $value) { 
$Session_ID = $_POST['Session_ID'];
$setstatus = $db->sql_query("UPDATE ".$prefix."_tl_session SET status ='$value' WHERE Session_ID = '$Key'");
die('<br />$setstatus = '.$setstatus.'<br />');

Wouldn't $value be the value of the checkbox and $Key be the Session_ID

I use this:

die('<br />$setstatus = '.$setstatus.'<br />');

To get the interpreted value of $setstatus.

It comes back as $setstatus = 1

sneakyimp

Perhaps you should echo out each query that is running to see what is actually being called?

Also, you don't check for errors from the query...it could be failing every time.