diff --git a/app/__init__.py b/app/__init__.py index a1862aaa7..738ad0970 100644 --- a/app/__init__.py +++ b/app/__init__.py @@ -205,8 +205,18 @@ def create_app(config_class=DevConfig): app = Flask(__name__, static_url_path="/ScoDoc/static", static_folder="static") app.wsgi_app = ReverseProxied(app.wsgi_app) app.logger.setLevel(logging.DEBUG) + + # Evite de logguer toutes les requetes dans notre log + logging.getLogger("werkzeug").disabled = True + app.config.from_object(config_class) + # Vérifie/crée lien sym pour les URL statiques + link_filename = f"{app.root_path}/static/links/{sco_version.SCOVERSION}" + if not os.path.exists(link_filename): + app.logger.info(f"creating symlink {link_filename}") + os.symlink("..", link_filename) + db.init_app(app) migrate.init_app(app, db) login.init_app(app) diff --git a/app/api/etudiants.py b/app/api/etudiants.py index a193553fa..4649a6698 100644 --- a/app/api/etudiants.py +++ b/app/api/etudiants.py @@ -230,7 +230,9 @@ def etudiant_formsemestres(etudid: int = None, nip: int = None, ine: int = None) formsemestres = query.order_by(FormSemestre.date_debut) - return jsonify([formsemestre.to_dict() for formsemestre in formsemestres]) + return jsonify( + [formsemestre.to_dict(convert_parcours=True) for formsemestre in formsemestres] + ) @bp.route( diff --git a/app/api/formsemestres.py b/app/api/formsemestres.py index 725454b18..139b35ef1 100644 --- a/app/api/formsemestres.py +++ b/app/api/formsemestres.py @@ -54,7 +54,7 @@ def formsemestre(formsemestre_id: int): formsemestre: FormSemestre = models.FormSemestre.query.filter_by( id=formsemestre_id ).first_or_404() - data = formsemestre.to_dict() + data = formsemestre.to_dict(convert_parcours=True) # Pour le moment on a besoin de fixer le departement # pour accéder aux préferences dept = Departement.query.get(formsemestre.dept_id) @@ -84,7 +84,9 @@ def formsemestre_apo(etape_apo: str): FormSemestreEtape.formsemestre_id == FormSemestre.id, ) - return jsonify([formsemestre.to_dict() for formsemestre in formsemestres]) + return jsonify( + [formsemestre.to_dict(convert_parcours=True) for formsemestre in formsemestres] + ) @bp.route("/formsemestre//bulletins", methods=["GET"]) diff --git a/app/but/apc_edit_ue.py b/app/but/apc_edit_ue.py new file mode 100644 index 000000000..dd8e60a86 --- /dev/null +++ b/app/but/apc_edit_ue.py @@ -0,0 +1,106 @@ +############################################################################## +# ScoDoc +# Copyright (c) 1999 - 2022 Emmanuel Viennet. All rights reserved. +# See LICENSE +############################################################################## + +""" +Edition associations UE <-> Ref. Compétence +""" +from flask import g, url_for +from app import db, log +from app.models import Formation, UniteEns +from app.models.but_refcomp import ApcNiveau +from app.scodoc import sco_codes_parcours + + +def form_ue_choix_niveau(formation: Formation, ue: UniteEns) -> str: + """Form. HTML pour associer une UE à un niveau de compétence""" + if ue.type != sco_codes_parcours.UE_STANDARD: + return "" + ref_comp = ue.formation.referentiel_competence + if ref_comp is None: + return f"""
+
Pas de référentiel de compétence associé à cette formation !
+
associer un référentiel de compétence +
+
""" + annee = (ue.semestre_idx + 1) // 2 # 1, 2, 3 + niveaux_by_parcours = ref_comp.get_niveaux_by_parcours(annee) + + # Les niveaux déjà associés à d'autres UE du même semestre + autres_ues = formation.ues.filter_by(semestre_idx=ue.semestre_idx) + niveaux_autres_ues = { + oue.niveau_competence_id for oue in autres_ues if oue.id != ue.id + } + options = [] + if niveaux_by_parcours["TC"]: # TC pour Tronc Commun + options.append("""""") + for n in niveaux_by_parcours["TC"]: + if n.id in niveaux_autres_ues: + disabled = "disabled" + else: + disabled = "" + options.append( + f"""""" + ) + options.append("""""") + for parcour in ref_comp.parcours: + if len(niveaux_by_parcours[parcour.id]): + options.append(f"""""") + for n in niveaux_by_parcours[parcour.id]: + if n.id in niveaux_autres_ues: + disabled = "disabled" + else: + disabled = "" + options.append( + f"""""" + ) + options.append("""""") + options_str = "\n".join(options) + return f""" +
+
+ Niveau de compétence associé: + +
+
+ """ + + +def set_ue_niveau_competence(ue_id: int, niveau_id: int): + """Associe le niveau et l'UE""" + log(f"set_ue_niveau_competence( {ue_id}, {niveau_id} )") + ue = UniteEns.query.get_or_404(ue_id) + + autres_ues = ue.formation.ues.filter_by(semestre_idx=ue.semestre_idx) + niveaux_autres_ues = { + oue.niveau_competence_id for oue in autres_ues if oue.id != ue.id + } + if niveau_id in niveaux_autres_ues: + log( + f"set_ue_niveau_competence: denying association of {ue} to already associated {niveau_id}" + ) + return "", 409 # conflict + if niveau_id == "": + # suppression de l'association + ue.niveau_competence = None + else: + niveau = ApcNiveau.query.get_or_404(niveau_id) + ue.niveau_competence = niveau + db.session.add(ue) + db.session.commit() + return "", 204 diff --git a/app/but/bulletin_but.py b/app/but/bulletin_but.py index f9dbb8706..3b90187b9 100644 --- a/app/but/bulletin_but.py +++ b/app/but/bulletin_but.py @@ -14,6 +14,7 @@ from flask import url_for, g from app.comp.res_but import ResultatsSemestreBUT from app.models import FormSemestre, Identite +from app.models import but_validations from app.models.groups import GroupDescr from app.models.ues import UniteEns from app.scodoc import sco_bulletins, sco_utils as scu @@ -244,7 +245,7 @@ class BulletinBUT: f"{fmt_note(bonus_vect[ue.id])} sur {ue.acronyme}" for ue in res.ues if ue.type != UE_SPORT - and res.modimpls_in_ue(ue.id, etudid) + and res.modimpls_in_ue(ue, etudid) and ue.id in res.bonus_ues and bonus_vect[ue.id] > 0.0 ] @@ -274,6 +275,11 @@ class BulletinBUT: etat_inscription = etud.inscription_etat(formsemestre.id) nb_inscrits = self.res.get_inscriptions_counts()[scu.INSCRIT] published = (not formsemestre.bul_hide_xml) or force_publishing + if formsemestre.formation.referentiel_competence is None: + etud_ues_ids = {ue.id for ue in res.ues if res.modimpls_in_ue(ue, etud.id)} + else: + etud_ues_ids = res.etud_ues_ids(etud.id) + d = { "version": "0", "type": "BUT", @@ -318,9 +324,13 @@ class BulletinBUT: ects_tot = sum([ue.ects or 0 for ue in res.ues]) if res.ues else 0.0 ects_acquis = sum([d.get("ects", 0) for d in decisions_ues.values()]) semestre_infos["ECTS"] = {"acquis": ects_acquis, "total": ects_tot} - semestre_infos.update( - sco_bulletins_json.dict_decision_jury(etud.id, formsemestre.id) - ) + if sco_preferences.get_preference("bul_show_decision", formsemestre.id): + semestre_infos.update( + sco_bulletins_json.dict_decision_jury(etud.id, formsemestre.id) + ) + semestre_infos.update( + but_validations.dict_decision_jury(etud, formsemestre) + ) if etat_inscription == scu.INSCRIT: # moyenne des moyennes générales du semestre semestre_infos["notes"] = { @@ -365,10 +375,7 @@ class BulletinBUT: ) for ue in res.ues # si l'UE comporte des modules auxquels on est inscrit: - if ( - (ue.type == UE_SPORT) - or self.res.modimpls_in_ue(ue.id, etud.id) - ) + if ((ue.type == UE_SPORT) or ue.id in etud_ues_ids) }, "semestre": semestre_infos, }, diff --git a/app/but/forms/jury_but_forms.py b/app/but/forms/jury_but_forms.py new file mode 100644 index 000000000..0f3719600 --- /dev/null +++ b/app/but/forms/jury_but_forms.py @@ -0,0 +1,18 @@ +############################################################################## +# ScoDoc +# Copyright (c) 1999 - 2022 Emmanuel Viennet. All rights reserved. +# See LICENSE +############################################################################## + +"""ScoDoc 9.3 : Formulaires / jurys BUT +""" + + +from flask_wtf import FlaskForm +from wtforms import SubmitField + + +class FormSemestreValidationAutoBUTForm(FlaskForm): + "simple form de confirmation" + submit = SubmitField("Lancer le calcul") + cancel = SubmitField("Annuler") diff --git a/app/but/forms/refcomp_forms.py b/app/but/forms/refcomp_forms.py index ce16292df..ee67cba28 100644 --- a/app/but/forms/refcomp_forms.py +++ b/app/but/forms/refcomp_forms.py @@ -13,7 +13,9 @@ from wtforms import SelectField, SubmitField class FormationRefCompForm(FlaskForm): - referentiel_competence = SelectField("Référentiels déjà chargés") + referentiel_competence = SelectField( + "Choisir parmi les référentiels déjà chargés :" + ) submit = SubmitField("Valider") cancel = SubmitField("Annuler") @@ -23,7 +25,7 @@ class RefCompLoadForm(FlaskForm): "Choisir un référentiel de compétences officiel BUT" ) upload = FileField( - label="Ou bien sélectionner un fichier XML au format Orébut", + label="... ou bien sélectionner un fichier XML au format Orébut (réservé aux développeurs !)", validators=[ FileAllowed( [ diff --git a/app/but/import_refcomp.py b/app/but/import_refcomp.py index 0f97cd958..9acd1c9db 100644 --- a/app/but/import_refcomp.py +++ b/app/but/import_refcomp.py @@ -4,7 +4,6 @@ # See LICENSE ############################################################################## from xml.etree import ElementTree -from typing import TextIO import sqlalchemy @@ -57,13 +56,13 @@ def orebut_import_refcomp(xml_data: str, dept_id: int, orig_filename=None): try: c = ApcCompetence(**ApcCompetence.attr_from_xml(competence.attrib)) db.session.flush() - except sqlalchemy.exc.IntegrityError: + except sqlalchemy.exc.IntegrityError as exc: # ne devrait plus se produire car pas d'unicité de l'id: donc inutile db.session.rollback() raise ScoValueError( f"""Un référentiel a déjà été chargé avec les mêmes compétences ! ({competence.attrib["id"]}) """ - ) + ) from exc ref.competences.append(c) # --- SITUATIONS situations = competence.find("situations") diff --git a/app/but/jury_but.py b/app/but/jury_but.py new file mode 100644 index 000000000..0a7660345 --- /dev/null +++ b/app/but/jury_but.py @@ -0,0 +1,1008 @@ +############################################################################## +# ScoDoc +# Copyright (c) 1999 - 2022 Emmanuel Viennet. All rights reserved. +# See LICENSE +############################################################################## + +"""Jury BUT: logique de gestion + +Utilisation: + 1) chargement page jury, pour un étudiant et un formsemestre BUT quelconque + - DecisionsProposeesAnnee(formsemestre) + cherche l'autre formsemestre de la même année scolaire (peut ne pas exister) + cherche les RCUEs de l'année (BUT1, 2, 3) + pour un redoublant, le RCUE peut considérer un formsemestre d'une année antérieure. + + on instancie des DecisionsProposees pour les + différents éléments (UEs, RCUEs, Année, Diplôme) + Cela donne + - les codes possibles (dans .codes) + - le code actuel si une décision existe déjà (dans code_valide) + - pour les UEs, le rcue s'il y en a un) + + 2) Validation pour l'utilisateur (form)) => enregistrement code + - on vérifie que le code soumis est bien dans les codes possibles + - on enregistre la décision (dans ScolarFormSemestreValidation pour les UE, + ApcValidationRCUE pour les RCUE, et ApcValidationAnnee pour les années) + - Si RCUE validé, on déclenche d'éventuelles validations: + ("La validation des deux UE du niveau d’une compétence emporte la validation + de l’ensemble des UE du niveau inférieur de cette même compétence.") + +Les jurys de semestre BUT impairs entrainent systématiquement la génération d'une +autorisation d'inscription dans le semestre pair suivant: `ScolarAutorisationInscription`. +Les jurys de semestres pairs non (S2, S4, S6): il y a une décision sur l'année (ETP) + - autorisation en S_2n+1 (S3 ou S5) si: ADM, ADJ, PASD, PAS1CN + - autorisation en S2n-1 (S1, S3 ou S5) si: RED + - rien si pour les autres codes d'année. + +Le formulaire permet de choisir des codes d'UE, RCUE et Année (ETP). +Mais normalement, les codes d'UE sont à choisir: les RCUE et l'année s'en déduisent. +Si l'utilisateur coche "décision manuelle", il peut alors choisir les codes RCUE et années. + +La soumission du formulaire: + - etud, formation + - UEs: [(formsemestre, ue, code), ...] + - RCUE: [(formsemestre, ue, code), ...] le formsemestre est celui d'indice pair du niveau + (S2, S4 ou S6), il sera regoupé avec celui impair de la même année ou de la suivante. + - Année: [(formsemestre, code)] + +DecisionsProposeesAnnee: + si 1/2 des rcue et aucun < 8 + pour S5 condition sur les UE de BUT1 et BUT2 + => charger les DecisionsProposeesRCUE + +DecisionsProposeesRCUE: les RCUEs pour cette année + validable, compensable, ajourné. Utilise classe RegroupementCoherentUE + +DecisionsProposeesUE: décisions de jury sur une UE du BUT + initialisation sans compensation (ue isolée), mais + DecisionsProposeesRCUE appelera .set_compensable() + si on a la possibilité de la compenser dans le RCUE. +""" +import html +from operator import attrgetter +import re +from typing import Union + +from flask import g, url_for + +from app import db +from app import log +from app.comp.res_but import ResultatsSemestreBUT +from app.comp import res_sem +from app.models import formsemestre + +from app.models.but_refcomp import ( + ApcAnneeParcours, + ApcCompetence, + ApcNiveau, + ApcParcours, + ApcParcoursNiveauCompetence, +) +from app.models import Scolog, ScolarAutorisationInscription +from app.models.but_validations import ( + ApcValidationAnnee, + ApcValidationRCUE, + RegroupementCoherentUE, +) +from app.models.etudiants import Identite +from app.models.formations import Formation +from app.models.formsemestre import FormSemestre, FormSemestreInscription +from app.models.ues import UniteEns +from app.models.validations import ScolarFormSemestreValidation +from app.scodoc import sco_codes_parcours as sco_codes +from app.scodoc.sco_codes_parcours import RED, UE_STANDARD +from app.scodoc import sco_utils as scu +from app.scodoc.sco_exceptions import ScoException, ScoValueError + + +class NoRCUEError(ScoValueError): + """Erreur en cas de RCUE manquant""" + + def __init__(self, deca: "DecisionsProposeesAnnee", ue: UniteEns): + if all(u.niveau_competence for u in deca.ues_pair): + warning_pair = "" + else: + warning_pair = """
certaines UE du semestre pair ne sont pas associées à un niveau de compétence
""" + if all(u.niveau_competence for u in deca.ues_impair): + warning_impair = "" + else: + warning_impair = """
certaines UE du semestre impair ne sont pas associées à un niveau de compétence
""" + msg = ( + f"""

Pas de RCUE pour l'UE {ue.acronyme}

+ {warning_impair} + {warning_pair} +
UE {ue.acronyme}: niveau {html.escape(str(ue.niveau_competence))}
+
UEs impaires: {html.escape(', '.join(str(u.niveau_competence or "pas de niveau") + for u in deca.ues_impair))} +
+ """ + + deca.infos() + ) + super().__init__(msg) + + +class DecisionsProposees: + """Une décision de jury proposé, constituée d'une liste de codes et d'une explication. + Super-classe, spécialisée pour les UE, les RCUE, les années et le diplôme. + + validation : None ou une instance de d'une classe avec un champ code + ApcValidationRCUE, ApcValidationAnnee ou ScolarFormSemestreValidation + """ + + # Codes toujours proposés sauf si include_communs est faux: + codes_communs = [ + sco_codes.RAT, + sco_codes.DEF, + sco_codes.ABAN, + sco_codes.DEM, + sco_codes.UEBSL, + ] + + def __init__( + self, + etud: Identite = None, + code: Union[str, list[str]] = None, + explanation="", + code_valide=None, + include_communs=True, + ): + self.etud = etud + self.codes = [] + "Les codes attribuables par ce jury" + if include_communs: + self.codes = self.codes_communs.copy() + if isinstance(code, list): + self.codes = code + self.codes + elif code is not None: + self.codes = [code] + self.codes + self.validation = None + "Validation enregistrée" + self.code_valide: str = code_valide + "Code décision actuel enregistré" + self.explanation: str = explanation + "Explication à afficher à côté de la décision" + self.recorded = False + "true si la décision vient d'être enregistrée" + + def __repr__(self) -> str: + return f"""<{self.__class__.__name__} valid={self.code_valide + } codes={self.codes} explanation={self.explanation}""" + + +class DecisionsProposeesAnnee(DecisionsProposees): + """Décisions de jury sur une année (ETP) du BUT + + Le texte: + La poursuite d'études dans un semestre pair d’une même année est de droit + pour tout étudiant. La poursuite d’études dans un semestre impair est + possible si et seulement si l’étudiant a obtenu : + - la moyenne à plus de la moitié des regroupements cohérents d’UE; + - et une moyenne égale ou supérieure à 8 sur 20 à chaque RCUE. + La poursuite d'études dans le semestre 5 nécessite de plus la validation + de toutes les UE des semestres 1 et 2 dans les conditions de validation + des points 4.3 (moy_ue >= 10) et 4.4 (compensation rcue), ou par décision + de jury. + """ + + # Codes toujours proposés sauf si include_communs est faux: + codes_communs = [ + sco_codes.RAT, + sco_codes.ABAN, + sco_codes.ABL, + sco_codes.ATJ, + sco_codes.DEF, + sco_codes.DEM, + sco_codes.EXCLU, + ] + + def __init__( + self, + etud: Identite, + formsemestre: FormSemestre, + ): + super().__init__(etud=etud) + self.formsemestre_id = formsemestre.id + formsemestre_impair, formsemestre_pair = self.comp_formsemestres(formsemestre) + assert ( + (formsemestre_pair is None) + or (formsemestre_impair is None) + or ( + ((formsemestre_pair.semestre_id - formsemestre_impair.semestre_id) == 1) + and ( + formsemestre_pair.formation.referentiel_competence_id + == formsemestre_impair.formation.referentiel_competence_id + ) + ) + ) + + self.formsemestre_impair = formsemestre_impair + "le 1er semestre de l'année scolaire considérée (S1, S3, S5)" + self.formsemestre_pair = formsemestre_pair + "le second formsemestre de la même année scolaire (S2, S4, S6)" + self.annee_but = ( + (formsemestre_impair.semestre_id + 1) // 2 + if formsemestre_impair + else (formsemestre_pair.semestre_id + 1) // 2 + ) + "le rang de l'année dans le BUT: 1, 2, 3" + assert self.annee_but in (1, 2, 3) + self.rcues_annee = [] + "RCUEs de l'année" + if self.formsemestre_impair is not None: + self.validation = ApcValidationAnnee.query.filter_by( + etudid=self.etud.id, + formsemestre_id=formsemestre_impair.id, + ordre=self.annee_but, + ).first() + else: + self.validation = None + if self.validation is not None: + self.code_valide = self.validation.code + self.parcour = None + "Le parcours considéré (celui du semestre pair, ou à défaut impair)" + if self.formsemestre_pair is not None: + self.res_pair: ResultatsSemestreBUT = res_sem.load_formsemestre_results( + self.formsemestre_pair + ) + else: + self.res_pair = None + if self.formsemestre_impair is not None: + self.res_impair: ResultatsSemestreBUT = res_sem.load_formsemestre_results( + self.formsemestre_impair + ) + else: + self.res_impair = None + + self.ues_impair, self.ues_pair = self.compute_ues_annee() # pylint: disable=all + self.decisions_ues = { + ue.id: DecisionsProposeesUE(etud, formsemestre_impair, ue) + for ue in self.ues_impair + } + "{ue_id : DecisionsProposeesUE} pour toutes les UE de l'année" + self.decisions_ues.update( + { + ue.id: DecisionsProposeesUE(etud, formsemestre_pair, ue) + for ue in self.ues_pair + } + ) + self.rcues_annee = self.compute_rcues_annee() + + formation = ( + self.formsemestre_impair.formation + if self.formsemestre_impair + else self.formsemestre_pair.formation + ) + self.niveaux_competences = ApcNiveau.niveaux_annee_de_parcours( + self.parcour, self.annee_but, formation.referentiel_competence + ).all() # non triés + "liste des niveaux de compétences associés à cette année" + self.decisions_rcue_by_niveau = self.compute_decisions_niveaux() + "les décisions rcue associées aux niveau_id" + self.dec_rcue_by_ue = self._dec_rcue_by_ue() + "{ ue_id : DecisionsProposeesRCUE } pour toutes les UE associées à un niveau" + self.nb_competences = len(self.niveaux_competences) + "le nombre de niveaux de compétences à valider cette année" + rcues_avec_niveau = [d.rcue for d in self.decisions_rcue_by_niveau.values()] + self.nb_validables = len( + [rcue for rcue in rcues_avec_niveau if rcue.est_validable()] + ) + "le nombre de comp. validables (éventuellement par compensation)" + self.nb_rcues_under_8 = len( + [rcue for rcue in rcues_avec_niveau if not rcue.est_suffisant()] + ) + "le nb de comp. sous la barre de 8/20" + # année ADM si toutes RCUE validées (sinon PASD) + self.admis = self.nb_validables == self.nb_competences + "vrai si l'année est réussie, tous niveaux validables" + self.valide_moitie_rcue = self.nb_validables > (self.nb_competences // 2) + # Peut passer si plus de la moitié validables et tous > 8 + self.passage_de_droit = self.valide_moitie_rcue and (self.nb_rcues_under_8 == 0) + # XXX TODO ajouter condition pour passage en S5 + + # Enfin calcule les codes des UE: + for dec_ue in self.decisions_ues.values(): + dec_ue.compute_codes() + + # Reste à attribuer ADM, ADJ, PASD, PAS1NCI, RED, NAR + expl_rcues = ( + f"{self.nb_validables} niveau validable(s) sur {self.nb_competences}" + ) + if self.admis: + self.codes = [sco_codes.ADM] + self.codes + self.explanation = expl_rcues + elif self.passage_de_droit: + 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 + self.codes = [ + sco_codes.RED, + sco_codes.NAR, + sco_codes.PAS1NCI, + sco_codes.ADJ, + ] + self.codes + self.explanation = expl_rcues + f" et {self.nb_rcues_under_8} < 8" + else: + self.codes = [ + sco_codes.RED, + sco_codes.NAR, + sco_codes.PAS1NCI, + sco_codes.ADJ, + ] + self.codes + self.explanation = ( + expl_rcues + + f""" et {self.nb_rcues_under_8} + niveau{'x' if self.nb_rcues_under_8 > 1 else ''} < 8""" + ) + # + + def infos(self) -> str: + "informations, for debugging purpose" + return f"""DecisionsProposeesAnnee + + """ + + def annee_scolaire(self) -> int: + "L'année de début de l'année scolaire" + formsemestre = self.formsemestre_impair or self.formsemestre_pair + return formsemestre.annee_scolaire() + + def annee_scolaire_str(self) -> str: + "L'année scolaire, eg '2021 - 2022'" + formsemestre = self.formsemestre_impair or self.formsemestre_pair + return formsemestre.annee_scolaire_str().replace(" ", "") + + def comp_formsemestres( + self, formsemestre: FormSemestre + ) -> tuple[FormSemestre, FormSemestre]: + """les deux formsemestres de l'année scolaire à laquelle appartient formsemestre.""" + if not formsemestre.formation.is_apc(): # garde fou + return None, None + if formsemestre.semestre_id % 2 == 0: + other_semestre_id = formsemestre.semestre_id - 1 + else: + other_semestre_id = formsemestre.semestre_id + 1 + annee_scolaire = formsemestre.annee_scolaire() + other_formsemestre = None + for inscr in self.etud.formsemestre_inscriptions: + if ( + # Même spécialité BUT (tolère ainsi des variantes de formation) + ( + inscr.formsemestre.formation.referentiel_competence + == formsemestre.formation.referentiel_competence + ) + # L'autre semestre + and (inscr.formsemestre.semestre_id == other_semestre_id) + # de la même année scolaire: + and (inscr.formsemestre.annee_scolaire() == annee_scolaire) + ): + other_formsemestre = inscr.formsemestre + if formsemestre.semestre_id % 2 == 0: + return other_formsemestre, formsemestre + return formsemestre, other_formsemestre + + def compute_ues_annee(self) -> list[list[UniteEns], list[UniteEns]]: + """UEs à valider cette année pour cet étudiant, selon son parcours. + Ramène [ listes des UE du semestre impair, liste des UE du semestre pair ]. + """ + etudid = self.etud.id + ues_sems = [] + for (formsemestre, res) in ( + (self.formsemestre_impair, self.res_impair), + (self.formsemestre_pair, self.res_pair), + ): + if (formsemestre is None) or (not formsemestre.formation.is_apc()): + ues = [] + else: + formation: Formation = formsemestre.formation + # Parcour dans lequel l'étudiant est inscrit, et liste des UEs + if res.etuds_parcour_id[etudid] is None: + # pas de parcour: prend toutes les UEs (non bonus) + ues = [ue for ue in res.etud_ues(etudid) if ue.type == UE_STANDARD] + ues.sort(key=lambda u: u.numero) + else: + parcour = ApcParcours.query.get(res.etuds_parcour_id[etudid]) + if parcour is not None: + self.parcour = parcour + ues = ( + formation.query_ues_parcour(parcour) + .filter_by(semestre_idx=formsemestre.semestre_id) + .order_by(UniteEns.numero) + .all() + ) + ues_sems.append(ues) + return ues_sems + + def check_ues_ready_jury(self) -> list[str]: + """Vérifie que les toutes les UEs (hors bonus) de l'année sont + bien associées à des niveaux de compétences. + Renvoie liste vide si ok, sinon liste de message explicatifs + """ + messages = [] + for ue in self.ues_impair + self.ues_pair: + if ue.niveau_competence is None: + messages.append( + f"UE {ue.acronyme} non associée à un niveau de compétence" + ) + if ue.semestre_idx is None: + messages.append( + f"UE {ue.acronyme} n'a pas d'indice de semestre dans la formation" + ) + return messages + + def compute_rcues_annee(self) -> list[RegroupementCoherentUE]: + """Liste des regroupements d'UE à considérer cette année. + Pour le moment on ne considère pas de RCUE à cheval sur plusieurs années (redoublants). + Si on n'a pas les deux semestres, aucun RCUE. + Raises ScoValueError s'il y a des UE sans RCUE. + """ + if self.formsemestre_pair is None or self.formsemestre_impair is None: + return [] + rcues_annee = [] + ues_impair_sans_rcue = {ue.id for ue in self.ues_impair} + for ue_pair in self.ues_pair: + rcue = None + for ue_impair in self.ues_impair: + if ue_pair.niveau_competence_id == ue_impair.niveau_competence_id: + rcue = RegroupementCoherentUE( + self.etud, + self.formsemestre_impair, + ue_impair, + self.formsemestre_pair, + ue_pair, + ) + ues_impair_sans_rcue.discard(ue_impair.id) + break + if rcue is None: + raise NoRCUEError(deca=self, ue=ue_pair) + rcues_annee.append(rcue) + if len(ues_impair_sans_rcue) > 0: + ue = UniteEns.query.get(ues_impair_sans_rcue.pop()) + raise NoRCUEError(deca=self, ue=ue) + return rcues_annee + + def compute_decisions_niveaux(self) -> dict[int, "DecisionsProposeesRCUE"]: + """Pour chaque niveau de compétence de cette année, construit + le DecisionsProposeesRCUE, + ou None s'il n'y en a pas + (ne devrait pas arriver car compute_rcues_annee vérifie déjà cela). + Return: { niveau_id : DecisionsProposeesRCUE } + """ + # Retrouve le RCUE associé à chaque niveau + rc_niveaux = [] + for niveau in self.niveaux_competences: + rcue = None + for rc in self.rcues_annee: + if rc.ue_1.niveau_competence_id == niveau.id: + rcue = rc + break + if rcue is not None: + dec_rcue = DecisionsProposeesRCUE(self, rcue) + rc_niveaux.append((dec_rcue, niveau.id)) + # prévient les UE concernées :-) + self.decisions_ues[dec_rcue.rcue.ue_1.id].set_rcue(dec_rcue.rcue) + self.decisions_ues[dec_rcue.rcue.ue_2.id].set_rcue(dec_rcue.rcue) + # Ordonne par numéro d'UE + rc_niveaux.sort(key=lambda x: x[0].rcue.ue_1.numero) + decisions_rcue_by_niveau = {x[1]: x[0] for x in rc_niveaux} + return decisions_rcue_by_niveau + + def _dec_rcue_by_ue(self) -> dict[int, "DecisionsProposeesRCUE"]: + """construit dict { ue_id : DecisionsProposeesRCUE } + à partir de self.decisions_rcue_by_niveau""" + d = {} + for dec_rcue in self.decisions_rcue_by_niveau.values(): + d[dec_rcue.rcue.ue_1.id] = dec_rcue + d[dec_rcue.rcue.ue_2.id] = dec_rcue + return d + + def next_annee_semestre_id(self, code: str) -> int: + """L'indice du semestre dans lequel l'étudiant est autorisé à + poursuivre l'année suivante. None si aucun.""" + if self.formsemestre_pair is None: + return None # seulement sur année + if code == RED: + return self.formsemestre_pair.semestre_id - 1 + elif ( + code in sco_codes.BUT_CODES_PASSAGE + and self.formsemestre_pair.semestre_id < sco_codes.ParcoursBUT.NB_SEM + ): + return self.formsemestre_pair.semestre_id + 1 + return None + + def record_form(self, form: dict): + """Enregistre les codes de jury en base + form dict: + - 'code_ue_1896' : 'AJ' code pour l'UE id 1896 + - 'code_rcue_6" : 'ADM' code pour le RCUE du niveau 6 + - 'code_annee' : 'ADM' code pour l'année + + Si les code_rcue et le code_annee ne sont pas fournis, + et qu'il n'y en a pas déjà, enregistre ceux par défaut. + """ + for key in form: + code = form[key] + # Codes d'UE + m = re.match(r"^code_ue_(\d+)$", key) + if m: + ue_id = int(m.group(1)) + dec_ue = self.decisions_ues.get(ue_id) + if not dec_ue: + raise ScoValueError(f"UE invalide ue_id={ue_id}") + dec_ue.record(code) + else: + # Codes de RCUE + m = re.match(r"^code_rcue_(\d+)$", key) + if m: + niveau_id = int(m.group(1)) + dec_rcue = self.decisions_rcue_by_niveau.get(niveau_id) + if not dec_rcue: + raise ScoValueError(f"RCUE invalide niveau_id={niveau_id}") + dec_rcue.record(code) + elif key == "code_annee": + # Code annuel + self.record(code) + + self.record_all() + db.session.commit() + + def record(self, code: str, no_overwrite=False): + """Enregistre le code de l'année, et au besoin l'autorisation d'inscription. + Si no_overwrite, ne fait rien si un code est déjà enregistré. + """ + if code and not code in self.codes: + raise ScoValueError( + f"code annee {html.escape(code)} invalide pour formsemestre {html.escape(self.formsemestre)}" + ) + if code == self.code_valide or (self.code_valide is not None and no_overwrite): + self.recorded = True + return # no change + if self.validation: + db.session.delete(self.validation) + db.session.flush() + if code is None: + self.validation = None + else: + self.validation = ApcValidationAnnee( + etudid=self.etud.id, + formsemestre=self.formsemestre_impair, + ordre=self.annee_but, + annee_scolaire=self.annee_scolaire(), + code=code, + ) + Scolog.logdb( + method="jury_but", + etudid=self.etud.id, + msg=f"Validation année BUT{self.annee_but}: {code}", + ) + db.session.add(self.validation) + # --- Autorisation d'inscription dans semestre suivant ? + if self.formsemestre_pair is not None: + if code is None: + ScolarAutorisationInscription.delete_autorisation_etud( + etudid=self.etud.id, + origin_formsemestre_id=self.formsemestre_pair.id, + ) + else: + next_semestre_id = self.next_annee_semestre_id(code) + if next_semestre_id is not None: + ScolarAutorisationInscription.autorise_etud( + self.etud.id, + self.formsemestre_pair.formation.formation_code, + self.formsemestre_pair.id, + next_semestre_id, + ) + + self.recorded = True + + def record_all(self): + """Enregistre les codes qui n'ont pas été spécifiés par le formulaire, et sont donc en mode "automatique" """ + decisions = ( + list(self.decisions_ues.values()) + + list(self.decisions_rcue_by_niveau.values()) + + [self] + ) + for dec in decisions: + if not dec.recorded: + # rappel: le code par défaut est en tête + code = dec.codes[0] if dec.codes else None + # s'il n'y a pas de code, efface + dec.record(code, no_overwrite=True) + + def erase(self): + """Efface les décisions de jury de cet étudiant + pour cette année: décisions d'UE, de RCUE, d'année, + et autorisations d'inscription émises. + """ + for dec_ue in self.decisions_ues.values(): + dec_ue.erase() + for dec_rcue in self.decisions_rcue_by_niveau.values(): + dec_rcue.erase() + if self.formsemestre_impair: + ScolarAutorisationInscription.delete_autorisation_etud( + self.etud.id, self.formsemestre_impair.id + ) + if self.formsemestre_pair: + ScolarAutorisationInscription.delete_autorisation_etud( + self.etud.id, self.formsemestre_pair.id + ) + validations = ApcValidationAnnee.query.filter_by( + etudid=self.etud.id, + formsemestre_id=self.formsemestre_impair.id, + ordre=self.annee_but, + ) + for validation in validations: + db.session.delete(validation) + db.session.flush() + + def get_autorisations_passage(self) -> list[int]: + """Les liste des indices de semestres auxquels on est autorisé à + s'inscrire depuis cette année""" + formsemestre = self.formsemestre_pair or self.formsemestre_impair + if not formsemestre: + return [] + return [ + a.semestre_id + for a in ScolarAutorisationInscription.query.filter_by( + etudid=self.etud.id, + origin_formsemestre_id=formsemestre.id, + ) + ] + + def descr_niveaux_validation(self, line_sep: str = "\n") -> str: + """Description textuelle des niveaux validés (enregistrés) + pour PV jurys + """ + validations = [ + dec_rcue.descr_validation() + for dec_rcue in self.decisions_rcue_by_niveau.values() + ] + return line_sep.join(v for v in validations if v) + + def descr_ues_validation(self, line_sep: str = "\n") -> str: + """Description textuelle des UE validées (enregistrés) + pour PV jurys + """ + validations = [] + for res in (self.res_impair, self.res_pair): + if res: + dec_ues = [ + self.decisions_ues[ue.id] + for ue in res.ues + if ue.type == UE_STANDARD and ue.id in self.decisions_ues + ] + valids = [dec_ue.descr_validation() for dec_ue in dec_ues] + validations.append(", ".join(v for v in valids if v)) + return line_sep.join(validations) + + +class DecisionsProposeesRCUE(DecisionsProposees): + """Liste des codes de décisions que l'on peut proposer pour + le RCUE de cet étudiant dans cette année. + + ADM, CMP, ADJ, AJ, RAT, DEF, ABAN + """ + + codes_communs = [ + sco_codes.ADJ, + sco_codes.ATJ, + sco_codes.RAT, + sco_codes.DEF, + sco_codes.ABAN, + ] + + def __init__( + self, dec_prop_annee: DecisionsProposeesAnnee, rcue: RegroupementCoherentUE + ): + super().__init__(etud=dec_prop_annee.etud) + self.rcue = rcue + if rcue is None: # RCUE non dispo, eg un seul semestre + self.codes = [] + return + self.parcour = dec_prop_annee.parcour + self.validation = rcue.query_validations().first() + if self.validation is not None: + self.code_valide = self.validation.code + if rcue.est_compensable(): + self.codes.insert(0, sco_codes.CMP) + elif rcue.est_validable(): + self.codes.insert(0, sco_codes.ADM) + else: + self.codes.insert(0, sco_codes.AJ) + + def record(self, code: str, no_overwrite=False): + """Enregistre le code""" + if code and not code in self.codes: + raise ScoValueError( + f"code UE invalide pour ue_id={self.ue.id}: {html.escape(code)}" + ) + if code == self.code_valide or (self.code_valide is not None and no_overwrite): + self.recorded = True + return # no change + parcours_id = self.parcour.id if self.parcour is not None else None + if self.validation: + db.session.delete(self.validation) + db.session.flush() + if code is None: + self.validation = None + else: + self.validation = ApcValidationRCUE( + etudid=self.etud.id, + formsemestre_id=self.rcue.formsemestre_2.id, + ue1_id=self.rcue.ue_1.id, + ue2_id=self.rcue.ue_2.id, + parcours_id=parcours_id, + code=code, + ) + Scolog.logdb( + method="jury_but", + etudid=self.etud.id, + msg=f"Validation RCUE {repr(self.rcue)}", + ) + db.session.add(self.validation) + self.recorded = True + + def erase(self): + """Efface la décision de jury de cet étudiant pour cet RCUE""" + # par prudence, on requete toutes les validations, en cas de doublons + validations = self.rcue.query_validations() + for validation in validations: + db.session.delete(validation) + db.session.flush() + + def descr_validation(self) -> str: + """Description validation niveau enregistrée, pour PV jury. + Si le niveau est validé, done son acronyme, sinon chaine vide. + """ + if self.code_valide in sco_codes.CODES_RCUE_VALIDES: + if ( + self.rcue and self.rcue.ue_1 and self.rcue.ue_1.niveau_competence + ): # prudence ! + niveau_titre = self.rcue.ue_1.niveau_competence.competence.titre or "" + ordre = self.rcue.ue_1.niveau_competence.ordre + else: + return "?" # oups ? + return f"{niveau_titre} niv. {ordre}" + return "" + + +class DecisionsProposeesUE(DecisionsProposees): + """Décisions de jury sur une UE du BUT + + Liste des codes de décisions que l'on peut proposer pour + cette UE d'un étudiant dans un semestre. + + Si DEF ou DEM ou ABAN ou ABL sur année BUT: seulement DEF, DEM, ABAN, ABL + + si moy_ue > 10, ADM + sinon si compensation dans RCUE: CMP + sinon: ADJ, AJ + + et proposer toujours: RAT, DEF, ABAN, DEM, UEBSL (codes_communs) + """ + + # Codes toujours proposés sauf si include_communs est faux: + codes_communs = [ + sco_codes.RAT, + sco_codes.DEF, + sco_codes.ABAN, + sco_codes.ATJ, + sco_codes.DEM, + sco_codes.UEBSL, + ] + + def __init__( + self, + etud: Identite, + formsemestre: FormSemestre, + ue: UniteEns, + ): + super().__init__(etud=etud) + self.formsemestre = formsemestre + self.ue: UniteEns = ue + self.rcue: RegroupementCoherentUE = None + "Le rcu auquel est rattaché cette UE, ou None" + # Une UE peut être validée plusieurs fois en cas de redoublement (qu'elle soit capitalisée ou non) + # mais ici on a restreint au formsemestre donc une seule (prend la première) + self.validation = ScolarFormSemestreValidation.query.filter_by( + etudid=self.etud.id, formsemestre_id=formsemestre.id, ue_id=ue.id + ).first() + if self.validation is not None: + self.code_valide = self.validation.code + if ue.type == sco_codes.UE_SPORT: + self.explanation = "UE bonus, pas de décision de jury" + self.codes = [] # aucun code proposé + return + + # Moyenne de l'UE ? + res: ResultatsSemestreBUT = res_sem.load_formsemestre_results(formsemestre) + if not ue.id in res.etud_moy_ue: + self.explanation = "UE sans résultat" + return + if not etud.id in res.etud_moy_ue[ue.id]: + self.explanation = "Étudiant sans résultat dans cette UE" + return + self.moy_ue = res.etud_moy_ue[ue.id][etud.id] + + def set_rcue(self, rcue: RegroupementCoherentUE): + """Rattache cette UE à un RCUE. Cela peut modifier les codes + proposés (si compensation)""" + self.rcue = rcue + + def compute_codes(self): + """Calcul des .codes attribuables et de l'explanation associée""" + if self.moy_ue > (sco_codes.ParcoursBUT.BARRE_MOY - sco_codes.NOTES_TOLERANCE): + self.codes.insert(0, sco_codes.ADM) + self.explanation = (f"Moyenne >= {sco_codes.ParcoursBUT.BARRE_MOY}/20",) + elif self.rcue and self.rcue.est_compensable(): + self.codes.insert(0, sco_codes.CMP) + self.explanation = "compensable dans le RCUE" + else: + # Échec à valider cette UE + self.codes = [sco_codes.AJ, sco_codes.ADJ] + self.codes + self.explanation = "notes insuffisantes" + + def record(self, code: str, no_overwrite=False): + """Enregistre le code""" + if code and not code in self.codes: + raise ScoValueError( + f"code UE invalide pour ue_id={self.ue.id}: {html.escape(code)}" + ) + if code == self.code_valide or (self.code_valide is not None and no_overwrite): + self.recorded = True + return # no change + if self.validation: + db.session.delete(self.validation) + db.session.flush() + if code is None: + self.validation = None + else: + self.validation = ScolarFormSemestreValidation( + etudid=self.etud.id, + formsemestre_id=self.formsemestre.id, + ue_id=self.ue.id, + code=code, + moy_ue=self.moy_ue, + ) + Scolog.logdb( + method="jury_but", + etudid=self.etud.id, + msg=f"Validation UE {self.ue.id}", + ) + db.session.add(self.validation) + self.recorded = True + + def erase(self): + """Efface la décision de jury de cet étudiant pour cette UE""" + # par prudence, on requete toutes les validations, en cas de doublons + validations = ScolarFormSemestreValidation.query.filter_by( + etudid=self.etud.id, formsemestre_id=self.formsemestre.id, ue_id=self.ue.id + ) + for validation in validations: + db.session.delete(validation) + db.session.flush() + + def descr_validation(self) -> str: + """Description validation niveau enregistrée, pour PV jury. + Si l'UE est validée, donne son acronyme, sinon chaine vide. + """ + if self.code_valide in sco_codes.CODES_UE_VALIDES: + return f"{self.ue.acronyme}" + return "" + + +class BUTCursusEtud: # WIP TODO + """Validation du cursus d'un étudiant""" + + def __init__(self, formsemestre: FormSemestre, etud: Identite): + if formsemestre.formation.referentiel_competence is None: + raise ScoException("BUTCursusEtud: pas de référentiel de compétences") + assert len(etud.formsemestre_inscriptions) > 0 + self.formsemestre = formsemestre + self.etud = etud + # + # La dernière inscription en date va donner le parcours (donc les compétences à valider) + self.last_inscription = sorted( + etud.formsemestre_inscriptions, key=attrgetter("formsemestre.date_debut") + )[-1] + + def est_diplomable(self) -> bool: + """Vrai si toutes les compétences sont validables""" + return all( + self.competence_validable(competence) + for competence in self.competences_du_parcours() + ) + + def est_diplome(self) -> bool: + """Vrai si BUT déjà validé""" + # vrai si la troisième année est validée + # On cherche les validations de 3ieme annee (ordre=3) avec le même référentiel + # de formation que nous. + return ( + ApcValidationAnnee.query.filter_by(etudid=self.etud.id, ordre=3) + .join(FormSemestre, FormSemestre.id == ApcValidationAnnee.formsemestre_id) + .join(Formation, FormSemestre.formation_id == Formation.id) + .filter( + Formation.referentiel_competence_id + == self.formsemestre.formation.referentiel_competence_id + ) + .count() + > 0 + ) + + def competences_du_parcours(self) -> list[ApcCompetence]: + """Construit liste des compétences du parcours, qui doivent être + validées pour obtenir le diplôme. + Le parcours est celui de la dernière inscription. + """ + parcour = self.last_inscription.parcour + query = self.formsemestre.formation.formation.query_competences_parcour(parcour) + if query is None: + return [] + return query.all() + + def competence_validee(self, competence: ApcCompetence) -> bool: + """Vrai si la compétence est validée, c'est à dire que tous ses + niveaux sont validés (ApcValidationRCUE). + """ + # XXX A REVOIR + validations = ( + ApcValidationRCUE.query.filter_by(etudid=self.etud.id) + .join(UniteEns, UniteEns.id == ApcValidationRCUE.ue1_id) + .join(ApcNiveau, ApcNiveau.id == UniteEns.niveau_competence_id) + .join(ApcCompetence, ApcCompetence.id == ApcNiveau.competence_id) + ) + + def competence_validable(self, competence: ApcCompetence): + """Vrai si la compétence est "validable" automatiquement, c'est à dire + que les conditions de notes sont satisfaites pour l'acquisition de + son niveau le plus élevé, qu'il ne manque que l'enregistrement de la décision. + + En vertu de la règle "La validation des deux UE du niveau d’une compétence + emporte la validation de l'ensemble des UE du niveau inférieur de cette + même compétence.", + il suffit de considérer le dernier niveau dans lequel l'étudiant est inscrit. + """ + pass + + def ues_emportees(self, niveau: ApcNiveau) -> list[tuple[FormSemestre, UniteEns]]: + """La liste des UE à valider si on valide ce niveau. + Ne liste que les UE qui ne sont pas déjà acquises. + + Selon la règle donnée par l'arrêté BUT: + * La validation des deux UE du niveau d’une compétence emporte la validation de + l'ensemble des UE du niveau inférieur de cette même compétence. + """ + pass diff --git a/app/but/jury_but_pv.py b/app/but/jury_but_pv.py new file mode 100644 index 000000000..095c5efc9 --- /dev/null +++ b/app/but/jury_but_pv.py @@ -0,0 +1,137 @@ +############################################################################## +# ScoDoc +# Copyright (c) 1999 - 2022 Emmanuel Viennet. All rights reserved. +# See LICENSE +############################################################################## + +"""Jury BUT: table synthèse résultats semestre / PV +""" +from flask import g, request, url_for + +from openpyxl.styles import Font, Border, Side, Alignment, PatternFill + +from app import log +from app.but import jury_but +from app.models.etudiants import Identite +from app.models.formsemestre import FormSemestre +from app.scodoc.gen_tables import GenTable +from app.scodoc import sco_excel +from app.scodoc.sco_exceptions import ScoValueError +from app.scodoc import sco_preferences +from app.scodoc import sco_utils as scu + + +def _descr_cursus_but(etud: Identite) -> str: + "description de la liste des semestres BUT suivis" + # prend simplement tous les semestre de type APC, ce qui sera faux si + # l'étudiant change de spécialité au sein du même département + # (ce qui ne peut normalement pas se produire) + indices = sorted( + [ + ins.formsemestre.semestre_id + if ins.formsemestre.semestre_id is not None + else -1 + for ins in etud.formsemestre_inscriptions + if ins.formsemestre.formation.is_apc() + ] + ) + return ", ".join(f"S{indice}" for indice in indices) + + +def pvjury_table_but(formsemestre_id: int, format="html") -> list[dict]: + """Page récapitulant les décisions de jury BUT + formsemestre peut être pair ou impair + """ + formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) + assert formsemestre.formation.is_apc() + title = "Procès-verbal de jury BUT annuel" + + if format == "html": + line_sep = "
" + else: + line_sep = "\n" + # remplace pour le BUT la fonction sco_pvjury.pvjury_table + annee_but = (formsemestre.semestre_id + 1) // 2 + titles = { + "nom": "Nom", + "cursus": "Cursus", + "ues": "UE validées", + "niveaux": "Niveaux de compétences validés", + "decision_but": f"Décision BUT{annee_but}", + "diplome": "Résultat au diplôme", + "devenir": "Devenir", + "observations": "Observations", + } + rows = [] + for etudid in formsemestre.etuds_inscriptions: + etud: Identite = Identite.query.get(etudid) + try: + deca = jury_but.DecisionsProposeesAnnee(etud, formsemestre) + if deca.annee_but != annee_but: # wtf ? + log( + f"pvjury_table_but: inconsistent annee_but {deca.annee_but} != {annee_but}" + ) + continue + except ScoValueError: + deca = None + row = { + "nom": etud.etat_civil_pv(line_sep=line_sep), + "_nom_order": etud.sort_key, + "_nom_target_attrs": f'class="etudinfo" id="{etud.id}"', + "_nom_td_attrs": f'id="{etud.id}" class="etudinfo"', + "_nom_target": url_for( + "scolar.ficheEtud", + scodoc_dept=g.scodoc_dept, + etudid=etud.id, + ), + "cursus": _descr_cursus_but(etud), + "ues": deca.descr_ues_validation(line_sep=line_sep) if deca else "-", + "niveaux": deca.descr_niveaux_validation(line_sep=line_sep) + if deca + else "-", + "decision_but": deca.code_valide if deca else "", + "devenir": ", ".join([f"S{i}" for i in deca.get_autorisations_passage()]), + } + + rows.append(row) + + rows.sort(key=lambda x: x["_nom_order"]) + + # Style excel... passages à la ligne sur \n + xls_style_base = sco_excel.excel_make_style() + xls_style_base["alignment"] = Alignment(wrapText=True, vertical="top") + + tab = GenTable( + base_url=f"{request.base_url}?formsemestre_id={formsemestre_id}", + caption=title, + columns_ids=titles.keys(), + html_caption=title, + html_class="pvjury_table_but table_leftalign", + html_title=f"""
{title} + + version excel
+ + """, + html_with_td_classes=True, + origin=f"Généré par {scu.sco_version.SCONAME} le {scu.timedate_human_repr()}", + page_title=title, + pdf_title=title, + preferences=sco_preferences.SemPreferences(), + rows=rows, + table_id="formation_table_recap", + titles=titles, + xls_columns_width={ + "nom": 32, + "cursus": 12, + "ues": 32, + "niveaux": 32, + "decision_but": 14, + "diplome": 17, + "devenir": 8, + "observations": 12, + }, + xls_style_base=xls_style_base, + ) + return tab.make_page(format=format, javascripts=["js/etud_info.js"], init_qtip=True) diff --git a/app/but/jury_but_recap.py b/app/but/jury_but_recap.py new file mode 100644 index 000000000..c2944d98c --- /dev/null +++ b/app/but/jury_but_recap.py @@ -0,0 +1,422 @@ +############################################################################## +# ScoDoc +# Copyright (c) 1999 - 2022 Emmanuel Viennet. All rights reserved. +# See LICENSE +############################################################################## + +"""Jury BUT: table recap annuelle et liens saisie +""" + +import time +import numpy as np +from flask import g, url_for + +from app.but import jury_but +from app.but.jury_but import ( + DecisionsProposeesAnnee, + DecisionsProposeesRCUE, + DecisionsProposeesUE, +) +from app.comp.res_but import ResultatsSemestreBUT +from app.comp import res_sem +from app.models.etudiants import Identite +from app.models.formsemestre import FormSemestre + +from app.scodoc.sco_codes_parcours import ( + BUT_BARRE_RCUE, + BUT_BARRE_UE, + BUT_BARRE_UE8, + BUT_RCUE_SUFFISANT, +) +from app.scodoc import sco_formsemestre_status +from app.scodoc import html_sco_header +from app.scodoc import sco_utils as scu +from app.scodoc.sco_exceptions import ScoValueError + + +def formsemestre_saisie_jury_but( + formsemestre2: FormSemestre, + read_only: bool = False, + selected_etudid: int = None, + mode="jury", +) -> str: + """formsemestre est un semestre PAIR + Si readonly, ne montre pas le lien "saisir la décision" + + => page html complète + + Si mode == "recap", table recap des codes, sans liens de saisie. + """ + # Quick & Dirty + # pour chaque etud de res2 trié + # S1: UE1, ..., UEn + # S2: UE1, ..., UEn + # + # UE1_s1, UE1_s2, moy_rcue, UE2... , Nbrcue_validables, Nbrcue<8, passage_de_droit, valide_moitie_rcue + # + # Pour chaque etud de res2 trié + # DecisionsProposeesAnnee(etud, formsemestre2) + # Pour le 1er etud, faire un check_ues_ready_jury(self) -> page d'erreur + # -> rcue .ue_1, .ue_2 -> stroe moy ues, rcue.moy_rcue, etc + if formsemestre2.semestre_id % 2 != 0: + raise ScoValueError("Cette page ne fonctionne que sur les semestres pairs") + + if formsemestre2.formation.referentiel_competence is None: + raise ScoValueError( + """ +

Pas de référentiel de compétences associé à la formation !

+

Pour associer un référentiel, passer par le menu Semestre / + Voir la formation... et suivre le lien "associer à un référentiel + de compétences" + """ + ) + + rows, titles, column_ids = get_table_jury_but( + formsemestre2, read_only=read_only, mode=mode + ) + if not rows: + return ( + '

aucun étudiant !
' + ) + filename = scu.sanitize_filename( + f"""jury-but-{formsemestre2.titre_num()}-{time.strftime("%Y-%m-%d")}""" + ) + klass = "table_jury_but_bilan" if mode == "recap" else "" + table_html = build_table_jury_but_html( + filename, rows, titles, column_ids, selected_etudid=selected_etudid, klass=klass + ) + H = [ + html_sco_header.sco_header( + page_title=f"{formsemestre2.sem_modalite()}: jury BUT annuel", + no_side_bar=True, + init_qtip=True, + javascripts=["js/etud_info.js", "js/table_recap.js"], + ), + sco_formsemestre_status.formsemestre_status_head( + formsemestre_id=formsemestre2.id + ), + ] + if mode == "recap": + H.append( + f"""

Décisions de jury enregistrées pour les étudiants de ce semestre

+ + """ + ) + H.append( + f""" + + {table_html} + + + + {html_sco_header.sco_footer()} + """ + ) + return "\n".join(H) + + +def build_table_jury_but_html( + filename: str, rows, titles, column_ids, selected_etudid: int = None, klass="" +) -> str: + """assemble la table html""" + footer_rows = [] # inutilisé pour l'instant + H = [ + f"""
""" + ] + # header + H.append( + f""" + + {scu.gen_row(column_ids, titles, "th")} + + """ + ) + # body + H.append("") + for row in rows: + H.append(f"{scu.gen_row(column_ids, row, selected_etudid=selected_etudid)}\n") + H.append("\n") + # footer + H.append("") + idx_last = len(footer_rows) - 1 + for i, row in enumerate(footer_rows): + H.append(f'{scu.gen_row(column_ids, row, "th" if i == idx_last else "td")}\n') + H.append( + """ + +
+
+ """ + ) + return "".join(H) + + +class RowCollector: + """Une ligne de la table""" + + def __init__( + self, + cells: dict = None, + titles: dict = None, + convert_values=True, + column_classes: dict = None, + ): + self.titles = titles + self.row = cells or {} # col_id : str + self.column_classes = column_classes # col_id : str, css class + self.idx = 0 + self.last_etud_cell_idx = 0 + if convert_values: + self.fmt_note = scu.fmt_note + else: + self.fmt_note = lambda x: x + + def __setitem__(self, key, value): + self.row[key] = value + + def __getitem__(self, key): + return self.row[key] + + def get_row_dict(self): + "La ligne, comme un dict" + # create empty cells + for col_id in self.titles: + if col_id not in self.row: + self.row[col_id] = "" + klass = self.column_classes.get(col_id) + if klass: + self.row[f"_{col_id}_class"] = klass + return self.row + + def add_cell( + self, + col_id: str, + title: str, + content: str, + classes: str = "", + idx: int = None, + column_class="", + ): + """Add a row to our table. classes is a list of css class names""" + self.idx = idx if idx is not None else self.idx + self.row[col_id] = content + if classes: + self.row[f"_{col_id}_class"] = classes + f" c{self.idx}" + if not col_id in self.titles: + self.titles[col_id] = title + self.titles[f"_{col_id}_col_order"] = self.idx + if classes: + self.titles[f"_{col_id}_class"] = classes + self.column_classes[col_id] = column_class + self.idx += 1 + + def add_etud_cells(self, etud: Identite, formsemestre: FormSemestre): + "Les cells code, nom, prénom etc." + # --- Codes (seront cachés, mais exportés en excel) + self.add_cell("etudid", "etudid", etud.id, "codes") + self.add_cell("code_nip", "code_nip", etud.code_nip or "", "codes") + # --- Identité étudiant (adapté de res_comon/get_table_recap, à factoriser XXX TODO) + self.add_cell("civilite_str", "Civ.", etud.civilite_str, "identite_detail") + self.add_cell("nom_disp", "Nom", etud.nom_disp(), "identite_detail") + self["_nom_disp_order"] = etud.sort_key + self.add_cell("prenom", "Prénom", etud.prenom, "identite_detail") + self.add_cell("nom_short", "Nom", etud.nom_short, "identite_court") + self["_nom_short_order"] = etud.sort_key + self["_nom_short_target"] = url_for( + "notes.formsemestre_bulletinetud", + scodoc_dept=g.scodoc_dept, + formsemestre_id=formsemestre.id, + etudid=etud.id, + ) + self["_nom_short_target_attrs"] = f'class="etudinfo" id="{etud.id}"' + self["_nom_disp_target"] = self["_nom_short_target"] + self["_nom_disp_target_attrs"] = self["_nom_short_target_attrs"] + self.last_etud_cell_idx = self.idx + + def add_ue_cells(self, dec_ue: DecisionsProposeesUE): + "cell de moyenne d'UE" + col_id = f"moy_ue_{dec_ue.ue.id}" + note_class = "" + val = dec_ue.moy_ue + if isinstance(val, float): + if val < BUT_BARRE_UE: + note_class = " moy_inf" + elif val >= BUT_BARRE_UE: + note_class = " moy_ue_valid" + if val < BUT_BARRE_UE8: + note_class = " moy_ue_warning" # notes très basses + self.add_cell( + col_id, + dec_ue.ue.acronyme, + self.fmt_note(val), + "col_ue" + note_class, + column_class="col_ue", + ) + self.add_cell( + col_id + "_code", + dec_ue.ue.acronyme, + dec_ue.code_valide or "", + "col_ue_code recorded_code", + column_class="col_ue", + ) + + def add_rcue_cells(self, dec_rcue: DecisionsProposeesRCUE): + "2 cells: moyenne du RCUE, code enregistré" + rcue = dec_rcue.rcue + col_id = f"moy_rcue_{rcue.ue_1.niveau_competence_id}" # le niveau_id + note_class = "" + val = rcue.moy_rcue + if isinstance(val, float): + if val < BUT_BARRE_RCUE: + note_class = " moy_ue_inf" + elif val >= BUT_BARRE_RCUE: + note_class = " moy_ue_valid" + if val < BUT_RCUE_SUFFISANT: + note_class = " moy_ue_warning" # notes très basses + self.add_cell( + col_id, + f"
{rcue.ue_1.acronyme}
{rcue.ue_2.acronyme}
", + self.fmt_note(val), + "col_rcue" + note_class, + column_class="col_rcue", + ) + self.add_cell( + col_id + "_code", + f"
{rcue.ue_1.acronyme}
{rcue.ue_2.acronyme}
", + dec_rcue.code_valide or "", + "col_rcue_code recorded_code", + column_class="col_rcue", + ) + + def add_nb_rcues_cell(self, deca: DecisionsProposeesAnnee): + "cell avec nb niveaux validables / total" + klass = " " + if deca.nb_rcues_under_8 > 0: + klass += "moy_ue_warning" + elif deca.nb_validables < deca.nb_competences: + klass += "moy_ue_inf" + else: + klass += "moy_ue_valid" + self.add_cell( + "rcues_validables", + "RCUEs", + f"""{deca.nb_validables}/{deca.nb_competences}""" + + ((" " + scu.EMO_WARNING) if deca.nb_rcues_under_8 > 0 else ""), + "col_rcue col_rcues_validables" + klass, + ) + self["_rcues_validables_data"] = { + "etudid": deca.etud.id, + "nomprenom": deca.etud.nomprenom, + } + if len(deca.rcues_annee) > 0: + # permet un tri par nb de niveaux validables + moyenne gen indicative S_pair + if deca.res_pair and deca.etud.id in deca.res_pair.etud_moy_gen: + moy = deca.res_pair.etud_moy_gen[deca.etud.id] + if np.isnan(moy): + moy_gen_d = "x" + else: + moy_gen_d = f"{int(moy*1000):05}" + else: + moy_gen_d = "x" + self["_rcues_validables_order"] = f"{deca.nb_validables:04d}-{moy_gen_d}" + else: + # etudiants sans RCUE: pas de semestre impair, ... + # les classe à la fin + self[ + "_rcues_validables_order" + ] = f"{deca.nb_validables:04d}-00000-{deca.etud.sort_key}" + + +def get_table_jury_but( + formsemestre2: FormSemestre, read_only: bool = False, mode="jury" +) -> tuple[list[dict], list[str], list[str]]: + """Construit la table des résultats annuels pour le jury BUT""" + res2: ResultatsSemestreBUT = res_sem.load_formsemestre_results(formsemestre2) + titles = {} # column_id : title + column_classes = {} + rows = [] + for etudid in formsemestre2.etuds_inscriptions: + etud: Identite = Identite.query.get(etudid) + deca = jury_but.DecisionsProposeesAnnee(etud, formsemestre2) + row = RowCollector(titles=titles, column_classes=column_classes) + row.add_etud_cells(etud, formsemestre2) + row.idx = 100 # laisse place pour les colonnes de groupes + # --- Nombre de niveaux + row.add_nb_rcues_cell(deca) + # --- Les RCUEs + for rcue in deca.rcues_annee: + dec_rcue = deca.dec_rcue_by_ue.get(rcue.ue_1.id) + if dec_rcue is not None: # None si l'UE n'est pas associée à un niveau + row.add_ue_cells(deca.decisions_ues[rcue.ue_1.id]) + row.add_ue_cells(deca.decisions_ues[rcue.ue_2.id]) + row.add_rcue_cells(dec_rcue) + # --- Le code annuel existant + row.add_cell( + "code_annee", + "Année", + f"""{deca.code_valide or ''}""", + "col_code_annee", + ) + # --- Le lien de saisie + if mode != "recap": + row.add_cell( + "lien_saisie", + "", + f""" + + {"voir" if read_only else ("modif." if deca.code_valide else "saisie")} + décision + """, + "col_lien_saisie_but", + ) + rows.append(row) + rows_dict = [row.get_row_dict() for row in rows] + if len(rows_dict) > 0: + res2.recap_add_partitions(rows_dict, titles, col_idx=row.last_etud_cell_idx + 1) + column_ids = [title for title in titles if not title.startswith("_")] + column_ids.sort(key=lambda col_id: titles.get("_" + col_id + "_col_order", 1000)) + rows_dict.sort(key=lambda row: row["_nom_disp_order"]) + return rows_dict, titles, column_ids diff --git a/app/but/jury_but_validation_auto.py b/app/but/jury_but_validation_auto.py new file mode 100644 index 000000000..99512168b --- /dev/null +++ b/app/but/jury_but_validation_auto.py @@ -0,0 +1,34 @@ +############################################################################## +# ScoDoc +# Copyright (c) 1999 - 2022 Emmanuel Viennet. All rights reserved. +# See LICENSE +############################################################################## + +"""Jury BUT: clacul des décisions de jury annuelles "automatiques" +""" + +from flask import g, url_for + +from app import db +from app.but import jury_but +from app.models.etudiants import Identite +from app.models.formsemestre import FormSemestre +from app.scodoc.sco_exceptions import ScoValueError + + +def formsemestre_validation_auto_but(formsemestre: FormSemestre) -> int: + """Calcul automatique des décisions de jury sur une année BUT. + Returns: nombre d'étudiants "admis" + """ + if not formsemestre.formation.is_apc(): + raise ScoValueError("fonction réservée aux formations BUT") + nb_admis = 0 + for etudid in formsemestre.etuds_inscriptions: + etud: Identite = Identite.query.get(etudid) + deca = jury_but.DecisionsProposeesAnnee(etud, formsemestre) + if deca.admis: # année réussie + deca.record_all() + nb_admis += 1 + + db.session.commit() + return nb_admis diff --git a/app/but/jury_but_view.py b/app/but/jury_but_view.py new file mode 100644 index 000000000..fa5171200 --- /dev/null +++ b/app/but/jury_but_view.py @@ -0,0 +1,173 @@ +############################################################################## +# ScoDoc +# Copyright (c) 1999 - 2022 Emmanuel Viennet. All rights reserved. +# See LICENSE +############################################################################## + +"""Jury BUT: affichage/formulaire +""" +from flask import g, url_for +from app.models.etudiants import Identite + +from app.scodoc import sco_utils as scu +from app.but.jury_but import DecisionsProposeesAnnee, DecisionsProposeesUE +from app.models import FormSemestre, FormSemestreInscription, UniteEns +from app.scodoc.sco_exceptions import ScoValueError + + +def show_etud(deca: DecisionsProposeesAnnee, read_only: bool = True) -> str: + """Affichage des décisions annuelles BUT + Si pas read_only, menus sélection codes jury. + """ + H = [] + if deca.code_valide and not read_only: + erase_span = f"""effacer décisions""" + else: + erase_span = "" + + H.append( + f"""
+
+ Décision de jury pour l'année : { + _gen_but_select("code_annee", deca.codes, deca.code_valide, + disabled=True, klass="manual") + } + ({'non ' if deca.code_valide is None else ''}enregistrée) + {erase_span} +
+
{deca.explanation}
+
""" + ) + + H.append( + f""" +
Niveaux de compétences et unités d'enseignement :
+
+
+
S{1}
+
S{2}
+
RCUE
+ """ + ) + for niveau in deca.niveaux_competences: + H.append( + f"""
+
{niveau.competence.titre}
+
""" + ) + dec_rcue = deca.decisions_rcue_by_niveau.get(niveau.id) + if dec_rcue is None: + break + # Semestre impair + H.append( + _gen_but_niveau_ue( + dec_rcue.rcue.ue_1, + dec_rcue.rcue.moy_ue_1, + deca.decisions_ues[dec_rcue.rcue.ue_1.id], + disabled=read_only, + ) + ) + # Semestre pair + H.append( + _gen_but_niveau_ue( + dec_rcue.rcue.ue_2, + dec_rcue.rcue.moy_ue_2, + deca.decisions_ues[dec_rcue.rcue.ue_2.id], + disabled=read_only, + ) + ) + # RCUE + H.append( + f"""
+
{scu.fmt_note(dec_rcue.rcue.moy_rcue)}
+
{ + _gen_but_select("code_rcue_"+str(niveau.id), + dec_rcue.codes, + dec_rcue.code_valide, + disabled=True, klass="manual" + ) + }
+
""" + ) + H.append("
") # but_annee + return "\n".join(H) + + +def _gen_but_select( + name: str, + codes: list[str], + code_valide: str, + disabled: bool = False, + klass: str = "", +) -> str: + "Le menu html select avec les codes" + h = "\n".join( + [ + f"""""" + for code in codes + ] + ) + return f""" + """ + + +def _gen_but_niveau_ue( + ue: UniteEns, moy_ue: float, dec_ue: DecisionsProposeesUE, disabled=False +): + return f"""
+
{ue.acronyme}
+
{scu.fmt_note(moy_ue)}
+
{ + _gen_but_select("code_ue_"+str(ue.id), + dec_ue.codes, + dec_ue.code_valide, disabled=disabled + ) + }
+
""" + + +# +def infos_fiche_etud_html(etudid: int) -> str: + """Section html pour fiche etudiant + provisoire pour BUT 2022 + """ + etud: Identite = Identite.query.get_or_404(etudid) + inscriptions = ( + FormSemestreInscription.query.join(FormSemestreInscription.formsemestre) + .filter( + FormSemestreInscription.etudid == etud.id, + ) + .order_by(FormSemestre.date_debut) + ) + formsemestres_but = [ + i.formsemestre for i in inscriptions if i.formsemestre.formation.is_apc() + ] + if len(formsemestres_but) == 0: + return "" + + # temporaire quick & dirty: affiche le dernier + try: + deca = DecisionsProposeesAnnee(etud, formsemestres_but[-1]) + if len(deca.rcues_annee) > 0: + return f"""
+ {show_etud(deca, read_only=True)} +
+ """ + except ScoValueError: + pass + + return "" diff --git a/app/comp/bonus_spo.py b/app/comp/bonus_spo.py index 6cf0767f0..88eb36fc9 100644 --- a/app/comp/bonus_spo.py +++ b/app/comp/bonus_spo.py @@ -767,6 +767,21 @@ class BonusStMalo(BonusIUTRennes1): __doc__ = BonusIUTRennes1.__doc__ +class BonusLaRocheSurYon(BonusSportAdditif): + """Bonus IUT de La Roche-sur-Yon + + Si une note de bonus est saisie, l'étudiant est gratifié de 0,2 points + sur sa moyenne générale ou, en BUT, sur la moyenne de chaque UE. + """ + + name = "bonus_larochesuryon" + displayed_name = "IUT de La Roche-sur-Yon" + seuil_moy_gen = 0.0 + seuil_comptage = 0.0 + proportion_point = 1e10 # le moindre point sature le bonus + bonus_max = 0.2 # à 0.2 + + class BonusLaRochelle(BonusSportAdditif): """Calcul bonus modules optionnels (sport, culture), règle IUT de La Rochelle. @@ -1023,6 +1038,54 @@ class BonusNantes(BonusSportAdditif): bonus_max = 0.5 # plafonnement à 0.5 points +class BonusOrleans(BonusSportAdditif): + """Calcul bonus modules optionnels (sport, culture), règle IUT d'Orléans +

Cadre général : + En reconnaissance de l'engagement des étudiants dans la vie associative, + sociale ou professionnelle, l’IUT d’Orléans accorde, sous conditions, + une bonification aux étudiants inscrits qui en font la demande en début + d’année universitaire. +

+

Cet engagement doit être régulier et correspondre à une activité réelle + et sérieuse qui bénéficie à toute la communauté étudiante de l’IUT, + de l’Université ou à l’ensemble de la collectivité.

+

Bonification : + Pour les DUT et LP, cette bonification interviendra sur la moyenne générale + des semestres pairs : +

+ Pour le BUT, cette bonification interviendra sur la moyenne de chacune + des UE des semestre pairs : + + La bonification ne peut dépasser +0,5 points par année universitaire. +

+

Avant février 2020 : + Un bonus de 2,5% de la note de sport est accordé à la moyenne générale. +

+ """ + + name = "bonus_iutorleans" + displayed_name = "IUT d'Orléans" + bonus_max = 0.5 + seuil_moy_gen = 0.0 # seuls les points au dessus du seuil sont comptés + proportion_point = 1 + classic_use_bonus_ues = False + + def compute_bonus(self, sem_modimpl_moys_inscrits, modimpl_coefs_etuds_no_nan): + if self.formsemestre.date_debut > datetime.date(2020, 2, 1): + self.proportion_point = 1.0 + else: + self.proportion_point = 2.5 / 100.0 + return super().compute_bonus( + sem_modimpl_moys_inscrits, modimpl_coefs_etuds_no_nan + ) + + class BonusPoitiers(BonusSportAdditif): """Calcul bonus optionnels (sport, culture), règle IUT de Poitiers. diff --git a/app/comp/moy_mod.py b/app/comp/moy_mod.py index 8f8bd1a89..aba032fd9 100644 --- a/app/comp/moy_mod.py +++ b/app/comp/moy_mod.py @@ -161,8 +161,11 @@ class ModuleImplResults: evals_notes = evals_notes.merge( eval_df, how="left", left_index=True, right_index=True ) - # Notes en attente: (on prend dans evals_notes pour ne pas avoir les dem.) - nb_att = sum(evals_notes[str(evaluation.id)] == scu.NOTES_ATTENTE) + # Notes en attente: (ne prend en compte que les inscrits, non démissionnaires) + nb_att = sum( + evals_notes[str(evaluation.id)][list(inscrits_module)] + == scu.NOTES_ATTENTE + ) self.evaluations_etat[evaluation.id] = EvaluationEtat( evaluation_id=evaluation.id, nb_attente=nb_att, is_complete=is_complete ) diff --git a/app/comp/moy_ue.py b/app/comp/moy_ue.py index 2d337a3d0..5f432387c 100644 --- a/app/comp/moy_ue.py +++ b/app/comp/moy_ue.py @@ -496,17 +496,26 @@ def compute_malus( """ ues_idx = [ue.id for ue in ues] malus = pd.DataFrame(index=modimpl_inscr_df.index, columns=ues_idx, dtype=float) + if len(sem_modimpl_moys.flat) == 0: # vide + return malus + if len(sem_modimpl_moys.shape) > 2: + # BUT: ne retient que la 1er composante du malus qui est scalaire + # au sens ou chaque note de malus n'affecte que la moyenne de l'UE + # de rattachement de son module. + sem_modimpl_moys_scalar = sem_modimpl_moys[:, :, 0] + else: # classic + sem_modimpl_moys_scalar = sem_modimpl_moys for ue in ues: if ue.type != UE_SPORT: modimpl_mask = np.array( [ (m.module.module_type == ModuleType.MALUS) - and (m.module.ue.id == ue.id) + and (m.module.ue.id == ue.id) # UE de rattachement for m in formsemestre.modimpls_sorted ] ) if len(modimpl_mask): - malus_moys = sem_modimpl_moys[:, modimpl_mask].sum(axis=1) + malus_moys = sem_modimpl_moys_scalar[:, modimpl_mask].sum(axis=1) malus[ue.id] = malus_moys malus.fillna(0.0, inplace=True) diff --git a/app/comp/res_but.py b/app/comp/res_but.py index d7fec7863..0b58521ae 100644 --- a/app/comp/res_but.py +++ b/app/comp/res_but.py @@ -6,6 +6,8 @@ """Résultats semestres BUT """ +from collections.abc import Generator +from re import U import time import numpy as np import pandas as pd @@ -28,6 +30,8 @@ class ResultatsSemestreBUT(NotesTableCompat): "modimpl_coefs_df", "modimpls_evals_poids", "sem_cube", + "etuds_parcour_id", # parcours de chaque étudiant + "ues_inscr_parcours_df", # inscriptions aux UE / parcours ) def __init__(self, formsemestre): @@ -35,7 +39,8 @@ class ResultatsSemestreBUT(NotesTableCompat): self.sem_cube = None """ndarray (etuds x modimpl x ue)""" - + self.etuds_parcour_id = None + """Parcours de chaque étudiant { etudid : parcour_id }""" if not self.load_cached(): t0 = time.time() self.compute() @@ -55,6 +60,7 @@ class ResultatsSemestreBUT(NotesTableCompat): self.modimpls_results, ) = moy_ue.notes_sem_load_cube(self.formsemestre) self.modimpl_inscr_df = inscr_mod.df_load_modimpl_inscr(self.formsemestre) + self.ues_inscr_parcours_df = self.load_ues_inscr_parcours() self.modimpl_coefs_df, _, _ = moy_ue.df_load_modimpl_coefs( self.formsemestre, modimpls=self.formsemestre.modimpls_sorted ) @@ -108,6 +114,9 @@ class ResultatsSemestreBUT(NotesTableCompat): # Clippe toutes les moyennes d'UE dans [0,20] self.etud_moy_ue.clip(lower=0.0, upper=20.0, inplace=True) + # Nanifie les moyennes d'UE hors parcours pour chaque étudiant + self.etud_moy_ue *= self.ues_inscr_parcours_df + # Moyenne générale indicative: # (note: le bonus sport a déjà été appliqué aux moyennes d'UE, et impacte # donc la moyenne indicative) @@ -149,16 +158,24 @@ class ResultatsSemestreBUT(NotesTableCompat): """ return self.modimpl_coefs_df.loc[ue.id].sum() - def modimpls_in_ue(self, ue_id, etudid, with_bonus=True) -> list[ModuleImpl]: + def modimpls_in_ue(self, ue: UniteEns, etudid, with_bonus=True) -> list[ModuleImpl]: """Liste des modimpl ayant des coefs non nuls vers cette UE et auxquels l'étudiant est inscrit. Inclus modules bonus le cas échéant. """ # sert pour l'affichage ou non de l'UE sur le bulletin et la table recap - coefs = self.modimpl_coefs_df # row UE, cols modimpl + if ue.type == UE_SPORT: + return [ + modimpl + for modimpl in self.formsemestre.modimpls_sorted + if modimpl.module.ue.id == ue.id + and self.modimpl_inscr_df[modimpl.id][etudid] + ] + coefs = self.modimpl_coefs_df # row UE (sans bonus), cols modimpl modimpls = [ modimpl for modimpl in self.formsemestre.modimpls_sorted - if (coefs[modimpl.id][ue_id] != 0) + if modimpl.module.ue.type != UE_SPORT + and (coefs[modimpl.id][ue.id] != 0) and self.modimpl_inscr_df[modimpl.id][etudid] ] if not with_bonus: @@ -175,3 +192,50 @@ class ResultatsSemestreBUT(NotesTableCompat): i = self.modimpl_coefs_df.columns.get_loc(modimpl_id) j = self.modimpl_coefs_df.index.get_loc(ue_id) return self.sem_cube[:, i, j] + + def load_ues_inscr_parcours(self) -> pd.DataFrame: + """Chargement des inscriptions aux parcours et calcul de la + matrice d'inscriptions (etuds, ue). + S'il n'y pas de référentiel de compétence, donc pas de parcours, + on considère l'étudiant inscrit à toutes les ue. + La matrice avec ue ne comprend que les UE non bonus. + 1.0 si étudiant inscrit à l'UE, NaN sinon. + """ + etuds_parcour_id = { + inscr.etudid: inscr.parcour_id for inscr in self.formsemestre.inscriptions + } + self.etuds_parcour_id = etuds_parcour_id + ue_ids = [ue.id for ue in self.ues if ue.type != UE_SPORT] + # matrice de 1, inscrits par défaut à toutes les UE: + ues_inscr_parcours_df = pd.DataFrame( + 1.0, index=etuds_parcour_id.keys(), columns=ue_ids, dtype=float + ) + if self.formsemestre.formation.referentiel_competence is None: + return ues_inscr_parcours_df + + ue_by_parcours = {} # parcours_id : {ue_id:0|1} + for parcour in self.formsemestre.formation.referentiel_competence.parcours: + ue_by_parcours[parcour.id] = { + ue.id: 1.0 + for ue in self.formsemestre.formation.query_ues_parcour( + parcour + ).filter_by(semestre_idx=self.formsemestre.semestre_id) + } + for etudid in etuds_parcour_id: + parcour = etuds_parcour_id[etudid] + if parcour is not None: + ues_inscr_parcours_df.loc[etudid] = ue_by_parcours[ + etuds_parcour_id[etudid] + ] + return ues_inscr_parcours_df + + def etud_ues_ids(self, etudid: int) -> list[int]: + """Liste des id d'UE auxquelles l'étudiant est inscrit (sans bonus). + (surchargée ici pour prendre en compte les parcours) + """ + s = self.ues_inscr_parcours_df.loc[etudid] + return s.index[s.notna()] + + def etud_ues(self, etudid: int) -> Generator[UniteEns]: + """Liste des UE auxquelles l'étudiant est inscrit.""" + return (UniteEns.query.get(ue_id) for ue_id in self.etud_ues_ids(etudid)) diff --git a/app/comp/res_common.py b/app/comp/res_common.py index 645f067e6..04bd92d13 100644 --- a/app/comp/res_common.py +++ b/app/comp/res_common.py @@ -112,6 +112,14 @@ class ResultatsSemestre(ResultatsCache): "dict { etudid : indice dans les inscrits }" return {e.id: idx for idx, e in enumerate(self.etuds)} + def etud_ues_ids(self, etudid: int) -> list[int]: + """Liste des UE auxquelles l'etudiant est inscrit, sans bonus + (surchargée en BUT pour prendre en compte les parcours) + """ + # Pour les formations classiques, etudid n'est pas utilisé + # car tous les étudiants sont inscrits à toutes les UE + return [ue.id for ue in self.ues if ue.type != UE_SPORT] + def modimpl_notes(self, modimpl_id: int, ue_id: int) -> np.ndarray: """Les notes moyennes des étudiants du sem. à ce modimpl dans cette ue. Utile pour stats bottom tableau recap. @@ -179,7 +187,7 @@ class ResultatsSemestre(ResultatsCache): ues = sorted(list(ues), key=lambda x: x.numero or 0) return ues - def modimpls_in_ue(self, ue_id, etudid, with_bonus=True) -> list[ModuleImpl]: + def modimpls_in_ue(self, ue: UniteEns, etudid, with_bonus=True) -> list[ModuleImpl]: """Liste des modimpl de cette UE auxquels l'étudiant est inscrit. Utile en formations classiques, surchargée pour le BUT. Inclus modules bonus le cas échéant. @@ -189,7 +197,7 @@ class ResultatsSemestre(ResultatsCache): modimpls = [ modimpl for modimpl in self.formsemestre.modimpls_sorted - if modimpl.module.ue.id == ue_id + if modimpl.module.ue.id == ue.id and self.modimpl_inscr_df[modimpl.id][etudid] ] if not with_bonus: @@ -391,7 +399,7 @@ class ResultatsSemestre(ResultatsCache): # --- TABLEAU RECAP def get_table_recap( - self, convert_values=False, include_evaluations=False, modejury=False + self, convert_values=False, include_evaluations=False, mode_jury=False ): """Result: tuple avec - rows: liste de dicts { column_id : value } @@ -542,7 +550,7 @@ class ResultatsSemestre(ResultatsCache): titles_bot[ f"_{col_id}_target_attrs" ] = f"""title="{ue.titre} S{ue.semestre_idx or '?'}" """ - if modejury: + if mode_jury: # pas d'autre colonnes de résultats continue # Bonus (sport) dans cette UE ? @@ -564,7 +572,7 @@ class ResultatsSemestre(ResultatsCache): # Les moyennes des modules (ou ressources et SAÉs) dans cette UE idx_malus = idx # place pour colonne malus à gauche des modules idx += 1 - for modimpl in self.modimpls_in_ue(ue.id, etudid, with_bonus=False): + for modimpl in self.modimpls_in_ue(ue, etudid, with_bonus=False): if ue_status["is_capitalized"]: val = "-c-" else: @@ -622,9 +630,10 @@ class ResultatsSemestre(ResultatsCache): f"_{col_id}_target_attrs" ] = f""" title="{modimpl.module.titre} ({nom_resp})" """ modimpl_ids.add(modimpl.id) + nb_ues_etud_parcours = len(self.etud_ues_ids(etudid)) ue_valid_txt = ( ue_valid_txt_html - ) = f"{nb_ues_validables}/{len(ues_sans_bonus)}" + ) = f"{nb_ues_validables}/{nb_ues_etud_parcours}" if nb_ues_warning: ue_valid_txt_html += " " + scu.EMO_WARNING add_cell( @@ -641,7 +650,17 @@ class ResultatsSemestre(ResultatsCache): elif nb_ues_validables < len(ues_sans_bonus): row["_ues_validables_class"] += " moy_inf" row["_ues_validables_order"] = nb_ues_validables # pour tri - if modejury: + if mode_jury: + dec_sem = self.validations.decisions_jury.get(etudid) + jury_code_sem = dec_sem["code"] if dec_sem else "" + idx = add_cell( + row, + "jury_code_sem", + "Jury", + jury_code_sem, + "jury_code_sem", + 1000, + ) idx = add_cell( row, "jury_link", @@ -651,11 +670,11 @@ class ResultatsSemestre(ResultatsCache): ) }">saisir décision""", "col_jury_link", - 1000, + idx, ) rows.append(row) - self._recap_add_partitions(rows, titles) + self.recap_add_partitions(rows, titles) self._recap_add_admissions(rows, titles) # tri par rang croissant @@ -762,7 +781,9 @@ class ResultatsSemestre(ResultatsCache): "apo": row_apo, } - def _recap_etud_groups_infos(self, etudid: int, row: dict, titles: dict): + def _recap_etud_groups_infos( + self, etudid: int, row: dict, titles: dict + ): # XXX non utilisé """Table recap: ajoute à row les colonnes sur les groupes pour cet etud""" # dec = self.get_etud_decision_sem(etudid) # if dec: @@ -818,7 +839,7 @@ class ResultatsSemestre(ResultatsCache): else: row[f"_{cid}_class"] = "admission" - def _recap_add_partitions(self, rows: list[dict], titles: dict): + def recap_add_partitions(self, rows: list[dict], titles: dict, col_idx: int = None): """Ajoute les colonnes indiquant les groupes rows est une liste de dict avec une clé "etudid" Les colonnes ont la classe css "partition" @@ -827,7 +848,7 @@ class ResultatsSemestre(ResultatsCache): self.formsemestre.id ) first_partition = True - col_order = 10 + col_order = 10 if col_idx is None else col_idx for partition in partitions: cid = f"part_{partition['partition_id']}" rg_cid = cid + "_rg" # rang dans la partition diff --git a/app/comp/res_compat.py b/app/comp/res_compat.py index 5ac18ff4e..d48c727de 100644 --- a/app/comp/res_compat.py +++ b/app/comp/res_compat.py @@ -54,6 +54,7 @@ class NotesTableCompat(ResultatsSemestre): self.ue_rangs_by_group = {} # { ue_id : {group_id : (Series, Series)}} self.expr_diagnostics = "" self.parcours = self.formsemestre.formation.get_parcours() + self._modimpls_dict_by_ue = {} # local cache def get_inscrits(self, include_demdef=True, order_by=False) -> list[Identite]: """Liste des étudiants inscrits @@ -145,6 +146,10 @@ class NotesTableCompat(ResultatsSemestre): """Liste des modules pour une UE (ou toutes si ue_id==None), triés par numéros (selon le type de formation) """ + # cached ? + modimpls_dict = self._modimpls_dict_by_ue.get(ue_id) + if modimpls_dict: + return modimpls_dict modimpls_dict = [] for modimpl in self.formsemestre.modimpls_sorted: if (ue_id is None) or (modimpl.module.ue.id == ue_id): @@ -152,6 +157,7 @@ class NotesTableCompat(ResultatsSemestre): # compat ScoDoc < 9.2: ajoute matières d["mat"] = modimpl.module.matiere.to_dict() modimpls_dict.append(d) + self._modimpls_dict_by_ue[ue_id] = modimpls_dict return modimpls_dict def compute_rangs(self): diff --git a/app/decorators.py b/app/decorators.py index d6c6ed234..83441275e 100644 --- a/app/decorators.py +++ b/app/decorators.py @@ -87,10 +87,10 @@ def permission_required(permission): def decorated_function(*args, **kwargs): scodoc_dept = getattr(g, "scodoc_dept", None) if not current_user.has_permission(permission, scodoc_dept): - abort(403) + return current_app.login_manager.unauthorized() return f(*args, **kwargs) - return login_required(decorated_function) + return decorated_function return decorator diff --git a/app/forms/main/config_apo.py b/app/forms/main/config_apo.py index 9a5e11989..946e6ff29 100644 --- a/app/forms/main/config_apo.py +++ b/app/forms/main/config_apo.py @@ -41,6 +41,7 @@ from app.scodoc import sco_codes_parcours def _build_code_field(code): return StringField( label=code, + default=code, description=sco_codes_parcours.CODES_EXPL[code], validators=[ validators.regexp( @@ -58,6 +59,8 @@ def _build_code_field(code): class CodesDecisionsForm(FlaskForm): "Formulaire code décisions Apogée" + ABAN = _build_code_field("ABAN") + ABL = _build_code_field("ABL") ADC = _build_code_field("ADC") ADJ = _build_code_field("ADJ") ADM = _build_code_field("ADM") @@ -68,8 +71,13 @@ class CodesDecisionsForm(FlaskForm): CMP = _build_code_field("CMP") DEF = _build_code_field("DEF") DEM = _build_code_field("DEM") + EXCLU = _build_code_field("EXCLU") NAR = _build_code_field("NAR") + PASD = _build_code_field("PASD") + PAS1NCI = _build_code_field("PAS1NCI") RAT = _build_code_field("RAT") + RED = _build_code_field("RED") + NOTES_FMT = StringField( label="Format notes exportées", description="""Format des notes. Par défaut %3.2f (deux chiffres après la virgule)""", diff --git a/app/forms/main/config_logos.py b/app/forms/main/config_logos.py index 0d0ac0d5a..b35ac34e1 100644 --- a/app/forms/main/config_logos.py +++ b/app/forms/main/config_logos.py @@ -43,7 +43,7 @@ from app.scodoc import sco_logos, html_sco_header from app.scodoc import sco_utils as scu from app.scodoc.sco_config_actions import LogoInsert - +from app.scodoc.sco_exceptions import ScoValueError from app.scodoc.sco_logos import find_logo @@ -108,6 +108,8 @@ def dept_key_to_id(dept_key): def logo_name_validator(message=None): def validate_logo_name(form, field): name = field.data if field.data else "" + if "." in name: + raise ValidationError(message) if not scu.is_valid_filename(name): raise ValidationError(message) @@ -199,9 +201,12 @@ class LogoForm(FlaskForm): def __init__(self, *args, **kwargs): kwargs["meta"] = {"csrf": False} super().__init__(*args, **kwargs) - self.logo = find_logo( + logo = find_logo( logoname=self.logo_id.data, dept_id=dept_key_to_id(self.dept_key.data) - ).select() + ) + if logo is None: + raise ScoValueError("logo introuvable") + self.logo = logo.select() self.description = None self.titre = None self.can_delete = True diff --git a/app/models/__init__.py b/app/models/__init__.py index d29b6bf3b..c7a183ec3 100644 --- a/app/models/__init__.py +++ b/app/models/__init__.py @@ -1,14 +1,25 @@ # -*- coding: UTF-8 -* """Modèles base de données ScoDoc -XXX version préliminaire ScoDoc8 #sco8 sans département """ +import sqlalchemy + CODE_STR_LEN = 16 # chaine pour les codes SHORT_STR_LEN = 32 # courtes chaine, eg acronymes APO_CODE_STR_LEN = 512 # nb de car max d'un code Apogée (il peut y en avoir plusieurs) GROUPNAME_STR_LEN = 64 +convention = { + "ix": "ix_%(column_0_label)s", + "uq": "uq_%(table_name)s_%(column_0_name)s", + "ck": "ck_%(table_name)s_%(constraint_name)s", + "fk": "fk_%(table_name)s_%(column_0_name)s_%(referred_table_name)s", + "pk": "pk_%(table_name)s", +} + +metadata_obj = sqlalchemy.MetaData(naming_convention=convention) + from app.models.raw_sql_init import create_database_functions from app.models.absences import Absence, AbsenceNotification, BilletAbsence @@ -65,5 +76,8 @@ from app.models.but_refcomp import ( ApcCompetence, ApcSituationPro, ApcAppCritique, + ApcParcours, ) +from app.models.but_validations import ApcValidationAnnee, ApcValidationRCUE + from app.models.config import ScoDocSiteConfig diff --git a/app/models/absences.py b/app/models/absences.py index 830d46f9e..405ea6bff 100644 --- a/app/models/absences.py +++ b/app/models/absences.py @@ -11,7 +11,9 @@ class Absence(db.Model): __tablename__ = "absences" id = db.Column(db.Integer, primary_key=True) - etudid = db.Column(db.Integer, db.ForeignKey("identite.id"), index=True) + etudid = db.Column( + db.Integer, db.ForeignKey("identite.id", ondelete="CASCADE"), index=True + ) jour = db.Column(db.Date) estabs = db.Column(db.Boolean()) estjust = db.Column(db.Boolean()) @@ -50,7 +52,7 @@ class AbsenceNotification(db.Model): id = db.Column(db.Integer, primary_key=True) etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), ) notification_date = db.Column( db.DateTime(timezone=True), server_default=db.func.now() diff --git a/app/models/but_pn.py b/app/models/but_pn.py index 35afbe16c..67993d023 100644 --- a/app/models/but_pn.py +++ b/app/models/but_pn.py @@ -1,12 +1,9 @@ """ScoDoc 9 models : Formation BUT 2021 + XXX inutilisé """ -from enum import unique -from typing import Any from app import db -from app.scodoc.sco_utils import ModuleType - class APCFormation(db.Model): """Formation par compétence""" diff --git a/app/models/but_refcomp.py b/app/models/but_refcomp.py index 0750f1a9f..b9626f828 100644 --- a/app/models/but_refcomp.py +++ b/app/models/but_refcomp.py @@ -7,12 +7,14 @@ """ from datetime import datetime +import flask_sqlalchemy from sqlalchemy.orm import class_mapper import sqlalchemy from app import db from app.scodoc.sco_utils import ModuleType +from app.scodoc.sco_exceptions import ScoValueError # from https://stackoverflow.com/questions/2537471/method-of-iterating-over-sqlalchemy-models-defined-columns @@ -82,7 +84,7 @@ class ApcReferentielCompetences(db.Model, XMLModel): formations = db.relationship("Formation", backref="referentiel_competence") def __repr__(self): - return f"" + return f"" def to_dict(self): """Représentation complète du ref. de comp. @@ -105,6 +107,52 @@ class ApcReferentielCompetences(db.Model, XMLModel): "parcours": {x.code: x.to_dict() for x in self.parcours}, } + def get_niveaux_by_parcours(self, annee) -> dict: + """ + Construit la liste des niveaux de compétences pour chaque parcours + de ce référentiel. + Les niveaux sont groupés par parcours, en isolant les niveaux de tronc commun. + Le tronc commun n'est pas identifié comme tel dans les référentiels Orébut: + on cherche les niveaux qui sont présents dans tous les parcours et les range sous + la clé "TC" (toujours présente mais éventuellement liste vide si pas de tronc commun). + + résultat: + { + "TC" : [ ApcNiveau ], + parcour.id : [ ApcNiveau ] + } + """ + parcours = self.parcours.order_by(ApcParcours.numero).all() + niveaux_by_parcours = { + parcour.id: ApcNiveau.niveaux_annee_de_parcours(parcour, annee, self) + for parcour in parcours + } + # Cherche tronc commun + niveaux_ids_tc = set.intersection( + *[ + {n.id for n in niveaux_by_parcours[parcour_id]} + for parcour_id in niveaux_by_parcours + ] + ) + # Enleve les niveaux du tronc commun + niveaux_by_parcours_no_tc = { + parcour.id: [ + niveau + for niveau in niveaux_by_parcours[parcour.id] + if niveau.id not in niveaux_ids_tc + ] + for parcour in parcours + } + # Niveaux du TC + niveaux_tc = [] + if len(parcours): + niveaux_parcours_1 = niveaux_by_parcours[parcours[0].id] + niveaux_tc = [ + niveau for niveau in niveaux_parcours_1 if niveau.id in niveaux_ids_tc + ] + niveaux_by_parcours_no_tc["TC"] = niveaux_tc + return niveaux_by_parcours_no_tc + class ApcCompetence(db.Model, XMLModel): "Compétence" @@ -144,9 +192,10 @@ class ApcCompetence(db.Model, XMLModel): ) def __repr__(self): - return f"" + return f"" def to_dict(self): + "repr dict recursive sur situations, composantes, niveaux" return { "id_orebut": self.id_orebut, "titre": self.titre, @@ -160,6 +209,16 @@ class ApcCompetence(db.Model, XMLModel): "niveaux": {x.annee: x.to_dict() for x in self.niveaux}, } + def to_dict_bul(self) -> dict: + "dict court pour bulletins" + return { + "id_orebut": self.id_orebut, + "titre": self.titre, + "titre_long": self.titre_long, + "couleur": self.couleur, + "numero": self.numero, + } + class ApcSituationPro(db.Model, XMLModel): "Situation professionnelle" @@ -186,13 +245,20 @@ class ApcComposanteEssentielle(db.Model, XMLModel): class ApcNiveau(db.Model, XMLModel): + """Niveau de compétence + Chaque niveau peut être associé à deux UE, + des semestres impair et pair de la même année. + """ + + __tablename__ = "apc_niveau" + id = db.Column(db.Integer, primary_key=True) competence_id = db.Column( db.Integer, db.ForeignKey("apc_competence.id"), nullable=False ) libelle = db.Column(db.Text(), nullable=False) - annee = db.Column(db.Text(), nullable=False) # "BUT2" - # L'ordre est l'année d'apparition de ce niveau + annee = db.Column(db.Text(), nullable=False) # "BUT1", "BUT2", "BUT3" + # L'ordre est le niveau (1,2,3) ou (1,2) suivant la competence ordre = db.Column(db.Integer, nullable=False) # 1, 2, 3 app_critiques = db.relationship( "ApcAppCritique", @@ -200,11 +266,14 @@ class ApcNiveau(db.Model, XMLModel): lazy="dynamic", cascade="all, delete-orphan", ) + ues = db.relationship("UniteEns", back_populates="niveau_competence") def __repr__(self): - return f"<{self.__class__.__name__} ordre={self.ordre}>" + return f"""<{self.__class__.__name__} ordre={self.ordre!r} annee={ + self.annee!r} {self.competence!r}>""" def to_dict(self): + "as a dict, recursif sur les AC" return { "libelle": self.libelle, "annee": self.annee, @@ -212,6 +281,64 @@ class ApcNiveau(db.Model, XMLModel): "app_critiques": {x.code: x.to_dict() for x in self.app_critiques}, } + def to_dict_bul(self): + "dict pour bulletins: indique la compétence, pas les ACs (pour l'instant ?)" + return { + "libelle": self.libelle, + "annee": self.annee, + "ordre": self.ordre, + "competence": self.competence.to_dict_bul(), + } + + @classmethod + def niveaux_annee_de_parcours( + cls, + parcour: "ApcParcours", + annee: int, + referentiel_competence: ApcReferentielCompetences = None, + ) -> flask_sqlalchemy.BaseQuery: + """Les niveaux de l'année du parcours + Si le parcour est None, tous les niveaux de l'année + """ + if annee not in {1, 2, 3}: + raise ValueError("annee invalide pour un parcours BUT") + if referentiel_competence is None: + raise ScoValueError( + "Pas de référentiel de compétences associé à la formation !" + ) + annee_formation = f"BUT{annee}" + if parcour is None: + return ApcNiveau.query.filter( + ApcNiveau.annee == annee_formation, + ApcCompetence.id == ApcNiveau.competence_id, + ApcCompetence.referentiel_id == referentiel_competence.id, + ) + else: + return ApcNiveau.query.filter( + ApcParcoursNiveauCompetence.annee_parcours_id == ApcAnneeParcours.id, + ApcParcours.id == ApcAnneeParcours.parcours_id, + ApcParcours.referentiel == parcour.referentiel, + ApcParcoursNiveauCompetence.competence_id == ApcCompetence.id, + ApcCompetence.id == ApcNiveau.competence_id, + ApcAnneeParcours.parcours == parcour, + ApcNiveau.annee == annee_formation, + ) + + +app_critiques_modules = db.Table( + "apc_modules_acs", + db.Column( + "module_id", + db.ForeignKey("notes_modules.id", ondelete="CASCADE"), + primary_key=True, + ), + db.Column( + "app_crit_id", + db.ForeignKey("apc_app_critique.id"), + primary_key=True, + ), +) + class ApcAppCritique(db.Model, XMLModel): "Apprentissage Critique BUT" @@ -220,12 +347,31 @@ class ApcAppCritique(db.Model, XMLModel): code = db.Column(db.Text(), nullable=False, index=True) libelle = db.Column(db.Text()) - modules = db.relationship( - "Module", - secondary="apc_modules_acs", - lazy="dynamic", - backref=db.backref("app_critiques", lazy="dynamic"), - ) + # modules = db.relationship( + # "Module", + # secondary="apc_modules_acs", + # lazy="dynamic", + # backref=db.backref("app_critiques", lazy="dynamic"), + # ) + + @classmethod + def app_critiques_ref_comp( + cls, + ref_comp: ApcReferentielCompetences, + annee: str, + competence: ApcCompetence = None, + ) -> flask_sqlalchemy.BaseQuery: + "Liste les AC de tous les parcours de ref_comp pour l'année indiquée" + assert annee in {"BUT1", "BUT2", "BUT3"} + query = cls.query.filter( + ApcAppCritique.niveau_id == ApcNiveau.id, + ApcNiveau.competence_id == ApcCompetence.id, + ApcNiveau.annee == annee, + ApcCompetence.referentiel_id == ref_comp.id, + ) + if competence is not None: + query = query.filter(ApcNiveau.competence == competence) + return query def to_dict(self) -> dict: return {"libelle": self.libelle} @@ -234,18 +380,40 @@ class ApcAppCritique(db.Model, XMLModel): return self.code + " - " + self.titre def __repr__(self): - return f"<{self.__class__.__name__} {self.code}>" + return f"<{self.__class__.__name__} {self.code!r}>" def get_saes(self): """Liste des SAE associées""" return [m for m in self.modules if m.module_type == ModuleType.SAE] -ApcAppCritiqueModules = db.Table( - "apc_modules_acs", - db.Column("module_id", db.ForeignKey("notes_modules.id")), - db.Column("app_crit_id", db.ForeignKey("apc_app_critique.id")), +parcours_modules = db.Table( + "parcours_modules", + db.Column( + "parcours_id", db.Integer, db.ForeignKey("apc_parcours.id"), primary_key=True + ), + db.Column( + "module_id", + db.Integer, + db.ForeignKey("notes_modules.id", ondelete="CASCADE"), + primary_key=True, + ), ) +"""Association parcours <-> modules (many-to-many)""" + +parcours_formsemestre = db.Table( + "parcours_formsemestre", + db.Column( + "parcours_id", db.Integer, db.ForeignKey("apc_parcours.id"), primary_key=True + ), + db.Column( + "formsemestre_id", + db.Integer, + db.ForeignKey("notes_formsemestre.id", ondelete="CASCADE"), + primary_key=True, + ), +) +"""Association parcours <-> formsemestre (many-to-many)""" class ApcParcours(db.Model, XMLModel): @@ -264,7 +432,7 @@ class ApcParcours(db.Model, XMLModel): ) def __repr__(self): - return f"<{self.__class__.__name__} {self.code}>" + return f"<{self.__class__.__name__} {self.code!r}>" def to_dict(self): return { @@ -281,9 +449,10 @@ class ApcAnneeParcours(db.Model, XMLModel): db.Integer, db.ForeignKey("apc_parcours.id"), nullable=False ) ordre = db.Column(db.Integer) + "numéro de l'année: 1, 2, 3" def __repr__(self): - return f"<{self.__class__.__name__} ordre={self.ordre}>" + return f"<{self.__class__.__name__} ordre={self.ordre!r} parcours={self.parcours.code!r}>" def to_dict(self): return { @@ -321,6 +490,7 @@ class ApcParcoursNiveauCompetence(db.Model): "annee_parcours", passive_deletes=True, cascade="save-update, merge, delete, delete-orphan", + lazy="dynamic", ), ) annee_parcours = db.relationship( @@ -333,4 +503,4 @@ class ApcParcoursNiveauCompetence(db.Model): ) def __repr__(self): - return f"<{self.__class__.__name__} {self.competence} {self.annee_parcours}>" + return f"<{self.__class__.__name__} {self.competence!r}<->{self.annee_parcours!r} niveau={self.niveau!r}>" diff --git a/app/models/but_validations.py b/app/models/but_validations.py new file mode 100644 index 000000000..6c2814424 --- /dev/null +++ b/app/models/but_validations.py @@ -0,0 +1,329 @@ +# -*- coding: UTF-8 -* + +"""Décisions de jury (validations) des RCUE et années du BUT +""" + +import flask_sqlalchemy +from sqlalchemy.sql import text +from typing import Union + +from app import db + +from app.models import CODE_STR_LEN +from app.models.but_refcomp import ApcNiveau +from app.models.etudiants import Identite +from app.models.ues import UniteEns +from app.models.formations import Formation +from app.models.formsemestre import FormSemestre +from app.scodoc import sco_codes_parcours as sco_codes + + +class ApcValidationRCUE(db.Model): + """Validation des niveaux de compétences + + aka "regroupements cohérents d'UE" dans le jargon BUT. + + le formsemestre est celui du semestre PAIR du niveau de compétence + """ + + __tablename__ = "apc_validation_rcue" + # Assure unicité de la décision: + __table_args__ = ( + db.UniqueConstraint("etudid", "formsemestre_id", "ue1_id", "ue2_id"), + ) + + id = db.Column(db.Integer, primary_key=True) + etudid = db.Column( + db.Integer, + db.ForeignKey("identite.id", ondelete="CASCADE"), + index=True, + nullable=False, + ) + formsemestre_id = db.Column( + db.Integer, db.ForeignKey("notes_formsemestre.id"), index=True, nullable=True + ) + # Les deux UE associées à ce niveau: + ue1_id = db.Column(db.Integer, db.ForeignKey("notes_ue.id"), nullable=False) + ue2_id = db.Column(db.Integer, db.ForeignKey("notes_ue.id"), nullable=False) + # optionnel, le parcours dans lequel se trouve la compétence: + parcours_id = db.Column(db.Integer, db.ForeignKey("apc_parcours.id"), nullable=True) + date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) + code = db.Column(db.String(CODE_STR_LEN), nullable=False, index=True) + + etud = db.relationship("Identite", backref="apc_validations_rcues") + formsemestre = db.relationship("FormSemestre", backref="apc_validations_rcues") + ue1 = db.relationship("UniteEns", foreign_keys=ue1_id) + ue2 = db.relationship("UniteEns", foreign_keys=ue2_id) + parcour = db.relationship("ApcParcours") + + def __repr__(self): + return f"<{self.__class__.__name__} {self.id} {self.etud} {self.ue1}/{self.ue2}:{self.code!r}>" + + def niveau(self) -> ApcNiveau: + """Le niveau de compétence associé à cet RCUE.""" + # Par convention, il est donné par la seconde UE + return self.ue2.niveau_competence + + def to_dict_bul(self) -> dict: + "Export dict pour bulletins" + return {"code": self.code, "niveau": self.niveau().to_dict_bul()} + + +# Attention: ce n'est pas un modèle mais une classe ordinaire: +class RegroupementCoherentUE: + """Le regroupement cohérent d'UE, dans la terminologie du BUT, est le couple d'UEs + de la même année (BUT1,2,3) liées au *même niveau de compétence*. + + La moyenne (10/20) au RCU déclenche la compensation des UE. + """ + + def __init__( + self, + etud: Identite, + formsemestre_1: FormSemestre, + ue_1: UniteEns, + formsemestre_2: FormSemestre, + ue_2: UniteEns, + ): + from app.comp import res_sem + from app.comp.res_but import ResultatsSemestreBUT + + # Ordonne les UE dans le sens croissant (S1,S2) ou (S3,S4)... + if formsemestre_1.semestre_id > formsemestre_2.semestre_id: + (ue_1, formsemestre_1), (ue_2, formsemestre_2) = ( + ( + ue_2, + formsemestre_2, + ), + (ue_1, formsemestre_1), + ) + assert formsemestre_1.semestre_id % 2 == 1 + assert formsemestre_2.semestre_id % 2 == 0 + assert abs(formsemestre_1.semestre_id - formsemestre_2.semestre_id) == 1 + assert ue_1.niveau_competence_id == ue_2.niveau_competence_id + self.etud = etud + self.formsemestre_1 = formsemestre_1 + "semestre impair" + self.ue_1 = ue_1 + self.formsemestre_2 = formsemestre_2 + "semestre pair" + self.ue_2 = ue_2 + # Stocke les moyennes d'UE + res: ResultatsSemestreBUT = res_sem.load_formsemestre_results(formsemestre_1) + if ue_1.id in res.etud_moy_ue and etud.id in res.etud_moy_ue[ue_1.id]: + self.moy_ue_1 = res.etud_moy_ue[ue_1.id][etud.id] + self.moy_ue_1_val = self.moy_ue_1 # toujours float, peut être NaN + else: + self.moy_ue_1 = None + self.moy_ue_1_val = 0.0 + res: ResultatsSemestreBUT = res_sem.load_formsemestre_results(formsemestre_2) + if ue_2.id in res.etud_moy_ue and etud.id in res.etud_moy_ue[ue_2.id]: + self.moy_ue_2 = res.etud_moy_ue[ue_2.id][etud.id] + self.moy_ue_2_val = self.moy_ue_2 + else: + self.moy_ue_2 = None + self.moy_ue_2_val = 0.0 + # Calcul de la moyenne au RCUE + if (self.moy_ue_1 is not None) and (self.moy_ue_2 is not None): + # Moyenne RCUE (les pondérations par défaut sont 1.) + self.moy_rcue = ( + self.moy_ue_1 * ue_1.coef_rcue + self.moy_ue_2 * ue_2.coef_rcue + ) / (ue_1.coef_rcue + ue_2.coef_rcue) + else: + self.moy_rcue = None + + def __repr__(self) -> str: + return f"<{self.__class__.__name__} {self.ue_1.acronyme}({self.moy_ue_1}) {self.ue_2.acronyme}({self.moy_ue_2})>" + + def query_validations( + self, + ) -> flask_sqlalchemy.BaseQuery: # list[ApcValidationRCUE] + """Les validations de jury enregistrées pour ce RCUE""" + niveau = self.ue_2.niveau_competence + + return ( + ApcValidationRCUE.query.filter_by( + etudid=self.etud.id, + ) + .join(UniteEns, UniteEns.id == ApcValidationRCUE.ue2_id) + .join(ApcNiveau, UniteEns.niveau_competence_id == ApcNiveau.id) + .filter(ApcNiveau.id == niveau.id) + ) + + def other_ue(self, ue: UniteEns) -> UniteEns: + """L'autre UE du regroupement. Si ue ne fait pas partie du regroupement, ValueError""" + if ue.id == self.ue_1.id: + return self.ue_2 + elif ue.id == self.ue_2.id: + return self.ue_1 + raise ValueError(f"ue {ue} hors RCUE {self}") + + def est_enregistre(self) -> bool: + """Vrai si ce RCUE, donc le niveau de compétences correspondant + a une décision jury enregistrée + """ + return self.query_validations().count() > 0 + + def est_compensable(self): + """Vrai si ce RCUE est validable par compensation + c'est à dire que sa moyenne est > 10 avec une UE < 10 + """ + return ( + (self.moy_rcue is not None) + and (self.moy_rcue > sco_codes.BUT_BARRE_RCUE) + and ( + (self.moy_ue_1_val < sco_codes.NOTES_BARRE_GEN) + or (self.moy_ue_2_val < sco_codes.NOTES_BARRE_GEN) + ) + ) + + def est_suffisant(self) -> bool: + """Vrai si ce RCUE est > 8""" + return (self.moy_rcue is not None) and ( + self.moy_rcue > sco_codes.BUT_RCUE_SUFFISANT + ) + + def est_validable(self) -> bool: + """Vrai si ce RCU satisfait les conditions pour être validé + Pour cela, il suffit que la moyenne des UE qui le constitue soit > 10 + """ + return (self.moy_rcue is not None) and ( + self.moy_rcue > sco_codes.BUT_BARRE_RCUE + ) + + def code_valide(self) -> Union[ApcValidationRCUE, None]: + "Si ce RCUE est ADM, CMP ou ADJ, la validation. Sinon, None" + validation = self.query_validations().first() + if (validation is not None) and ( + validation.code in sco_codes.CODES_RCUE_VALIDES + ): + return validation + return None + + +def find_rcues( + formsemestre: FormSemestre, ue: UniteEns, etud: Identite +) -> list[RegroupementCoherentUE]: + """Les RCUE (niveau de compétence) à considérer pour cet étudiant dans + ce semestre pour cette UE. + + Cherche les UEs du même niveau de compétence auxquelles l'étudiant est inscrit. + En cas de redoublement, il peut y en avoir plusieurs, donc plusieurs RCUEs. + + Résultat: la liste peut être vide. + """ + if (ue.niveau_competence is None) or (ue.semestre_idx is None): + return [] + + if ue.semestre_idx % 2: # S1, S3, S5 + other_semestre_idx = ue.semestre_idx + 1 + else: + other_semestre_idx = ue.semestre_idx - 1 + + cursor = db.session.execute( + text( + """SELECT + ue.id, formsemestre.id + FROM + notes_ue ue, + notes_formsemestre_inscription inscr, + notes_formsemestre formsemestre + + WHERE + inscr.etudid = :etudid + AND inscr.formsemestre_id = formsemestre.id + + AND formsemestre.semestre_id = :other_semestre_idx + AND ue.formation_id = formsemestre.formation_id + AND ue.niveau_competence_id = :ue_niveau_competence_id + AND ue.semestre_idx = :other_semestre_idx + """ + ), + { + "etudid": etud.id, + "other_semestre_idx": other_semestre_idx, + "ue_niveau_competence_id": ue.niveau_competence_id, + }, + ) + rcues = [] + for ue_id, formsemestre_id in cursor: + other_ue = UniteEns.query.get(ue_id) + other_formsemestre = FormSemestre.query.get(formsemestre_id) + rcues.append( + RegroupementCoherentUE(etud, formsemestre, ue, other_formsemestre, other_ue) + ) + # safety check: 1 seul niveau de comp. concerné: + assert len({rcue.ue_1.niveau_competence_id for rcue in rcues}) == 1 + return rcues + + +class ApcValidationAnnee(db.Model): + """Validation des années du BUT""" + + __tablename__ = "apc_validation_annee" + # Assure unicité de la décision: + __table_args__ = (db.UniqueConstraint("etudid", "annee_scolaire"),) + id = db.Column(db.Integer, primary_key=True) + etudid = db.Column( + db.Integer, + db.ForeignKey("identite.id", ondelete="CASCADE"), + index=True, + nullable=False, + ) + ordre = db.Column(db.Integer, nullable=False) + "numéro de l'année: 1, 2, 3" + formsemestre_id = db.Column( + db.Integer, db.ForeignKey("notes_formsemestre.id"), nullable=True + ) + "le semestre IMPAIR (le 1er) de l'année" + annee_scolaire = db.Column(db.Integer, nullable=False) # 2021 + date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) + code = db.Column(db.String(CODE_STR_LEN), nullable=False, index=True) + + etud = db.relationship("Identite", backref="apc_validations_annees") + formsemestre = db.relationship("FormSemestre", backref="apc_validations_annees") + + def __repr__(self): + return f"<{self.__class__.__name__} {self.id} {self.etud} BUT{self.ordre}/{self.annee_scolaire}:{self.code!r}>" + + def to_dict_bul(self) -> dict: + "dict pour bulletins" + return { + "annee_scolaire": self.annee_scolaire, + "date": self.date.isoformat(), + "code": self.code, + "ordre": self.ordre, + } + + +def dict_decision_jury(etud: Identite, formsemestre: FormSemestre) -> dict: + """ + Un dict avec les décisions de jury BUT enregistrées. + Ne reprend pas les décisions d'UE, non spécifiques au BUT. + """ + decisions = {} + # --- RCUEs: seulement sur semestres pairs XXX à améliorer + if formsemestre.semestre_id % 2 == 0: + # validations émises depuis ce formsemestre: + validations_rcues = ApcValidationRCUE.query.filter_by( + etudid=etud.id, formsemestre_id=formsemestre.id + ) + decisions["decision_rcue"] = [v.to_dict_bul() for v in validations_rcues] + else: + decisions["decision_rcue"] = [] + # --- Année: prend la validation pour l'année scolaire de ce semestre + validation = ( + ApcValidationAnnee.query.filter_by( + etudid=etud.id, + annee_scolaire=formsemestre.annee_scolaire(), + ) + .join(ApcValidationAnnee.formsemestre) + .join(FormSemestre.formation) + .filter(Formation.formation_code == formsemestre.formation.formation_code) + .first() + ) + if validation: + decisions["decision_annee"] = validation.to_dict_bul() + else: + decisions["decision_annee"] = None + return decisions diff --git a/app/models/config.py b/app/models/config.py index 53ac96e9b..cb65d5198 100644 --- a/app/models/config.py +++ b/app/models/config.py @@ -9,6 +9,8 @@ from app.comp import bonus_spo from app.scodoc.sco_exceptions import ScoValueError from app.scodoc.sco_codes_parcours import ( + ABAN, + ABL, ADC, ADJ, ADM, @@ -19,11 +21,17 @@ from app.scodoc.sco_codes_parcours import ( CMP, DEF, DEM, + EXCLU, NAR, + PASD, + PAS1NCI, RAT, + RED, ) CODES_SCODOC_TO_APO = { + ABAN: "ABAN", + ABL: "ABL", ADC: "ADMC", ADJ: "ADM", ADM: "ADM", @@ -34,8 +42,12 @@ CODES_SCODOC_TO_APO = { CMP: "COMP", DEF: "NAR", DEM: "NAR", + EXCLU: "EXC", NAR: "NAR", + PASD: "PASD", + PAS1NCI: "PAS1NCI", RAT: "ATT", + RED: "RED", "NOTES_FMT": "%3.2f", } @@ -161,9 +173,8 @@ class ScoDocSiteConfig(db.Model): @classmethod def get_code_apo(cls, code: str) -> str: """La représentation d'un code pour les exports Apogée. - Par exemple, à l'iUT du H., le code ADM est réprésenté par VAL + Par exemple, à l'IUT du H., le code ADM est réprésenté par VAL Les codes par défaut sont donnés dans sco_apogee_csv. - """ cfg = ScoDocSiteConfig.query.filter_by(name=code).first() if not cfg: @@ -172,6 +183,11 @@ class ScoDocSiteConfig(db.Model): code_apo = cfg.value return code_apo + @classmethod + def get_codes_apo_dict(cls) -> dict[str:str]: + "Un dict avec code jury : code exporté" + return {code: cls.get_code_apo(code) for code in CODES_SCODOC_TO_APO} + @classmethod def set_code_apo(cls, code: str, code_apo: str): """Enregistre nouvelle représentation du code""" diff --git a/app/models/etudiants.py b/app/models/etudiants.py index 0bce6d47e..30333a6b1 100644 --- a/app/models/etudiants.py +++ b/app/models/etudiants.py @@ -60,7 +60,9 @@ class Identite(db.Model): admission = db.relationship("Admission", backref="identite", lazy="dynamic") def __repr__(self): - return f"" + return ( + f"" + ) @classmethod def from_request(cls, etudid=None, code_nip=None): @@ -133,8 +135,10 @@ class Identite(db.Model): def sort_key(self) -> tuple: "clé pour tris par ordre alphabétique" return ( - scu.suppress_accents(self.nom_usuel or self.nom or "").lower(), - scu.suppress_accents(self.prenom or "").lower(), + scu.sanitize_string( + self.nom_usuel or self.nom or "", remove_spaces=False + ).lower(), + scu.sanitize_string(self.prenom or "", remove_spaces=False).lower(), ) def get_first_email(self, field="email") -> str: @@ -201,6 +205,19 @@ class Identite(db.Model): d.update(adresse.to_dict(convert_nulls_to_str=True)) return d + def inscriptions(self) -> list["FormSemestreInscription"]: + "Liste des inscriptions à des formsemestres, triée, la plus récente en tête" + from app.models.formsemestre import FormSemestre, FormSemestreInscription + + return ( + FormSemestreInscription.query.join(FormSemestreInscription.formsemestre) + .filter( + FormSemestreInscription.etudid == self.id, + ) + .order_by(desc(FormSemestre.date_debut)) + .all() + ) + def inscription_courante(self): """La première inscription à un formsemestre _actuellement_ en cours. None s'il n'y en a pas (ou plus, ou pas encore). @@ -212,7 +229,7 @@ class Identite(db.Model): ] return r[0] if r else None - def inscriptions_courantes(self) -> list: # -> list[FormSemestreInscription]: + def inscriptions_courantes(self) -> list["FormSemestreInscription"]: """Liste des inscriptions à des semestres _courants_ (il est rare qu'il y en ai plus d'une, mais c'est possible). Triées par date de début de semestre décroissante (le plus récent en premier). @@ -240,18 +257,6 @@ class Identite(db.Model): ] return r[0] if r else None - def inscription_etat(self, formsemestre_id): - """État de l'inscription de cet étudiant au semestre: - False si pas inscrit, ou scu.INSCRIT, DEMISSION, DEF - """ - # voir si ce n'est pas trop lent: - ins = models.FormSemestreInscription.query.filter_by( - etudid=self.id, formsemestre_id=formsemestre_id - ).first() - if ins: - return ins.etat - return False - def inscription_descr(self) -> dict: """Description de l'état d'inscription""" inscription_courante = self.inscription_courante() @@ -290,6 +295,18 @@ class Identite(db.Model): "situation": situation, } + def inscription_etat(self, formsemestre_id): + """État de l'inscription de cet étudiant au semestre: + False si pas inscrit, ou scu.INSCRIT, DEMISSION, DEF + """ + # voir si ce n'est pas trop lent: + ins = models.FormSemestreInscription.query.filter_by( + etudid=self.id, formsemestre_id=formsemestre_id + ).first() + if ins: + return ins.etat + return False + def descr_situation_etud(self) -> str: """Chaîne décrivant la situation _actuelle_ de l'étudiant. Exemple: @@ -361,6 +378,15 @@ class Identite(db.Model): return situation + def etat_civil_pv(self, line_sep="\n") -> str: + """Présentation, pour PV jury + M. Pierre Dupont + n° 12345678 + né(e) le 7/06/1974 + à Paris + """ + return f"""{self.nomprenom}{line_sep}n°{self.code_nip or ""}{line_sep}né{self.e} le {self.date_naissance.strftime("%d/%m/%Y") if self.date_naissance else ""}{line_sep}à {self.lieu_naissance or ""}""" + def photo_html(self, title=None, size="small") -> str: """HTML img tag for the photo, either in small size (h90) or original size (size=="orig") @@ -434,7 +460,7 @@ class Adresse(db.Model): adresse_id = db.synonym("id") etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), ) email = db.Column(db.Text()) # mail institutionnel emailperso = db.Column(db.Text) # email personnel (exterieur) @@ -468,7 +494,7 @@ class Admission(db.Model): adm_id = db.synonym("id") etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), ) # Anciens champs de ScoDoc7, à revoir pour être plus générique et souple # notamment dans le cadre du bac 2021 @@ -513,21 +539,21 @@ class Admission(db.Model): def to_dict(self, no_nulls=False): """Représentation dictionnaire,""" - e = dict(self.__dict__) - e.pop("_sa_instance_state", None) + d = dict(self.__dict__) + d.pop("_sa_instance_state", None) if no_nulls: - for k in e: - if e[k] is None: + for k in d.keys(): + if d[k] is None: col_type = getattr( sqlalchemy.inspect(models.Admission).columns, "apb_groupe" ).expression.type if isinstance(col_type, sqlalchemy.Text): - e[k] = "" + d[k] = "" elif isinstance(col_type, sqlalchemy.Integer): - e[k] = 0 + d[k] = 0 elif isinstance(col_type, sqlalchemy.Boolean): - e[k] = False - return e + d[k] = False + return d # Suivi scolarité / débouchés @@ -538,7 +564,7 @@ class ItemSuivi(db.Model): itemsuivi_id = db.synonym("id") etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), ) item_date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) situation = db.Column(db.Text) diff --git a/app/models/events.py b/app/models/events.py index b94549e76..4e566cbd9 100644 --- a/app/models/events.py +++ b/app/models/events.py @@ -32,6 +32,21 @@ class Scolog(db.Model): authenticated_user = db.Column(db.Text) # login, sans contrainte # zope_remote_addr suppressed + @classmethod + def logdb( + cls, method: str = None, etudid: int = None, msg: str = None, commit=False + ): + """Add entry in student's log (replacement for old scolog.logdb)""" + entry = Scolog( + method=method, + msg=msg, + etudid=etudid, + authenticated_user=current_user.user_name, + ) + db.session.add(entry) + if commit: + db.session.commit() + class ScolarNews(db.Model): """Nouvelles pour page d'accueil""" diff --git a/app/models/formations.py b/app/models/formations.py index 43d3d680c..d2970fce9 100644 --- a/app/models/formations.py +++ b/app/models/formations.py @@ -1,15 +1,25 @@ """ScoDoc 9 models : Formations """ +import flask_sqlalchemy import app from app import db from app.comp import df_cache from app.models import SHORT_STR_LEN +from app.models.but_refcomp import ( + ApcAnneeParcours, + ApcCompetence, + ApcNiveau, + ApcParcours, + ApcParcoursNiveauCompetence, +) from app.models.modules import Module +from app.models.moduleimpls import ModuleImpl from app.models.ues import UniteEns from app.scodoc import sco_cache from app.scodoc import sco_codes_parcours from app.scodoc import sco_utils as scu +from app.scodoc.sco_codes_parcours import UE_STANDARD class Formation(db.Model): @@ -45,7 +55,11 @@ class Formation(db.Model): modules = db.relationship("Module", lazy="dynamic", backref="formation") def __repr__(self): - return f"<{self.__class__.__name__}(id={self.id}, dept_id={self.dept_id}, acronyme='{self.acronyme}')>" + return f"<{self.__class__.__name__}(id={self.id}, dept_id={self.dept_id}, acronyme='{self.acronyme!r}')>" + + def to_html(self) -> str: + "titre complet pour affichage" + return f"""Formation {self.titre} ({self.acronyme}) [version {self.version}] code {self.formation_code}""" def to_dict(self): e = dict(self.__dict__) @@ -55,7 +69,10 @@ class Formation(db.Model): return e def get_parcours(self): - """get l'instance de TypeParcours de cette formation""" + """get l'instance de TypeParcours de cette formation + (le TypeParcours définit le genre de formation, à ne pas confondre + avec les parcours du BUT). + """ return sco_codes_parcours.get_parcours_from_code(self.type_parcours) def get_titre_version(self) -> str: @@ -97,6 +114,13 @@ class Formation(db.Model): else: keys = f"{self.id}.{semestre_idx}" df_cache.ModuleCoefsCache.delete_many(keys | {f"{self.id}"}) + # Invalidate aussi les poids de toutes les évals de la formation + for modimpl in ModuleImpl.query.filter( + ModuleImpl.module_id == Module.id, + Module.formation_id == self.id, + ): + modimpl.invalidate_evaluations_poids() + sco_cache.invalidate_formsemestre() def invalidate_cached_sems(self): @@ -148,6 +172,40 @@ class Formation(db.Model): if change: app.clear_scodoc_cache() + def query_ues_parcour(self, parcour: ApcParcours) -> flask_sqlalchemy.BaseQuery: + """Les UEs d'un parcours de la formation. + Exemple: pour avoir les UE du semestre 3, faire + `formation.query_ues_parcour(parcour).filter_by(semestre_idx=3)` + """ + return UniteEns.query.filter_by(formation=self).filter( + UniteEns.niveau_competence_id == ApcNiveau.id, + UniteEns.type == UE_STANDARD, + ApcParcoursNiveauCompetence.competence_id == ApcNiveau.competence_id, + ApcParcoursNiveauCompetence.annee_parcours_id == ApcAnneeParcours.id, + ApcAnneeParcours.parcours_id == parcour.id, + ) + + def query_competences_parcour( + self, parcour: ApcParcours + ) -> flask_sqlalchemy.BaseQuery: + """Les ApcCompetences d'un parcours de la formation. + None si pas de référentiel de compétences. + """ + if self.referentiel_competence_id is None: + return None + return ( + ApcCompetence.query.filter_by(referentiel_id=self.referentiel_competence_id) + .join( + ApcParcoursNiveauCompetence, + ApcParcoursNiveauCompetence.competence_id == ApcCompetence.id, + ) + .join( + ApcAnneeParcours, + ApcParcoursNiveauCompetence.annee_parcours_id == ApcAnneeParcours.id, + ) + .filter(ApcAnneeParcours.parcours_id == parcour.id) + ) + class Matiere(db.Model): """Matières: regroupe les modules d'une UE @@ -168,7 +226,7 @@ class Matiere(db.Model): def __repr__(self): return f"""<{self.__class__.__name__}(id={self.id}, ue_id={ - self.ue_id}, titre='{self.titre}')>""" + self.ue_id}, titre='{self.titre!r}')>""" def to_dict(self): """as a dict, with the same conversions as in ScoDoc7""" diff --git a/app/models/formsemestre.py b/app/models/formsemestre.py index e243c3983..99dd4a597 100644 --- a/app/models/formsemestre.py +++ b/app/models/formsemestre.py @@ -5,19 +5,31 @@ import datetime from functools import cached_property +from flask import flash import flask_sqlalchemy +from sqlalchemy.sql import text from app import db from app import log from app.models import APO_CODE_STR_LEN from app.models import SHORT_STR_LEN from app.models import CODE_STR_LEN +from app.models.but_refcomp import ( + ApcAnneeParcours, + ApcNiveau, + ApcParcours, + ApcParcoursNiveauCompetence, +) +from app.models.groups import GroupDescr, Partition import app.scodoc.sco_utils as scu -from app.models.ues import UniteEns +from app.models.but_refcomp import ApcParcours +from app.models.but_refcomp import parcours_formsemestre +from app.models.etudiants import Identite from app.models.modules import Module from app.models.moduleimpls import ModuleImpl -from app.models.etudiants import Identite +from app.models.ues import UniteEns + from app.scodoc import sco_codes_parcours from app.scodoc import sco_preferences from app.scodoc.sco_vdi import ApoEtapeVDI @@ -113,6 +125,14 @@ class FormSemestre(db.Model): # ne pas utiliser après migrate_scodoc7_dept_archives scodoc7_id = db.Column(db.Text(), nullable=True) + # BUT + parcours = db.relationship( + "ApcParcours", + secondary=parcours_formsemestre, + lazy="subquery", + backref=db.backref("formsemestres", lazy=True), + ) + def __init__(self, **kwargs): super(FormSemestre, self).__init__(**kwargs) if self.modalite is None: @@ -121,7 +141,7 @@ class FormSemestre(db.Model): def __repr__(self): return f"<{self.__class__.__name__} {self.id} {self.titre_num()}>" - def to_dict(self): + def to_dict(self, convert_parcours=False): "dict (compatible ScoDoc7)" d = dict(self.__dict__) d.pop("_sa_instance_state", None) @@ -140,6 +160,8 @@ class FormSemestre(db.Model): d["date_fin"] = d["date_fin_iso"] = "" d["responsables"] = [u.id for u in self.responsables] d["titre_formation"] = self.titre_formation() + if convert_parcours: + d["parcours"] = [p.to_dict() for p in self.parcours] return d def get_infos_dict(self) -> dict: @@ -197,6 +219,22 @@ class FormSemestre(db.Model): sem_ues = sem_ues.filter(UniteEns.type != sco_codes_parcours.UE_SPORT) return sem_ues.order_by(UniteEns.numero) + def query_ues_parcours_etud(self, etudid: int) -> flask_sqlalchemy.BaseQuery: + """UE que suit l'étudiant dans ce semestre BUT + en fonction du parcours dans lequel il est inscrit. + + Si voulez les UE d'un parcours, il est plus efficace de passer par + `formation.query_ues_parcour(parcour)`. + """ + return self.query_ues().filter( + FormSemestreInscription.etudid == etudid, + FormSemestreInscription.formsemestre == self, + UniteEns.niveau_competence_id == ApcNiveau.id, + ApcParcoursNiveauCompetence.competence_id == ApcNiveau.competence_id, + ApcParcoursNiveauCompetence.annee_parcours_id == ApcAnneeParcours.id, + ApcAnneeParcours.parcours_id == FormSemestreInscription.parcour_id, + ) + @cached_property def modimpls_sorted(self) -> list[ModuleImpl]: """Liste des modimpls du semestre (y compris bonus) @@ -223,6 +261,28 @@ class FormSemestre(db.Model): ) return modimpls + def modimpls_parcours(self, parcours: ApcParcours) -> list[ModuleImpl]: + """Liste des modimpls du semestre (sans les bonus (?)) dans le parcours donné. + - triée par type/numéro/code ?? + """ + cursor = db.session.execute( + text( + """ + SELECT modimpl.id + FROM notes_moduleimpl modimpl, notes_modules mod, + parcours_modules pm, parcours_formsemestre pf + WHERE modimpl.formsemestre_id = :formsemestre_id + AND modimpl.module_id = mod.id + AND pm.module_id = mod.id + AND pm.parcours_id = pf.parcours_id + AND pf.parcours_id = :parcours_id + AND pf.formsemestre_id = :formsemestre_id + """ + ), + {"formsemestre_id": self.id, "parcours_id": parcours.id}, + ) + return [ModuleImpl.query.get(modimpl_id) for modimpl_id in cursor] + def can_be_edited_by(self, user): """Vrai si user peut modifier ce semestre""" if not user.has_permission(Permission.ScoImplement): # pas chef @@ -289,6 +349,25 @@ class FormSemestre(db.Model): return "" return ", ".join(sorted([etape.etape_apo for etape in self.etapes if etape])) + def regroupements_coherents_etud(self) -> list[tuple[UniteEns, UniteEns]]: + """Calcule la liste des regroupements cohérents d'UE impliquant ce + formsemestre. + Pour une année donnée: l'étudiant est inscrit dans ScoDoc soit dans le semestre + impair, soit pair, soit les deux (il est rare mais pas impossible d'avoir une + inscription seulement en semestre pair, par exemple suite à un transfert ou un + arrêt temporaire du cursus). + + 1. Déterminer l'*autre* formsemestre: semestre précédent ou suivant de la même + année, formation compatible (même référentiel de compétence) dans lequel + l'étudiant est inscrit. + + 2. Construire les couples d'UE (regroupements cohérents): apparier les UE qui + ont le même `ApcParcoursNiveauCompetence`. + """ + if not self.formation.is_apc(): + return [] + raise NotImplementedError() # XXX + def responsables_str(self, abbrev_prenom=True) -> str: """chaîne "J. Dupond, X. Martin" ou "Jacques Dupond, Xavier Martin" @@ -305,6 +384,11 @@ class FormSemestre(db.Model): "True si l'user est l'un des responsables du semestre" return user.id in [u.id for u in self.responsables] + def annee_scolaire(self) -> int: + """L'année de début de l'année scolaire. + Par exemple, 2022 si le semestre va de septebre 2022 à février 2023.""" + return scu.annee_scolaire_debut(self.date_debut.year, self.date_debut.month) + def annee_scolaire_str(self): "2021 - 2022" return scu.annee_scolaire_repr(self.date_debut.year, self.date_debut.month) @@ -403,6 +487,19 @@ class FormSemestre(db.Model): etudid, self.date_debut.isoformat(), self.date_fin.isoformat() ) + def get_codes_apogee(self, category=None) -> set[str]: + """Les codes Apogée (codés en base comme "VRT1,VRT2") + category: None: tous, "etapes": étapes associées, "sem: code semestre", "annee": code annuel + """ + codes = set() + if category is None or category == "etapes": + codes |= {e.etape_apo for e in self.etapes if e} + if (category is None or category == "sem") and self.elt_sem_apo: + codes |= {x.strip() for x in self.elt_sem_apo.split(",") if x} + if (category is None or category == "annee") and self.elt_annee_apo: + codes |= {x.strip() for x in self.elt_annee_apo.split(",") if x} + return codes + def get_inscrits(self, include_demdef=False, order=False) -> list[Identite]: """Liste des étudiants inscrits à ce semestre Si include_demdef, tous les étudiants, avec les démissionnaires @@ -427,6 +524,85 @@ class FormSemestre(db.Model): """Map { etudid : inscription } (incluant DEM et DEF)""" return {ins.etud.id: ins for ins in self.inscriptions} + def setup_parcours_groups(self) -> None: + """Vérifie et créee si besoin la partition et les groupes de parcours BUT.""" + if not self.formation.is_apc(): + return + partition = Partition.query.filter_by( + formsemestre_id=self.id, partition_name=scu.PARTITION_PARCOURS + ).first() + if partition is None: + # Création de la partition de parcours + partition = Partition( + formsemestre_id=self.id, + partition_name=scu.PARTITION_PARCOURS, + numero=-1, + ) + db.session.add(partition) + db.session.flush() # pour avoir un id + flash(f"Partition Parcours créée.") + + for parcour in self.parcours: + if parcour.code: + group = GroupDescr.query.filter_by( + partition_id=partition.id, group_name=parcour.code + ).first() + if not group: + partition.groups.append(GroupDescr(group_name=parcour.code)) + db.session.commit() + + def update_inscriptions_parcours_from_groups(self) -> None: + """Met à jour les inscriptions dans les parcours du semestres en + fonction des groupes de parcours. + Les groupes de parcours sont ceux de la partition scu.PARTITION_PARCOURS + et leur nom est le code du parcours (eg "Cyber"). + """ + partition = Partition.query.filter_by( + formsemestre_id=self.id, partition_name=scu.PARTITION_PARCOURS + ).first() + if partition is None: # pas de partition de parcours + return + + # Efface les inscriptions aux parcours: + db.session.execute( + text( + """UPDATE notes_formsemestre_inscription + SET parcour_id=NULL + WHERE formsemestre_id=:formsemestre_id + """ + ), + { + "formsemestre_id": self.id, + }, + ) + # Inscrit les étudiants des groupes de parcours: + for group in partition.groups: + query = ApcParcours.query.filter_by(code=group.group_name) + if query.count() != 1: + log( + f"""update_inscriptions_parcours_from_groups: { + query.count()} parcours with code {group.group_name}""" + ) + continue + parcour = query.first() + db.session.execute( + text( + """UPDATE notes_formsemestre_inscription ins + SET parcour_id=:parcour_id + FROM group_membership gm + WHERE formsemestre_id=:formsemestre_id + AND gm.etudid = ins.etudid + AND gm.group_id = :group_id + """ + ), + { + "formsemestre_id": self.id, + "parcour_id": parcour.id, + "group_id": group.id, + }, + ) + db.session.commit() + # Association id des utilisateurs responsables (aka directeurs des etudes) du semestre notes_formsemestre_responsables = db.Table( @@ -580,7 +756,9 @@ class FormSemestreInscription(db.Model): id = db.Column(db.Integer, primary_key=True) formsemestre_inscription_id = db.synonym("id") - etudid = db.Column(db.Integer, db.ForeignKey("identite.id"), index=True) + etudid = db.Column( + db.Integer, db.ForeignKey("identite.id", ondelete="CASCADE"), index=True + ) formsemestre_id = db.Column( db.Integer, db.ForeignKey("notes_formsemestre.id"), @@ -600,11 +778,16 @@ class FormSemestreInscription(db.Model): ) # I inscrit, D demission en cours de semestre, DEF si "defaillant" etat = db.Column(db.String(CODE_STR_LEN), index=True) - # etape apogee d'inscription (experimental 2020) + # Etape Apogée d'inscription (ajout 2020) etape = db.Column(db.String(APO_CODE_STR_LEN)) + # Parcours (pour les BUT) + parcour_id = db.Column(db.Integer, db.ForeignKey("apc_parcours.id"), index=True) + parcour = db.relationship(ApcParcours) def __repr__(self): - return f"<{self.__class__.__name__} {self.id} etudid={self.etudid} sem={self.formsemestre_id} etat={self.etat}>" + return f"""<{self.__class__.__name__} {self.id} etudid={self.etudid} sem={ + self.formsemestre_id} etat={self.etat} { + ('parcours='+str(self.parcour)) if self.parcour else ''}>""" class NotesSemSet(db.Model): diff --git a/app/models/groups.py b/app/models/groups.py index 4c64ad543..27b763d11 100644 --- a/app/models/groups.py +++ b/app/models/groups.py @@ -23,7 +23,7 @@ class Partition(db.Model): ) # "TD", "TP", ... (NULL for 'all') partition_name = db.Column(db.String(SHORT_STR_LEN)) - # numero = ordre de presentation) + # Numero = ordre de presentation) numero = db.Column(db.Integer) # Calculer le rang ? bul_show_rank = db.Column( @@ -33,6 +33,10 @@ class Partition(db.Model): show_in_lists = db.Column( db.Boolean(), nullable=False, default=True, server_default="true" ) + # Editable ? (faux pour les groupes de parcours) + groups_editable = db.Column( + db.Boolean(), nullable=False, default=True, server_default="true" + ) groups = db.relationship( "GroupDescr", backref=db.backref("partition", lazy=True), @@ -106,7 +110,7 @@ class GroupDescr(db.Model): group_membership = db.Table( "group_membership", - db.Column("etudid", db.Integer, db.ForeignKey("identite.id")), + db.Column("etudid", db.Integer, db.ForeignKey("identite.id", ondelete="CASCADE")), db.Column("group_id", db.Integer, db.ForeignKey("group_descr.id")), db.UniqueConstraint("etudid", "group_id"), ) @@ -116,5 +120,5 @@ group_membership = db.Table( # __tablename__ = "group_membership" # __table_args__ = (db.UniqueConstraint("etudid", "group_id"),) # id = db.Column(db.Integer, primary_key=True) -# etudid = db.Column(db.Integer, db.ForeignKey("identite.id")) +# etudid = db.Column(db.Integer, db.ForeignKey("identite.id", ondelete="CASCADE")) # group_id = db.Column(db.Integer, db.ForeignKey("group_descr.id")) diff --git a/app/models/modules.py b/app/models/modules.py index 67ff3de0d..b36557722 100644 --- a/app/models/modules.py +++ b/app/models/modules.py @@ -3,6 +3,7 @@ from app import db from app.models import APO_CODE_STR_LEN +from app.models.but_refcomp import app_critiques_modules, parcours_modules from app.scodoc import sco_utils as scu from app.scodoc.sco_codes_parcours import UE_SPORT from app.scodoc.sco_utils import ModuleType @@ -44,13 +45,27 @@ class Module(db.Model): lazy=True, backref=db.backref("modules", lazy=True), ) + # BUT + parcours = db.relationship( + "ApcParcours", + secondary=parcours_modules, + lazy="subquery", + backref=db.backref("modules", lazy=True), + ) + + app_critiques = db.relationship( + "ApcAppCritique", + secondary=app_critiques_modules, + lazy="subquery", + backref=db.backref("modules", lazy=True), + ) def __init__(self, **kwargs): self.ue_coefs = [] super(Module, self).__init__(**kwargs) def __repr__(self): - return f"" + return f"" def to_dict(self): e = dict(self.__dict__) @@ -160,6 +175,12 @@ class Module(db.Model): # Liste seulement les coefs définis: return [(c.ue, c.coef) for c in self.get_ue_coefs_sorted()] + def get_codes_apogee(self) -> set[str]: + """Les codes Apogée (codés en base comme "VRT1,VRT2")""" + if self.code_apogee: + return {x.strip() for x in self.code_apogee.split(",") if x} + return set() + class ModuleUECoef(db.Model): """Coefficients des modules vers les UE (APC, BUT) diff --git a/app/models/notes.py b/app/models/notes.py index 6da4ef5d6..f88f87287 100644 --- a/app/models/notes.py +++ b/app/models/notes.py @@ -17,7 +17,7 @@ class BulAppreciations(db.Model): date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), index=True, ) formsemestre_id = db.Column( @@ -36,7 +36,7 @@ class NotesNotes(db.Model): id = db.Column(db.Integer, primary_key=True) etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), ) evaluation_id = db.Column( db.Integer, db.ForeignKey("notes_evaluation.id"), index=True @@ -56,7 +56,7 @@ class NotesNotesLog(db.Model): etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), ) evaluation_id = db.Column( db.Integer, diff --git a/app/models/ues.py b/app/models/ues.py index 48d81a14f..450482bc9 100644 --- a/app/models/ues.py +++ b/app/models/ues.py @@ -40,8 +40,15 @@ class UniteEns(db.Model): # coef UE, utilise seulement si l'option use_ue_coefs est activée: coefficient = db.Column(db.Float) + # coef. pour le calcul de moyennes de RCUE. Par défaut, 1. + coef_rcue = db.Column(db.Float, nullable=False, default=1.0, server_default="1.0") + color = db.Column(db.Text()) + # BUT + niveau_competence_id = db.Column(db.Integer, db.ForeignKey("apc_niveau.id")) + niveau_competence = db.relationship("ApcNiveau", back_populates="ues") + # relations matieres = db.relationship("Matiere", lazy="dynamic", backref="ue") modules = db.relationship("Module", lazy="dynamic", backref="ue") @@ -113,3 +120,9 @@ class UniteEns(db.Model): (Module.module_type != scu.ModuleType.SAE), (Module.module_type != scu.ModuleType.RESSOURCE), ).all() + + def get_codes_apogee(self) -> set[str]: + """Les codes Apogée (codés en base comme "VRT1,VRT2")""" + if self.code_apogee: + return {x.strip() for x in self.code_apogee.split(",") if x} + return set() diff --git a/app/models/validations.py b/app/models/validations.py index 0bf487f3a..42d7ba0d6 100644 --- a/app/models/validations.py +++ b/app/models/validations.py @@ -6,6 +6,7 @@ from app import db from app.models import SHORT_STR_LEN from app.models import CODE_STR_LEN +from app.models.events import Scolog class ScolarFormSemestreValidation(db.Model): @@ -19,7 +20,7 @@ class ScolarFormSemestreValidation(db.Model): formsemestre_validation_id = db.synonym("id") etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), index=True, ) formsemestre_id = db.Column( @@ -36,7 +37,7 @@ class ScolarFormSemestreValidation(db.Model): # NULL pour les UE, True|False pour les semestres: assidu = db.Column(db.Boolean) event_date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) - # NULL sauf si compense un semestre: + # NULL sauf si compense un semestre: (pas utilisé pour BUT) compense_formsemestre_id = db.Column( db.Integer, db.ForeignKey("notes_formsemestre.id"), @@ -54,7 +55,7 @@ class ScolarFormSemestreValidation(db.Model): ue = db.relationship("UniteEns", lazy="select", uselist=False) def __repr__(self): - return f"{self.__class__.__name__}({self.formsemestre_id}, {self.etudid}, code={self.code}, ue={self.ue_id}, moy_ue={self.moy_ue})" + return f"{self.__class__.__name__}({self.formsemestre_id}, {self.etudid}, code={self.code}, ue={self.ue}, moy_ue={self.moy_ue})" class ScolarAutorisationInscription(db.Model): @@ -66,10 +67,10 @@ class ScolarAutorisationInscription(db.Model): etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), ) formation_code = db.Column(db.String(SHORT_STR_LEN), nullable=False) - # semestre ou on peut s'inscrire: + # Indice du semestre où on peut s'inscrire: semestre_id = db.Column(db.Integer) date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) origin_formsemestre_id = db.Column( @@ -77,6 +78,44 @@ class ScolarAutorisationInscription(db.Model): db.ForeignKey("notes_formsemestre.id"), ) + @classmethod + def autorise_etud( + cls, + etudid: int, + formation_code: str, + origin_formsemestre_id: int, + semestre_id: int, + ): + """Enregistre une autorisation, remplace celle émanant du même semestre si elle existe.""" + cls.delete_autorisation_etud(etudid, origin_formsemestre_id) + autorisation = cls( + etudid=etudid, + formation_code=formation_code, + origin_formsemestre_id=origin_formsemestre_id, + semestre_id=semestre_id, + ) + db.session.add(autorisation) + Scolog.logdb("autorise_etud", etudid=etudid, msg=f"passage vers S{semestre_id}") + + @classmethod + def delete_autorisation_etud( + cls, + etudid: int, + origin_formsemestre_id: int, + ): + """Efface les autorisations de cette étudiant venant du sem. origine""" + autorisations = cls.query.filter_by( + etudid=etudid, origin_formsemestre_id=origin_formsemestre_id + ) + for autorisation in autorisations: + db.session.delete(autorisation) + Scolog.logdb( + "autorise_etud", + etudid=etudid, + msg=f"annule passage vers S{autorisation.semestre_id}", + ) + db.session.flush() + class ScolarEvent(db.Model): """Evenement dans le parcours scolaire d'un étudiant""" @@ -86,7 +125,7 @@ class ScolarEvent(db.Model): event_id = db.synonym("id") etudid = db.Column( db.Integer, - db.ForeignKey("identite.id"), + db.ForeignKey("identite.id", ondelete="CASCADE"), ) event_date = db.Column(db.DateTime(timezone=True), server_default=db.func.now()) formsemestre_id = db.Column( diff --git a/app/scodoc/TrivialFormulator.py b/app/scodoc/TrivialFormulator.py index c8acefaf5..ae7007683 100644 --- a/app/scodoc/TrivialFormulator.py +++ b/app/scodoc/TrivialFormulator.py @@ -207,12 +207,16 @@ class TF(object): else: self.values[field] = 1 if field not in self.values: - if "default" in descr: # first: default in form description - self.values[field] = descr["default"] - else: # then: use initvalues dict - self.values[field] = self.initvalues.get(field, "") - if self.values[field] == None: - self.values[field] = "" + if (descr.get("input_type", None) == "checkbox") and self.submitted(): + # aucune case cochée + self.values[field] = [] + else: + if "default" in descr: # first: default in form description + self.values[field] = descr["default"] + else: # then: use initvalues dict + self.values[field] = self.initvalues.get(field, "") + if self.values[field] is None: + self.values[field] = "" # convert numbers, except ids if field.endswith("id") and self.values[field]: @@ -392,9 +396,7 @@ class TF(object): if self.top_buttons: R.append(buttons_markup + "

") R.append('') - idx = 0 - for idx in range(len(self.formdescription)): - (field, descr) = self.formdescription[idx] + for field, descr in self.formdescription: if descr.get("readonly", False): R.append(self._ReadOnlyElement(field, descr)) continue @@ -408,7 +410,7 @@ class TF(object): input_type = descr.get("input_type", "text") item_dom_id = descr.get("dom_id", "") if item_dom_id: - item_dom_attr = ' id="%s"' % item_dom_id + item_dom_attr = f' id="{item_dom_id}"' else: item_dom_attr = "" # choix du template @@ -523,7 +525,6 @@ class TF(object): else: checked = "" else: # boolcheckbox - # open('/tmp/toto','a').write('GenForm: values[%s] = %s (%s)\n' % (field, values[field], type(values[field]))) if values[field] == "True": v = True elif values[field] == "False": diff --git a/app/scodoc/gen_tables.py b/app/scodoc/gen_tables.py index 1c708ca5c..0fab06eae 100644 --- a/app/scodoc/gen_tables.py +++ b/app/scodoc/gen_tables.py @@ -45,7 +45,7 @@ import random from collections import OrderedDict from xml.etree import ElementTree import json - +from openpyxl.utils import get_column_letter from reportlab.platypus import SimpleDocTemplate, Paragraph, Spacer, Frame, PageBreak from reportlab.platypus import Table, TableStyle, Image, KeepInFrame from reportlab.lib.colors import Color @@ -127,6 +127,8 @@ class GenTable(object): filename="table", # filename, without extension xls_sheet_name="feuille", xls_before_table=[], # liste de cellules a placer avant la table + xls_style_base=None, # style excel pour les cellules + xls_columns_width=None, # { col_id : largeur en "pixels excel" } pdf_title="", # au dessus du tableau en pdf pdf_table_style=None, pdf_col_widths=None, @@ -151,6 +153,8 @@ class GenTable(object): self.page_title = page_title self.pdf_link = pdf_link self.xls_link = xls_link + self.xls_style_base = xls_style_base + self.xls_columns_width = xls_columns_width or {} self.xml_link = xml_link # HTML parameters: if not table_id: # random id @@ -495,7 +499,8 @@ class GenTable(object): sheet = wb.create_sheet(sheet_name=self.xls_sheet_name) sheet.rows += self.xls_before_table style_bold = sco_excel.excel_make_style(bold=True) - style_base = sco_excel.excel_make_style() + style_base = self.xls_style_base or sco_excel.excel_make_style() + sheet.append_row(sheet.make_row(self.get_titles_list(), style_bold)) for line in self.get_data_list(xls_mode=True): sheet.append_row(sheet.make_row(line, style_base)) @@ -505,6 +510,16 @@ class GenTable(object): if self.origin: sheet.append_blank_row() # empty line sheet.append_single_cell_row(self.origin, style_base) + # Largeurs des colonnes + columns_ids = list(self.columns_ids) + for col_id, width in self.xls_columns_width.items(): + try: + idx = columns_ids.index(col_id) + col = get_column_letter(idx + 1) + sheet.set_column_dimension_width(col, width) + except ValueError: + pass + if wb is None: return sheet.generate() diff --git a/app/scodoc/html_sco_header.py b/app/scodoc/html_sco_header.py index 2cf0be400..0b2a3d587 100644 --- a/app/scodoc/html_sco_header.py +++ b/app/scodoc/html_sco_header.py @@ -59,35 +59,29 @@ BOOTSTRAP_MULTISELECT_CSS = [ def standard_html_header(): """Standard HTML header for pages outside depts""" # not used in ZScolar, see sco_header - return """ + return f"""ScoDoc: accueil - + - + -%s""" % ( - scu.SCO_ENCODING, - scu.CUSTOM_HTML_HEADER_CNX, - ) +{scu.CUSTOM_HTML_HEADER_CNX}""" def standard_html_footer(): """Le pied de page HTML de la page d'accueil.""" - return """ -

Problèmes et suggestions sur le logiciel: %s

+

Problèmes et suggestions sur le logiciel: {scu.SCO_USERS_LIST}

ScoDoc est un logiciel libre développé par Emmanuel Viennet.

-""" % ( - scu.SCO_USERS_LIST, - scu.SCO_USERS_LIST, - ) +""" -_HTML_BEGIN = """ +_HTML_BEGIN = f""" @@ -100,27 +94,27 @@ _HTML_BEGIN = """%(page_title)s - + - - - - + + + + - - - + + + - + - - + + - - + + """ @@ -138,9 +132,9 @@ def sco_header( # optional args page_title="", # page title no_side_bar=False, # hide sidebar - cssstyles=[], # additionals CSS sheets - javascripts=[], # additionals JS filenames to load - scripts=[], # script to put in page header + cssstyles=(), # additionals CSS sheets + javascripts=(), # additionals JS filenames to load + scripts=(), # script to put in page header bodyOnLoad="", # JS init_qtip=False, # include qTip init_google_maps=False, # Google maps @@ -148,6 +142,8 @@ def sco_header( titrebandeau="", # titre dans bandeau superieur head_message="", # message action (petit cadre jaune en haut) user_check=True, # verifie passwords temporaires + etudid=None, + formsemestre_id=None, ): "Main HTML page header for ScoDoc" from app.scodoc.sco_formsemestre_status import formsemestre_page_title @@ -191,7 +187,7 @@ def sco_header( # jQuery UI # can modify loaded theme here H.append( - '\n' + f'\n' ) if init_google_maps: # It may be necessary to add an API key: @@ -200,72 +196,65 @@ def sco_header( # Feuilles de style additionnelles: for cssstyle in cssstyles: H.append( - """\n""" - % cssstyle + f"""\n""" ) H.append( - """ - - - + f""" + + + - - + +""" - % params ) # jQuery H.append( - """ - """ + f""" + """ ) - H.append('') # qTip if init_qtip: H.append( - '' - ) - H.append( - '' + f""" + """ ) H.append( - '' + f""" + """ ) - - H.append('') if init_google_maps: H.append( - '' + f'' ) if init_datatables: H.append( - '' + f""" + """ ) - H.append('') # H.append( - # '' + # f'' # ) # JS additionels for js in javascripts: - H.append("""\n""" % js) + H.append(f"""\n""") H.append( - """ """ - % params ) # Scripts de la page: if scripts: @@ -281,25 +270,24 @@ def sco_header( H.append(scu.CUSTOM_HTML_HEADER) # if not no_side_bar: - H.append(html_sidebar.sidebar()) + H.append(html_sidebar.sidebar(etudid)) H.append("""
""") # En attendant le replacement complet de cette fonction, # inclusion ici des messages flask H.append(render_template("flashed_messages.html")) # # Barre menu semestre: - H.append(formsemestre_page_title()) + H.append(formsemestre_page_title(formsemestre_id)) # Avertissement si mot de passe à changer if user_check: if current_user.passwd_temp: H.append( - """
+ f"""
Attention !
Vous avez reçu un mot de passe temporaire.
- Vous devez le changer: cliquez ici + Vous devez le changer: cliquez ici
""" - % (scu.UsersURL, current_user.user_name) ) # if head_message: @@ -328,6 +316,6 @@ def html_sem_header( else: h = "" if with_h2: - return h + """

%s

""" % (title) + return h + f"""

{title}

""" else: return h diff --git a/app/scodoc/html_sidebar.py b/app/scodoc/html_sidebar.py index 5ab8eacf3..c3468cfee 100644 --- a/app/scodoc/html_sidebar.py +++ b/app/scodoc/html_sidebar.py @@ -73,7 +73,7 @@ def sidebar_common(): return "".join(H) -def sidebar(): +def sidebar(etudid: int = None): "Main HTML page sidebar" # rewritten from legacy DTML code from app.scodoc import sco_abs @@ -93,14 +93,14 @@ def sidebar(): """ ] # ---- Il y-a-t-il un etudiant selectionné ? - etudid = g.get("etudid", None) - if not etudid: + etudid = etudid if etudid is not None else g.get("etudid", None) + if etudid is None: if request.method == "GET": etudid = request.args.get("etudid", None) elif request.method == "POST": etudid = request.form.get("etudid", None) - if etudid: + if etudid is not None: etud = sco_etud.get_etud_info(filled=True, etudid=etudid)[0] params.update(etud) params["fiche_url"] = url_for( diff --git a/app/scodoc/sco_apogee_csv.py b/app/scodoc/sco_apogee_csv.py index 7fe9501df..e309bedaa 100644 --- a/app/scodoc/sco_apogee_csv.py +++ b/app/scodoc/sco_apogee_csv.py @@ -258,11 +258,16 @@ class ApoEtud(dict): self["nom"] = nom self["prenom"] = prenom self["naissance"] = naissance - self.cols = cols # { col_id : value } colid = 'apoL_c0001' + self.cols = cols + "{ col_id : value } colid = 'apoL_c0001'" + self.col_elts = {} + "{'V1RT': {'R': 'ADM', 'J': '', 'B': 20, 'N': '12.14'}}" self.new_cols = {} # { col_id : value to record in csv } - self.etud = None # etud ScoDoc + self.etud: Identite = None + "etudiant ScoDoc associé" self.etat = None # ETUD_OK, ... - self.is_NAR = False # set to True si NARé dans un semestre + self.is_NAR = False + "True si NARé dans un semestre" self.log = [] self.has_logged_no_decision = False self.export_res_etape = export_res_etape # VET, ... @@ -276,7 +281,7 @@ class ApoEtud(dict): ) def __repr__(self): - return "ApoEtud( nom='%s', nip='%s' )" % (self["nom"], self["nip"]) + return f"""ApoEtud( nom='{self["nom"]}', nip='{self["nip"]}' )""" def lookup_scodoc(self, etape_formsemestre_ids): """Cherche l'étudiant ScoDoc associé à cet étudiant Apogée. @@ -284,6 +289,10 @@ class ApoEtud(dict): met .etud à None. Sinon, cherche le semestre, et met l'état à ETUD_OK ou ETUD_NON_INSCRIT. """ + + # futur: #WIP + # etud: Identite = Identite.query.filter_by(code_nip=self["nip"]).first() + # self.etud = etud etuds = sco_etud.get_etud_info(code_nip=self["nip"], filled=True) if not etuds: # pas dans ScoDoc @@ -291,13 +300,16 @@ class ApoEtud(dict): self.log.append("non inscrit dans ScoDoc") self.etat = ETUD_ORPHELIN else: + # futur: #WIP + # formsemestre_ids = { + # ins.formsemestre_id for ins in etud.formsemestre_inscriptions + # } + # in_formsemestre_ids = formsemestre_ids.intersection(etape_formsemestre_ids) self.etud = etuds[0] # cherche le semestre ScoDoc correspondant à l'un de ceux de l'etape: formsemestre_ids = {s["formsemestre_id"] for s in self.etud["sems"]} - self.in_formsemestre_ids = formsemestre_ids.intersection( - etape_formsemestre_ids - ) - if not self.in_formsemestre_ids: + in_formsemestre_ids = formsemestre_ids.intersection(etape_formsemestre_ids) + if not in_formsemestre_ids: self.log.append( "connu dans ScoDoc, mais pas inscrit dans un semestre de cette étape" ) @@ -305,7 +317,7 @@ class ApoEtud(dict): else: self.etat = ETUD_OK - def associate_sco(self, apo_data): + def associate_sco(self, apo_data: "ApoData"): """Recherche les valeurs des éléments Apogée pour cet étudiant Set .new_cols """ @@ -327,7 +339,7 @@ class ApoEtud(dict): cur_sem, autre_sem = self.etud_semestres_de_etape(apo_data) for sem in apo_data.sems_etape: el = self.search_elt_in_sem(code, sem, cur_sem, autre_sem) - if el != None: + if el is not None: sco_elts[code] = el break self.col_elts[code] = el @@ -338,15 +350,15 @@ class ApoEtud(dict): self.new_cols[col_id] = sco_elts[code][ apo_data.cols[col_id]["Type Rés."] ] - except KeyError: + except KeyError as exc: log( - "associate_sco: missing key, etud=%s\ncode='%s'\netape='%s'" - % (self, code, apo_data.etape_apogee) + f"associate_sco: missing key, etud={self}\ncode='{code}'\netape='{apo_data.etape_apogee}'" ) raise ScoValueError( - """L'élément %s n'a pas de résultat: peut-être une erreur dans les codes sur le programme pédagogique (vérifier qu'il est bien associé à une UE ou semestre)?""" - % code - ) + f"""L'élément {code} n'a pas de résultat: peut-être une erreur + dans les codes sur le programme pédagogique + (vérifier qu'il est bien associé à une UE ou semestre)?""" + ) from exc # recopie les 4 premieres colonnes (nom, ..., naissance): for col_id in apo_data.col_ids[:4]: self.new_cols[col_id] = self.cols[col_id] @@ -356,7 +368,7 @@ class ApoEtud(dict): # codes = set([apo_data.cols[col_id].code for col_id in apo_data.col_ids]) # return codes - set(sco_elts) - def search_elt_in_sem(self, code, sem, cur_sem, autre_sem): + def search_elt_in_sem(self, code, sem, cur_sem, autre_sem) -> dict: """ VET code jury etape ELP élément pédagogique: UE, module @@ -820,10 +832,8 @@ class ApoData(object): elts[col["Code"]] = ApoElt([col]) return elts # { code apo : ApoElt } - def apo_read_etuds(self, f): - """Lecture des etudiants (et resultats) du fichier CSV Apogée - -> liste de dicts - """ + def apo_read_etuds(self, f) -> list[ApoEtud]: + """Lecture des etudiants (et resultats) du fichier CSV Apogée""" L = [] while True: line = f.readline() @@ -958,36 +968,38 @@ class ApoData(object): """ codes_by_sem = {} for sem in self.sems_etape: + formsemestre: FormSemestre = FormSemestre.query.get_or_404( + sem["formsemestre_id"] + ) + # L'ensemble des codes apo associés aux éléments: + codes_semestre = formsemestre.get_codes_apogee() + codes_modules = set().union( + *[ + modimpl.module.get_codes_apogee() + for modimpl in formsemestre.modimpls + ] + ) + codes_ues = set().union( + *[ + ue.get_codes_apogee() + for ue in formsemestre.query_ues(with_sport=True) + ] + ) s = set() codes_by_sem[sem["formsemestre_id"]] = s for col_id in self.col_ids[4:]: code = self.cols[col_id]["Code"] # 'V1RT' - # associé à l'étape, l'année ou les semestre: - if ( - sco_formsemestre.sem_has_etape(sem, code) - or (code in {x.strip() for x in sem["elt_sem_apo"].split(",")}) - or (code in {x.strip() for x in sem["elt_annee_apo"].split(",")}) - ): + # associé à l'étape, l'année ou le semestre: + if code in codes_semestre: s.add(code) continue # associé à une UE: - formsemestre = FormSemestre.query.get_or_404(sem["formsemestre_id"]) - nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) - for ue in nt.get_ues_stat_dict(): - if ue["code_apogee"]: - codes = {x.strip() for x in ue["code_apogee"].split(",")} - if code in codes: - s.add(code) - continue + if code in codes_ues: + s.add(code) + continue # associé à un module: - modimpls = nt.get_modimpls_dict() - for modimpl in modimpls: - module = modimpl["module"] - if module["code_apogee"]: - codes = {x.strip() for x in module["code_apogee"].split(",")} - if code in codes: - s.add(code) - continue + if code in codes_modules: + s.add(code) # log('codes_by_sem=%s' % pprint.pformat(codes_by_sem)) return codes_by_sem diff --git a/app/scodoc/sco_archives.py b/app/scodoc/sco_archives.py index 773fc0426..26ce9f8c7 100644 --- a/app/scodoc/sco_archives.py +++ b/app/scodoc/sco_archives.py @@ -47,6 +47,7 @@ qui est une description (humaine, format libre) de l'archive. """ +import chardet import datetime import glob import json @@ -55,7 +56,7 @@ import os import re import shutil import time -import chardet +from typing import Union import flask from flask import g, request @@ -232,14 +233,17 @@ class BaseArchiver(object): os.mkdir(archive_id) # if exists, raises an OSError finally: scu.GSL.release() - self.store(archive_id, "_description.txt", description.encode("utf-8")) + self.store(archive_id, "_description.txt", description) return archive_id - def store(self, archive_id: str, filename: str, data: bytes): + def store(self, archive_id: str, filename: str, data: Union[str, bytes]): """Store data in archive, under given filename. Filename may be modified (sanitized): return used filename The file is created or replaced. + data may be str or bytes """ + if isinstance(data, str): + data = data.encode(scu.SCO_ENCODING) self.initialize() filename = scu.sanitize_filename(filename) log("storing %s (%d bytes) in %s" % (filename, len(data), archive_id)) @@ -350,19 +354,21 @@ def do_formsemestre_archive( html_sco_header.sco_footer(), ] ) - data = data.encode(scu.SCO_ENCODING) PVArchive.store(archive_id, "Tableau_moyennes.html", data) # Bulletins en JSON data = gen_formsemestre_recapcomplet_json(formsemestre_id, xml_with_decisions=True) data_js = json.dumps(data, indent=1, cls=scu.ScoDocJSONEncoder) - data_js = data_js.encode(scu.SCO_ENCODING) if data: PVArchive.store(archive_id, "Bulletins.json", data_js) # Decisions de jury, en XLS data = sco_pvjury.formsemestre_pvjury(formsemestre_id, format="xls", publish=False) if data: - PVArchive.store(archive_id, "Decisions_Jury" + scu.XLSX_SUFFIX, data) + PVArchive.store( + archive_id, + "Decisions_Jury" + scu.XLSX_SUFFIX, + data, + ) # Classeur bulletins (PDF) data, _ = sco_bulletins_pdf.get_formsemestre_bulletins_pdf( formsemestre_id, version=bulVersion diff --git a/app/scodoc/sco_bulletins.py b/app/scodoc/sco_bulletins.py index df790649a..daf4a9419 100644 --- a/app/scodoc/sco_bulletins.py +++ b/app/scodoc/sco_bulletins.py @@ -158,9 +158,24 @@ def formsemestre_bulletinetud_dict(formsemestre_id, etudid, version="long"): I["server_name"] = request.url_root # Formation et parcours - I["formation"] = sco_formations.formation_list( - args={"formation_id": I["sem"]["formation_id"]} - )[0] + if I["sem"]["formation_id"]: + I["formation"] = sco_formations.formation_list( + args={"formation_id": I["sem"]["formation_id"]} + )[0] + else: # what's the fuck ? + I["formation"] = { + "acronyme": "?", + "code_specialite": "", + "dept_id": 1, + "formation_code": "?", + "formation_id": -1, + "id": -1, + "referentiel_competence_id": None, + "titre": "?", + "titre_officiel": "?", + "type_parcours": 0, + "version": 0, + } I["parcours"] = sco_codes_parcours.get_parcours_from_code( I["formation"]["type_parcours"] ) diff --git a/app/scodoc/sco_bulletins_standard.py b/app/scodoc/sco_bulletins_standard.py index e7c92ad78..e1e853720 100644 --- a/app/scodoc/sco_bulletins_standard.py +++ b/app/scodoc/sco_bulletins_standard.py @@ -439,15 +439,15 @@ class BulletinGeneratorStandard(sco_bulletins_generator.BulletinGenerator): ects_txt = "-" t = { - "titre": ue["acronyme"] + " " + ue["titre"], + "titre": ue["acronyme"] + " " + (ue["titre"] or ""), "_titre_html": plusminus - + ue["acronyme"] + + (ue["acronyme"] or "") + " " - + ue["titre"] + + (ue["titre"] or "") + ' ' - + ue["ue_descr_txt"] + + (ue["ue_descr_txt"] or "") + "", - "_titre_help": ue["ue_descr_txt"], + "_titre_help": ue["ue_descr_txt"] or "", "_titre_colspan": 2, "module": ue_descr, "note": ue["moy_ue_txt"], diff --git a/app/scodoc/sco_cache.py b/app/scodoc/sco_cache.py index 8fcc6f7ef..7975e3b2c 100644 --- a/app/scodoc/sco_cache.py +++ b/app/scodoc/sco_cache.py @@ -67,6 +67,7 @@ class ScoDocCache: timeout = None # ttl, infinite by default prefix = "" + verbose = False # if true, verbose logging (debug) @classmethod def _get_key(cls, oid): @@ -87,7 +88,10 @@ class ScoDocCache: def set(cls, oid, value): """Store value""" key = cls._get_key(oid) - # log(f"CACHE key={key}, type={type(value)}, timeout={cls.timeout}") + if cls.verbose: + log( + f"{cls.__name__}.set key={key}, type={type(value).__name__}, timeout={cls.timeout}" + ) try: status = CACHE.set(key, value, timeout=cls.timeout) if not status: @@ -101,11 +105,15 @@ class ScoDocCache: @classmethod def delete(cls, oid): """Remove from cache""" + # if cls.verbose: + # log(f"{cls.__name__}.delete({oid})") CACHE.delete(cls._get_key(oid)) @classmethod def delete_many(cls, oids): """Remove multiple keys at once""" + if cls.verbose: + log(f"{cls.__name__}.delete_many({oids})") # delete_many seems bugged: # CACHE.delete_many([cls._get_key(oid) for oid in oids]) for oid in oids: diff --git a/app/scodoc/sco_codes_parcours.py b/app/scodoc/sco_codes_parcours.py index e77c711d2..dc59aeb2e 100644 --- a/app/scodoc/sco_codes_parcours.py +++ b/app/scodoc/sco_codes_parcours.py @@ -35,7 +35,7 @@ from app import log @enum.unique class CodesParcours(enum.IntEnum): - """Codes numériques de sparcours, enregistrés en base + """Codes numériques des parcours, enregistrés en base dans notes_formations.type_parcours Ne pas modifier. """ @@ -68,7 +68,8 @@ NOTES_TOLERANCE = 0.00499999999999 # si note >= (BARRE-TOLERANCE), considere ok # (permet d'eviter d'afficher 10.00 sous barre alors que la moyenne vaut 9.999) # Barre sur moyenne générale utilisée pour compensations semestres: -NOTES_BARRE_GEN_COMPENSATION = 10.0 - NOTES_TOLERANCE +NOTES_BARRE_GEN = 10.0 +NOTES_BARRE_GEN_COMPENSATION = NOTES_BARRE_GEN - NOTES_TOLERANCE # ---------------------------------------------------------------- # Types d'UE: @@ -114,6 +115,8 @@ UE_SEM_DEFAULT = 1000000 # indice semestre des UE sans modules # ------------------------------------------------------------------ # Codes proposés par ADIUT / Apogee +ABAN = "ABAN" +ABL = "ABL" ADM = "ADM" # moyenne gen., barres UE, assiduité: sem. validé ADC = "ADC" # admis par compensation (eg moy(S1, S2) > 10) ADJ = "ADJ" # admis par le jury @@ -122,10 +125,16 @@ ATJ = "ATJ" # pb assiduité: décision repoussée au semestre suivant ATB = "ATB" AJ = "AJ" CMP = "CMP" # utile pour UE seulement (indique UE acquise car semestre acquis) -NAR = "NAR" -RAT = "RAT" # en attente rattrapage, sera ATT dans Apogée DEF = "DEF" # défaillance (n'est pas un code jury dans scodoc mais un état, comme inscrit ou demission) DEM = "DEM" +EXCLU = "EXCLU" +JSD = "JSD" # jury tenu mais pas de code (Jury Sans Décision) +NAR = "NAR" +PASD = "PASD" +PAS1NCI = "PAS1NCI" +RAT = "RAT" # en attente rattrapage, sera ATT dans Apogée +RED = "RED" +UEBSL = "UEBSL" # UE blanchie # codes actions REDOANNEE = "REDOANNEE" # redouble annee (va en Sn-1) @@ -143,22 +152,34 @@ ALL = "ALL" # Explication des codes (de semestre ou d'UE) CODES_EXPL = { + ABAN: "Non évalué pour manque d’assiduité: non présentation des notes de l'étudiant au jury", + ABL: "Année blanche", ADC: "Validé par compensation", ADJ: "Validé par le Jury", ADM: "Validé", - AJ: "Ajourné", + AJ: "Ajourné (ou UE/BC de BUT en attente pour problème de moyenne)", ATB: "Décision en attente d'un autre semestre (au moins une UE sous la barre)", ATJ: "Décision en attente d'un autre semestre (assiduité insuffisante)", ATT: "Décision en attente d'un autre semestre (faute d'atteindre la moyenne)", - CMP: "Code UE acquise car semestre acquis", - DEF: "Défaillant", - NAR: "Échec, non autorisé à redoubler", - RAT: "En attente d'un rattrapage", + CMP: """Code UE acquise car semestre acquis, ou, en BUT, acquise par + compensation UE avec l’UE de même compétence et de même année (ECTS acquis). + Utilisé aussi pour les blocs de compétences BUT (RCUE). + """, + DEF: "Défaillant, pas ou peu de notes par arrêt de la formation. Non évalué par manque assiduité.", DEM: "Démission", + EXCLU: "Exclusion: décision réservée à des décisions disciplinaires", + NAR: "Non admis, réorientation, non autorisé à redoubler", + PASD: """Année BUT: non admis, mais passage de droit: + Passage en Année Supérieure de Droit (+ de 50% des UE VAL et RCUE Ajourné(s) >=8) + """, + PAS1NCI: """Année BUT: Non admis, mais passage par décision de jury: + Passage en Année Supérieure avec au moins 1 Niveau de Compétence Insuffisant (RCUE<8) + """, + RAT: "En attente d'un rattrapage", + RED: "Année: Ajourné, mais autorisé à redoubler", + UEBSL: "UE blanchie", } -# Nota: ces explications sont personnalisables via le fichier -# de config locale /opt/scodoc/var/scodoc/config/scodoc_local.py -# variable: CONFIG.CODES_EXP + # Les codes de semestres: CODES_JURY_SEM = {ADC, ADJ, ADM, AJ, ATB, ATJ, ATT, DEF, NAR, RAT} @@ -167,7 +188,21 @@ CODES_SEM_ATTENTES = {ATT: True, ATB: True, ATJ: True} # semestre en attente CODES_SEM_REO = {NAR: 1} # reorientation -CODES_UE_VALIDES = {ADM: True, CMP: True} # UE validée +CODES_UE_VALIDES = {ADM: True, CMP: True, ADJ: True} # UE validée +CODES_RCUE_VALIDES = CODES_UE_VALIDES # Niveau RCUE validé +# Pour le BUT: +CODES_ANNEE_ARRET = {DEF, DEM, ABAN, ABL} +CODES_RCUE = {ADM, AJ, CMP} +BUT_BARRE_UE8 = 8.0 - NOTES_TOLERANCE +BUT_BARRE_UE = BUT_BARRE_RCUE = 10.0 - NOTES_TOLERANCE +BUT_RCUE_SUFFISANT = 8.0 - NOTES_TOLERANCE +BUT_CODES_PASSAGE = { + ADM, + ADJ, + PASD, + PAS1NCI, + ATJ, +} def code_semestre_validant(code: str) -> bool: diff --git a/app/scodoc/sco_edit_apc.py b/app/scodoc/sco_edit_apc.py index c53094ef5..7e71f51d1 100644 --- a/app/scodoc/sco_edit_apc.py +++ b/app/scodoc/sco_edit_apc.py @@ -76,7 +76,7 @@ def html_edit_formation_apc( ues_by_sem[semestre_idx] = formation.ues.filter_by( semestre_idx=semestre_idx ).order_by(UniteEns.semestre_idx, UniteEns.numero, UniteEns.acronyme) - ects = [ue.ects for ue in ues_by_sem[semestre_idx]] + ects = [ue.ects for ue in ues_by_sem[semestre_idx] if ue.type != UE_SPORT] if None in ects: ects_by_sem[semestre_idx] = 'manquant' else: @@ -127,27 +127,33 @@ def html_edit_formation_apc( formation=formation, titre=f"Ressources du S{semestre_idx}", create_element_msg="créer une nouvelle ressource", - matiere_parent=matiere_parent, + # matiere_parent=matiere_parent, modules=ressources_in_sem, module_type=ModuleType.RESSOURCE, editable=editable, tag_editable=tag_editable, icons=icons, scu=scu, - ), + semestre_id=semestre_idx, + ) + if ues_by_sem[semestre_idx].count() > 0 + else "", render_template( "pn/form_mods.html", formation=formation, titre=f"Situations d'Apprentissage et d'Évaluation (SAÉs) S{semestre_idx}", create_element_msg="créer une nouvelle SAÉ", - matiere_parent=matiere_parent, + # matiere_parent=matiere_parent, modules=saes_in_sem, module_type=ModuleType.SAE, editable=editable, tag_editable=tag_editable, icons=icons, scu=scu, - ), + semestre_id=semestre_idx, + ) + if ues_by_sem[semestre_idx].count() > 0 + else "", render_template( "pn/form_mods.html", formation=formation, @@ -159,7 +165,10 @@ def html_edit_formation_apc( tag_editable=tag_editable, icons=icons, scu=scu, - ), + semestre_id=semestre_idx, + ) + if ues_by_sem[semestre_idx].count() > 0 + else """créer une UE pour pouvoir ajouter des modules""", ] return "\n".join(H) diff --git a/app/scodoc/sco_edit_formation.py b/app/scodoc/sco_edit_formation.py index 606fc7421..62fc2bbf4 100644 --- a/app/scodoc/sco_edit_formation.py +++ b/app/scodoc/sco_edit_formation.py @@ -245,7 +245,11 @@ def formation_edit(formation_id=None, create=False): return ( "\n".join(H) + tf_error_message( - "Valeurs incorrectes: il existe déjà une formation avec même titre, acronyme et version." + f"""Valeurs incorrectes: il existe déjà une formation avec même titre, + acronyme et version. + """ ) + tf[1] + html_sco_header.sco_footer() diff --git a/app/scodoc/sco_edit_module.py b/app/scodoc/sco_edit_module.py index f99bb8a18..b94a60178 100644 --- a/app/scodoc/sco_edit_module.py +++ b/app/scodoc/sco_edit_module.py @@ -29,7 +29,7 @@ (portage from DTML) """ import flask -from flask import url_for, render_template +from flask import flash, url_for, render_template from flask import g, request from flask_login import current_user @@ -39,6 +39,7 @@ from app.models import APO_CODE_STR_LEN from app.models import Formation, Matiere, Module, UniteEns from app.models import FormSemestre, ModuleImpl from app.models import ScolarNews +from app.models.but_refcomp import ApcAppCritique, ApcParcours import app.scodoc.notesdb as ndb import app.scodoc.sco_utils as scu @@ -121,235 +122,13 @@ def module_create( Sinon, donne le choix de l'UE de rattachement et utilise la première matière de cette UE (si elle n'existe pas, la crée). """ - if matiere_id: - matiere = Matiere.query.get_or_404(matiere_id) - ue = matiere.ue - formation = ue.formation - else: - formation = Formation.query.get_or_404(formation_id) - parcours = formation.get_parcours() - is_apc = parcours.APC_SAE - ues = formation.ues.order_by( - UniteEns.semestre_idx, UniteEns.numero, UniteEns.acronyme - ).all() - # cherche le numero adéquat (pour placer le module en fin de liste) - modules = formation.modules.all() - if modules: - default_num = max([m.numero or 0 for m in modules]) + 10 - else: - default_num = 10 - - if is_apc and module_type is not None: - object_name = scu.MODULE_TYPE_NAMES[module_type] - else: - object_name = "Module" - H = [ - html_sco_header.sco_header(page_title=f"Création {object_name}"), - ] - if not matiere_id: - H += [ - f"""

