Thanks for the feedback. I changed this now. *addon - plugin.image.photoapp *version - 1.0.2 *url - https://github.com/PhotoAppPlugin/plugin.image.photoapp/tree/v1.0.2 *revision - tag v1.0.2 *branch - master *xbmc version - isengard
> Am 28.11.2015 um 15:57 schrieb ronie <ro...@kodi.tv>: > >> On 17-11-15 00:45, Claude Marksteiner wrote: >> *addon - plugin.image.photoapp >> *version - 1.0.1 >> *url -https://github.com/PhotoAppPlugin/plugin.image.photoapp/tree/v1.0.1 >> *revision - tag v1.0.1 >> *branch - master >> *xbmc version - isengard > > hey Claude, > > thanx for submitting your addon to our repo! > > i've reviewed your code and have two minor change requests: > 1) there's no need to put 'import xbmcvfs' in a try/except, > this module is hardcoded in kodi and can't be removed, it will always be > available. > > 2) use 'addon = xbmcaddon.Addon()', there's no need to specify the id there, > it will be auto-resolved by kodi. > > cheers, > ronie ------------------------------------------------------------------------------ _______________________________________________ Xbmc-addons mailing list Xbmc-addons@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/xbmc-addons