app/scodoc/sco_dept.py

@@ -405,6 +405,8 @@ def delete_dept(dept_id: int) -> str:
             "delete from scolar_news where dept_id = %(dept_id)s",
             "delete from notes_semset where dept_id = %(dept_id)s",
             "delete from notes_formations where dept_id = %(dept_id)s",
+            "delete from itemsuivi_tags where dept_id = %(dept_id)s",
+            "delete from identite where dept_id = %(dept_id)s",
             "delete from departement where id = %(dept_id)s",
             "drop table tags_temp",
             "drop table formations_temp",

app/scodoc/sco_preferences.py

@@ -626,10 +626,11 @@ class BasePreferences:
                 "forcer_module",
                 {
                     "initvalue": 0,
-                    "title": "Forcer la déclaration du module.",
+                    "title": "Imposer la déclaration du module",
                     "input_type": "boolcheckbox",
                     "labels": ["non", "oui"],
                     "category": "assi",
+                    "explanation": "toute saisie d'absence doit indiquer le module concerné",
                 },
             ),
             # (

app/views/scolar.py

@@ -42,6 +42,7 @@ from flask_json import as_json
 from flask_login import current_user
 from flask_wtf import FlaskForm
 from flask_wtf.file import FileField, FileAllowed
+import sqlalchemy as sa
 from wtforms import SubmitField
 
 import app
@@ -1853,22 +1854,16 @@ def etud_copy_in_other_dept(etudid: int):
     )
 
 
-@bp.route("/etudident_delete", methods=["GET", "POST"])
+@bp.route("/etudident_delete/<int:etudid>", methods=["GET", "POST"])
 @scodoc
 @permission_required(Permission.EtudInscrit)
 @scodoc7func
-def etudident_delete(etudid, dialog_confirmed=False):
+def etudident_delete(etudid: int = -1, dialog_confirmed=False):
     "Delete a student"
-    cnx = ndb.GetDBConnexion()
-    etuds = sco_etud.etudident_list(cnx, {"etudid": etudid})
-    if not etuds:
-        raise ScoValueError("Étudiant inexistant !")
-    else:
-        etud = etuds[0]
-    sco_etud.fill_etuds_info([etud])
+    etud = Identite.get_etud(etudid)
     if not dialog_confirmed:
         return scu.confirm_dialog(
-            """<h2>Confirmer la suppression de l'étudiant <b>{e[nomprenom]}</b> ?</h2>
+            f"""<h2>Confirmer la suppression de l'étudiant <b>{etud.nomprenom}</b> ?</h2>
             </p>
             <p style="top-margin: 2ex; bottom-margin: 2ex;">Prenez le temps de vérifier
             que vous devez vraiment supprimer cet étudiant !
@@ -1877,16 +1872,13 @@ def etudident_delete(etudid, dialog_confirmed=False):
             efface toute trace de l'étudiant: inscriptions, <b>notes</b>, absences...
             dans <b>tous les semestres</b> qu'il a fréquenté.
             </p>
-            <p>Dans la plupart des cas, vous avez seulement besoin de le <ul>désinscrire</ul>
-            d'un semestre ? (dans ce cas passez par sa fiche, menu associé au semestre)</p>
+            <p>Dans la plupart des cas, vous avez seulement besoin de le <b>désinscrire</b>
+            d'un semestre ! (pour cela, passez par sa fiche, menu associé au semestre)</p>
 
-            <p><a href="{fiche_url}">Vérifier la fiche de {e[nomprenom]}</a>
-            </p>""".format(
-                e=etud,
-                fiche_url=url_for(
+            <p><a class="stdlink" href="{url_for(
                     "scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=etudid
-                ),
-            ),
+                )}">Vérifier la fiche de {etud.nomprenom}</a>
+            </p>""",
             dest_url="",
             cancel_url=url_for(
                 "scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=etudid
@@ -1894,13 +1886,19 @@ def etudident_delete(etudid, dialog_confirmed=False):
             OK="Supprimer définitivement cet étudiant",
             parameters={"etudid": etudid},
         )
-    log("etudident_delete: etudid=%(etudid)s nomprenom=%(nomprenom)s" % etud)
+    log(f"etudident_delete: {etud}")
+    formsemestre_ids_to_inval = [
+        ins.formsemestre_id for ins in etud.formsemestre_inscriptions
+    ]
+
     # delete in all tables !
     # c'est l'ancienne façon de gérer les cascades dans notre pseudo-ORM :)
     tables = [
         "notes_appreciations",
         "scolar_autorisation_inscription",
         "scolar_formsemestre_validation",
+        "apc_validation_rcue",
+        "apc_validation_annee",
         "scolar_events",
         "notes_notes_log",
         "notes_notes",
@@ -1914,14 +1912,14 @@ def etudident_delete(etudid, dialog_confirmed=False):
         "absences_notifications",
         "billet_absence",
     ]
-    cursor = cnx.cursor(cursor_factory=ndb.ScoDocCursor)
     for table in tables:
-        cursor.execute("delete from %s where etudid=%%(etudid)s" % table, etud)
-    cursor.execute("delete from identite where id=%(etudid)s", etud)
-    cnx.commit()
+        db.session.execute(
+            sa.text(f"""delete from {table} where etudid=:etudid"""), {"etudid": etudid}
+        )
+    db.session.delete(etud)
+    db.session.commit()
     # Inval semestres où il était inscrit:
-    to_inval = [s["formsemestre_id"] for s in etud["sems"]]
-    for formsemestre_id in to_inval:
+    for formsemestre_id in formsemestre_ids_to_inval:
         sco_cache.invalidate_formsemestre(formsemestre_id=formsemestre_id)
     flash("Étudiant supprimé !")
     return flask.redirect(scu.ScoURL())