Création {object_name} dans la formation {formation.acronyme} -

- """ - ] - else: - H += [ - f"""

Création {object_name} dans la matière {matiere.titre}, - (UE {ue.acronyme}), semestre {ue.semestre_idx}

- """ - ] - - H += [ - render_template( - "scodoc/help/modules.html", - is_apc=is_apc, - semestre_id=semestre_id, - ) - ] - - descr = [ - ( - "code", - { - "size": 10, - "explanation": "code du module, ressource ou SAÉ. Exemple M1203, R2.01, ou SAÉ 3.4. Ce code doit être unique dans la formation.", - "allow_null": False, - "validator": lambda val, field, formation_id=formation_id: check_module_code_unicity( - val, field, formation_id - ), - }, - ), - ( - "titre", - { - "size": 30, - "explanation": "nom du module. Exemple: Introduction à la démarche ergonomique", - }, - ), - ( - "abbrev", - { - "size": 20, - "explanation": "nom abrégé (pour les bulletins). Exemple: Intro. à l'ergonomie", - }, - ), - ] - - if is_apc: - module_types = scu.ModuleType # tous les types - else: - # ne propose pas SAE et Ressources: - module_types = set(scu.ModuleType) - { - scu.ModuleType.RESSOURCE, - scu.ModuleType.SAE, - } - - descr += [ - ( - "module_type", - { - "input_type": "menu", - "title": "Type", - "explanation": "", - "labels": [x.name.capitalize() for x in module_types], - "allowed_values": [str(int(x)) for x in module_types], - }, - ), - ( - "heures_cours", - { - "title": "Heures de cours", - "size": 4, - "type": "float", - "explanation": "nombre d'heures de cours (optionnel)", - }, - ), - ( - "heures_td", - { - "title": "Heures de TD", - "size": 4, - "type": "float", - "explanation": "nombre d'heures de Travaux Dirigés (optionnel)", - }, - ), - ( - "heures_tp", - { - "title": "Heures de TP", - "size": 4, - "type": "float", - "explanation": "nombre d'heures de Travaux Pratiques (optionnel)", - }, - ), - ] - if is_apc: - descr += [ - ( - "sep_ue_coefs", - { - "input_type": "separator", - "title": """ -
(les coefficients vers les UE se fixent sur la page dédiée) -
""", - }, - ), - ] - else: - descr += [ - ( - "coefficient", - { - "size": 4, - "type": "float", - "explanation": "coefficient dans la formation (PPN)", - "allow_null": False, - }, - ), - ] - - if matiere_id: - descr += [ - ("ue_id", {"default": ue.id, "input_type": "hidden"}), - ("matiere_id", {"default": matiere_id, "input_type": "hidden"}), - ] - else: - # choix de l'UE de rattachement - descr += [ - ( - "ue_id", - { - "input_type": "menu", - "type": "int", - "title": "UE de rattachement", - "explanation": "utilisée notamment pour les malus", - "labels": [ - f"S{u.semestre_idx if u.semestre_idx is not None else '.'} / {u.acronyme} {u.titre}" - for u in ues - ], - "allowed_values": [u.id for u in ues], - }, - ), - ] - - descr += [ - # ('ects', { 'size' : 4, 'type' : 'float', 'title' : 'ECTS', 'explanation' : 'nombre de crédits ECTS (inutilisés: les crédits sont associés aux UE)' }), - ("formation_id", {"default": formation.id, "input_type": "hidden"}), - ( - "code_apogee", - { - "title": "Code Apogée", - "size": 25, - "explanation": "(optionnel) code élément pédagogique Apogée ou liste de codes ELP séparés par des virgules", - "validator": lambda val, _: len(val) < APO_CODE_STR_LEN, - }, - ), - ( - "numero", - { - "size": 2, - "explanation": "numéro (1,2,3,4...) pour ordre d'affichage", - "type": "int", - "default": default_num, - }, - ), - ] - args = scu.get_request_args() - tf = TrivialFormulator( - request.base_url, - args, - descr, - submitlabel="Créer ce module", + return module_edit( + create=True, + matiere_id=matiere_id, + module_type=module_type, + semestre_id=semestre_id, + formation_id=formation_id, ) - if tf[0] == 0: - return "\n".join(H) + tf[1] + html_sco_header.sco_footer() - else: - if not matiere_id: - # formulaire avec choix UE de rattachement - ue = UniteEns.query.get(tf[2]["ue_id"]) - if ue is None: - raise ValueError("UE invalide") - matiere = ue.matieres.first() - if matiere: - tf[2]["matiere_id"] = matiere.id - else: - matiere_id = sco_edit_matiere.do_matiere_create( - {"ue_id": ue.id, "titre": ue.titre, "numero": 1}, - ) - tf[2]["matiere_id"] = matiere_id - - tf[2]["semestre_id"] = ue.semestre_idx - - _ = do_module_create(tf[2]) - - return flask.redirect( - url_for( - "notes.ue_table", - scodoc_dept=g.scodoc_dept, - formation_id=formation.id, - semestre_idx=tf[2]["semestre_id"], - ) - ) def can_delete_module(module): @@ -359,7 +138,6 @@ def can_delete_module(module): def do_module_delete(oid): "delete module" - module = Module.query.get_or_404(oid) mod = module_list({"module_id": oid})[0] # sco7 if module_is_locked(module.id): @@ -379,9 +157,14 @@ def do_module_delete(oid): # S'il y a des moduleimpls, on ne peut pas detruire le module ! mods = sco_moduleimpl.moduleimpl_list(module_id=oid) if mods: - err_page = f"""

