From ef171364a697368be1a424c74ed3acd5d59eb886 Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Fri, 2 Feb 2024 18:23:35 +0100 Subject: [PATCH] =?UTF-8?q?Edition/import=20=C3=A9tudiants=20/=20traitemen?= =?UTF-8?q?t=20erreurs=20codes=20dupliqu=C3=A9s?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/models/etudiants.py | 65 ++++++++++++++++++++++++++++- app/scodoc/sco_etud.py | 75 +++------------------------------- app/scodoc/sco_import_etuds.py | 11 +++-- app/views/scolar.py | 24 +++++++---- 4 files changed, 94 insertions(+), 81 deletions(-) diff --git a/app/models/etudiants.py b/app/models/etudiants.py index a03058e99..d7eb0312d 100644 --- a/app/models/etudiants.py +++ b/app/models/etudiants.py @@ -19,7 +19,7 @@ from app.models.departements import Departement from app.models.scolar_event import ScolarEvent from app.scodoc import notesdb as ndb from app.scodoc.sco_bac import Baccalaureat -from app.scodoc.sco_exceptions import ScoInvalidParamError, ScoValueError +from app.scodoc.sco_exceptions import ScoGenError, ScoInvalidParamError, ScoValueError import app.scodoc.sco_utils as scu @@ -233,6 +233,15 @@ class Identite(models.ScoDocModel): log(f"Identite.create {etud}") return etud + def from_dict(self, args, **kwargs) -> bool: + """Check arguments, then modify. + Add to session but don't commit. + True if modification. + """ + check_etud_duplicate_code(args, "code_nip") + check_etud_duplicate_code(args, "code_ine") + return super().from_dict(args, **kwargs) + @classmethod def filter_model_attributes(cls, data: dict, excluded: set[str] = None) -> dict: """Returns a copy of dict with only the keys belonging to the Model and not in excluded.""" @@ -728,6 +737,60 @@ class Identite(models.ScoDocModel): ) +def check_etud_duplicate_code(args, code_name, edit=True): + """Vérifie que le code n'est pas dupliqué. + Raises ScoGenError si problème. + """ + etudid = args.get("etudid", None) + if not args.get(code_name, None): + return + etuds = Identite.query.filter_by( + **{code_name: str(args[code_name]), "dept_id": g.scodoc_dept_id} + ).all() + duplicate = False + if edit: + duplicate = (len(etuds) > 1) or ( + (len(etuds) == 1) and etuds[0].id != args["etudid"] + ) + else: + duplicate = len(etuds) > 0 + if duplicate: + listh = [] # liste des doubles + for etud in etuds: + listh.append(f"Autre étudiant: {etud.html_link_fiche()}") + if etudid: + submit_label = "retour à la fiche étudiant" + dest_endpoint = "scolar.fiche_etud" + parameters = {"etudid": etudid} + else: + if "tf_submitted" in args: + del args["tf_submitted"] + submit_label = "Continuer" + dest_endpoint = "scolar.etudident_create_form" + parameters = args + else: + submit_label = "Annuler" + dest_endpoint = "notes.index_html" + parameters = {} + + err_page = f"""

Code étudiant ({code_name}) dupliqué !

+

Le {code_name} {args[code_name]} est déjà utilisé: un seul étudiant peut avoir + ce code. Vérifier votre valeur ou supprimer l'autre étudiant avec cette valeur. +

+ +

+ {submit_label} +

+ """ + + log(f"*** error: code {code_name} duplique: {args[code_name]}") + + raise ScoGenError(err_page) + + def make_etud_args( etudid=None, code_nip=None, use_request=True, raise_exc=False, abort_404=True ) -> dict: diff --git a/app/scodoc/sco_etud.py b/app/scodoc/sco_etud.py index f6b8a18df..7a6a49e93 100644 --- a/app/scodoc/sco_etud.py +++ b/app/scodoc/sco_etud.py @@ -39,6 +39,7 @@ from app import db, email from app import log from app.models import Admission, Identite from app.models.etudiants import ( + check_etud_duplicate_code, input_civilite, input_civilite_etat_civil, make_etud_args, @@ -229,74 +230,12 @@ def check_nom_prenom_homonyms( return True, query.all() -def _check_duplicate_code(cnx, args, code_name, disable_notify=False, edit=True): - """Vérifie que le code n'est pas dupliqué. - Raises ScoGenError si problème. - """ - etudid = args.get("etudid", None) - if args.get(code_name, None): - etuds = identite_list(cnx, {code_name: str(args[code_name])}) - duplicate = False - if edit: - duplicate = (len(etuds) > 1) or ( - (len(etuds) == 1) and etuds[0]["id"] != args["etudid"] - ) - else: - duplicate = len(etuds) > 0 - if duplicate: - listh = [] # liste des doubles - for e in etuds: - listh.append( - f"""Autre étudiant: {e['nom']} {e['prenom']}""" - ) - if etudid: - OK = "retour à la fiche étudiant" - dest_endpoint = "scolar.fiche_etud" - parameters = {"etudid": etudid} - else: - if "tf_submitted" in args: - del args["tf_submitted"] - OK = "Continuer" - dest_endpoint = "scolar.etudident_create_form" - parameters = args - else: - OK = "Annuler" - dest_endpoint = "notes.index_html" - parameters = {} - if not disable_notify: - err_page = f"""

