launchpad-reviewers team mailing list archive

[Steve Kowalik <stevenk@xxxxxxxxxx>]

The proposal to merge lp:~stevenk/launchpad/destroy-getusedbugtags into lp:launchpad has been updated.

Description changed to:

Destroy IBugTarget.getUsedBugTags(), get_bug_tags() -- they are only used in one place in BugEditView's validate method to make sure that new bug tags are explicitly created. Except the function is horrid and has at least XSS hole, perhaps more.

Due to the aforementioned death, BugEditView has been reduced to a shadow of its former self, and its template has been deleted so it makes use of the generic-edit template.

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/destroy-getusedbugtags/+merge/118024
-- 
https://code.launchpad.net/~stevenk/launchpad/destroy-getusedbugtags/+merge/118024
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stevenk/launchpad/destroy-getusedbugtags into lp:launchpad.