#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.

Reply via email to