[RESOLVED] Dynamic Check Boxes

[middleman666]

Hi All,

I've made this code and it works, but the form has to be submitted twice before the changes are made to the read out.

I know it's an ordering issue, but I'm not sure of the solution.

Here's the snippet of my form code showing the checkboxes:

    <?php			  	  

	$ABquery=mysql_query("SELECT * FROM acts WHERE ClientID = '".$clientID."' AND ShowID = '".$SID."' ORDER BY TimeOnStage DESC");  

	while($ABrow=mysql_fetch_array($ABquery)) {
		$ActID = $ABrow['id'];
		$BandID = $ABrow['BandID'];
		$Bresult=mysql_query("SELECT * FROM band WHERE id = '".$BandID."'"); 
        $Brow=mysql_fetch_array($Bresult);

	$ActArray[] = "Act".$ActID;

	echo '<tr>
	       <td class="HDivider"><strong>'.$Brow['Band'].'</strong></td>
	       <td><input name="Act'.$ActID.'" type="checkbox" id="Act'.$ActID.'" value="del"></td>
		  </tr>';
}


  //===============REMOVE ACTS===============================================
  $i=0; 
  while($i<count($ActArray)) { 

		$Box = $ActArray[$i];
		$ActPost = $_POST[$Box];
		$Act = substr($ActArray[$i],3,999);
		//echo $ActArray[$i].' = '.$ActPost.' = '.$Act.'<br>';

		if ($_POST[$Box] == 'del') {			

			$ABquery=mysql_query("SELECT * FROM acts WHERE ClientID = '".$clientID."' AND id = '".$Act."'");  
			$ABrow=mysql_fetch_array($ABquery);
			$BandID = $ABrow['BandID'];
				$Bresult=mysql_query("SELECT * FROM band WHERE id = '".$BandID."'"); 
				$Brow=mysql_fetch_array($Bresult);

			mysql_query("DELETE FROM acts WHERE id='".$Act."' AND ClientID = '".$clientID."' LIMIT 1"); 

			echo '<tr><td colspan="2">'.$Brow['Band'].' has been removed.<br /></td></tr>';

		}

	$i++; 
  } 
  //=========================================================================	

?> 

Basically, I want to be able to run the REMOVE ACTS part before reading out the checkboxes. I've tried simply putting this before the boxes are read out, but then it doesn't seem to find the $_POST's.

Anyone have any suggestions?

Thanks.

[leatherback]

As you are not doing any update, nor using the $_POST data in the code you submitted.. No. It is not possible to say how you should post your code.

In short:
- Delete
- Update
- Display query

[middleman666]

Cheers for the reply.

I managed to give it a bit of a bodge treatment to get it to work without having to be submitted twice.

Here's the reworked snippet of code:

<?php			  	  

	$ABquery=mysql_query("SELECT * FROM acts WHERE ClientID = '".$clientID."' AND ShowID = '".$SID."' ORDER BY TimeOnStage DESC");  

	while($ABrow=mysql_fetch_array($ABquery)) {
		$ActID = $ABrow['id'];
		$BandID = $ABrow['BandID'];
		$Bresult=mysql_query("SELECT * FROM band WHERE id = '".$BandID."'"); 
        $Brow=mysql_fetch_array($Bresult);

	$ActArray[] = "Act".$ActID;

	echo '<tr>
	       <td></td>
	       <td><input name="Act'.$ActID.'" type="hidden" id="Act'.$ActID.'"></td>
		  </tr>';
}


  //===============REMOVE ACTS===============================================
  $i=0; 
  while($i<count($ActArray)) { 

		$Box = $ActArray[$i];
		$ActPost = $_POST[$Box];
		$Act = substr($ActArray[$i],3,999);
		//echo $ActArray[$i].' = '.$ActPost.' = '.$Act.'<br>';

		if ($_POST[$Box] == 'del') {			

			$ABquery=mysql_query("SELECT * FROM acts WHERE ClientID = '".$clientID."' AND id = '".$Act."'");  
			$ABrow=mysql_fetch_array($ABquery);
			$BandID = $ABrow['BandID'];
				$Bresult=mysql_query("SELECT * FROM band WHERE id = '".$BandID."'"); 
				$Brow=mysql_fetch_array($Bresult);

			mysql_query("DELETE FROM acts WHERE id='".$Act."' AND ClientID = '".$clientID."' LIMIT 1"); 

			//echo '<tr><td colspan="2">'.$Brow['Band'].' has been removed.<br /></td></tr>';

		}

	$i++; 
  } 
  //=========================================================================	

$ABquery=mysql_query("SELECT * FROM acts WHERE ClientID = '".$clientID."' AND ShowID = '".$SID."' ORDER BY TimeOnStage DESC");  
while($ABrow=mysql_fetch_array($ABquery)) {
	$ActID = $ABrow['id'];
	$BandID = $ABrow['BandID'];
	$Bresult=mysql_query("SELECT * FROM band WHERE id = '".$BandID."'"); 
    $Brow=mysql_fetch_array($Bresult);

	$ActArray[] = "Act".$ActID;

	echo '<tr>
	       <td class="HDivider"><strong>'.$Brow['Band'].'</strong></td>
	       <td><input name="Act'.$ActID.'" type="checkbox" id="Act'.$ActID.'" value="del"></td>
		  </tr>';
}

?> 

Basically it first does a dummy reading out of the checkboxes which are simply hidden fields, then it does the usual deleting of any that have been checked and finnaly it reads them out again, this time without the ones that have just been deleted.

It's a bit ugly, but as long as it works I think I can live with it for now.