Dernières opérations
+
")
+ return "\n".join(H)
diff --git a/app/models/formations.py b/app/models/formations.py
index c0f375ddc..f76607d3c 100644
--- a/app/models/formations.py
+++ b/app/models/formations.py
@@ -146,7 +146,8 @@ class Formation(db.Model):
db.session.add(ue)
db.session.commit()
- app.clear_scodoc_cache()
+ if change:
+ app.clear_scodoc_cache()
class Matiere(db.Model):
diff --git a/app/models/formsemestre.py b/app/models/formsemestre.py
index 4ec90052b..0d126fc7f 100644
--- a/app/models/formsemestre.py
+++ b/app/models/formsemestre.py
@@ -286,7 +286,7 @@ class FormSemestre(db.Model):
"""
if not self.etapes:
return ""
- return ", ".join([str(x.etape_apo) for x in self.etapes])
+ return ", ".join(sorted([str(x.etape_apo) for x in self.etapes]))
def responsables_str(self, abbrev_prenom=True) -> str:
"""chaîne "J. Dupond, X. Martin"
@@ -374,6 +374,16 @@ class FormSemestre(db.Model):
return self.titre
return f"{self.titre} {self.formation.get_parcours().SESSION_NAME} {self.semestre_id}"
+ def sem_modalite(self) -> str:
+ """Le semestre et la modialité, ex "S2 FI" ou "S3 APP" """
+ if self.semestre_id > 0:
+ descr_sem = f"S{self.semestre_id}"
+ else:
+ descr_sem = ""
+ if self.modalite:
+ descr_sem += " " + self.modalite
+ return descr_sem
+
def get_abs_count(self, etudid):
"""Les comptes d'absences de cet étudiant dans ce semestre:
tuple (nb abs, nb abs justifiées)
@@ -423,7 +433,7 @@ notes_formsemestre_responsables = db.Table(
class FormSemestreEtape(db.Model):
- """Étape Apogée associées au semestre"""
+ """Étape Apogée associée au semestre"""
__tablename__ = "notes_formsemestre_etapes"
id = db.Column(db.Integer, primary_key=True)
diff --git a/app/models/groups.py b/app/models/groups.py
index f6452cf7c..9cf5f2364 100644
--- a/app/models/groups.py
+++ b/app/models/groups.py
@@ -74,6 +74,10 @@ class GroupDescr(db.Model):
f"""<{self.__class__.__name__} {self.id} "{self.group_name or '(tous)'}">"""
)
+ def get_nom_with_part(self) -> str:
+ "Nom avec partition: 'TD A'"
+ return f"{self.partition.partition_name or ''} {self.group_name or '-'}"
+
group_membership = db.Table(
"group_membership",
diff --git a/app/models/moduleimpls.py b/app/models/moduleimpls.py
index 292ec8ffd..7574ed7e8 100644
--- a/app/models/moduleimpls.py
+++ b/app/models/moduleimpls.py
@@ -9,7 +9,6 @@ from app.comp import df_cache
from app.models.etudiants import Identite
from app.models.modules import Module
-import app.scodoc.notesdb as ndb
from app.scodoc import sco_utils as scu
diff --git a/app/scodoc/gen_tables.py b/app/scodoc/gen_tables.py
index 7f5531c6f..01b94c064 100644
--- a/app/scodoc/gen_tables.py
+++ b/app/scodoc/gen_tables.py
@@ -121,6 +121,7 @@ class GenTable(object):
html_with_td_classes=False, # put class=column_id in each - """
+ ]
+
+ for news in news_list:
+ H.append(
+ f"""
- {news.formatted_date()}{news} """ + ) + + H.append("
+ Pour être informé des évolutions de ScoDoc,
+ vous pouvez vous
+
+ abonner à la liste de diffusion.
+
+ """
+ )
+
+ H.append("' - % request.base_url + f"""
+
Voir tous les semestres ({len(othersems)} verrouillés) +
""" ) H.append( - """ - """ - % scu.NotesURL() + f"""+
+ """ ) # if current_user.has_permission(Permission.ScoEtudInscrit): @@ -149,23 +150,26 @@ Chercher étape courante:Gestion des étudiants
-
-
- créer un nouvel étudiant -
- importer de nouveaux étudiants (ne pas utiliser sauf cas particulier, utilisez plutôt le lien dans - le tableau de bord semestre si vous souhaitez inscrire les - étudiants importés à un semestre) +
- créer un nouvel étudiant + +
- importer de nouveaux étudiants + (ne pas utiliser sauf cas particulier, utilisez plutôt le lien dans + le tableau de bord semestre si vous souhaitez inscrire les + étudiants importés à un semestre) +
+ f"""
Exports Apogée
""" - % scu.NotesURL() ) # H.append( @@ -177,7 +181,13 @@ Chercher étape courante: tuple[bool, str]: def do_matiere_delete(oid): "delete matiere and attached modules" - from app.scodoc import sco_formations from app.scodoc import sco_edit_ue from app.scodoc import sco_edit_module - from app.scodoc import sco_news cnx = ndb.GetDBConnexion() # check @@ -197,11 +195,11 @@ def do_matiere_delete(oid): # news formation = Formation.query.get(ue["formation_id"]) - sco_news.add( - typ=sco_news.NEWS_FORM, - object=ue["formation_id"], + ScolarNews.add( + typ=ScolarNews.NEWS_FORM, + obj=ue["formation_id"], text=f"Modification de la formation {formation.acronyme}", - max_frequency=3, + max_frequency=10 * 60, ) formation.invalidate_cached_sems() diff --git a/app/scodoc/sco_edit_module.py b/app/scodoc/sco_edit_module.py index 2f6b3f430..ece30a345 100644 --- a/app/scodoc/sco_edit_module.py +++ b/app/scodoc/sco_edit_module.py @@ -38,6 +38,7 @@ from app import models from app.models import APO_CODE_STR_LEN from app.models import Formation, Matiere, Module, UniteEns from app.models import FormSemestre, ModuleImpl +from app.models import ScolarNews import app.scodoc.notesdb as ndb import app.scodoc.sco_utils as scu @@ -53,7 +54,6 @@ from app.scodoc import html_sco_header from app.scodoc import sco_codes_parcours from app.scodoc import sco_edit_matiere from app.scodoc import sco_moduleimpl -from app.scodoc import sco_news _moduleEditor = ndb.EditableTable( "notes_modules", @@ -98,18 +98,16 @@ def module_list(*args, **kw): def do_module_create(args) -> int: "Create a module. Returns id of new object." # create - from app.scodoc import sco_formations - cnx = ndb.GetDBConnexion() r = _moduleEditor.create(cnx, args) # news formation = Formation.query.get(args["formation_id"]) - sco_news.add( - typ=sco_news.NEWS_FORM, - object=formation.id, + ScolarNews.add( + typ=ScolarNews.NEWS_FORM, + obj=formation.id, text=f"Modification de la formation {formation.acronyme}", - max_frequency=3, + max_frequency=10 * 60, ) formation.invalidate_cached_sems() return r @@ -396,11 +394,11 @@ def do_module_delete(oid): # news formation = module.formation - sco_news.add( - typ=sco_news.NEWS_FORM, - object=mod["formation_id"], + ScolarNews.add( + typ=ScolarNews.NEWS_FORM, + obj=mod["formation_id"], text=f"Modification de la formation {formation.acronyme}", - max_frequency=3, + max_frequency=10 * 60, ) formation.invalidate_cached_sems() diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py index ffe4d64fc..c6811d454 100644 --- a/app/scodoc/sco_edit_ue.py +++ b/app/scodoc/sco_edit_ue.py @@ -37,6 +37,7 @@ from app import db from app import log from app.models import APO_CODE_STR_LEN, SHORT_STR_LEN from app.models import Formation, UniteEns, ModuleImpl, Module +from app.models import ScolarNews from app.models.formations import Matiere import app.scodoc.notesdb as ndb import app.scodoc.sco_utils as scu @@ -55,15 +56,11 @@ from app.scodoc import html_sco_header from app.scodoc import sco_cache from app.scodoc import sco_codes_parcours from app.scodoc import sco_edit_apc -from app.scodoc import sco_edit_formation from app.scodoc import sco_edit_matiere from app.scodoc import sco_edit_module -from app.scodoc import sco_etud from app.scodoc import sco_formsemestre from app.scodoc import sco_groups from app.scodoc import sco_moduleimpl -from app.scodoc import sco_news -from app.scodoc import sco_permissions from app.scodoc import sco_preferences from app.scodoc import sco_tag_module @@ -138,11 +135,11 @@ def do_ue_create(args): ue = UniteEns.query.get(ue_id) flash(f"UE créée (code {ue.ue_code})") formation = Formation.query.get(args["formation_id"]) - sco_news.add( - typ=sco_news.NEWS_FORM, - object=args["formation_id"], + ScolarNews.add( + typ=ScolarNews.NEWS_FORM, + obj=args["formation_id"], text=f"Modification de la formation {formation.acronyme}", - max_frequency=3, + max_frequency=10 * 60, ) formation.invalidate_cached_sems() return ue_id @@ -222,11 +219,11 @@ def do_ue_delete(ue_id, delete_validations=False, force=False): sco_cache.invalidate_formsemestre() # news F = sco_formations.formation_list(args={"formation_id": ue.formation_id})[0] - sco_news.add( - typ=sco_news.NEWS_FORM, - object=ue.formation_id, - text="Modification de la formation %(acronyme)s" % F, - max_frequency=3, + ScolarNews.add( + typ=ScolarNews.NEWS_FORM, + obj=ue.formation_id, + text=f"Modification de la formation {F['acronyme']}", + max_frequency=10 * 60, ) # if not force: diff --git a/app/scodoc/sco_etape_apogee_view.py b/app/scodoc/sco_etape_apogee_view.py index 21fc2410f..f0ceecc92 100644 --- a/app/scodoc/sco_etape_apogee_view.py +++ b/app/scodoc/sco_etape_apogee_view.py @@ -34,8 +34,6 @@ from zipfile import ZipFile import flask from flask import url_for, g, send_file, request -# from werkzeug.utils import send_file - import app.scodoc.sco_utils as scu from app import log from app.scodoc import html_sco_header diff --git a/app/scodoc/sco_etud.py b/app/scodoc/sco_etud.py index 710f85bc4..598ec96b0 100644 --- a/app/scodoc/sco_etud.py +++ b/app/scodoc/sco_etud.py @@ -637,7 +637,7 @@ def create_etud(cnx, args={}): Returns: etud, l'étudiant créé. """ - from app.scodoc import sco_news + from app.models import ScolarNews # creation d'un etudiant etudid = etudident_create(cnx, args) @@ -671,9 +671,8 @@ def create_etud(cnx, args={}): etud = etudident_list(cnx, {"etudid": etudid})[0] fill_etuds_info([etud]) etud["url"] = url_for("scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=etudid) - sco_news.add( - typ=sco_news.NEWS_INSCR, - object=None, # pas d'object pour ne montrer qu'un etudiant + ScolarNews.add( + typ=ScolarNews.NEWS_INSCR, text='Nouvel étudiant %(nomprenom)s' % etud, url=etud["url"], ) diff --git a/app/scodoc/sco_evaluation_check_abs.py b/app/scodoc/sco_evaluation_check_abs.py index b8287c25d..b464cd673 100644 --- a/app/scodoc/sco_evaluation_check_abs.py +++ b/app/scodoc/sco_evaluation_check_abs.py @@ -25,7 +25,7 @@ # ############################################################################## -"""Vérification des abasneces à une évaluation +"""Vérification des absences à une évaluation """ from flask import url_for, g diff --git a/app/scodoc/sco_evaluation_db.py b/app/scodoc/sco_evaluation_db.py index 59dbc5fe8..7aec90889 100644 --- a/app/scodoc/sco_evaluation_db.py +++ b/app/scodoc/sco_evaluation_db.py @@ -28,7 +28,6 @@ """Gestion evaluations (ScoDoc7, sans SQlAlchemy) """ -import datetime import pprint import flask @@ -37,6 +36,7 @@ from flask_login import current_user from app import log +from app.models import ScolarNews from app.models.evaluations import evaluation_enrich_dict, check_evaluation_args import app.scodoc.sco_utils as scu import app.scodoc.notesdb as ndb @@ -44,9 +44,7 @@ from app.scodoc.sco_exceptions import AccessDenied, ScoValueError from app.scodoc import sco_cache from app.scodoc import sco_edit_module -from app.scodoc import sco_formsemestre from app.scodoc import sco_moduleimpl -from app.scodoc import sco_news from app.scodoc import sco_permissions_check @@ -179,9 +177,9 @@ def do_evaluation_create( mod = sco_edit_module.module_list(args={"module_id": M["module_id"]})[0] mod["moduleimpl_id"] = M["moduleimpl_id"] mod["url"] = "Notes/moduleimpl_status?moduleimpl_id=%(moduleimpl_id)s" % mod - sco_news.add( - typ=sco_news.NEWS_NOTE, - object=moduleimpl_id, + ScolarNews.add( + typ=ScolarNews.NEWS_NOTE, + obj=moduleimpl_id, text='Création d\'une évaluation dans %(titre)s' % mod, url=mod["url"], ) @@ -240,9 +238,9 @@ def do_evaluation_delete(evaluation_id): mod["url"] = ( scu.NotesURL() + "/moduleimpl_status?moduleimpl_id=%(moduleimpl_id)s" % mod ) - sco_news.add( - typ=sco_news.NEWS_NOTE, - object=moduleimpl_id, + ScolarNews.add( + typ=ScolarNews.NEWS_NOTE, + obj=moduleimpl_id, text='Suppression d\'une évaluation dans %(titre)s' % mod, url=mod["url"], ) diff --git a/app/scodoc/sco_evaluation_recap.py b/app/scodoc/sco_evaluation_recap.py new file mode 100644 index 000000000..7f1120b78 --- /dev/null +++ b/app/scodoc/sco_evaluation_recap.py @@ -0,0 +1,148 @@ +############################################################################## +# ScoDoc +# Copyright (c) 1999 - 2022 Emmanuel Viennet. All rights reserved. +# See LICENSE +############################################################################## + +"""Tableau recap. de toutes les évaluations d'un semestre +avec leur état. + +Sur une idée de Pascal Bouron, de Lyon. +""" +import time +from flask import g, url_for + +from app.models import Evaluation, FormSemestre +from app.comp import res_sem +from app.comp.res_compat import NotesTableCompat +from app.comp.moy_mod import ModuleImplResults +from app.scodoc import html_sco_header +import app.scodoc.sco_utils as scu + + +def evaluations_recap(formsemestre_id: int) -> str: + """Page récap. de toutes les évaluations d'un semestre""" + formsemestre: FormSemestre = FormSemestre.query.get_or_404(formsemestre_id) + rows, titles = evaluations_recap_table(formsemestre) + column_ids = titles.keys() + filename = scu.sanitize_filename( + f"""evaluations-{formsemestre.titre_num()}-{time.strftime("%Y-%m-%d")}""" + ) + if not rows: + return ''
+ H = [
+ html_sco_header.sco_header(
+ page_title="Évaluations du semestre",
+ javascripts=["js/evaluations_recap.js"],
+ ),
+ f"""
Les étudiants démissionnaires ou défaillants ne sont pas pris en compte dans cette table.
+ """
+ )
+ H.append(
+ html_sco_header.sco_footer(),
+ )
+ return "".join(H)
+
+
+def evaluations_recap_table(formsemestre: FormSemestre) -> list[dict]:
+ """Tableau recap. de toutes les évaluations d'un semestre
+ Colonnes:
+ - code (UE ou module),
+ - titre
+ - complete
+ - publiée
+ - inscrits (non dem. ni def.)
+ - nb notes manquantes
+ - nb ATT
+ - nb ABS
+ - nb EXC
+ """
+ rows = []
+ titles = {
+ "type": "",
+ "code": "Code",
+ "titre": "",
+ "date": "Date",
+ "complete": "Comptée",
+ "inscrits": "Inscrits",
+ "manquantes": "Manquantes", # notes eval non entrées
+ "nb_abs": "ABS",
+ "nb_att": "ATT",
+ "nb_exc": "EXC",
+ }
+ nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre)
+ line_idx = 0
+ for modimpl in nt.formsemestre.modimpls_sorted:
+ modimpl_results: ModuleImplResults = nt.modimpls_results[modimpl.id]
+ row = {
+ "type": modimpl.module.type_abbrv().upper(),
+ "_type_order": f"{line_idx:04d}",
+ "code": modimpl.module.code,
+ "_code_target": url_for(
+ "notes.moduleimpl_status",
+ scodoc_dept=g.scodoc_dept,
+ moduleimpl_id=modimpl.id,
+ ),
+ "titre": modimpl.module.titre,
+ "_titre_class": "titre",
+ "inscrits": modimpl_results.nb_inscrits_module,
+ "date": "-",
+ "_date_order": "",
+ "_tr_class": f"module {modimpl.module.type_abbrv()}",
+ }
+ rows.append(row)
+ line_idx += 1
+ for evaluation_id in modimpl_results.evals_notes:
+ e = Evaluation.query.get(evaluation_id)
+ eval_etat = modimpl_results.evaluations_etat[evaluation_id]
+ row = {
+ "type": "",
+ "_type_order": f"{line_idx:04d}",
+ "titre": e.description or "sans titre",
+ "_titre_target": url_for(
+ "notes.evaluation_listenotes",
+ scodoc_dept=g.scodoc_dept,
+ evaluation_id=evaluation_id,
+ ),
+ "_titre_target_attrs": 'class="discretelink"',
+ "date": e.jour.strftime("%d/%m/%Y") if e.jour else "",
+ "_date_order": e.jour.isoformat() if e.jour else "",
+ "complete": "oui" if eval_etat.is_complete else "non",
+ "_complete_target": "#",
+ "_complete_target_attrs": 'class="bull_link" title="prise en compte dans les moyennes"'
+ if eval_etat.is_complete
+ else 'class="bull_link incomplete" title="il manque des notes"',
+ "manquantes": len(modimpl_results.evals_etudids_sans_note[e.id]),
+ "inscrits": modimpl_results.nb_inscrits_module,
+ "nb_abs": sum(modimpl_results.evals_notes[e.id] == scu.NOTES_ABSENCE),
+ "nb_att": eval_etat.nb_attente,
+ "nb_exc": sum(
+ modimpl_results.evals_notes[e.id] == scu.NOTES_NEUTRALISE
+ ),
+ "_tr_class": "evaluation"
+ + (" incomplete" if not eval_etat.is_complete else ""),
+ }
+ rows.append(row)
+ line_idx += 1
+
+ return rows, titles
diff --git a/app/scodoc/sco_evaluations.py b/app/scodoc/sco_evaluations.py
index 43b585907..6152e8810 100644
--- a/app/scodoc/sco_evaluations.py
+++ b/app/scodoc/sco_evaluations.py
@@ -36,32 +36,27 @@ from flask import g
from flask_login import current_user
from flask import request
-from app import log
-
from app.comp import res_sem
from app.comp.res_compat import NotesTableCompat
from app.models import FormSemestre
+from app.models import ScolarNews
import app.scodoc.sco_utils as scu
from app.scodoc.sco_utils import ModuleType
import app.scodoc.notesdb as ndb
-from app.scodoc.sco_exceptions import AccessDenied, ScoValueError
-import sco_version
from app.scodoc.gen_tables import GenTable
from app.scodoc import html_sco_header
from app.scodoc import sco_evaluation_db
from app.scodoc import sco_abs
-from app.scodoc import sco_cache
from app.scodoc import sco_edit_module
from app.scodoc import sco_edit_ue
-from app.scodoc import sco_formsemestre
from app.scodoc import sco_formsemestre_inscriptions
from app.scodoc import sco_groups
from app.scodoc import sco_moduleimpl
-from app.scodoc import sco_news
from app.scodoc import sco_permissions_check
from app.scodoc import sco_preferences
from app.scodoc import sco_users
+import sco_version
# --------------------------------------------------------------------
@@ -633,13 +628,16 @@ def evaluation_describe(evaluation_id="", edit_in_place=True):
'voir toutes les notes du module'
% moduleimpl_id
)
- mod_descr = '%s %s (resp. %s) %s' % (
- moduleimpl_id,
- Mod["code"] or "",
- Mod["titre"] or "?",
- nomcomplet,
- resp,
- link,
+ mod_descr = (
+ '%s %s (resp. %s) %s'
+ % (
+ moduleimpl_id,
+ Mod["code"] or "",
+ Mod["titre"] or "?",
+ nomcomplet,
+ resp,
+ link,
+ )
)
etit = E["description"] or ""
diff --git a/app/scodoc/sco_excel.py b/app/scodoc/sco_excel.py
index 978247800..5d1b2d729 100644
--- a/app/scodoc/sco_excel.py
+++ b/app/scodoc/sco_excel.py
@@ -185,13 +185,13 @@ def excel_make_style(
class ScoExcelSheet:
- """Représente une feuille qui peut être indépendante ou intégrée dans un SCoExcelBook.
+ """Représente une feuille qui peut être indépendante ou intégrée dans un ScoExcelBook.
En application des directives de la bibliothèque sur l'écriture optimisée, l'ordre des opérations
est imposé:
* instructions globales (largeur/maquage des colonnes et ligne, ...)
* construction et ajout des cellules et ligne selon le sens de lecture (occidental)
ligne de haut en bas et cellules de gauche à droite (i.e. A1, A2, .. B1, B2, ..)
- * pour finit appel de la méthode de génération
+ * pour finir appel de la méthode de génération
"""
def __init__(self, sheet_name="feuille", default_style=None, wb=None):
@@ -260,7 +260,7 @@ class ScoExcelSheet:
for i, val in enumerate(value):
self.ws.column_dimensions[self.i2col(i)].width = val
# No keys: value is a list of widths
- elif type(cle) == str: # accepts set_column_with("D", ...)
+ elif isinstance(cle, str): # accepts set_column_with("D", ...)
self.ws.column_dimensions[cle].width = value
else:
self.ws.column_dimensions[self.i2col(cle)].width = value
@@ -337,7 +337,8 @@ class ScoExcelSheet:
return cell
- def make_row(self, values: list, style=None, comments=None):
+ def make_row(self, values: list, style=None, comments=None) -> list:
+ "build a row"
# TODO make possible differents styles in a row
if comments is None:
comments = [None] * len(values)
diff --git a/app/scodoc/sco_exceptions.py b/app/scodoc/sco_exceptions.py
index 35d2d9d6e..17d39a4c2 100644
--- a/app/scodoc/sco_exceptions.py
+++ b/app/scodoc/sco_exceptions.py
@@ -63,6 +63,17 @@ class ScoFormatError(ScoValueError):
pass
+class ScoInvalidParamError(ScoValueError):
+ """Paramètres requete invalides.
+ A utilisée lorsqu'une route est appelée avec des paramètres invalides
+ (id strings, ...)
+ """
+
+ def __init__(self, msg=None, dest_url=None):
+ msg = msg or "Adresse invalide. Vérifiez vos signets."
+ super().__init__(msg, dest_url=dest_url)
+
+
class ScoPDFFormatError(ScoValueError):
"erreur génération PDF (templates platypus, ...)"
diff --git a/app/scodoc/sco_find_etud.py b/app/scodoc/sco_find_etud.py
index 9c5630480..91c53cca6 100644
--- a/app/scodoc/sco_find_etud.py
+++ b/app/scodoc/sco_find_etud.py
@@ -53,12 +53,10 @@ def form_search_etud(
):
"form recherche par nom"
H = []
- if title:
- H.append("%s
" % title) H.append( f""""] # Navigation suivant/precedent - if etud_index_prev != None: - etud_p = sco_etud.get_etud_info(etudid=T[etud_index_prev][-1], filled=True)[0] - Footer.append( - 'Etud. précédent (%s)' - % (formsemestre_id, etud_index_prev, etud_p["nomprenom"]) + if etud_index_prev is not None: + etud_prev = Identite.query.get(T[etud_index_prev][-1]) + url_prev = url_for( + "notes.formsemestre_validation_etud_form", + scodoc_dept=g.scodoc_dept, + formsemestre_id=formsemestre_id, + etud_index=etud_index_prev, ) - if etud_index_next != None: - etud_n = sco_etud.get_etud_info(etudid=T[etud_index_next][-1], filled=True)[0] - Footer.append( - 'Etud. suivant (%s)' - % (formsemestre_id, etud_index_next, etud_n["nomprenom"]) + else: + url_prev = None + if etud_index_next is not None: + etud_next = Identite.query.get(T[etud_index_next][-1]) + url_next = url_for( + "notes.formsemestre_validation_etud_form", + scodoc_dept=g.scodoc_dept, + formsemestre_id=formsemestre_id, + etud_index=etud_index_next, ) - Footer.append("
") - Footer.append(html_sco_header.sco_footer()) + else: + url_next = None + footer = ["""") + + footer.append(html_sco_header.sco_footer()) H.append('| ')
if not check:
@@ -171,7 +200,7 @@ def formsemestre_validation_etud_form(
"""
)
)
- return "\n".join(H + Footer)
+ return "\n".join(H + footer)
H.append(
formsemestre_recap_parcours_table(
@@ -180,21 +209,10 @@ def formsemestre_validation_etud_form(
)
if check:
if not desturl:
- desturl = url_for(
- "notes.formsemestre_recapcomplet",
- scodoc_dept=g.scodoc_dept,
- modejury=1,
- hidemodules=1,
- hidebac=1,
- pref_override=0,
- formsemestre_id=formsemestre_id,
- sortcol=sortcol
- or None, # pour refaire tri sorttable du tableau de notes
- _anchor="etudid%s" % etudid, # va a la bonne ligne
- )
+ desturl = url_tableau
H.append(f'')
- return "\n".join(H + Footer)
+ return "\n".join(H + footer)
decision_jury = Se.nt.get_etud_decision_sem(etudid)
@@ -210,7 +228,7 @@ def formsemestre_validation_etud_form(
"""
)
)
- return "\n".join(H + Footer)
+ return "\n".join(H + footer)
# Infos si pas de semestre précédent
if not Se.prev:
@@ -348,7 +366,7 @@ def formsemestre_validation_etud_form(
else:
H.append("sans semestres décalés")
- return "".join(H + Footer)
+ return "".join(H + footer)
def formsemestre_validation_etud(
@@ -437,14 +455,6 @@ def _redirect_valid_choice(formsemestre_id, etudid, Se, choice, desturl, sortcol
# sinon renvoie au listing general,
-# if choice.new_code_prev:
-# flask.redirect( 'formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s&check=1&desturl=%s' % (formsemestre_id, etudid, desturl) )
-# else:
-# if not desturl:
-# desturl = 'formsemestre_recapcomplet?modejury=1&hidemodules=1&formsemestre_id=' + str(formsemestre_id)
-# flask.redirect(desturl)
-
-
def _dispcode(c):
if not c:
return ""
@@ -948,19 +958,23 @@ def do_formsemestre_validation_auto(formsemestre_id):
)
if conflicts:
H.append(
- """ Attention: %d étudiants non modifiés car décisions différentes - déja saisies :
Attention: {len(conflicts)} étudiants non modifiés + car décisions différentes déja saisies : +
Dernières opérations']
- H.append(' ")
- return "\n".join(H)
-
-
-def _send_news_by_mail(n):
- """Notify by email"""
- infos = _get_formsemestre_infos_from_news(n)
- formsemestre_id = infos.get("formsemestre_id", None)
- prefs = sco_preferences.SemPreferences(formsemestre_id=formsemestre_id)
- destinations = prefs["emails_notifications"] or ""
- destinations = [x.strip() for x in destinations.split(",")]
- destinations = [x for x in destinations if x]
- if not destinations:
- return
- #
- txt = n["text"]
- if infos:
- txt += "\n\nSemestre %(titremois)s\n\n" % infos["sem"]
- txt += (
- """%(descr_sem)s
- """
- % infos
- )
- txt += "\n\nEffectué par: %(nomcomplet)s\n" % n["user_info"]
-
- txt = (
- "\n"
- + txt
- + """\n
---- Ceci est un message de notification automatique issu de ScoDoc
---- vous recevez ce message car votre adresse est indiquée dans les paramètres de ScoDoc.
-"""
- )
-
- # Transforme les URL en URL absolue
- base = scu.ScoURL()
- txt = re.sub('href=.*?"', 'href="' + base + "/", txt)
-
- # Transforme les liens HTML en texte brut: 'texte' devient 'texte: url'
- # (si on veut des messages non html)
- txt = re.sub(r'
- Pour être informé des évolutions de ScoDoc,
- vous pouvez vous
-
- abonner à la liste de diffusion.
-
- """
- % scu.SCO_ANNONCES_WEBSITE
- )
-
- H.append("") - if modejury: - H.append( - """Calcul automatique des décisions du jury """ - % (formsemestre_id,) - ) - else: - H.append( - """Saisie des décisions du jury""" - % formsemestre_id - ) - H.append("") - if sco_preferences.get_preference("use_ue_coefs", formsemestre_id): + if len(formsemestre.inscriptions) > 0: + H.append("") + H.append( + f"""""" + ) + if sco_permissions_check.can_validate_sem(formsemestre_id): + H.append("") + if modejury: H.append( - """ - utilise les coefficients d'UE pour calculer la moyenne générale. - """ + f"""Calcul automatique des décisions du jury""" ) - H.append(html_sco_header.sco_footer()) + else: + H.append( + f"""Saisie des décisions du jury""" + ) + H.append("") + if sco_preferences.get_preference("use_ue_coefs", formsemestre_id): + H.append( + """ +utilise les coefficients d'UE pour calculer la moyenne générale. + """ + ) + H.append(html_sco_header.sco_footer()) # HTML or binary data ? if len(H) > 1: return "".join(H) @@ -206,697 +185,73 @@ def formsemestre_recapcomplet( return H -def do_formsemestre_recapcomplet( +def _do_formsemestre_recapcomplet( formsemestre_id=None, format="html", # html, xml, xls, xlsall, json - hidemodules=False, # ne pas montrer les modules (ignoré en XML) - hidebac=False, # pas de colonne Bac (ignoré en XML) xml_nodate=False, # format XML sans dates (sert pour debug cache: comparaison de XML) modejury=False, # saisie décisions jury sortcol=None, # indice colonne a trier dans table T xml_with_decisions=False, - disable_etudlink=False, - rank_partition_id=None, # si None, calcul rang global force_publishing=True, + selected_etudid=None, ): """Calcule et renvoie le tableau récapitulatif.""" formsemestre = FormSemestre.query.get_or_404(formsemestre_id) - if format == "html" and not modejury: + + filename = scu.sanitize_filename( + f"""recap-{formsemestre.titre_num()}-{time.strftime("%Y-%m-%d")}""" + ) + + if format == "html" or format == "evals": res: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) - data, filename = gen_formsemestre_recapcomplet_html(formsemestre, res) - else: - data, filename, format = make_formsemestre_recapcomplet( - formsemestre_id=formsemestre_id, - format=format, - hidemodules=hidemodules, - hidebac=hidebac, - xml_nodate=xml_nodate, + data = gen_formsemestre_recapcomplet_html( + formsemestre, + res, + include_evaluations=(format == "evals"), modejury=modejury, - sortcol=sortcol, - xml_with_decisions=xml_with_decisions, - disable_etudlink=disable_etudlink, - rank_partition_id=rank_partition_id, - force_publishing=force_publishing, + filename=filename, + selected_etudid=selected_etudid, ) - # --- - if format == "xml" or format == "html": return data - elif format == "csv": - return scu.send_file(data, filename=filename, mime=scu.CSV_MIMETYPE) - elif format.startswith("xls") or format.startswith("xlsx"): - return scu.send_file(data, filename=filename, mime=scu.XLSX_MIMETYPE) - elif format == "json": - js = json.dumps(data, indent=1, cls=scu.ScoDocJSONEncoder) - return scu.send_file( - js, filename=filename, suffix=scu.JSON_SUFFIX, mime=scu.JSON_MIMETYPE + elif format.startswith("xls") or format == "csv": + res: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) + include_evaluations = format in {"xlsall", "csv "} + if format != "csv": + format = "xlsx" + data, filename = gen_formsemestre_recapcomplet_excel( + formsemestre, + res, + include_evaluations=include_evaluations, + format=format, + filename=filename, ) - else: - raise ValueError("unknown format %s" % format) - - -def make_formsemestre_recapcomplet( - formsemestre_id=None, - format="html", # html, xml, xls, xlsall, json - hidemodules=False, # ne pas montrer les modules (ignoré en XML) - hidebac=False, # pas de colonne Bac (ignoré en XML) - xml_nodate=False, # format XML sans dates (sert pour debug cache: comparaison de XML) - modejury=False, # saisie décisions jury - sortcol=None, # indice colonne a trier dans table T - xml_with_decisions=False, - disable_etudlink=False, - rank_partition_id=None, # si None, calcul rang global - force_publishing=True, # donne bulletins JSON/XML meme si non publiés -): - """Grand tableau récapitulatif avec toutes les notes de modules - pour tous les étudiants, les moyennes par UE et générale, - trié par moyenne générale décroissante. - """ - civ_nom_prenom = False # 3 colonnes différentes ou une seule avec prénom abrégé ? - if format == "xml": - return _formsemestre_recapcomplet_xml( + return scu.send_file(data, filename=filename, mime=scu.get_mime_suffix(format)) + elif format == "xml": + data = gen_formsemestre_recapcomplet_xml( formsemestre_id, xml_nodate, xml_with_decisions=xml_with_decisions, force_publishing=force_publishing, ) + return scu.send_file(data, filename=filename, suffix=scu.XML_SUFFIX) elif format == "json": - return _formsemestre_recapcomplet_json( + data = gen_formsemestre_recapcomplet_json( formsemestre_id, xml_nodate=xml_nodate, xml_with_decisions=xml_with_decisions, force_publishing=force_publishing, ) - if format[:3] == "xls": - civ_nom_prenom = True # 3 cols: civilite, nom, prenom - keep_numeric = True # pas de conversion des notes en strings - else: - keep_numeric = False + return scu.sendJSON(data, filename=filename) - if hidebac: - admission_extra_cols = [] - else: - admission_extra_cols = [ - "type_admission", - "classement", - "apb_groupe", - "apb_classement_gr", - ] - - formsemestre = FormSemestre.query.get_or_404(formsemestre_id) - # A ré-écrire XXX - sem = sco_formsemestre.do_formsemestre_list( - args={"formsemestre_id": formsemestre_id} - )[0] - parcours = formsemestre.formation.get_parcours() - - nt: NotesTableCompat = res_sem.load_formsemestre_results(formsemestre) - modimpls = formsemestre.modimpls_sorted - ues = nt.get_ues_stat_dict() # incluant le(s) UE de sport - - partitions, partitions_etud_groups = sco_groups.get_formsemestre_groups( - formsemestre_id - ) - if rank_partition_id and format == "html": - # Calcul rang sur une partition et non sur l'ensemble - # seulement en format HTML (car colonnes rangs toujours presentes en xls) - rank_partition = sco_groups.get_partition(rank_partition_id) - rank_label = "Rg (%s)" % rank_partition["partition_name"] - else: - rank_partition = sco_groups.get_default_partition(formsemestre_id) - rank_label = "Rg" - - T = nt.get_table_moyennes_triees() - if not T: - return "", "", format - - # Construit une liste de listes de chaines: le champs du tableau resultat (HTML ou CSV) - F = [] - h = [rank_label] - if civ_nom_prenom: - h += ["Civilité", "Nom", "Prénom"] - else: - h += ["Nom"] - if not hidebac: - h.append("Bac") - - # Si CSV ou XLS, indique tous les groupes - if format[:3] == "xls" or format == "csv": - for partition in partitions: - h.append("%s" % partition["partition_name"]) - else: - h.append("Gr") - - h.append("Moy") - # Ajoute rangs dans groupe seulement si CSV ou XLS - if format[:3] == "xls" or format == "csv": - for partition in partitions: - h.append("rang_%s" % partition["partition_name"]) - - cod2mod = {} # code : moduleimpl - mod_evals = {} # moduleimpl_id : liste de toutes les evals de ce module - for ue in ues: - if ue["type"] != UE_SPORT: - h.append(ue["acronyme"]) - else: # UE_SPORT: - # n'affiche pas la moyenne d'UE dans ce cas - # mais laisse col. vide si modules affichés (pour séparer les UE) - if not hidemodules: - h.append("") - pass - if not hidemodules and not ue["is_external"]: - for modimpl in modimpls: - if modimpl.module.ue_id == ue["ue_id"]: - code = modimpl.module.code - h.append(code) - cod2mod[code] = modimpl # pour fabriquer le lien - if format == "xlsall": - evals = nt.modimpls_results[ - modimpl.id - ].get_evaluations_completes(modimpl) - # evals = nt.get_mod_evaluation_etat_list(... - mod_evals[modimpl.id] = evals - h += _list_notes_evals_titles(code, evals) - - h += admission_extra_cols - h += ["code_nip", "etudid"] - F.append(h) - - def fmtnum(val): # conversion en nombre pour cellules excel - if keep_numeric: - try: - return float(val) - except: - return val - else: - return val - - # Compte les decisions de jury - codes_nb = scu.DictDefault(defaultvalue=0) - # - is_dem = {} # etudid : bool - for t in T: - etudid = t[-1] - dec = nt.get_etud_decision_sem(etudid) - if dec: - codes_nb[dec["code"]] += 1 - etud_etat = nt.get_etud_etat(etudid) - if etud_etat == "D": - gr_name = "Dém." - is_dem[etudid] = True - elif etud_etat == DEF: - gr_name = "Déf." - is_dem[etudid] = False - else: - group = sco_groups.get_etud_main_group(etudid, formsemestre_id) - gr_name = group["group_name"] or "" - is_dem[etudid] = False - if rank_partition_id: - rang_gr, _, rank_gr_name = sco_bulletins.get_etud_rangs_groups( - etudid, formsemestre_id, partitions, partitions_etud_groups, nt - ) - if rank_gr_name[rank_partition_id]: - rank = "%s %s" % ( - rank_gr_name[rank_partition_id], - rang_gr[rank_partition_id], - ) - else: - rank = "" - else: - rank = nt.get_etud_rang(etudid) - - e = nt.identdict[etudid] - if civ_nom_prenom: - sco_etud.format_etud_ident(e) - l = [rank, e["civilite_str"], e["nom_disp"], e["prenom"]] # civ, nom prenom - else: - l = [rank, nt.get_nom_short(etudid)] # rang, nom, - - e["admission"] = {} - if not hidebac: - e["admission"] = nt.etuds_dict[etudid].admission.first() - if e["admission"]: - bac = nt.etuds_dict[etudid].admission[0].get_bac() - l.append(bac.abbrev()) - else: - l.append("") - - if format[:3] == "xls" or format == "csv": # tous les groupes - for partition in partitions: - group = partitions_etud_groups[partition["partition_id"]].get( - etudid, None - ) - if group: - l.append(group["group_name"]) - else: - l.append("") - else: - l.append(gr_name) # groupe - - # Moyenne générale - l.append(fmtnum(scu.fmt_note(t[0], keep_numeric=keep_numeric))) - # Ajoute rangs dans groupes seulement si CSV ou XLS - if format[:3] == "xls" or format == "csv": - rang_gr, _, gr_name = sco_bulletins.get_etud_rangs_groups( - etudid, formsemestre_id, partitions, partitions_etud_groups, nt - ) - - for partition in partitions: - l.append(rang_gr[partition["partition_id"]]) - - # Nombre d'UE au dessus de 10 - # t[i] est une chaine :-) - # nb_ue_ok = sum( - # [t[i] > 10 for i, ue in enumerate(ues, start=1) if ue["type"] != UE_SPORT] - # ) - ue_index = [] # indices des moy UE dans l (pour appliquer style css) - for i, ue in enumerate(ues, start=1): - if ue["type"] != UE_SPORT: - l.append( - fmtnum(scu.fmt_note(t[i], keep_numeric=keep_numeric)) - ) # moyenne etud dans ue - else: # UE_SPORT: - # n'affiche pas la moyenne d'UE dans ce cas - if not hidemodules: - l.append("") - ue_index.append(len(l) - 1) - if not hidemodules and not ue["is_external"]: - j = 0 - for modimpl in modimpls: - if modimpl.module.ue_id == ue["ue_id"]: - l.append( - fmtnum( - scu.fmt_note( - t[j + len(ues) + 1], keep_numeric=keep_numeric - ) - ) - ) # moyenne etud dans module - if format == "xlsall": - l += _list_notes_evals(mod_evals[modimpl.id], etudid) - j += 1 - if not hidebac: - for k in admission_extra_cols: - l.append(getattr(e["admission"], k, "") or "") - l.append( - nt.identdict[etudid]["code_nip"] or "" - ) # avant-derniere colonne = code_nip - l.append(etudid) # derniere colonne = etudid - F.append(l) - - # Dernière ligne: moyennes, min et max des UEs et modules - if not hidemodules: # moy/min/max dans chaque module - mods_stats = {} # moduleimpl_id : stats - for modimpl in modimpls: - mods_stats[modimpl.id] = nt.get_mod_stats(modimpl.id) - - def add_bottom_stat(key, title, corner_value=""): - l = ["", title] - if civ_nom_prenom: - l += ["", ""] - if not hidebac: - l.append("") - if format[:3] == "xls" or format == "csv": - l += [""] * len(partitions) - else: - l += [""] - l.append(corner_value) - if format[:3] == "xls" or format == "csv": - for _ in partitions: - l += [""] # rangs dans les groupes - for ue in ues: - if ue["type"] != UE_SPORT: - if key == "nb_valid_evals": - l.append("") - elif key == "coef": - if sco_preferences.get_preference("use_ue_coefs", formsemestre_id): - l.append("%2.3f" % ue["coefficient"]) - else: - l.append("") - else: - if key == "ects": - if keep_numeric: - l.append(ue[key]) - else: - l.append(str(ue[key])) - else: - l.append(scu.fmt_note(ue[key], keep_numeric=keep_numeric)) - else: # UE_SPORT: - # n'affiche pas la moyenne d'UE dans ce cas - if not hidemodules: - l.append("") - # ue_index.append(len(l) - 1) - if not hidemodules and not ue["is_external"]: - for modimpl in modimpls: - if modimpl.module.ue_id == ue["ue_id"]: - if key == "coef": - coef = modimpl.module.coefficient - if format[:3] != "xls": - coef = str(coef) - l.append(coef) - elif key == "ects": - l.append("") # ECTS module ? - else: - val = mods_stats[modimpl.id][key] - if key == "nb_valid_evals": - if ( - format[:3] != "xls" - ): # garde val numerique pour excel - val = str(val) - else: # moyenne du module - val = scu.fmt_note(val, keep_numeric=keep_numeric) - l.append(val) - - if format == "xlsall": - l += _list_notes_evals_stats(mod_evals[modimpl.id], key) - if modejury: - l.append("") # case vide sur ligne "Moyennes" - - l += [""] * len(admission_extra_cols) # infos admission vides ici - F.append(l + ["", ""]) # ajoute cellules code_nip et etudid inutilisees ici - - add_bottom_stat( - "min", "Min", corner_value=scu.fmt_note(nt.moy_min, keep_numeric=keep_numeric) - ) - add_bottom_stat( - "max", "Max", corner_value=scu.fmt_note(nt.moy_max, keep_numeric=keep_numeric) - ) - add_bottom_stat( - "moy", - "Moyennes", - corner_value=scu.fmt_note(nt.moy_moy, keep_numeric=keep_numeric), - ) - add_bottom_stat("coef", "Coef") - add_bottom_stat("nb_valid_evals", "Nb évals") - add_bottom_stat("ects", "ECTS") - - # Génération de la table au format demandé - if format == "html": - # Table format HTML - H = [ - """ - -
Décisions du jury
Pour les formations par compétences (comme le BUT), la moyenne générale est purement indicative et ne devrait pas être communiquée aux étudiants. """ - ) - return "\n".join(H), "", "html" - elif format == "csv": - CSV = scu.CSV_LINESEP.join( - [scu.CSV_FIELDSEP.join([str(x) for x in l]) for l in F] - ) - semname = sem["titre_num"].replace(" ", "_") - date = time.strftime("%d-%m-%Y") - filename = "notes_modules-%s-%s.csv" % (semname, date) - return CSV, filename, "csv" - elif format[:3] == "xls": - semname = sem["titre_num"].replace(" ", "_") - date = time.strftime("%d-%m-%Y") - if format == "xls": - filename = "notes_modules-%s-%s%s" % (semname, date, scu.XLSX_SUFFIX) - else: - filename = "notes_modules_evals-%s-%s%s" % (semname, date, scu.XLSX_SUFFIX) - sheet_name = "notes %s %s" % (semname, date) - if len(sheet_name) > 31: - sheet_name = "notes %s %s" % ("...", date) - xls = sco_excel.excel_simple_table( - titles=["etudid", "code_nip"] + F[0][:-2], - lines=[ - [x[-1], x[-2]] + x[:-2] for x in F[1:] - ], # reordonne cols (etudid et nip en 1er), - sheet_name=sheet_name, - ) - return xls, filename, "xls" - else: - raise ValueError("unknown format %s" % format) + raise ScoValueError(f"Format demandé invalide: {format}") -def _ligne_titres(ue_index, F, cod2mod, modejury, with_modules_links=True): - """Cellules de la ligne de titre (haut ou bas)""" - cells = '{F[0][i]} | """
- else:
- cells += f"{F[0][i]} | "
- if modejury:
- cells += 'Décision | '
- return cells + "... incluant le tableau.
"""
- rows, footer_rows, titles, column_ids = res.get_table_recap(convert_values=True)
+ table_html = None
+ if not (modejury or selected_etudid):
+ if include_evaluations:
+ table_html = sco_cache.TableRecapWithEvalsCache.get(formsemestre.id)
+ else:
+ table_html = sco_cache.TableRecapCache.get(formsemestre.id)
+ if modejury or (table_html is None):
+ table_html = _gen_formsemestre_recapcomplet_html(
+ formsemestre,
+ res,
+ include_evaluations,
+ modejury,
+ filename,
+ selected_etudid=selected_etudid,
+ )
+ if not modejury:
+ if include_evaluations:
+ sco_cache.TableRecapWithEvalsCache.set(formsemestre.id, table_html)
+ else:
+ sco_cache.TableRecapCache.set(formsemestre.id, table_html)
+
+ return table_html
+
+
+def _gen_formsemestre_recapcomplet_html(
+ formsemestre: FormSemestre,
+ res: NotesTableCompat,
+ include_evaluations=False,
+ modejury=False,
+ filename: str = "",
+ selected_etudid=None,
+) -> str:
+ """Génère le html"""
+ rows, footer_rows, titles, column_ids = res.get_table_recap(
+ convert_values=True, include_evaluations=include_evaluations, modejury=modejury
+ )
if not rows:
return (
- '',
- "",
+ ' '
)
H = [
- f""" |