From bde51dc6391dccca6f8342a2e36b32a07aa249ff Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Wed, 8 Jun 2022 17:42:52 +0200
Subject: [PATCH] Fix: check suppression formation/ue

---
 app/models/ues.py                |  9 +++++++++
 app/scodoc/sco_edit_formation.py |  5 +++--
 app/scodoc/sco_edit_ue.py        | 16 ++++------------
 3 files changed, 16 insertions(+), 14 deletions(-)

diff --git a/app/models/ues.py b/app/models/ues.py
index 518bd7219..48d81a14f 100644
--- a/app/models/ues.py
+++ b/app/models/ues.py
@@ -75,6 +75,15 @@ class UniteEns(db.Model):
 
         return sco_edit_ue.ue_is_locked(self.id)
 
+    def can_be_deleted(self) -> bool:
+        """True si l'UE n'est pas utilisée dans des formsemestre
+        et n'a pas de module rattachés
+        """
+        # "pas un seul module de cette UE n'a de modimpl...""
+        return (self.modules.count() == 0) or not any(
+            m.modimpls.all() for m in self.modules
+        )
+
     def guess_semestre_idx(self) -> None:
         """Lorsqu'on prend une ancienne formation non APC,
         les UE n'ont pas d'indication de semestre.
diff --git a/app/scodoc/sco_edit_formation.py b/app/scodoc/sco_edit_formation.py
index aabfaddce..606fc7421 100644
--- a/app/scodoc/sco_edit_formation.py
+++ b/app/scodoc/sco_edit_formation.py
@@ -66,8 +66,9 @@ def formation_delete(formation_id=None, dialog_confirmed=False):
     sems = sco_formsemestre.do_formsemestre_list({"formation_id": formation_id})
     if sems:
         H.append(
-            """<p class="warning">Impossible de supprimer cette formation, car les sessions suivantes l'utilisent:</p>
-<ul>"""
+            """<p class="warning">Impossible de supprimer cette formation,
+            car les sessions suivantes l'utilisent:</p>
+            <ul>"""
         )
         for sem in sems:
             H.append(
diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py
index a0f4ff743..28cc1e083 100644
--- a/app/scodoc/sco_edit_ue.py
+++ b/app/scodoc/sco_edit_ue.py
@@ -142,14 +142,6 @@ def do_ue_create(args):
     return ue_id
 
 
-def can_delete_ue(ue: UniteEns) -> bool:
-    """True si l'UE n'est pas utilisée dans des formsemestre
-    et n'a pas de module rattachés
-    """
-    # "pas un seul module de cette UE n'a de modimpl...""
-    return (ue.modules.count() == 0) and not any(m.modimpls.all() for m in ue.modules)
-
-
 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
@@ -158,9 +150,9 @@ def do_ue_delete(ue_id, delete_validations=False, force=False):
     ue = UniteEns.query.get_or_404(ue_id)
     formation_id = ue.formation_id
     semestre_idx = ue.semestre_idx
-    if not can_delete_ue(ue):
+    if not ue.can_be_deleted():
         raise ScoNonEmptyFormationObject(
-            "UE",
+            f"UE (id={ue.id}, dud)",
             msg=ue.titre,
             dest_url=url_for(
                 "notes.ue_table",
@@ -540,9 +532,9 @@ def ue_delete(ue_id=None, delete_validations=False, dialog_confirmed=False):
                 semestre_idx=ue.semestre_idx,
             ),
         )
-    if not can_delete_ue(ue):
+    if not ue.can_be_deleted():
         raise ScoNonEmptyFormationObject(
-            "UE",
+            f"UE",
             msg=ue.titre,
             dest_url=url_for(
                 "notes.ue_table",