Topic: populating a select list

Hi there, I have the following code: ( which does not seem to work... any ideas ? ). I dont seem to get any errors, but yet the form remains unpopulated.

code tp add the item....

document.data.location[location.length] = new Option(OptionText, OptionValue);

form item...

<select id="dd_location2" name="location">
	<option value="0"></option>
	<option value="4" >Other Locations</option>
</select>

I've also tried this way as well, no dice.

document.getElementById('dd_location2').length = new Option(OptionText, OptionValue);

(Edited by protoculture on 10-25-2006 18:30)

Holy moly!!!



(Edited by poi on 10-25-2006 18:48)

Actually I had already done that and led me to the code you see before you.

the question still stands... can anyone see anything obvious in the code I might be missing?

Right.

... but the first code snippet in the first link returned by google works.

an even more generic search leeds straight to Peter-Paul Koch's excellent quirksmode.org

quote:

---

protoculture said:

can anyone see anything obvious in the code I might be missing?

---

Aside from the fact that you are not specifying what to populate the list with?

Check the links posted....lots of good info in those top couple of results.

Of course, if you can be more specific as to *what* you want to automatically populate the list with, you might get more specific answers as well.

Um the first link on the google seach seems to return the same method he has

document.data.location[location.length] = new Option(OptionText, OptionValue);


That shoulds be

document.data.location[document.data.location.length] = new Option(OptionText, OptionValue);


Can't test right now but it seems it should actually be:

document.data.location.options[document.data.location.options.length] = new Option(OptionText, OptionValue);



.:[ Never resist a perfect moment ]:.

Ok, ok the first link on my first google search is a bit incorrect :

I only read their ClearOptions() and AddToOptionList() functions and they are correct. It's how they use that is not. They pass the select itself instead of its options property. Tssss. But hey you can't expect 100% correct JS infos from a site called ASPWatch.

However the code on PPK's site is correct and it shows up in second place on my first search, and first place on my second search. Obviously someone didn't do his homework seriously.


Also, to make sure I'm really anal here, if you read out loud:

document.getElementById('dd_location2').length = new Option(OptionText, OptionValue);

you can see it doesn't make sense. Length equal new option. No, that's not what you want. What you want is to add an option to the collection of options of a select.



ps: can anyone see it touches a nerve when I see that kind of threads ?



(Edited by poi on 10-26-2006 20:48)