Opened 12 years ago
Closed 12 years ago
#19056 closed Bug (fixed)
Admin password change page relies on user.username
Reported by: | Russell Keith-Magee | Owned by: | nobody |
---|---|---|---|
Component: | contrib.admin | 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 |
Pull Requests: | How to create a pull request | ||
Description ¶
The admin "change password" template currently renders "original.username" to identify the user whose password is to be changed.
Pluggable user models means that username isn't always available. It should use __unicode__ instead.
Change History (6)
comment:1 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:2 by , 12 years ago
this also applies to the 'title' context variable set in UserAdmin's def user_change_password(self, request, id, form_url=''):
somewhere about:
context = { 'title': _('Change password: %s') % escape(user.username), 'adminForm': adminForm,
this should be changed to
context = { 'title': _('Change password: %s') % escape(user.get_username()), 'adminForm': adminForm,
i'm on the run right now, but perhaps somebody could commit that.
Greetings,
Gabe
comment:3 by , 12 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:4 by , 12 years ago
Severity: | Normal → Release blocker |
---|---|
Triage Stage: | Unreviewed → Accepted |
Good catch -- it's would also be worth doing a quick search for .username to see if there is anywhere else that the attribute is being used directly.
comment:5 by , 12 years ago
I have pull request for this (tiny) change on Github. I probably should have opened a ticket. https://github.com/django/django/pull/511
I quick grep of django.contrib.auth reveals this to be the sole remaining reference (aside from in tests).
I will close it the request if you commit your own patch.
comment:6 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
In 4c75344cc1d3c74ed73b7a8d6aab92a173afe8f5: