Opened 13 years ago

Closed 13 years ago

#19079 closed Cleanup/optimization (fixed)

make USERNAME_FIELD a required attribute of user objects

Reported by: Preston Holmes Owned by: nobody
Component: contrib.auth Version: dev
Severity: Release blocker Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

currently it is documented as required, but is not present in any base class

instead a number of places we fall back to looking for a 'username' attribute

per:

https://code.djangoproject.com/ticket/19077#comment:6

it would clean things up a bit to simply require this as a defined attribute, and represent it appropriately on the base classes

Change History (2)

comment:1 by Russell Keith-Magee, 13 years ago

Severity: Normal → Release blocker
Triage Stage: Unreviewed → Accepted

I've got a patch to address this almost ready to go.

comment:2 by Russell Keith-Magee <russell@…>, 13 years ago

Resolution: → fixed
Status: new → closed

In c433fcb3fb34fccd69782979f0e7cd5f2d4a4893:

Fixed #19077, #19079 -- Made USERNAME_FIELD a required field, and modified UserAdmin to match.

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