Fix: creation matieres sans numeros

app/models/modules.py

@@ -1,6 +1,5 @@
 """ScoDoc 9 models : Modules
 """
-from operator import attrgetter
 from flask import current_app
 
 from app import db

app/scodoc/sco_edit_matiere.py

@@ -32,6 +32,7 @@ import flask
 from flask import g, url_for, request
 from app.models.events import ScolarNews
 from app.models.formations import Matiere
+from app.models.ues import UniteEns
 
 import app.scodoc.notesdb as ndb
 import app.scodoc.sco_utils as scu
@@ -100,13 +101,12 @@ def do_matiere_create(args):
 
 def matiere_create(ue_id=None):
     """Creation d'une matiere"""
-    from app.scodoc import sco_edit_ue
-
-    UE = sco_edit_ue.ue_list(args={"ue_id": ue_id})[0]
+    ue: UniteEns = UniteEns.query.get_or_404(ue_id)
+    default_numero = max([mat.numero for mat in ue.matieres] or [9]) + 1
     H = [
         html_sco_header.sco_header(page_title="Création d'une matière"),
-        """<h2>Création d'une matière dans l'UE %(titre)s (%(acronyme)s)</h2>""" % UE,
-        """<p class="help">Les matières sont des groupes de modules dans une UE
+        f"""<h2>Création d'une matière dans l'UE {ue.titre} ({ue.acronyme})</h2>
+        <p class="help">Les matières sont des groupes de modules dans une UE
 d'une formation donnée. Les matières servent surtout pour la
 présentation (bulletins, etc) mais <em>n'ont pas de rôle dans le calcul
 des notes.</em>
@@ -126,13 +126,21 @@ associé.
         scu.get_request_args(),
         (
             ("ue_id", {"input_type": "hidden", "default": ue_id}),
-            ("titre", {"size": 30, "explanation": "nom de la matière."}),
+            (
+                "titre",
+                {
+                    "size": 30,
+                    "explanation": "nom de la matière.",
+                },
+            ),
             (
                 "numero",
                 {
                     "size": 2,
                     "explanation": "numéro (1,2,3,4...) pour affichage",
                     "type": "int",
+                    "default": default_numero,
+                    "allow_null": False,
                 },
             ),
         ),
@@ -140,7 +148,7 @@ associé.
     )
 
     dest_url = url_for(
-        "notes.ue_table", scodoc_dept=g.scodoc_dept, formation_id=UE["formation_id"]
+        "notes.ue_table", scodoc_dept=g.scodoc_dept, formation_id=ue.formation_id
     )
 
     if tf[0] == 0:

app/scodoc/sco_edit_module.py

@@ -659,6 +659,7 @@ def module_edit(
                 "explanation": "numéro (1, 2, 3, 4, ...) pour ordre d'affichage",
                 "type": "int",
                 "default": default_num,
+                "allow_null": False,
             },
         ),
     ]

sco_version.py

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