Commit 5f1ff061 authored by Vincent Le gallic's avatar Vincent Le gallic

C'est pas parce qu'on est trésorier qu'on peut rendre la BDD incohérente

cantinvalidate prend la priorité sur transactions_admin : sinon, on peut dévalider
une transaction d'adhésion. Or l'adhesion elle-même ne peut pas être supprimée.
parent d58dd739
......@@ -3774,14 +3774,14 @@ def _can_toggle_transaction(self, transaction):
La règle est la suivante : on peut valider/dévalider une transaction si on aurait le droit
de faire la transaction que va effectuer cette validation/dévalidation.
"""
if self._has_acl("transactions_admin"):
# Quand on a ces droits (à distribuer parcimonieusement car aussi puissants que overforced)
# on peut changer l'état d'une transaction sans condition
return True
if transaction["cantinvalidate"]:
self._debug(3, u"can't toggle transaction : transaction %s est en cantinvalidate" % transaction["id"])
self._send(None, 312, u"Cette transaction ne peut pas être validée/dévalidée.")
return False
if self._has_acl("transactions_admin"):
# Quand on a ces droits (à distribuer parcimonieusement car aussi puissants que overforced)
# on peut changer l'état d'une transaction sans condition
return True
con, cur = BaseFonctions.getcursor()
cur.execute("SELECT toggle_transaction_timeout FROM configurations WHERE used = true;")
duree = cur.fetchone()[0]
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment