Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#13051 closed (fixed)

reversing namespaces doesn't work according to the docs.

Reported by: apollo13 Owned by: gabrielhurley
Component: Documentation Version: master
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

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)

ns.diff (2.5 KB) - added by apollo13 5 years ago.
13051_docs_patch.diff (1.3 KB) - added by gabrielhurley 5 years ago.
patches docs instead of changing code behavior

Download all attachments as: .zip

Change History (9)

Changed 5 years ago by apollo13

comment:1 Changed 5 years ago by russellm

  • Component changed from Uncategorized to Documentation
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement set
  • Triage Stage changed from Unreviewed to Accepted

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.

comment:2 Changed 5 years ago by gabrielhurley

  • Owner changed from nobody to gabrielhurley
  • Status changed from new to assigned

I'll write up a patch for the docs here in a minute.

Changed 5 years ago by gabrielhurley

patches docs instead of changing code behavior

comment:3 Changed 5 years ago by gabrielhurley

  • Patch needs improvement unset

comment:4 Changed 5 years ago by russellm

  • Triage Stage changed from Accepted to Ready for checkin

comment:5 Changed 5 years ago by russellm

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [13161]) Fixed #13051 -- Corrected the documentation of URL namespacing to match the tested implementation behavior. Thanks to apollo13 for the report, and Gabriel Hurley for the patch.

comment:6 Changed 5 years ago by russellm

(In [13164]) [1.1.X] Fixed #13051 -- Corrected the documentation of URL namespacing to match the tested implementation behavior. Thanks to apollo13 for the report, and Gabriel Hurley for the patch.

Backport of r13161 from trunk.

comment:7 Changed 4 years ago by jacob

  • milestone 1.2 deleted

Milestone 1.2 deleted

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