Al cambiar el nombre de la función no responde

[ferrins]

Hola!

Que tal? Bueno pues yo tengo un poco de lio, no entiendo porque me pasa lo siguiente, tengo un formulario que cuando le das a enviar comprueba si todos los campos estan llenos, todo funciona perfectamente, pero me he dado cuenta que si cambio el nombre de la función en el formulario y también en el archivo adjunto donde tengo el codigo, entonces no me valida el formulario!

En el formulario llamo a la función con un onSubmit="return checkarts()" y en el archivo adjunto .js la función es: function checkarts(){} Pero cuando cambio el nombre en ambas partes no funciona...

Que os parece? Es esto normal?

Gracias!!

[caricatos]

Hola:

Por lo que dices, debería funcionar, pero veo que no llamas a la función con el formulario como parámetro... no habrás cambiado también otra cosa como el nombre del formulario o el órden...

Con los datos que das es muy difícil ayudar...

Saludos

[ferrins]

Os pongo el código a ver que os parece:

La función:

function checkarts() {
var f1 = document.forms[0];
var wm = "Us heu oblidat les següents dades:\n\r\n";
var noerror = 1;
var t1 = f1.titol;
if (t1.value == "" || t1.value == " ") {
wm += "Manca el títol de l\'article\r\n";
noerror = 0;
}
var t1 = f1.cos_article;
if (t1.value == "" || t1.value == " ") {
wm += "Manca el text de l\'article\r\n";
noerror = 0;
}
var t1 = f1.nom_seccio;
if (t1.value == "" || t1.value == " ") {
wm += "Manca la secció de l\'article\r\n";
noerror = 0;
}
if (noerror == 0) {
alert(wm);
return false;
}
else return true;
}

Y ahora el formulario:

<form method="post" action="insert_art.php" enctype="text/plain" onSubmit="return checkarts()">
<fieldset>
<label>Títol</label>
<input type="text" class="title" name="titol" maxlength="75" size="48"/><br>
<label>Secció</label> <? select_sec_art(); ?><br>
<div style="clear: both; "><label>Text article</label>
<textarea class="body" name="cos_article" rows="10" cols="60"></textarea></div>
<div style="clear:both; "><label>&nbsp;</label><input type="submit" name="Insertar" value="Insertar"></div>
</fieldset></form>

[caricatos]

Hola:

Miré unas pocas líneas y veo unas cosillas que podrías mejorar...
Al formulario es preferible ponerle un nombre para referenciarlo y no usar un índice y en las validaciones es mejor pasar como parámetro el propio formulario: onsubmit="return checkarts(this)" usando this como parámetro (si tuvieras más formularios podrías tener problemas) y la definición sería simplificada: function checkarts(f1)

La verdad es que con códigos "mezclados con otros lenguajes" son difícil de interpretar.

Saludos