[RESOLVED] Posting data from checkboxes generated by an array

facarroll

I think I'm still a newbie, and will probably always be so.
I'm trying to write a website that allows students to self test. I allow a subscribing administrator (read teacher) to create up to six classroom and one inactive (students no longer fit any of the six groups) lists of students to undertake testing. The administrator can give their groups unique names. My current problem is to create a script to allow the administrator to move student names into a different classroom group.
I've created a form using checkboxes for the selected students and radio buttons for nominating their new group. The problem is that the list of student names can be of any length, and this is displaying correctly for the entry form, but I have no idea about how to post this collection of data, and how to update the database. Ive included the code I've written, and have shown a full line of comment slashes either side of what I believe is the problematic code. I've also attached a .doc file thatshows a screenshot of the form on a webpage.
If anyone wants to help, I can supply further details of the database.
I'm also looking for someone who may be willing to help me out with similar problems from time to time for a fee, bank transfer, negotiable.

<?php
session_start(); 
// Place Session variable 'recid' into local variable
	$recid 		 = $_SESSION['recid'];
	$username    = $_SESSION['name'];
	$school    = $_SESSION['college'];
//Connect to the database
include_once "demo_conn.php";
// Query members data from the database and organise it for display
$sql = mysql_query("SELECT group1, group2, group3, group4, group5, group6 FROM members WHERE recid='$recid' LIMIT 1");
while($row = mysql_fetch_array($sql)){
$group1 = $row['group1'];
$group2 = $row['group2'];
$group3 = $row['group3'];
$group4 = $row['group4'];
$group5 = $row['group5'];
$group6 = $row['group6'];
}
////////////////////////////////////
// Process the form 
if ($_POST['group1']) {
	$snames = $_POST['snames[]'];
    $userGroup = $_POST['group1'];
    $userGroup = $_POST['group2'];
    $userGroup = $_POST['group3'];
    $userGroup = $_POST['group4'];
	$userGroup = $_POST['group5'];
	$userGroup = $_POST['group6'];
	$userGroup = $_POST['group7'];
    $sql = mysql_query("UPDATE users SET userGroup='userGroup' WHERE recid='$recid' AND userId='$snames'"); 
    echo 'Your group names have been updated.<br /><br />
To return to your editing page, <a href="member_account.php">click here</a>';
exit();
} // close if posted
////////////////////////////////////
?>




<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<head>
<title>Edit Your User Group Names</title>
<link rel="stylesheet" href="style.css" type="text/css" media="screen" />
</head>
<body>
<table align="center" cellpadding="8" cellspacing="8">
  <tr>
    <td width="100"></td>
    <td width="160"><div align="left"> <font size="-1">Select <?php echo "$group1"; ?> students.</font></div></td>
    <td width="10"></td>
    <td width="150"><font size="-1">Send students to ....&nbsp;&nbsp;</font><font color="#CC0000">&nbsp;</font></td>
    <td width="30"></td>
    <td width="190"><div align="left"><font size="-1"></font></div></td>
  </tr>
<?php
// Query member data from the database
$query1 = mysql_query("SELECT userId FROM users WHERE managerId='".$recid."' AND userGroup='".$group1."' ORDER BY userId ASC");
while($row1 = mysql_fetch_array($query1))
{
$firstGroup .=$row1['userId']. ' <input type="checkbox" name="snames[]" value="1" /> ' . "&nbsp;&nbsp;&nbsp;&nbsp;<br /><br>";
}
?>
  <form method="post" enctype="multipart/form-data" action="send_stud_gp_test.php">
    <tr>
      <td>&nbsp;<br /></td>
    </tr>
    <tr>
      <td width="100"></td>
      <td width="160" style="text-align:right;"><?php echo $firstGroup;?></td>
      <td width="10"></td>
      <td width="150" style="text-align:left;">
          <input type="radio" name="group2" value="1" />
          &nbsp;&nbsp;&nbsp;<?php echo $group2; ?>
        <p>
          <input type="radio" name="group3" value="1" />
          &nbsp;&nbsp;&nbsp;<?php echo $group3; ?>
        <p>
          <input type="radio" name="group4" value="1" />
          &nbsp;&nbsp;&nbsp;<?php echo $group4; ?>
        <p>
          <input type="radio" name="group5" value="1" />
          &nbsp;&nbsp;&nbsp;<?php echo $group5; ?>
        <p>
          <input type="radio" name="group6" value="1" />
          &nbsp;&nbsp;&nbsp;<?php echo $group6; ?>
        <p>
          <input type="radio" name="group7" value="1" />
          &nbsp;&nbsp;&nbsp;Deactivate student.
        <p>
          <input type="submit" value="submit" name="Send Data">
  </form>
  	  </td>  

    </tr>  

</table>
</body>
</html>

Krik

First use the userId as the value for each checkbox

<input type="checkbox" name="snames[]" value="<?php echo $row1['userId']; ?>" />

Also your group radio buttons should all have he same name else multiple buttons can be selected. And use the group name to as the value

<input type="radio" name="group" value="<?php echo $group2; ?>" />&nbsp;&nbsp;&nbsp;<?php echo $group2; ?>
<p>
<input type="radio" name="group" value="<?php echo $group3; ?>" />&nbsp;&nbsp;&nbsp;<?php echo $group3; ?>
<p>
<input type="radio" name="group" value="<?php echo $group4; ?>" />&nbsp;&nbsp;&nbsp;<?php echo $group4; ?>
<p>
<input type="radio" name="group" value="<?php echo $group5; ?>" />&nbsp;&nbsp;&nbsp;<?php echo $group5; ?>
<p>
<input type="radio" name="group" value="<?php echo $group6; ?>" />&nbsp;&nbsp;&nbsp;<?php echo $group6; ?>
<p>
<input type="radio" name="group" value="" />&nbsp;&nbsp;&nbsp;Deactivate student.
<p>

Then loop through the $POST['snames'] array updating the data for each loop

if ($_POST['group']) {
    $userGroup = $_POST['group'];

foreach($_POST['snames'] as $v) {
    mysql_query("UPDATE users SET userGroup='$userGroup' WHERE recid='$recid' AND userId='$v'");
}
}

Note that

    $userGroup = $_POST['group1'];
    $userGroup = $_POST['group2'];
    $userGroup = $_POST['group3'];
    $userGroup = $_POST['group4'];
    $userGroup = $_POST['group5'];
    $userGroup = $_POST['group6'];
    $userGroup = $_POST['group7'];

sets "$userGroup" to the value of "$_POST['group7']" so you would not be able to set any user to the value of $_POST['group4'], for example.

$snames = $_POST['snames[]']

is incorect syntax

$snames = $_POST['snames']

is how you would assign the $_POST['snames'] array to $snames.

facarroll

Thanks a lot. I've had another reply on another forum also. Between both replies I think I'll be able to understand. I'll get back to you maybe tomorrow.
Thanks again.