Problemas con el checkbox de un formulario

[almasysm]

Hola a todos, como soy nuevo aquí, perdonad mis posibles fallos

Estoy haciendo un formulario que se envia con php, dentro del formulario hay una casilla de verificación. He conseguido que aparezca un mensaje de aviso en caso de no estar activada pero haga lo que haga al dar a aceptar me envia igualmente el formulario. Os remito parte del código, gracias de antemano.


<script>
function DoTheCheck() {
if(document.econieve.normas.checked == false);
{ alert('DEBES ACEPTAR LAS NORMAS DEL CLUB');
document.econieve.normas.focus()
return false
}
}

</script>

<form action="formulario_eco.php" method="post" name="econieve" id="econieve" onsubmit="DoTheCheck()" >

[Perr0]

mirando rapidamente
me parece que le debes anteponer return al evento onsubmit

<form action="formulario_eco.php" method="post" name="econieve" id="econieve" onsubmit="return DoTheCheck()" >

salu2

[almasysm]

No me ha servido, lo que cambia es que este o no este activada la casilla sale el mensaje y no se envia el correo.
Gracias de todos modos

[Perr0]

ahora si

PD: no ocupes los ; para terminar, no son necesario salvo quieras enredar el codigo y ordenarlo de forma horizontal, el return que te dije si va de todos modos

Código HTML:

<script>
function DoTheCheck(x) {
	var x=document.econieve.normas.checked
	if(x == false)
	{
		document.econieve.normas.focus()
		alert('DEBES ACEPTAR LAS NORMAS DEL CLUB')
		return false
	}
}

</script>

<form action="formulario_eco.php" method="post" name="econieve" id="econieve" onsubmit="return DoTheCheck()" >

  <p>
  <input name="normas" id="normas" type="checkbox" value="">
  Acepto</p>
  <p>
    <input name="Enviar" type="submit" value="Enviar" id="Enviar">
  </p>
</form> 

salu2

[_cronos2]

Hola @Perr0, yo diría que el código que has puesto y la explicación que has dado no tienen sentido. Primero, los puntos y coma se deben poner igual, no son sólo para cuando el código está horizontal. Además, los puntos y coma no suelen ser el problema a la hora de ejecutar código, y si lo son es precisamente porque faltan, no porque sobran. Por último, a una función que se le pasa un argumento hay que llamarla con un argumento, además no sé si se puede declarar una variable que se llame igual que el argumento.
Saludos (:

[almasysm]

He probado la solucion de Perr0 y a simple vista funciona!!!
Muchísimas gracias Perr0, soy un recién iniciado en estas lides (hoy de echo) y estaba desesperado.

[Perr0]

_cronos2 te recalco en rojo el punto y coma (;) que arrojaba problema para mi


<script>
function DoTheCheck() {
if(document.econieve.normas.checked == false);
{ alert('DEBES ACEPTAR LAS NORMAS DEL CLUB');
document.econieve.normas.focus()
return false
}
}

</script>

ademas hay que agregar el return en el evento onsubmit

salu2

[_cronos2]

Esque ahí sí sobra porque aunque la línea acabe ahí despues del if debe ir una llave, a no ser que el código se ejecute en una línea, pero en todo caso después del paréntesis nunca va ; , pero los demás hay que ponerlos igual. Y lo del return estoy de acuerdo.
Saludos :D