[RESOLVED] How pass radiobutton values?

albynas

Hi,
I am stuck with passing radiobutton value to DB. Whats wrong?

if($submit)
{
	$visible	= $_POST['visible'];

$sql = "UPDATE darbas SET visible='$visible' WHERE id='$id'";

$result = mysql_query($sql);

elseif($id)
{
	$sql = "SELECT * FROM darbas WHERE id='$id'";

$result = mysql_query($sql);

while($row = mysql_fetch_assoc($result))
{
	$visible	= $_POST['visible'];

?>

<br><div align="center"><h3>Edit</h3></div>

<form method="post" action="<?php echo $PHP_SELF ?>">
<input type="hidden" name="id" value="<? echo $row['id']?>">
<table border="1" cellspacing="0" cellpadding="0" align="center" class="text">
<tr align="center">
<td width="120"><b>Visible</b></td>
<td width="300" align="left" valign="top">
      Yes<input name="<? echo $visible; ?>" type="radio" value="yes">
      No<input name="<? echo $visible; ?>" type="radio" value="no">
    </td>
    </tr>
</table>
<div align="center"><input type="submit" name="submit" value="Koreguoti"></div>

</form>

<?
}
}
?>

Thanks.

Weedpacket

if($submit)

Perhaps change this to if(isset($_POST['submit']))?

And do the same for $id as well.

And you might want to change $PHP_SELF to $_SERVER['PHP_SELF']

albynas

Hi,

thanks for replay, but I think the problem might be in these lines:

Yes<input name="<? echo $visible; ?>" type="radio" value="yes">
No<input name="<? echo $visible; ?>" type="radio" value="no">

maxpup979

in debugging this kind of error, all you need to do is see where you are losing the value. use print or echos to show you the value of the radio buttons at various points. Also check that the name of the radio buttons is coming through appropriately (view source is your friend).

If you think that the problem is with those lines, then try this instead:

Yes<input name="<?php echo $_POST['visible']; ?>" type="radio" value="yes">
No<input name="<?php echo $_POST['visible']; ?>" type="radio" value="no">

albynas

Thank u for ur replay. It realy helped. I view source and see that visible name is empty. Now Im going to find out why. Thanks.

Weedpacket

Have a look at the FAQ in the Newbies forum - question one.

albynas

I resolved this problem by this:

Yes<input name="visible" type="radio" value="yes">

Thanks again