Problema en Firefox con funcion DNI

[carscx]

Buenas,

Encontre la siguiente funcion en JS para calcular la letra del DNI automaticamente. El caso es que en IE6, 7 y 8 si funciona, pero en Firefox no.

Esta es la funcion:

Código JavaScript:

function calcularNIF(dni) {
 
  if (!/^\d{8}$/.test(dni)) {
 
   alert ("El numero introducido no es valido");
 
   return false;
 
  }
 
 
 
  var letras = 'TRWAGMYFPDXBNJZSQVHLCKE';
 
  var numero = dni%23;
 
  alta_cliente.letra.value = letras.substring(numero,numero+1);
 
  return true;
 
 }
 

Y estos los campos del formulario:

Código HTML:

<p>DNI</p></td><td><input type="text" class="dni" value="DNI" name="dni" onKeypress="if (event.keyCode < 45 || event.keyCode > 57) event.returnValue = false;" onBlur="calcularNIF(this.value);" maxlength="8"onFocus="clearText(this)" /><input type="text" class="letradni" onfocus="this.blur()" name="letra" maxlength="1" size="1">
 

No se cual puede ser el fallo, espero vuestra ayuda, muchas gracias.

[David]

Revisa la Consola de Errores.

Por un lado, prueba usando document.getElementById() con el id para hacer referencia a los elementos del formulario.

[carscx]

¿Que consola de errores? Perdona mi ignorancia, pero no tengo ni idea de JS, donde le tengo que aplicar el document.getElementById() en el formulario o en la función.

gracias de antemano
saludos,

[buzu]

imagino que alta_cliente es el valor del name de tu formulario. Puedes probar con document.forms['alta_cliente']['letra'].value en lugar de alta_cliente.letra.value

[carscx]

Iniciado por buzu

imagino que alta_cliente es el valor del name de tu formulario. Puedes probar con document.forms['alta_cliente']['letra'].value en lugar de alta_cliente.letra.value

Con eso me funciono a la perfección muchas gracias.

Hay otro asunto, que es que tengo en ese mismo campo bloqueado para que no entren letras y solo numeros en IE lo consegui con el siguiente atributo en el campo del formulario. Pero en Firefox no funciona.

Código:

   1.
      <p>DNI</p></td><td><input type="text" class="dni" value="DNI" name="dni" onKeypress="if (event.keyCode < 45 || event.keyCode > 57) event.returnValue = false;" onBlur="calcularNIF(this.value);" maxlength="8"onFocus="clearText(this)" /><input type="text" class="letradni" onfocus="this.blur()" name="letra" maxlength="1" size="1">

Es el que remarco en rojo

[David]

Es que para los navegadores que no son Internet Explorer es event.which en lugar de event.keyCode y event.preventDefault() para cancelar la acción del evento en lugar de event.returnValue = false.

[carscx]

Entonces dejara de funcionar en IE???

[David]

Debes usar condicionales para que funcione en ambos:

Código Javascript:

key = event.which || event.keyCode;
 

Y para cancelar la acción del evento:

Código Javascript:

if (event.preventDefault) {
    event.preventDefault();
} else {
    event.returnValue = false;
}