Strange Error with JS Function

[oracleguy]

I have this page that uses this same javascript function many times:


function addSpeed(optText,optValue) {

document.Order.SysSpeed.options[document.Order.SysSpeed.options.length] = new Option(optText, optValue);

}


And there is a problem inside this switch statement. The red lines specificly.


switch(document.Order.SysModel.value)
{
case "Vel":

addSpeed('Intel Pentium 4 1.60GHz/400MHz','Intel Pentium 4 1.60GHz/400MHz');
addSpeed('Intel Pentium 4 1.80GHz/400MHz','Intel Pentium 4 1.80GHz/400MHz');
addSpeed('Intel Pentium 4 2.00GHz/400MHz','Intel Pentium 4 2.00GHz/400MHz');

break
case "AdvVel":

addSpeed('Intel Pentium 4 2.00GHz/400MHz','Intel Pentium 4 2.00GHz/400MHz');
addSpeed('Intel Pentium 4 2.20GHz/400MHz','Intel Pentium 4 2.20GHz/400MHz');
addSpeed('Intel Pentium 4 2.40GHz/400MHz','Intel Pentium 4 2.40GHz/400MHz');

break
case "VelVivo":

addSpeed('Intel Pentium 4 2.00GHz/400MHz','Intel Pentium 4 2.00GHz/400MHz');
addSpeed('Intel Pentium 4 2.20GHz/400MHz','Intel Pentium 4 2.20GHz/400MHz');
addSpeed('Intel Pentium 4 2.40GHz/400MHz','Intel Pentium 4 2.40GHz/400MHz');

break
case "VelBus":

addSpeed('Intel Pentium 4 1.60GHz/400MHz','Intel Pentium 4 1.60GHz/400MHz');
addSpeed('Intel Pentium 4 1.80GHz/400MHz','Intel Pentium 4 1.80GHz/400MHz');
addSpeed('Intel Pentium 4 2.00GHz/400MHz','Intel Pentium 4 2.00GHz/400MHz');

break
case "AdvVelBus":

addSpeed('Intel Pentium 4 2.00GHz/400MHz','Intel Pentium 4 2.00GHz/400MHz');
addSpeed('Intel Pentium 4 2.20GHz/400MHz','Intel Pentium 4 2.20GHz/400MHz');
addSpeed('Intel Pentium 4 2.40GHz/400MHz','Intel Pentium 4 2.40GHz/400MHz');
}


Look at this page, http://v6.orangepc.net/order.asp

If you select 'Velocity' as the system line and then select 'ViVo' as the model, the speed is blank, but the speed fills in for all the others.

I use an ASP Function to generate the addSpeed(...) line from a database.

So why is this happening? I can't seem to figure it out.

[ahosang]

It's a typo:
You have another capital V in your addOption function:
addOption('ViVo', 'VelViVo');

but in your switch statement, you have a common v:
case 'VelVivo':