Opened 12 years ago

Closed 11 years ago

#19161 closed Bug (fixed)

Missing clean_password in custom user documentation

Reported by: DavidW Owned by: nobody
Component: Documentation 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: yes UI/UX: no

Description (last modified by Aymeric Augustin)

https://docs.djangoproject.com/en/dev/topics/auth/#auth-custom-user

class UserChangeForm(forms.ModelForm):
    """A form for updateing users. Includes all the fields on
    the user, but replaces the password field with admin's
    pasword hash display field.
    """
    password = ReadOnlyPasswordHashField()

    class Meta:
        model = MyUser

Needs the following added:

    def clean_password(self):
        # Regardless of what the user provides, return the initial value.
        # This is done here, rather than on the field, because the
        # field does not have access to the initial value
        return self.initial["password"]

Or you get a not-null constraint violation on form submit.

Change History (6)

comment:1 by Łukasz Rekucki, 12 years ago

Has patch: unset

comment:2 by Aymeric Augustin, 12 years ago

Description: modified (diff)

Fixed formatting (please use preview).

comment:3 by Russell Keith-Magee, 12 years ago

Triage Stage: UnreviewedAccepted

comment:4 by Russell Keith-Magee, 12 years ago

Severity: NormalRelease blocker

Actually, this should be a release blocker, since it's a fundamentally misleading piece of documentation in a new feature.

comment:5 by Claude Paroz, 11 years ago

Summary: Line missing on Dev documentationMissing clean_password in custom user documentation

comment:6 by Tim Graham, 11 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.
Back to Top