← Back to branch summary

~launchpad-pqm/launchpad/devel

« back to all changes in this revision

Viewing changes to lib/lp/bugs/browser/bugsubscription.py

  • Committer: Brad Crittenden
  • Date: 2011-07-21 18:07:26 UTC
  • mto: This revision was merged to the branch mainline in revision 13487.
  • Revision ID: bac@canonical.com-20110721180726-905vbp94smtqfc7f
Incorporate brilliant suggestions from review.

Show diffs side-by-side

added added

removed removed

Lines of Context:
lib/lp/bugs/browser/bugsubscription.py
49
49
from lp.bugs.enum import BugNotificationLevel
50
50
from lp.bugs.interfaces.bug import IBug
51
51
from lp.bugs.interfaces.bugsubscription import IBugSubscription
52
 
from lp.bugs.interfaces.bugtask import IBugTask
53
52
from lp.bugs.model.personsubscriptioninfo import PersonSubscriptions
54
53
from lp.bugs.model.structuralsubscription import (
55
54
    get_structural_subscriptions_for_bug,
555
554
                'self_link': absoluteURL(person, self.api_request),
556
555
                'is_team': person.is_team,
557
556
                'can_edit': can_edit,
558
 
                'subscribed_by': subscription.display_subscribed_by,
 
557
                'display_subscribed_by': subscription.display_subscribed_by,
559
558
                }
560
559
            record = {
561
560
                'subscriber': subscriber,
568
567
    @property
569
568
    def subscriber_data_js(self):
570
569
        """Return subscriber_ids in a form suitable for JavaScript use."""
571
 
        if IBug.providedBy(self.context):
572
 
            bug = self.context
573
 
        elif IBugTask.providedBy(self.context):
574
 
            bug = self.context.bug
575
 
 
 
570
        bug = IBug(self.context)
576
571
        data = self.direct_subscriber_data(bug)
577
572
 
578
573
        others = list(bug.getIndirectSubscribers())