From 11ef8857e2fd899d51ddd73ee102805d576f410f Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Tue, 28 Sep 2021 07:27:55 +0200
Subject: [PATCH] fix urls

---
 app/scodoc/sco_formsemestre_validation.py | 54 +++++++++++++++--------
 1 file changed, 36 insertions(+), 18 deletions(-)

diff --git a/app/scodoc/sco_formsemestre_validation.py b/app/scodoc/sco_formsemestre_validation.py
index 99f5b5267..7c49c792a 100644
--- a/app/scodoc/sco_formsemestre_validation.py
+++ b/app/scodoc/sco_formsemestre_validation.py
@@ -160,10 +160,11 @@ def formsemestre_validation_etud_form(
     if etud_etat != "I":
         H.append(
             tf_error_message(
-                """Impossible de statuer sur cet étudiant:
-                il est démissionnaire ou défaillant (voir <a href="%s">sa fiche</a>)
+                f"""Impossible de statuer sur cet étudiant:
+                il est démissionnaire ou défaillant (voir <a href="{
+                    url_for("scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=etudid)
+                }">sa fiche</a>)
                 """
-                % url_for("scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=etudid)
             )
         )
         return "\n".join(H + Footer)
@@ -175,16 +176,19 @@ def formsemestre_validation_etud_form(
     )
     if check:
         if not desturl:
-            desturl = (
-                "formsemestre_recapcomplet?modejury=1&hidemodules=1&hidebac=1&pref_override=0&formsemestre_id="
-                + str(formsemestre_id)
+            desturl = url_for(
+                "notes.formsemestre_recapcomplet",
+                scodoc_dept=g.scodoc_dept,
+                modejury=1,
+                hidemodules=1,
+                hidebac=1,
+                pref_override=0,
+                formsemestre_id=formsemestre_id,
+                sortcol=sortcol
+                or None,  # pour refaire tri sorttable du tableau de notes
+                _anchor="etudid%s" % etudid,  # va a la bonne ligne
             )
-            if sortcol:
-                desturl += (
-                    "&sortcol=" + sortcol
-                )  # pour refaire tri sorttable du tableau de notes
-            desturl += "#etudid%s" % etudid  # va a la bonne ligne
-        H.append('<ul><li><a href="%s">Continuer</a></li></ul>' % desturl)
+        H.append(f'<ul><li><a href="{desturl}">Continuer</a></li></ul>')
 
         return "\n".join(H + Footer)
 
@@ -194,8 +198,12 @@ def formsemestre_validation_etud_form(
     if nt.etud_has_notes_attente(etudid):
         H.append(
             tf_error_message(
-                """Impossible de statuer sur cet étudiant: il a des notes en attente dans des évaluations de ce semestre (voir <a href="formsemestre_status?formsemestre_id=%s">tableau de bord</a>)"""
-                % formsemestre_id
+                f"""Impossible de statuer sur cet étudiant: il a des notes en
+                attente dans des évaluations de ce semestre (voir <a href="{
+                    url_for( "notes.formsemestre_status",
+                    scodoc_dept=g.scodoc_dept, formsemestre_id=formsemestre_id)
+                }">tableau de bord</a>)
+                """
             )
         )
         return "\n".join(H + Footer)
@@ -210,14 +218,24 @@ def formsemestre_validation_etud_form(
         if not Se.prev_decision:
             H.append(
                 tf_error_message(
-                    """Le jury n\'a pas statué sur le semestre précédent ! (<a href="formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s">le faire maintenant</a>)"""
-                    % (Se.prev["formsemestre_id"], etudid)
+                    f"""Le jury n'a pas statué sur le semestre précédent ! (<a href="{
+                    url_for("notes.formsemestre_validation_etud_form",
+                        scodoc_dept=g.scodoc_dept,
+                        formsemestre_id=Se.prev["formsemestre_id"],
+                        etudid=etudid)
+                    }">le faire maintenant</a>)
+                    """
                 )
             )
             if decision_jury:
                 H.append(
-                    '<a href="formsemestre_validation_suppress_etud?etudid=%s&formsemestre_id=%s" class="stdlink">Supprimer décision existante</a>'
-                    % (etudid, formsemestre_id)
+                    f"""<a href="{
+                    url_for("notes.formsemestre_validation_suppress_etud",
+                        scodoc_dept=g.scodoc_dept,
+                        etudid=etudid, formsemestre_id=formsemestre_id
+                    )
+                    }" class="stdlink">Supprimer décision existante</a>
+                    """
                 )
             H.append(html_sco_header.sco_footer())
             return "\n".join(H)