Destruction du module impossible car il est utilisé dans des semestres existants !

-

Il faut d'abord supprimer le semestre (ou en retirer ce module). Mais il est peut être préférable de - laisser ce programme intact et d'en créer une nouvelle version pour la modifier sans affecter les semestres déjà en place. + err_page = f""" +

Destruction du module impossible car il est utilisé dans des + semestres existants !

+

Il faut d'abord supprimer le semestre (ou en retirer + ce module). + Mais il est peut être préférable de laisser ce programme intact et + d'en créer une nouvelle version pour la modifier sans affecter + les semestres déjà en place.

reprendre @@ -465,37 +248,65 @@ def do_module_edit(vals: dict) -> None: def check_module_code_unicity(code, field, formation_id, module_id=None): "true si code module unique dans la formation" - Mods = module_list(args={"code": code, "formation_id": formation_id}) + modules = module_list(args={"code": code, "formation_id": formation_id}) if module_id: # edition: supprime le module en cours - Mods = [m for m in Mods if m["module_id"] != module_id] + modules = [m for m in modules if m["module_id"] != module_id] - return len(Mods) == 0 + return len(modules) == 0 -def module_edit(module_id=None): - """Edit a module""" - from app.scodoc import sco_formations +def module_edit( + module_id=None, + create=False, + matiere_id=None, + module_type=None, + semestre_id=None, + formation_id=None, +): + """Formulaire édition ou création module. + Si create, création nouveau module. + Si matiere_id est spécifié, le module sera créé dans cette matière (cas normal). + Sinon, donne le choix de l'UE de rattachement et utilise la première matière + de cette UE (si elle n'existe pas, la crée). + """ from app.scodoc import sco_tag_module - if not module_id: - raise ScoValueError("invalid module !") - modules = module_list(args={"module_id": module_id}) - if not modules: - raise ScoValueError("invalid module !") - module = modules[0] - a_module = models.Module.query.get(module_id) - unlocked = not module_is_locked(module_id) - formation_id = module["formation_id"] - formation = sco_formations.formation_list(args={"formation_id": formation_id})[0] - parcours = sco_codes_parcours.get_parcours_from_code(formation["type_parcours"]) + # --- Détermination de la formation + orig_semestre_idx = semestre_id + ue = None + if create: + if matiere_id: + matiere = Matiere.query.get_or_404(matiere_id) + ue = matiere.ue + formation = ue.formation + orig_semestre_idx = ue.semestre_idx if semestre_id is None else semestre_id + else: + formation = Formation.query.get_or_404(formation_id) + module = None + unlocked = True + else: + if not module_id: + raise ValueError("missing module_id !") + module = models.Module.query.get_or_404(module_id) + ue = module.ue + module_dict = module.to_dict() + formation = module.formation + unlocked = not module_is_locked(module_id) + + parcours = sco_codes_parcours.get_parcours_from_code(formation.type_parcours) is_apc = parcours.APC_SAE # BUT - in_use = len(a_module.modimpls.all()) > 0 # il y a des modimpls + if not create: + orig_semestre_idx = module.ue.semestre_idx if is_apc else module.semestre_id + if orig_semestre_idx is None: + orig_semestre_idx = 1 + # il y a-t-il des modimpls ? + in_use = (module is not None) and (len(module.modimpls.all()) > 0) matieres = Matiere.query.filter( - Matiere.ue_id == UniteEns.id, UniteEns.formation_id == formation_id + Matiere.ue_id == UniteEns.id, UniteEns.formation_id == formation.id ).order_by(UniteEns.semestre_idx, UniteEns.numero, Matiere.numero) if in_use: # restreint aux matières du même semestre - matieres = matieres.filter(UniteEns.semestre_idx == a_module.ue.semestre_idx) + matieres = matieres.filter(UniteEns.semestre_idx == module.ue.semestre_idx) if is_apc: # ne conserve que la 1ere matière de chaque UE, @@ -503,7 +314,8 @@ def module_edit(module_id=None): matieres = [ mat for mat in matieres - if a_module.matiere.id == mat.id or mat.id == mat.ue.matieres.first().id + if ((module is not None) and (module.matiere.id == mat.id)) + or (mat.id == mat.ue.matieres.first().id) ] mat_names = [ "S%s / %s" % (mat.ue.semestre_idx, mat.ue.acronyme) for mat in matieres @@ -511,64 +323,131 @@ def module_edit(module_id=None): else: mat_names = ["%s / %s" % (mat.ue.acronyme, mat.titre or "") for mat in matieres] - ue_mat_ids = ["%s!%s" % (mat.ue.id, mat.id) for mat in matieres] - module["ue_matiere_id"] = "%s!%s" % (module["ue_id"], module["matiere_id"]) + if module: # edition + ue_mat_ids = ["%s!%s" % (mat.ue.id, mat.id) for mat in matieres] + module_dict["ue_matiere_id"] = "%s!%s" % ( + module_dict["ue_id"], + module_dict["matiere_id"], + ) semestres_indices = list(range(1, parcours.NB_SEM + 1)) + # Toutes les UEs de la formation (tout parcours): + ues = formation.ues.order_by( + UniteEns.semestre_idx, UniteEns.numero, UniteEns.acronyme + ).all() + # L'UE de rattachement par défaut: 1ere du semestre + ue_default = ( + formation.ues.filter_by(semestre_idx=orig_semestre_idx) + .order_by(UniteEns.numero, UniteEns.acronyme) + .first() + ) + + # --- Titre de la page + if create: + if is_apc and module_type is not None: + object_name = scu.MODULE_TYPE_NAMES[module_type] + else: + object_name = "Module" + page_title = f"Création {object_name}" + if matiere_id: + title = f"""Création {object_name} dans la matière + {matiere.titre}, + (UE {ue.acronyme}), semestre {ue.semestre_idx} + """ + else: + title = f"""Création {object_name} dans la formation + {formation.acronyme}""" + else: + page_title = f"Modification du module {module.code or module.titre or ''}" + title = f"""Modification du module {module.code or ''} {module.titre or ''} + (formation {formation.acronyme}, version {formation.version}) + """ H = [ html_sco_header.sco_header( - page_title=f"Modification du module {a_module.code or a_module.titre or ''}", + page_title=page_title, cssstyles=["libjs/jQuery-tagEditor/jquery.tag-editor.css"], javascripts=[ "libjs/jQuery-tagEditor/jquery.tag-editor.min.js", "libjs/jQuery-tagEditor/jquery.caret.min.js", "js/module_tag_editor.js", + "js/module_edit.js", ], ), - f"""

