form e controllo - oggetto è nullo o non è oggetto

[dreaman74]

Ho un form con metodo 'post'
al submit assegno l'evento 'onClick' richiamando la funzione 'inviaOrdine'
Mi resituisce il seguente errore:
'nome.value' è nullo o non è un oggetto

function inviaOrdine(controllo){
var controlloNome = controllo.nome.value;

if (controlloNome == ""){
alert('Devi inserire il nome!');
controllo.nome.focus();
return false;
}
else {
	var rv
	rv = confirm('Cliccando su OK invii l\'ordine definitivamente!');
	if (rv) controllo.form.action=('invioOrdine.asp');
	return rv;
	}
}

[Andrea1979]

Bhè, che ti aspetti? Che dovrebbe essere "nome" e che dovrebbe essere "controllo"?

[dreaman74]

Scusami, non mastico bene javascript, cosa intendi?

[dreaman74]

TRamite il submit richiamo la funzione:

<input type="submit" name="ordineok" value="INVIA ORDINE" class="pulsanteInviaOrdine" onClick="return inviaOrdine(this);">

[Andrea1979]

facciam prima: posta il form

[Andrea1979]

ok, quel "this" sta a significare "pulsante di submit", ergo la funzione chiama "controllo" il tuo pulsante di submit e prova a recuperare un fatidico attributo chiamato "nome" del pulsante di submit che chiaramente non esiste.... che cosa vorresti ottenere?

[dreaman74]

Premetto che il form è dinamico, quindi lo creo tramite un linguaggio server side "ASP".
Ora ti riscrivo il form, omettendo le parti dinamiche...

<form id="carrello" method="post" class="form">
... parte server side
'AVVISO SE L'UTENTE NON HA EFFETTUATO IL LOGIN
'tramite messaggio
...
NOME E COGNOME <input type="text" name="nome">
VIA <input type="text" name="via"
CAP <input type="text" name="cap">
PAESE <input type="text" name="paese">
PROVINCIA
<select name="provincia">
...
</select>
TELEFONO DI CONTATTO <input type="text" name="telefono">
EMAIL <input type="text" name="email"> (facoltativa)
INDICAZIONI VARIE
<textarea name="indicazioni"></textarea>
<input type="submit" name="ordineok" value=""INVIA ORDINE" onClick="return inviaOrdine(this);">
</form>

E? stato semplificato al massimo omettendo anche i tag di impaginazione.

[br1]

I controlli sull'invio del form vanno effettuati sull'evento onsubmit associato al form stesso e non sull'onclick del pulsante submit.

<form ... onsubmit="return controlli(this)">
...
<input type="submit">
</form>

ciao

Edit: se si adotta la sintassi indicata da Andrea1979 il form verra' inviato qualunque sia l'esito dei controlli, dato che il valore di ritorno della funzione non influisce sul flusso degli eventi.

[Andrea1979]

Hai due opzioni: lasci la chiamata alla funzione javascript all'onclick del submit, e modifichi con:

onclick="return inviaOrdine(this.form);"

Oppure la metti all'onsubmit del tag form e la lasci così com'è:

<form id="carrello" method="post" class="form" onsubmit="return inviaOrdine(this);">

Così siamo sicuri che stiamo lavorando sul form nella funzione javascript e non su un bottone o altro elemento.

A questo punto dovrebbe funzionare, visto che controllo sarà il riferimento al form e il form contiene effettivamente un input di nome "nome" etc etc

[dreaman74]

Credo che questa soluzione sia ideale:
onclick="return inviaOrdine(this.form);"


infatti ho delle select "sensibili" all'onSubmit, per il refresh automatico del form: sarebbe un pò dannoso fare il controllo ad ogni submit, preferisco fare il controllo solo all'onClick del pulsante "invia ordine".

Può andare bene?