Code

Opened 5 years ago

Closed 3 years ago

#12062 closed Bug (duplicate)

django.contrib.localflavor.cl.CLRutField isn't doing proper validation

Reported by: aotarola Owned by: nobody
Component: contrib.localflavor Version:
Severity: Normal Keywords:
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX:

Description

django.contrib.localflavor.cl.CLRutField ignores validation for lowercase values.


A valid chilean rut ( chilean national identification number) is for example: 76796630-K (case for "K" must be uppercase), but not all people are aware of this,
and they keep using the lowercase version, so they spend a lot of time in online forms figuring this out.

The lowercase problem must be handled every time a form has a CLRutfield.

I believe this issue must be handled by CLRutfield's validation.

The attached patch solves this problem.

Attachments (1)

enhanced_chilean_rut_validation.diff (599 bytes) - added by aotarola 5 years ago.

Download all attachments as: .zip

Change History (5)

Changed 5 years ago by aotarola

comment:1 Changed 5 years ago by jcd

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Design decision needed

comment:2 Changed 4 years ago by russellm

  • milestone 1.2 deleted
  • Version 1.1 deleted

comment:3 Changed 3 years ago by julien

  • Easy pickings unset
  • Needs tests set
  • Severity set to Normal
  • Type set to Bug

comment:4 Changed 3 years ago by claudep

  • Resolution set to duplicate
  • Status changed from new to closed

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.