id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
17967	"""change password"" link in the admin header should be easier to disable"	Dougal Sutherland	Aviral Dasgupta	"The ""change password"" link in the admin header doesn't make sense for all installations. For example, sites that use `django-auth-ldap` as the primary authentication scheme don't necessarily allow users to change their password, because it [[https://bitbucket.org/psagers/django-auth-ldap/issue/8/add-the-ablility-to-change-password#comment-402102|might not make sense to do so]]. In that case, having the ""change password"" link in the header of the admin is confusing for users, because it doesn't work / do what they expect it to.

I've overriden the `base.html` template and made this change:

{{{
35c35,37
<                 <a href=""{% url 'admin:password_change' %}"">{% trans 'Change password' %}</a> /
---
>                 {% if user.has_usable_password %}
>                     <a href=""{% url 'admin:password_change' %}"">{% trans 'Change password' %}</a> /
>                 {% endif %}
}}}
which is appropriate for my sites, though I'm not sure if it is in general.

But this is annoying, because then when you update Django (as I did today) any changes to `base.html` just won't happen and there will probably be problems.

If `has_usable_password` isn't a good way to distinguish in general, then there should either be a setting for whether this should be displayed, or the `<div id=""user-tools"">` should be in its own template for easy overriding."	New feature	closed	contrib.admin	1.4	Normal	fixed		Aviral Dasgupta	Accepted	1	1	1	0	1	0