Launchpad itself

Merge lp://staging/~gmb/launchpad/fix-update-bug-watch into lp://staging/launchpad

fix-update-bug-watch
Merge into devel

Proposed by Graham Binns on 2010-12-14

Status:

Merged

Approved by:

Gavin Panella on 2010-12-14

Approved revision:

no longer in the source branch.

Merged at revision:

12069

Proposed branch:

lp://staging/~gmb/launchpad/fix-update-bug-watch

Merge into:

lp://staging/launchpad

Diff against target:

132 lines (+25/-22)

2 files modified

lib/lp/bugs/scripts/checkwatches/bugwatchupdater.py (+4/-9)
lib/lp/bugs/scripts/checkwatches/tests/test_bugwatchupdater.py (+21/-13)

To merge this branch:

bzr merge lp://staging/~gmb/launchpad/fix-update-bug-watch

Related bugs:

Bug #568505: BugWatchUpdater.updateBugWatch() shouldn't need 10 parameters	High	Fix Released
Bug #578714: BugWatchUpdater.updateBugWatch() shouldn't take can_foo parameters	Medium	Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
Gavin Panella (community)		2010-12-14	Approve on 2010-12-14
Review via email: mp+43643@code.staging.launchpad.net

Commit message

[r=allenap][ui=none][bug=568505,578714] Extraneous parameters have been removed from BugWatchUpdater.updateBugWatch().

Description of the change

This branch fixes the linked bugs by removing the extraneous parameters
from BugWatchUpdater.updateBugWatch(). I've also altered the tests for
that method.

Revision history for this message