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
35584	Discard user full names from admin log entries	Kamil Paduszyński		"The `get_full_name` of the `auth.AbstractUser` model is used to indicate a user in the admin's object history:

{{{
#!default
<td>{{ action.user.get_username }}{% if action.user.get_full_name %} ({{ action.user.get_full_name }}){% endif %}</td>
}}}

In some circumstances, personal data like `first_name` or `last_name` (used in the mentioned method) are generally handled by a different model, where the `AUTH_USER_MODEL` is for the authentication data and permissions only.

Given I discard `first_name` and `last_name` from my user model and mark the respective methods as raising `NotImplementedError`, I have to remember to override the entire huge `content` block just to update a single line of code... In the case of the `get_short_name` method used in the `admin/base.html` template, the issue is much easier to resolve due to a better template organization (there, an update in a single block `welcome-msg` is needed).

Therefore, I think it would be better to remove the reference to `get_full_name` from the template, simply by replacing it by:

{{{
#!default
<td>{{ action.user.get_username }}</td>
}}}

In my opinion, a username is enough to identify a user associated with a log."	Cleanup/optimization	closed	contrib.admin	5.0	Normal	wontfix		Tim Graham Josh Schneier	Unreviewed	0	0	0	0	0	1