From 06caacbb470b4cdbf4f87c7fa99f73d8b564be36 Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Tue, 22 Nov 2022 15:25:33 +0100 Subject: [PATCH] minor code cleaning --- app/scodoc/sco_saisie_notes.py | 147 +++++++++++++++++++-------------- 1 file changed, 87 insertions(+), 60 deletions(-) diff --git a/app/scodoc/sco_saisie_notes.py b/app/scodoc/sco_saisie_notes.py index 4e33d5e77..2c8d8fa7a 100644 --- a/app/scodoc/sco_saisie_notes.py +++ b/app/scodoc/sco_saisie_notes.py @@ -651,29 +651,28 @@ def notes_add( def saisie_notes_tableur(evaluation_id, group_ids=()): """Saisie des notes via un fichier Excel""" - evals = sco_evaluation_db.do_evaluation_list({"evaluation_id": evaluation_id}) - if not evals: - raise ScoValueError("invalid evaluation_id") - E = evals[0] - M = sco_moduleimpl.moduleimpl_list(moduleimpl_id=E["moduleimpl_id"])[0] - formsemestre_id = M["formsemestre_id"] - if not sco_permissions_check.can_edit_notes(current_user, E["moduleimpl_id"]): + evaluation = Evaluation.query.get_or_404(evaluation_id) + moduleimpl_id = evaluation.moduleimpl.id + formsemestre_id = evaluation.moduleimpl.formsemestre_id + if not sco_permissions_check.can_edit_notes(current_user, moduleimpl_id): return ( html_sco_header.sco_header() - + "

Modification des notes impossible pour %s

" - % current_user.user_name - + """

(vérifiez que le semestre n'est pas verrouillé et que vous - avez l'autorisation d'effectuer cette opération)

-

Continuer

+ + f""" +

Modification des notes impossible pour {current_user.user_name}

+

(vérifiez que le semestre n'est pas verrouillé et que vous + avez l'autorisation d'effectuer cette opération) +

+

Continuer

""" - % E["moduleimpl_id"] + html_sco_header.sco_footer() ) - if E["description"]: - page_title = 'Saisie des notes de "%s"' % E["description"] - else: - page_title = "Saisie des notes" + page_title = "Saisie des notes" + ( + f"""de {evaluation.description}""" if evaluation.description else "" + ) # Informations sur les groupes à afficher: groups_infos = sco_groups_view.DisplayedGroupsInfos( @@ -700,16 +699,23 @@ def saisie_notes_tableur(evaluation_id, group_ids=()): H.append("") H.append( - """
+ f""" -
+
+ +
""" - % (evaluation_id, groups_infos.groups_query_args, evaluation_id, evaluation_id) ) H.append( @@ -752,70 +758,91 @@ def saisie_notes_tableur(evaluation_id, group_ids=()): if updiag[0]: H.append(updiag[1]) H.append( - """

Notes chargées.    - + f"""

Notes chargées.    + Revenir au tableau de bord du module     - Charger d'autres notes dans cette évaluation + Charger d'autres notes dans cette évaluation

""" - % E ) else: - H.append("""

Notes non chargées !

""" + updiag[1]) H.append( - """ -

- Reprendre -

""" - % E + f""" +

Notes non chargées !

+ {updiag[1]} +

+ Reprendre +

+ """ ) + # H.append("""

Autres opérations

""" - % E - ) + +
  • Effacer toutes les notes de cette évaluation + (ceci permet ensuite de supprimer l'évaluation si besoin) +
  • +
  • Revenir au module +
  • +
  • Revenir au formulaire de saisie +
  • + - H.append( - """

    Explications

    +

    Explications

      -
    1. Etape 1: +
    2. Etape 1:
      1. choisir le ou les groupes d'étudiants;
      2. télécharger le fichier Excel à remplir.
    3. -
    4. Etape 2 (cadre vert): Indiquer le fichier Excel téléchargé à l'étape 1 et dans lequel on a saisi des notes. Remarques: -
        -
      • le fichier Excel peut être incomplet: on peut ne saisir que quelques notes et répéter l'opération (en téléchargeant un nouveau fichier) plus tard;
      • -
      • seules les valeurs des notes modifiées sont prises en compte;
      • -
      • seules les notes sont extraites du fichier Excel;
      • -
      • on peut optionnellement ajouter un commentaire (type "copies corrigées par Dupont", ou "Modif. suite à contestation") dans la case "Commentaire". -
      • -
      • le fichier Excel doit impérativement être celui chargé à l'étape 1 pour cette évaluation. Il n'est pas possible d'utiliser une liste d'appel ou autre document Excel téléchargé d'une autre page.
      • -
      +
    5. Etape 2 (cadre vert): Indiquer le fichier Excel + téléchargé à l'étape 1 et dans lequel on a saisi des notes. + Remarques: +
        +
      • le fichier Excel peut être incomplet: on peut ne saisir que quelques notes + et répéter l'opération (en téléchargeant un nouveau fichier) plus tard; +
      • +
      • seules les valeurs des notes modifiées sont prises en compte; +
      • +
      • seules les notes sont extraites du fichier Excel; +
      • +
      • on peut optionnellement ajouter un commentaire (type "copies corrigées + par Dupont", ou "Modif. suite à contestation") dans la case "Commentaire". +
      • +
      • le fichier Excel doit impérativement être celui chargé à l'étape 1 + pour cette évaluation. Il n'est pas possible d'utiliser une liste d'appel + ou autre document Excel téléchargé d'une autre page. +
      • +
    """ - ) + ) H.append(html_sco_header.sco_footer()) return "\n".join(H)