From a5507c55b85d8d534031eb2a78a15ceb20a2f2de Mon Sep 17 00:00:00 2001
From: Antoine Durand-Gasselin <adg@crans.org>
Date: Sat, 16 Oct 2010 18:28:38 +0200
Subject: [PATCH] =?UTF-8?q?[attributs]=20C'est=20plus=20rapide=20si=20on?=
 =?UTF-8?q?=20check=20l'uniqueness=20dans=20la=20classe=20m=C3=A8re?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 attributs.py | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/attributs.py b/attributs.py
index f1f6940..ad167b7 100644
--- a/attributs.py
+++ b/attributs.py
@@ -111,16 +111,15 @@ class Attr(object):
     def _check_uniqueness(self):
         """Vérifie l'unicité dans la base de la valeur (mailAlias, chbre,
         etc...)"""
-        pass
-
-    def _check_values(self, values):
-        """Vérifie que les valeurs sont valides (typiquement chbre)"""
-        pass
+        attr = self.__class__.__name__
+        if attr in [ "mid", "uid", "cid", "fid",]:  #... etc
+            assert not self.conn.search('%s=%s' % (attr, str(self)))
 
     def _check_users_restrictions(self, values):
         """Vérifie les restrictions supplémentaires imposées selon les
         niveaux de droits (<= 3 mailAlias, pas de mac identiques,
         etc...)"""
+        ### On l'implémente dans les classes filles !
         pass
 
     def can_modify(self):