diff options
| author | Jan Cholasta <jcholast@redhat.com> | 2016-05-30 08:12:32 +0200 |
|---|---|---|
| committer | Jan Cholasta <jcholast@redhat.com> | 2016-06-03 09:00:34 +0200 |
| commit | 748a4c31bf9018482b411578127eada176f47cec (patch) | |
| tree | d34738fd860ed476d403ebc4c22db5d7b81582fd | |
| parent | e9f0e9d8fafb35acf0adbd99d40c16edf510ff9f (diff) | |
| download | freeipa-748a4c31bf9018482b411578127eada176f47cec.tar.gz freeipa-748a4c31bf9018482b411578127eada176f47cec.tar.xz freeipa-748a4c31bf9018482b411578127eada176f47cec.zip | |
cli: make optional positional command arguments actually optional
Fix commands defined in ipalib.cli not to assume optional positional
arguments have a value of None when not specified.
https://fedorahosted.org/freeipa/ticket/4739
Reviewed-By: David Kupka <dkupka@redhat.com>
| -rw-r--r-- | ipalib/cli.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipalib/cli.py b/ipalib/cli.py index 1c88a9b1f..10c1fe4b8 100644 --- a/ipalib/cli.py +++ b/ipalib/cli.py @@ -773,7 +773,7 @@ class help(frontend.Local): super(help, self)._on_finalize() - def run(self, key, outfile=None, **options): + def run(self, key=None, outfile=None, **options): if outfile is None: outfile = sys.stdout writer = self._writer(outfile) @@ -923,7 +923,7 @@ class show_api(frontend.Command): takes_args = ('namespaces*',) - def run(self, namespaces): + def run(self, namespaces=None): if namespaces is None: names = tuple(self.api) else: |