crans/lc_ldap
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

On laisse la possiblité de fournir le login quand on rajoute une ligne d'historique.

Browse source 
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.
This commit is contained in:
Vincent Le Gallic 2012-12-01 05:24:42 +01:00
parent 042f6fc7c3
commit e26db64944
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
lc_ldap.py
View file

@ -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):