Can someone see if this has any errors?

[Dethman]

if($_POST['defcon']){
$defcon=$_POST['defcon_rate'];
  $q=mysql_query("INSERT INTO `user_users` SET `alert`='$defcon' WHERE `userid`='$player['userid']'");
if( $q ){
	header("location: lair.php?strErr=You Have Upgraded Your Defcon Rate");
}
}

 

I think there may be something wrong with this can someone help?

 

Thanks,

Brian Flores AKA Dethman

[ShimmyShine]

Try this:

 

if(isset($_POST['defcon'])){
$defcon = $_POST['defcon_rate'];
$player = $player['userid'];
  $q = "UPDATE `user_users` SET `alert`='$defcon' WHERE `userid`='$player'";
  $result = mysql_query($q);
if($result){
	header("location: lair.php?strErr=You Have Upgraded Your Defcon Rate");
} else {
    echo "Error";
  }
}

 

Shimmy

[DarkWater]

Yeah, you can't use SET in an INSERT statement:

$q = "INSERT INTO `user_users` SET `alert`='$defcon' WHERE `userid`='$player'";

 

Maybe you meant UPDATE?

[trq]

SET is perfectly valid in an INSERT statement.

 

The big question here is, does it work? Your meant to tell us something isn't working, describing what you want it to do and what it actually does do, from there we attempt to help. Simply stating you 'think' something may be wrong gets us knowhere.

[ShimmyShine]

Forgot to change that part

 

Shimmy