Problema en validacion de formulario

[matiasbmx]

Buenas, he estado haciendo la validacion de un formulario, lo valida, me tira el error pero al darle aceptar (al alert), lo manda igual!..
aca los codes

Código PHP:

  <script> /* Abrimos etiqueta de código */ 
function validar_formulario(){ /* Abrimos la función validar_formulario */ 
if (document.form1.titulo.value.length==0){ 
alert('Debe ingresar un titulo') 
document.form1.titulo.focus() 
} 
if (document.form1.autor.value.length==0){ 
alert('Debe ingresar un autor') 
document.form1.autor.focus() 
} 
if (document.form1.categoria.value.length==0){ 
alert('Debe ingresar una categoria') 
document.form1.categoria.focus() 
} 
if (document.form1.articulo.value.length<100){ 
alert('Debe ingresar almenos 100 caracteres') 
document.form1.articulo.focus() 
Publicar=false 
} 
form1.submit(); 
} 
</script>  
  

Código PHP:

  <form name="form1" action="procesanoticia.php" method="post" onsubmit='return validar_formulario()'>  
Titulo:  
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="text" name="titulo"><br />  
Autor:  
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="text" name="autor"><br /> 
Categoría: 
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="text" name="categoria"> <br /> 
<p>&nbsp;</p>  
Escribe el articulo/noticia:<br>  
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<textarea name="articulo" cols="50" rows="10"></textarea><br /> 
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input type="submit" value="Publicar" /> 
</form>  
   

[caricatos]

Hola:

Tanto   dificulta la lectura del código (innecesariamente)... sobre la validación es una respuesta trivial que debes revisar...

onsubmit='return validar_formulario()...

...validar_formulario() debe devolver false para su cancelación, y no vale ninguna otra cosa... incluso los errores cancelan la supuesta cancelación.

Si fuerzas los envíos (form.submit()), las validaciones no tienen ningún sentido.

Revisa esas cosillas y nos cuentas.

Saludos

[matiasbmx]

ya le saque los   , etntiendo la respuesta del false, pero nose como ponerlo en el code.. aalgo asi como ''onsubmit='return

validar_formulario()=false'' ???
podrias orientarme ?

[caricatos]

Hola:

En la definición del formulario ya tienes el onsubmit="return validar_formulario()", así que tienes que editar esa función para que devuelva (return) false en las cancelaciones.

Uno de los "if" termina con Publicar=false;... eso lo podrías repetir en los restantes if, y como última línea, en vez del odioso submit(), poner return Publicar... que tendrías que inicializar al principio con true.

Cuando lo tengas, avisa que te puedo plantear una mejora (pero vayamos por partes, como dijo Jack)

Saludos

[matiasbmx]

Iniciado por caricatos

Hola:

En la definición del formulario ya tienes el onsubmit="return validar_formulario()", así que tienes que editar esa función para que devuelva (return) false en las cancelaciones.

Uno de los "if" termina con Publicar=false;... eso lo podrías repetir en los restantes if, y como última línea, en vez del odioso submit(), poner return Publicar... que tendrías que inicializar al principio con true.

Cuando lo tengas, avisa que te puedo plantear una mejora (pero vayamos por partes, como dijo Jack)

Saludos

bueno, trate de hacer lo que dices, pero no conozco las funciones, y puede haber un error de escritura, que se yo...
asi quedo:

Código PHP:

  function validar_formulario(){ /* Abrimos la función validar_formulario */ 
if (document.form1.titulo.value.length==0){ 
alert('Debe ingresar un titulo') 
document.form1.titulo.focus() 
Publicar=false; 
} 
if (document.form1.autor.value.length==0){ 
alert('Debe ingresar un autor') 
document.form1.autor.focus() 
Publicar=false; 
} 
if (document.form1.categoria.value.length==0){ 
alert('Debe ingresar una categoria') 
document.form1.categoria.focus() 
Publicar=false; 
} 
if (document.form1.articulo.value.length<100){ 
alert('Debe ingresar almenos 100 caracteres') 
document.form1.articulo.focus() 
Publicar()=false; 
} 
true=form1.Publicar(); 
}  
   

y no funciona, sigue igual

[caricatos]

Hola:

Vamos a ver:

Código:

function validar_formulario(){ /* Abrimos la función validar_formulario */ 
Publicar = true;
if (document.form1.titulo.value.length==0){ 
alert('Debe ingresar un titulo') 
document.form1.titulo.focus() 
Publicar=false; 
} 
if (document.form1.autor.value.length==0){ 
alert('Debe ingresar un autor') 
document.form1.autor.focus() 
Publicar=false; 
} 
if (document.form1.categoria.value.length==0){ 
alert('Debe ingresar una categoria') 
document.form1.categoria.focus() 
Publicar=false; 
} 
if (document.form1.articulo.value.length<100){ 
alert('Debe ingresar almenos 100 caracteres') 
document.form1.articulo.focus() 
Publicar=false; 
} 
//true=form1.Publicar(); 
return Publicar
}

Supongo que así funcionaría... ahora un retoque:

Código:

function validar_formulario(){ /* Abrimos la función validar_formulario */ 
mensajes = new Array();
if (document.form1.titulo.value.length==0){ 
mensajes.pusht('Debe ingresar un titulo') 
document.form1.titulo.focus() 
//Publicar=false; 
} 
if (document.form1.autor.value.length==0){ 
mensajes.push('Debe ingresar un autor') 
document.form1.autor.focus() 
//Publicar=false; 
} 
if (document.form1.categoria.value.length==0){ 
mensajes.push('Debe ingresar una categoria') 
document.form1.categoria.focus() 
//Publicar=false; 
} 
if (document.form1.articulo.value.length<100){ 
mensajes.push('Debe ingresar almenos 100 caracteres') 
document.form1.articulo.focus() 
//Publicar()=false; 
} 
//true=form1.Publicar(); 
if (mensajes.length > 0) alert(mensajes.join("\n"));
return (mensajes.length == 0);
}

No sé si los focos se quedarán bien, pero de esta manera solo se mostrará una alerta si hay fallos de validación.

Saludos

[matiasbmx]

funciona perfecto!
gracias por la ayuda!