From 3eb038b491d80f7675d6b8fefa7edc8ca4a5dd64 Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Fri, 7 Jan 2022 15:11:24 +0100 Subject: [PATCH] formsemestre header sans notes_table --- app/models/formsemestre.py | 19 ++++++++ app/scodoc/html_sco_header.py | 2 +- app/scodoc/sco_archives.py | 3 +- app/scodoc/sco_evaluation_check_abs.py | 1 - app/scodoc/sco_evaluations.py | 1 - app/scodoc/sco_formsemestre.py | 2 +- app/scodoc/sco_formsemestre_custommenu.py | 2 +- app/scodoc/sco_formsemestre_edit.py | 8 ++-- app/scodoc/sco_formsemestre_inscriptions.py | 2 - app/scodoc/sco_formsemestre_status.py | 48 ++++++++++----------- app/scodoc/sco_formsemestre_validation.py | 4 +- app/scodoc/sco_inscr_passage.py | 2 +- app/scodoc/sco_preferences.py | 2 +- app/scodoc/sco_pvjury.py | 3 -- app/scodoc/sco_ue_external.py | 1 - app/views/absences.py | 4 +- app/views/notes.py | 8 +--- app/views/scolar.py | 3 +- 18 files changed, 57 insertions(+), 58 deletions(-) diff --git a/app/models/formsemestre.py b/app/models/formsemestre.py index 928dd6093b..a55188b6be 100644 --- a/app/models/formsemestre.py +++ b/app/models/formsemestre.py @@ -8,6 +8,7 @@ from functools import cached_property import flask_sqlalchemy from app import db +from app import log from app.models import APO_CODE_STR_LEN from app.models import SHORT_STR_LEN from app.models import CODE_STR_LEN @@ -158,6 +159,24 @@ class FormSemestre(db.Model): """ return (self.date_debut <= date_debut) and (date_fin <= self.date_fin) + def est_sur_une_annee(self): + """Test si sem est entièrement sur la même année scolaire. + (ce n'est pas obligatoire mais si ce n'est pas le + cas les exports Apogée risquent de mal fonctionner) + Pivot au 1er août. + """ + if self.date_debut > self.date_fin: + log(f"Warning: semestre {self.id} begins after ending !") + annee_debut = self.date_debut.year + if self.date_debut.month < 8: # août + # considere que debut sur l'anne scolaire precedente + annee_debut -= 1 + annee_fin = self.date_fin.year + if self.date_fin.month < 9: + # 9 (sept) pour autoriser un début en sept et une fin en aout + annee_fin -= 1 + return annee_debut == annee_fin + def est_decale(self): """Vrai si semestre "décalé" c'est à dire semestres impairs commençant entre janvier et juin diff --git a/app/scodoc/html_sco_header.py b/app/scodoc/html_sco_header.py index e2b4d0bd65..23a8b8340c 100644 --- a/app/scodoc/html_sco_header.py +++ b/app/scodoc/html_sco_header.py @@ -313,7 +313,7 @@ def sco_footer(): def html_sem_header( - title, sem=None, with_page_header=True, with_h2=True, page_title=None, **args + title, with_page_header=True, with_h2=True, page_title=None, **args ): "Titre d'une page semestre avec lien vers tableau de bord" # sem now unused and thus optional... diff --git a/app/scodoc/sco_archives.py b/app/scodoc/sco_archives.py index b7cdc1f147..159d3305a7 100644 --- a/app/scodoc/sco_archives.py +++ b/app/scodoc/sco_archives.py @@ -405,7 +405,6 @@ def formsemestre_archive(formsemestre_id, group_ids=[]): H = [ html_sco_header.html_sem_header( "Archiver les PV et résultats du semestre", - sem=sem, javascripts=sco_groups_view.JAVASCRIPTS, cssstyles=sco_groups_view.CSSSTYLES, init_qtip=True, @@ -524,7 +523,7 @@ def formsemestre_list_archives(formsemestre_id): } L.append(a) - H = [html_sco_header.html_sem_header("Archive des PV et résultats ", sem)] + H = [html_sco_header.html_sem_header("Archive des PV et résultats ")] if not L: H.append("

aucune archive enregistrée

") else: diff --git a/app/scodoc/sco_evaluation_check_abs.py b/app/scodoc/sco_evaluation_check_abs.py index 78c18045be..5f52ad5e7b 100644 --- a/app/scodoc/sco_evaluation_check_abs.py +++ b/app/scodoc/sco_evaluation_check_abs.py @@ -222,7 +222,6 @@ def formsemestre_check_absences_html(formsemestre_id): H = [ html_sco_header.html_sem_header( "Vérification absences aux évaluations de ce semestre", - sem, ), """

Vérification de la cohérence entre les notes saisies et les absences signalées. Sont listés tous les modules avec des évaluations.
Aucune action n'est effectuée: diff --git a/app/scodoc/sco_evaluations.py b/app/scodoc/sco_evaluations.py index 82f5d13aa6..15b690a3ca 100644 --- a/app/scodoc/sco_evaluations.py +++ b/app/scodoc/sco_evaluations.py @@ -467,7 +467,6 @@ def formsemestre_evaluations_cal(formsemestre_id): H = [ html_sco_header.html_sem_header( "Evaluations du semestre", - sem, cssstyles=["css/calabs.css"], ), '

', diff --git a/app/scodoc/sco_formsemestre.py b/app/scodoc/sco_formsemestre.py index 272c5c3831..629da14565 100644 --- a/app/scodoc/sco_formsemestre.py +++ b/app/scodoc/sco_formsemestre.py @@ -450,7 +450,7 @@ def sem_in_annee_scolaire(sem, year=False): ) -def sem_une_annee(sem): +def sem_une_annee(sem): # XXX deprecated: use FormSemestre.est_sur_une_annee() """Test si sem est entièrement sur la même année scolaire. (ce n'est pas obligatoire mais si ce n'est pas le cas les exports Apogée ne vont pas fonctionner) pivot au 1er août. diff --git a/app/scodoc/sco_formsemestre_custommenu.py b/app/scodoc/sco_formsemestre_custommenu.py index ee18a8ec0a..e879d622ab 100644 --- a/app/scodoc/sco_formsemestre_custommenu.py +++ b/app/scodoc/sco_formsemestre_custommenu.py @@ -84,7 +84,7 @@ def formsemestre_custommenu_edit(formsemestre_id): scu.NotesURL() + "/formsemestre_status?formsemestre_id=%s" % formsemestre_id ) H = [ - html_sco_header.html_sem_header("Modification du menu du semestre ", sem), + html_sco_header.html_sem_header("Modification du menu du semestre "), """

Ce menu, spécifique à chaque semestre, peut être utilisé pour placer des liens vers vos applications préférées.

Procédez en plusieurs fois si vous voulez ajouter plusieurs items.

""", ] diff --git a/app/scodoc/sco_formsemestre_edit.py b/app/scodoc/sco_formsemestre_edit.py index 060289e95f..59e6d924c9 100644 --- a/app/scodoc/sco_formsemestre_edit.py +++ b/app/scodoc/sco_formsemestre_edit.py @@ -96,7 +96,6 @@ def formsemestre_editwithmodules(formsemestre_id): H = [ html_sco_header.html_sem_header( "Modification du semestre", - sem, javascripts=["libjs/AutoSuggest.js"], cssstyles=["css/autosuggest_inquisitor.css"], bodyOnLoad="init_tf_form('')", @@ -900,7 +899,6 @@ def formsemestre_clone(formsemestre_id): H = [ html_sco_header.html_sem_header( "Copie du semestre", - sem, javascripts=["libjs/AutoSuggest.js"], cssstyles=["css/autosuggest_inquisitor.css"], bodyOnLoad="init_tf_form('')", @@ -1240,7 +1238,7 @@ def formsemestre_delete(formsemestre_id): sem = sco_formsemestre.get_formsemestre(formsemestre_id) F = sco_formations.formation_list(args={"formation_id": sem["formation_id"]})[0] H = [ - html_sco_header.html_sem_header("Suppression du semestre", sem), + html_sco_header.html_sem_header("Suppression du semestre"), """
Attention !

A n'utiliser qu'en cas d'erreur lors de la saisie d'une formation. Normalement, un semestre ne doit jamais être supprimé (on perd la mémoire des notes et de tous les événements liés à ce semestre !).

@@ -1519,7 +1517,7 @@ def formsemestre_edit_uecoefs(formsemestre_id, err_ue_id=None):

