Sql query not working

[Duskwalker]

Im sorry, I just keep trying to code things that are beyond my php skill..

if ($view == pvp) {


	if ($stat[pvp] = N) {
		print "You currently have Peace mode on. What this does, is makes other players unable to attack you in the game. Do you want to to turn Peace mode off? <a href='account.php?view=pvp&step=1'>Yes</a>";
	} else {
		print "You currently have Peace mode off. What this does, is allows other players to attack you in the game. Do you want to to turn Peace mode on? <a href='account.php?view=pvp&step=2'>Yes</a>";
	}


if ($step == 1) {
print "You have turned Peace mode off. Other players can now attack you.<br><br><a href='account.php'>back</a>";
	$sql = "UPDATE players SET pvp = N WHERE id = '".$stat[id]."'";
	mysql_query("$sql") or print "Didnt work, go get Duskwalker.";


}
if ($step == 2) {
print "You have turned Peace mode on. Other players cannot attack you.<br><br><a href='account.php'>back</a>";
	$sql = "UPDATE players SET pvp = Y WHERE id = '".$stat[id]."'";
	mysql_query("$sql") or print "Didnt work, go get Duskwalker.";

}
}

Thanks in advance. Everything is right I believe except the mysql_query

-Dusk

[rich_d]

Actually there's more here than just the query you ought to look at.

1. if ($view == pvp)

What's pvp? $pvp surely? (even if it's an integer)

2. if ($stat[pvp] = N)

That should be ==, not just a single equals sign. That's for variable assigning, not comparison.

3. mysql_query("$sql")

You don't need the quote marks. Although they won't break it, you shouldn't have them.

4. Use echo instead of print.

5. $sql = "UPDATE players SET pvp = N WHERE id = '".$stat[id]."'";

N needs to be quoted and also you can do it like this:

$sql = "UPDATE players SET pvp='N' WHERE id={$stat[id]}";

Hope that gets you started.

Cheers,

Rich