This mostly looks good; I just have one question below.
> === modified file 'lib/lp/registry/browser/pillar.py'
> --- lib/lp/registry/browser/pillar.py 2012-03-21 14:30:16 +0000
> +++ lib/lp/registry/browser/pillar.py 2012-03-22 13:04:20 +0000
> @@ -303,15 +303,17 @@
> size=config.launchpad.default_batch_size,
> range_factory=StormRangeFactory(sharees))
>
> - def shareeData(self):
> - """Return an `ITableBatchNavigator` for sharees."""
> + @property
> + def sharees(self):
> + """An `IBatchNavigator` for sharees."""
> if self._batch_navigator is None:
> - unbatchedSharees = self.unbatchedShareeData()
> + unbatchedSharees = self.unbatched_sharees
> self._batch_navigator = self._getBatchNavigator(unbatchedSharees)
> return self._batch_navigator
>
> - def unbatchedShareeData(self):
> - """Return all the sharees for a pillar."""
> + @property
> + def unbatched_sharees(self):
> + """All the sharees for a pillar."""
> return self._getSharingService().getPillarSharees(self.context)
>
> def initialize(self):
I'm not sure I understand the logic of making these properties; don't we
usually avoid that for things that might be expensive or slow, like
unbatched_sharees?
Hey Ian--
This mostly looks good; I just have one question below.
> === modified file 'lib/lp/ registry/ browser/ pillar. py' registry/ browser/ pillar. py 2012-03-21 14:30:16 +0000 registry/ browser/ pillar. py 2012-03-22 13:04:20 +0000 launchpad. default_ batch_size, StormRangeFacto ry(sharees) ) igator` for sharees.""" navigator is None: hareeData( ) sharees navigator = self._getBatchN avigator( unbatchedSharee s) navigator Data(self) : sharees( self): gService( ).getPillarShar ees(self. context)
> --- lib/lp/
> +++ lib/lp/
> @@ -303,15 +303,17 @@
> size=config.
> range_factory=
>
> - def shareeData(self):
> - """Return an `ITableBatchNav
> + @property
> + def sharees(self):
> + """An `IBatchNavigator` for sharees."""
> if self._batch_
> - unbatchedSharees = self.unbatchedS
> + unbatchedSharees = self.unbatched_
> self._batch_
> return self._batch_
>
> - def unbatchedSharee
> - """Return all the sharees for a pillar."""
> + @property
> + def unbatched_
> + """All the sharees for a pillar."""
> return self._getSharin
>
> def initialize(self):
I'm not sure I understand the logic of making these properties; don't we
usually avoid that for things that might be expensive or slow, like
unbatched_sharees?