Valida pero envia igual

Pazosele · #1 (**permalink**) 13/04/2005, 05:27

Tengo una pagina con radiobutons y textboxes.
he usado este codigo de la FAQ, pero envia el formulario independientemente de detectar que haya o no campos sin responder.. alguna idea para que no envie hasta que no se hayan rellenado?

Código PHP:

  function comprobar(){var mal=false;  
    numero=document.forms[0].elements.length;  
    for(a=0;a<numero;a++){  
    if (document.forms[0].elements[a].value=="") {document.forms[0].elements[a].style.backgroundColor="#ffffcc";mal=true;}  
    //else{document.forms[0].elements[a].style.backgroundColor="white";}  esto lo he anulado pq a mi no me hacia falta 
    }  
    if(mal){alert("Por favor, rellene las cajas coloreadas");}  
    else if (mal=false){document.forms[0].submit()}  
    }

El formulario esta escrito asi:

Código PHP:

  <FORM METHOD="post" NAME="Formulario" ID="Formulario" action="javascript:alert('enviando')" > 
 
. 
. 
. 
. 
<INPUT " TYPE="submit" NAME="accion" VALUE="Enviar" onclick="comprobar()">  
</FORM>

Carlitos · #2 (**permalink**) 13/04/2005, 08:26

Hola

Corrige esta línea:

else if (mal=false){document.forms[0].submit()}

por esta:

else {document.forms[0].submit()}

KarlanKas · #3 (**permalink**) 13/04/2005, 08:55

Veo que no solo has quitado lo del color de fondo... es muy feo quitar los créditos en los scripts ajenos...

http://www.forosdelweb.com/showpost....&postcount=123

Pazosele · #4 (**permalink**) 14/04/2005, 05:00

Quitando eso, es decir, tal como esta en la FAQ, tampoco me funciona.. me dice que faltan rellenar campos, peor luego lo envia igual..
ALquien sabe porque?

PD: KARLANKAS, solo he pegado el codigo que podia dar conflicto, peor no sufras que en el codigo entero si que sale tu nombre ;)

(lo puedes ver en otro post que puse antes con el codigo entero)

KarlanKas · #5 (**permalink**) 14/04/2005, 05:13

Para que funcione tienes que cambiar el tipo de input del submit de submit a button.

Código PHP:

  <INPUT TYPE="button" NAME="accion" VALUE="Enviar" onclick="comprobar()">

O si quieres quitar el onclick de ese botón y cambiar un poco el código:

Código PHP:

  function comprobar(){ 
    var mal=false; 
    numero=document.forms[0].elements.length; 
    for(a=0;a<numero;a++){ 
    if (document.forms[0].elements[a].value=="") {document.forms[0].elements[a].style.backgroundColor="#ffffcc";mal=true;} 
    //else{document.forms[0].elements[a].style.backgroundColor="white";}  esto lo he anulado pq a mi no me hacia falta 
    } 
    if(mal){alert("Por favor, rellene las cajas coloreadas");} 
    return !mal 
    }     </script> 
</head> 
 
<body> 
<form action="javascript:alert('enviando')" 
      method="post" 
      name="formulario" 
      id="formulario" 
      style="font:normal 10px/10px verdana;border:solid 1px black;text-align:right;width:300px;" 
      onsubmit="return comprobar()"> 
... 
<INPUT TYPE="submit" NAME="accion" VALUE="Enviar" >

Perdona el comentario de ayer... es que ayer tuve un día un poco malo...

Pazosele · #6 (**permalink**) 14/04/2005, 05:27

Muchas gracias!
Me has ayudado un monton!

Por lo de ayer ni te preocupes ;)
Saludos!

y nos vemos por el foro!
(seguro que pronto tengo alguna preguntilla mas) XD