8000 Fix disabling check digit validation of Mexican CURP · QuantumNovice/python-stdnum@02dec52 · GitHub
  • [go: up one dir, main page]

    Skip to content

    Commit 02dec52

    Browse files
    committed
    Fix disabling check digit validation of Mexican CURP
    The validation functions supported an optional parameter to disable check digit validation in the number that didn't actually affect validation and was most likely accidentally copied from the RFC module. Fixes 50874a9 Closes arthurdejong#285
    1 parent dcf4730 commit 02dec52

    File tree

    1 file changed

    +3
    -3
    lines changed

    1 file changed

    +3
    -3
    lines changed

    stdnum/mx/curp.py

    Lines changed: 3 additions & 3 deletions
    Original file line numberDiff line numberDiff line change
    @@ -109,7 +109,7 @@ def calc_check_digit(number):
    109109
    return str((10 - check % 10) % 10)
    110110

    111111

    112-
    def validate(number, validate_check_digits=False):
    112+
    def validate(number, validate_check_digits=True):
    113113
    """Check if the number is a valid CURP."""
    114114
    number = compact(number)
    115115
    if len(number) != 18:
    @@ -122,12 +122,12 @@ def validate(number, validate_check_digits=False):
    122122
    get_gender(number)
    123123
    if number[11:13] not in _valid_states:
    124124
    raise InvalidComponent()
    125-
    if number[-1] != calc_check_digit(number):
    125+
    if validate_check_digits and number[-1] != calc_check_digit(number):
    126126
    raise InvalidChecksum()
    127127
    return number
    128128

    129129

    130-
    def is_valid(number, validate_check_digits=False):
    130+
    def is_valid(number, validate_check_digits=True):
    131131
    """Check if the number provided is a valid CURP."""
    132132
    try:
    133133
    return bool(validate(number, validate_check_digits))

    0 commit comments

    Comments
     (0)
    0