diff --git a/gestion/ldap_crans.py b/gestion/ldap_crans.py
index 5f55ac60..6e123727 100644
--- a/gestion/ldap_crans.py
+++ b/gestion/ldap_crans.py
@@ -166,17 +166,15 @@ def preattr(val):
     Retourne [ len(str(val).strip), str(val).strip en utf-8 ]
     """
 
-    t = type(val)
-
-    if t == list and len(val) == 1:
+    if isinstance(val, list) and len(val) == 1:
         return preattr(val[0])
 
-    elif t == str or t == int:
+    elif isinstance(val, str) or isinstance(val, int):
         val = str(val).strip()
         # On passe tout en utf-8 pour ne pas avoir de problèmes
         # d'accents dans la base
         return [len(val), unicode(val, 'iso-8859-1').encode('utf-8')]
-    elif t == unicode:
+    elif isinstance(val, unicode):
         val = val.strip()
         return [len(val), val.encode('utf-8')]
     else:
@@ -803,7 +801,7 @@ class CransLdap:
                     if r[i] == None:
                         continue
                     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:
                             bons_dn2.append(dn)
                             result[i].append(self.make(a, mode))
@@ -2854,7 +2852,7 @@ Contactez nounou si la MAC est bien celle d'une carte.""", 3)
                     break
 
             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:
             # L'ip est elle dans le bon sous-réseau ?