funzione checkall non funziona con mozilla

[marcio_hack]

Perchè questa funzione per checkare tutti i checkbox cliccando su uno solo di nome checkall_box:

function checkall() {
//checks all the checkboxes
for (i = 0; i < pvt_form.elements.length; i++) {

var e = pvt_form.elements[i];
e.checked = pvt_form.elements.checkall_box.checked;

	}
}

non funziona con mozilla firefox e come posso renderla compatibile?

[Mich_]

La sintassi e` sbagliata.

Ecco quella corretta:

function checkall() {
//checks all the checkboxes
  for (i = 0; i < document.pvt_form.elements.length; i++) {

    var e = document.pvt_form.elements[i];
    e.checked = document.pvt_form.elements.checkall_box.checked;
  }
}

[var]

fai così:

function checkall(f) {
 var e=f.elements;
 for (var i=0; i<e.length; i++)
  if (e[i].type=="checkbox" && e[i].name!="checkall_box") e[i].checked=f.checkall_box.checked;
}

da richiamare passando un riferimento al form (checkall(this.form)).

ciao

[marcio_hack]

Originariamente inviato da var
fai così:

function checkall(f) {
 var e=f.elements;
 for (var i=0; i<e.length; i++)
  if (e[i].type=="checkbox" && e[i].name!="checkall_box") e[i].checked=f.checkall_box.checked;
}

da richiamare passando un riferimento al form (checkall(this.form)).

ciao

grazie,ma che cambia in sostanza che dovrebbe farlo funzionare con mozilla?

[Mich_]

var ed io abbiamo notato lo stesso errore.
Il nome del form non e` definito come oggetto da solo. Ha necessita` di un document. davanti.
Alcuni browser (ad esempio IE) permettono quegli errori, con questo creando alcuni problemi di compatibilta` nei nomi.

Io ti ho dato una soluzine il piu` simile possibile alla tua; var ha modificato lo script in modo piu` razionale, facendoti passare l'oggetto form alla funzione (vedi modifica della istruzione di chiamata), in modo da aggirare il problema a monte.