Comprobar formulario PHP + JS

[Nietzsche]

Hola, tengo un problema en mi código y no se exactamente cual es.
Se trata de un formulario en HTML + PHP que al presionar el botón "Enviar" (O una imagen con la palabra "Añadir" en este caso, ya que añade coches a una BBDD) se lanza una funcion de comprobacion en JavaScript y si están rellenos los campos obligatorios debería insertar en la BBDD, sino no.

El codigo PHP:

Código PHP:

  ... 
... 
<tr> 
   <td colspan="2">Foto int.:</td> 
   <td colspan="2"><input type="file" name="interior" value=""></td> 
</tr> 
<tr> 
   <td colspan="2">Foto atrás:</td> 
   <td colspan="2"><input type="file" name="trasera" value=""></td> 
</tr> 
<tr> 
   <td colspan="4"><br><input name="enviar" type="image" id="enviar" onclick="comprobar();" title="Añadir coche" src="images/anadir-coche.jpg" alt="Añadir coche"/></td> 
</tr>  
   

El caso es que si alguno de los campos obligatorios no esta rellenado da el error, pero aun así inserta en la BBDD.

Código JS:

Código:

// Funcion que valida los campos obligatorios
function comprobar(){ 
    var mal=false; 
    with (document.forms[0]){ 
     numero = elements.length; 
     for (a = 0; a < numero; a++){ 
      if ((elements[a].name == "sel_marca") || (elements[a].name == "modelo") || (elements[a].name == "precio") || (elements[a].name == "color"))
		{ 
       	if ((elements[a].value == "") || (elements[a].value == "default"))
			{ 
        	elements[a].style.backgroundColor = "#ffffcc"; 
        	mal = true; 
       		} 
		else 
			{ 
        	elements[a].style.backgroundColor = "white"; 
       		} 
      	}
     } 
     if (mal) 
      alert("Por favor, rellene las cajas coloreadas"); 
     else 
      submit(); 
    } 
   }

A ver si alguien me echa una mano.

Gracias y saludos!

[jerkan]

Despues de hacer el alert deberías poner un 'return false;'

[Nietzsche]

Hola,
he cambiado el código como me dices:

Código:

     if (mal) {
      alert("Por favor, rellene las cajas coloreadas"); 
      return false;
      }
     else 
      submit(); 
    }

Y sigue ocurriendo lo mismo

[jerkan]

En el atributo onclick del tag input pon:

Código HTML:

onclick="return comprobar();"

[caricatos]

Hola:

Poner en la función javascript la instrucción submit() fuerza el envío del formulario, y se dejan de evaluar las alternativas.
Los input type="image" son una especie de variante del tipo submit (envían el formulario y añaden en el envío las coordenadas donde se ha pulsado), así que esa comprobación/validación deberías hacerla desde el mismo tag form, devolviendo false para cancelar y true para validar...

<form onsubmit="return comprobar(this)"...

y tu "comprobar" más o menos así:

Código:

if (mal) 
      alert("Por favor, rellene las cajas coloreadas"); 
return !mal;

Por cierto, no veo nada de php

Saludos

[Nietzsche]

Muchas gracias, ya funciona.

Si, la verdad es que este tema deberia ir en Javascript más que en PHP

[caricatos]

Iniciado por Nietzsche

Muchas gracias, ya funciona.

Si, la verdad es que este tema deberia ir en Javascript más que en PHP

Me alegro.

De todas maneras ya lo he reportado, así que en cuanto algún moderador pueda, moverán el mensaje.

Saludos

[jam1138]

Iniciado por caricatos

De todas maneras ya lo he reportado, así que en cuanto algún moderador pueda, moverán el mensaje.

Saludos caricatos .

Hay que ganarse el sueldo... Tema movido desde PHP