From 90a942f4ee229e1d4baa25ba4d5d55ee846f2920 Mon Sep 17 00:00:00 2001
From: Mike McLean <mikem@redhat.com>
Date: Oct 18 2018 11:57:42 +0000
Subject: PR#1127: only pass new incl_blocked call opt if it is explicitly needed


Merges #1127
https://pagure.io/koji/pull-request/1127

Fixes: #1126
https://pagure.io/koji/issue/1126
list-groups cli command fails on older server

---

diff --git a/cli/koji_cli/commands.py b/cli/koji_cli/commands.py
index 5b89b0c..df4600d 100644
--- a/cli/koji_cli/commands.py
+++ b/cli/koji_cli/commands.py
@@ -2584,7 +2584,9 @@ def anon_handle_list_groups(goptions, session, args):
     if len(args) < 1 or len(args) > 2:
         parser.error(_("Incorrect number of arguments"))
         assert False  # pragma: no cover
-    opts = {'incl_blocked': options.incl_blocked}
+    opts = {}
+    if options.incl_blocked:
+        opts['incl_blocked'] = True
     activate_session(session, goptions)
     event = koji.util.eventFromOpts(session, options)
     if event:
@@ -5107,7 +5109,10 @@ def anon_handle_show_groups(goptions, session, args):
         parser.error(_("--show-blocked doesn't make sense for comps/spec output"))
     activate_session(session, goptions)
     tag = args[0]
-    groups = session.getTagGroups(tag, incl_blocked=options.incl_blocked)
+    callopts = {}
+    if options.incl_blocked:
+        callopts['incl_blocked'] = True
+    groups = session.getTagGroups(tag, **callopts)
     if options.comps:
         print(koji.generate_comps(groups, expand_groups=options.expand))
     elif options.spec: