Fix: lien validation jury

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)

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()