From ff94fc795ee048dcfb91e20b659fdc6069fc28e9 Mon Sep 17 00:00:00 2001 From: Clement Verna Date: Oct 19 2016 18:32:38 +0000 Subject: Nicer way to check if we have milestone --- diff --git a/pagure_importer/utils/importer_trac.py b/pagure_importer/utils/importer_trac.py index 36b17db..dce5a13 100644 --- a/pagure_importer/utils/importer_trac.py +++ b/pagure_importer/utils/importer_trac.py @@ -129,11 +129,8 @@ class TracImporter(): pagure_issue_tags = filter( lambda x: x != '', trac_ticket['keywords'].split(' ')) - try: - if trac_ticket['milestone'] != '': - pagure_issue_tags.append(str(trac_ticket['milestone'])) - except KeyError: - pass + if 'milestone' in trac_ticket and trac_ticket['milestone'] != '': + pagure_issue_tags.append(str(trac_ticket['milestone'])) # Remove ',' between the tags and convert all tags to lower case pagure_issue_tags = list(set([j for k in [i.lower().split(',')