← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~bac/launchpad/bug-799901 into lp:launchpad

 

Indeed, Robert is right. For that sole reason I implemented a separate method which gets all the data with one query for this page, and that method needs extending: IBug.getDirectSubscribersWithDetails()

It returns a ResultSet of (Person, Subscription) tuples, and should be extended to return (Person, SubscribedBy, Subscription) tuples where SubscribedBy is a ClassAlias(Person) as appropriate.

Don't worry about changing the method on the model because this is the only place it is used in.

-- 
https://code.launchpad.net/~bac/launchpad/bug-799901/+merge/68590
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~bac/launchpad/bug-799901 into lp:launchpad.


References