Author evan_
Recipients evan_
Date 2016-06-05.08:59:16
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1465117156.8.0.428303445441.issue27227@psf.upfronthosting.co.za>
In-reply-to
Content
When using nargs='*' with choices, it is impossible to specify 0 args:

from argparse import ArgumentParser
parser = ArgumentParser()
parser.add_argument('foo', choices=['foo'], nargs='*')
args = parser.parse_args([])  # <-- fails, error message below
assert args.foo == []
# usage: args.py [-h] [{foo} [{foo} ...]]
# args.py: error: argument foo: invalid choice: [] (choose from 'foo')

The problem appears to be this block of code trying to validate `value` immediately after it sets it to `[]`:

        # when nargs='*' on a positional, if there were no command-line
        # args, use the default if it is anything other than None
        elif (not arg_strings and action.nargs == ZERO_OR_MORE and
              not action.option_strings):
            if action.default is not None:
                value = action.default
            else:
                value = arg_strings
            self._check_value(action, value)

The fix seems to be as simple as moving the check under `if action.default is not None`.

(NOTE: This would be also adequately solved by patches already attached to http://bugs.python.org/issue9625, however the minimal solution to this problem is simpler.)
History
Date User Action Args
2016-06-05 08:59:16evan_setrecipients: + evan_
2016-06-05 08:59:16evan_setmessageid: <1465117156.8.0.428303445441.issue27227@psf.upfronthosting.co.za>
2016-06-05 08:59:16evan_linkissue27227 messages
2016-06-05 08:59:16evan_create