From 2add3e12cc3ee295c75f8db9049624d21d0f9b6e Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Fri, 24 Sep 2021 00:28:09 +0200
Subject: [PATCH] Fix: lien validation jury

---
 app/scodoc/sco_formsemestre_validation.py | 10 ++++------
 app/scodoc/sco_parcours_dut.py            | 21 ++++++++-------------
 2 files changed, 12 insertions(+), 19 deletions(-)

diff --git a/app/scodoc/sco_formsemestre_validation.py b/app/scodoc/sco_formsemestre_validation.py
index 447fae7d3..b7b59ba3e 100644
--- a/app/scodoc/sco_formsemestre_validation.py
+++ b/app/scodoc/sco_formsemestre_validation.py
@@ -356,7 +356,7 @@ def formsemestre_validation_etud(
     #
     Se.valide_decision(selected_choice, REQUEST)  # enregistre
     return _redirect_valid_choice(
-        formsemestre_id, etudid, Se, selected_choice, desturl, sortcol, REQUEST
+        formsemestre_id, etudid, Se, selected_choice, desturl, sortcol
     )
 
 
@@ -402,19 +402,17 @@ def formsemestre_validation_etud_manu(
     Se.valide_decision(choice, REQUEST)  # enregistre
     if redirect:
         return _redirect_valid_choice(
-            formsemestre_id, etudid, Se, choice, desturl, sortcol, REQUEST
+            formsemestre_id, etudid, Se, choice, desturl, sortcol
         )
 
 
-def _redirect_valid_choice(
-    formsemestre_id, etudid, Se, choice, desturl, sortcol, REQUEST
-):
+def _redirect_valid_choice(formsemestre_id, etudid, Se, choice, desturl, sortcol):
     adr = "formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s&check=1" % (
         formsemestre_id,
         etudid,
     )
     if sortcol:
-        adr += "&sortcol=" + sortcol
+        adr += "&sortcol=" + str(sortcol)
     # if desturl:
     #    desturl += "&desturl=" + desturl
     return flask.redirect(adr)
diff --git a/app/scodoc/sco_parcours_dut.py b/app/scodoc/sco_parcours_dut.py
index 2fb5c732c..83f030335 100644
--- a/app/scodoc/sco_parcours_dut.py
+++ b/app/scodoc/sco_parcours_dut.py
@@ -588,7 +588,6 @@ class SituationEtudParcoursGeneric(object):
             self.etudid,
             decision.code_etat,
             decision.assiduite,
-            REQUEST=REQUEST,
         )
         # -- modification du code du semestre precedent
         if self.prev and decision.new_code_prev:
@@ -619,7 +618,6 @@ class SituationEtudParcoursGeneric(object):
                 self.etudid,
                 decision.new_code_prev,
                 decision.assiduite,  # attention: en toute rigueur il faudrait utiliser une indication de l'assiduite au sem. precedent, que nous n'avons pas...
-                REQUEST=REQUEST,
             )
 
             sco_cache.invalidate_formsemestre(
@@ -897,9 +895,7 @@ def formsemestre_update_validation_sem(
     return to_invalidate
 
 
-def formsemestre_validate_ues(
-    formsemestre_id, etudid, code_etat_sem, assiduite, REQUEST=None
-):
+def formsemestre_validate_ues(formsemestre_id, etudid, code_etat_sem, assiduite):
     """Enregistre codes UE, selon état semestre.
     Les codes UE sont toujours calculés ici, et non passés en paramètres
     car ils ne dépendent que de la note d'UE et de la validation ou non du semestre.
@@ -933,14 +929,13 @@ def formsemestre_validate_ues(
                 cnx, nt, formsemestre_id, etudid, ue_id, code_ue
             )
 
-        if REQUEST:
-            logdb(
-                cnx,
-                method="validate_ue",
-                etudid=etudid,
-                msg="ue_id=%s code=%s" % (ue_id, code_ue),
-                commit=False,
-            )
+        logdb(
+            cnx,
+            method="validate_ue",
+            etudid=etudid,
+            msg="ue_id=%s code=%s" % (ue_id, code_ue),
+            commit=False,
+        )
     cnx.commit()