Skip to content

Fix instance.save(update fields=[...]) #33

Merged
merged 5 commits into from Mar 28, 2013

2 participants

@treyhunner
Collaborator

This adds a test for using the new update_fields in the model save method in Django 1.5 and modifies ModelTracker to pass the test.

This isn't ready to merge yet because there is still no test for update_fields when saving the model for the first time and I'm not yet sure how Django models handle that case yet.

Fixes gh-28.

@treyhunner
Collaborator

More changes:

  • save() now passes keyword arguments to parent save and the update_fields test checks for this
  • test added for using ModelTracker before a model instance is saved in the database

I think this is ready for merging.

@carljm carljm merged commit 4156fa4 into carljm:master Mar 28, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.