la validacion se interrumpe si una sentencia if da verdadero

[radvla]

Hola a todos, he decido abrir otro hilo, pues el anterior está muy complicado.

OS pongo la función que estoy usando para validar un formulario con onsubmit.Para probar he forzado a que de siempre falso.

Hay una sóla condición destinada a saber si un campo está vacío.

No se porque cuando la condición IF se cumple la función se interrumpe, se devuelve verdadero a las etiquetas form que la llamaron y se envía el formulario con campos vacíos.



<script language="javascript">
function verificar1() {

alert("estoy en la función");

valor3 = document.getElementById("BIEN_SERVICIO_TRES").valu e;
alert("he recogido el campo TREScon longitud=");
alert(valor3.length);

if( valor3.length==0 ) {
ALERT("EL CAMPO TRES ESTÁ VACIO");
ok=false;
alert(ok);}
else{
ok=true;
alert(ok); }

return false;
}
</script>

a esa función se llama así:

<form id = "cambiodebienes" name="cambiodebienes" action="http://localhost/leccionesphp/vercuadrosdetexto.php" method="GET" onSubmit = "return verificar1();" >


SOCORRRO!!!

Gracias

[PHPeros]

No existe ninguna funcion llamada ALERT

[radvla]

ya l he corregido y lo he puesto en minúsculas:

alert("EL CAMPO TRES ESTÁ VACIO");

se sigue interrumpiendo la función cuando se verifica que el campo 3 es nulo con la condición if( valor3.length==0 ), en ese momento no se dan loa avisos alert(ok) por lo que se que se ha interrumpido sin que haya ningún return true.

Socorro!!!!!!

gracias

[Alexis88]

Solo por curiosidad, ¿la siguiente línea la has escrito del mismo modo en tu código o fue un error de tipeo al elaborar este hilo?:

Código Javascript:

valor3 = document.getElementById("BIEN_SERVICIO_TRES").valu e;
 

Porque así no está bien, debe ser así:

Código Javascript:

valor3 = document.getElementById("BIEN_SERVICIO_TRES").value;
 

Pero muy aparte de eso, creo que te estás complicando con algo muy sencillo, bastaría con que lo hagas así:

Código Javascript:

function verificar1() {
    var valor3 = document.getElementById("BIEN_SERVICIO_TRES").value;
 
    if(!valor3.length) {
        alert("EL CAMPO TRES ESTÁ VACIO");
        return false;
    }
    else{
        return true;
    }
}
 

Quité las demás alertas porque son un tanto incómodas y no creo que a los usuarios que vayan a utilizar tu aplicación les agrade ver tantas alertas.

Saludos