"Overriding predefined model methods": use *args, **kwargs in code sample

[Daniel Hahler]

The "Overriding predefined model methods" section in the models doc provides sample code to override save, and I think that the code should use "*args, kwargs" so that your own implementation is future proof.

I'm attaching a patch for docs/topics/db/models.txt

[Daniel Hahler]

Patch against trunk

[Alex Robbins]

I'm not sure that you want to pass through all args and kwargs. If I pass anything other than force_insert or force_update to my function then I will get TypeError (unexpected keyword argument) when that argument is passed into the model's save method. The model's save method doesn't accept kwargs, so passing that to it is asking for trouble.

Just my 2 cents.
Alex

[Alex Robbins]

I'm going to set this invalid since I think it won't work. Feel free to set it back if I am wrong.