Opened 11 years ago

Closed 11 years ago

#21948 closed Cleanup/optimization (fixed)

Docs on extending admin templates should mention TEMPLATE_LOADERS

Reported by: django@… Owned by: nobody
Component: Documentation Version: 1.6
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description (last modified by Tim Graham)

The docs at ​https://docs.djangoproject.com/en/dev/ref/contrib/admin/#overriding-admin-templates should mention that you *must* set:

TEMPLATE_LOADERS = (
     'django.template.loaders.filesystem.Loader',
     ...
)

and that django.template.loaders.filesystem.Loader must come before django.template.loaders.app_directories.Loader.

This confused me for a long time, mainly because the default settings.py created with django-admin.py startproject myproject doesn't contain any of those settings.

(P.S. you also need to set TEMPLATE_DIRS of course. This is mentioned in the docs linked above, but I think it could be clearer)

Thanks

Change History (4)

comment:1 by anonymous, 11 years ago

Component: Uncategorized → Documentation

comment:2 by Tim Graham, 11 years ago

Description: modified (diff)
Easy pickings: set
Triage Stage: Unreviewed → Accepted
Type: Uncategorized → Cleanup/optimization

These are the default loaders so that's why it's probably not mentioned.

comment:3 by alextreme, 11 years ago

A small addition to clarify that the filesystem template loader must be used when extending the admin templates, in case the user has deviated from the default:

​https://github.com/django/django/pull/2361

comment:4 by Tim Graham <timograham@…>, 11 years ago

Resolution: → fixed
Status: new → closed

In ea869b514846cdd4803e3a87b08686113c14f7f2:

Fixed #21948 -- Noted TEMPLATE_LOADERS requirement when overriding admin templates.

Thanks django at patjack.co.uk for the suggestion.

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