Radio Button validation

I've had no luck applying client valitation to radio buttons. Pleas look at the attached and point me in the right direction.

Also where is the documentation for applying client validation to radio buttons. THis is not obvious.

Thanks
Nick

I went back and looked at the validation via the wizard and it has changed teh membership validation to the first name agian!

No predicting what willbe written!

when working with a radio group, the list of elements will have a main option for radio group:
WAATKRegistrationForm.memb_typ_us

and an indexed option for each element in the radio group:
WAATKRegistrationForm.memb_typ_us[0]
WAATKRegistrationForm.memb_typ_us[1]


the index at the end refers to specific radio buttons in the group. you should apply the validation to the main radio group:
WAATKRegistrationForm.memb_typ_us

In the code you provided, i do not see any validation applied to the radio group, only the "first_usr", "last_usr" and "email_usr" elements have validation applied.

Try this one please: BM_RegistrationWEBASSIST.php
Also where is the the radio button validation detailed in the documentation?

Thanks

Nick

It looks like there is a bug with the radio group value lookup code, i have looged this in our bugbase


look for the following code:

else if (inputType == "radiogroup")  {
	  alert(formElement.length)
    for (var x=0; x<formElement.length; x++)  {
      if (formElement[x].checked && formElement[x].value!="")  {
        value = formElement[x].value;
        break;
      }
    }
  }

and change it to:

else if (inputType == "radiogroup")  {
    if (formElement.length)  {
      for (var x=0; x<formElement.length; x++)  {
        if (formElement[x].checked && formElement[x].value!="")  {
          value = formElement[x].value;
          break;
        }
      }
    }
    else if (formElement.checked)
      value = formElement.value;
  }

BM_RegistrationWEBASSIST.php

Now works if you choose the first button but not the other two! I noticed there is are two entries for the group:

else if (inputType == "radio") {
if (formElement.length) {
for (var x=0; x<formElement.length; x++) {
if (formElement[x].checked && formElement[x].value!="") {
value = formElement[x].value;
break;
}
}
}
else if (formElement.checked)
value = formElement.value;
}
else if (inputType == "radiogroup") {
if (formElement.length) {
for (var x=0; x<formElement.length; x++) {
if (formElement[x].checked && formElement[x].value!="") {
value = formElement[x].value;
break;
}
}
}
else if (formElement.checked)
value = formElement.value;
}

OK, put that code back to what it used to be, I think I have the correct fix.


find the following line:

if (formElement.length && inputType.toLowerCase()!="radio" && inputType.toLowerCase()!="select") formElement=formElement[0];

and change it to:

if (formElement.length && inputType.toLowerCase()!="radio" && inputType.toLowerCase()!="select") formElement=formElement[0];

I am logging this as a bug to corrected in a future version.

Sorry but the code is the same as produced but the wizard. Are you sure you entered the edited code?

Nick

sorry, the edited code should be:

if (formElement.length && inputType.toLowerCase()!="radiogroup" && inputType.toLowerCase()!="select") formElement=formElement[0];

Now we're talking val-i-dation!

Thanks!

Nick