Canada localflavor improvements/fixes
|Reported by:||mmulley||Owned by:||nobody|
|Severity:||Normal||Keywords:||localflavor, ca, localflavorsplit|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||yes|
There's a discussion in #10365 about the merits of changing the abbreviation and potentially breaking existing data. It's a tough question, since the abbreviation isn't simply an internal representation: it's a very common use case to use the postal abbreviation in addresses, and so the current version of ca.localflavor will generate incorrect output. This doesn't help with the issue of existing data, though, and I'm not quite sure what to do there.
Two patches are attached. One updates to the correct abbreviations. The other maintains the current, incorrect abbreviations but normalizes the correct abbreviations (NL, YT) to the incorrect ones (NF, YK) instead of rejecting them, as the current version does.
Both patches also include some other fixes and improvements:
- More abbreviations and misspellings are accepted and normalized for province names
- French province names are now accepted
- CAPostalCodeField accepts and normalizes common postal code formats ('h4u5h7' -> 'H4U 5H7')
Change History (16)
comment:1 Changed 8 years ago by
|Patch needs improvement:||set|
|Triage Stage:||Unreviewed → Accepted|