From dd16cc98b0d67f1448bf9de25f8adce512b1431c Mon Sep 17 00:00:00 2001 From: Petr Viktorin Date: Mon, 10 Aug 2015 18:29:33 +0200 Subject: Use six.string_types instead of "basestring" Reviewed-By: Christian Heimes Reviewed-By: Jan Cholasta --- ipalib/plugins/baseuser.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'ipalib/plugins/baseuser.py') diff --git a/ipalib/plugins/baseuser.py b/ipalib/plugins/baseuser.py index 5eede7a98..c3e290371 100644 --- a/ipalib/plugins/baseuser.py +++ b/ipalib/plugins/baseuser.py @@ -22,6 +22,8 @@ import string import posixpath import os +import six + from ipalib import api, errors from ipalib import Flag, Int, Password, Str, Bool, StrEnum, DateTime, Bytes from ipalib.plugable import Registry @@ -89,7 +91,7 @@ def validate_nsaccountlock(entry_attrs): if 'nsaccountlock' in entry_attrs: nsaccountlock = entry_attrs['nsaccountlock'] if not isinstance(nsaccountlock, (bool, Bool)): - if not isinstance(nsaccountlock, basestring): + if not isinstance(nsaccountlock, six.string_types): raise errors.OnlyOneValueAllowed(attr='nsaccountlock') if nsaccountlock.lower() not in ('true', 'false'): raise errors.ValidationError(name='nsaccountlock', @@ -400,7 +402,7 @@ class baseuser(LDAPObject): if not isinstance(email, (list, tuple)): email = [email] for m in email: - if isinstance(m, basestring): + if isinstance(m, six.string_types): if '@' not in m and defaultdomain: m = m + u'@' + defaultdomain if not Email(m): -- cgit