call_command() ignores dest from make_option() and uses kwarg name instead

[Markus Amalthea Magnuson]

Let's say I've defined a custom manage command and added an option to it like this:

option_list = BaseCommand.option_list + (
    make_option('--myoption', action='store', dest='my_opt_var'),
)

I can now call this from the command-line like this:

./manage.py mycommand --my_opt_var=abc

And there will now be an entry in the options dictionary: options['my_opt_var']

However, if I call the same command from code:

call_command('mycommand', myoption='abc')

The added entry will instead use the name of the keyword argument: options['myoption']

I would expect call_command() to use what I've specified through the dest parameter of make_option()

[Tim Graham]

Note that we've switched to argparse in master (1.8). Could you check if the issue still exists?

[Markus Amalthea Magnuson]

Yeah, I actually only tried this in 1.7b3, will try it in master and update this ticket with more info.

[Markus Amalthea Magnuson]

Seems like this is still the case in master. These are the steps I did to reproduce:

1. Start a new project and a new app, and add the new app to settings.
2. In the new app, create the path management/commands/my_command.py with all the intermediate __init__.py files.
3. In my_command.py paste the following code:

   from django.core.management.base import BaseCommand
   
   
   class Command(BaseCommand):
   
       def add_arguments(self, parser):
           parser.add_argument('--myflag',
                               action='store',
                               dest='mydest',
                               default=False)
   
       def handle(self, *args, **options):
           print(options)
   

4. From the command-line, run ./manage.py shell, and then run these lines:

   >>> from django.core import management
   >>> management.call_command('my_command', myflag='abc')
   

This will print:

{'settings': None, 'pythonpath': None, 'verbosity': 1, 'traceback': False, 'no_color': False, 'mydest': False, 'myflag': 'abc'}

I would expect it to print:

{'settings': None, 'pythonpath': None, 'verbosity': 1, 'traceback': False, 'no_color': False, 'mydest': 'abc'}

[Tim Graham]

Looks reasonable.

[Areski Belaid]

You will have similar problem with Django 1.6.5 except that 'mydest' will be None instead of False.

[Claude Paroz]

​https://github.com/django/django/pull/3038

[Claude Paroz <claude@…>]

In 2cc8ffe258008096a70791115b2daa12f0ef0192:

Fixed #22985 -- Made call_command accept option name parameter

Thanks giulettamasina for the report and Tim Graham for the review.