From 6af35d587c6351e82f7ae771be0129fd19c422e8 Mon Sep 17 00:00:00 2001
From: Nicolas Dandrimont <Nicolas.Dandrimont@crans.org>
Date: Sun, 17 Oct 2010 14:25:23 +0200
Subject: [PATCH] [lc_ldap, attributs] Exceptions string, pas unicode...

---
 attributs.py |  2 +-
 lc_ldap.py   | 22 +++++++++++-----------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/attributs.py b/attributs.py
index 7797626..17c646c 100644
--- a/attributs.py
+++ b/attributs.py
@@ -96,7 +96,7 @@ class Attr(object):
         """Vérifie qu'il y a un nombre correct de valeur =1, <=1, {0,1},
         etc..."""
         if self.singlevalue and len(values) > 1:
-            raise ValueError(u'%s doit avoir au maximum une valeur (affecte %s)' % (self.__class__, values))
+            raise ValueError('%s doit avoir au maximum une valeur (affecte %s)' % (self.__class__, values))
         if not self.optional and len(values) == 0:
             raise ValueError('%s doit avoir au moins une valeur' % self.__class__)
 
diff --git a/lc_ldap.py b/lc_ldap.py
index 7ddec09..2518708 100644
--- a/lc_ldap.py
+++ b/lc_ldap.py
@@ -211,7 +211,7 @@ class lc_ldap(ldap.ldapobject.LDAPObject):
             else:
                 break
         else:
-            raise EnvironmentError(u'Aucun %s libre dans la plage [%d, %d]' %
+            raise EnvironmentError('Aucun %s libre dans la plage [%d, %d]' %
                                    (attr, plage[0], i))
         return i
 
@@ -267,7 +267,7 @@ class CransLdapObject(object):
         self._modifs = None # C'est là qu'on met les modifications
 
         if not isinstance(conn, lc_ldap):
-            raise TypeError(u"conn doit être une instance de lc_ldap")
+            raise TypeError("conn doit être une instance de lc_ldap")
         self.conn = conn
         self.dn = dn
 
@@ -302,7 +302,7 @@ class CransLdapObject(object):
     def save(self):
         "Vérifie que self._modifs contient des valeurs correctes et enregistre les modifications"
         if self.mode not in ['w', 'rw']:
-            raise EnvironmentError(u"Objet en lecture seule, réessayer en lecture/écriture")
+            raise EnvironmentError("Objet en lecture seule, réessayer en lecture/écriture")
 
         # Vérifications et Historique
         #histo = self._gen_hist(self._modifs)
@@ -315,7 +315,7 @@ class CransLdapObject(object):
         # Vérification des modifications
         self.attrs = ldif_to_uldif(self.conn.search_s(self.dn, 0)[0][1])
         if self.attrs != self._modifs:
-            raise EnvironmentError(u"Les modifications apportées à l'objet %s n'ont pas été correctement sauvegardées\nexpected = %s, found = %s" % (self.dn, self._modifs, self.attrs))
+            raise EnvironmentError("Les modifications apportées à l'objet %s n'ont pas été correctement sauvegardées\nexpected = %s, found = %s" % (self.dn, self._modifs, self.attrs))
 
     def get_modlist(self):
         """Renvoie le dico des modifs"""
@@ -338,7 +338,7 @@ class CransLdapObject(object):
         """Définit les nouvelles valeurs d'un attribut"""
 
         if self.mode not in ['w', 'rw']:
-            raise EnvironmentError(u"Objet en lecture seule, réessayer en lecture/écriture")
+            raise EnvironmentError("Objet en lecture seule, réessayer en lecture/écriture")
 
         if not isinstance(new_vals, list):
             new_vals = [new_vals]
@@ -365,7 +365,7 @@ class CransLdapObject(object):
         elif len(new_vals) == 1:
             new_vals = [ new_val ]
         else:
-            raise ValueError(u"%s has multiple values, must specify old_val")
+            raise ValueError("%s has multiple values, must specify old_val")
         return self.set_ldapattr(attr, new_vals)
 
     def del_ldapattr(self, attr, val):
@@ -415,7 +415,7 @@ class CransLdapObject(object):
             elif olds == news and len(oldvals) == len(newvals) == len(olds):
                 msg = u"[%s].shuffle()" % field
             else:
-                raise ValueError(u"Les valeurs pour %s : %s -> %s ne semblent pas différentes" % (field, oldvals, newvals))
+                raise ValueError("Les valeurs pour %s : %s -> %s ne semblent pas différentes" % (field, oldvals, newvals))
             histo.append(self.conn._hist(msg))
 
         return histo
@@ -487,7 +487,7 @@ class CransLdapObject(object):
             debut = new[0] = int(time.time())
         else:
             try: debut = new[0] = int(new[0])
-            except: raise ValueError(u'Date de début blacklist invalide')
+            except: raise ValueError('Date de début blacklist invalide')
 
         if new[1] == 'now':
             fin = new[1] = int(time.time())
@@ -495,12 +495,12 @@ class CransLdapObject(object):
             fin = -1
         else:
             try: fin = new[1] = int(new[1])
-            except: raise ValueError(u'Date de fin blacklist invalide')
+            except: raise ValueError('Date de fin blacklist invalide')
 
         if debut == fin:
-            raise ValueError(u'Dates de début et de fin identiques')
+            raise ValueError('Dates de début et de fin identiques')
         elif fin != -1 and debut > fin:
-            raise ValueError(u'Date de fin avant date de début')
+            raise ValueError('Date de fin avant date de début')
 
         # On dépasse la fin de sanction d'1min pour être sûr qu'elle est périmée.
         fin = fin + 60