Deleting Rows - Should be an easy one!

Philth

Hello folks,

Hopefully this will be an easy one for most of you guys. I don't delve into PHP much so I'd appreciate some help.

The aim of the script below is to display a collection of rows with the aim of specifying one for eventual deletion.

It returns the "row deleted" echo fine, but doesn't actually delete anything.

Expand|Select|Wrap|Line Numbers

 
    <?

include("connect.php"); 
 
if(!isset($cmd)) 

{

   $result = mysql_query("select * from events order by eventName"); 
 
   while($r=mysql_fetch_array($result)) 

   { 
 
      $title=$r["eventName"];

      $id=$r["eventNumber"];

        $location=$r["eventLocation"];

          $date=$r["eventDate"];
 
     echo "<span class='smalltitle'>$title</span><br><span class='bodytext'>$location<br>$date</span><br> "; 

      echo "<a href='delete_event.php?cmd=delete&id=$id'>$title - Delete</a><br><br>";
 
    }

}

?>

Expand|Select|Wrap|Line Numbers

 
<?

if($_GET["cmd"]=="delete")

{

    $sql = "DELETE FROM events WHERE eventNumber=$id";

    $result = mysql_query($sql);

    echo "Row deleted!";

}

?>

Many thanks.

Markus

If there's an error generated by mysql, it won't tell you - you have to find out for yourself.

Update your code to this:

Expand|Select|Wrap|Line Numbers

 
<?

if($_GET["cmd"]=="delete")

{

    $sql = "DELETE FROM events WHERE eventNumber=$id";

    // Notice mysql_error()

    $result = mysql_query($sql) or die("Error: ". mysql_error());

    echo "Row deleted!";

}

?>

Deleting Rows - Should be an easy one!

re: Deleting Rows - Should be an easy one!

Similar PHP bytes

/bytes/development

/bytes/it

/bytes/about