no me valida el dni

[jawir]

Tengo el siguiente formulario y quiero que mire si en el dni hay numero y letras el srcipt del dni lo he cogio del foro, pero ponga lo que ponga en el dni me inserta los datos en la base de datos, porque no me valida?
Gracias de antemano

Código PHP:

  <script type="text/JavaScript">
 
function validar()
{
abc=document.formulario.nif.value
dni=abc.substring(0,abc.length-1)
let=abc.charAt(abc.length-1)
if (!isNaN(let))
 {
  alert('Falta la letra')
  document.formulario.nif.focus()
  return false
 }
else
 {
  cadena="TRWAGMYFPDXBNJZSQVHLCKET"
  posicion = dni % 23
  letra = cadena.substring(posicion,posicion+1)
  if (letra!=let.toUpperCase())
   {
    alert("Nif no válido")
    document.formulario.nif.focus()
    return false
   }
 }
alert("Nif válido")
}
 
</script> 
  

Código PHP:

  <form action="php/registrocliente.php" method="POST" name="formulario">
    <div align="center"><strong>ALTA NUEVO CLIENTE <br>    
    <br>
    </strong>
    </div>
    <table width="379" height="125" border="2" align="center" bordercolor="#0000FF" bgcolor="#FFFFCC" style="border:2px solid #000000;">
 
 
<tr>
<td align="left"> <span class="Estilo2">DNI:</span>
<input type="text" size="10" maxlength="10"name="dni" />
</td>
</tr>
 
 
<tr>
<td align="left"> <span class="Estilo2">Nombre del Cliente:</span>
<input type="text" size="30" maxlength="30"name="nombre" />
</td>
</tr>
 
<tr>
<td align="left"> <span class="Estilo2">Apellidos:</span>
<input type="text" size="30" maxlength="30"name="apellidos" />
</td>
</tr>
 
<tr>
<td align="left"> <span class="Estilo2">Direccion:</span>
<input type="text" size="50" maxlength="50"name="direccion" />
 
</tr>
</td>
 
<tr>
<td align="left"> <span class="Estilo2">Pais:</span>
<input type="text" size="20" maxlength="20"name="pais" />
</td>
</tr>
 
<tr>
<td align="left"> <span class="Estilo2">Fecha Nacimiento(AAAA/MM/DD):</span>
<input type="text" size="10" maxlength="10"name="fecha" />
</td>
 
</tr>
 
<tr>
<td align="left"> <span class="Estilo2">Telefono:</span>
<input type="text" size="20" maxlength="20"name="telefono" />
</td>
</tr>
 
<tr>
<td align="left"> <span class="Estilo2">Movil:</span>
<input type="text" size="20" maxlength="20"name="movil" />
</td>
</tr>
 
<tr>
<td align="left"> <span class="Estilo2">Matricula:</span>
<input type="text" size="15" maxlength="15"name="matricula" />
</td>
</tr>
 
<tr>
<td align="center">
<input type="submit" onClick="validar();" value="Registro" />