allow_empty_list)
if multi is not False and default is None:
default = []
- self.impl_validate(default, is_multi=is_multi)
+ try:
+ self.impl_validate(default, is_multi=is_multi)
+ except ContextError:
+ pass
self._set_default_values(default, default_multi, is_multi)
##callback is False in optiondescription
if callback is not False:
descr = context.cfgimpl_get_description()
all_cons_vals = []
- for opt in all_cons_opts:
- try:
+ try:
+ for opt in all_cons_opts:
#get value
if (isinstance(opt, DynSymLinkOption) and option._dyn == opt._dyn) or \
option == opt:
option == opt:
all_cons_vals.append(opt_value)
elif self.impl_is_submulti():
- try:
- all_cons_vals.append(opt_value[index][submulti_index])
- except IndexError, err:
- log.debug('indexerror in submulti opt in _launch_consistency: {0}'.format(err))
- #value is not already set, could be higher index
- #so return if no value
- return
+ all_cons_vals.append(opt_value[index][submulti_index])
else:
- try:
- all_cons_vals.append(opt_value[index])
- except IndexError, err:
- #value is not already set, could be higher index
- #so return if no value and not default_value
- log.debug('indexerror in _launch_consistency: {0}'.format(err))
- return
- except PropertiesOptionError as err:
- log.debug('propertyerror in _launch_consistency: {0}'.format(err))
- if transitive:
- raise err
- else:
- return
+ all_cons_vals.append(opt_value[index])
+ except IndexError, err:
+ #value is not already set, could be higher index
+ #so return if no value and not default_value
+ log.debug('indexerror in _launch_consistency: {0}'.format(err))
+ return
+ except PropertiesOptionError as err:
+ log.debug('propertyerror in _launch_consistency: {0}'.format(err))
+ if transitive:
+ raise err
+ else:
+ return
getattr(self, func)(all_cons_opts, all_cons_vals, warnings_only)
def impl_validate(self, value, context=undefined, validate=True,
def impl_is_master_slaves(self, type_='both'):
"""FIXME
"""
- try:
- self._master_slaves
+ master_slaves = getattr(self, '_master_slaves', None)
+ if master_slaves is not None:
if type_ in ('both', 'master') and \
- self._master_slaves.is_master(self):
+ master_slaves.is_master(self):
return True
if type_ in ('both', 'slave') and \
- not self._master_slaves.is_master(self):
+ not master_slaves.is_master(self):
return True
- except:
- pass
return False
def impl_get_master_slaves(self):
unknown_keys,
valid_keys))
# prepare all attributes
- try:
- option = require['option']
- expected = require['expected']
- action = require['action']
- except KeyError: # pragma: optional cover
+ if 'option' not in require or 'expected' not in require or \
+ 'action' not in require:
raise ValueError(_("malformed requirements for option: {0}"
" require must have option, expected and"
" action keys").format(name))
+ option = require['option']
+ expected = require['expected']
+ action = require['action']
if action == 'force_store_value': # pragma: optional cover
raise ValueError(_("malformed requirements for option: {0}"
" action cannot be force_store_value"
def _impl_setstate(self, descr):
self._impl_setopt(descr.impl_get_opt_by_path(self._state_opt))
del(self._state_opt)
- try:
- del(self._stated)
- except AttributeError: # pragma: optional cover
- pass
+ del(self._stated)
self._set_readonly(True)
def impl_get_information(self, key, default=undefined):