Solved

Undefined Value in drop down box countries and states

Posted on 2010-11-17
4
385 Views
Last Modified: 2012-06-21
I am using this script here and am getting an undefined option on the country drop down as the last option. Does anyone know what would be causing this?

http://javascript.internet.com/forms/country-state-drop-down.html

The same thing happens on my script.. it also happens on the original source link, it is the last country option.
0
Comment
Question by:nismor240sx
  • 2
4 Comments
 
LVL 1

Accepted Solution

by:
leehanken earned 175 total points
ID: 34160376
There is an error in the javascript file in the populateCountry function. The line which says:

  for (var loop = 0; loop < countryLineArray.length; loop++) {

should say

  for (var loop = 0; loop < countryLineArray.length-1; loop++) {

otherwise the loop goes on too many times and adds an invalid item to the list of countries.

Just insert the -1 and it will work.
// Populates the country selected with the counties from the country list
function populateCountry(defaultCountry) {
  if ( postCountry != '' ) {
    defaultCountry = postCountry;
  }
  var countryLineArray = country.split('|');  // Split into lines
  var selObj = document.getElementById('countrySelect');
  selObj.options[0] = new Option('Select Country','');
  selObj.selectedIndex = 0;
  for (var loop = 0; loop < countryLineArray.length-1; loop++) {
    lineArray = countryLineArray[loop].split(':');
    countryCode  = TrimString(lineArray[0]);
    countryName  = TrimString(lineArray[1]);
    if ( countryCode != '' ) {
      selObj.options[loop + 1] = new Option(countryName, countryCode);
    }
    if ( defaultCountry == countryCode ) {
      selObj.selectedIndex = loop + 1;
    }
  }
}

Open in new window

0
 
LVL 63

Assisted Solution

by:Zvonko
Zvonko earned 75 total points
ID: 34160395
Remove the last separator pipe character | from the country data string and the empty Option will go away:

ZM:Zambia|\
ZW:Zimbabwe|\
';

Like this:

ZM:Zambia|\
ZW:Zimbabwe';
 
0
 
LVL 1

Author Closing Comment

by:nismor240sx
ID: 34160496
Thank you.
0
 
LVL 1

Author Comment

by:nismor240sx
ID: 34160500
You were both correct, thanks for helping me figure it out. Since my script is dynamic using ASP scripts and a database, the first answer was the easiest one for me to implement.
0

Featured Post

Problems using Powershell and Active Directory?

Managing Active Directory does not always have to be complicated.  If you are spending more time trying instead of doing, then it's time to look at something else. For nearly 20 years, AD admins around the world have used one tool for day-to-day AD management: Hyena. Discover why

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
Browser Chrome downloads colored link 5 41
Help with removing item from Session Array 3 23
Jquery keyup 4 19
Add shadow behind div 5 11
I've been trying to accomplish this for a while and it just struck me yesterday how to accomplish this task. I have done searches all over the internet looking for ways to email pages from my applications and finally I have done it!!! Every single s…
This article demonstrates how to create a simple responsive confirmation dialog with Ok and Cancel buttons using HTML, CSS, jQuery and Promises
The viewer will learn the basics of jQuery, including how to invoke it on a web page. Reference your jQuery libraries: (CODE) Include your new external js/jQuery file: (CODE) Write your first lines of code to setup your site for jQuery.: (CODE)
The viewer will learn the basics of jQuery including how to code hide show and toggles. Reference your jQuery libraries: (CODE) Include your new external js/jQuery file: (CODE) Write your first lines of code to setup your site for jQuery…

778 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question