Fix temporaire du bug modification assiduité (WIP: gestion du module autre dysfoncytionnelle)

app/models/assiduites.py

@@ -210,6 +210,14 @@ class Assiduite(db.Model):
 
     def set_moduleimpl(self, moduleimpl_id: int | str) -> bool:
         """TODO"""
+        # je ne comprend pas cette fonction WIP
+        # moduleimpl_id peut être == "autre", ce qui plante
+        # ci-dessous un fix temporaire en attendant explication de @iziram
+        try:
+            moduleimpl_id_int = int(moduleimpl_id)
+        except ValueError:
+            raise ScoValueError("invalid moduleimpl_id")
+        # /fix
         moduleimpl: ModuleImpl = ModuleImpl.query.get(moduleimpl_id)
         if moduleimpl is not None:
             # Vérification de l'inscription de l'étudiant

app/views/assiduites.py

@@ -1458,7 +1458,10 @@ def _action_modifier_assiduite(assi: Assiduite):
             module = int(module)
         except ValueError:
             pass
-
+    # TODO revoir, documenter (voir set_moduleimpl)
+    # ne pas appeler module ici un paramètre qui s'appelle moduleimpl_id dans la fonction
+    # module == instance de Module
+    # moduleimpl_id : id, toujours integer
     assi.set_moduleimpl(module)
 
     db.session.add(assi)