Merge lp:~danilo/launchpad-work-items-tracker/bug-1124087 into lp:~linaro-automation/launchpad-work-items-tracker/linaro

[Данило Шеган]

Concatenate component (actually team) names when a card lists multiple components like http://cards.linaro.org/browse/CARD-372

[Stevan Radaković (stevanr)]

Looks good, thanks.
Approve +1

review: Approve

=== modified file 'collect_jira'
--- collect_jira	2013-01-04 10:46:26 +0000
+++ collect_jira	2013-02-13 13:02:21 +0000
@@ -55,6 +55,11 @@
     return data
 
 
+def construct_team_from_components(components):
+    '''Construct a team name from multiple component['name'] values.'''
+    assert len(components) > 0, "Cards with no components should be ignored."
+    return u', '.join([team['name'] for team in components])
+
 def jira_import(collector, cfg, opts):
     '''Collect roadmap items from JIRA into DB.'''
 
@@ -103,9 +108,10 @@
         if len(components) == 0:
             dbg('Skipping card without component (name = %s, key = %s)' %
                 (name, key))
-        # JIRA allows listing multiple components
-        assert len(components) == 1
-        model_card.team = unicode(components[0]['name'])
+            continue
+        # JIRA allows listing multiple components, combine them into a single
+        # team description.
+        model_card.team = construct_team_from_components(components)
         model_card.priority = unicode(fields['priority']['name'])
         size_fields = []
         timetracking = fields['timetracking']