Opened 16 years ago

Closed 16 years ago

Last modified 12 years ago

#8276 closed Uncategorized (fixed)

Change fields' names in django.contrib.localflavor.pl.forms

Reported by: Piotr Lewandowski <django@…> Owned by: nobody
Component: contrib.localflavor Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

I would like to propose backward incompatible change django.contrib.localflavor.pl.forms:

Some classes present in django.contrib.localflavor.pl.forms are named in a werid way - they are neither short and useful, nor offical translations of corresponding numbers.

Old nameNew name
PLNationalIdentificationNumberFieldPLPESELField
PLTaxNumberFieldPLNIPField
PLNationalBusinessRegisterFieldPLREGONField
PLVoivodeshipSelectPLProvinceSelect
PLAdministrativeUnitSelectPLCountiesSelect

Not only Polish people in everyday language use those acronyms, but even Polish officials refer to those numbers as PESEL, REGON and NIP:

I'm willing to prepare the patch if those changes are accepted.

Attachments (2)

polish_forms_r8521.diff (891 bytes ) - added by Manuel Saelices 16 years ago.
A patch that fixes typo error
polish_form_r8522.diff (20.9 KB ) - added by Piotr Lewandowski <django@…> 16 years ago.
Previous patch does not include tests

Download all attachments as: .zip

Change History (13)

comment:1 by Malcolm Tredinnick, 16 years ago

milestone: 1.0 beta
Triage Stage: UnreviewedAccepted

Using the "correct" names sounds right and, failing any other indication, the common usage is correct. So this seems reasonable. I'll get it in before the beta.

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

Malcolm, can you also take a look at ticket #8197? Are there any chances to get that in before the beta?

PLCountiesSelect (aka PLAdministrativeUnitSelect) is using outdated counties list - I don't know whether it's worth filling separate ticket.

In the long run, fields like PLProvinceSelect, PLCountiesSelect and not-yet-existing PLCommunesSelect and PLCitiesSelect could use data from pl_regions.py introduced in #8197 so they'd be up-to-date. I may prepare a path combining this feature and issues mentioned in #8197 and #8276.

comment:3 by Malcolm Tredinnick, 16 years ago

#8197 is unrelated to this ticket. Adding new stuff like that can wait, since there are design questions there.

comment:4 by Malcolm Tredinnick, 16 years ago

Resolution: fixed
Status: newclosed

(In [8345]) Fixed #8276 -- Changed the names of a few Polish localflavor classes to match
their common names/acronyms, similar to other localflavors.

Backwards incompatible if you're using these classes.

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

Resolution: fixed
Status: closedreopened

My mistake - PLCountiesSelect should be rather named PLCountySelect - to be coherent with ro and uk localflavors.

by Manuel Saelices, 16 years ago

Attachment: polish_forms_r8521.diff added

A patch that fixes typo error

comment:6 by Manuel Saelices, 16 years ago

Has patch: set

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

Attachment: polish_form_r8522.diff added

Previous patch does not include tests

comment:7 by Malcolm Tredinnick, 16 years ago

milestone: 1.0 beta1.0

comment:8 by Jakub Wilk <ubanus@…>, 16 years ago

Cc: ubanus@… added

comment:9 by Jacob, 16 years ago

Resolution: fixed
Status: reopenedclosed

(In [8643]) Fixed #8276: corrected another couple of names in localflavor.pl.forms. Patch by Piotr Lewandowski.

comment:10 by ubanus@…, 13 years ago

Cc: ubanus@… removed
Easy pickings: unset
Severity: Normal
Type: Uncategorized

comment:11 by Jacob, 12 years ago

milestone: 1.0

Milestone 1.0 deleted

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