Fixed #18306 -- Deferred models should automatically issue update_fields... #65

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

Contributor
niwinz commented May 15, 2012

Ticket: https://code.djangoproject.com/ticket/18306

Is an initial patch, run all test cases mentioned in the ticket.
I added documentation, where I felt I should be, but I'm not sure.

The only thing not clear to me, as it should behave _state.update_fields.

  • Normal case, save all fields -> this attribute must be None (or all fields?)
  • In all other cases, filled with the result of update_fields.
Member

Great patch! I think defered_* should be renamed to deferred_ (two r).

Contributor
niwinz commented May 16, 2012

;) I have renamed the variables!

Member

I will close this pull request for now, until the issues raised in #18306 are resolved.

@akaariai akaariai closed this May 18, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment