On Wed, Jul 01, 2009 at 05:14:09AM -0000 or thereabouts, Tim Bentley wrote:
> Review: Needs Fixing
> Sorry but this will break the other plugins.
>
> adding self.ServiceItemIconName will be required by all plugins now
>
Should I add it to the other plugins then...?
> Also is incorrect.
> - service_item.addIcon(u':/media/media_image.png')
> 92 + service_item.addIcon(self.ServiceItemIconName)
>
> should be
> service_item.addIcon(u':/media/media_' + 'self.ShortPluginName + u'image.png')
> This does not work for bibles as the media_image for bibles is _verses instead of _bibles.
which would also fix this problem.
>
> Use of assert causes problems and rejections on a number of boxes can we have throws error instead.
On Wed, Jul 01, 2009 at 05:14:09AM -0000 or thereabouts, Tim Bentley wrote: mIconName will be required by all plugins now
> Review: Needs Fixing
> Sorry but this will break the other plugins.
>
> adding self.ServiceIte
>
Should I add it to the other plugins then...?
> Also is incorrect. item.addIcon( u':/media/ media_image. png') item.addIcon( self.ServiceIte mIconName) item.addIcon( u':/media/ media_' + 'self.ShortPlug inName + u'image.png')
> - service_
> 92 + service_
>
> should be
> service_
> This does not work for bibles as the media_image for bibles is _verses instead of _bibles.
which would also fix this problem.
>
> Use of assert causes problems and rejections on a number of boxes can we have throws error instead.
OK.
Cheers,
Martin