lehmann/ScoDoc-Front
1
0
Fork 0
forked from ScoDoc/ScoDoc
Code Pull Requests Activity

Jury BUT: fix enregistrement décisions + message cohérence

Browse Source
This commit is contained in:
Emmanuel Viennet 2023-01-11 11:09:12 -03:00 committed by iziram
parent 85f00c7cb6
commit 4915852d66
1 changed files with 67 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

96
app/but/jury_but.py
View File

@ -91,7 +91,12 @@ from app.models.ues import UniteEns
from app.models.validations import ScolarFormSemestreValidation from app.models.validations import ScolarFormSemestreValidation
from app.scodoc import sco_cache from app.scodoc import sco_cache
from app.scodoc import sco_codes_parcours as sco_codes from app.scodoc import sco_codes_parcours as sco_codes
from app.scodoc.sco_codes_parcours import CODES_UE_VALIDES, RED, UE_STANDARD from app.scodoc.sco_codes_parcours import (
CODES_RCUE_VALIDES,
CODES_UE_VALIDES,
RED,
UE_STANDARD,
)
from app.scodoc import sco_utils as scu from app.scodoc import sco_utils as scu
from app.scodoc.sco_exceptions import ScoNoReferentielCompetences, ScoValueError from app.scodoc.sco_exceptions import ScoNoReferentielCompetences, ScoValueError
@ -365,7 +370,6 @@ class DecisionsProposeesAnnee(DecisionsProposees):
"s" if plural else ""} sur {self.nb_competences}""" "s" if plural else ""} sur {self.nb_competences}"""
if self.admis: if self.admis:
self.codes = [sco_codes.ADM] + self.codes self.codes = [sco_codes.ADM] + self.codes
self.explanation = expl_rcues
# elif not self.jury_annuel: # elif not self.jury_annuel:
# self.codes = [] # pas de décision annuelle sur semestres impairs # self.codes = [] # pas de décision annuelle sur semestres impairs
elif self.inscription_etat != scu.INSCRIT: elif self.inscription_etat != scu.INSCRIT:
@ -381,9 +385,9 @@ class DecisionsProposeesAnnee(DecisionsProposees):
sco_codes.ABL, sco_codes.ABL,
sco_codes.EXCLU, sco_codes.EXCLU,
] ]
expl_rcues = ""
elif self.passage_de_droit: elif self.passage_de_droit:
self.codes = [sco_codes.PASD, sco_codes.ADJ] + self.codes self.codes = [sco_codes.PASD, sco_codes.ADJ] + self.codes
self.explanation = expl_rcues
elif self.valide_moitie_rcue: # mais au moins 1 rcue insuffisante elif self.valide_moitie_rcue: # mais au moins 1 rcue insuffisante
self.codes = [ self.codes = [
sco_codes.RED, sco_codes.RED,
@ -391,7 +395,7 @@ class DecisionsProposeesAnnee(DecisionsProposees):
sco_codes.PAS1NCI, sco_codes.PAS1NCI,
sco_codes.ADJ, sco_codes.ADJ,
] + self.codes ] + self.codes
self.explanation = expl_rcues + f" et {self.nb_rcues_under_8} < 8" expl_rcues += f" et {self.nb_rcues_under_8} < 8"
else: else:
self.codes = [ self.codes = [
sco_codes.RED, sco_codes.RED,
@ -400,17 +404,21 @@ class DecisionsProposeesAnnee(DecisionsProposees):
sco_codes.ADJ, sco_codes.ADJ,
sco_codes.PASD, # voir #488 (discutable, conventions locales) sco_codes.PASD, # voir #488 (discutable, conventions locales)
] + self.codes ] + self.codes
self.explanation = ( expl_rcues += f""" et {self.nb_rcues_under_8} niveau{'x' if self.nb_rcues_under_8 > 1 else ''} < 8"""
expl_rcues
+ f""" et {self.nb_rcues_under_8}
niveau{'x' if self.nb_rcues_under_8 > 1 else ''} < 8"""
)
# Si l'un des semestres est extérieur, propose ADM # Si l'un des semestres est extérieur, propose ADM
if ( if (
self.formsemestre_impair and self.formsemestre_impair.modalite == "EXT" self.formsemestre_impair and self.formsemestre_impair.modalite == "EXT"
) or (self.formsemestre_pair and self.formsemestre_pair.modalite == "EXT"): ) or (self.formsemestre_pair and self.formsemestre_pair.modalite == "EXT"):
self.codes.insert(0, sco_codes.ADM) self.codes.insert(0, sco_codes.ADM)
self.explanation = f"<div>{expl_rcues}</div>"
messages = self.descr_pb_coherence()
if messages:
self.explanation += (
'<div class="warning">'
+ '</div><div class="warning">'.join(messages)
+ "</div>"
)
# #
def infos(self) -> str: def infos(self) -> str:
@ -670,16 +678,16 @@ class DecisionsProposeesAnnee(DecisionsProposees):
# Code annuel # Code annuel
code_annee = code code_annee = code
with sco_cache.DeferredSemCacheManager(): with sco_cache.DeferredSemCacheManager():
# Enregistre les codes, dans l'ordre UE, RCUE, Année # Enregistre les codes, dans l'ordre UE, RCUE, Année
for dec_ue, code in codes_ues: for dec_ue, code in codes_ues:
dec_ue.record(code) dec_ue.record(code)
for dec_rcue, code in codes_rcues: for dec_rcue, code in codes_rcues:
dec_rcue.record(code) dec_rcue.record(code)
self.record(code_annee) self.record(code_annee)
self.record_all() self.record_all()
db.session.commit() db.session.commit()
def record(self, code: str, no_overwrite=False): def record(self, code: str, no_overwrite=False):
"""Enregistre le code de l'année, et au besoin l'autorisation d'inscription. """Enregistre le code de l'année, et au besoin l'autorisation d'inscription.
@ -697,7 +705,7 @@ class DecisionsProposeesAnnee(DecisionsProposees):
return # no change return # no change
if self.validation: if self.validation:
db.session.delete(self.validation) db.session.delete(self.validation)
db.session.flush() db.session.commit()
if code is None: if code is None:
self.validation = None self.validation = None
else: else:
@ -708,12 +716,14 @@ class DecisionsProposeesAnnee(DecisionsProposees):
annee_scolaire=self.annee_scolaire(), annee_scolaire=self.annee_scolaire(),
code=code, code=code,
) )
db.session.add(self.validation)
db.session.commit()
Scolog.logdb( Scolog.logdb(
method="jury_but", method="jury_but",
etudid=self.etud.id, etudid=self.etud.id,
msg=f"Validation année BUT{self.annee_but}: {code}", msg=f"Validation année BUT{self.annee_but}: {code}",
) )
db.session.add(self.validation)
# --- Autorisation d'inscription dans semestre suivant ? # --- Autorisation d'inscription dans semestre suivant ?
if self.formsemestre_pair is not None: if self.formsemestre_pair is not None:
if code is None: if code is None:
@ -796,13 +806,14 @@ class DecisionsProposeesAnnee(DecisionsProposees):
ordre=self.annee_but, ordre=self.annee_but,
) )
for validation in validations: for validation in validations:
db.session.delete(validation)
Scolog.logdb( Scolog.logdb(
"jury_but", "jury_but",
etudid=self.etud.id, etudid=self.etud.id,
msg=f"Validation année BUT{self.annee_but}: effacée", msg=f"Validation année BUT{self.annee_but}: effacée",
) )
db.session.delete(validation)
# Efface éventuelle validation de semestre # Efface éventuelles validations de semestre
# (en principe inutilisées en BUT) # (en principe inutilisées en BUT)
# et autres UEs (en cas de changement d'architecture de formation depuis le jury ?) # et autres UEs (en cas de changement d'architecture de formation depuis le jury ?)
# #
@ -811,7 +822,7 @@ class DecisionsProposeesAnnee(DecisionsProposees):
): ):
db.session.delete(validation) db.session.delete(validation)
db.session.flush() db.session.commit()
self.invalidate_formsemestre_cache() self.invalidate_formsemestre_cache()
def get_autorisations_passage(self) -> list[int]: def get_autorisations_passage(self) -> list[int]:
@ -854,6 +865,27 @@ class DecisionsProposeesAnnee(DecisionsProposees):
validations.append(", ".join(v for v in valids if v)) validations.append(", ".join(v for v in valids if v))
return line_sep.join(validations) return line_sep.join(validations)
def descr_pb_coherence(self) -> list[str]:
"""Description d'éventuels problèmes de cohérence entre
les décisions *enregistrées* d'UE et de RCUE.
Note: en principe, la cohérence RCUE/UE est assurée au moment de
l'enregistrement (record).
Mais la base peut avoir été modifiée par d'autres voies.
"""
messages = []
for dec_rcue in self.decisions_rcue_by_niveau.values():
if dec_rcue.code_valide in CODES_RCUE_VALIDES:
for ue in (dec_rcue.rcue.ue_1, dec_rcue.rcue.ue_2):
dec_ue = self.decisions_ues.get(ue.id)
if dec_ue:
if dec_ue.code_valide not in CODES_UE_VALIDES:
messages.append(
f"L'UE {ue.acronyme} n'est pas validée mais son RCUE l'est !"
)
else:
messages.append(f"L'UE {ue.acronyme} n'a pas décision (???)")
return messages
def list_ue_parcour_etud( def list_ue_parcour_etud(
formsemestre: FormSemestre, etud: Identite, res: ResultatsSemestreBUT formsemestre: FormSemestre, etud: Identite, res: ResultatsSemestreBUT
@ -969,7 +1001,7 @@ class DecisionsProposeesRCUE(DecisionsProposees):
parcours_id = self.parcour.id if self.parcour is not None else None parcours_id = self.parcour.id if self.parcour is not None else None
if self.validation: if self.validation:
db.session.delete(self.validation) db.session.delete(self.validation)
db.session.flush() db.session.commit()
if code is None: if code is None:
self.validation = None self.validation = None
else: else:
@ -984,12 +1016,14 @@ class DecisionsProposeesRCUE(DecisionsProposees):
parcours_id=parcours_id, parcours_id=parcours_id,
code=code, code=code,
) )
db.session.add(self.validation)
db.session.commit()
Scolog.logdb( Scolog.logdb(
method="jury_but", method="jury_but",
etudid=self.etud.id, etudid=self.etud.id,
msg=f"Validation {self.rcue}: {code}", msg=f"Validation {self.rcue}: {code}",
commit=True,
) )
db.session.add(self.validation)
# Modifie au besoin les codes d'UE # Modifie au besoin les codes d'UE
if code == "ADJ": if code == "ADJ":
deca = self.deca deca = self.deca
@ -1156,6 +1190,7 @@ class DecisionsProposeesUE(DecisionsProposees):
method="jury_but", method="jury_but",
etudid=self.etud.id, etudid=self.etud.id,
msg=f"Validation UE {self.ue.id} {self.ue.acronyme}: effacée", msg=f"Validation UE {self.ue.id} {self.ue.acronyme}: effacée",
commit=True,
) )
else: else:
self.validation = ScolarFormSemestreValidation( self.validation = ScolarFormSemestreValidation(
@ -1165,12 +1200,14 @@ class DecisionsProposeesUE(DecisionsProposees):
code=code, code=code,
moy_ue=self.moy_ue, moy_ue=self.moy_ue,
) )
db.session.add(self.validation)
db.session.commit()
Scolog.logdb( Scolog.logdb(
method="jury_but", method="jury_but",
etudid=self.etud.id, etudid=self.etud.id,
msg=f"Validation UE {self.ue.id} {self.ue.acronyme}({self.moy_ue}): {code}", msg=f"Validation UE {self.ue.id} {self.ue.acronyme}({self.moy_ue}): {code}",
commit=True,
) )
db.session.add(self.validation)
log(f"DecisionsProposeesUE: recording {self.validation}") log(f"DecisionsProposeesUE: recording {self.validation}")
sco_cache.invalidate_formsemestre(formsemestre_id=self.formsemestre.id) sco_cache.invalidate_formsemestre(formsemestre_id=self.formsemestre.id)
@ -1185,13 +1222,14 @@ class DecisionsProposeesUE(DecisionsProposees):
) )
for validation in validations: for validation in validations:
log(f"DecisionsProposeesUE: deleting {validation}") log(f"DecisionsProposeesUE: deleting {validation}")
db.session.delete(validation)
Scolog.logdb( Scolog.logdb(
method="jury_but", method="jury_but",
etudid=self.etud.id, etudid=self.etud.id,
msg=f"Validation UE {validation.ue.id} {validation.ue.acronyme}: effacée", msg=f"Validation UE {validation.ue.id} {validation.ue.acronyme}: effacée",
) )
db.session.delete(validation)
db.session.flush() db.session.commit()
def descr_validation(self) -> str: def descr_validation(self) -> str:
"""Description validation niveau enregistrée, pour PV jury. """Description validation niveau enregistrée, pour PV jury.