#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 , 15 years ago
comment:1 by , 15 years ago
Component: | Uncategorized → Documentation |
---|---|
Patch needs improvement: | set |
Triage Stage: | Unreviewed → Accepted |
comment:2 by , 15 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
I'll write up a patch for the docs here in a minute.
by , 15 years ago
Attachment: | 13051_docs_patch.diff added |
---|
patches docs instead of changing code behavior
comment:3 by , 15 years ago
Patch needs improvement: | unset |
---|
comment:4 by , 15 years ago
Triage Stage: | Accepted → Ready for checkin |
---|
comment:5 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:6 by , 15 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.