tickets: 8273
This data as json
id | created | changetime | last_pulled_from_trac | stage | status | component | type | severity | version | resolution | summary | description | owner | reporter | keywords | easy | has_patch | needs_better_patch | needs_tests | needs_docs | ui_ux |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
8273 | 2008-08-13 02:13:52 | 2012-10-15 19:37:28 | 2022-03-06 03:41:49.004388 | Accepted | closed | contrib.localflavor | Cleanup/optimization | Normal | dev | wontfix | Reduce amount of redundant code in django.contrib.localflavor | There is a plenty of somehow redundant code in {{{django.contrib.localflavor}}} which leads to potentially more bugs and what is quite inconvenient -- many translation strings. I would like to propose the patch that handles all fields with a checksum verification. Pros: * a lot fewer translation strings, which is more consistent and means less work when translating Django to 50 languages, * DRY compatible (and all benefits that it implies). Cons: * backwards incompatible, * new code may contain bugs (although all tests were OK). I volunteer for further modifications of {{{django.contrib.localflavor}}} code to use {{{NumberWithChecksumField}}} if this patch is accepted. | nobody | Piotr Lewandowski <django@icomputing.pl> | 0 | 1 | 1 | 0 | 0 | 0 |