From e26db649444a4d0dd2b2a44277791c0c0b014328 Mon Sep 17 00:00:00 2001 From: Vincent Le Gallic Date: Sat, 1 Dec 2012 05:24:42 +0100 Subject: [PATCH] =?UTF-8?q?On=20laisse=20la=20possiblit=C3=A9=20de=20fourn?= =?UTF-8?q?ir=20le=20login=20quand=20on=20rajoute=20une=20ligne=20d'histor?= =?UTF-8?q?ique.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit C'est utilisé par l'intranet2 actuellement parce que $USER et $SUDO_USER ne contiennent rien, et de toutes façons ce serait sans rapport avec l'user logué. Par contre j'aime pas le fait qu'on doive fournir ce login dès qu'on veut faire un truc. (là il faut le fournir à update_solde) mais je vois pas trop comment faire autrement. --- lc_ldap.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lc_ldap.py b/lc_ldap.py index e81f5db..940c981 100644 --- a/lc_ldap.py +++ b/lc_ldap.py @@ -576,8 +576,9 @@ class proprio(CransLdapObject): return bool_carte return True - def update_solde(self, diff, comment=u""): + def update_solde(self, diff, comment=u"", login=None): """Modifie le solde du proprio. diff peut être négatif ou positif.""" + login = login or current_user assert isinstance(diff, int) or isinstance(diff, float) assert isinstance(comment, unicode) @@ -590,7 +591,7 @@ class proprio(CransLdapObject): transaction = u"credit" if diff >=0 else u"debit" new_solde = u"%.2f" % new_solde - self.history_add(current_user, u"%s %.2f Euros [%s]" % (transaction, abs(diff), comment)) + self.history_add(login, u"%s %.2f Euros [%s]" % (transaction, abs(diff), comment)) self["solde"] = new_solde def machines(self):