Updating SQL database value in PHP script

Question

I have an IPN script that is doing some work on the amount of a payment received, and when a certain amount is received, it is updating their license code in the database after verifying it with PayPal's IPN service.

This SQL isn't right, it's not updating. The rest of my code is fine because it sends an email, but where's the SQL error at? It's really late and I'm spacing out...

if ($amt == "77.00")
    { 
     mysql_query("UPDATE login_users SET license_code = 3 WHERE username = ". $username ."") or die(mysql_error()); 
      // Change license code in database
    }

You forgot quotes around " . $username . ". You have to put them if you want your variable to be considered as a string. — Vincent Savard
– Vincent Savard, Commented Aug 11, 2011 at 7:24

Pekka · Accepted Answer · 2011-08-11 07:25:02Z

2

You need quotes around the user name if it's a string.

 WHERE username = '". $username ."'

Also make sure $username is properly sanitized:

$username = mysql_real_escape_string(... wherever the value is coming from ...);

answered Aug 11, 2011 at 7:25

Pekka

451k150 gold badges989 silver badges1.1k bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Brenden Clerget Over a year ago

Thanks a lot Pekka, your answer worked straight out of the box. And yes, regardless that it's been verified with Paypal I should probably do that, thanks a lot.

JK. · Accepted Answer · 2011-08-11 07:23:10Z

0

Put SET before WHERE and add single quotes around username

"UPDATE login_users SET license_code = 3 WHERE username = '". $username ."'";

answered Aug 11, 2011 at 7:23

JK.

5,1261 gold badge29 silver badges26 bronze badges

1 Comment

Brenden Clerget Over a year ago

Just updated my code above, Dreamweaver says no syntax errors with mine, had to add a trailing " to yours because it's appending a variable, but it's not updating still for some reason...

Collectives™ on Stack Overflow

Updating SQL database value in PHP script

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related