Launchpad itself

Wire up getSubscriptionsForBug() into the notifications machinery

Bug #655567 reported by Gavin Panella
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Gavin Panella
Launchpad itself 10.11 "Tenacious Turkey"

Bug Description

Currently getSubscriptionsForBug() sits alone, unused. It's time to
get the bug notification machinery to use it instead of the old boring
IStructuralSubscriptionTarget methods.

Related branches

lp:~allenap/launchpad/wire-up-filter-subs-bug-655567
Graham Binns (community): Approve (code)
Diff: 547 lines (+165/-159)
9 files modified
database/schema/security.cfg (+7/-0)
lib/lp/bugs/configure.zcml (+1/-0)
lib/lp/bugs/interfaces/bug.py (+8/-2)
lib/lp/bugs/model/bug.py (+61/-36)
lib/lp/bugs/model/tests/test_bug.py (+87/-1)
lib/lp/registry/doc/structural-subscriptions.txt (+0/-55)
lib/lp/registry/interfaces/person.py (+0/-11)
lib/lp/registry/model/person.py (+0/-53)
utilities/format-new-and-modified-imports (+1/-1)
lp:~allenap/launchpad/wire-up-filter-subs-bug-655567-devel
Abel Deuring (community): Approve (code)
Diff: 881 lines (+239/-250)
12 files modified
database/schema/security.cfg (+7/-0)
lib/lp/bugs/configure.zcml (+1/-0)
lib/lp/bugs/interfaces/bug.py (+8/-2)
lib/lp/bugs/model/bug.py (+61/-36)
lib/lp/bugs/model/tests/test_bug.py (+91/-9)
lib/lp/registry/doc/structural-subscriptions.txt (+0/-55)
lib/lp/registry/interfaces/person.py (+0/-11)
lib/lp/registry/interfaces/structuralsubscription.py (+2/-2)
lib/lp/registry/model/person.py (+0/-53)
lib/lp/registry/model/structuralsubscription.py (+5/-19)
lib/lp/registry/tests/test_structuralsubscriptiontarget.py (+63/-62)
utilities/format-new-and-modified-imports (+1/-1)
Gavin Panella (allenap)
tags: added: story-subscribe-to-search
Revision history for this message
Launchpad QA Bot (lpqabot) wrote : Bug fixed by a commit

Fixed in db-stable r9888 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/db-stable/revision/9888>.

Changed in malone:
milestone: 10.10 → 10.11
tags: added: qa-needstesting
Changed in malone:
status: In Progress → Fix Committed
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

Fixed in stable r11721 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/11721>.

Revision history for this message
Gavin Panella (allenap) wrote :

Seems to be working okay, but I noticed bug 663947 as I went along.

tags: added: qa-ok
removed: qa-needstesting
Robert Collins (lifeless)
Changed in malone:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.