#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
-------------------------------------+-------------------------------------
Changes (by julien):
* status: new => closed
* resolution: => duplicate
Comment:
Thanks a lot for taking the time to detail out all the steps to reproduce
this problem. Here the `_object_dict` cache is used for performance
reasons but obviously it isn't thread-safe, at least in the context of
`list_editable`.
However, I think this is just one particular issue among the several that
would need to be addressed as part of #11313. If we can fix this then we
should fix it as a whole rather than piecemeal. For this reason I'll close
this ticket as a duplicate of #11313. Let's continue the conversation over
there.
By the way, you've made a good point about the documentation. A warning
should be added indicating that `list_editable` isn't thread safe. I'll
add a comment about that in the other ticket. Thank you!
--
Ticket URL: <https://code.djangoproject.com/ticket/17118#comment:3>
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.
