UserUmaskOption: Implement validate_value instead of set_value

set_value() is implemented by the base class, subclasses only need to
override validate_value().
This commit is contained in:
Xavier Claessens 2018-05-13 10:36:58 -04:00 committed by Nirbheek Chauhan
parent 75cce9e157
commit 531722229b
1 changed files with 4 additions and 5 deletions

View File

@ -109,11 +109,10 @@ class UserUmaskOption(UserIntegerOption):
def __init__(self, name, description, value, yielding=None):
super().__init__(name, description, 0, 0o777, value, yielding)
def set_value(self, newvalue):
if newvalue is None or newvalue == 'preserve':
self.value = None
else:
super().set_value(newvalue)
def validate_value(self, value):
if value is None or value == 'preserve':
return None
return super().validate_value(value)
def toint(self, valuestring):
try: