#13051 closed (fixed)
reversing namespaces doesn't work according to the docs.
| Reported by: | Florian Apolloner | Owned by: | Gabriel Hurley |
|---|---|---|---|
| Component: | Documentation | Version: | dev |
| Severity: | Keywords: | ||
| 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
according to http://docs.djangoproject.com/en/1.1/topics/http/urls/#url-namespaces point 4, django should take the first deployed instance if there is no current_app defin ed (and no default app). Currently it takes the last. The attached patch fixes it. Note: Tests did test this behavior, but they acciendently tested for the last instance.
Attachments (2)
Change History (9)
by , 14 years ago
comment:1 by , 14 years ago
| Component: | Uncategorized → Documentation |
|---|---|
| Patch needs improvement: | set |
| Triage Stage: | Unreviewed → Accepted |
comment:2 by , 14 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
I'll write up a patch for the docs here in a minute.
by , 14 years ago
| Attachment: | 13051_docs_patch.diff added |
|---|
patches docs instead of changing code behavior
comment:3 by , 14 years ago
| Patch needs improvement: | unset |
|---|
comment:4 by , 14 years ago
| Triage Stage: | Accepted → Ready for checkin |
|---|
comment:5 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
comment:6 by , 14 years ago
Note:
See TracTickets
for help on using tickets.
Accepted, but I'm inclined to say the problem here is actually the documentation. The current implementation is in the wild, and the current behavior is explicitly tested. Ultimately, the decision of whether you use the first or the last instance is arbitrary; in the interests of minimal impact on deployed instances, I'm inclined to say we fix the documentation to reflect what the implementation actually does.