Isset for radio box

Clarkinator

What I need to do is on submit it does:

if($_POST['looksub']){
//Looksub is the ID of the Submit button
	mysql_connect($host, $user, $pass) or die("Cannot connect: ".mysql_error());
if ( isset($_POST["look1"]) ) 
{
//some stuff
}elsif ( isset($_POST["look2"]) )
{
//some other stuff
}

look1 and look2 are radio boxes.

This should mean that on Submit it checks to see if the radio box look1 is selected and if it is then it should do what's in that bit but it doesn't work. I feel that the problem is the isset.

bradgrafelman

What does the HTML for the radio boxes look like?

Clarkinator

<label>
                  <input name="lookuprdo" type="radio" id="charlook" value="radio" checked="checked" />
                  Character</label>
                <br />
                <label>
                  <input type="radio" name="lookuprdo" value="radio" id="acctlook" />
                  Account</label>
                <br />
                <label>
                  <input type="radio" name="lookuprdo" value="radio" id="iplook" />
                  IP</label>

The ID's are different from the first post because it was something that got complicated. But there are 3 of them and I think I can do what I need within the IF statements if I know what I need to have them as.

bradgrafelman

If the radio buttons have a name of lookuprdo, then why are you checking $POST['look1'] and $POST['look2'] ?

Clarkinator

It was the name that I had to cross check not the ID? Bugger.

bradgrafelman

Clarkinator wrote:
It was the name that I had to cross check not the ID?

Well yes, since the 'id' attribute of an HTML form element isn't included when submitting the form. :p

Also... why do you give all 3 of the radios above the same value? If you do that, there's no way to tell which radio button was actually selected...

Clarkinator

The name is because its this that means I can't select more than 1. Without it, I can select them all.

bradgrafelman

clarkinator wrote:
Without it, I can select them all.

If you want the user to be able to check multiple values instead of either one or none, then you shouldn't be using radio buttons at all - that's what checkboxes are for.

Clarkinator

No, I want them to only be able to select 1 which is what the name does. If the name is different I can select more than 1.

dagon

Clarkinator;10956660 wrote:
No, I want them to only be able to select 1 which is what the name does. If the name is different I can select more than 1.

bradgrafelman said value not name

your correct about use of name, but value is what will be submitted, so if they are all the same you will not be able to tell which radio button was actully selected.