[Resolved] Delete User... MySql

[krs1]

Currently I have a link that goes to DeleteUser.php?id=x... And I have never done a delete command before... needless to say I am having some problems:p. Here is what I have....

<?php
$delete = $_GET['id'];
include '../includes/Connect.php';
mysql_select_db("$dbDatabase", $db) or die ("Couldn't select the database.");

$query = mysql_query("DELETE FROM users WHERE user_id='$delete'", $db);
$result=mysql_query($query);
$num=mysql_numrows($result);
$i=0;
while ($i < $num) {
header( "Location: [url]http://www.caycan.ca/plains/admin.php[/url]" );
}
?>
[code=php]

Can someone give me a thwack in the right direction? :bemused:

[devinemke]

you don't use [man]mysql_num_rows[/man] for a DELETE query. if you want to see how many rows were deleted then use [man]mysql_affected_rows[/man]. but even if you use that, why are you calling [man]header[/man] during each while loop iteration?

[krs1]

woah... whoops i am tired... i was playing around trying it to display something.... I had changed delete to select.... .....

[krs1]

this is what i ment i have:

<?php
$delete = $_GET['id'];
include '../includes/Connect.php';
mysql_select_db("$dbDatabase", $db) or die ("Couldn't select the database.");

$Query =mysql_query("DELETE FROM users WHERE user_id='$delete'", $db);
$result=mysql_query($query);
header( "Location: http://www.caycan.ca/plains/admin.php" );
?>

[gufmn]

Where is $query defined? And I don't mean $Query.

[bradgrafelman]

$Query =mysql_query("DELETE FROM users WHERE user_id='$delete'", $db); 
$result=mysql_query($query); 

This is extraneous and is only going to result in errors/warnings. Also, unless you have more than 1 MySQL connection open, the resource for it is not necessary.

Just try this:

$Query =mysql_query("DELETE FROM users WHERE user_id='$delete'");

instead of the two lines above.

[krs1]

Thanks solving my mental block... I'll figure this p-h-p thing sooner or later...:p