Dos funciones on submit

almasysm · #1 (**permalink**) 03/11/2010, 12:23

Hola
He estado creando un formulario con dos validaciones, una de los campos obligatorios a rellenar y la otra una casilla de aceptación de las normas. Pero no consigo que funcionen las dos, me ignora una de ellas aunque si me lanza el mensaje de alerta (la de DoTheCheck).
Os agradecería una ayudita
Gracias
Os dejo el código

<script type="text/javascript">
function MM_validateForm() { //v4.0
if (document.getElementById){
var i,p,q,nm,test,num,min,max,errors='',args=MM_valida teForm.arguments;
for (i=0; i<(args.length-2); i+=3) { test=args[i+2]; val=document.getElementById(args[i]);
if (val) { nm=val.name; if ((val=val.value)!="") {
if (test.indexOf('isEmail')!=-1) { p=val.indexOf('@');
if (p<1 || p==(val.length-1)) errors+='- '+nm+' must contain an e-mail address.\n';
} else if (test!='R') { num = parseFloat(val);
if (isNaN(val)) errors+='- '+nm+' must contain a number.\n';
if (test.indexOf('inRange') != -1) { p=test.indexOf(':');
min=test.substring(8,p); max=test.substring(p+1);
if (num<min || max<num) errors+='- '+nm+' must contain a number between '+min+' and '+max+'.\n';
} } } else if (test.charAt(0) == 'R') errors += '- '+nm+' is required.\n'; }
} if (errors) alert('The following error(s) occurred:\n'+errors);
document.MM_returnValue = (errors == '');
} }

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>
<style type="text/css">
body {
background-image: url(fondo-formulario3.jpg);
}
</style>
</head>

<body>

<form action="formulario.php" method="post" name="eco" id="eco" onsubmit="MM_validateForm('Nombre','','R','Apellid os','','R','DNI-NIF','','R','tele1','','R','sms','','R','Email','' ,'R','Dirección','','R','Población', '','R','cp','','R','Alergias','','R','enf','','R') ;return document.MM_returnValue" >

_cronos2 · #2 (**permalink**) 03/11/2010, 13:25

Después de un return no se sigue ejecutando el código, la solución es usar un único return pero uniendo los dos valores. Es decir return a&&b, así si alguno de los dos es false devolverá false. La función doTheCheck podría optimizarse así:

Código Javascript:

Ver originalfunction DoTheCheck(x) {
 x?void(0):alert('DEBES ACEPTAR LAS NORMAS DEL CLUB');
 return x;
 /*
  No sirve de mucho darle el foco a un checkbox
  a no ser que el formulario sea muy largo,
  al menos en mi opinión.
 */
}

Código HTML:

Ver original<form action="formulario.php" method="post" name="eco" id="eco" onsubmit="return MM_validateForm('Nombre','','R','Apellid os','','R','DNI-NIF','','R','tele1','','R','sms','','R','Email','' ,'R','Direcci&oacute;n','','R','Poblaci&oacute;n', '','R','cp','','R','Alergias','','R','enf','','R')&&DoTheCheck(this.elements['normas'].checked);" >

Saludos (:

almasysm · #3 (**permalink**) 03/11/2010, 16:14

No me funciona, me dice que los campos de texto no están rellenados y luego se envia vacio.
Gracias de todos modos por tu atención

_cronos2 · #4 (**permalink**) 03/11/2010, 16:33

Ok esque pensé que la función MM_... tenía un return. Entonces prueba cambiando toda la función del return por document.MM_returnValue.
Saludos (: