Merge lp://staging/~gary/launchpad/bug723999-2c into lp://staging/launchpad
Status: | Merged |
---|---|
Merged at revision: | 12548 |
Proposed branch: | lp://staging/~gary/launchpad/bug723999-2c |
Merge into: | lp://staging/launchpad |
Prerequisite: | lp://staging/~gary/launchpad/bug723999-2b |
Diff against target: |
685 lines (+235/-218) 5 files modified
lib/lp/bugs/doc/bugsubscription.txt (+9/-2) lib/lp/bugs/interfaces/bug.py (+5/-5) lib/lp/bugs/model/bug.py (+68/-45) lib/lp/bugs/model/structuralsubscription.py (+151/-120) lib/lp/bugs/subscribers/bug.py (+2/-46) |
To merge this branch: | bzr merge lp://staging/~gary/launchpad/bug723999-2c |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Benji York (community) | code | Approve | |
Review via email:
|
Description of the change
This branch is slightly meatier than than the previous two in the series, and we're starting to see the light at the end of the tunnel.
This branch makes the following changes.
- I factored the code of bug.getAlsoNoti
- I made the function accept a bugtask or a bug, rather than only a bug. This meant that I could eliminate an essentially identical function of lp.bugs.
- I made the get_also_
- I made the get_also_
- I made the get_also_
- I made get_structural_
- I pulled out two functions from the _get_structural
Note that I tried to make lint completely happy (particularly notable is that lint is what caused me to move the XXX comment in lib/lp/
./lib/lp/
435: E301 expected 1 blank line, found 0
Line 435 is the one below beginning with "@operation_
...
def newMessage(owner, subject, content):
"""Create a new message, and link it to this object."""
@operation_
...
I don't see what the problem is. I've decided to ignore it.
Thank you!
Gary
This branch is good to go. The descriptive and thorough introduction
helped quite a bit in understanding the changes.
As for the lint output, I determined that the comment embedded in the
decorator call is the culprit. I suggest filing a bug against
pocketlint and/or whichever Python linter it's using.