diff options
| author | xarx00 <xarx@sms.cz> | 2019-03-04 15:15:51 +0100 |
|---|---|---|
| committer | Max Wittig <max.wittig95@gmail.com> | 2019-07-21 18:53:34 +0200 |
| commit | 8d1552a0ad137ca5e14fabfc75f7ca034c2a78ca (patch) | |
| tree | f0f3894ae8f72b157cc05708884de9ee1e846619 | |
| parent | 8306ef21be731336c9706c9908133cfcb3b6a5f4 (diff) | |
| download | gitlab-8d1552a0ad137ca5e14fabfc75f7ca034c2a78ca.tar.gz | |
fix(cli): don't fail when the short print attr value is None
Fixes #717
Fixes #727
| -rw-r--r-- | gitlab/cli.py | 4 | ||||
| -rw-r--r-- | gitlab/tests/test_cli.py | 2 | ||||
| -rw-r--r-- | gitlab/v4/cli.py | 4 |
3 files changed, 5 insertions, 5 deletions
diff --git a/gitlab/cli.py b/gitlab/cli.py index 0433a81..01d8851 100644 --- a/gitlab/cli.py +++ b/gitlab/cli.py @@ -182,7 +182,7 @@ def main(): if args.fields: fields = [x.strip() for x in args.fields.split(",")] debug = args.debug - action = args.action + action = args.whaction what = args.what args = args.__dict__ @@ -193,7 +193,7 @@ def main(): "verbose", "debug", "what", - "action", + "whaction", "version", "output", ): diff --git a/gitlab/tests/test_cli.py b/gitlab/tests/test_cli.py index bc49d8b..1485499 100644 --- a/gitlab/tests/test_cli.py +++ b/gitlab/tests/test_cli.py @@ -116,7 +116,7 @@ class TestV4CLI(unittest.TestCase): parser = cli._get_parser(gitlab.v4.cli) args = parser.parse_args(["project", "list"]) self.assertEqual(args.what, "project") - self.assertEqual(args.action, "list") + self.assertEqual(args.whaction, "list") def test_parser(self): parser = cli._get_parser(gitlab.v4.cli) diff --git a/gitlab/v4/cli.py b/gitlab/v4/cli.py index f0ed199..6fc41ac 100644 --- a/gitlab/v4/cli.py +++ b/gitlab/v4/cli.py @@ -312,7 +312,7 @@ def extend_parser(parser): object_group = subparsers.add_parser(arg_name) object_subparsers = object_group.add_subparsers( - title="action", dest="action", help="Action to execute." + title="action", dest="whaction", help="Action to execute." ) _populate_sub_parser_by_class(cls, object_subparsers) object_subparsers.required = True @@ -406,7 +406,7 @@ class LegacyPrinter(object): id = getattr(obj, obj._id_attr) print("%s: %s" % (obj._id_attr.replace("_", "-"), id)) if hasattr(obj, "_short_print_attr"): - value = getattr(obj, obj._short_print_attr) + value = getattr(obj, obj._short_print_attr) or "None" value = value.replace("\r", "").replace("\n", " ") # If the attribute is a note (ProjectCommitComment) then we do # some modifications to fit everything on one line |