diff --git a/app/models/evaluations.py b/app/models/evaluations.py index 71d54d82e..04d0dcf91 100644 --- a/app/models/evaluations.py +++ b/app/models/evaluations.py @@ -13,6 +13,8 @@ from app.models.ues import UniteEns from app.scodoc.sco_exceptions import ScoValueError import app.scodoc.notesdb as ndb +DEFAULT_EVALUATION_TIME = datetime.time(8, 0) + class Evaluation(db.Model): """Evaluation (contrôle, examen, ...)""" @@ -111,12 +113,24 @@ class Evaluation(db.Model): if self.heure_debut and ( not self.heure_fin or self.heure_fin == self.heure_debut ): - return f"""à {self.heure_debut.strftime("%H:%M")}""" + return f"""à {self.heure_debut.strftime("%Hh%M")}""" elif self.heure_debut and self.heure_fin: - return f"""de {self.heure_debut.strftime("%H:%M")} à {self.heure_fin.strftime("%H:%M")}""" + return f"""de {self.heure_debut.strftime("%Hh%M")} à {self.heure_fin.strftime("%Hh%M")}""" else: return "" + def descr_duree(self) -> str: + "Description de la durée pour affichages" + if self.heure_debut is None and self.heure_fin is None: + return "" + debut = self.heure_debut or DEFAULT_EVALUATION_TIME + fin = self.heure_fin or DEFAULT_EVALUATION_TIME + d = (fin.hour * 60 + fin.minute) - (debut.hour * 60 + debut.minute) + duree = f"{d//60}h" + if d % 60: + duree += f"{d%60:02d}" + return duree + def clone(self, not_copying=()): """Clone, not copying the given attrs Attention: la copie n'a pas d'id avant le prochain commit diff --git a/app/scodoc/sco_moduleimpl_status.py b/app/scodoc/sco_moduleimpl_status.py index 3ab89db7e..305f03b14 100644 --- a/app/scodoc/sco_moduleimpl_status.py +++ b/app/scodoc/sco_moduleimpl_status.py @@ -203,16 +203,22 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): sem = sco_formsemestre.get_formsemestre(formsemestre_id) F = sco_formations.formation_list(args={"formation_id": sem["formation_id"]})[0] mod_inscrits = sco_moduleimpl.do_moduleimpl_inscription_list( - moduleimpl_id=M["moduleimpl_id"] + moduleimpl_id=moduleimpl_id ) nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) - mod_evals = sco_evaluation_db.do_evaluation_list({"moduleimpl_id": moduleimpl_id}) - mod_evals.sort( - key=lambda x: (x["numero"], x["jour"], x["heure_debut"]), reverse=True - ) # la plus RECENTE en tête - + # mod_evals = sco_evaluation_db.do_evaluation_list({"moduleimpl_id": moduleimpl_id}) + # mod_evals.sort( + # key=lambda x: (x["numero"], x["jour"], x["heure_debut"]), reverse=True + # ) + # la plus RECENTE en tête + evaluations = modimpl.evaluations.order_by( + Evaluation.numero.desc(), + Evaluation.jour.desc(), + Evaluation.heure_debut.desc(), + ).all() + nb_evaluations = len(evaluations) # sem_locked = not sem["etat"] can_edit_evals = ( @@ -329,8 +335,10 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): # ) H.append("") H.append( - 'Absences dans ce module' - % moduleimpl_id + f"""Absences dans ce module""" ) # Adapté à partir d'une suggestion de DS (Le Havre) # Liens saisies absences seulement si permission et date courante dans le semestre @@ -372,13 +380,14 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): """

%d évaluations : """ - % (len(mod_evals), moduleimpl_id) + % (nb_evaluations, moduleimpl_id) ) # # Liste les noms de partitions partitions = sco_groups.get_partitions_list(sem["formsemestre_id"]) H.append( - """Afficher les groupes de """ ) been_selected = False for partition in partitions: @@ -414,27 +423,27 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): url_for("notes.evaluation_create", scodoc_dept=g.scodoc_dept, moduleimpl_id=M['moduleimpl_id']) }">Créer nouvelle évaluation """ - if mod_evals: + if nb_evaluations > 0: top_table_links += f""" Trier par date """ - if mod_evals: + if nb_evaluations > 0: H.append( '
' + top_table_links + "
" ) H.append("""""") - eval_index = len(mod_evals) - 1 + eval_index = nb_evaluations - 1 first_eval = True - for eval_dict in mod_evals: + for evaluation in evaluations: H.append( _ligne_evaluation( modimpl, - eval_dict, + evaluation, first_eval=first_eval, partition_id=partition_id, arrow_down=arrow_down, @@ -443,7 +452,7 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): can_edit_evals=can_edit_evals, can_edit_notes=can_edit_notes, eval_index=eval_index, - nb_evals=len(mod_evals), + nb_evals=nb_evaluations, is_apc=nt.is_apc, ) ) @@ -495,7 +504,7 @@ def moduleimpl_status(moduleimpl_id=None, partition_id=None): def _ligne_evaluation( modimpl: ModuleImpl, - eval_dict: dict, + evaluation: Evaluation, first_eval: bool = True, partition_id=None, arrow_down=None, @@ -509,8 +518,7 @@ def _ligne_evaluation( ) -> str: """Ligne décrivant une évaluation dans le tableau de bord moduleimpl.""" H = [] - # TODO unifier pour ne plus utiliser eval_dict - evaluation: Evaluation = Evaluation.query.get(eval_dict["evaluation_id"]) + # evaluation: Evaluation = Evaluation.query.get(eval_dict["evaluation_id"]) etat = sco_evaluations.do_evaluation_etat( evaluation.id, partition_id=partition_id, @@ -543,7 +551,7 @@ def _ligne_evaluation( elif is_apc: # visualisation des poids H.append(_evaluation_poids_html(evaluation)) - + H.append("""
""") if evaluation.jour: H.append( f"""Le {evaluation.jour.strftime("%d/%m/%Y")} {evaluation.descr_heure()}""" @@ -571,7 +579,7 @@ def _ligne_evaluation( ) # H.append( - f""" + f"""
{ eval_index:2} @@ -597,6 +605,7 @@ def _ligne_evaluation( H.append( f""" +
@@ -611,7 +620,7 @@ def _ligne_evaluation( if etat["evalcomplete"]: etat_txt = """(prise en compte)""" etat_descr = "notes utilisées dans les moyennes" - elif eval_dict["publish_incomplete"]: + elif evaluation.publish_incomplete: etat_txt = """(prise en compte immédiate)""" etat_descr = ( "il manque des notes, mais la prise en compte immédiate a été demandée" @@ -677,7 +686,7 @@ def _ligne_evaluation( }">{scu.icontag("status_orange_img", title="il manque des notes")}""" ) # - if eval_dict["visibulletin"]: + if evaluation.visibulletin: H.append( scu.icontag( "status_visible_img", title="visible dans bulletins intermédiaires" @@ -696,8 +705,8 @@ def _ligne_evaluation( # H.append( f""" - - + + """ ) H.append( @@ -710,7 +719,7 @@ def _ligne_evaluation( ) if etat["moy"]: H.append( - f"""{etat["moy"]} / {eval_dict["note_max"]:g} + f"""{etat["moy"]} / {evaluation.note_max:g}   (
 {eval_dict["duree"]}{eval_dict["coefficient"]:g}{evaluation.descr_duree()}{evaluation.coefficient:g}