deleting a record...not working

Goonboonie

Hi,

I am having problems with this code. The delete part of it is not deleting a record from the mySQL database when the submit button is pushed on the page that contains this code.

require_once ("x0x0x0x.php");
$db=mysql_select_db(DB_NAME);

if (!$db) {
echo "Could not change into the database";
exit;
}
if(isset($_POST['submit'])) {

$query = "Delete FROM LIST_table WHERE listee_id='$listee_id'";
mysql_query($query);

} else {

$query = "SELECT * FROM LIST_table WHERE listee_id='$radiobutton'";
$result = mysql_query($query) or die("Cannot perform record query.");
$row = mysql_fetch_array($result);

?> <table width="100%" border="0" cellspacing="0" cellpadding="0">
<tr>

I have the page acting on itself to peform the delete operation.

This page displays a single record and is supposed to allow a user to delete the record that is being viewed. Instead when the delete button is pushed the web page refreshes in the browser and the values for things like name, address, phone number, etc are blank.

Thanks

Goonboonie

LordShryku

Use [man]mysql_error[/man] to find out why.

Frederick

You need to add a hidden tag in your form to hold the id number of your record so when you submit the form, the id continues with the actions

echo "<input type=\"hidden\" name=\"listee_id\" value=\"$listee_id\">\n";

Goonboonie

Hi,

Thanks for your suggestions. I have the following code in the php page to store the listee_id number.

I don't know what else needs to be done. 🙁

rgermain

Ok,
Like LordShryku said you need to find out what your error is by using [man]mysql_error[/man].

Example:

<? 
$query = "SELECT * FROM some_tbl WHERE id = some_id";
$ result = mysql_query($query) or die("Error 001: " . mysql_error);
?>

This will tell you what the error is and the only reason I add the "Error 001:" is in case I have more than one query. That way I know right where to look within my code.

dnast

I use

<? 
$query = "SELECT * FROM some_tbl WHERE id = some_id";
$result = mysql_query($query)
  or die("Database Error (Line ".__LINE__."): ".mysql_error());
?>

And it tells you exactly what line the error is on. I like the "Error 001" idea if your text editor doesn't show line numbers, though.

Goonboonie

Hi,

Thanks for all your suggestions. I found out something interesting by simply changing this if(isset($POST['submit']))
When I capitalized the word submit as in if(isset($POST['Submit']
the delete function worked off of the previous page.

The previous page (admin page) as I mentioned is a table of all the records in the database but with limited information. With each record is a radio button and corresponding submit button labelled full record.

When the radio button is clicked and matching full record button this calls the display_record.php the page in question. However since I changed the code as described above, the display_record.php only shows a blank page. When I view the database the corresponding record that was to be displayed is deleted.

if (!$db) {
echo "Could not change into the database";
exit;
}
if(isset($_POST['Submit1'])) {

$query = "Delete FROM LIST_table WHERE listee_id='$_POST[radiobutton]'";
mysql_query($query);

} else {

$query = "SELECT * FROM LIST_table WHERE listee_id='$radiobutton'";
$result = mysql_query($query) or die("Cannot perform record query.");
$row = mysql_fetch_array($result);

The else statement above is never executed when the word submit is capitalized. What I want to happen is that the matching record will display when display_record.php is called and that the user can delete only if they click the delete button on that web page.

I have never typed that much in my life. I hope I cleared things up a bit for you.

Thanks again

Goonboonie

This was the code that was causing me grief. I changed it so now it reads like this : value="<?php echo $row['radiobutton'] ?>">

Thanks for all your help. Your suggestions forced me to analyze the code more closely. It's always an easy problem with a hard solution.