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.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to