#20088: Django admin log depends on User model having a field named id
-------------------------------------+-------------------------------------
Reporter: Ryan Leckey | Owner: jcatalan
<leckey.ryan@…> | Status: closed
Type: Bug | Version: master
Component: contrib.admin | Resolution: fixed
Severity: Normal | Triage Stage:
Keywords: | Unreviewed
configurable,user,model | Needs documentation: 0
Has patch: 0 | Patch needs improvement: 1
Needs tests: 0 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by Juan Catalano <jc@…>):
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"054ce2aa02c88221ffa5665a4b9a5739cbbd0a39"]:
{{{
#!CommitTicketReference repository=""
revision="054ce2aa02c88221ffa5665a4b9a5739cbbd0a39"
Fixed #20088 -- Changed get_admin_log not to depend on User id field
Before this change, the get_admin_log method would expect User model's
FK to be named `id`. When changing that FK name, admin/index.html
rendering would fail.
This includes:
* Changed the use of id for the use of pk property.
* Added a regression test that fails without the patch.
This commit refs #20088.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/20088#comment:6>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
