Merge lp:~brian-murray/arsenal/reset-recorded-tasks into lp:~bryce/arsenal/2.x

Proposed by Brian Murray
Status: Merged
Merged at revision: 1120
Proposed branch: lp:~brian-murray/arsenal/reset-recorded-tasks
Merge into: lp:~bryce/arsenal/2.x
Diff against target: 12 lines (+2/-0)
1 file modified
scripts/collect-bug-data (+2/-0)
To merge this branch: bzr merge lp:~brian-murray/arsenal/reset-recorded-tasks
Reviewer Review Type Date Requested Status
Bryce Harrington Pending
Review via email: mp+143426@code.launchpad.net

Description of the change

I discovered an issue with collect-bug-data where it was trying to used cached bugs that had tasks that were empty. This fixes that.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'scripts/collect-bug-data'
--- scripts/collect-bug-data 2012-12-14 19:03:29 +0000
+++ scripts/collect-bug-data 2013-01-16 00:00:35 +0000
@@ -716,6 +716,8 @@
716 Dbg.verbose("LP: #%s removed unassigned task %s\n" % (bug_number, task.web_link))716 Dbg.verbose("LP: #%s removed unassigned task %s\n" % (bug_number, task.web_link))
717 recorded_task.clear()717 recorded_task.clear()
718 continue718 continue
719 # reset recorded_tasks to exclude those that have been cleared
720 recorded_tasks[bug_number] = [recorded_task for task in recorded_tasks[bug_number] if recorded_task]
719 # if the bug_number no longer has any recorded tasks remove it721 # if the bug_number no longer has any recorded tasks remove it
720 if all(len(recorded_task)==0 for recorded_task in recorded_tasks[bug_number]):722 if all(len(recorded_task)==0 for recorded_task in recorded_tasks[bug_number]):
721 Dbg.verbose("LP: #%s has been removed because it has no tasks\n" % bug_number)723 Dbg.verbose("LP: #%s has been removed because it has no tasks\n" % bug_number)

Subscribers

People subscribed via source and target branches

to all changes: