Monday, May 20, 2024
76
rated 0 times [  77] [ 1]  / answers: 1 / hits: 119896  / 13 Years ago, wed, march 16, 2011, 12:00:00

This one is driving me nuts. It’s got to be something simple and stupid that I am overlooking.
I have a multiple select box in a form. I am just trying to get the values that are selected. In my loop, if I use alert then I have no problem. As soon as try to concatenate the values I get the error ‘SelBranch[...].selected' is null or not an object



      <form name=InventoryList method=post action=InventoryList.asp>
<select name=SelBranch class=bnotes size=5 multiple=multiple>
<option value=All>All</option>
<option value=001 Renton>001 Renton</option>
<option value=002 Spokane>002 Spokane</option>
<option value=003 Missoula>003 Missoula</option>
<option value=004 Chehalis>004 Chehalis</option>
<option value=005 Portland>005 Portland</option>
<option value=006 Anchorage>006 Anchorage</option>
<option value=018 PDC>018 PDC</option>
</select>

<input type=button name=ViewReport value=View class=bnotes onclick=GetInventory();>

</form>


<script language=JavaScript>
function GetInventory()
{
var InvForm = document.forms.InventoryList;
var SelBranchVal = ;
var x = 0;

for (x=0;x<=InvForm.SelBranch.length;x++)
{
if (InvForm.SelBranch[x].selected)
{
//alert(InvForm.SelBranch[x].value);
SelBranchVal = SelBranchVal + , + InvForm.SelBranch[x].value;
}
}
alert(SelBranchVal);
}


</script>

More From » drop-down-menu

 Answers
11

The for loop is getting one extra run. Change



for (x=0;x<=InvForm.SelBranch.length;x++)


to



for (x=0; x < InvForm.SelBranch.length; x++)

[#93242] Tuesday, March 15, 2011, 13 Years  [reply] [flag answer]
Only authorized users can answer the question. Please sign in first, or register a free account.
kurtisl

Total Points: 559
Total Questions: 110
Total Answers: 97

Location: Tokelau
Member since Sun, May 7, 2023
1 Year ago
;