Url to contrib.admindocs is still hardcoded in base template of contrib.admin

[Remco Wendt]

In revision 7947 of newforms-admin, the base template of contrib.admin app still hardlinks to the admindocs with:

<a href="{{ root_path }}doc/">{% trans 'Documentation' %}</a>

This behaviour is incorrect, since users now decide for themselves where the admindocs recide. The hard coding of the url effectively forces a user to add:

    (r'^admin/doc/', include('django.contrib.admindocs.urls')),

Above their admin app mapping in urls.py. In the current situation changing the url to for example be:

    (r'^admin/documentation/', include('django.contrib.admindocs.urls')),

In #6962 it was already mentioned to use a named url, since there is coupling between the admin and admindocs app anyway.

[Brian Rosner]

Not critical for 1.0-alpha.

[MattBowen]

adds names for the admindocs urls and uses the named URL in admin's base.html

[MattBowen]

I used the namespace django-admindocs-* for all the named pattens; it's a little long, but it will likely be unique for applications (unless someone is looking for trouble), and it probably won't need to be typed often anyway. This change has the interesting side effect of making the Documentation url simple point to the admin's root if the admindocs URLs are not loaded. This gets rid of the 404, but is arguably a more confusing behavior.

[MattBowen]

It looks like I missed some spots in bookmarkets, so I'm revising my patch.

[anonymous]

Ran this against rev. 8229, changed the admin doc urls, and this does what it's supposed to. Marking ready for checkin.

[adamfast]

Wasn't logged in. Above notes by me.

[Malcolm Tredinnick]

I have some Secret Inside Knowledge(tm) here. Matt's patch was done at the DC sprint on August 1 and after he attached it, we realised there was one problematic case: bookmarklets. They assume there is a single admin site (no longer necessarily true) and seem to have to make some assumptions about the URL of that site.

The patch here is a good start, but it doesn't quite solve all the problems (see comment:6). We need to make a decision about bookmarklets -- carve them off to somewhere else,? Require configuration? Inventive third option we didn't think of yet? But current patch only shuffles the bugs until that is worked out.

(Remember, campers, tickets aren't the place to do design work, so resist the urge to post random suggestions on that item in the comments. We have a mailing list for that.)

[Jacob]

(In [8717]) Fixed #7810: added named URLs for admin docs, and use them in the admin base template. Thanks, MattBowen.

[Jacob]

Milestone 1.0 deleted