checkbox script works in IE, but not in Firefox

[uremog]

I have a set of of check boxes. onClick, the checkboxes call check_radio and recup_checkbox. the referenced radios function as group selectors. check_radio just unchecks the radios if someone manually selects checkboxes.

i think the problem is in the add_selection and del_selection as suggested by the alerts, which are for debugging only. it gets as far as the alert(add/del selection) alerts in FF. all the alert tests pass in IE.

as the title says, this works fine in IE, but fails to pass InputCheck in Firefox, which checks if the search term (enam) is not empty and if the selection of checkboxes are not empty (the whole set of radios and checkboxes are under the "selection" form). selection.value always returns "" (empty string).

could anyone help me with how to change this to make it FF friendly?

Note:i took out some of the functions that i didn:t mention to make this shorter. this is all used with database info, so the number of checkboxes is variable. the ****s remind me whats a test script

[HTML]
<SCRIPT type="text/javascript">
<!-- <![CDATA[
function recup_checkbox( checkbox)
{
alert("recup_ch eckbox"); //******
// This function is used to fill the differents forms with the
// office selection
sel=checkbox.va lue;
if (checkbox.check ed==true) //if the checkbox is checked then we add selection
{
add_selection(' form1','selecti on',sel);
}
if (checkbox.check ed==false) //else we delete selection
{
del_selection(' form1','selecti on',sel)
}
}

function add_selection(f orm_name,input_ name,elet)
{
alert("add selection"); //******
//this function add office code in the customer list
//form_name is a name of a form we want to change
//input_name is the name of the input value we want to change
//elet is the element to add
//the list is formatted to be used in a SQL request
selected_custom er=document.for ms(form_name).e lements(input_n ame).value
if (document.forms (form_name)!=nu ll)
{
alert("document .forms name != null"); //******
if (document.forms (form_name).ele ments(input_nam e).value=="") //if the list is empty, no need to add "-"
{
selected_custom er = elet;
alert("adding selection1:"+do cument.form1.se lection.value); //******
}
else
{
selected_custom er = "-" + elet;
alert("adding selection2:"+do cument.form1.se lection.value); //******
}
document.forms( form_name).elem ents(input_name ).value += selected_custom er;
}
}

function del_selection(f orm_name,input_ name,elet)
{
alert("del selection"); //******
//this function delete office code in the customer list
//form_name is a name of a form we want to change
//input_name is the name of the input value we want to change
//checkbox is a checkbox object
//the list is formatted to be used in a SQL request
if (document.forms (form_name)!=nu ll)
{
alert("document .forms name != null"); //******
search = elet;
selected_custom er = document.forms( form_name).elem ents(input_name ).value;
selected_custom er=selected_cus tomer.replace(s earch,'');
selected_custom er = selected_custom er.replace('--','-');
if (selected_custo mer.substr(sele cted_customer.l ength-1,selected_cust omer.length)=="-")
{
selected_custom er=selected_cus tomer.substr(0, selected_custom er.length-1);
alert("removing selection1:"+do cument.form1.se lection.value); //******
}
if (selected_custo mer.substr(0,1) =="-")
{
selected_custom er=selected_cus tomer.substr(1, selected_custom er.length-1);
alert("removing selection2:"+do cument.form1.se lection.value); //******
}
document.forms( form_name).elem ents(input_name ).value = selected_custom er;
}
}

function check_radio(obj Chkbx)
{
alert("removing all ji selection"); //******
for (i=0;i<document .form1.ji_selec tion.length;i++ )
{
document.form1. ji_selection[i].checked=false
}
}

function Input_Check(ena m)
{
if ((enam == "") && (document.form1 .selection.valu e==""))
{
alert("Enter model number and select office(s) !");
alert("selectio n value:"+documen t.form1.selecti on.value);
return false;
}
if ((enam != "") && (document.form1 .selection.valu e==""))
{
alert("You must select office(s) !");
alert("selectio n value:"+documen t.form1.selecti on.value);
return false;
}
if ((enam == "") && (document.form1 .selection.valu e!=""))
{
alert("Enter model number!");
alert("selectio n value:"+documen t.form1.selecti on.value);
return false;
}
if ((enam != "") && (document.form1 .selection.valu e!=""))
{
ret=afCheck(doc ument.form1.ena m);
return true;
}

}
// ]]> -->
</SCRIPT>
[/HTML]

[acoder]

What errors do you see? Check the error console.

[uremog]

thank you. i'm really not an html person, but i do use java, so with the error console i think i:ll be able to fix everything.

edit: and indeed i got it. whoever wrote the original was making bad calls to elements[] and form[], using elements() and form(), which apparently pass in IE, but firefox (and probably other browsers) are too strict for that.

thanks for the help.

[acoder]

thank you. i'm really not an html person, but i do use java, so with the error console i think i:ll be able to fix everything.

edit: and indeed i got it. whoever wrote the original was making bad calls to elements[] and form[], using elements() and form(), which apparently pass in IE, but firefox (and probably other browsers) are too strict for that.

thanks for the help.

Glad you got it. Yet another quirk from IE which is why you (not you, uremog, but just generally) should never test in IE first.