From 99ba3a8915834eccacca14f54cb0f4cd0f5d67da Mon Sep 17 00:00:00 2001
From: Valentin Samir <valentin.samir@crans.org>
Date: Sun, 23 Feb 2014 00:02:00 +0100
Subject: [PATCH] [lc_ldap, objets] vivent les __repr__

---
 lc_ldap.py |  6 ++++++
 objets.py  | 16 ++++++++++++++--
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/lc_ldap.py b/lc_ldap.py
index 0402189..447bffe 100644
--- a/lc_ldap.py
+++ b/lc_ldap.py
@@ -132,6 +132,12 @@ class lc_ldap(ldap.ldapobject.LDAPObject, object):
             self.droits = []
         self._username_given = user
 
+    def __repr__(self):
+        if self.dn:
+            return str(self.__class__) + " : " + self.dn
+        else:
+            return super(lc_ldap, self).__repr__()
+
     def gravedig(self, type, filter=None, date=None):
         """Cherche dans le cimetière un objet de type ``type``,
         correspondant au filtre ``filter`` entre les dates ``date[0]`` et ``date[1]``
diff --git a/objets.py b/objets.py
index 33aea7a..11b82d4 100644
--- a/objets.py
+++ b/objets.py
@@ -610,7 +610,7 @@ class proprio(CransLdapObject):
     attribs = [attributs.nom, attributs.chbre, attributs.paiement, attributs.info, attributs.blacklist, attributs.controle, attributs.historique]
 
     def __repr__(self):
-        return str(self.__class__) + " : " + str(self['nom'][0])
+        return str(self.__class__) + " : nom=" + str(self['nom'][0])
 
     def __init__(self, conn, dn, mode='ro', ldif=None):
         super(proprio, self).__init__(conn, dn, mode, ldif)
@@ -720,7 +720,7 @@ class machine(CransLdapObject):
                attributs.dnsIpv6, attributs.machineAlias]
 
     def __repr__(self):
-        return str(self.__class__) + " : " + str(self['host'][0])
+        return str(self.__class__) + " : host=" + str(self['host'][0])
 
     def __init__(self, conn, dn, mode='ro', ldif = None):
         super(machine, self).__init__(conn, dn, mode, ldif)
@@ -883,6 +883,9 @@ class adherent(proprio):
                                  ]
     ldap_name = "adherent"
 
+    def __repr__(self):
+        return str(self.__class__) + " : aid=" + str(self['aid'][0])
+
     def __init__(self, conn, dn, mode='ro', ldif=None):
         super(adherent, self).__init__(conn, dn, mode, ldif)
         self.full = False
@@ -967,6 +970,9 @@ class club(proprio):
     attribs = proprio.attribs + [attributs.cid, attributs.responsable, attributs.imprimeurClub]
     ldap_name = "club"
 
+    def __repr__(self):
+        return str(self.__class__) + " : cid=" + str(self['cid'][0])
+
 @crans_object
 class machineFixe(machine):
     u"""Machine fixe"""
@@ -1050,6 +1056,9 @@ class facture(CransLdapObject):
 
     _proprio = None
 
+    def __repr__(self):
+        return str(self.__class__) + " : fid=" + str(self['fid'][0])
+
     #### GROS HACK pour rester comptatible avec ldap_crans où l'article representant les frais n'est ajouté qu'une fois le paiement reçu
     def __init__(self, conn, dn, mode='ro', ldif=None):
         super(facture, self).__init__(conn, dn, mode, ldif)
@@ -1098,6 +1107,9 @@ class baseCert(CransLdapObject):
 
     _machine = None
 
+    def __repr__(self):
+        return str(self.__class__) + " : xid=" + str(self['xid'][0])
+
     def __init__(self, conn, dn, mode='ro', ldif=None):
         super(baseCert, self).__init__(conn, dn, mode, ldif)
         if "TLSACert" in self['objectClass']: