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

[gest_crans_lc] carte_ok -> adherent["carteEtudiant"]

Browse source 
Parce que ça n'est pas la sémantique de carte_ok que de savoir si la case
carte étudiant est coché et que c'est ce qu'on veux.
Pour info, carte_ok() retourne True si pas de blacklist carte (ie carte donné
ou periode transitoire ou sursis carte)
This commit is contained in:
Valentin Samir 2014-11-24 16:18:37 +01:00
parent e78049cd82
commit 0366876368
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
gestion/gest_crans_lc.py
View file

@ -1993,7 +1993,7 @@ les valeurs valident sont :
a = attributs a = attributs
choices = [] choices = []
if self.has_right(a.tresorier, adherent) or not adherent.carte_controle(): if self.has_right(a.tresorier, adherent) or not adherent.carte_controle():
choices.append((a.carteEtudiant.ldap_name, "Carte étudiant présentée", 1 if adherent.carte_ok() or values.get(a.carteEtudiant.ldap_name, False) else 0)) choices.append((a.carteEtudiant.ldap_name, "Carte étudiant présentée", 1 if adherent[a.carteEtudiant.ldap_name] or values.get(a.carteEtudiant.ldap_name, False) else 0))
if self.has_right(a.tresorier, adherent): if self.has_right(a.tresorier, adherent):
choices.append(("controleCarte", "La carte a-t-elle été controlée", 1 if adherent.carte_controle() or values.get("controleCarte", False) else 0)) choices.append(("controleCarte", "La carte a-t-elle été controlée", 1 if adherent.carte_controle() or values.get("controleCarte", False) else 0))
@ -2030,9 +2030,9 @@ les valeurs valident sont :
if not adherent["controle"][0]: if not adherent["controle"][0]:
adherent["controle"] = [] adherent["controle"] = []
# Si la carte n'est pas validé ou qu'on est trésorier, on sauvegarde les changements # Si la carte n'est pas validé ou qu'on est trésorier, on sauvegarde les changements
if values[a.carteEtudiant.ldap_name] and not adherent.carte_ok() and (not adherent.carte_controle() or self.has_right(a.tresorier, adherent)): if values[a.carteEtudiant.ldap_name] and not adherent[a.carteEtudiant.ldap_name] and (not adherent.carte_controle() or self.has_right(a.tresorier, adherent)):
adherent[a.carteEtudiant.ldap_name] = u"TRUE" adherent[a.carteEtudiant.ldap_name] = u"TRUE"
elif not values[a.carteEtudiant.ldap_name] and adherent.carte_ok() and (not adherent.carte_controle() or self.has_right(a.tresorier, adherent)): elif not values[a.carteEtudiant.ldap_name] and adherent[a.carteEtudiant.ldap_name] and (not adherent.carte_controle() or self.has_right(a.tresorier, adherent)):
adherent[a.carteEtudiant.ldap_name] = [] adherent[a.carteEtudiant.ldap_name] = []
if adherent["controle"]: if adherent["controle"]:
adherent["controle"]=unicode(adherent["controle"][0]).replace('c','') adherent["controle"]=unicode(adherent["controle"][0]).replace('c','')