Michael Pasternak has uploaded a new change for review. Change subject: cli: do not consider None as valid option ......................................................................
cli: do not consider None as valid option Change-Id: I7d4d48600d6e5f30d2cac28f6a22f6a8626b70a4 Signed-off-by: Michael pasternak <[email protected]> --- M src/ovirtcli/utils/autocompletionhelper.py 1 file changed, 3 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine-cli refs/changes/33/21233/1 diff --git a/src/ovirtcli/utils/autocompletionhelper.py b/src/ovirtcli/utils/autocompletionhelper.py index 37146fe..7c2124d 100644 --- a/src/ovirtcli/utils/autocompletionhelper.py +++ b/src/ovirtcli/utils/autocompletionhelper.py @@ -56,8 +56,9 @@ @staticmethod def _resolve_value(val, common_options=[], specific_options={}, specific_arguments={}): + if val == 'None': return val return val + ( - '-identifier ' if val not in common_options \ + '-identifier' if val not in common_options \ and (not AutoCompletionHelper._is_verb_in_dict_values( specific_options, val ) @@ -113,7 +114,7 @@ obj = spl[1].strip() repl = AutoCompletionHelper._get_verb_replecations(mp[obj], s_text) i_completions = [('--' if (not AutoCompletionHelper._is_verb_in_dict_values(specific_arguments, f)) else '') - + f + + f + ' ' if text in mp[obj] or repl == 1 or len(mp[obj]) == 1 or (len(mp[obj]) == 2 and 'None' in mp[obj]) == 1 else f -- To view, visit http://gerrit.ovirt.org/21233 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I7d4d48600d6e5f30d2cac28f6a22f6a8626b70a4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
