#18306: Deferred models should automatically issue update_fields when saving
-------------------------------------+-------------------------------------
Reporter: akaariai | Owner: nobody
Type: | Status: new
Cleanup/optimization | Version: master
Component: Database layer | Resolution:
(models, ORM) | Triage Stage: Accepted
Severity: Normal | Needs documentation: 1
Keywords: | Patch needs improvement: 1
Has patch: 1 | UI/UX: 0
Needs tests: 0 |
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by akaariai):
* needs_better_patch: 0 => 1
* has_patch: 0 => 1
* needs_docs: 0 => 1
* stage: Unreviewed => Accepted
Comment:
The implementation looks suspiciously clean... :) However there is going
to be some problems to get the updating of manually set fields to work
correctly. (the a.age = 31 case in original description). Apart of this
problem the patch needs some documentation (only()/defer() docs need a
mention of this, maybe also in .save()).
If it turns out to be at all problematic to support the
model._state.update_fields attribute lets just leave that out from this
ticket. I'd like to try that idea out, but there is no need to complicate
this ticket with that idea.
--
Ticket URL: <https://code.djangoproject.com/ticket/18306#comment:2>
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 django-updates@googlegroups.com.
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
