Opened 18 years ago
Closed 18 years ago
#5431 closed (fixed)
Argentinean localflavor
| Reported by: | Ramiro Morales | Owned by: | nobody |
|---|---|---|---|
| Component: | contrib.localflavor | Version: | dev |
| Severity: | Keywords: | aArgentina localflavor | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Find attached the patch that implemements a widget and some field for Argentina-specific information, it included test cases.
Note: I can't get Mercurial to represent the empty django/contrib/localflavor/ar/__init__.py file on the patch it generates, so if this get commited, the file should be created manually, sorry for the burden :(
Attachments (2)
Change History (6)
by , 18 years ago
| Attachment: | argentinean_localflavor_r6139.diff added |
|---|
comment:1 by , 18 years ago
| Triage Stage: | Unreviewed → Ready for checkin |
|---|
follow-up: 3 comment:2 by , 18 years ago
comment:3 by , 18 years ago
| Keywords: | aArgentina added; argentina removed |
|---|
Replying to mtredinnick:
Line 112 of the patch can't be correct. It's referencing a variable that doesn't exist. Do you mean
cuit[:-1]there?
You are completely right, sorry. New version of the patch attached with that fix.
/me installs pychecker.
by , 18 years ago
| Attachment: | argentinean_localflavor_r6139-2.diff added |
|---|
comment:4 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Line 112 of the patch can't be correct. It's referencing a variable that doesn't exist. Do you mean
cuit[:-1]there?