Verificar un formulario

[Adler]

Hola, esta pregunta hago es muy sencilla la solución, pero no consigo verla.

La situación es la siguiente:

Un formulario que se sumita usando una función, ya que el botón es una imagen

Código:

function Enviar() {
document.forms.nmsg.action = "./escribirmsg.asp?accion=enviar"
document.forms.nmsg.Submit()
}

hasta ahí todo bien, ahora la verificar el campo, me aparece la alerta, y cuando pincho en Aceptar, envia el formulario

Código:

function Verificar() {
checktema = document.forms.tema.value;
if (checktema.length < 1)
{
alert("Has de escribir un tema para este mensaje");
document.forms.tema.select();
document.forms.tema.focus();
return (false);
}
else
{
return (true);
}
}

Lo que pretendo es que no se envie el formulario, cuando se cumple la condición. ¿Cual es el error?

Gracias

[FNX_NET]

debe ser por que es un submit y no un button


prueba con eso

ademas te falta punto y como en el primer queote

[caricatos]

Hola:

Los input type="image" suelen hacer submit si están bien puestos, y a la vez envían las coordenadas en donde se ha pinchado...

El error puede estar en cualquier parte porque deberíamos ver el resto del código... posiblemente la verificación no esté bien puesta... por lo poco que he visto debería ser así:

<form onsubmit="return Verificar()"...

Te recomiendo que en las verificaciones pongas el formulario como parámetro (creo que es el mejor sistema)

Saludos

[Adler]

Gracias a ambos por vuestras respuestas. El problema estaba que se me olvidó apuntar hacia el formulario

document.forms.nmsg.tema.value;

Me ha quedado una dudar en lo que respondió Caricato

Te recomiendo que en las verificaciones pongas el formulario como parámetro (creo que es el mejor sistema)

¿Qué quieres decir con poner el formulario como parámetro?

Gracias

[caricatos]

Hola:

Las validaciones bien hechas se hacen desde el tag form, provocando en las funciones validadoras que entreguen al formulario los valores true/false (aceptación/cancelación), y si se tienen muchos formularios, con una sola función validadora se pueden validar muchos formularios con esa sencilla premisa...

Un modelo de formulario y función validadora sería así:
<form onsubmit="return validar(this)"...

Si todos los campos tienen un campo "obligatorio" que no pueda estar vacío, con una declaración sencilla, vale para todos:

function validar(f) {
vale = f.obligatorio.value != "";
if (!vale) alert("rellene el campo obligatorio");
return vale;
}

f de la función equivale a "this" de la declaración en el tag form...

Si sigues este consejo evitarás un problema muy recurrente de los webmasters novatos.

Saludos