From 5d8f5a9d23ebc9e2e9344b4105e0a7f42e80e4c2 Mon Sep 17 00:00:00 2001
From: Olivier Iffrig <iffrig@crans.org>
Date: Sun, 27 Feb 2011 15:40:39 +0100
Subject: [PATCH] =?UTF-8?q?[attributs,=20lc=5Fldap]=20Cr=C3=A9ation=20de?=
 =?UTF-8?q?=20clubs?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 TODO         | 1 -
 attributs.py | 2 +-
 lc_ldap.py   | 4 +++-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/TODO b/TODO
index b421139..23572c0 100644
--- a/TODO
+++ b/TODO
@@ -5,7 +5,6 @@
   * Une liste vide pour les machines en tant que valeurs non instanciée,
     ça n'a pas l'air top...
  * Ajouter les tests de permission dans la création de machine
- * Création de clubs
  * quid des factures ?!
  * Définir une méthode reconnect ?!
  * Décider comment on fait les tests d'unicité d'attribut ?
diff --git a/attributs.py b/attributs.py
index 07ba21b..66fe752 100644
--- a/attributs.py
+++ b/attributs.py
@@ -428,7 +428,7 @@ class responsable(Attr):
         self.value = self.conn.search('aid=%s' % resp)[0]
 
     def __unicode__(self):
-        return self.value.attrs['aid'][0]
+        return self.value.attrs['aid'][0].__unicode__()
 
 
 class blacklist(Attr):
diff --git a/lc_ldap.py b/lc_ldap.py
index ae48ad7..0234530 100644
--- a/lc_ldap.py
+++ b/lc_ldap.py
@@ -185,7 +185,9 @@ class lc_ldap(ldap.ldapobject.LDAPObject):
 
     def newClub(self, uldif):
         """Crée un nouveau club"""
-        raise NotImplementedError()
+        cid = uldif.setdefault('cid', [ unicode(self._find_id('cid')) ])
+        uldif['objectClass'] = [u'club']
+        return self._create_entity('cid=%s,%s' % (cid[0], base_dn), uldif)
 
     def newFacture(self, uldif):
         """Crée une nouvelle facture"""