Problema con if y evento onsubmit de javascript

AleXei22 · #1 (**permalink**) 28/07/2010, 14:12

Hola saludos a todos: tengo un pequeño problema con un formulario y un codigo Javascript, lo que quiero hacer es en un pequeño formulario ingresar un codigo y si el codigo es correcto que redirija a otra pagina web y si el codigo es incorrecto que envie un mensaje de error, este es el codigo que he hecho pero no me funciona, alguien me podria ayudar y chequear que es lo que le falta al codigo, muchas gracias

Código:

<html>
<head>
<title>Ubicacion</title>
</head>

<body><center>
<table width="27%" border="1">
  <tr>
    <td>BETA</td>
  </tr>
  <tr>
    <td><form onSubmit="CheckForm()" id="ubicacion" name="ubicacion" method="post">
    <label>codigo: 
    	<input name="codigo" type="text" maxlength="5" />
    </label>
          <input type="submit" name="aceptar" id="aceptar" value="Aceptar" />
<Script language="javascript">
function CheckForm()
	{   
		if (ubicacion.codigo_chofer.value == "0001") {
			ubicacion.action="ubicacion01.html"
				} else {
			alert('Ha introducido un codigo que no existe, vuelva a intentar');
		}
	}
</Script>
    </form></td>
  </tr>
</table>
</center></body>
</html>

zerokilled · #2 (**permalink**) 28/07/2010, 14:21

note la diferencia. estas haciendo referencia a un elemento que no existe. al no existir e intentar leer una propiedad, javascript genera error y se detiene la ejecuccion de javascript. pero este error no se nota porque el control es devuelto al navegador, en este caso para enviar el formulario.

Código:

...
    	<input name="codigo" type="text" maxlength="5" />

...
		if (ubicacion.codigo_chofer.value == "0001") {

_cronos2 · #3 (**permalink**) 28/07/2010, 14:30

Otro detalle es que si no se inserta 0001, el alert sale, pero el formulario se envía igual; debes poner 'return false;'
Saludos (:

AleXei22 · #4 (**permalink**) 28/07/2010, 14:52

Gracias zerokilled no me habia percatado que el los nombres no eran iguales, muchas gracias por decirlo y tambien gracias a _cronos2 por la sugerencia del return false