Modification du module {a_module.code or ''} {a_module.titre or ''}""", - """ (formation %(acronyme)s, version %(version)s)

""" % formation, + f"""

{title}

""", render_template( "scodoc/help/modules.html", is_apc=is_apc, + semestre_id=semestre_id, formsemestres=FormSemestre.query.filter( ModuleImpl.formsemestre_id == FormSemestre.id, ModuleImpl.module_id == module_id, ) .order_by(FormSemestre.date_debut) - .all(), + .all() + if not create + else None, ), ] if not unlocked: H.append( - """
Formation verrouillée, seuls certains éléments peuvent être modifiés
""" + """
Formation verrouillée, seuls + certains éléments peuvent être modifiés
""" ) if is_apc: module_types = scu.ModuleType # tous les types else: # ne propose pas SAE et Ressources, sauf si déjà de ce type... - module_types = ( - set(scu.ModuleType) - {scu.ModuleType.RESSOURCE, scu.ModuleType.SAE} - ) | { - scu.ModuleType(a_module.module_type) - if a_module.module_type - else scu.ModuleType.STANDARD + module_types = set(scu.ModuleType) - { + scu.ModuleType.RESSOURCE, + scu.ModuleType.SAE, } + if module: + module_types |= { + scu.ModuleType(module.module_type) + if module.module_type + else scu.ModuleType.STANDARD + } + # Numéro du module + # cherche le numero adéquat (pour placer le module en fin de liste) + if module: + default_num = module.numero + else: + modules = formation.modules.all() + if modules: + default_num = max([m.numero or 0 for m in modules]) + 10 + else: + default_num = 10 descr = [ ( "code", { "size": 10, - "explanation": "code du module (issu du programme, exemple M1203 ou R2.01. Doit être unique dans la formation)", + "explanation": """code du module (issu du programme, exemple M1203, + R2.01, ou SAÉ 3.4. Doit être unique dans la formation)""", "allow_null": False, - "validator": lambda val, field, formation_id=formation_id: check_module_code_unicity( - val, field, formation_id, module_id=module_id + "validator": lambda val, field, formation_id=formation.id: check_module_code_unicity( + val, field, formation_id, module_id=module.id if module else None ), }, ), - ("titre", {"size": 30, "explanation": "nom du module"}), - ("abbrev", {"size": 20, "explanation": "nom abrégé (pour bulletins)"}), + ( + "titre", + { + "size": 30, + "explanation": """nom du module. Exemple: + Introduction à la démarche ergonomique""", + }, + ), + ( + "abbrev", + { + "size": 20, + "explanation": """nom abrégé (pour bulletins). + Exemple: Intro. à l'ergonomie""", + }, + ), ( "module_type", { @@ -583,50 +462,64 @@ def module_edit(module_id=None): ( "heures_cours", { - "title": "Heures CM :", + "title": "Heures cours :", "size": 4, "type": "float", - "explanation": "nombre d'heures de cours", + "explanation": "nombre d'heures de cours (optionnel)", }, ), ( "heures_td", { - "title": "Heures TD :", + "title": "Heures de TD :", "size": 4, "type": "float", - "explanation": "nombre d'heures de Travaux Dirigés", + "explanation": "nombre d'heures de Travaux Dirigés (optionnel)", }, ), ( "heures_tp", { - "title": "Heures TP :", + "title": "Heures de TP :", "size": 4, "type": "float", - "explanation": "nombre d'heures de Travaux Pratiques", + "explanation": "nombre d'heures de Travaux Pratiques (optionnel)", }, ), ] if is_apc: - coefs_lst = a_module.ue_coefs_list() - if coefs_lst: - coefs_descr_txt = ", ".join( - [f"{ue.acronyme}: {c}" for (ue, c) in coefs_lst] - ) + if module: + coefs_lst = module.ue_coefs_list() + if coefs_lst: + coefs_descr_txt = ", ".join( + [f"{ue.acronyme}: {c}" for (ue, c) in coefs_lst] + ) + else: + coefs_descr_txt = """non définis""" + descr += [ + ( + "ue_coefs", + { + "readonly": True, + "title": "Coefficients vers les UE ", + "default": coefs_descr_txt, + "explanation": """
(passer par la page d'édition de la + formation pour modifier les coefficients)""", + }, + ) + ] else: - coefs_descr_txt = """non définis""" - descr += [ - ( - "ue_coefs", - { - "readonly": True, - "title": "Coefficients vers les UE ", - "default": coefs_descr_txt, - "explanation": "
(passer par la page d'édition de la formation pour modifier les coefficients)", - }, - ) - ] + descr += [ + ( + "sep_ue_coefs", + { + "input_type": "separator", + "title": """ +
(les coefficients vers les UE se fixent sur la page dédiée) +
""", + }, + ), + ] else: # Module classique avec coef scalaire: descr += [ ( @@ -641,30 +534,72 @@ def module_edit(module_id=None): ), ] descr += [ - ("formation_id", {"input_type": "hidden"}), - ("ue_id", {"input_type": "hidden"}), - ("module_id", {"input_type": "hidden"}), ( - "ue_matiere_id", + "formation_id", { - "input_type": "menu", - "title": "Rattachement :" if is_apc else "Matière :", - "explanation": ( - "UE de rattachement" - + ( - " module utilisé, ne peut pas être changé de semestre" - if in_use - else "" - ) - ) - if is_apc - else "un module appartient à une seule matière.", - "labels": mat_names, - "allowed_values": ue_mat_ids, - "enabled": unlocked, + "input_type": "hidden", + "default": formation.id, + }, + ), + ( + "semestre_id", + { + "input_type": "hidden", + "default": orig_semestre_idx, }, ), ] + if module: + descr += [ + ("ue_id", {"input_type": "hidden"}), + ("module_id", {"input_type": "hidden"}), + ( + "ue_matiere_id", + { + "input_type": "menu", + "title": "Rattachement :" if is_apc else "Matière :", + "explanation": ( + "UE de rattachement, utilisée notamment pour les malus" + + ( + " (module utilisé, ne peut pas être changé de semestre)" + if in_use + else "" + ) + ) + if is_apc + else "un module appartient à une seule matière.", + "labels": mat_names, + "allowed_values": ue_mat_ids, + "enabled": unlocked, + }, + ), + ] + else: # Création + if matiere_id: + descr += [ + ("ue_id", {"default": ue.id, "input_type": "hidden"}), + ("matiere_id", {"default": matiere_id, "input_type": "hidden"}), + ] + else: + # choix de l'UE de rattachement + descr += [ + ( + "ue_id", + { + "input_type": "menu", + "type": "int", + "title": "UE de rattachement", + "explanation": "utilisée notamment pour les malus", + "labels": [ + f"S{u.semestre_idx if u.semestre_idx is not None else '.'} / {u.acronyme} {u.titre}" + for u in ues + ], + "allowed_values": [u.id for u in ues], + "default": ue_default.id if ue_default is not None else "", + }, + ), + ] + if is_apc: # le semestre du module est toujours celui de son UE descr += [ @@ -685,8 +620,7 @@ def module_edit(module_id=None): "input_type": "menu", "type": "int", "title": parcours.SESSION_NAME.capitalize(), - "explanation": "%s de début du module dans la formation standard" - % parcours.SESSION_NAME, + "explanation": f"{parcours.SESSION_NAME} de début du module dans la formation standard", "labels": [str(x) for x in semestres_indices], "allowed_values": semestres_indices, "enabled": unlocked, @@ -699,7 +633,7 @@ def module_edit(module_id=None): { "title": "Code Apogée", "size": 25, - "explanation": """(optionnel) code élément pédagogique Apogée ou liste de codes ELP + "explanation": """(optionnel) code élément pédagogique Apogée ou liste de codes ELP séparés par des virgules (ce code est propre à chaque établissement, se rapprocher du référent Apogée). """, @@ -710,26 +644,123 @@ def module_edit(module_id=None): "numero", { "size": 2, - "explanation": "numéro (1,2,3,4...) pour ordre d'affichage", + "explanation": "numéro (1, 2, 3, 4, ...) pour ordre d'affichage", "type": "int", + "default": default_num, }, ), ] + + if is_apc: + # Choix des parcours + ref_comp = formation.referentiel_competence + if ref_comp: + descr += [ + ( + "parcours", + { + "input_type": "checkbox", + "vertical": True, + "dom_id": "tf_module_parcours", + "labels": [parcour.libelle for parcour in ref_comp.parcours] + + ["Tous (tronc commun)"], + "allowed_values": [ + str(parcour.id) for parcour in ref_comp.parcours + ] + + ["-1"], + "explanation": """Parcours dans lesquels est utilisé ce module.
+ Attention: si le module ne doit pas avoir les mêmes coefficients suivant le parcours, + il faut en créer plusieurs versions, car dans ScoDoc chaque module a ses coefficients.""", + }, + ) + ] + if module: + module_dict["parcours"] = [ + str(parcour.id) for parcour in module.parcours + ] + module_dict["app_critiques"] = [ + str(app_crit.id) for app_crit in module.app_critiques + ] + # Choix des Apprentissages Critiques + if ue is not None: + annee = f"BUT{(orig_semestre_idx+1)//2}" + app_critiques = ApcAppCritique.app_critiques_ref_comp(ref_comp, annee) + descr += ( + [ + ( + "app_critiques", + { + "title": "Apprentissages Critiques", + "input_type": "checkbox", + "vertical": True, + "dom_id": "tf_module_app_critiques", + "labels": [ + f"{app_crit.code}  {app_crit.libelle}" + for app_crit in app_critiques + ], + "allowed_values": [ + str(app_crit.id) for app_crit in app_critiques + ], + "html_data": [], + "explanation": """Apprentissages Critiques liés à ce module. + (si vous changez le semestre, revenez ensuite sur cette page + pour associer les AC.) + """, + }, + ) + ] + if (ue.niveau_competence is not None) + else [ + ( + "app_critiques", + { + "input_type": "separator", + "title": f"""{scu.EMO_WARNING } + L'UE {ue.acronyme} {ue.titre} + n'est pas associée à un niveau de compétences + """, + }, + ) + ] + ) + else: + descr += [ + ( + "parcours", + { + "input_type": "separator", + "title": f"""{scu.EMO_WARNING } + Pas de parcours: + associer un référentiel de compétence + """, + }, + ) + ] # force module semestre_idx to its UE - if a_module.ue.semestre_idx: - module["semestre_id"] = a_module.ue.semestre_idx - # Filet de sécurité si jamais l'UE n'a pas non plus de semestre: - if not module["semestre_id"]: - module["semestre_id"] = 1 + if module: + if module.ue.semestre_idx is None: + # Filet de sécurité si jamais l'UE n'a pas non plus de semestre: + module_dict["semestre_id"] = 1 + else: + module_dict["semestre_id"] = module.ue.semestre_idx + tf = TrivialFormulator( request.base_url, scu.get_request_args(), descr, - html_foot_markup="""
""".format( - module_id, ",".join(sco_tag_module.module_tag_list(module_id)) - ), - initvalues=module, - submitlabel="Modifier ce module", + html_foot_markup=f"""
+ """ + if not create + else "", + initvalues=module_dict if module else {}, + submitlabel="Modifier ce module" if module else "Créer ce module", + cancelbutton="Annuler", ) # if tf[0] == 0: @@ -739,39 +770,77 @@ def module_edit(module_id=None): url_for( "notes.ue_table", scodoc_dept=g.scodoc_dept, - formation_id=formation_id, - semestre_idx=module["semestre_id"], + formation_id=formation.id, + semestre_idx=orig_semestre_idx, ) ) else: - # l'UE de rattachement peut changer - tf[2]["ue_id"], tf[2]["matiere_id"] = tf[2]["ue_matiere_id"].split("!") - x, y = tf[2]["ue_matiere_id"].split("!") - tf[2]["ue_id"] = int(x) - tf[2]["matiere_id"] = int(y) - old_ue_id = a_module.ue.id - new_ue_id = tf[2]["ue_id"] - if (old_ue_id != new_ue_id) and in_use: - new_ue = UniteEns.query.get_or_404(new_ue_id) - if new_ue.semestre_idx != a_module.ue.semestre_idx: - # pas changer de semestre un module utilisé ! - raise ScoValueError( - "Module utilisé: il ne peut pas être changé de semestre !" - ) - # En APC, force le semestre égal à celui de l'UE - if is_apc: - selected_ue = UniteEns.query.get(tf[2]["ue_id"]) - if selected_ue is None: - raise ValueError("UE invalide") - tf[2]["semestre_id"] = selected_ue.semestre_idx - # Check unicité code module dans la formation - do_module_edit(tf[2]) + if create: + if not matiere_id: + # formulaire avec choix UE de rattachement + ue = UniteEns.query.get(tf[2]["ue_id"]) + if ue is None: + raise ValueError("UE invalide") + matiere = ue.matieres.first() + if matiere: + tf[2]["matiere_id"] = matiere.id + else: + matiere_id = sco_edit_matiere.do_matiere_create( + {"ue_id": ue.id, "titre": ue.titre, "numero": 1}, + ) + tf[2]["matiere_id"] = matiere_id + + tf[2]["semestre_id"] = ue.semestre_idx + module_id = do_module_create(tf[2]) + module = Module.query.get(module_id) + else: # EDITION MODULE + # l'UE de rattachement peut changer + tf[2]["ue_id"], tf[2]["matiere_id"] = tf[2]["ue_matiere_id"].split("!") + x, y = tf[2]["ue_matiere_id"].split("!") + tf[2]["ue_id"] = int(x) + tf[2]["matiere_id"] = int(y) + old_ue_id = module.ue.id + new_ue_id = tf[2]["ue_id"] + if (old_ue_id != new_ue_id) and in_use: + new_ue = UniteEns.query.get_or_404(new_ue_id) + if new_ue.semestre_idx != module.ue.semestre_idx: + # pas changer de semestre un module utilisé ! + raise ScoValueError( + "Module utilisé: il ne peut pas être changé de semestre !" + ) + # En APC, force le semestre égal à celui de l'UE + if is_apc: + selected_ue = UniteEns.query.get(tf[2]["ue_id"]) + if selected_ue is None: + raise ValueError("UE invalide") + tf[2]["semestre_id"] = selected_ue.semestre_idx + # Check unicité code module dans la formation + # ??? TODO + # + do_module_edit(tf[2]) + # Modifie les parcours + if ("parcours" in tf[2]) and formation.referentiel_competence: + if "-1" in tf[2]["parcours"]: # "tous" + module.parcours = formation.referentiel_competence.parcours.all() + else: + module.parcours = [ + ApcParcours.query.get(int(parcour_id_str)) + for parcour_id_str in tf[2]["parcours"] + ] + # Modifie les AC + if "app_critiques" in tf[2]: + module.app_critiques = [ + ApcAppCritique.query.get(int(ac_id_str)) + for ac_id_str in tf[2]["app_critiques"] + ] + db.session.add(module) + db.session.commit() return flask.redirect( url_for( "notes.ue_table", scodoc_dept=g.scodoc_dept, - formation_id=formation_id, - semestre_idx=tf[2]["semestre_id"], + formation_id=formation.id, + semestre_idx=tf[2]["semestre_id"] if is_apc else 1, ) ) @@ -846,13 +915,23 @@ def module_count_moduleimpls(module_id): return len(mods) -def formation_add_malus_modules(formation_id, titre=None, redirect=True): +def formation_add_malus_modules( + formation_id: int, semestre_id: int = None, titre=None, redirect=True +): """Création d'un module de "malus" dans chaque UE d'une formation""" formation = Formation.query.get_or_404(formation_id) - for ue in formation.ues: - ue_add_malus_module(ue, titre=titre) + nb = 0 + ues = formation.ues + if semestre_id is not None: + ues = ues.filter_by(semestre_idx=semestre_id) + for ue in ues: + if ue.type == sco_codes_parcours.UE_STANDARD: + if ue_add_malus_module(ue, titre=titre) != None: + nb += 1 + + flash(f"Modules de malus ajoutés dans {nb} UEs du S{semestre_id}") formation.invalidate_cached_sems() @@ -871,9 +950,9 @@ def ue_add_malus_module(ue: UniteEns, titre=None, code=None) -> int: """ modules_malus = [m for m in ue.modules if m.module_type == scu.ModuleType.MALUS] if len(modules_malus) > 0: - return modules_malus[0].id # déjà existant + return None # déjà existant - titre = titre or "" + titre = titre or f"Malus {ue.acronyme}" code = code or f"MALUS{ue.numero}" # Tout module doit avoir un semestre_id (indice 1, 2, ...) @@ -885,7 +964,7 @@ def ue_add_malus_module(ue: UniteEns, titre=None, code=None) -> int: # c'est ennuyeux: dans ce cas, on pourrait demander à indiquer explicitement # le semestre ? ou affecter le malus au semestre 1 ??? raise ScoValueError( - "Impossible d'ajouter un malus s'il n'y a pas d'autres modules" + "Impossible d'ajouter un malus si l'UE n'a pas de numéro de semestre et ne comporte pas d'autres modules" ) else: semestre_id = ue.semestre_idx @@ -899,7 +978,7 @@ def ue_add_malus_module(ue: UniteEns, titre=None, code=None) -> int: matiere = matieres_malus[0] else: if ue.matieres.count() > 0: - numero = max([mat.numero for mat in ue.matieres]) + 10 + numero = max([(mat.numero or 0) for mat in ue.matieres]) + 10 else: numero = 0 matiere = Matiere(ue_id=ue.id, titre=titre_matiere_malus, numero=numero) diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py index 33ffc69cf..def072c3c 100644 --- a/app/scodoc/sco_edit_ue.py +++ b/app/scodoc/sco_edit_ue.py @@ -35,6 +35,7 @@ from flask_login import current_user from app import db from app import log +from app.but import apc_edit_ue from app.models import APO_CODE_STR_LEN, SHORT_STR_LEN from app.models import Formation, UniteEns, ModuleImpl, Module from app.models import ScolarNews @@ -51,7 +52,6 @@ from app.scodoc.sco_exceptions import ( ) from app.scodoc import html_sco_header -from app.scodoc import sco_cache from app.scodoc import sco_codes_parcours from app.scodoc import sco_edit_apc from app.scodoc import sco_edit_matiere @@ -77,6 +77,7 @@ _ueEditor = ndb.EditableTable( "is_external", "code_apogee", "coefficient", + "coef_rcue", "color", ), sortkey="numero", @@ -121,12 +122,7 @@ def do_ue_create(args): # create ue_id = _ueEditor.create(cnx, args) - # Invalidate cache: vire les poids de toutes les évals de la formation - for modimpl in ModuleImpl.query.filter( - ModuleImpl.module_id == Module.id, Module.formation_id == args["formation_id"] - ): - modimpl.invalidate_evaluations_poids() - formation = Formation.query.get(args["formation_id"]) + formation: Formation = Formation.query.get(args["formation_id"]) formation.invalidate_module_coefs() # news ue = UniteEns.query.get(ue_id) @@ -144,11 +140,10 @@ def do_ue_create(args): def do_ue_delete(ue_id, delete_validations=False, force=False): "delete UE and attached matieres (but not modules)" - from app.scodoc import sco_formations from app.scodoc import sco_parcours_dut ue = UniteEns.query.get_or_404(ue_id) - formation_id = ue.formation_id + formation = ue.formation semestre_idx = ue.semestre_idx if not ue.can_be_deleted(): raise ScoNonEmptyFormationObject( @@ -157,7 +152,7 @@ def do_ue_delete(ue_id, delete_validations=False, force=False): dest_url=url_for( "notes.ue_table", scodoc_dept=g.scodoc_dept, - formation_id=formation_id, + formation_id=formation.id, semestre_idx=semestre_idx, ), ) @@ -181,7 +176,7 @@ def do_ue_delete(ue_id, delete_validations=False, force=False): cancel_url=url_for( "notes.ue_table", scodoc_dept=g.scodoc_dept, - formation_id=formation_id, + formation_id=formation.id, semestre_idx=semestre_idx, ), parameters={"ue_id": ue.id, "dialog_confirmed": 1}, @@ -192,7 +187,11 @@ def do_ue_delete(ue_id, delete_validations=False, force=False): "DELETE FROM scolar_formsemestre_validation WHERE ue_id=%(ue_id)s", {"ue_id": ue.id}, ) - + # delete old formulas + ndb.SimpleQuery( + "DELETE FROM notes_formsemestre_ue_computation_expr WHERE ue_id=%(ue_id)s", + {"ue_id": ue.id}, + ) # delete all matiere in this UE mats = sco_edit_matiere.matiere_list({"ue_id": ue.id}) for mat in mats: @@ -207,13 +206,13 @@ def do_ue_delete(ue_id, delete_validations=False, force=False): _ueEditor.delete(cnx, ue.id) # > UE delete + supr. validations associées etudiants (cas compliqué, mais rarement # utilisé: acceptable de tout invalider): - sco_cache.invalidate_formsemestre() + formation.invalidate_module_coefs() + # -> invalide aussi .invalidate_formsemestre() # news - F = sco_formations.formation_list(args={"formation_id": formation_id})[0] ScolarNews.add( typ=ScolarNews.NEWS_FORM, - obj=formation_id, - text=f"Modification de la formation {F['acronyme']}", + obj=formation.id, + text=f"Modification de la formation {formation.acronyme}", max_frequency=10 * 60, ) # @@ -222,7 +221,7 @@ def do_ue_delete(ue_id, delete_validations=False, force=False): url_for( "notes.ue_table", scodoc_dept=g.scodoc_dept, - formation_id=formation_id, + formation_id=formation.id, semestre_idx=semestre_idx, ) ) @@ -248,13 +247,16 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No title = f"Modification de l'UE {ue.acronyme} {ue.titre}" initvalues = ue_dict submitlabel = "Modifier les valeurs" - can_change_semestre_id = (ue.modules.count() == 0) or (ue.semestre_idx is None) + can_change_semestre_id = ( + (ue.modules.count() == 0) or (ue.semestre_idx is None) + ) and ue.niveau_competence is None else: ue = None title = "Création d'une UE" initvalues = { "semestre_idx": default_semestre_idx, "color": ue_guess_color_default(formation_id, default_semestre_idx), + "coef_rcue": 1.0, } submitlabel = "Créer cette UE" can_change_semestre_id = True @@ -277,6 +279,11 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No

