Merge lp:~gmb/launchpad/refactor-bugnotification-recipient-rationales-bug-594208 into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Graham Binns |
Approved revision: | no longer in the source branch. |
Merged at revision: | 11040 |
Proposed branch: | lp:~gmb/launchpad/refactor-bugnotification-recipient-rationales-bug-594208 |
Merge into: | lp:launchpad |
Diff against target: |
418 lines (+173/-150) 8 files modified
lib/canonical/launchpad/mailnotification.py (+9/-144) lib/lp/bugs/configure.zcml (+1/-1) lib/lp/bugs/doc/bug-change.txt (+1/-1) lib/lp/bugs/doc/bugnotificationrecipients.txt (+1/-1) lib/lp/bugs/mail/bugnotificationrecipients.py (+158/-0) lib/lp/bugs/model/bug.py (+1/-1) lib/lp/bugs/scripts/tests/test_bugnotification.py (+1/-1) lib/lp/registry/doc/structural-subscriptions.txt (+1/-1) |
To merge this branch: | bzr merge lp:~gmb/launchpad/refactor-bugnotification-recipient-rationales-bug-594208 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jeroen T. Vermeulen (community) | code | Approve | |
Review via email: mp+27687@code.launchpad.net |
Commit message
BugNotification
Description of the change
This branch moves BugNotification
This turned out to be a much easier branch than I anticipated. I thought that some refactoring would be needed to make BugNotification
I've moved the class out of mailnotification.py and updated all the import sites I could find. I've added an XXX to mailnotification.py for the import of NotificationRec
Fine with me. As discussed on IRC, there's plenty more work to be done on the code that's moving (dealing with supervisors of duplicate bugs, factoring out the repetition in text composition, replacing reason = reason + x with reason += x) but as you point out that's best left to later branches after first moving the code out of conflicts' way.