From ff802959f433492c383b1b346babe33295320684 Mon Sep 17 00:00:00 2001
From: glondu <none>
Date: Sat, 1 Apr 2006 15:03:27 +0200
Subject: [PATCH] On met les choses au clair en ce qui concerne
 rewriteMailHeaders. Les

nouveaux adhrents n'auront plus la rcriture des en-ttes par dfaut.

darcs-hash:20060401130327-68412-34ed66d6a3692367180721989ccc171dbd9964c4.gz
---
 gestion/ldap_crans.py | 36 ++++++++++++++++++++++--------------
 1 file changed, 22 insertions(+), 14 deletions(-)

diff --git a/gestion/ldap_crans.py b/gestion/ldap_crans.py
index 6cfe643b..54a03fe6 100755
--- a/gestion/ldap_crans.py
+++ b/gestion/ldap_crans.py
@@ -1028,7 +1028,7 @@ class base_classes_crans(crans_ldap):
             
             # Formate les entrées de l'historique de la forme champ (ancien -> nouveau)
             # On suppose que le champ apparaît forcément dans l'enregistrement
-            for champ in ['chbre', 'nom', 'prenom', 'mail', 'tel',
+            for champ in ['chbre', 'nom', 'prenom', 'mail', 'tel', 'rewriteMailHeaders',
                           'puissance', 'canal', 'prise', 'responsable',
                           'contourneGreylist', 'macAddress', 'ipHostNumber', 'host']:
                 if champ in self.modifs:
@@ -1384,17 +1384,21 @@ class base_proprietaire(base_classes_crans):
         return actuel
 
     def contourneGreylist(self, contourneGreylist=None):
-        """ Retourne ou change la greylist pour le compte
-        True  : contourne le GreyListing
-        False :ne contourne pas le greylisting """
+        """
+        Retourne ou change la greylist pour le compte
+         * True  : contourne le GreyListing
+         * False : ne contourne pas le greylisting
+        Au niveau de la base, on considère la présence ou l'absence
+        d'un champ contourneGreylist=OK.
+        """
         
         # Pour postfix il faut retourner :
         # OK : contourne la greyliste
         # cf. man 5 access
         
-        # si l'adhérent n'a pas de compte, on lève une exeption
+        # si l'adhérent n'a pas de compte, on lève une exception
         if not self.compte():
-            raise NotImplementedError, u"L'adhérent na pas de compte"
+            raise NotImplementedError, u"L'adhérent n'a pas de compte"
 
         # tente de modifier la valeur
         if contourneGreylist == True:
@@ -1986,24 +1990,28 @@ class Adherent(base_proprietaire):
         return new
     
     def rewriteMailHeaders(self, rewrite=None):
-        """ Réécriture des entêtes mail avec l'alias canonique
-        True  : réécrit les en-têtes
-        False : ne réécrit pas les en-têtes """
+        """
+        Réécriture des entêtes mail avec l'alias canonique
+         * True  : réécrit les en-têtes
+         * False : ne réécrit pas les en-têtes
+        Au niveau de la base, on considère l'absence ou la présence
+        de rewriteMailHeaders=TRUE.
+        """
         
-        # si l'adhérent n'a pas de compte, on lève une exeption
+        # si l'adhérent n'a pas de compte, on lève une exception
         if not self.compte():
-            raise NotImplementedError, u"L'adhérent na pas de compte"
+            raise NotImplementedError, u"L'adhérent n'a pas de compte"
 
         # tente de modifier la valeur
         if rewrite == True:
-            self._set('rewriteMailHeaders', ['oui'])
+            self._set('rewriteMailHeaders', ['TRUE'])
         elif rewrite == False:
-            self._set('rewriteMailHeaders', ['non'])
+            self._set('rewriteMailHeaders', [])
         elif rewrite != None:
             raise ValueError, u"rewriteMailHeaders prend un booléen comme argument"
 
         # renvoie la valeur trouvée dans la base
-        return self._data.get('rewriteMailHeaders', ['oui']) == ['oui']
+        return bool(self._data.get('rewriteMailHeaders', []))
         
 class Club(base_proprietaire):
     """ Classe de définition d'un club """