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(
- """
Impossible de supprimer cette formation, car les sessions suivantes l'utilisent:
-"""
+ """Impossible de supprimer cette formation,
+ car les sessions suivantes l'utilisent:
+ """
)
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",