intanto ti invito a postare sempre l' html-tipo risultante da operazioni lato server,
perche' non e' sempre scontato che, specialmente in questo forum, tutti capiscano asp o php

poi ti invito a postare il codice tra tag [ code] [ /code] o [ php][ /php] (ovviamente senza gli spazi dopo [) che mantiene l' indentazione e facilita la lettura

poi, se non c'e' piu' di un elemento con lo stesso nome non si tratta di un array, quindi non ha length
e mi sembra tu comunque non esegua il controllo se sia segnato o meno in quel caso
codice:
...
var bottone = document.mod.scelta;
if(!bottone.length){
  if(bottone.checked) controllo = true;
}
else ...
ciao