How to update radio button's data to mysql database with php?

Question

CODE am trying:

what about update radio button's logic ?? Is this OK ??

            if(isset($_POST['btnSubmit']))
{   
    $book_save = $_POST['book_title'];
    $author_save = $_POST['author_name'];
    $gender_save = $_POST['gender'];

    mysql_query("UPDATE bookss SET book_title ='$book_save',author_name ='$author_save',gender='$gender_save' WHERE ID = '$id'")
                or die(mysql_error());

i refer 2 links in SO but m not getting properly with radio button .... Suggestions are always Welcome ...

Please, don't use mysql_* functions in new code. They are no longer maintained and are officially deprecated. See the red box? Learn about prepared statements instead, and use PDO or MySQLi - this article will help you decide which. — Kermit
– Kermit, Commented Jan 18, 2013 at 19:07

user1991510 · Accepted Answer · 2013-01-18 19:22:20Z

1

I agree with everyone above, are you hard coding the radio button values? if so, I'm not sure how SQL injection would effect radio button entries. In any case, if you're stuck to this structure I'd at least do something like this: $book_save = mysql_real_escape_string($_POST['book_title']);

answered Jan 18, 2013 at 19:22

user1991510

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

1 Comment

Eric Over a year ago

You can't think of a $_POST value as a "radio button entry". I can create a form on my site, or even locally on my computer, and have it post to your form, and attempt to inject different values for your radio buttons than what you hard code. For that reason, every element of the $_POST array should be validated/escaped before anything is used in a query string (unless you're using prepared statements).

SeanWM · Accepted Answer · 2013-01-18 19:06:53Z

0

What you're doing is correct. I would suggest you escape your data though. You're open to sql injection. I'd also look into using PDO as the mysql functions are deprecated.

answered Jan 18, 2013 at 19:06

SeanWM

17.1k10 gold badges54 silver badges85 bronze badges

Collectives™ on Stack Overflow

How to update radio button's data to mysql database with php?

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related