#10365 closed (duplicate)
localflavor.ca.ca_provinces incorrect.
Reported by: | bartek | Owned by: | nobody |
---|---|---|---|
Component: | contrib.localflavor | Version: | 1.0 |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Design decision needed | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
In the tuples provided in this file, Newfoundland and Labrador have an abbreviation of "NF"
This is incorrect, and is actually "NL". I am unsure if this is a recent change but that is the case according to official Canadian websites and even systems like PayPal Pro
Change History (5)
comment:1 Changed 13 years ago by
milestone: | → 1.1 |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:2 Changed 13 years ago by
Triage Stage: | Accepted → Design decision needed |
---|
comment:4 Changed 13 years ago by
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Closing as a dupe of #10456, since it's itnertwined with the changes over there (as per comment 6, there)
Note: See
TracTickets for help on using
tickets.
I'd be -1 on making this change. The abbreviation in question isn't a display string: it's the value stored in the database. We haven't made any error with the human-consumable string.
The big argument against changing it is that anybody who is already using this field will now have invalid data in their database. Suddenly they will have stored NF and it won't map to any display string (and entering the same data after the change will put different data into the database).
Moving to "design decision needed"; it's not a no-brainer and it's not a functional bug at the moment.