form php e validità campi

[Traxsung]

Ragazzi sto provando a validare il mio usando javascript, ma sto riscontrando diverse difficolta.

Questo qui di seguito è il mio form php

<form name="modulo" action="registra.php" method="post">
    <table align="center" border="0">
      <tr>
        <td>

Nome e cognome</p>
        

Ragione Sociale:  </p></td>
        <td><input name="NOME" type="text" maxlength="20" /></td>
      </tr>
      <tr>
        <td>

Partita Iva </p>
        

 Codice Fiscale    : </p></td>
        <td><input name="PICF" type="text" maxlength="20" /></td>
      </tr>
      <tr>
        <td>Indirizzo: </td>
        <td><input name="INDIRIZZO" type="text" /></td>
      </tr>
      <tr>
        <td>Telefono: </td>
        <td><input name="TELEFONO" type="text" maxlength="15" /></td>
      </tr>
      <tr>
        <td>Citt&agrave;: </td>
        <td><input name="CITTA" type="text" maxlength="15" /></td>
      </tr>
      <tr>
        <td>Email: </td>
        <td><input name="EMAIL" type="text" /></td>
      </tr>
    
    <tr><td></td><td><input value="Invia Registrazione" type="submit" onClick="Modulo()"/></td></tr>
    </table> 

mentre questo è il codice di validificazione in javascript

<script language="javascript">
 <!--
  function Modulo() {
     var NOME = document.modulo.nome.value;
     var PICF = document.modulo.partitaiva.value;
     var INDIRIZZO = document.modulo.indirizzo.value;
     var TELEFONO = document.modulo.telefono.value;
	 var CITTA = document.modulo.citta.value;
	 var EMAIL = document.modulo.email.value;
     var email_reg_exp = /^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-]{2,})+\.)+([a-zA-Z0-9]{2,})+$/;
	 

        if ((NOME == "") || (NOME == "undefined")) {
           alert("Il campo Nome e Cognome/Ragione Sociale obbligatorio.");
           document.modulo.nome.focus();
           return false;
        }
        else if ((PICF == "") || (NOME == "PICF")) {
          alert("Il campo Partita Iva/Codice Fiscale è obbligatorio.");
           document.modulo.partitaiva.focus();
           return false;
		    }
        else if ((INDIRIZZO == "") || (NOME == "INDIRIZZO")) {
           alert("Il campo Indirizzo è obbligatorio.");
           document.modulo.indirizzo.focus();
           return false;
		    }
        else if ((TELEFONO == "") || (NOME == "TELEFONO")) {
           alert("Il campo Telefono è obbligatorio.");
           document.modulo.telefono.focus();
           return false;
		    }
        else if ((CITTA == "") || (NOME == "CITTA")) {
           alert("Il campo Citta è obbligatorio.");
           document.modulo.citta.focus();
           return false;
        }
        else if ((email == "") || (email == "undefined")) {
           alert("Il campo E-mail è obbligatorio.");
           document.modulo.email.focus();
           return false;
        }
       else if (!email_reg_exp.test(email)) {
           alert("Inserisci un indirizzo e-mail corretto.");
           document.modulo.email.focus();
           return false;
      

        
        }
		document.modulo.submit();
  }
 //-->
</script>

In pratica è come se non vedesse la parte in javascript proseguendo con il form (nel senso che invia i dati con i campi vuoti al database)


Dove sbaglio?


Grazie