It looks good, but I see now that you can update the PlayerID... that means that you actually should be using $_GET['PlayerID'] on line36. Try that. If you were updating the PlayerID in your tests it wouldn't find or update the record since the playerID changed.