Opened 10 years ago

Closed 6 years ago

#10206 closed Bug (wontfix)

admin/auth/user/* templates should be in django.contrib.auth app

Reported by: izibi Owned by: Chris Beaven
Component: contrib.admin Version: 1.0
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


In case somebody wants to create a own auth app with a user model he might get a problem with these templates in the admin.

So these templates should be moved to the django.contrib.auth app to prevent this problem.

Change History (5)

comment:1 Changed 10 years ago by Jacob

Triage Stage: UnreviewedDesign decision needed

I'm pretty sure these templates are admin-specific.

comment:2 Changed 8 years ago by Chris Beaven

Owner: changed from nobody to Chris Beaven
Severity: Normal
Status: newassigned
Triage Stage: Design decision neededAccepted
Type: Bug

Admin specific, but completely in context to the auth application. If the auth app is not installed, these templates should technically not be accessible.

comment:3 Changed 7 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:4 Changed 7 years ago by Aymeric Augustin

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:5 Changed 6 years ago by Aymeric Augustin

Resolution: wontfix
Status: assignedclosed

admin requires auth; "the auth app is not installed" isn't really possible.

The other way round is possible: you can use auth without the admin. In that case, the current templates wouldn't work, because they extend admin/base.html which wouldn't be accessible.

Therefore keeping these templates in the admin app is correct.

One could argue that "using the admin with an API-compatible re-implementation of auth that requires different templates" is possible, but that's much less common than "using auth but not the admin", all the more since Django now supports custom user models.

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