From ac5c433f5a42bcdc0412d098588486620f863b66 Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Sun, 19 Dec 2021 11:08:03 +0100 Subject: [PATCH] Affichage poids sur tableau bord module + fix #222 --- app/models/evaluations.py | 15 +++++++++---- app/scodoc/sco_liste_notes.py | 15 +++++++++---- app/scodoc/sco_moduleimpl_status.py | 35 +++++++++++++++++++++-------- app/static/css/scodoc.css | 3 +++ sco_version.py | 2 +- 5 files changed, 52 insertions(+), 18 deletions(-) diff --git a/app/models/evaluations.py b/app/models/evaluations.py index 9a237c5c98..e0733fb77e 100644 --- a/app/models/evaluations.py +++ b/app/models/evaluations.py @@ -76,11 +76,11 @@ class Evaluation(db.Model): db.session.add(copy) return copy - def set_ue_poids(self, ue, poids: float): + def set_ue_poids(self, ue, poids: float) -> None: """Set poids évaluation vers cette UE""" self.update_ue_poids_dict({ue.id: poids}) - def set_ue_poids_dict(self, ue_poids_dict: dict): + def set_ue_poids_dict(self, ue_poids_dict: dict) -> None: """set poids vers les UE (remplace existants) ue_poids_dict = { ue_id : poids } """ @@ -91,16 +91,23 @@ class Evaluation(db.Model): self.ue_poids = L self.moduleimpl.invalidate_evaluations_poids() # inval cache - def update_ue_poids_dict(self, ue_poids_dict: dict): + def update_ue_poids_dict(self, ue_poids_dict: dict) -> None: """update poids vers UE (ajoute aux existants)""" current = self.get_ue_poids_dict() current.update(ue_poids_dict) self.set_ue_poids_dict(current) - def get_ue_poids_dict(self): + def get_ue_poids_dict(self) -> dict: """returns { ue_id : poids }""" return {p.ue.id: p.poids for p in self.ue_poids} + def get_ue_poids_str(self) -> str: + """string describing poids, for excel cells and pdfs + Note: si les poids ne sont pas initialisés (poids par défaut), + ils ne sont pas affichés. + """ + return ", ".join([f"{p.ue.acronyme}: {p.poids}" for p in self.ue_poids]) + class EvaluationUEPoids(db.Model): """Poids des évaluations (BUT) diff --git a/app/scodoc/sco_liste_notes.py b/app/scodoc/sco_liste_notes.py index 517d172df3..09e5480f7f 100644 --- a/app/scodoc/sco_liste_notes.py +++ b/app/scodoc/sco_liste_notes.py @@ -32,6 +32,7 @@ import flask from flask import url_for, g, request from app import models +from app.models.evaluations import Evaluation from app.models.moduleimpls import ModuleImpl import app.scodoc.sco_utils as scu import app.scodoc.notesdb as ndb @@ -395,6 +396,7 @@ def _make_table_notes( key_mgr, note_sur_20, keep_numeric, + format=format, ) columns_ids.append(e["evaluation_id"]) # @@ -408,7 +410,7 @@ def _make_table_notes( # Si module, ajoute la (les) "moyenne(s) du module: if mode == "module": if len(evals) > 1: - # Moyenne de l'étudant dans le module + # Moyenne de l'étudiant dans le module # Affichée même en APC à titre indicatif _add_moymod_column( sem["formsemestre_id"], @@ -637,6 +639,7 @@ def _add_eval_columns( K, note_sur_20, keep_numeric, + format="html", ): """Add eval e""" nb_notes = 0 @@ -645,6 +648,7 @@ def _add_eval_columns( sum_notes = 0 notes = [] # liste des notes numeriques, pour calcul histogramme uniquement evaluation_id = e["evaluation_id"] + e_o = Evaluation.query.get(evaluation_id) # XXX en attendant ré-écriture NotesDB = sco_evaluation_db.do_evaluation_get_all_notes(evaluation_id) for row in rows: etudid = row["etudid"] @@ -706,9 +710,12 @@ def _add_eval_columns( row_coefs[evaluation_id] = "coef. %s" % e["coefficient"] if is_apc: - row_poids[evaluation_id] = _mini_table_eval_ue_poids( - evaluation_id, evals_poids, ues - ) + if format == "html": + row_poids[evaluation_id] = _mini_table_eval_ue_poids( + evaluation_id, evals_poids, ues + ) + else: + row_poids[evaluation_id] = e_o.get_ue_poids_str() if note_sur_20: nmax = 20.0 else: diff --git a/app/scodoc/sco_moduleimpl_status.py b/app/scodoc/sco_moduleimpl_status.py index 921fbd113f..c9f5aaa289 100644 --- a/app/scodoc/sco_moduleimpl_status.py +++ b/app/scodoc/sco_moduleimpl_status.py @@ -31,9 +31,10 @@ import time from flask import g, url_for from flask_login import current_user -from app.auth.models import User +from app.auth.models import User from app.models import ModuleImpl +from app.models.evaluations import Evaluation import app.scodoc.sco_utils as scu from app.scodoc.sco_permissions import Permission @@ -391,8 +392,9 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): ) H.append("""""") eval_index = len(mod_evals) - 1 - first = True + first_eval = True for eval in mod_evals: + evaluation = Evaluation.query.get(eval["evaluation_id"]) # TODO unifier etat = sco_evaluations.do_evaluation_etat( eval["evaluation_id"], partition_id=partition_id, @@ -406,9 +408,7 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): else: tr_class = "mievr" tr_class_1 = "mievr" - if first: - first = False - else: + if not first_eval: H.append("""""") tr_class_1 += " mievr_spaced" H.append("""""") # if etat["nb_notes"] == 0: - H.append("""""") + H.append("""""" % tr_class) + if modimpl.module.is_apc(): + H.append( + f"""""" + ) + else: + H.append('') + H.append("""""") else: # il y a deja des notes saisies gr_moyennes = etat["gr_moyennes"] + first_group = True for gr_moyenne in gr_moyennes: H.append("""""" % tr_class) - H.append("""""") + H.append("""""") + if first_group and modimpl.module.is_apc(): + H.append( + f"""""" + ) + else: + H.append("""""") + first_group = False if gr_moyenne["group_name"] is None: name = "Tous" # tous else: name = "Groupe %s" % gr_moyenne["group_name"] H.append( - """""") + first_eval = False # if caneditevals or not sem["etat"]: diff --git a/app/static/css/scodoc.css b/app/static/css/scodoc.css index 4de3920925..58f819c5bf 100644 --- a/app/static/css/scodoc.css +++ b/app/static/css/scodoc.css @@ -1476,6 +1476,9 @@ span.evalindex { margin-left: 3px; } +table.moduleimpl_evaluations td.eval_poids { + color:rgb(0, 0, 255); +} /* Formulaire edition des partitions */ form#editpart table { diff --git a/sco_version.py b/sco_version.py index cdee3253ec..b33f520ab2 100644 --- a/sco_version.py +++ b/sco_version.py @@ -1,7 +1,7 @@ # -*- mode: python -*- # -*- coding: utf-8 -*- -SCOVERSION = "9.1.10" +SCOVERSION = "9.1.11" SCONAME = "ScoDoc"
 
""" % tr_class_1) @@ -586,19 +586,35 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): H.append("""
 """ % tr_class) - H.append("""
{ + evaluation.get_ue_poids_str()}
  { + evaluation.get_ue_poids_str()}%s  """ % name + """%s  """ % name ) if gr_moyenne["gr_nb_notes"] > 0: H.append("%(gr_moy)s" % gr_moyenne) @@ -637,6 +653,7 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): H.append("""""") H.append("") H.append("""