#10456: Canada localflavor improvements/fixes
-------------------------------------------------+--------------------------
Reporter: mmulley | Owner: nobody
Status: new | Milestone:
Component: django.contrib.localflavor | Version: SVN
Resolution: | Keywords: localflavor
ca
Stage: Accepted | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 0 |
-------------------------------------------------+--------------------------
Comment (by mtredinnick):
I think there might have been some cross-purposes discussion going on
here, now that I've tried to sort out the various patches on the tickets.
There are two separate issues at play.
1. The `PROVINCE_CHOICES` list. In that list, the first tuple is a value
stored in the database that is used for lookups and is not a display
string. That's the one I'd prefer not to change, since it's not a
functionality bug and changing it requires end-user code changes.
2. The normalised value returned from `CAProvinceField`. That is a user-
displayable field (it's something they can input directly) and that's a
functionality bug, since we're normalising to the wrong value. We can and
probably should break backwards-compatibility for that.
The problem is then that `CAProvinceSelect` can potentially be layered
over `CAProvinceField`, which adds weight to the option of changing them
both. Going to think about it a bit more to see if any good solution pops
up, because there are competing and contradictory requirements here. I'm
leaning slightly towards changing both at the moment.
Given how intertwined everything is, I'm going to close #10365 as a dupe
of this, since it's going to be hard to close one and not the other and
the more relevant comments are here.
--
Ticket URL: <http://code.djangoproject.com/ticket/10456#comment:6>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
