Améliore quelques routes et msg erreurs

app/scodoc/sco_groups_view.py

@@ -302,7 +302,12 @@ class DisplayedGroupsInfos(object):
             if group_ids:
                 group_ids = [group_ids]  # cas ou un seul parametre, pas de liste
         else:
+            try:
                 group_ids = [int(g) for g in group_ids]
+            except ValueError as exc:
+                raise ScoValueError(
+                    "identifiant de groupe invalide (mettre à jour vos bookmarks ?)"
+                ) from exc
         if not formsemestre_id and moduleimpl_id:
             mods = sco_moduleimpl.moduleimpl_list(moduleimpl_id=moduleimpl_id)
             if len(mods) != 1:

app/views/notes.py

@@ -397,7 +397,7 @@ def ue_table(formation_id=None, semestre_idx=1, msg=""):
     )
 
 
-@bp.route("/ue_infos/<ue_id>")
+@bp.route("/ue_infos/<int:ue_id>")
 @scodoc
 @permission_required(Permission.ScoView)
 def ue_infos(ue_id):

app/views/pn_modules.py

@@ -68,7 +68,7 @@ from app.scodoc.sco_permissions import Permission
 
 
 @bp.route("/table_modules_ue_coefs/<int:formation_id>")
-@bp.route("/table_modules_ue_coefs/<int:formation_id>/<semestre_idx>")
+@bp.route("/table_modules_ue_coefs/<int:formation_id>/<int:semestre_idx>")
 @scodoc
 @permission_required(Permission.ScoView)
 def table_modules_ue_coefs(formation_id, semestre_idx=None):

app/views/scodoc.py

@@ -121,7 +121,7 @@ def create_dept():
     )
 
 
-@bp.route("/ScoDoc/toggle_dept_vis/<dept_id>", methods=["GET", "POST"])
+@bp.route("/ScoDoc/toggle_dept_vis/<int:dept_id>", methods=["GET", "POST"])
 @admin_required
 def toggle_dept_vis(dept_id):
     """Cache ou rend visible un dept"""

sco_version.py

@@ -1,7 +1,7 @@
 # -*- mode: python -*-
 # -*- coding: utf-8 -*-
 
-SCOVERSION = "9.1.56"
+SCOVERSION = "9.1.57"
 
 SCONAME = "ScoDoc"