[SOLVED] Delete query not working, though....

[kernelgpf]

I'm trying to make it so people can remove friends from their friendlist, but the query won't work. Both variables are set, query works if I put it in the SQL box in phpmyadmin.. here's my code.

[code]
<?php
include "header.php"; //DB connection
if($_GET['action'] == "removefriend"){
$pid=$_GET['playerid'];

mysql_query("DELETE FROM `friendslist` WHERE `playerid`='$pid' AND `friendof`='$sid'");

echo "$sql";

print "<p>Successfully removed friend.<p>";

}



$query=mysql_query("select playerid,note,friendid from friendslist where friendof='$sid'");
$num=mysql_num_rows($query);
if($num == "0"){
print "No friends? What a shame. Get out there a socialize!";
include "footer.php";
exit;
}
else{
print "You have <b>$num</b> friends.<p>";
}
print "<table border=2><th width=100>Friend</th><th width=300>Note</th><th width=200>Remove?</th>";
for($i=0;$i<$num;$i++){
$row=mysql_fetch_array($query);
$query2=mysql_query("select basicname from users where id='$row[playerid]'");
$row2=mysql_fetch_array($query2);
print "<tr><td class=tstyle3><a href='/userprofiles.php?id=$row[playerid]'>$row2[basicname]</a></td><td class=tstyle3>$row[note]</td><td class=tstyle3> <a href='friendlist.php?action=removefriend&playerid=$row[friendid]'>[x] remove</a></td></tr>";
}
include "footer.php";
?>[/code]

[trq]

What does....

[code=php:0]
mysql_query("DELETE FROM `friendslist` WHERE `playerid`='$pid' AND `friendof`='$sid'") or die(mysql_error());
[/code]

produce?

[HuggieBear]

You aren't defining $sid anywhere.

You have $pid defined ok, but not $sid.

Regards
Huggie

[kernelgpf]

$sid is defined, it's a session variable set in header.php. And the mysql_error() doesn't put out anything.

I've printed both $pid and $sid to make sure they work- they do.

[simcoweb]

Ok, where is the session variable being transferred to? There's no mention of a session_start(); in the posted. code. Is it in the header file?

[kernelgpf]

Yeah, in the header.php file.

[simcoweb]

Can we see it, pretty please? ;)

[trq]

Can we try this?

[code=php:0]
$sql = "DELETE FROM `friendslist` WHERE `playerid`='$pid' AND `friendof`='$sid'"
// mysql_query($sql);
die($sql);
[/code]

What does this produce?

[kernelgpf]

thorpe> "DELETE FROM `friendslist` WHERE `playerid`='29' AND `friendof`='7'"

[HuggieBear]

If your passing integers then there's no need for the single quotes.

Regards
Huggie

[kernelgpf]

..okay, but I took them away and my query still isn't working. =/

[kernelgpf]

I figured out the problem. Thanks guys. ^^