Opened 6 years ago

Closed 4 years ago

#14015 closed New feature (invalid)

django.contrib.localflavor.it added italian phone number validation

Reported by: Andrea Rossi Owned by: nobody
Component: contrib.localflavor Version: 1.2
Severity: Normal Keywords: django.contrib.localflavor.it, ITPhoneNumberField, form, phone, number, validation, localflavorsplit
Cc: andrea@… Triage Stage: Accepted
Has patch: yes Needs documentation: yes
Needs tests: yes Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Hi to all,
I found myself in need of an Italian phone numbers validator, so I wrote it as a localflavor element, and now I'm sharing it with the community of Django.
The validation process is quite simple but yet very functional. The (eventual) international prefix (both +39 or 0039) is stripped from the number entered (something like 0039 010 1234567 -> 010 1234567).
The local prefix (in example 010, the prefix for Genoa) is checked against a list of valid prefixes (both fixed line or mobile), and if the number meet all the requirements (containing only number, having a prefix between 2 and 4 digits, and a number itself between 5 and 8 digits...) is returned as a cleaned value.

Hope of being of some Help,
Greetings,
Andrea

Attachments (1)

django.contrib.localflavor.it.tar.gz (3.6 KB) - added by Andrea Rossi 6 years ago.

Download all attachments as: .zip

Change History (7)

Changed 6 years ago by Andrea Rossi

comment:1 Changed 6 years ago by Russell Keith-Magee

Triage Stage: UnreviewedAccepted

comment:2 Changed 6 years ago by Julien Phalip

Needs documentation: set
Needs tests: set
Patch needs improvement: set
Severity: Normal
Type: Uncategorized

Could you please provide a diff patch instead of the whole files? It'd be great if you could also write some doc, and some tests either in source:django/trunk/tests/regressiontests/localflavor or source:django/trunk/tests/regressiontests/forms/localflavor ?

comment:3 Changed 6 years ago by Julien Phalip

Type: UncategorizedNew feature

comment:4 Changed 5 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:5 Changed 5 years ago by Aymeric Augustin

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:6 Changed 4 years ago by Aymeric Augustin

Keywords: localflavorsplit added
Resolution: invalid
Status: newclosed

django.contrib.localflavor is now deprecated — see https://docs.djangoproject.com/en/dev/ref/contrib/localflavor/

A repository was created for each localflavor at https://github.com/django/django-localflavor-? (Replace with the country code.)

If you're still interested in this ticket, could you create a pull request on that repository?

Sorry for not resolving this issue earlier, and thanks for your input!

Note: See TracTickets for help on using tickets.
Back to Top