Note: sauf exception, l'UE n'a pas de coefficient associé. Seuls les modules ont des coefficients.

""", + f""" +

UE du semestre S{ue.semestre_idx}

+ """ + if is_apc and ue + else "", ] ue_types = parcours.ALLOWED_UE_TYPES @@ -308,8 +315,7 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No "type": "int", "allow_null": False, "title": parcours.SESSION_NAME.capitalize(), - "explanation": "%s de l'UE dans la formation" - % parcours.SESSION_NAME, + "explanation": f"{parcours.SESSION_NAME} de l'UE dans la formation", "labels": ["non spécifié"] + [str(x) for x in semestres_indices], "allowed_values": [""] + semestres_indices, }, @@ -339,22 +345,43 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No "allow_null": not is_apc, # ects requis en APC }, ), - ( - "coefficient", - { - "size": 4, - "type": "float", - "title": "Coefficient", - "explanation": """les coefficients d'UE ne sont utilisés que + ] + if is_apc: # coef pour la moyenne RCUE + form_descr.append( + ( + "coef_rcue", + { + "size": 4, + "type": "float", + "title": "Coef. RCUE", + "explanation": """pondération utilisée pour le calcul de la moyenne du RCUE. Laisser à 1, sauf si votre établissement a explicitement décidé de pondérations. + """, + "defaut": 1.0, + "allow_null": False, + "enabled": is_apc, + }, + ) + ) + else: # non APC, coef d'UE + form_descr.append( + ( + "coefficient", + { + "size": 4, + "type": "float", + "title": "Coefficient", + "explanation": """les coefficients d'UE ne sont utilisés que lorsque l'option Utiliser les coefficients d'UE pour calculer la moyenne générale est activée. Par défaut, le coefficient d'une UE est simplement la somme des coefficients des modules dans lesquels l'étudiant a des notes. Jamais utilisé en BUT. """, - "enabled": not is_apc, - }, - ), + "enabled": not is_apc, + }, + ) + ) + form_descr += [ ( "ue_code", { @@ -410,8 +437,12 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No form_descr, initvalues=initvalues, submitlabel=submitlabel, + cancelbutton="Revenir à la formation", ) if tf[0] == 0: + niveau_competence_div = "" + if ue and is_apc: + niveau_competence_div = apc_edit_ue.form_ue_choix_niveau(formation, ue) if ue and ue.modules.count() and ue.semestre_idx is not None: modules_div = f"""
{ue.modules.count()} modules sont rattachés @@ -420,7 +451,7 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
    """ for m in ue.modules: modules_div += f"""
  • {m.code} {m.titre}
  • """ + "notes.module_edit",scodoc_dept=g.scodoc_dept, module_id=m.id)}">{m.code} {m.titre or "sans titre"}""" modules_div += """
""" else: modules_div = "" @@ -429,12 +460,13 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No return ( "\n".join(H) + tf[1] + + niveau_competence_div + modules_div + bonus_div + ue_div + html_sco_header.sco_footer() ) - else: + elif tf[2]: if create: if not tf[2]["ue_code"]: del tf[2]["ue_code"] @@ -467,14 +499,26 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No else: do_ue_edit(tf[2]) flash("UE modifiée") - return flask.redirect( - url_for( - "notes.ue_table", - scodoc_dept=g.scodoc_dept, - formation_id=formation_id, - semestre_idx=tf[2]["semestre_idx"], - ) + + if tf[2]: + dest_semestre_idx = tf[2]["semestre_idx"] + elif ue: + dest_semestre_idx = ue.semestre_idx + elif default_semestre_idx: + dest_semestre_idx = default_semestre_idx + elif "semestre_idx" in request.form: + dest_semestre_idx = request.form["semestre_idx"] + else: + dest_semestre_idx = 1 + + return flask.redirect( + url_for( + "notes.ue_table", + scodoc_dept=g.scodoc_dept, + formation_id=formation_id, + semestre_idx=dest_semestre_idx, ) + ) def _add_ue_semestre_id(ues: list[dict], is_apc): @@ -646,9 +690,7 @@ def ue_table(formation_id=None, semestre_idx=1, msg=""): # was ue_list ], page_title=f"Programme {formation.acronyme}", ), - f"""

Formation {formation.titre} ({formation.acronyme}) - [version {formation.version}] code {formation.formation_code} - {lockicon} + f"""

{formation.to_html()} {lockicon}

""", ] @@ -711,7 +753,8 @@ du programme" (menu "Semestre") si vous avez un semestre en cours); else: descr_refcomp = f"""Référentiel de compétences: + scodoc_dept=g.scodoc_dept, refcomp_id=formation.referentiel_competence.id)}" + class="stdlink"> {formation.referentiel_competence.type_titre} {formation.referentiel_competence.specialite_long}  """ msg_refcomp = "changer" @@ -727,7 +770,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours); f"""
  • éditer les coefficients des ressources et SAÉs + }">Éditer les coefficients des ressources et SAÉs
  • """ @@ -816,6 +859,7 @@ du programme" (menu "Semestre") si vous avez un semestre en cours); ) }">Créer une nouvelle version (non verrouillée) + """ ) H.append( @@ -914,7 +958,7 @@ def _html_select_semestre_idx(formation_id, semestre_ids, semestre_idx): def _ue_table_ues( parcours, - ues, + ues: list[dict], editable, tag_editable, has_perm_change, @@ -923,7 +967,7 @@ def _ue_table_ues( arrow_none, delete_icon, delete_disabled_icon, -): +) -> str: """Édition de programme: liste des UEs (avec leurs matières et modules). Pour les formations classiques (non APC/BUT) """ @@ -951,9 +995,9 @@ def _ue_table_ues( if ue["semestre_id"] == sco_codes_parcours.UE_SEM_DEFAULT: lab = "Pas d'indication de semestre:" else: - lab = "Semestre %s:" % ue["semestre_id"] + lab = f"""Semestre {ue["semestre_id"]}:""" H.append( - '
    %s
    ' % lab + f'
    {lab}
    ' ) H.append('
      ') H.append('
    • ') @@ -1304,8 +1348,9 @@ def do_ue_edit(args, bypass_lock=False, dont_invalidate_cache=False): formation = Formation.query.get(ue["formation_id"]) if not dont_invalidate_cache: - # Invalide les semestres utilisant cette formation: - formation.invalidate_cached_sems() + # Invalide les semestres utilisant cette formation + # ainsi que les poids et coefs + formation.invalidate_module_coefs() # essai edition en ligne: diff --git a/app/scodoc/sco_excel.py b/app/scodoc/sco_excel.py index 9ba10a304..b21652093 100644 --- a/app/scodoc/sco_excel.py +++ b/app/scodoc/sco_excel.py @@ -59,7 +59,7 @@ class COLORS(Enum): LIGHT_YELLOW = "FFFFFF99" -# Un style est enregistré comme un dictionnaire qui précise la valeur d'un attributdans la liste suivante: +# Un style est enregistré comme un dictionnaire qui précise la valeur d'un attribut dans la liste suivante: # font, border, number_format, fill,... # (cf https://openpyxl.readthedocs.io/en/stable/styles.html#working-with-styles) @@ -288,7 +288,7 @@ class ScoExcelSheet: value -- contenu de la cellule (texte, numérique, booléen ou date) style -- style par défaut (dictionnaire cf. excel_make_style) de la feuille si non spécifié """ - # adapatation des valeurs si nécessaire + # adaptation des valeurs si nécessaire if value is None: value = "" elif value is True: diff --git a/app/scodoc/sco_formsemestre.py b/app/scodoc/sco_formsemestre.py index 11792057c..32c2d9e34 100644 --- a/app/scodoc/sco_formsemestre.py +++ b/app/scodoc/sco_formsemestre.py @@ -141,11 +141,18 @@ def do_formsemestre_list(*a, **kw): def _formsemestre_enrich(sem): - """Ajoute champs souvent utiles: titre + annee et dateord (pour tris)""" + """Ajoute champs souvent utiles: titre + annee et dateord (pour tris). + XXX obsolete: préférer formsemestre.to_dict() ou, mieux, les méthodes de FormSemestre. + """ # imports ici pour eviter refs circulaires from app.scodoc import sco_formsemestre_edit - F = sco_formations.formation_list(args={"formation_id": sem["formation_id"]})[0] + formations = sco_formations.formation_list( + args={"formation_id": sem["formation_id"]} + ) + if not formations: + raise ScoValueError("pas de formation pour ce semestre !") + F = formations[0] parcours = sco_codes_parcours.get_parcours_from_code(F["type_parcours"]) # 'S1', 'S2', ... ou '' pour les monosemestres if sem["semestre_id"] != NO_SEMESTRE_ID: diff --git a/app/scodoc/sco_formsemestre_edit.py b/app/scodoc/sco_formsemestre_edit.py index 3ccba7728..c4e329462 100644 --- a/app/scodoc/sco_formsemestre_edit.py +++ b/app/scodoc/sco_formsemestre_edit.py @@ -39,23 +39,21 @@ from app.models import Module, ModuleImpl, Evaluation, EvaluationUEPoids, UniteE from app.models import ScolarNews from app.models.formations import Formation from app.models.formsemestre import FormSemestre +from app.models.but_refcomp import ApcParcours import app.scodoc.notesdb as ndb import app.scodoc.sco_utils as scu from app.scodoc import sco_cache from app.scodoc import sco_groups from app import log -from app.scodoc.TrivialFormulator import TrivialFormulator, TF +from app.scodoc.TrivialFormulator import TrivialFormulator from app.scodoc.sco_exceptions import AccessDenied, ScoValueError from app.scodoc.sco_permissions import Permission from app.scodoc.sco_vdi import ApoEtapeVDI from app.scodoc import html_sco_header from app.scodoc import sco_codes_parcours from app.scodoc import sco_compute_moy -from app.scodoc import sco_edit_matiere from app.scodoc import sco_edit_module -from app.scodoc import sco_edit_ue from app.scodoc import sco_etud -from app.scodoc import sco_evaluations from app.scodoc import sco_evaluation_db from app.scodoc import sco_formations from app.scodoc import sco_formsemestre @@ -119,12 +117,12 @@ def formsemestre_editwithmodules(formsemestre_id): vals = scu.get_request_args() if not vals.get("tf_submitted", False): H.append( - """

      Seuls les modules cochés font partie de ce semestre. + """

      Seuls les modules cochés font partie de ce semestre. Pour les retirer, les décocher et appuyer sur le bouton "modifier".

      -

      Attention : s'il y a déjà des évaluations dans un module, +

      Attention : s'il y a déjà des évaluations dans un module, il ne peut pas être supprimé !

      -

      Les modules ont toujours un responsable. +

      Les modules ont toujours un responsable. Par défaut, c'est le directeur des études.

      Un semestre ne peut comporter qu'une seule UE "bonus sport/culture"

      @@ -153,7 +151,7 @@ def do_formsemestre_createwithmodules(edit=False): formsemestre = FormSemestre.query.get_or_404(formsemestre_id) if not current_user.has_permission(Permission.ScoImplement): if not edit: - # il faut ScoImplement pour creer un semestre + # il faut ScoImplement pour créer un semestre raise AccessDenied("vous n'avez pas le droit d'effectuer cette opération") else: if not sem["resp_can_edit"] or current_user.id not in sem["responsables"]: @@ -175,6 +173,7 @@ def do_formsemestre_createwithmodules(edit=False): formation = Formation.query.get(formation_id) if formation is None: raise ScoValueError("Formation inexistante !") + is_apc = formation.is_apc() if not edit: initvalues = {"titre": _default_sem_title(formation)} semestre_id = int(vals["semestre_id"]) @@ -210,12 +209,12 @@ def do_formsemestre_createwithmodules(edit=False): if NB_SEM == 1: semestre_id_list = [-1] else: - if edit and formation.is_apc(): + if edit and is_apc: # en APC, ne permet pas de changer de semestre semestre_id_list = [formsemestre.semestre_id] else: semestre_id_list = list(range(1, NB_SEM + 1)) - if not formation.is_apc(): + if not is_apc: # propose "pas de semestre" seulement en classique semestre_id_list.insert(0, -1) @@ -226,7 +225,7 @@ def do_formsemestre_createwithmodules(edit=False): else: semestre_id_labels.append(f"S{sid}") # Liste des modules dans cette formation - if formation.is_apc(): + if is_apc: modules = formation.modules.order_by(Module.module_type, Module.numero) else: modules = ( @@ -318,10 +317,10 @@ def do_formsemestre_createwithmodules(edit=False): { "size": 40, "title": "Nom de ce semestre", - "explanation": """n'indiquez pas les dates, ni le semestre, ni la modalité dans + "explanation": f"""n'indiquez pas les dates, ni le semestre, ni la modalité dans le titre: ils seront automatiquement ajoutés """ - % _default_sem_title(formation), + value="remettre titre par défaut" onClick="document.tf.titre.value='{ + _default_sem_title(formation)}';"/>""", }, ), ( @@ -343,11 +342,9 @@ def do_formsemestre_createwithmodules(edit=False): "allowed_values": semestre_id_list, "labels": semestre_id_labels, "explanation": "en BUT, on ne peut pas modifier le semestre après création" - if formation.is_apc() - else "", - "attributes": ['onchange="change_semestre_id();"'] - if formation.is_apc() + if is_apc else "", + "attributes": ['onchange="change_semestre_id();"'] if is_apc else "", }, ), ) @@ -386,7 +383,7 @@ def do_formsemestre_createwithmodules(edit=False): mf = mf_manual for n in range(1, scu.EDIT_NB_ETAPES + 1): - mf["title"] = "Etape Apogée (%d)" % n + mf["title"] = f"Etape Apogée ({n})" modform.append(("etape_apo" + str(n), mf.copy())) modform.append( ( @@ -443,15 +440,19 @@ def do_formsemestre_createwithmodules(edit=False): ) ) if edit: - formtit = ( - """ -

      Modifier les coefficients des UE capitalisées

      -

      Sélectionner les modules, leurs responsables et les étudiants à inscrire:

      + formtit = f""" +

      Modifier les coefficients des UE capitalisées

      +

      Sélectionner les modules, leurs responsables et les étudiants + à inscrire:

      """ - % formsemestre_id - ) else: - formtit = """

      Sélectionner les modules et leurs responsables

      Si vous avez des parcours (options), ne sélectionnez que les modules du tronc commun.

      """ + formtit = """

      Sélectionner les modules et leurs responsables

      +

      Si vous avez des parcours (options), dans un premier + ne sélectionnez que les modules du tronc commun, puis après inscriptions, + revenez ajouter les modules de parcours en sélectionnant les groupes d'étudiants + à y inscrire. +

      """ modform += [ ( @@ -531,12 +532,53 @@ def do_formsemestre_createwithmodules(edit=False): "explanation": "empêcher le calcul des moyennes d'UE et générale.", }, ), + ] + # Choix des parcours + if is_apc: + ref_comp = formation.referentiel_competence + if ref_comp: + modform += [ + ( + "parcours", + { + "input_type": "checkbox", + "vertical": True, + "dom_id": "tf_module_parcours", + "labels": [parcour.libelle for parcour in ref_comp.parcours], + "allowed_values": [ + str(parcour.id) for parcour in ref_comp.parcours + ], + "explanation": """Parcours proposés dans ce semestre. + S'il s'agit d'un semestre de "tronc commun", ne pas indiquer de parcours.""", + }, + ) + ] + if edit: + sem["parcours"] = [str(parcour.id) for parcour in formsemestre.parcours] + else: + modform += [ + ( + "parcours", + { + "input_type": "separator", + "title": f"""{scu.EMO_WARNING } + Pas de parcours: + vérifier la formation + """, + }, + ) + ] + + # Choix des modules + modform += [ ( "sep", { "input_type": "separator", "title": "", - "template": "
    %s" % formtit, + "template": f"
    {formtit}", }, ), ] @@ -544,8 +586,8 @@ def do_formsemestre_createwithmodules(edit=False): nbmod = 0 for semestre_id in semestre_ids: - if formation.is_apc(): - # pour restreindre l'édition aux module du semestre sélectionné + if is_apc: + # pour restreindre l'édition aux modules du semestre sélectionné tr_class = f'class="sem{semestre_id}"' else: tr_class = "" @@ -560,7 +602,7 @@ def do_formsemestre_createwithmodules(edit=False): "sep", { "input_type": "separator", - "title": "Semestre %s" % semestre_id, + "title": f"Semestre {semestre_id}", "template": templ_sep, }, ) @@ -568,13 +610,13 @@ def do_formsemestre_createwithmodules(edit=False): for mod in mods: if mod["semestre_id"] == semestre_id and ( (not edit) # creation => tous modules - or (not formation.is_apc()) # pas BUT, on peut mixer les semestres + or (not is_apc) # pas BUT, on peut mixer les semestres or (semestre_id == formsemestre.semestre_id) # module du semestre or (mod["module_id"] in module_ids_set) # module déjà présent ): nbmod += 1 if edit: - select_name = "%s!group_id" % mod["module_id"] + select_name = f"{mod['module_id']}!group_id" def opt_selected(gid): if gid == vals.get(select_name): @@ -603,13 +645,16 @@ def do_formsemestre_createwithmodules(edit=False): group["group_name"], ) fcg += "" - itemtemplate = ( - f"""" - ) + itemtemplate = f""" + + + + """ else: - itemtemplate = f"""""" + itemtemplate = f""" + + + """ modform.append( ( "MI" + str(mod["module_id"]), @@ -742,7 +787,8 @@ def do_formsemestre_createwithmodules(edit=False): for module_id in tf[2]["tf-checked"]: mod_resp_id = User.get_user_id_from_nomplogin(tf[2][module_id]) if mod_resp_id is None: - # Si un module n'a pas de responsable (ou inconnu), l'affecte au 1er directeur des etudes: + # Si un module n'a pas de responsable (ou inconnu), + # l'affecte au 1er directeur des etudes: mod_resp_id = tf[2]["responsable_id"] tf[2][module_id] = mod_resp_id @@ -763,7 +809,7 @@ def do_formsemestre_createwithmodules(edit=False): module_ids_checked = [int(x[2:]) for x in tf[2]["tf-checked"]] _formsemestre_check_ue_bonus_unicity(module_ids_checked) if not edit: - if formation.is_apc(): + if is_apc: _formsemestre_check_module_list( module_ids_checked, tf[2]["semestre_id"] ) @@ -777,14 +823,6 @@ def do_formsemestre_createwithmodules(edit=False): "responsable_id": tf[2][f"MI{module_id}"], } _ = sco_moduleimpl.do_moduleimpl_create(modargs) - flash("Nouveau semestre créé") - return flask.redirect( - url_for( - "notes.formsemestre_status", - scodoc_dept=g.scodoc_dept, - formsemestre_id=formsemestre_id, - ) - ) else: # Modification du semestre: # on doit creer les modules nouvellement selectionnés @@ -794,7 +832,7 @@ def do_formsemestre_createwithmodules(edit=False): module_ids_tocreate = [ x for x in module_ids_checked if not x in module_ids_existing ] - if formation.is_apc(): + if is_apc: _formsemestre_check_module_list( module_ids_tocreate, tf[2]["semestre_id"] ) @@ -868,27 +906,48 @@ def do_formsemestre_createwithmodules(edit=False): modargs, formsemestre_id=formsemestre_id ) mod = sco_edit_module.module_list({"module_id": module_id})[0] - - if msg: - msg_html = ( - '
    Attention !
    • ' - + "
    • ".join(msg) - + "
    " - ) - if ok: - msg_html += "

    Modification effectuée

    " - else: - msg_html += "

    Modification effectuée (mais modules cités non supprimés)

    " - msg_html += ( - 'retour au tableau de bord' - % formsemestre_id - ) - return msg_html + # --- Association des parcours + formsemestre = FormSemestre.query.get(formsemestre_id) + if "parcours" in tf[2]: + formsemestre.parcours = [ + ApcParcours.query.get(int(parcour_id_str)) + for parcour_id_str in tf[2]["parcours"] + ] + db.session.add(formsemestre) + db.session.commit() + # --- Crée ou met à jour les groupes de parcours BUT + formsemestre.setup_parcours_groups() + # --- Fin + if edit: + if msg: + msg_html = ( + '
    Attention !
    • ' + + "
    • ".join(msg) + + "
    " + ) + if ok: + msg_html += "

    Modification effectuée

    " else: - return flask.redirect( - "formsemestre_status?formsemestre_id=%s&head_message=Semestre modifié" - % formsemestre_id - ) + msg_html += "

    Modification effectuée (mais modules cités non supprimés)

    " + msg_html += ( + 'retour au tableau de bord' + % formsemestre_id + ) + return msg_html + else: + return flask.redirect( + "formsemestre_status?formsemestre_id=%s&head_message=Semestre modifié" + % formsemestre_id + ) + else: + flash("Nouveau semestre créé") + return flask.redirect( + url_for( + "notes.formsemestre_status", + scodoc_dept=g.scodoc_dept, + formsemestre_id=formsemestre_id, + ) + ) def _formsemestre_check_module_list(module_ids, semestre_idx): diff --git a/app/scodoc/sco_formsemestre_inscriptions.py b/app/scodoc/sco_formsemestre_inscriptions.py index b3fe98532..3f95add7c 100644 --- a/app/scodoc/sco_formsemestre_inscriptions.py +++ b/app/scodoc/sco_formsemestre_inscriptions.py @@ -35,6 +35,7 @@ from flask import url_for, g, request from app.comp import res_sem from app.comp.res_compat import NotesTableCompat from app.models import FormSemestre +from app.models.groups import GroupDescr, Partition import app.scodoc.sco_utils as scu from app import log from app.scodoc.scolog import logdb @@ -257,14 +258,14 @@ def do_formsemestre_inscription_with_modules( """Inscrit cet etudiant à ce semestre et TOUS ses modules STANDARDS (donc sauf le sport) """ + formsemestre = FormSemestre.query.get_or_404(formsemestre_id) # inscription au semestre args = {"formsemestre_id": formsemestre_id, "etudid": etudid} if etat is not None: args["etat"] = etat do_formsemestre_inscription_create(args, method=method) log( - "do_formsemestre_inscription_with_modules: etudid=%s formsemestre_id=%s" - % (etudid, formsemestre_id) + f"do_formsemestre_inscription_with_modules: etudid={etudid} formsemestre_id={formsemestre_id}" ) # inscriptions aux groupes # 1- inscrit au groupe 'tous' @@ -275,10 +276,16 @@ def do_formsemestre_inscription_with_modules( # 2- inscrit aux groupes for group_id in group_ids: if group_id and not group_id in gdone: - sco_groups.set_group(etudid, group_id) - gdone[group_id] = 1 + group = GroupDescr.query.get_or_404(group_id) + if group.partition.groups_editable: + sco_groups.set_group(etudid, group_id) + gdone[group_id] = 1 + else: + log( + f"do_formsemestre_inscription_with_modules: group {group:r} belongs to non editable partition" + ) - # inscription a tous les modules de ce semestre + # Inscription à tous les modules de ce semestre modimpls = sco_moduleimpl.moduleimpl_withmodule_list( formsemestre_id=formsemestre_id ) @@ -288,6 +295,8 @@ def do_formsemestre_inscription_with_modules( {"moduleimpl_id": mod["moduleimpl_id"], "etudid": etudid}, formsemestre_id=formsemestre_id, ) + # Mise à jour des inscriptions aux parcours: + formsemestre.update_inscriptions_parcours_from_groups() def formsemestre_inscription_with_modules_etud( @@ -534,11 +543,14 @@ def formsemestre_inscription_option(etudid, formsemestre_id): ue_status = nt.get_etud_ue_status(etudid, ue_id) if ue_status and ue_status["is_capitalized"]: sem_origin = sco_formsemestre.get_formsemestre(ue_status["formsemestre_id"]) - ue_descr += ' (capitalisée le %s)' % ( - sem_origin["formsemestre_id"], - etudid, - sem_origin["titreannee"], - ndb.DateISOtoDMY(ue_status["event_date"]), + ue_descr += ( + ' (capitalisée le %s)' + % ( + sem_origin["formsemestre_id"], + etudid, + sem_origin["titreannee"], + ndb.DateISOtoDMY(ue_status["event_date"]), + ) ) descr.append( ( diff --git a/app/scodoc/sco_formsemestre_status.py b/app/scodoc/sco_formsemestre_status.py index 299318ed7..ec4fb6dac 100644 --- a/app/scodoc/sco_formsemestre_status.py +++ b/app/scodoc/sco_formsemestre_status.py @@ -40,6 +40,7 @@ from app.comp import res_sem from app.comp.res_compat import NotesTableCompat from app.models import Module from app.models.formsemestre import FormSemestre +from app.models.moduleimpls import ModuleImpl import app.scodoc.sco_utils as scu from app.scodoc.sco_utils import ModuleType import app.scodoc.notesdb as ndb @@ -148,7 +149,10 @@ def formsemestre_status_menubar(sem): { "title": "Voir la formation %(acronyme)s (v%(version)s)" % F, "endpoint": "notes.ue_table", - "args": {"formation_id": sem["formation_id"]}, + "args": { + "formation_id": sem["formation_id"], + "semestre_idx": sem["semestre_id"], + }, "enabled": True, "helpmsg": "Tableau de bord du semestre", }, @@ -325,7 +329,7 @@ def formsemestre_status_menubar(sem): }, { "title": "Créer/modifier les partitions...", - "endpoint": "scolar.editPartitionForm", + "endpoint": "scolar.edit_partition_form", "args": {"formsemestre_id": formsemestre_id}, "enabled": sco_groups.sco_permissions_check.can_change_groups( formsemestre_id @@ -345,7 +349,7 @@ def formsemestre_status_menubar(sem): "title": "%s" % partition["partition_name"], "endpoint": "scolar.affect_groups", "args": {"partition_id": partition["partition_id"]}, - "enabled": enabled, + "enabled": enabled and partition["groups_editable"], } ) menuGroupes.append( @@ -406,10 +410,9 @@ def formsemestre_status_menubar(sem): }, { "title": "Saisie des décisions du jury", - "endpoint": "notes.formsemestre_recapcomplet", + "endpoint": "notes.formsemestre_saisie_jury", "args": { "formsemestre_id": formsemestre_id, - "modejury": 1, }, "enabled": sco_permissions_check.can_validate_sem(formsemestre_id), }, @@ -499,20 +502,24 @@ def retreive_formsemestre_from_request() -> int: # Element HTML decrivant un semestre (barre de menu et infos) -def formsemestre_page_title(): +def formsemestre_page_title(formsemestre_id=None): """Element HTML decrivant un semestre (barre de menu et infos) Cherche dans la requete si un semestre est défini (formsemestre_id ou moduleimpl ou evaluation ou group) """ - formsemestre_id = retreive_formsemestre_from_request() + formsemestre_id = ( + formsemestre_id + if formsemestre_id is not None + else retreive_formsemestre_from_request() + ) # if not formsemestre_id: return "" try: formsemestre_id = int(formsemestre_id) - formsemestre = FormSemestre.query.get(formsemestre_id) - except: - log("can't find formsemestre_id %s" % formsemestre_id) + except ValueError: + log(f"formsemestre_id: invalid type {formsemestre_id:r}") return "" + formsemestre = FormSemestre.query.get_or_404(formsemestre_id) h = render_template( "formsemestre_page_title.html", @@ -578,7 +585,9 @@ def fill_formsemestre(sem): # Description du semestre sous forme de table exportable -def formsemestre_description_table(formsemestre_id, with_evals=False): +def formsemestre_description_table( + formsemestre_id, with_evals=False, with_parcours=False +): """Description du semestre sous forme de table exportable Liste des modules et de leurs coefficients """ @@ -618,7 +627,7 @@ def formsemestre_description_table(formsemestre_id, with_evals=False): ue_info["Coef._class"] = "ue_coef" R.append(ue_info) - ModInscrits = sco_moduleimpl.do_moduleimpl_inscription_list( + mod_inscrits = sco_moduleimpl.do_moduleimpl_inscription_list( moduleimpl_id=M["moduleimpl_id"] ) enseignants = ", ".join( @@ -629,7 +638,7 @@ def formsemestre_description_table(formsemestre_id, with_evals=False): "Code": M["module"]["code"] or "", "Module": M["module"]["abbrev"] or M["module"]["titre"], "_Module_class": "scotext", - "Inscrits": len(ModInscrits), + "Inscrits": len(mod_inscrits), "Responsable": sco_users.user_info(M["responsable_id"])["nomprenom"], "_Responsable_class": "scotext", "Enseignants": enseignants, @@ -648,10 +657,15 @@ def formsemestre_description_table(formsemestre_id, with_evals=False): moduleimpl_id=M["moduleimpl_id"], ), } - R.append(l) if M["module"]["coefficient"]: sum_coef += M["module"]["coefficient"] + if with_parcours: + module = Module.query.get(M["module_id"]) + l["parcours"] = ", ".join(sorted([pa.code for pa in module.parcours])) + + R.append(l) + if with_evals: # Ajoute lignes pour evaluations evals = nt.get_mod_evaluation_etat_list(M["moduleimpl_id"]) @@ -676,7 +690,10 @@ def formsemestre_description_table(formsemestre_id, with_evals=False): sums = {"_css_row_class": "moyenne sortbottom", "ects": sum_ects, "Coef.": sum_coef} R.append(sums) - columns_ids = ["UE", "Code", "Module", "Coef."] + columns_ids = ["UE", "Code", "Module"] + if with_parcours: + columns_ids += ["parcours"] + columns_ids += ["Coef."] if sco_preferences.get_preference("bul_show_ects", formsemestre_id): columns_ids += ["ects"] columns_ids += ["Inscrits", "Responsable", "Enseignants"] @@ -696,6 +713,7 @@ def formsemestre_description_table(formsemestre_id, with_evals=False): titles["description"] = "" titles["coefficient"] = "Coef. éval." titles["evalcomplete_str"] = "Complète" + titles["parcours"] = "Parcours" titles["publish_incomplete_str"] = "Toujours Utilisée" title = "%s %s" % (parcours.SESSION_NAME.capitalize(), formsemestre.titre_mois()) @@ -720,21 +738,26 @@ def formsemestre_description_table(formsemestre_id, with_evals=False): ) -def formsemestre_description(formsemestre_id, format="html", with_evals=False): +def formsemestre_description( + formsemestre_id, format="html", with_evals=False, with_parcours=False +): """Description du semestre sous forme de table exportable Liste des modules et de leurs coefficients """ with_evals = int(with_evals) - tab = formsemestre_description_table(formsemestre_id, with_evals=with_evals) - tab.html_before_table = """
    - - + + indiquer les évaluations + indiquer les parcours BUT + """ return tab.make_page(format=format) @@ -854,7 +877,7 @@ def _make_listes_sem(sem, with_absences=True): H.append( f"""

    )""" ) H.append("") + if sem.parcours: + H.append( + f""" +

    + + + """ + ) evals = sco_evaluations.do_evaluation_etat_in_sem(formsemestre_id) H.append( - '' - % (modimpl["moduleimpl_id"], mod_descr, mod.abbrev or mod.titre) + % (modimpl["moduleimpl_id"], mod_descr, mod.abbrev or mod.titre or "") ) H.append('' % len(mod_inscrits)) H.append( diff --git a/app/scodoc/sco_formsemestre_validation.py b/app/scodoc/sco_formsemestre_validation.py index f4896a8dc..2af209c2a 100644 --- a/app/scodoc/sco_formsemestre_validation.py +++ b/app/scodoc/sco_formsemestre_validation.py @@ -111,7 +111,7 @@ def formsemestre_validation_etud_form( url_tableau = url_for( "notes.formsemestre_recapcomplet", scodoc_dept=g.scodoc_dept, - modejury=1, + mode_jury=1, formsemestre_id=formsemestre_id, selected_etudid=etudid, # va a la bonne ligne ) @@ -581,19 +581,27 @@ def formsemestre_recap_parcours_table( else: pm = plusminus % sem["formsemestre_id"] - H.append( - '' - % (bgcolor, num_sem, pm) + inscr = formsemestre.etuds_inscriptions.get(etudid) + parcours_name = ( + f' {inscr.parcour.code}' + if (inscr and inscr.parcour) + else "" ) - H.append('' % sem) H.append( - '' - % (a_url, sem["formsemestre_id"], etudid, sem["titreannee"]) + f""" + + + + """ ) - if decision_sem: + if nt.is_apc: + H.append('') + elif decision_sem: H.append('' % decision_sem["code"]) else: - H.append('') + H.append("") H.append('' % ass) # abs # acronymes UEs auxquelles l'étudiant est inscrit (ou capitalisé) ues = nt.get_ues_stat_dict(filter_sport=True) @@ -973,7 +981,7 @@ def do_formsemestre_validation_auto(formsemestre_id): H.append("") H.append( f"""continuer""" ) H.append(html_sco_header.sco_footer()) diff --git a/app/scodoc/sco_groups.py b/app/scodoc/sco_groups.py index 79ed91cad..f17e017e6 100644 --- a/app/scodoc/sco_groups.py +++ b/app/scodoc/sco_groups.py @@ -76,10 +76,12 @@ partitionEditor = ndb.EditableTable( "numero", "bul_show_rank", "show_in_lists", + "editable", ), input_formators={ "bul_show_rank": bool, "show_in_lists": bool, + "editable": bool, }, ) @@ -105,14 +107,19 @@ def get_group(group_id: int): return r[0] -def group_delete(group, force=False): +def group_delete(group_id: int): """Delete a group.""" # if not group['group_name'] and not force: # raise ValueError('cannot suppress this group') # remove memberships: - ndb.SimpleQuery("DELETE FROM group_membership WHERE group_id=%(group_id)s", group) + ndb.SimpleQuery( + "DELETE FROM group_membership WHERE group_id=%(group_id)s", + {"group_id": group_id}, + ) # delete group: - ndb.SimpleQuery("DELETE FROM group_descr WHERE id=%(group_id)s", group) + ndb.SimpleQuery( + "DELETE FROM group_descr WHERE id=%(group_id)s", {"group_id": group_id} + ) def get_partition(partition_id): @@ -264,6 +271,17 @@ def get_group_members(group_id, etat=None): return r +def check_group_name(group_name, partition, raiser=False): + """If groupe name exists in partition : if raiser -> Raise ScoValueError else-> return true""" + exists = group_name in [g["group_name"] for g in get_partition_groups(partition)] + if exists: + if raiser: + raise ScoValueError("Le nom de groupe existe déjà dans la partition") + else: + return True + return False + + # obsolete: sco_groups_view.DisplayedGroupsInfos # def get_groups_members(group_ids, etat=None): # """Liste les étudiants d'une liste de groupes @@ -621,10 +639,12 @@ def comp_origin(etud, cur_sem): return "" # parcours normal, ne le signale pas -def set_group(etudid, group_id): +def set_group(etudid: int, group_id: int) -> bool: """Inscrit l'étudiant au groupe. Return True if ok, False si deja inscrit. - Warning: don't check if group_id exists (the caller should check). + Warning: + - don't check if group_id exists (the caller should check). + - don't check if group's partition is editable """ cnx = ndb.GetDBConnexion() cursor = cnx.cursor(cursor_factory=ndb.ScoDocCursor) @@ -686,7 +706,12 @@ def change_etud_group_in_partition(etudid, group_id, partition=None): % (formsemestre_id, partition["partition_name"], group["group_name"]), ) cnx.commit() - # 4- invalidate cache + + # 5- Update parcours + formsemestre = FormSemestre.query.get(formsemestre_id) + formsemestre.update_inscriptions_parcours_from_groups() + + # 6- invalidate cache sco_cache.invalidate_formsemestre( formsemestre_id=formsemestre_id ) # > change etud group @@ -698,14 +723,28 @@ def setGroups( groupsToCreate="", # name and members of new groups groupsToDelete="", # groups to delete ): - """Affect groups (Ajax request) + """Affect groups (Ajax request): renvoie du XML groupsLists: lignes de la forme "group_id;etudid;...\n" groupsToCreate: lignes "group_name;etudid;...\n" groupsToDelete: group_id;group_id;... + + Ne peux pas modifier les groupes des partitions non éditables. """ from app.scodoc import sco_formsemestre + def xml_error(msg, code=404): + data = ( + f'Error: {msg}' + ) + response = make_response(data, code) + response.headers["Content-Type"] = scu.XML_MIMETYPE + return response + partition = get_partition(partition_id) + if not partition["groups_editable"]: + msg = "setGroups: partition non editable" + log(msg) + return xml_error(msg, code=403) formsemestre_id = partition["formsemestre_id"] if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") @@ -727,8 +766,8 @@ def setGroups( continue try: group_id = int(group_id) - except ValueError as exc: - log("setGroups: ignoring invalid group_id={group_id}") + except ValueError: + log(f"setGroups: ignoring invalid group_id={group_id}") continue group = get_group(group_id) # Anciens membres du groupe: @@ -778,6 +817,10 @@ def setGroups( for etudid in fs[1:-1]: change_etud_group_in_partition(etudid, group_id, partition) + # Update parcours + formsemestre = FormSemestre.query.get(formsemestre_id) + formsemestre.update_inscriptions_parcours_from_groups() + data = ( 'Groupes enregistrés' ) @@ -798,15 +841,15 @@ def create_group(partition_id, group_name="", default=False) -> int: if not group_name and not default: raise ValueError("invalid group name: ()") # checkGroupName(group_name) - if group_name in [g["group_name"] for g in get_partition_groups(partition)]: - raise ValueError( - "group_name %s already exists in partition" % group_name + if check_group_name(group_name, partition): + raise ScoValueError( + f"group_name {group_name} already exists in partition" ) # XXX FIX: incorrect error handling (in AJAX) cnx = ndb.GetDBConnexion() group_id = groupEditor.create( cnx, {"partition_id": partition_id, "group_name": group_name} ) - log("create_group: created group_id=%s" % group_id) + log("create_group: created group_id={group_id}") # return group_id @@ -817,21 +860,18 @@ def delete_group(group_id, partition_id=None): affectation aux groupes) partition_id est optionnel et ne sert que pour verifier que le groupe est bien dans cette partition. + S'il s'agit d'un groupe de parcours, affecte l'inscription des étudiants aux parcours. """ - group = get_group(group_id) + group = GroupDescr.query.get_or_404(group_id) if partition_id: - if partition_id != group["partition_id"]: + if partition_id != group.partition_id: raise ValueError("inconsistent partition/group") - else: - partition_id = group["partition_id"] - partition = get_partition(partition_id) - if not sco_permissions_check.can_change_groups(partition["formsemestre_id"]): + if not sco_permissions_check.can_change_groups(group.partition.formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") - log( - "delete_group: group_id=%s group_name=%s partition_name=%s" - % (group_id, group["group_name"], partition["partition_name"]) - ) - group_delete(group) + log(f"delete_group: group={group} partition={group.partition}") + formsemestre = group.partition.formsemestre + group_delete(group.id) + formsemestre.update_inscriptions_parcours_from_groups() def partition_create( @@ -881,7 +921,7 @@ def partition_create( if redirect: return flask.redirect( url_for( - "scolar.editPartitionForm", + "scolar.edit_partition_form", scodoc_dept=g.scodoc_dept, formsemestre_id=formsemestre_id, ) @@ -900,11 +940,12 @@ def get_arrow_icons_tags(): return arrow_up, arrow_down, arrow_none -def editPartitionForm(formsemestre_id=None): +def edit_partition_form(formsemestre_id=None): """Form to create/suppress partitions""" # ad-hoc form if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") + formsemestre = FormSemestre.query.get_or_404(formsemestre_id) partitions = get_partitions_list(formsemestre_id) arrow_up, arrow_down, arrow_none = get_arrow_icons_tags() suppricon = scu.icontag( @@ -914,7 +955,7 @@ def editPartitionForm(formsemestre_id=None): H = [ html_sco_header.sco_header( page_title="Partitions...", - javascripts=["js/editPartitionForm.js"], + javascripts=["js/edit_partition_form.js"], ), # limite à SHORT_STR_LEN r""" + {% include 'bul_foot.html' %} diff --git a/app/templates/but/documentation_codes_jury.html b/app/templates/but/documentation_codes_jury.html new file mode 100644 index 000000000..27e4aa0c6 --- /dev/null +++ b/app/templates/but/documentation_codes_jury.html @@ -0,0 +1,233 @@ +
    +

    Ci-dessous la signification de chaque code est expliquée, + ainsi que la correspondance avec les codes préconisés par + l'AMUE pour Apogée dans un document informel qui a circulé début + 2022 (les éventuelles erreurs n'engagent personne). +

    +
    Codes d'année
    +
    +
    %(label)s%(elem)s""" - + fcg - + "
    %(label)s%(elem)s{fcg}
    %(label)s%(elem)s
    %(label)s%(elem)s
    Parcours: {', '.join(parcours.code for parcours in sem.parcours)}
    Evaluations: %(nb_evals_completes)s ok, %(nb_evals_en_cours)s en cours, %(nb_evals_vides)s vides' + '
    Évaluations: %(nb_evals_completes)s ok, %(nb_evals_en_cours)s en cours, %(nb_evals_vides)s vides' % evals ) if evals["last_modif"]: @@ -1175,7 +1206,7 @@ def formsemestre_tableau_modules( ) H.append( '%s%s%s%s%(mois_debut)s%s{num_sem}{pm}{sem['mois_debut']}{formsemestre.titre_annee()}{parcours_name}BUT%sen coursen cours%s
    + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
    ScoDoc{{nom_univ}}AMUESignification
    ADM{{codes["ADM"]}}Admis
    ADJ{{codes["ADJ"]}}Admis par décision jury
    PASD{{codes["PASD"]}}PASDNon admis, mais passage de droit
    PAS1NCI{{codes["PAS1NCI"]}}PAS1NCINon admis, mais passage par décision de jury (Passage en Année + Supérieure avec au moins 1 Niveau de Compétence Insuffisant (RCUE<8)) +
    RED{{codes["RED"]}}REDAjourné, mais autorisé à redoubler
    NAR{{codes["NAR"]}}REONon admis, réorientation
    DEM{{codes["DEM"]}}Démission
    ABAN{{codes["ABAN"]}}ABANABANdon constaté (sans lettre de démission)
    ATJ{{codes["ATJ"]}}ndNon validé pour une autre raison, voir règlement local
    RAT{{codes["RAT"]}}En attente d’un rattrapage
    EXCLU{{codes["EXCLU"]}}EXCEXClusion, décision réservée à des décisions disciplinaires
    DEF{{codes["DEF"]}}(défaillance) Non évalué par manque assiduité
    ABL{{codes["ABL"]}}ABLAnnée Blanche
    + + +
    Codes RCUE (niveaux de compétences annuels)
    + +
    + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
    ScoDoc{{nom_univ}}AMUESignification
    ADM + {{codes["ADM"]}}VAL + Acquis +
    CMP{{codes["CMP"]}}Acquis par compensation annuelle
    ADJ{{codes["ADJ"]}}CODJAcquis par décision du jury
    AJ{{codes["AJ"]}}AJAttente pour problème de moyenne
    ATJ{{codes["ATJ"]}}ndNon validé pour une autre raison, voir règlement local
    RAT{{codes["RAT"]}}En attente d’un rattrapage
    DEF{{codes["DEF"]}}Défaillant
    ABAN{{codes["ABAN"]}}Non évalué pour manque assiduité
    +
    + +
    Codes des Unités d'Enseignement (UE)
    + +
    + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
    ScoDoc{{nom_univ}}AMUESignification
    ADM{{codes["ADM"]}}VALAcquis (ECTS acquis)
    CMP{{codes["CMP"]}}COMPAcquis par compensation UE compensée avec l’UE de même compétence et de même année (ECTS acquis) +
    ADJ{{codes["ADJ"]}}Acquis par décision de jury (ECTS acquis)
    AJ{{codes["AJ"]}}AJAttente pour problème de moyenne
    ATJ{{codes["ATJ"]}}ndNon validé pour une autre raison, voir règlement local
    RAT{{codes["RAT"]}}En attente d’un rattrapage
    DEF{{codes["DEF"]}}ABANDéfaillant Pas ou peu de notes par arrêt de la formation
    ABAN{{codes["ABAN"]}}ABANNon évalué pour manque d’assiduité Non présentation des notes de l’étudiant au jury
    DEM{{codes["DEM"]}}Démission
    UEBSL{{codes["UEBSL"]}}UEBSLUE blanchie
    +
    + \ No newline at end of file diff --git a/app/templates/but/formsemestre_validation_auto_but.html b/app/templates/but/formsemestre_validation_auto_but.html new file mode 100644 index 000000000..c9397fc0d --- /dev/null +++ b/app/templates/but/formsemestre_validation_auto_but.html @@ -0,0 +1,30 @@ +{# -*- mode: jinja-html -*- #} +{% extends "sco_page.html" %} +{% import 'bootstrap/wtf.html' as wtf %} + +{% block styles %} +{{super()}} +{% endblock %} + +{% block app_content %} + +

    Calcul automatique des décisions de jury annuelle BUT

    +
      +
    • Seuls les étudiants qui valident l'année seront affectés: + tous les niveaux de compétences (RCUE) validables + (moyenne annuelle au dessus de 10); +
    • +
    • l'assiduité n'est pas prise en compte;
    • +
    +

    + Il est nécessaire de relire soigneusement les décisions à l'issue de cette procédure ! +

    + + +
    +
    + {{ wtf.quick_form(form) }} +
    +
    + +{% endblock %} \ No newline at end of file diff --git a/app/templates/but/refcomp_assoc.html b/app/templates/but/refcomp_assoc.html index 9f8335be2..9ba07dee3 100644 --- a/app/templates/but/refcomp_assoc.html +++ b/app/templates/but/refcomp_assoc.html @@ -6,11 +6,25 @@

    Associer un référentiel de compétences

    Association d'un référentiel de compétence à la formation - {{formation.titre}} ({{formation.acronyme}}) + {{formation.titre}} ({{formation.acronyme}})
    -
    -
    - {{ wtf.quick_form(form) }} +
    + + Référentiel actuellement associé: + {% if formation.referentiel_competence is not none %} + {{ formation.referentiel_competence.specialite_long }} + supprimer + {% else %} + aucun + {% endif %} +
    +
    + {{ wtf.quick_form(form) }} +
    diff --git a/app/templates/but/refcomp_show.html b/app/templates/but/refcomp_show.html index 0e4ffef68..62b3a0154 100644 --- a/app/templates/but/refcomp_show.html +++ b/app/templates/but/refcomp_show.html @@ -10,7 +10,7 @@ - +
    Référentiel chargé le {{ref.scodoc_date_loaded.strftime("%d/%m/%Y à %H:%M") if ref.scodoc_date_loaded else ""}} à diff --git a/app/templates/config_codes_decisions.html b/app/templates/config_codes_decisions.html index 0c2f32b24..5f92aa8d3 100644 --- a/app/templates/config_codes_decisions.html +++ b/app/templates/config_codes_decisions.html @@ -6,12 +6,12 @@
    -

    Ces codes (ADM, AJ, ...) sont utilisés pour représenter les décisions de jury -et les validations de semestres ou d'UE. les valeurs indiquées ici sont utilisées -dans les exports Apogée. -

    -

    Ne les modifier que si vous savez ce que vous faites ! -

    +

    Ces codes (ADM, AJ, ...) sont utilisés pour représenter les décisions de jury + et les validations de semestres ou d'UE. + Les valeurs indiquées ici sont utilisées dans les exports Apogée. +

    +

    Ne les modifier que si vous savez ce que vous faites ! +

    diff --git a/app/templates/confirm_dialog.html b/app/templates/confirm_dialog.html new file mode 100644 index 000000000..8067f6c74 --- /dev/null +++ b/app/templates/confirm_dialog.html @@ -0,0 +1,22 @@ +{# -*- mode: jinja-html -*- #} +{% extends 'base.html' %} +{% import 'bootstrap/wtf.html' as wtf %} + +{% block app_content %} + +

    {{ title }}

    + +
    + {{ explanation }} +
    +
    + + + {% if cancel_url %} + + {% endif %} + +
    + +{% endblock %} \ No newline at end of file diff --git a/app/templates/pn/form_mods.html b/app/templates/pn/form_mods.html index 10927bc88..480cf267c 100644 --- a/app/templates/pn/form_mods.html +++ b/app/templates/pn/form_mods.html @@ -84,17 +84,29 @@ url_for("notes.module_create", scodoc_dept=g.scodoc_dept, module_type=module_type|int, - matiere_id=matiere_parent.id + matiere_id=matiere_parent.id, + semestre_id=semestre_id, )}}" {% else %}"{{ url_for("notes.module_create", scodoc_dept=g.scodoc_dept, module_type=module_type|int, - formation_id=formation.id + formation_id=formation.id, + semestre_id=semestre_id, )}}" - {% endif %} + {% endif %} >{{create_element_msg}} + + {% if module_type==scu.ModuleType.STANDARD %} +
  • ajouter un module de malus dans chaque UE du S{{semestre_id}} +
  • + {% endif %} {% endif %} {% endif %} diff --git a/app/templates/pn/form_modules_ue_coefs.html b/app/templates/pn/form_modules_ue_coefs.html index f022eccd3..75ed3de6e 100644 --- a/app/templates/pn/form_modules_ue_coefs.html +++ b/app/templates/pn/form_modules_ue_coefs.html @@ -2,10 +2,12 @@

    {% if not read_only %}Édition des c{% else %}C{%endif%}oefficients des modules vers les UEs

    {% if not read_only %} - Double-cliquer pour changer une valeur. +

    Double-cliquer pour changer une valeur. Les valeurs sont automatiquement enregistrées au fur et à mesure. +

    {% endif %} - +

    Chaque ligne représente une ressource ou SAÉ, et chaque colonne une Unité d'Enseignement (UE). +

    Semestre: