[SOLVED] Deleting rows in db

[Lee-Bartlett]

Ok so i want to try minipulate my update code and im not completly sure where i might be going wrong, This code makes a drop down boxs, and pulls the names from my database. So i made a include file which didnt work but am i going down the right track to do this? Wanna do a simple, click on drop down, get name, click go and it deletes that row.

 

codes are

 

<?php  require_once("includes/db_connection.php");


$sql = "SELECT * from tblbasicform";
$res = mysql_query($sql) or die(mysql_error());

echo "<form action=\"\" method=\"post\">";
echo "<select name=\"name\">";

while($row = mysql_fetch_array($res)){

echo "<option>". $row['name'];
        echo "</option>";
$name = $row['name'];
    $email = $row['email'];
    $location = $row['location'];
$buissnes_name = $row['buissnes_name'];
$type = $row['type'];
  }
echo "</select><br>";
echo "<input type=submit value=\"go\">";
echo "</form>";

include("delete.php");

?>

 

delete.php now

 

<?php
if($_GET["name"]=="delete")
{
    $sql = "DELETE FROM news WHERE name=$name";
    $res = mysql_query($sql);
    echo "Row deleted!";
}
?>

[unrelenting]

Are you getting an error?

 

BTW, you don't need the include. Just add the if statement after the require once at the top.

[F1Fan]

Quite a few issues. First, you're using the POST method in your form, but trying to use GET in your delete.php. Second, you're not assigning values to your <option> tags (like <option value='somevalue'>). Third, you are checking to see if $_GET['name'] (should be $_POST) is set, but then assuming that $name will be equal to $_GET (this can work if the server is set up properly). Finally, you're checking if your submitted value is equal to "delete," which it will not be.

[Lee-Bartlett]

hehe quite few problems then

[DarkWater]

Also, you don't have '' around your string in MySQL, nor do you do any error checking on the query.

[Lee-Bartlett]

Hmm seems to be getting bit advance for me, is there a way to dumb this down a little, to somthing much simpler?

[F1Fan]

This is relatively simple stuff. I would suggest going through a tutorial like this one:

http://w3schools.com/php/php_mysql_intro.asp

[Lee-Bartlett]

Ye i always tend to over think stuff, i can do the mysql string to work by its self, but atm i struggle to intergrate it with a form or a drop down, somthing like that.

[Lee-Bartlett]

Hmm that was reltivly simple, cool