#15175 closed (fixed)
bug when validate chilean national identification number
Reported by: | rvimer | Owned by: | ersame |
---|---|---|---|
Component: | contrib.localflavor | Version: | dev |
Severity: | Keywords: | localflavor | |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Bug when validate chilean national identification number with verifier digit lower case ('k').
Steps to reproduce the bug
>>> from django.contrib.localflavor.cl.forms import CLRutField >>> rut = CLRutField() >>> rut.clean('13449333-k') Traceback (most recent call last): File "<console>", line 1, in <module> File "/home/rene/backup/programFiles/django/trunk/django/contrib/localflavor/cl/forms.py", line 56, in clean raise ValidationError(self.error_messages['checksum']) ValidationError: [u'The Chilean RUT is not valid.'] >>> rut.clean('13449333-K') u'13.449.333-K' >>>
Attachments (3)
Change History (9)
by , 14 years ago
Attachment: | form.patch added |
---|
comment:1 by , 14 years ago
Needs tests: | set |
---|---|
Patch needs improvement: | set |
Triage Stage: | Unreviewed → Accepted |
by , 14 years ago
Attachment: | localflavour_cl_form.diff added |
---|
comment:2 by , 14 years ago
Needs tests: | unset |
---|---|
Owner: | changed from | to
Patch needs improvement: | unset |
Status: | new → assigned |
Hi,
This is my first patch and my hello to django developper.
It's an easy ticket and I would like to know if I have done it well.
Regards.
comment:3 by , 14 years ago
Triage Stage: | Accepted → Ready for checkin |
---|
by , 14 years ago
Attachment: | localflavour_cl_form_2.diff added |
---|
comment:4 by , 14 years ago
Modified ersames' patch. I didn't really see the need for the additional line declaring 'verifier' as a variable.
Note:
See TracTickets
for help on using tickets.
Patch form.py