Hola energy182:
Trataré de explicarte...
A primera vista hay un pequeño problema de coherencia...
Fíjate que la función tiene unparámetro que no se usa. Aunque no debería influír, deberías evitar ese tipo de incoherencias.
Para ese tipo de función puedes poner como parámetro el "value", y se te simplificarían las cosas...
<SELECT id=select1 onChange="formHandler(this.value)"...>
(El resto del select no debería cambiarse.
Luego la función sería mucho más sencilla...
function formHandler(sitio){
//var valor= document.form.site.value
if (sitio == "Seleccione uno...") {
alert("Atención. El valor introducido no es correcto");
return (false);
}
else{
window.location.href = sitio;
return (true);
}
}
No sé si los return false / true son útiles, pero me parecen innecesarios...
Si necesitas una mejor explicación, no dudes en comentarlo.
Saludos
