passing an argument to a custom save() method

Question

How do I pass an argument to my custom save method, preserving proper *args, **kwargs for passing to te super method? I was trying something like:

form.save(my_value)

and

def save(self, my_value=None, *args, **kwargs): super(MyModel, self).save(*args, **kwargs) print my_value

But this doesn't seem to work. What am I doing wrong?

Edit: I found this example (see the last message, for passing 'reorder'): http://groups.google.com/group/django-users/browse_thread/thread/b285698ea3cabfc9/6ce8a4517875cb40?lnk=raot

This is essentially what I am trying to do, but my_value is said to be an unexpected argument for some reason.

Defined method save(self, my_value=None, *args, **kwargs) expects keyword argument: save(my_value=some_value) (check this question about positional and keyword arguments) — Denis
– Denis, Commented Jul 11, 2021 at 9:22

Daniel Naab · Accepted Answer · 2010-03-29 02:40:34Z

26

Keyword arguments must follow the positional arguments. Try this instead:

def save(self, my_value, *args, **kwargs): ....

or:

def save(self, *args, **kwargs): my_value = kwargs.pop('my_value', None)

edited Mar 29, 2010 at 2:40

answered Mar 29, 2010 at 2:26

Daniel Naab

23.2k8 gold badges57 silver badges56 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Nikita Over a year ago

Doesn't seem to do the trick - python says it's a syntax error

ImportError · Accepted Answer · 2020-09-26 11:39:27Z

You can try the following.

Override the save method as:

def save(self, my_val, *args, **kwargs): print my_val # do_something here return super(MyModel, self).save(*args, **kwargs)

On calling the save method as:

MyModel.save(my_val="fooooo")

this will print my_val and save it.

mastaBlasta · Accepted Answer · 2019-11-04 16:31:10Z

The existing answers are incomplete.

The problem with Model.save is that there are multiple ways to get to it, most notably through Model.objects.create(). A queryset create() call will not pass additional arguments to Model.save and your code might not behave as expected.

You can easily see the source code for create() and other methods (get_or_create, update_or_create) here: https://github.com/django/django/blob/2.2.7/django/db/models/query.py#L415

The bulk_create method of QuerySet does not call save at all. Which means any custom logic will be entirely ignored.

Because of these risks I would consider the premise of the question an anti-pattern to be avoided. Rather than overloading the built in save method, create an entirely different (factory) method to encompass your special behavior e.g. mymodel.special_save().

Collectives™ on Stack Overflow

passing an argument to a custom save() method

3 Answers 3

1 Comment

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Linked

Related