platanus / angular-rut

57 stars 21 forks source link

Validación aprueba rut no válido #15

Open cmolina opened 7 years ago

cmolina commented 7 years ago

Estimados, Al ingresar "1.727.k17-2" es marcado como un RUT válido, cuando no lo es.

El markup que utilizo es el siguiente

<input ng-rut type="text" ng-model="ctrl.client.Rut" md-no-asterisk name="rut" ng-maxlength="13" required>

La versión de angular-rut usada es la 1.0.1.

cmolina commented 7 years ago

Al parecer el problema se encuentra en el uso de parseInt en

https://github.com/platanus/angular-rut/blob/v1.0.1/src/rut.js#L18

La razón está en MDN,

If parseInt encounters a character that is not a numeral in the specified radix, it ignores it and all succeeding characters and returns the integer value parsed up to that point.

Bastaría con verificar que el substring _value.slice(0,-1) es efectivamente sólo números.

cmolina commented 7 years ago

¿Existe alguna razón por la que el PR no ha sido aceptado? @gmq

EDIT: acabo de leer la respuesta en https://github.com/platanus/angular-rut/issues/12