Merge lp:~brian-murray/arsenal/days-with-importance into lp:~bryce/arsenal/2.x

[Brian Murray]

I discovered that you can't search on importance with collect-bug-data so I fixed that. I also added in searching for how long a bug has had it's importance set.

[Bryce Harrington (bryce)]

Looks good

review: Approve

=== modified file 'scripts/collect-bug-data'
--- scripts/collect-bug-data	2012-07-10 16:15:03 +0000
+++ scripts/collect-bug-data	2012-07-16 19:55:22 +0000
@@ -307,6 +307,12 @@
         Dbg.verbose('search: status: \'%s\'\n' % ','.join(search_status))
 
         try:
+            search_importance = search_criteria['importance']
+        except KeyError:
+            search_importance = []
+        Dbg.verbose('search: importance: \'%s\'\n' % ','.join(search_importance))
+
+        try:
             search_order = search_criteria['order_by']
         except KeyError:
             search_order = '-datecreated'
@@ -344,6 +350,11 @@
         except KeyError:
             statuslength = False
 
+        try:
+            implength = search_criteria['days-with-importance']
+        except KeyError:
+            implength = False
+
         # ensure that only one of these are selected
         count = 0
         if search_assignees != '':
@@ -362,6 +373,7 @@
         search_args = {'tags': search_tags,
                 'tags_combinator': search_tags_combinator,
                 'status': search_status,
+                'importance': search_importance,
                 'modified_since': last_run_time,
                 'order_by': search_order}
 
@@ -376,6 +388,8 @@
                         tasks.set_filter_reopened()
                     if statuslength:
                         tasks.set_filter_status_length(statuslength)
+                    if implength:
+                        tasks.set_filter_status_length(implength)
                     reported_tasks.append(tasks)
                 else:
                     for participant in search_reporter.participants:
@@ -387,6 +401,8 @@
                             tasks.set_filter_reopened()
                         if statuslength:
                             tasks.set_filter_status_length(statuslength)
+                        if implength:
+                            tasks.set_filter_status_length(implength)
                         for task in tasks:
                             reported_tasks.append(task)
                 if search_limit:
@@ -404,6 +420,8 @@
                         tasks.set_filter_reopened()
                     if statuslength:
                         tasks.set_filter_status_length(statuslength)
+                    if implength:
+                        tasks.set_filter_status_length(implength)
                     assigned_tasks.append(tasks)
                     search_assignee_names.append(search_assignee.name)
                 else:
@@ -416,6 +434,8 @@
                             tasks.set_filter_reopened()
                         if statuslength:
                             tasks.set_filter_status_length(statuslength)
+                        if implength:
+                            tasks.set_filter_status_length(implength)
                         for task in tasks:
                             assigned_tasks.append(task)
                 if search_limit:
@@ -434,6 +454,8 @@
                     tasks.set_filter_reopened()
                 if statuslength:
                     tasks.set_filter_status_length(statuslength)
+                if implength:
+                    tasks.set_filter_status_length(implength)
                 if search_limit:
                     return tasks[:search_limit]
                 else:
@@ -444,6 +466,8 @@
                 tasks.set_filter_reopened()
             if statuslength:
                 tasks.set_filter_status_length(statuslength)
+            if implength:
+                tasks.set_filter_status_length(implength)
             if search_limit:
                 return tasks[:search_limit]
             else: