Opened 16 years ago

Closed 16 years ago

#8478 closed (duplicate)

Change Canadian postal code for Newfoundland

Reported by: Doug Harris Owned by: Doug Harris
Component: contrib.localflavor Version: dev
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

django.contrib.localflavor.ca.ca_provinces has "NF" as the postal code for Newfoundland and Labrador. According to the Canada Post web site, the proper abbreviation is "NL".

The wikipedia article on Canadian subnational postal abbreviations shows "NF" as a former abbreviation.

Attachments (1)

8520.diff (1.2 KB ) - added by Piotr Lewandowski <django@…> 16 years ago.
django.contrib.localflavor.ca.ca_provinces 'NF' -> 'NL', trailing spaces removed

Download all attachments as: .zip

Change History (5)

by Piotr Lewandowski <django@…>, 16 years ago

Attachment: 8520.diff added

django.contrib.localflavor.ca.ca_provinces 'NF' -> 'NL', trailing spaces removed

comment:1 by Piotr Lewandowski <django@…>, 16 years ago

Has patch: set
Triage Stage: UnreviewedAccepted

comment:2 by Doug Harris, 16 years ago

Patch needs improvement: set

This needs a better patch. The code itself should refer to NL in the PROVINCES_NORMALIZED (including recognizing the old "nf" and returning "NL"). Also, there should be changes to tests/regressiontests/forms/localflavor/ca.py

I will work on this, but I'm posting this comment to make sure that these items are considered before this ticket is closed.

comment:3 by Piotr Lewandowski <django@…>, 16 years ago

Owner: changed from nobody to Doug Harris

You're 100% right. I'm assigning you to this ticket, since I'll no longer work on it.

comment:4 by Malcolm Tredinnick, 16 years ago

Resolution: duplicate
Status: newclosed

Closing as a duplicate of #10365, since that ticket contains the comment of why this isn't even necessarily a bug and why changing it is probably a bad idea.

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