use more descriptive print_fas_names argument

This is because both users and groups can (co-)maintain components.

Signed-off-by: Nils Philippsen <nils@redhat.com>
This commit is contained in:
Nils Philippsen 2019-11-21 15:28:27 +01:00
parent 06b0f23201
commit 26c3288951

View file

@ -213,7 +213,7 @@ class BugzillaProxy:
return self.userCache[username]['bugzilla_email'].lower()
def add_edit_component(self, package, collection, owner, description=None,
qacontact=None, cclist=None, output_user=False):
qacontact=None, cclist=None, print_fas_names=False):
'''Add or update a component to have the values specified.
'''
# Turn the cclist into something usable by bugzilla
@ -284,7 +284,7 @@ class BugzillaProxy:
print(f'[EDITCOMP] {data["product"]}/{data["component"]}')
for key in ["initialowner", "description", "initialqacontact", "initialcclist"]:
if data.get(key):
if output_user and key in ('initialowner', 'initialcclist'):
if print_fas_names and key in ('initialowner', 'initialcclist'):
if key == 'initialowner':
value = owner
else: