crans/scripts
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[ldap_crans.py] string.join est obsolète, tests de types un peu plus corrects

Browse source 
Ignore-this: 1d8276de2456055a393e389f972ca1f6

darcs-hash:20090916205237-108b1-1b6e3fcf5df5d341888bef4844b52588afdaa222.gz
This commit is contained in:
Olivier Iffrig 2009-09-16 22:52:37 +02:00
parent a4455e0286
commit 34787874db
1 changed files with 5 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

12
gestion/ldap_crans.py
View file

@ -166,17 +166,15 @@ def preattr(val):
Retourne [ len(str(val).strip), str(val).strip en utf-8 ] Retourne [ len(str(val).strip), str(val).strip en utf-8 ]
""" """
t = type(val) if isinstance(val, list) and len(val) == 1:
if t == list and len(val) == 1:
return preattr(val[0]) return preattr(val[0])
elif t == str or t == int: elif isinstance(val, str) or isinstance(val, int):
val = str(val).strip() val = str(val).strip()
# On passe tout en utf-8 pour ne pas avoir de problèmes # On passe tout en utf-8 pour ne pas avoir de problèmes
# d'accents dans la base # d'accents dans la base
return [len(val), unicode(val, 'iso-8859-1').encode('utf-8')] return [len(val), unicode(val, 'iso-8859-1').encode('utf-8')]
elif t == unicode: elif isinstance(val, unicode):
val = val.strip() val = val.strip()
return [len(val), val.encode('utf-8')] return [len(val), val.encode('utf-8')]
else: else:
@ -803,7 +801,7 @@ class CransLdap:
if r[i] == None: if r[i] == None:
continue continue
for a in r[i]: for a in r[i]:
dn = string.join(a[0].split(',')[-4:], ',') dn = ','.join(a[0].split(',')[-4:])
if dn in bons_dn and not a[0] in bons_dn2: if dn in bons_dn and not a[0] in bons_dn2:
bons_dn2.append(dn) bons_dn2.append(dn)
result[i].append(self.make(a, mode)) result[i].append(self.make(a, mode))
@ -2854,7 +2852,7 @@ Contactez nounou si la MAC est bien celle d'une carte.""", 3)
break break
if not len(pool_ip): if not len(pool_ip):
raise RuntimeError(u"Plus d'IP libres dans %s." % string.join(net, ' et ')) raise RuntimeError(u"Plus d'IP libres dans %s." % ' et '.join(net, ' et '))
else: else:
# L'ip est elle dans le bon sous-réseau ? # L'ip est elle dans le bon sous-réseau ?