screenlets-dev team mailing list archive

[tabbernuk <boamaod@xxxxxxxxx>]

Maybe I get something wrong, but it seems that is not the name/id None
leads to an error, but the value None is the culprit. In case of
FeedReaderPlus it's StringOption with name/id "link_type" which appears
to have None for the value. Actually it should render into list of
selections and is defined like this:

                self.add_option(StringOption(_('Options'), 'link_type',
                        self.link_type, _('Link type'), 
                        _('Which link is followed when you click on an item'),
                        ['item', 'feed', 'off']))

I looked at the code in base.py and seems that the choices demands
explicit definition like choices=['item', 'feed', 'off']. It's no
problem to fix this in FeedReaderPlus, but I'd like to fix it also in
options code for compatibility. By fixing I mean that the argument on
position five of StringOption should render into choices. Which would be
the best place to define it for StringOption?

-- 
You received this bug notification because you are a member of
Screenlets Dev Team, which is subscribed to Screenlets.
https://bugs.launchpad.net/bugs/769795

Title:
  Labels with value None in options rework problematic