app/views/notes.py

@@ -286,21 +286,28 @@ def formsemestre_bulletinetud(
     if not isinstance(formsemestre_id, int):
         raise ScoInvalidIdType("formsemestre_id must be an integer !")
     formsemestre = FormSemestre.query.get_or_404(formsemestre_id)
+    dept = formsemestre.dept_id
+    # une fois le formsemestre déterminé, on peut rechercher l'étudiant
+    # (y compris sur le nip ou le code ine ar on connait le département)
+    if etudid:
+        etud = models.Identite.query.get_or_404(etudid)
+    elif code_nip:
+        etud = (
+            models.Identite.query.filter_by(code_nip=str(code_nip))
+            .filter_by(dept_id=dept)
+            .first_or_404()
+        )
+    elif code_ine:
+        etud = (
+            models.Identite.query.filter_by(code_ine=str(code_ine))
+            .filter_by(dept_id=dept)
+            .first_or_404()
+        )
+    else:
+        raise ScoValueError(
+            "Paramètre manquant: spécifier code_nip ou etudid ou code_ine"
+        )
     if formsemestre.formation.is_apc() and format != "oldjson":
-        if etudid:
-            etud = models.Identite.query.get_or_404(etudid)
-        elif code_nip:
-            etud = models.Identite.query.filter_by(
-                code_nip=str(code_nip)
-            ).first_or_404()
-        elif code_ine:
-            etud = models.Identite.query.filter_by(
-                code_ine=str(code_ine)
-            ).first_or_404()
-        else:
-            raise ScoValueError(
-                "Paramètre manquant: spécifier code_nip ou etudid ou code_ine"
-            )
         if format == "json":
             r = bulletin_but.ResultatsSemestreBUT(formsemestre)
             return jsonify(r.bulletin_etud(etud, formsemestre))
@@ -492,6 +499,7 @@ sco_publish(
     methods=["GET", "POST"],
 )
 
+
 #
 @bp.route("/")
 @bp.route("/index_html")
@@ -655,7 +663,6 @@ sco_publish(
     Permission.ScoView,
 )
 
-
 # --- Matieres
 sco_publish(
     "/do_matiere_create",
@@ -669,7 +676,6 @@ sco_publish(
     Permission.ScoChangeFormation,
 )
 
-
 # --- Modules
 sco_publish(
     "/do_module_delete",
@@ -887,8 +893,8 @@ def edit_enseignants_form(moduleimpl_id):
         H.append(
             f"""
             <li>{nom} (<a class="stdlink" href="{
-                url_for('notes.edit_enseignants_form_delete', scodoc_dept=g.scodoc_dept, moduleimpl_id=moduleimpl_id, ens_id=ens["ens_id"])
-                }">supprimer</a>)
+            url_for('notes.edit_enseignants_form_delete', scodoc_dept=g.scodoc_dept, moduleimpl_id=moduleimpl_id, ens_id=ens["ens_id"])
+            }">supprimer</a>)
             </li>"""
         )
     H.append("</ul>")
@@ -1698,9 +1704,9 @@ def evaluation_delete(evaluation_id):
             "\n".join(H)
             + f"""<p>OK, évaluation supprimée.</p>
         <p><a class="stdlink" href="{
-            url_for("notes.moduleimpl_status", scodoc_dept=g.scodoc_dept, 
-            moduleimpl_id=E["moduleimpl_id"])
-            }">Continuer</a></p>"""
+        url_for("notes.moduleimpl_status", scodoc_dept=g.scodoc_dept,
+                moduleimpl_id=E["moduleimpl_id"])
+        }">Continuer</a></p>"""
             + html_sco_header.sco_footer()
         )
 
@@ -2527,6 +2533,7 @@ sco_publish(
     Permission.ScoView,
 )
 
+
 # --------------------------------------------------------------------
 # DEBUG