Test unitaire superficiel de (presque) toutes les vues du tableau de bord semestre

app/scodoc/sco_formsemestre_status.py

@@ -540,58 +540,6 @@ def formsemestre_page_title(formsemestre_id=None):
     )
 
 
-def fill_formsemestre(sem):
-    """Add some useful fields to help display formsemestres"""
-    sem["notes_url"] = scu.NotesURL()
-    formsemestre_id = sem["formsemestre_id"]
-    if not sem["etat"]:
-        sem[
-            "locklink"
-        ] = f"""<a href="{url_for('notes.formsemestre_change_lock',
-            scodoc_dept=g.scodoc_dept,formsemestre_id=formsemestre_id )
-            }">{scu.icontag("lock_img", border="0", title="Semestre verrouillé")}</a>"""
-    else:
-        sem["locklink"] = ""
-    if sco_preferences.get_preference("bul_display_publication", formsemestre_id):
-        if sem["bul_hide_xml"]:
-            eyeicon = scu.icontag("hide_img", border="0", title="Bulletins NON publiés")
-        else:
-            eyeicon = scu.icontag("eye_img", border="0", title="Bulletins publiés")
-        sem[
-            "eyelink"
-        ] = f"""<a href="{
-            url_for('notes.formsemestre_change_publication_bul', 
-                scodoc_dept=g.scodoc_dept,formsemestre_id=formsemestre_id)
-        }">{eyeicon}</a>"""
-    else:
-        sem["eyelink"] = ""
-    F = sco_formations.formation_list(args={"formation_id": sem["formation_id"]})[0]
-    sem["formation"] = F
-    parcours = codes_cursus.get_cursus_from_code(F["type_parcours"])
-    if sem["semestre_id"] != -1:
-        sem["num_sem"] = f""", {parcours.SESSION_NAME} {sem["semestre_id"]}"""
-    else:
-        sem["num_sem"] = ""  # formation sans semestres
-    if sem["modalite"]:
-        sem["modalitestr"] = f""" en {sem["modalite"]}"""
-    else:
-        sem["modalitestr"] = ""
-
-    sem["etape_apo_str"] = "Code étape Apogée: " + (
-        sco_formsemestre.formsemestre_etape_apo_str(sem) or "Pas de code étape"
-    )
-
-    inscrits = sco_formsemestre_inscriptions.do_formsemestre_inscription_list(
-        args={"formsemestre_id": formsemestre_id}
-    )
-    sem["nbinscrits"] = len(inscrits)
-    uresps = [
-        sco_users.user_info(responsable_id) for responsable_id in sem["responsables"]
-    ]
-    sem["resp"] = ", ".join([u["prenomnom"] for u in uresps])
-    sem["nomcomplet"] = ", ".join([u["nomcomplet"] for u in uresps])
-
-
 # Description du semestre sous forme de table exportable
 def formsemestre_description_table(
     formsemestre_id: int, with_evals=False, with_parcours=False

app/scodoc/sco_recapcomplet.py

@@ -277,7 +277,8 @@ def _formsemestre_recapcomplet_to_file(
             include_evaluations=include_evaluations,
             filename=filename,
         )
-        return scu.send_file(data, filename=filename, mime=scu.get_mime_suffix(format))
+        mime, suffix = scu.get_mime_suffix(tabformat)
+        return scu.send_file(data, filename=filename, mime=mime, suffix=suffix)
     elif tabformat == "xml":
         data = gen_formsemestre_recapcomplet_xml(
             formsemestre.id,

app/static/css/gt_table.css

@@ -649,7 +649,7 @@ table.dataTable.gt_table {
   padding-right: 5px;
 }
 
-/* Tables non centrées */
+/* Tables non centrées (inutile) */
 table.dataTable.gt_table.gt_left {
   margin-left: 16px;
 }

app/static/css/scodoc.css

@@ -4078,7 +4078,8 @@ button.unselect {
 } 
 */
 
-div.table_recap table.table_recap {
+div.table_recap table.table_recap,
+div.evaluations_recap table.evaluations_recap {
   width: auto;
   margin-left: 0px;
   /* font-family: Consolas, monaco, monospace; */

app/templates/formsemestre_page_title.j2

@@ -21,7 +21,7 @@
             scodoc_dept=g.scodoc_dept, formsemestre_id=formsemestre.id)
         }}">{{formsemestre.etuds_inscriptions|length}} inscrits</a></span><span class="lock">
             {%-if not formsemestre.etat -%}
-            <a href="{{ url_for( 'notes.formsemestre_change_lock', 
+            <a href="{{ url_for( 'notes.formsemestre_flip_lock', 
         scodoc_dept=g.scodoc_dept, formsemestre_id=formsemestre.id )}}">{{
                 scu.icontag("lock_img", border="0", title="Semestre verrouillé")|safe
                 }}</a>