MySQL Weird Delete Problem

Anon

Can't seem to delete a record from a table. I can echo the query statement and paste it on the MySQL command line and it works fine, but not from my PHP script. No error is raised, just nothing happens. Besides dumb logic errors on my part, is there any weird situations I should be on the look out for? Do I just have too many indecies?

create table counts
(
count_id INT unsigned not null
auto_increment primary key,
user_id SMALLINT unsigned not null,
contribtyp_id SMALLINT unsigned not null,
count_year YEAR not null,
count TINYINT unsigned not null,
count_timestamp TIMESTAMP(10),
index(user_id),
index(user_id, contribtyp_id)
);
TIA,
Alex

tomhath

Are you sure you executed the DELETE query?

Post the php, there's no (good) reason it shouldn't work.

Anon

if ($f_count_count > 1)
{ //decrement
$f_count_count -= 1;
$query = "UPDATE counts
SET count='$f_count_count'
WHERE count_id='$f_count_id'";
}else{
$query = "DELETE from counts
WHERE count_id='$f_count_id'";
}
$result = mysql_query($query) or die (" Unable to Decrement Count Record " . mysql_error() );

Thanks so much for taking an interest in my problem. (I'm not worthy...:-( )

steved

If your count_id column is a numeric of some sort, which it appears to be, you should probably remove the single quotes from your delete statement like:

$query = "DELETE from counts
WHERE count_id = $f_count_id";

The same would apply to your update statement.

Anon

Thanks, but it didn't help.

Anon

OK, I just solved this, but it's still not what you might expect so let me tell you about it. After the delete takes place, the following is executed:

$URL="fac_mbr_viewall.php?dok=1";
header ("Location: $URL");

I assumed (key word) this command would exit the page immediately - instead the PHP below this statement continues to execute prior to the redirect. Guess what...the code below this adds back the record I just deleted.

A thousand Thanks to everyone who reviewed my problem.

Alex

edwardsbc

Yup, exactly correct. A note to other readers, after issuing a

header(location:...);

you nearly always want to put a

exit;