ripeto che sembrerebbe corretto, a meno che non ci siano errori nell'html (tipo che hai nominato due elementi allo stesso modo
al limite fai una cosa piu' snella che ti evita di ciclare tutti quei campi che non devi controllare
codice:
function successivo (modulo) {
var ok=true;
var elementi_da_controllare = new Array("LettoDatiPersonali", "SottoscrittoDP", "NoteTrasparenza", "FoglioInformativo");
for (i=0; i<elementi_da_controllare.length; i++) {
var elemento=document.forms[modulo].elements[elementi_da_controllare[i]];
if (elemento.checked==false) {
elemento.style.backgroundColor="#ecf1f6";
ok=false;
} else {
elemento.style.backgroundColor="#ffffff";
}
}
if (ok==false) {
alert("Controllare i campi evidenziati");
}else{
document.forms[modulo].submit();
}
}