[Bonzo]

All very messy and NeverMind will probably complain : )

I am generating a image table from an array - This works OK

Now I want to select a " check box " ( or more than one ) to delete that entry.

All that happens is the last entry in the table comes up for deletion every time !

PHP Code:

echo "<td align='center'><a href = display.php?pic=$row[image] target=_blank>

        <img src = thumb/$row[image].JPG alt=$row[image]></a>

        <FORM ACTION = 'delete.php' METHOD = POST>

        <INPUT TYPE=HIDDEN input NAME=delete value='$row[image].JPG'>

        <INPUT TYPE='CHECKBOX'>

        </td> 

Can someone tell me where I am going wrong.

Anthony

[NeverMind]

HEY!!
I don't usualy complain unless something worth complaining happen
anyway the form should be started before the loop of the table elements!
and you don't need to have a hidden input for that !
the checkbox itself should have the value (name of image)..
and one note, I think many browser will complain (not me!) about this messed up HTML syntax !! it will work but it's really messed up ! sometimes you use single quotes sometimes you don't sometimes you use spaces and sometimes not !!
have a coding standered, it will help You in maintaining your code !
like this:

PHP Code:

echo'<FORM ACTION="delete.php" METHOD="POST">';



//LOOP OF TABLE (if there was one)

//one element of the table:

echo '<td align="center"><input type="checkbox" name="delete[]" value="'.$row['image'].'.JPG" />

<a href="display.php?pic='.$row['image'].'" target="_blank"> 

<img border="0" src="thumb/'.$row['image'].'.JPG" alt="'.$row['image'].'"></a> 

this code shall make an array containing all images' names that will be deleted ..
so loop through the array and DELETE FROM table ..

PHP Code:

$delete = array();

$delete = $_POST['delete'];



for ($i=0; $i<count($delete); $i++) {

  mysql_query("DELETE FROM table_1 WHERE image='$delete[$i]'")or

  die(mysql_error());

}