Default parameters in syncdb.Command().Execute()

[kacah222@…]

Trying to run syncdb.Command().Execute() command there is an error

TypeError: int() argument must be a string or a number, not 'NoneType'

Reason: no default values specified in syncdb.handle_noargs

Fix in attach.

[Ramiro Morales]

See also:

• ​https://groups.google.com/forum/?hl=en&fromgroups#!topic/django-users/77xN8sgN6Fs/discussion
• r17028 / #13760

[Claude Paroz]

I suddenly realized that call_command was working around this by picking defaults from klass.option_list.

    # Grab out a list of defaults from the options. optparse does this for us
    # when the script runs from the command line, but since call_command can
    # be called programatically, we need to simulate the loading and handling
    # of defaults (see #10080 for details).

Either we stick with the comment in call_command (and specifically add in the docs that execute should not be called directly):

This is the primary API you should use for calling specific commands.

Or we move the defaults value grabbing from call_command to execute (conditionally only when it is not run from command line).

This might be the real design decision to make.

[Claude Paroz]

Doc updated with note about execute

[Carl Meyer]

In [17629]:

Fixed #17799 - Documented that the execute() method of a management command is not suitable for calling the command from code; call_command should be used instead. Thanks kacah222 for the report and claudep for the patch.

[Ramiro Morales]

In [17678]:

Fixed #17820 -- Fixed more occurrences of redundant handling of management commands options.

They had been missen in the first attempts or had been introduced afterwards.

Refs #13760, #10080, #17799.