""" H = [ - html_sco_header.html_sem_header("Coefficients des UE du semestre", sem), + html_sco_header.html_sem_header("Coefficients des UE du semestre"), help, ] # @@ -1631,7 +1629,7 @@ def formsemestre_edit_uecoefs(formsemestre_id, err_ue_id=None): formsemestre_id=formsemestre_id ) # > modif coef UE cap (modifs notes de _certains_ etudiants) - header = html_sco_header.html_sem_header("Coefficients des UE du semestre", sem) + header = html_sco_header.html_sem_header("Coefficients des UE du semestre") return ( header + "\n".join(z) diff --git a/app/scodoc/sco_formsemestre_inscriptions.py b/app/scodoc/sco_formsemestre_inscriptions.py index 94d7539ce5..48d24f519d 100644 --- a/app/scodoc/sco_formsemestre_inscriptions.py +++ b/app/scodoc/sco_formsemestre_inscriptions.py @@ -374,7 +374,6 @@ def formsemestre_inscription_with_modules( H = [ html_sco_header.html_sem_header( "Inscription de %s dans ce semestre" % etud["nomprenom"], - sem, ) ] F = html_sco_header.sco_footer() @@ -802,7 +801,6 @@ def formsemestre_inscrits_ailleurs(formsemestre_id): H = [ html_sco_header.html_sem_header( "Inscriptions multiples parmi les étudiants du semestre ", - sem, ) ] insd = list_inscrits_ailleurs(formsemestre_id) diff --git a/app/scodoc/sco_formsemestre_status.py b/app/scodoc/sco_formsemestre_status.py index 7ca529269a..303cb20007 100644 --- a/app/scodoc/sco_formsemestre_status.py +++ b/app/scodoc/sco_formsemestre_status.py @@ -36,6 +36,7 @@ from flask_login import current_user from app import log from app.models import Module +from app.models.formsemestre import FormSemestre import app.scodoc.sco_utils as scu from app.scodoc.sco_utils import ModuleType import app.scodoc.notesdb as ndb @@ -722,7 +723,7 @@ def formsemestre_description_table(formsemestre_id, with_evals=False): % (request.base_url, formsemestre_id, with_evals), page_title=title, html_title=html_sco_header.html_sem_header( - "Description du semestre", sem, with_page_header=False + "Description du semestre", with_page_header=False ), pdf_title=title, preferences=sco_preferences.SemPreferences(formsemestre_id), @@ -915,34 +916,35 @@ def html_expr_diagnostic(diagnostics): def formsemestre_status_head(formsemestre_id=None, page_title=None): """En-tête HTML des pages "semestre" """ - semlist = sco_formsemestre.do_formsemestre_list( - args={"formsemestre_id": formsemestre_id} - ) - if not semlist: - raise ScoValueError("Session inexistante (elle a peut être été supprimée ?)") - sem = semlist[0] - F = sco_formations.formation_list(args={"formation_id": sem["formation_id"]})[0] - parcours = sco_codes_parcours.get_parcours_from_code(F["type_parcours"]) + sem = FormSemestre.query.get(formsemestre_id) + if not sem: + raise ScoValueError("Semestre inexistant (il a peut être été supprimé ?)") + formation = sem.formation + parcours = formation.get_parcours() page_title = page_title or "Modules de " H = [ html_sco_header.html_sem_header( - page_title, sem, with_page_header=False, with_h2=False + page_title, with_page_header=False, with_h2=False ), f"""") @@ -965,18 +967,16 @@ Il y a des notes en attente ! Le classement des étudiants n'a qu'une valeur ind ) H.append("
Formation: - {F['titre']}""", + {formation.titre} + """, ] - if sem["semestre_id"] >= 0: - H.append(", %s %s" % (parcours.SESSION_NAME, sem["semestre_id"])) - if sem["modalite"]: - H.append(" en %(modalite)s" % sem) - if sem["etapes"]: + if sem.semestre_id >= 0: + H.append(", %s %s" % (parcours.SESSION_NAME, sem.semestre_id)) + if sem.modalite: + H.append(f" en {sem.modalite}") + if sem.etapes: H.append( - "   (étape %s)" - % (sem["etapes_apo_str"] or "-") + f"""   (étape { + sem.etapes_apo_str() or "-" + })""" ) H.append("
") sem_warning = "" - if sem["bul_hide_xml"]: + if sem.bul_hide_xml: sem_warning += "Bulletins non publiés sur le portail. " - if sem["block_moyennes"]: + if sem.block_moyennes: sem_warning += "Calcul des moyennes bloqué !" if sem_warning: H.append('

' + sem_warning + "

") - if sem["semestre_id"] >= 0 and not sco_formsemestre.sem_une_annee(sem): + if sem.semestre_id >= 0 and not sem.est_sur_une_annee(): H.append( '

Attention: ce semestre couvre plusieurs années scolaires !

' ) - # elif sco_preferences.get_preference( 'bul_display_publication', formsemestre_id): - # H.append('

Bulletins publiés sur le portail

') return "".join(H) diff --git a/app/scodoc/sco_formsemestre_validation.py b/app/scodoc/sco_formsemestre_validation.py index 9b40f780a8..36549bb36b 100644 --- a/app/scodoc/sco_formsemestre_validation.py +++ b/app/scodoc/sco_formsemestre_validation.py @@ -847,9 +847,7 @@ def formsemestre_validation_auto(formsemestre_id): "Formulaire saisie automatisee des decisions d'un semestre" sem = sco_formsemestre.get_formsemestre(formsemestre_id) H = [ - html_sco_header.html_sem_header( - "Saisie automatique des décisions du semestre", sem - ), + html_sco_header.html_sem_header("Saisie automatique des décisions du semestre"), """
  • Seuls les étudiants qui obtiennent le semestre seront affectés (code ADM, moyenne générale et diff --git a/app/scodoc/sco_inscr_passage.py b/app/scodoc/sco_inscr_passage.py index d486b2427a..1ce755ccb7 100644 --- a/app/scodoc/sco_inscr_passage.py +++ b/app/scodoc/sco_inscr_passage.py @@ -418,7 +418,7 @@ def build_page( H = [ html_sco_header.html_sem_header( - "Passages dans le semestre", sem, with_page_header=False + "Passages dans le semestre", with_page_header=False ), """
    """ % request.base_url, """ diff --git a/app/scodoc/sco_preferences.py b/app/scodoc/sco_preferences.py index 52105ea435..3dd413f504 100644 --- a/app/scodoc/sco_preferences.py +++ b/app/scodoc/sco_preferences.py @@ -2150,7 +2150,7 @@ class SemPreferences(object): ) # a bug ! sem = sco_formsemestre.get_formsemestre(self.formsemestre_id) H = [ - html_sco_header.html_sem_header("Préférences du semestre", sem), + html_sco_header.html_sem_header("Préférences du semestre"), """

    Les paramètres définis ici ne s'appliqueront qu'à ce semestre.

    Attention: cliquez sur "Enregistrer les modifications" en bas de page pour appliquer vos changements !

    diff --git a/app/scodoc/sco_pvjury.py b/app/scodoc/sco_pvjury.py index 83593225bc..084b1ca0e1 100644 --- a/app/scodoc/sco_pvjury.py +++ b/app/scodoc/sco_pvjury.py @@ -541,7 +541,6 @@ def formsemestre_pvjury(formsemestre_id, format="html", publish=True): H = [ html_sco_header.html_sem_header( "Décisions du jury pour le semestre", - sem, init_qtip=True, javascripts=["js/etud_info.js"], ), @@ -627,7 +626,6 @@ def formsemestre_pvjury_pdf(formsemestre_id, group_ids=[], etudid=None): H = [ html_sco_header.html_sem_header( "Edition du PV de jury %s" % etuddescr, - sem=sem, javascripts=sco_groups_view.JAVASCRIPTS, cssstyles=sco_groups_view.CSSSTYLES, init_qtip=True, @@ -804,7 +802,6 @@ def formsemestre_lettres_individuelles(formsemestre_id, group_ids=[]): H = [ html_sco_header.html_sem_header( "Édition des lettres individuelles", - sem=sem, javascripts=sco_groups_view.JAVASCRIPTS, cssstyles=sco_groups_view.CSSSTYLES, init_qtip=True, diff --git a/app/scodoc/sco_ue_external.py b/app/scodoc/sco_ue_external.py index cd9d7635e8..b90afd0dd2 100644 --- a/app/scodoc/sco_ue_external.py +++ b/app/scodoc/sco_ue_external.py @@ -220,7 +220,6 @@ def external_ue_create_form(formsemestre_id, etudid): H = [ html_sco_header.html_sem_header( "Ajout d'une UE externe pour %(nomprenom)s" % etud, - sem, javascripts=["js/sco_ue_external.js"], ), """

    Cette page permet d'indiquer que l'étudiant a suivi une UE diff --git a/app/views/absences.py b/app/views/absences.py index 96215c4f5c..209a1f183f 100644 --- a/app/views/absences.py +++ b/app/views/absences.py @@ -940,9 +940,7 @@ def EtatAbsencesGr( init_qtip=True, javascripts=["js/etud_info.js"], ), - html_title=html_sco_header.html_sem_header( - "%s" % title, sem, with_page_header=False - ) + html_title=html_sco_header.html_sem_header("%s" % title, with_page_header=False) + "

    Période du %s au %s (nombre de demi-journées)
    " % (debut, fin), base_url="%s&formsemestre_id=%s&debut=%s&fin=%s" % (groups_infos.base_url, formsemestre_id, debut, fin), diff --git a/app/views/notes.py b/app/views/notes.py index da404fdaf7..395c52ce8d 100644 --- a/app/views/notes.py +++ b/app/views/notes.py @@ -969,7 +969,6 @@ def edit_moduleimpl_resp(moduleimpl_id): html_sco_header.html_sem_header( 'Modification du responsable du module %s' % (moduleimpl_id, M["module"]["titre"]), - sem, javascripts=["libjs/AutoSuggest.js"], cssstyles=["css/autosuggest_inquisitor.css"], bodyOnLoad="init_tf_form('')", @@ -1090,7 +1089,6 @@ def edit_moduleimpl_expr(moduleimpl_id): html_sco_header.html_sem_header( 'Modification règle de calcul du module %s' % (moduleimpl_id, M["module"]["titre"]), - sem, ), _EXPR_HELP % { @@ -1197,7 +1195,6 @@ def view_module_abs(moduleimpl_id, format="html"): 'Absences du module %s' % (moduleimpl_id, M["module"]["titre"]), page_title="Absences du module %s" % (M["module"]["titre"]), - sem=sem, ) ] if not T and format == "html": @@ -1246,7 +1243,6 @@ def edit_ue_expr(formsemestre_id, ue_id): html_sco_header.html_sem_header( "Modification règle de calcul de l'UE %s (%s)" % (ue["acronyme"], ue["titre"]), - sem, ), _EXPR_HELP % {"target": "de l'UE", "objs": "modules", "ordre": ""}, ] @@ -1384,7 +1380,7 @@ def formsemestre_enseignants_list(formsemestre_id, format="html"): html_class="table_leftalign", filename=scu.make_filename("Enseignants-" + sem["titreannee"]), html_title=html_sco_header.html_sem_header( - "Enseignants du semestre", sem, with_page_header=False + "Enseignants du semestre", with_page_header=False ), base_url="%s?formsemestre_id=%s" % (request.base_url, formsemestre_id), caption="Tous les enseignants (responsables ou associés aux modules de ce semestre) apparaissent. Le nombre de saisies d'absences est le nombre d'opérations d'ajout effectuées sur ce semestre, sans tenir compte des annulations ou double saisies.", @@ -1897,7 +1893,7 @@ def formsemestre_bulletins_choice( """Choix d'une version de bulletin""" sem = sco_formsemestre.get_formsemestre(formsemestre_id) H = [ - html_sco_header.html_sem_header(title, sem), + html_sco_header.html_sem_header(title), """ diff --git a/app/views/scolar.py b/app/views/scolar.py index 6bddb9475a..7ec555d248 100644 --- a/app/views/scolar.py +++ b/app/views/scolar.py @@ -1701,8 +1701,7 @@ def check_group_apogee(group_id, etat=None, fix=False, fixmail=False): cnx = ndb.GetDBConnexion() H = [ html_sco_header.html_sem_header( - "Etudiants du %s" % (group["group_name"] or "semestre"), - sem, + "Etudiants du %s" % (group["group_name"] or "semestre") ), '', "",
    NomNom usuelPrénomMailNIP (ScoDoc)Apogée