Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#15334 closed (wontfix)

Documentation on namespaced URL reversing has an error

Reported by: RaceCondition Owned by: nobody
Component: Documentation Version: master
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

"If there is no current instance - say, if we were rendering a page somewhere else on the site - myapp:index will resolve to the LAST REGISTERED instance of myapp. Since there is NO DEFAULT INSTANCE, the LAST INSTANCE of myapp that is registered will be used."

I assume the first "last registered" should instead be "default".

Change History (3)

comment:1 Changed 5 years ago by gabrielhurley

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to wontfix
  • Status changed from new to closed

To my reading the current docs are correct.

If, as you suggested, the first instance of "last registered instance" were changed to "default instance" the paragraph would contradict itself, saying that it will resolve to the default instance when there is no default instance.

The only way I could see improving that snippet would be to change the period before "since" to a semicolon or an em dash so that the two sentences are more clearly logically connected.

If the word "since" were an "if", what you say would be true.

If I am misunderstanding or misreading, please reopen the ticket with further explanation.

comment:2 Changed 5 years ago by RaceCondition

Isn't there a duplication in there?

First "myapp:index will resolve to the LAST REGISTERED instance of myapp"
and then "..., the LAST INSTANCE of myapp that is registered will be used." ?

Reading that paragraph feels like somebody clumsily cut and pasted some text. Sorry if I'm missing something.

comment:3 Changed 4 years ago by jacob

  • milestone 1.3 deleted

Milestone 1.3 deleted

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