From ef44a71e39dd38148d2499438d12d404eae9b5b7 Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Fri, 23 Aug 2024 19:02:33 +0200
Subject: [PATCH] templification saisie_notes_tableur

---
 app/scodoc/sco_saisie_excel.py | 43 +++++++++++++++++-----------------
 1 file changed, 22 insertions(+), 21 deletions(-)

diff --git a/app/scodoc/sco_saisie_excel.py b/app/scodoc/sco_saisie_excel.py
index c95f154b9..1f2383b82 100644
--- a/app/scodoc/sco_saisie_excel.py
+++ b/app/scodoc/sco_saisie_excel.py
@@ -51,7 +51,6 @@ from flask_login import current_user
 from app.models import Evaluation, FormSemestre, Identite, ModuleImpl, ScolarNews
 from app.scodoc.sco_excel import COLORS, ScoExcelSheet
 from app.scodoc import (
-    html_sco_header,
     sco_cache,
     sco_evaluations,
     sco_evaluation_db,
@@ -969,19 +968,21 @@ def saisie_notes_tableur(evaluation_id: int, group_ids=()):
     moduleimpl_id = evaluation.moduleimpl.id
     formsemestre_id = evaluation.moduleimpl.formsemestre_id
     if not evaluation.moduleimpl.can_edit_notes(current_user):
-        return (
-            html_sco_header.sco_header()
-            + f"""
-                <h2>Modification des notes impossible pour {current_user.user_name}</h2>
-                <p>(vérifiez que le semestre n'est pas verrouillé et que vous
-                avez l'autorisation d'effectuer cette opération)
-                </p>
-                <p><a class="stdlink" href="{
-                url_for("notes.moduleimpl_status", scodoc_dept=g.scodoc_dept,
-                    moduleimpl_id=moduleimpl_id)
-                }">Continuer</a></p>
-               """
-            + html_sco_header.sco_footer()
+        dest_url = url_for(
+            "notes.moduleimpl_status",
+            scodoc_dept=g.scodoc_dept,
+            moduleimpl_id=moduleimpl_id,
+        )
+        raise ScoValueError(
+            f"""
+            <h2>Modification des notes impossible pour {current_user.user_name}</h2>
+            <p>(vérifiez que le semestre n'est pas verrouillé et que vous
+            avez l'autorisation d'effectuer cette opération)
+            </p>
+            <p><a class="stdlink" href="{dest_url}">Continuer</a></p>
+            """,
+            safe=True,
+            dest_url="",
         )
 
     page_title = "Saisie des notes" + (
@@ -997,11 +998,6 @@ def saisie_notes_tableur(evaluation_id: int, group_ids=()):
     )
 
     H = [
-        html_sco_header.sco_header(
-            page_title=page_title,
-            javascripts=sco_groups_view.JAVASCRIPTS,
-            cssstyles=sco_groups_view.CSSSTYLES,
-        ),
         sco_evaluations.evaluation_describe(evaluation_id=evaluation_id),
         """<span class="eval_title">Saisie des notes par fichier</span>""",
     ]
@@ -1176,8 +1172,13 @@ def saisie_notes_tableur(evaluation_id: int, group_ids=()):
         </div>
 """
         )
-    H.append(html_sco_header.sco_footer())
-    return "\n".join(H)
+    return render_template(
+        "sco_page.j2",
+        content="\n".join(H),
+        page_title=page_title,
+        javascripts=sco_groups_view.JAVASCRIPTS,
+        cssstyles=sco_groups_view.CSSSTYLES,
+    )
 
 
 def formsemestre_import_notes(