On 11/27/2013 04:14 AM, Vratislav Podzimek wrote:
> +
> >+ def _device_valid(self, dev):
> >+ """ Check whether the entered CCW entry is valid.
"""
> >+ widget = self.builder.get_object("deviceEntry")
All objects should be pulled only once, in the refresh method (or in a
separate initialize method).
It should be in an initialize method, since refresh can also be called
more than once. pylint can catch a lot of that kind of thing for you,
which it reports as attributes defined outside of __init__ (we also
allow attributes to be defined in initialize()).
pylint currently complains about _configureGrid, _cancelButton,
_discoveryError, _startButton, _zfcpNotebook, _okButton, _deviceEntry
and _conditionNotebook. Most of those are currently defined in refresh().