Merge lp:~wgrant/launchpad/unuse-nullbugtask into lp:launchpad
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | William Grant | ||||||||
Approved revision: | no longer in the source branch. | ||||||||
Merged at revision: | 12607 | ||||||||
Proposed branch: | lp:~wgrant/launchpad/unuse-nullbugtask | ||||||||
Merge into: | lp:launchpad | ||||||||
Diff against target: |
479 lines (+43/-198) 6 files modified
lib/lp/bugs/browser/bugnomination.py (+1/-9) lib/lp/bugs/browser/bugtask.py (+6/-130) lib/lp/bugs/browser/configure.zcml (+21/-43) lib/lp/bugs/browser/tests/bug-views.txt (+2/-2) lib/lp/bugs/browser/tests/bugtask-edit-views.txt (+13/-13) lib/lp/bugs/templates/bugtask-index.pt (+0/-1) |
||||||||
To merge this branch: | bzr merge lp:~wgrant/launchpad/unuse-nullbugtask | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Penhey (community) | code | Approve | |
Steve Kowalik (community) | code* | Approve | |
Review via email: mp+53363@code.launchpad.net |
Commit message
[r=stevenk,
Description of the change
NullBugTask was created to provide a button to report a bug in a new context, if the bug was navigated to below a context for which a task did not yet exist. In early 2009 this behaviour was largely removed; non-existent tasks now redirect to the default task instead.
But one case was missed: DistributionSou
This branch fixes that case by redirecting during traversal, preventing a NullBugTask from ever being encountered. This allows us to remove a lot of special cases, mostly trivially. One non-trivial case is BugTaskTraversa
Removal of NullBugTask itself is a little more difficult, so that will be done in a later branch.