mirror of
https://salsa.debian.org/freeipa-team/freeipa.git
synced 2025-02-03 12:11:25 -06:00
Fix regressions introduced by pylint false positive fixes.
ticket 1198
This commit is contained in:
parent
fceb579d6e
commit
3edb8ed545
@ -383,15 +383,15 @@ class Command(HasParam):
|
|||||||
|
|
||||||
takes_options = tuple()
|
takes_options = tuple()
|
||||||
takes_args = tuple()
|
takes_args = tuple()
|
||||||
args = lambda: None
|
args = None
|
||||||
options = lambda: None
|
options = None
|
||||||
params = None
|
params = None
|
||||||
obj = None
|
obj = None
|
||||||
|
|
||||||
use_output_validation = True
|
use_output_validation = True
|
||||||
output = None
|
output = None
|
||||||
has_output = ('result',)
|
has_output = ('result',)
|
||||||
output_params = lambda: None
|
output_params = None
|
||||||
has_output_params = tuple()
|
has_output_params = tuple()
|
||||||
|
|
||||||
msg_summary = None
|
msg_summary = None
|
||||||
@ -464,10 +464,10 @@ class Command(HasParam):
|
|||||||
>>> list(c._repr_iter(login=u'Okay.', passwd=u'Private!'))
|
>>> list(c._repr_iter(login=u'Okay.', passwd=u'Private!'))
|
||||||
["u'Okay.'", "passwd=u'********'"]
|
["u'Okay.'", "passwd=u'********'"]
|
||||||
"""
|
"""
|
||||||
for arg in self.args():
|
for arg in self.args(): #pylint: disable=E1102
|
||||||
value = params.get(arg.name, None)
|
value = params.get(arg.name, None)
|
||||||
yield repr(arg.safe_value(value))
|
yield repr(arg.safe_value(value))
|
||||||
for option in self.options():
|
for option in self.options(): #pylint: disable=E1102
|
||||||
if option.name not in params:
|
if option.name not in params:
|
||||||
continue
|
continue
|
||||||
value = params[option.name]
|
value = params[option.name]
|
||||||
@ -492,7 +492,7 @@ class Command(HasParam):
|
|||||||
|
|
||||||
def __args_2_params(self, values):
|
def __args_2_params(self, values):
|
||||||
multivalue = False
|
multivalue = False
|
||||||
for (i, arg) in enumerate(self.args()):
|
for (i, arg) in enumerate(self.args()): #pylint: disable=E1102
|
||||||
assert not multivalue
|
assert not multivalue
|
||||||
if len(values) > i:
|
if len(values) > i:
|
||||||
if arg.multivalue:
|
if arg.multivalue:
|
||||||
@ -771,7 +771,7 @@ class Command(HasParam):
|
|||||||
return 1
|
return 1
|
||||||
return 2
|
return 2
|
||||||
self.params = NameSpace(
|
self.params = NameSpace(
|
||||||
sorted(tuple(self.args()) + tuple(self.options()), key=get_key),
|
sorted(tuple(self.args()) + tuple(self.options()), key=get_key), #pylint: disable=E1102
|
||||||
sort=False
|
sort=False
|
||||||
)
|
)
|
||||||
self.output = NameSpace(self._iter_output(), sort=False)
|
self.output = NameSpace(self._iter_output(), sort=False)
|
||||||
@ -920,7 +920,7 @@ class Command(HasParam):
|
|||||||
|
|
||||||
rv = 0
|
rv = 0
|
||||||
|
|
||||||
order = [p.name for p in self.output_params()]
|
order = [p.name for p in self.output_params()] #pylint: disable=E1102
|
||||||
if options.get('all', False):
|
if options.get('all', False):
|
||||||
order.insert(0, 'dn')
|
order.insert(0, 'dn')
|
||||||
print_all = True
|
print_all = True
|
||||||
@ -930,8 +930,8 @@ class Command(HasParam):
|
|||||||
if options.get('raw', False):
|
if options.get('raw', False):
|
||||||
labels = None
|
labels = None
|
||||||
else:
|
else:
|
||||||
labels = dict((p.name, unicode(p.label)) for p in self.output_params())
|
labels = dict((p.name, unicode(p.label)) for p in self.output_params()) #pylint: disable=E1102
|
||||||
flags = dict((p.name, p.flags) for p in self.output_params())
|
flags = dict((p.name, p.flags) for p in self.output_params()) #pylint: disable=E1102
|
||||||
|
|
||||||
for o in self.output:
|
for o in self.output:
|
||||||
outp = self.output[o]
|
outp = self.output[o]
|
||||||
@ -1020,7 +1020,7 @@ class Object(HasParam):
|
|||||||
backend = None
|
backend = None
|
||||||
methods = None
|
methods = None
|
||||||
properties = None
|
properties = None
|
||||||
params = lambda: None
|
params = None
|
||||||
primary_key = None
|
primary_key = None
|
||||||
params_minus_pk = None
|
params_minus_pk = None
|
||||||
|
|
||||||
@ -1037,7 +1037,7 @@ class Object(HasParam):
|
|||||||
self.__get_attrs('Property'), sort=False, name_attr='attr_name'
|
self.__get_attrs('Property'), sort=False, name_attr='attr_name'
|
||||||
)
|
)
|
||||||
self._create_param_namespace('params')
|
self._create_param_namespace('params')
|
||||||
pkeys = filter(lambda p: p.primary_key, self.params())
|
pkeys = filter(lambda p: p.primary_key, self.params()) #pylint: disable=E1102
|
||||||
if len(pkeys) > 1:
|
if len(pkeys) > 1:
|
||||||
raise ValueError(
|
raise ValueError(
|
||||||
'%s (Object) has multiple primary keys: %s' % (
|
'%s (Object) has multiple primary keys: %s' % (
|
||||||
@ -1048,7 +1048,7 @@ class Object(HasParam):
|
|||||||
if len(pkeys) == 1:
|
if len(pkeys) == 1:
|
||||||
self.primary_key = pkeys[0]
|
self.primary_key = pkeys[0]
|
||||||
self.params_minus_pk = NameSpace(
|
self.params_minus_pk = NameSpace(
|
||||||
filter(lambda p: not p.primary_key, self.params()), sort=False
|
filter(lambda p: not p.primary_key, self.params()), sort=False #pylint: disable=E1102
|
||||||
)
|
)
|
||||||
else:
|
else:
|
||||||
self.params_minus_pk = self.params
|
self.params_minus_pk = self.params
|
||||||
@ -1063,7 +1063,7 @@ class Object(HasParam):
|
|||||||
if len(names) == 1 and not isinstance(names[0], (Param, str)):
|
if len(names) == 1 and not isinstance(names[0], (Param, str)):
|
||||||
names = names[0]
|
names = names[0]
|
||||||
minus = frozenset(names)
|
minus = frozenset(names)
|
||||||
for param in self.params():
|
for param in self.params(): #pylint: disable=E1102
|
||||||
if param.name in minus or param in minus:
|
if param.name in minus or param in minus:
|
||||||
continue
|
continue
|
||||||
yield param
|
yield param
|
||||||
|
@ -138,7 +138,7 @@ class LazyText(object):
|
|||||||
subclasses.
|
subclasses.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
__slots__ = ('domain', 'localedir', 'key')
|
__slots__ = ('domain', 'localedir', 'key', 'args')
|
||||||
|
|
||||||
def __init__(self, domain=None, localedir=None):
|
def __init__(self, domain=None, localedir=None):
|
||||||
"""
|
"""
|
||||||
@ -220,7 +220,7 @@ class Gettext(LazyText):
|
|||||||
`NGettextFactory`.
|
`NGettextFactory`.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
__slots__ = ('msg', 'args')
|
__slots__ = ('msg')
|
||||||
|
|
||||||
def __init__(self, msg, domain=None, localedir=None):
|
def __init__(self, msg, domain=None, localedir=None):
|
||||||
super(Gettext, self).__init__(domain, localedir)
|
super(Gettext, self).__init__(domain, localedir)
|
||||||
@ -366,7 +366,7 @@ class NGettext(LazyText):
|
|||||||
See `NGettextFactory` for additional details.
|
See `NGettextFactory` for additional details.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
__slots__ = ('singular', 'plural', 'args')
|
__slots__ = ('singular', 'plural')
|
||||||
|
|
||||||
def __init__(self, singular, plural, domain=None, localedir=None):
|
def __init__(self, singular, plural, domain=None, localedir=None):
|
||||||
super(NGettext, self).__init__(domain, localedir)
|
super(NGettext, self).__init__(domain, localedir)
|
||||||
|
Loading…
Reference in New Issue
Block a user