invia solo se verifica [era: submit]

[Rossella_75]

ciao a tutti,

ho una funzione per controllare i campi, la verifica la fà solo che invia cmq il form mentre non dovrebbe inviarlo!

questa è la funzione

function ValidateForm(){
var f=document.form1;
var campo1 = f.tipo.value;
var campo2 = f.forfait.value;
var campo3 = f.pulizia.value;
var campo4 = f.culla.value;
if(campo1 == ""){
alert("Il campo1 è obbligatorio");
f.campo1.focus();
return false;
}
if(campo2 == ""){
alert("Il campo2 è obbligatorio");
f.campo2.focus();
return false;
}
if(campo3 == ""){
alert("Il campo3 è obbligatorio");
f.campo3.focus();
return false;
}
if(campo4 == ""){
alert("Il campo4 è obbligatorio");
f.campo4.focus();
return false;
}
if(campo5 == ""){
alert("Il campo5 è obbligatorio");
f.campo5.focus();
return false;
}
}

mentre al fomr ho fatto cosi

<form  method="post" name="form1" action "<?php echo $editFormAction; ?>" onSubmit="return ValidateForm();">

:master: Grazie

[cavicchiandrea]

Originariamente inviato da Rossella_75
ciao a tutti,

ho una funzione per controllare i campi, la verifica la fà solo che invia cmq il form mentre non dovrebbe inviarlo!

questa è la funzione

function ValidateForm(){
var f=document.form1;
var campo1 = f.tipo.value;
var campo2 = f.forfait.value;
var campo3 = f.pulizia.value;
var campo4 = f.culla.value;
if(campo1 == ""){
alert("Il campo1 è obbligatorio");
f.campo1.focus();
return false;
}
if(campo2 == ""){
alert("Il campo2 è obbligatorio");
f.campo2.focus();
return false;
}
if(campo3 == ""){
alert("Il campo3 è obbligatorio");
f.campo3.focus();
return false;
}
if(campo4 == ""){
alert("Il campo4 è obbligatorio");
f.campo4.focus();
return false;
}
if(campo5 == ""){
alert("Il campo5 è obbligatorio");
f.campo5.focus();
return false;
}
}

mentre al fomr ho fatto cosi

<form  method="post" name="form1" action "<?php echo $editFormAction; ?>" onSubmit="return ValidateForm();">

:master: Grazie

non puoi mandare il focus() ad una variabile

f.campo4.focus(); :incupito: f.culla.focus() :D

in + var campo5 non esiste ma la verifichi

if(campo5 =="") :master:

[Rossella_75]

infatti scusami avevo sbagliato a postare....