#17118: list_editable will update wrong rows on a multiuser system if
pagination is
enabled
-------------------------------------+-------------------------------------
Reporter: emilianodelvalle@… | Owner: nobody
Type: Bug | Status: closed
Component: contrib.admin | Version: 1.3
Severity: Normal | Resolution: duplicate
Keywords: list_editable list | Triage Stage:
multiuser | Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by chad.lyon@…):
That work around doesn't work but is really close. I think you want to
avoid using the query set altogether when list editing because it could
possibly return different results than it did when the list to be edited
was POSTed. This actually works as a work around:
{{{
def _existing_object(self, pk):
return self.model._default_manager.get_query_set().get(pk = pk)
}}}
...but has side effects. For example, what if another user deletes one of
the items in the list to be edited? This workaround will raise
{{{DoesNotExist}}}. Django committers need to get busy on the solution
talked about in #11313 because this bug is really ugly.
--
Ticket URL: <https://code.djangoproject.com/ticket/17118#comment:4>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.