foreach Warning!

[balkan7]

i have problem whit delete product, just delete latest product added, here is the code of warning:
[code]<td><input type='checkbox' name='row[]' value='$row[id]'></td>
<tr><td><p align='right'><input type='submit' name='delete' value='Delete'></td></tr>
[/code]

[code]case "delete":
      //if nothing is submitted in the
      //row[] array, then error!
      if(empty($_POST['row']))
      {
          echo "Nothing to delete!";
      }
      if(isset($_POST['delete']))
      {
          $delete_array = $_POST['row'];
     
          //loop through each individual
          //item in the array
         
          foreach($delete_array as $val)
          {
                //delete them!
                $query = "DELETE FROM software WHERE id = '$val'";
                $result = mysql_query($query) or die(mysql_error());
          }
          echo "Product has been successfull delete!";
      }
      break;
[/code]

and show me:
[code]Nothing to delete!
Warning: Invalid argument supplied for foreach() in /home/xxx/public_html/test/index.php on line 256
Product has been successfull delete![/code]

[Vikas Jayna]

You can check whether [b]$delete_array[/b] is an array using [b]is_array()[/b] function and only then process the foreach loop. Like this:-
[code]if(is_array($delete_array))
{
          foreach($delete_array as $val)
          {
                //delete them!
                $query = "DELETE FROM software WHERE id = '$val'";
                $result = mysql_query($query) or die(mysql_error());
          }
}[/code]

[balkan7]

but where i can set this code?
[code]if(isset($_POST['delete']))
      {
[/code]

[Vikas Jayna]

You already have the [b]foreach[/b] loop, only the [b]if condition[/b] is to be added with the [b]foreach[/b] nested within.

Good luck!

[balkan7]

i was fixed.
but only take action delete for one product, if you checked many products for delete, just delete one :)

[Adika]

Here is the solution:

[code]
case "delete":
      //if nothing is submitted in the
      //row[] array, then error!
    if(isset($_POST['delete'])) {
        if(empty($_POST['row']))
      {
          echo "Nothing to delete!";
      } else {
          $delete_array = $_POST['row'];
     
          //loop through each individual
          //item in the array
         
          foreach($delete_array as $val)
          {
                //delete them!
                $query = "DELETE FROM software WHERE id = '$val'";
                $result = mysql_query($query) or die(mysql_error());
          }
          echo "Product has been successfull delete!";
      }
  }
      break;
[/code]

Just a simple else statement was needed and a little line change.