> + def addManyTasks(self, owner, targets): > + """See `IBug`.""" > + new_tasks = getUtility(IBugTaskSet).createManyTasks(self, owner, > targets) > + return new_tasks
That method doesn't seem to have any reason to exist... its indirection for no benefit. I suggest either using it from IBugTaskSet or not putting it on IBugTaskSet at all and having the body in IBug. -Rob -- https://code.launchpad.net/~rharding/launchpad/bugnom_874250/+merge/105317 Your team Launchpad code reviewers is subscribed to branch 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