Code étudiant ({code_name}) dupliqué !

-

Le {code_name} {args[code_name]} est déjà utilisé: un seul étudiant peut avoir - ce code. Vérifier votre valeur ou supprimer l'autre étudiant avec cette valeur. -

- -

- {OK} -

- """ - else: - err_page = f"""

Code étudiant ({code_name}) dupliqué !

""" - log(f"*** error: code {code_name} duplique: {args[code_name]}") - raise ScoGenError(err_page) - - def identite_edit(cnx, args, disable_notify=False): """Modifie l'identite d'un étudiant. Si pref notification et difference, envoie message notification, sauf si disable_notify """ - _check_duplicate_code( - cnx, args, "code_nip", disable_notify=disable_notify, edit=True - ) - _check_duplicate_code( - cnx, args, "code_ine", disable_notify=disable_notify, edit=True - ) + check_etud_duplicate_code(args, "code_nip", edit=True) + check_etud_duplicate_code(args, "code_ine", edit=True) notify_to = None if not disable_notify: try: @@ -325,16 +264,14 @@ def identite_edit(cnx, args, disable_notify=False): def identite_create(cnx, args): "check unique etudid, then create" - _check_duplicate_code(cnx, args, "code_nip", edit=False) - _check_duplicate_code(cnx, args, "code_ine", edit=False) + check_etud_duplicate_code(args, "code_nip", edit=False) + check_etud_duplicate_code(args, "code_ine", edit=False) if "etudid" in args: etudid = args["etudid"] r = identite_list(cnx, {"etudid": etudid}) if r: - raise ScoValueError( - "Code identifiant (etudid) déjà utilisé ! (%s)" % etudid - ) + raise ScoValueError(f"Code identifiant (etudid) déjà utilisé ! ({etudid})") return _identiteEditor.create(cnx, args) diff --git a/app/scodoc/sco_import_etuds.py b/app/scodoc/sco_import_etuds.py index 7107217b3..ddab91ee0 100644 --- a/app/scodoc/sco_import_etuds.py +++ b/app/scodoc/sco_import_etuds.py @@ -508,15 +508,20 @@ def students_import_admission( H = [html_sco_header.sco_header(page_title="Import données admissions")] H.append("

Import terminé !

") H.append( - '

Continuer

' - % url_for( + f"""

Continuer

""" ) if diag: - H.append("

Diagnostic:

" % "
  • ".join(diag)) + H.append( + f"""

    Diagnostic:

    + """ + ) return "\n".join(H) + html_sco_header.sco_footer() diff --git a/app/views/scolar.py b/app/views/scolar.py index b9e57cbe2..f3025ddb6 100644 --- a/app/views/scolar.py +++ b/app/views/scolar.py @@ -2306,23 +2306,31 @@ def form_students_import_infos_admissions(formsemestre_id=None): html_sco_header.sco_header( page_title="Export données admissions (Parcoursup ou autre)", ), - """

    Téléchargement des informations sur l'admission des étudiants

    + f"""

    Téléchargement des informations sur l'admission + des étudiants

    - Exporter les informations de ScoDoc (classeur Excel) (ce fichier peut être ré-importé après d'éventuelles modifications) + Exporter les informations de ScoDoc (classeur Excel) (ce fichier + peut être ré-importé après d'éventuelles modifications)

    Vous n'avez pas le droit d'importer les données

    - """ - % {"formsemestre_id": formsemestre_id}, + """, ] return "\n".join(H) + F # On a le droit d'importer: H = [ html_sco_header.sco_header(page_title="Import données admissions Parcoursup"), - f"""

    Téléchargement des informations sur l'admission des étudiants depuis feuilles import Parcoursup

    -
    -

    A utiliser pour renseigner les informations sur l'origine des étudiants (lycées, bac, etc). Ces informations sont facultatives mais souvent utiles pour mieux connaitre les étudiants et aussi pour effectuer des statistiques (résultats suivant le type de bac...). Les données sont affichées sur les fiches individuelles des étudiants.

    -
    + f"""

    Téléchargement des informations sur l'admission des étudiants + depuis feuilles import Parcoursup

    +
    +

    A utiliser pour renseigner les informations sur l'origine des étudiants (lycées, bac, etc). + Ces informations sont facultatives mais souvent utiles pour mieux connaitre les étudiants + et aussi pour effectuer des statistiques (résultats suivant le type de bac...). + Les données sont affichées sur les fiches individuelles des étudiants. +

    +

    Importer ici la feuille excel utilisée pour envoyer le classement Parcoursup. Seuls les étudiants actuellement inscrits dans ce semestre ScoDoc seront affectés,