Changes between Version 2 and Version 3 of Ticket #35890, comment 6


Ignore:
Timestamp:
Nov 13, 2024, 9:21:11 AM (8 days ago)
Author:
Simon Charette

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #35890, comment 6

    v2 v3  
    66In your particular case, `Child.save(update_fields={...})`, won't update `"modified_at"` unless it's specified explicitly so your expectations are already broken for any direct or indirect (through third-party apps) usage of `save(update_fields)`; `update_or_create` is only a single instance of that.
    77
    8 The changes made to handle local concrete fields in #32095 ([https://github.com/django/django/pull/13526 original PR by Florian], [https://github.com/django/django/pull/16072 eventually merged changes by Sarah]) unfortunately didn't cover this use case as [https://github.com/django/django/pull/13526#discussion_r503304956 it wasn't pointed out clearly enough as a point of ambiguity].
     8The changes made to handle local concrete fields in #32095 ([https://github.com/django/django/pull/13526 original PR by Florian], [https://github.com/django/django/pull/16072 eventually merged changes by Sarah]) unfortunately didn't cover this use case as [https://github.com/django/django/pull/13526#discussion_r503304956 it wasn't pointed out clearly enough as a point of ambiguity] and none of the test models made use of MTI.
    99
    1010> Why do you think it should be that way?
Back to Top