From 8fedde52e70360bf406915b32760675a036e4fed Mon Sep 17 00:00:00 2001 From: Emmanuel Viennet Date: Thu, 29 Jul 2021 17:31:15 +0300 Subject: [PATCH] refactoring: removing useless args for Flask --- app/scodoc/gen_tables.py | 2 - app/scodoc/html_sco_header.py | 29 +- app/scodoc/html_sidebar.py | 61 ++-- app/scodoc/pe_view.py | 4 +- app/scodoc/sco_abs_views.py | 22 +- app/scodoc/sco_apogee_compare.py | 8 +- app/scodoc/sco_archives.py | 10 +- app/scodoc/sco_archives_etud.py | 4 +- app/scodoc/sco_bulletins.py | 30 +- app/scodoc/sco_cost_formation.py | 2 +- app/scodoc/sco_debouche.py | 28 +- app/scodoc/sco_dept.py | 12 +- app/scodoc/sco_dump_db.py | 4 +- app/scodoc/sco_edit_formation.py | 12 +- app/scodoc/sco_edit_matiere.py | 12 +- app/scodoc/sco_edit_module.py | 12 +- app/scodoc/sco_edit_ue.py | 6 +- app/scodoc/sco_edt_cal.py | 2 - app/scodoc/sco_etape_apogee.py | 2 +- app/scodoc/sco_etape_apogee_view.py | 8 +- app/scodoc/sco_evaluations.py | 13 +- app/scodoc/sco_export_results.py | 2 - app/scodoc/sco_find_etud.py | 4 +- app/scodoc/sco_formsemestre.py | 12 +- app/scodoc/sco_formsemestre_edit.py | 15 +- app/scodoc/sco_formsemestre_exterieurs.py | 8 +- app/scodoc/sco_formsemestre_inscriptions.py | 8 +- app/scodoc/sco_formsemestre_status.py | 130 ++++----- app/scodoc/sco_formsemestre_validation.py | 6 +- app/scodoc/sco_groups.py | 83 +++--- app/scodoc/sco_groups_edit.py | 6 +- app/scodoc/sco_groups_view.py | 16 +- app/scodoc/sco_import_etuds.py | 12 +- app/scodoc/sco_inscr_passage.py | 4 +- app/scodoc/sco_liste_notes.py | 2 +- app/scodoc/sco_lycee.py | 4 - app/scodoc/sco_moduleimpl_inscriptions.py | 2 - app/scodoc/sco_moduleimpl_status.py | 8 +- app/scodoc/sco_page_etud.py | 13 +- app/scodoc/sco_permissions_check.py | 56 ++-- app/scodoc/sco_placement.py | 2 +- app/scodoc/sco_poursuite_dut.py | 2 +- app/scodoc/sco_preferences.py | 2 +- app/scodoc/sco_pvjury.py | 6 +- app/scodoc/sco_recapcomplet.py | 4 +- app/scodoc/sco_report.py | 8 +- app/scodoc/sco_saisie_notes.py | 18 +- app/scodoc/sco_semset.py | 6 +- app/scodoc/sco_synchro_etuds.py | 8 +- app/scodoc/sco_trombino.py | 10 +- app/scodoc/sco_users.py | 2 - app/scodoc/sco_utils.py | 4 +- app/views/absences.py | 12 +- app/views/entreprises.py | 5 +- app/views/notes.py | 40 +-- app/views/scolar.py | 31 +- app/views/users.py | 8 +- scotests/sco_fake_gen.py | 2 +- scotests/test_absence.py | 4 +- scotests/test_absence2.py | 4 +- scotests/test_api7doc.py | 214 +++++++++----- scotests/test_groupes.py | 303 ++++++++++++-------- tests/unit/test_caches.py | 7 +- 63 files changed, 633 insertions(+), 743 deletions(-) diff --git a/app/scodoc/gen_tables.py b/app/scodoc/gen_tables.py index 078017e8c..3791c8142 100644 --- a/app/scodoc/gen_tables.py +++ b/app/scodoc/gen_tables.py @@ -628,8 +628,6 @@ class GenTable(object): H.append( self.html_header or html_sco_header.sco_header( - context, - REQUEST, page_title=page_title, javascripts=javascripts, init_qtip=init_qtip, diff --git a/app/scodoc/html_sco_header.py b/app/scodoc/html_sco_header.py index 2401ee7cb..7381cc734 100644 --- a/app/scodoc/html_sco_header.py +++ b/app/scodoc/html_sco_header.py @@ -30,6 +30,8 @@ import html +from flask import g +from flask import request from flask_login import current_user import app.scodoc.sco_utils as scu @@ -136,10 +138,7 @@ def scodoc_top_html_header(page_title="ScoDoc: bienvenue"): # Header: def sco_header( - context, - REQUEST=None, # optional args - container=None, # objet qui a lancé la demande page_title="", # page title no_side_bar=False, # hide sidebar cssstyles=[], # additionals CSS sheets @@ -159,15 +158,13 @@ def sco_header( from app.scodoc.sco_formsemestre_status import formsemestre_page_title context = None # XXX TODO à enlever #context - # context est une instance de ZScolar. container est une instance qui "acquiert" ZScolar - # Add a HTTP header (can be used by Apache to log requests) - if REQUEST.AUTHENTICATED_USER: - REQUEST.RESPONSE.setHeader("X-ScoDoc-User", str(REQUEST.AUTHENTICATED_USER)) - - # Get more parameters from REQUEST - if not head_message and "head_message" in REQUEST.form: - head_message = REQUEST.form["head_message"] + # Get head message from http request: + if not head_message: + if request.method == "POST": + head_message = request.form.get("head_message", "") + elif request.method == "GET": + head_message = request.args.get("head_message", "") params = { "page_title": page_title or VERSION.SCONAME, @@ -175,7 +172,7 @@ def sco_header( "ScoURL": scu.ScoURL(), "encoding": scu.SCO_ENCODING, "titrebandeau_mkup": "" + titrebandeau + "", - "authuser": str(REQUEST.AUTHENTICATED_USER), + "authuser": current_user.user_name, } if bodyOnLoad: params["bodyOnLoad_mkup"] = """onload="%s" """ % bodyOnLoad @@ -307,11 +304,11 @@ def sco_header( H.append(scu.CUSTOM_HTML_HEADER) # if not no_side_bar: - H.append(html_sidebar.sidebar(REQUEST)) + H.append(html_sidebar.sidebar()) H.append("""
""") # # Barre menu semestre: - H.append(formsemestre_page_title(context, REQUEST)) + H.append(formsemestre_page_title(context)) # Avertissement si mot de passe à changer if user_check: @@ -354,9 +351,7 @@ def html_sem_header( "Titre d'une page semestre avec lien vers tableau de bord" # sem now unused and thus optional... if with_page_header: - h = sco_header( - context, REQUEST, page_title="%s" % (page_title or title), **args - ) + h = sco_header(page_title="%s" % (page_title or title), **args) else: h = "" if with_h2: diff --git a/app/scodoc/html_sidebar.py b/app/scodoc/html_sidebar.py index a172df4f0..8bcdb487f 100644 --- a/app/scodoc/html_sidebar.py +++ b/app/scodoc/html_sidebar.py @@ -28,30 +28,32 @@ """ Génération de la "sidebar" (marge gauche des pages HTML) """ -from flask import url_for, g +from flask import url_for +from flask import g +from flask import request +from flask_login import current_user import app.scodoc.sco_utils as scu from app.scodoc import sco_preferences from app.scodoc.sco_permissions import Permission -def sidebar_common(REQUEST=None): +def sidebar_common(): "partie commune à toutes les sidebar" - authuser = REQUEST.AUTHENTICATED_USER params = { "ScoURL": scu.ScoURL(), "UsersURL": scu.UsersURL(), "NotesURL": scu.NotesURL(), "AbsencesURL": scu.AbsencesURL(), "LogoutURL": url_for("auth.logout"), - "authuser": str(authuser), + "authuser": current_user.user_name, } H = [ 'ScoDoc 8', '' % params, - sidebar_dept(REQUEST), - """

Scolarité

+ sidebar_dept(), + """

Scolarité

Semestres
Programmes
Absences
@@ -59,14 +61,14 @@ def sidebar_common(REQUEST=None): % params, ] - if authuser.has_permission(Permission.ScoUsersAdmin) or authuser.has_permission( - Permission.ScoUsersView - ): + if current_user.has_permission( + Permission.ScoUsersAdmin + ) or current_user.has_permission(Permission.ScoUsersView): H.append( """Utilisateurs
""" % params ) - if authuser.has_permission(Permission.ScoChangePreferences): + if current_user.has_permission(Permission.ScoChangePreferences): H.append( """Paramétrage
""" % params @@ -75,7 +77,7 @@ def sidebar_common(REQUEST=None): return "".join(H) -def sidebar(REQUEST=None): +def sidebar(): "Main HTML page sidebar" # rewritten from legacy DTML code from app.scodoc import sco_abs @@ -86,7 +88,7 @@ def sidebar(REQUEST=None): "SCO_USER_MANUAL": scu.SCO_USER_MANUAL, } - H = ['
', sidebar_common(REQUEST)] + H = ['
', sidebar_common()] H.append( """
Chercher étudiant:
@@ -97,9 +99,14 @@ def sidebar(REQUEST=None): """ % params ) - # ---- s'il y a un etudiant selectionné: - if "etudid" in REQUEST.form: - etudid = REQUEST.form["etudid"] + # ---- Il y-a-t-il un etudiant selectionné ? + etudid = None + if request.method == "GET": + etudid = request.args.get("etudid", None) + elif request.method == "POST": + etudid = request.form.get("etudid", None) + + if etudid: etud = sco_etud.get_etud_info(filled=1, etudid=etudid)[0] params.update(etud) params["fiche_url"] = url_for( @@ -126,7 +133,7 @@ def sidebar(REQUEST=None): ) H.append("
    ") - if REQUEST.AUTHENTICATED_USER.has_permission(Permission.ScoAbsChange): + if current_user.has_permission(Permission.ScoAbsChange): H.append( """
  • Ajouter
    • @@ -168,25 +175,17 @@ def sidebar(REQUEST=None): return "".join(H) -def sidebar_dept(REQUEST=None): +def sidebar_dept(): """Partie supérieure de la marge de gauche""" - infos = { - "BASE0": REQUEST.BASE0, - "DeptIntranetTitle": sco_preferences.get_preference("DeptIntranetTitle"), - "DeptIntranetURL": sco_preferences.get_preference("DeptIntranetURL"), - "DeptName": sco_preferences.get_preference("DeptName"), - "ScoURL": scu.ScoURL(), - } - H = [ - """

    Dépt. %(DeptName)s

    - Accueil
    """ - % infos + f"""

    Dépt. {sco_preferences.get_preference("DeptName")}

    +Accueil
    """ ] - if infos["DeptIntranetURL"]: + dept_intranet_url = sco_preferences.get_preference("DeptIntranetURL") + if dept_intranet_url: H.append( - '%(DeptIntranetTitle)s
    ' - % infos + f"""{ + sco_preferences.get_preference("DeptIntranetTitle")}
    """ ) # Entreprises pas encore supporté en ScoDoc8 # H.append( diff --git a/app/scodoc/pe_view.py b/app/scodoc/pe_view.py index faf2098a6..e98faf503 100644 --- a/app/scodoc/pe_view.py +++ b/app/scodoc/pe_view.py @@ -48,9 +48,7 @@ from app.scodoc import pe_avislatex def _pe_view_sem_recap_form(context, formsemestre_id, REQUEST=None): H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Avis de poursuite d'études" - ), + html_sco_header.sco_header(page_title="Avis de poursuite d'études"), """

    Génération des avis de poursuites d'études

    Cette fonction génère un ensemble de fichiers permettant d'éditer des avis de poursuites d'études. diff --git a/app/scodoc/sco_abs_views.py b/app/scodoc/sco_abs_views.py index 06f2749fe..80693c8b4 100644 --- a/app/scodoc/sco_abs_views.py +++ b/app/scodoc/sco_abs_views.py @@ -132,8 +132,6 @@ def doSignaleAbsence( M = "dans le module %s" % modimpl["module"]["code"] H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Signalement d'une absence pour %(nomprenom)s" % etud, ), """

    Signalement d'absences

    """, @@ -221,8 +219,6 @@ def SignaleAbsenceEtud(context, REQUEST=None): # etudid implied H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Signalement d'une absence pour %(nomprenom)s" % etud, ), """
    @@ -340,8 +336,6 @@ def doJustifAbsence( # H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Justification d'une absence pour %(nomprenom)s" % etud, ), """

    Justification d'absences

    """, @@ -378,8 +372,6 @@ def JustifAbsenceEtud(context, REQUEST=None): # etudid implied etudid = etud["etudid"] H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Justification d'une absence pour %(nomprenom)s" % etud, ), """
    @@ -450,8 +442,6 @@ def doAnnuleAbsence( # H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Annulation d'une absence pour %(nomprenom)s" % etud, ), """

    Annulation d'absences pour %(nomprenom)s

    """ % etud, @@ -489,8 +479,6 @@ def AnnuleAbsenceEtud(context, REQUEST=None): # etudid implied H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Annulation d'une absence pour %(nomprenom)s" % etud, ), """
    @@ -591,8 +579,6 @@ def doAnnuleJustif( # H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Annulation d'une justification pour %(nomprenom)s" % etud, ), """

    Annulation de justifications pour %(nomprenom)s

    """ % etud, @@ -679,7 +665,7 @@ def EtatAbsences(context, REQUEST=None): """Etat des absences: choix du groupe""" # crude portage from 1999 DTML H = [ - html_sco_header.sco_header(context, REQUEST, page_title="Etat des absences"), + html_sco_header.sco_header(page_title="Etat des absences"), """

    Etat des absences pour un groupe

    """, formChoixSemestreGroupe(context), @@ -760,8 +746,6 @@ def CalAbs(context, REQUEST=None): # etud implied # H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Calendrier des absences de %(nomprenom)s" % etud, cssstyles=["css/calabs.css"], ), @@ -864,9 +848,7 @@ def ListeAbsEtud( # Mise en forme HTML: H = [] H.append( - html_sco_header.sco_header( - context, REQUEST, page_title="Absences de %s" % etud["nomprenom"] - ) + html_sco_header.sco_header(page_title="Absences de %s" % etud["nomprenom"]) ) H.append( """

    Absences de %s (à partir du %s)

    """ diff --git a/app/scodoc/sco_apogee_compare.py b/app/scodoc/sco_apogee_compare.py index a6401f0e2..832190682 100644 --- a/app/scodoc/sco_apogee_compare.py +++ b/app/scodoc/sco_apogee_compare.py @@ -64,9 +64,7 @@ _help_txt = """ def apo_compare_csv_form(context, REQUEST=None): """Form: submit 2 CSV files to compare them.""" H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Comparaison de fichiers Apogée" - ), + html_sco_header.sco_header(page_title="Comparaison de fichiers Apogée"), """

    Comparaison de fichiers Apogée

    """, @@ -96,9 +94,7 @@ def apo_compare_csv(context, A_file, B_file, autodetect=True, REQUEST=None): B = _load_apo_data(B_file, autodetect=autodetect) H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Comparaison de fichiers Apogée" - ), + html_sco_header.sco_header(page_title="Comparaison de fichiers Apogée"), "

    Comparaison de fichiers Apogée

    ", _help_txt, '
    ', diff --git a/app/scodoc/sco_archives.py b/app/scodoc/sco_archives.py index b0c4a86d4..a6a37d43d 100644 --- a/app/scodoc/sco_archives.py +++ b/app/scodoc/sco_archives.py @@ -283,7 +283,7 @@ def do_formsemestre_archive( if not group_ids: # tous les inscrits du semestre - group_ids = [sco_groups.get_default_group(context, formsemestre_id)] + group_ids = [sco_groups.get_default_group(formsemestre_id)] groups_infos = sco_groups_view.DisplayedGroupsInfos( context, group_ids, formsemestre_id=formsemestre_id, REQUEST=REQUEST ) @@ -304,8 +304,6 @@ def do_formsemestre_archive( data = "\n".join( [ html_sco_header.sco_header( - context, - REQUEST, page_title="Moyennes archivées le %s" % date, head_message="Moyennes archivées le %s" % date, no_side_bar=True, @@ -372,7 +370,7 @@ def formsemestre_archive(context, REQUEST, formsemestre_id, group_ids=[]): """Make and store new archive for this formsemestre. (all students or only selected groups) """ - if not sco_permissions_check.can_edit_pv(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_edit_pv(formsemestre_id): raise AccessDenied( "opération non autorisée pour %s" % str(REQUEST.AUTHENTICATED_USER) ) @@ -380,7 +378,7 @@ def formsemestre_archive(context, REQUEST, formsemestre_id, group_ids=[]): sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) if not group_ids: # tous les inscrits du semestre - group_ids = [sco_groups.get_default_group(context, formsemestre_id)] + group_ids = [sco_groups.get_default_group(formsemestre_id)] groups_infos = sco_groups_view.DisplayedGroupsInfos( context, group_ids, formsemestre_id=formsemestre_id, REQUEST=REQUEST ) @@ -555,7 +553,7 @@ def formsemestre_delete_archive( context, REQUEST, formsemestre_id, archive_name, dialog_confirmed=False ): """Delete an archive""" - if not sco_permissions_check.can_edit_pv(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_edit_pv(formsemestre_id): raise AccessDenied( "opération non autorisée pour %s" % str(REQUEST.AUTHENTICATED_USER) ) diff --git a/app/scodoc/sco_archives_etud.py b/app/scodoc/sco_archives_etud.py index df332fe95..195396570 100644 --- a/app/scodoc/sco_archives_etud.py +++ b/app/scodoc/sco_archives_etud.py @@ -133,8 +133,6 @@ def etud_upload_file_form(context, REQUEST, etudid): etud = sco_etud.get_etud_info(filled=1, REQUEST=REQUEST)[0] H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Chargement d'un document associé à %(nomprenom)s" % etud, ), """

    Chargement d'un document associé à %(nomprenom)s

    @@ -266,7 +264,7 @@ def etudarchive_import_files_form(context, group_id, REQUEST=None): """Formulaire pour importation fichiers d'un groupe""" H = [ html_sco_header.sco_header( - context, REQUEST, page_title="Import de fichiers associés aux étudiants" + page_title="Import de fichiers associés aux étudiants" ), """

    Téléchargement de fichier associés aux étudiants

    Les fichiers associés (dossiers d'admission, certificats, ...), de types quelconques (pdf, doc, images) diff --git a/app/scodoc/sco_bulletins.py b/app/scodoc/sco_bulletins.py index 89ad2945f..f7ba8648c 100644 --- a/app/scodoc/sco_bulletins.py +++ b/app/scodoc/sco_bulletins.py @@ -38,7 +38,10 @@ from email.header import Header from reportlab.lib.colors import Color import six.moves.urllib.request, six.moves.urllib.parse, six.moves.urllib.error -from flask import g, url_for + +from flask import g +from flask import url_for +from flask_login import current_user from app.scodoc import sco_emails import app.scodoc.sco_utils as scu @@ -837,14 +840,13 @@ def formsemestre_bulletinetud( return "".join(H) -def can_send_bulletin_by_mail(context, formsemestre_id, REQUEST): +def can_send_bulletin_by_mail(context, formsemestre_id): """True if current user is allowed to send a bulletin by mail""" - authuser = REQUEST.AUTHENTICATED_USER sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) return ( sco_preferences.get_preference("bul_mail_allowed_for_all", formsemestre_id) - or authuser.has_permission(Permission.ScoImplement) - or str(authuser) in sem["responsables"] + or current_user.has_permission(Permission.ScoImplement) + or current_user.user_name in sem["responsables"] ) @@ -921,7 +923,7 @@ def do_formsemestre_bulletinetud( elif format == "pdfmail": # format pdfmail: envoie le pdf par mail a l'etud, et affiche le html # check permission - if not can_send_bulletin_by_mail(context, formsemestre_id, REQUEST): + if not can_send_bulletin_by_mail(context, formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") if nohtml: @@ -1112,7 +1114,7 @@ def _formsemestre_bulletinetud_header_html( }, "enabled": etud["email"] and can_send_bulletin_by_mail( - context, formsemestre_id, REQUEST + context, formsemestre_id ), # possible slt si on a un mail... }, { @@ -1127,7 +1129,7 @@ def _formsemestre_bulletinetud_header_html( }, "enabled": etud["emailperso"] and can_send_bulletin_by_mail( - context, formsemestre_id, REQUEST + context, formsemestre_id ), # possible slt si on a un mail... }, { @@ -1168,9 +1170,7 @@ def _formsemestre_bulletinetud_header_html( "formsemestre_id": formsemestre_id, "etudid": etudid, }, - "enabled": sco_permissions_check.can_validate_sem( - context, REQUEST, formsemestre_id - ), + "enabled": sco_permissions_check.can_validate_sem(formsemestre_id), }, { "title": "Enregistrer note d'une UE externe", @@ -1179,9 +1179,7 @@ def _formsemestre_bulletinetud_header_html( "formsemestre_id": formsemestre_id, "etudid": etudid, }, - "enabled": sco_permissions_check.can_validate_sem( - context, REQUEST, formsemestre_id - ), + "enabled": sco_permissions_check.can_validate_sem(formsemestre_id), }, { "title": "Entrer décisions jury", @@ -1190,9 +1188,7 @@ def _formsemestre_bulletinetud_header_html( "formsemestre_id": formsemestre_id, "etudid": etudid, }, - "enabled": sco_permissions_check.can_validate_sem( - context, REQUEST, formsemestre_id - ), + "enabled": sco_permissions_check.can_validate_sem(formsemestre_id), }, { "title": "Editer PV jury", diff --git a/app/scodoc/sco_cost_formation.py b/app/scodoc/sco_cost_formation.py index 800b614b1..e517246fa 100644 --- a/app/scodoc/sco_cost_formation.py +++ b/app/scodoc/sco_cost_formation.py @@ -58,7 +58,7 @@ def formsemestre_table_estim_cost( (dans ce cas, retoucher le tableau excel exporté). """ sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) - sco_formsemestre_status.fill_formsemestre(context, sem, REQUEST=REQUEST) + sco_formsemestre_status.fill_formsemestre(sem) Mlist = sco_moduleimpl.do_moduleimpl_withmodule_list( context, formsemestre_id=formsemestre_id ) diff --git a/app/scodoc/sco_debouche.py b/app/scodoc/sco_debouche.py index 9bb6bad90..6c7cf7a66 100644 --- a/app/scodoc/sco_debouche.py +++ b/app/scodoc/sco_debouche.py @@ -196,7 +196,7 @@ def table_debouche_etudids(context, etudids, keep_numeric=True): def report_debouche_ask_date(context, REQUEST=None): """Formulaire demande date départ""" return ( - html_sco_header.sco_header(context, REQUEST) + html_sco_header.sco_header() + """ Date de départ de la recherche:

    """ @@ -210,22 +210,6 @@ def report_debouche_ask_date(context, REQUEST=None): # # ---------------------------------------------------------------------------- -# OBSOLETE (this field has been copied to itemsuivi) -# def debouche_set(context, object, value, REQUEST=None): -# """Set debouche (field in admission table, may be deprecated ?) -# """ -# if not sco_permissions_check.can_edit_suivi(context, REQUEST): -# raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") -# adm_id = object -# debouche = value.strip('-_ \t') -# cnx = ndb.GetDBConnexion() -# adms = sco_etud.admission_list(cnx, {'etudid' : etudid}) -# if not adms: -# raise ValueError('no admission info for %s !' % etudid) -# adm = adms[0] -# adm['debouche'] = debouche -# admission_edit(cnx, adm) - _itemsuiviEditor = ndb.EditableTable( "itemsuivi", @@ -266,7 +250,7 @@ def itemsuivi_get(cnx, itemsuivi_id, ignore_errors=False): def itemsuivi_suppress(context, itemsuivi_id, REQUEST=None): """Suppression d'un item""" - if not sco_permissions_check.can_edit_suivi(context, REQUEST): + if not sco_permissions_check.can_edit_suivi(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") cnx = ndb.GetDBConnexion() item = itemsuivi_get(cnx, itemsuivi_id, ignore_errors=True) @@ -280,7 +264,7 @@ def itemsuivi_create( context, etudid, item_date=None, situation="", REQUEST=None, format=None ): """Creation d'un item""" - if not sco_permissions_check.can_edit_suivi(context, REQUEST): + if not sco_permissions_check.can_edit_suivi(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") cnx = ndb.GetDBConnexion() itemsuivi_id = _itemsuivi_create( @@ -298,7 +282,7 @@ def itemsuivi_set_date(context, itemsuivi_id, item_date, REQUEST=None): """set item date item_date is a string dd/mm/yyyy """ - if not sco_permissions_check.can_edit_suivi(context, REQUEST): + if not sco_permissions_check.can_edit_suivi(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") # log('itemsuivi_set_date %s : %s' % (itemsuivi_id, item_date)) cnx = ndb.GetDBConnexion() @@ -309,7 +293,7 @@ def itemsuivi_set_date(context, itemsuivi_id, item_date, REQUEST=None): def itemsuivi_set_situation(context, object, value, REQUEST=None): """set situation""" - if not sco_permissions_check.can_edit_suivi(context, REQUEST): + if not sco_permissions_check.can_edit_suivi(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") itemsuivi_id = object situation = value.strip("-_ \t") @@ -365,7 +349,7 @@ def itemsuivi_tag_set(context, itemsuivi_id="", taglist=[], REQUEST=None): a string with tag names separated by commas ("un;deux") or a list of strings (["un", "deux"]) """ - if not sco_permissions_check.can_edit_suivi(context, REQUEST): + if not sco_permissions_check.can_edit_suivi(): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") if not taglist: taglist = [] diff --git a/app/scodoc/sco_dept.py b/app/scodoc/sco_dept.py index edb52dae2..72c07eec2 100644 --- a/app/scodoc/sco_dept.py +++ b/app/scodoc/sco_dept.py @@ -29,6 +29,7 @@ """ from flask import g +from flask_login import current_user import app.scodoc.sco_utils as scu from app.scodoc.gen_tables import GenTable @@ -144,8 +145,7 @@ Chercher étape courante:

    Gestion des étudiants

    @@ -158,7 +158,7 @@ Chercher étape courante:

    Exports Apogée

    @@ -178,11 +178,7 @@ Chercher étape courante: Suppression de la formation %(titre)s (%(acronyme)s)""" % F, ] @@ -130,9 +128,7 @@ def formation_edit(context, formation_id=None, create=False, REQUEST=None): """Edit or create a formation""" if create: H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Création d'une formation" - ), + html_sco_header.sco_header(page_title="Création d'une formation"), """

    Création d'une formation

    Une "formation" décrit une filière, comme un DUT ou une Licence. La formation se subdivise en unités pédagogiques (UE, matières, modules). Elle peut se diviser en plusieurs semestres (ou sessions), qui seront mis en place séparément. @@ -154,9 +150,7 @@ def formation_edit(context, formation_id=None, create=False, REQUEST=None): is_locked = sco_formations.formation_has_locked_sems(context, formation_id) submitlabel = "Modifier les valeurs" H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Modification d'une formation" - ), + html_sco_header.sco_header(page_title="Modification d'une formation"), """

    Modification de la formation %(acronyme)s

    """ % initvalues, ] if is_locked: diff --git a/app/scodoc/sco_edit_matiere.py b/app/scodoc/sco_edit_matiere.py index 1a24c5a5b..8693ccbb8 100644 --- a/app/scodoc/sco_edit_matiere.py +++ b/app/scodoc/sco_edit_matiere.py @@ -98,9 +98,7 @@ def matiere_create(context, ue_id=None, REQUEST=None): UE = sco_edit_ue.do_ue_list(context, args={"ue_id": ue_id})[0] H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Création d'une matière" - ), + html_sco_header.sco_header(page_title="Création d'une matière"), """

    Création d'une matière dans l'UE %(titre)s (%(acronyme)s)

    """ % UE, """

    Les matières sont des groupes de modules dans une UE d'une formation donnée. Les matières servent surtout pour la @@ -197,9 +195,7 @@ def matiere_delete(context, matiere_id=None, REQUEST=None): M = do_matiere_list(context, args={"matiere_id": matiere_id})[0] UE = sco_edit_ue.do_ue_list(context, args={"ue_id": M["ue_id"]})[0] H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Suppression d'une matière" - ), + html_sco_header.sco_header(page_title="Suppression d'une matière"), "

    Suppression de la matière %(titre)s" % M, " dans l'UE (%(acronyme)s))

    " % UE, ] @@ -242,9 +238,7 @@ def matiere_edit(context, matiere_id=None, REQUEST=None): ue_names = ["%(acronyme)s (%(titre)s)" % u for u in ues] ue_ids = [u["ue_id"] for u in ues] H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Modification d'une matière" - ), + html_sco_header.sco_header(page_title="Modification d'une matière"), """

    Modification de la matière %(titre)s""" % F, """(formation %(acronyme)s, version %(version)s)

    """ % Fo, ] diff --git a/app/scodoc/sco_edit_module.py b/app/scodoc/sco_edit_module.py index 0468bfdbd..7e840abcd 100644 --- a/app/scodoc/sco_edit_module.py +++ b/app/scodoc/sco_edit_module.py @@ -131,7 +131,7 @@ def module_create(context, matiere_id=None, REQUEST=None): parcours = sco_codes_parcours.get_parcours_from_code(Fo["type_parcours"]) semestres_indices = list(range(1, parcours.NB_SEM + 1)) H = [ - html_sco_header.sco_header(context, REQUEST, page_title="Création d'un module"), + html_sco_header.sco_header(page_title="Création d'un module"), """

    Création d'un module dans la matière %(titre)s""" % M, """ (UE %(acronyme)s)

    """ % UE, _MODULE_HELP, @@ -287,9 +287,7 @@ def module_delete(context, module_id=None, REQUEST=None): raise ScoValueError("Module inexistant !") Mod = Mods[0] H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Suppression d'un module" - ), + html_sco_header.sco_header(page_title="Suppression d'un module"), """

    Suppression du module %(titre)s (%(code)s)

    """ % Mod, ] @@ -368,8 +366,6 @@ def module_edit(context, module_id=None, REQUEST=None): H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Modification du module %(titre)s" % Mod, cssstyles=["libjs/jQuery-tagEditor/jquery.tag-editor.css"], javascripts=[ @@ -539,9 +535,7 @@ def module_list(context, formation_id, REQUEST=None): raise ScoValueError("invalid formation !") F = sco_formations.formation_list(context, args={"formation_id": formation_id})[0] H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Liste des modules de %(titre)s" % F - ), + html_sco_header.sco_header(page_title="Liste des modules de %(titre)s" % F), """

    Listes des modules dans la formation %(titre)s (%(acronyme)s)

    """ % F, '
      ', diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py index 5b52f2231..e7c8bc543 100644 --- a/app/scodoc/sco_edit_ue.py +++ b/app/scodoc/sco_edit_ue.py @@ -211,9 +211,7 @@ def ue_edit(context, ue_id=None, create=False, formation_id=None, REQUEST=None): parcours = sco_codes_parcours.get_parcours_from_code(Fo["type_parcours"]) H = [ - html_sco_header.sco_header( - context, REQUEST, page_title=title, javascripts=["js/edit_ue.js"] - ), + html_sco_header.sco_header(page_title=title, javascripts=["js/edit_ue.js"]), "

      " + title, " (formation %(acronyme)s, version %(version)s)

      " % Fo, """ @@ -470,8 +468,6 @@ def ue_list(context, formation_id=None, msg="", REQUEST=None): ) H = [ html_sco_header.sco_header( - context, - REQUEST, cssstyles=["libjs/jQuery-tagEditor/jquery.tag-editor.css"], javascripts=[ "libjs/jinplace-1.2.1.min.js", diff --git a/app/scodoc/sco_edt_cal.py b/app/scodoc/sco_edt_cal.py index b95d0ed9c..9ad1a7105 100644 --- a/app/scodoc/sco_edt_cal.py +++ b/app/scodoc/sco_edt_cal.py @@ -166,8 +166,6 @@ def experimental_calendar(context, group_id=None, formsemestre_id=None, REQUEST= return "\n".join( [ html_sco_header.sco_header( - context, - REQUEST, javascripts=[ "libjs/purl.js", "libjs/moment.min.js", diff --git a/app/scodoc/sco_etape_apogee.py b/app/scodoc/sco_etape_apogee.py index 9e666aafb..91a2595de 100644 --- a/app/scodoc/sco_etape_apogee.py +++ b/app/scodoc/sco_etape_apogee.py @@ -387,7 +387,7 @@ apo_csv_store(context, csv_data, annee_scolaire, sem_id) -groups_infos = sco_groups_view.DisplayedGroupsInfos(context, [sco_groups.get_default_group(context, formsemestre_id)], formsemestre_id=formsemestre_id, REQUEST=REQUEST) +groups_infos = sco_groups_view.DisplayedGroupsInfos(context, [sco_groups.get_default_group(formsemestre_id)], formsemestre_id=formsemestre_id) nt = sco_cache.NotesTableCache.get( formsemestre_id) diff --git a/app/scodoc/sco_etape_apogee_view.py b/app/scodoc/sco_etape_apogee_view.py index 363a79ffb..40d46520d 100644 --- a/app/scodoc/sco_etape_apogee_view.py +++ b/app/scodoc/sco_etape_apogee_view.py @@ -69,7 +69,7 @@ def apo_semset_maq_status( if not semset_id: raise ValueError("invalid null semset_id") semset = sco_semset.SemSet(context, semset_id=semset_id) - semset.fill_formsemestres(REQUEST) + semset.fill_formsemestres() # autorise export meme si etudiants Apo manquants: allow_missing_apo = int(allow_missing_apo) # autorise export meme s'il manque des décisions de jury: @@ -105,8 +105,6 @@ def apo_semset_maq_status( H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Export Apogée", javascripts=["js/apo_semset_maq_status.js"], ), @@ -570,8 +568,6 @@ def _view_etuds_page( H = [ html_sco_header.sco_header( - context, - REQUEST, page_title=title, init_qtip=True, javascripts=["js/etud_info.js"], @@ -719,8 +715,6 @@ def view_apo_csv(context, etape_apo="", semset_id="", format="html", REQUEST=Non H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Maquette Apogée enregistrée pour %s" % etape_apo, init_qtip=True, javascripts=["js/etud_info.js"], diff --git a/app/scodoc/sco_evaluations.py b/app/scodoc/sco_evaluations.py index 5b0d35704..e1a54898d 100644 --- a/app/scodoc/sco_evaluations.py +++ b/app/scodoc/sco_evaluations.py @@ -253,7 +253,7 @@ def do_evaluation_create( evaluation_type=None, numero=None, REQUEST=None, - **kw # ceci pour absorber les arguments excedentaires de tf #sco8 + **kw, # ceci pour absorber les arguments excedentaires de tf #sco8 ): """Create an evaluation""" context = None # #context @@ -623,10 +623,10 @@ def do_evaluation_get_all_notes( """Toutes les notes pour une evaluation: { etudid : { 'value' : value, 'date' : date ... }} Attention: inclut aussi les notes des étudiants qui ne sont plus inscrits au module. """ - # log('do_evaluation_get_all_notes( e=%s fs=%s )' % (evaluation_id, filter_suppressed)) do_cache = ( filter_suppressed and table == "notes_notes" and (by_uid is None) ) # pas de cache pour (rares) appels via undo_notes ou specifiant un enseignant + log(f"do_evaluation_get_all_notes: {evaluation_id} (do_cache={do_cache})") if do_cache: r = sco_cache.EvaluationCache.get(evaluation_id) if r != None: @@ -651,7 +651,8 @@ def do_evaluation_get_all_notes( for x in res: d[x["etudid"]] = x if do_cache: - sco_cache.EvaluationCache.set(evaluation_id, d) + status = sco_cache.EvaluationCache.set(evaluation_id, d) + log(f"EvaluationCache.set: {evaluation_id}\t{status}") return d @@ -1082,7 +1083,7 @@ def evaluation_describe(evaluation_id="", edit_in_place=True, REQUEST=None): % (jour, E["heure_debut"], E["heure_fin"]) ) if E["jour"]: - group_id = sco_groups.get_default_group(context, formsemestre_id) + group_id = sco_groups.get_default_group(formsemestre_id) H.append( '(absences ce jour)' % ( @@ -1129,7 +1130,7 @@ def evaluation_create_form( if not readonly: if not sco_permissions_check.can_edit_evaluation(moduleimpl_id=moduleimpl_id): return ( - html_sco_header.sco_header(context, REQUEST) + html_sco_header.sco_header() + "

      Opération non autorisée

      " + "Modification évaluation impossible pour %s" % current_user.get_nomplogin() @@ -1359,7 +1360,7 @@ def evaluation_create_form( dest_url = "moduleimpl_status?moduleimpl_id=%s" % M["moduleimpl_id"] if tf[0] == 0: - head = html_sco_header.sco_header(context, REQUEST, page_title=page_title) + head = html_sco_header.sco_header(page_title=page_title) return head + "\n".join(H) + "\n" + tf[1] + help + html_sco_header.sco_footer() elif tf[0] == -1: return REQUEST.RESPONSE.redirect(dest_url) diff --git a/app/scodoc/sco_export_results.py b/app/scodoc/sco_export_results.py index 72acee374..d1e6fc212 100644 --- a/app/scodoc/sco_export_results.py +++ b/app/scodoc/sco_export_results.py @@ -278,8 +278,6 @@ def scodoc_table_results( H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Export résultats", init_qtip=True, javascripts=html_sco_header.BOOTSTRAP_MULTISELECT_JS diff --git a/app/scodoc/sco_find_etud.py b/app/scodoc/sco_find_etud.py index 764a4b362..dda5a5877 100644 --- a/app/scodoc/sco_find_etud.py +++ b/app/scodoc/sco_find_etud.py @@ -86,9 +86,7 @@ def form_search_etud( if add_headers: return ( - html_sco_header.sco_header( - context, REQUEST, page_title="Choix d'un étudiant" - ) + html_sco_header.sco_header(page_title="Choix d'un étudiant") + "\n".join(H) + html_sco_header.sco_footer() ) diff --git a/app/scodoc/sco_formsemestre.py b/app/scodoc/sco_formsemestre.py index e04c2489b..701ca6316 100644 --- a/app/scodoc/sco_formsemestre.py +++ b/app/scodoc/sco_formsemestre.py @@ -219,8 +219,9 @@ def etapes_apo_str(etapes): return ", ".join([str(x) for x in etapes]) -def do_formsemestre_create(context, args, REQUEST, silent=False): +def do_formsemestre_create(args, silent=False): "create a formsemestre" + context = None # XXX #context from app.scodoc import sco_groups from app.scodoc import sco_news @@ -235,11 +236,12 @@ def do_formsemestre_create(context, args, REQUEST, silent=False): # create default partition partition_id = sco_groups.partition_create( - context, formsemestre_id, default=True, redirect=0, REQUEST=REQUEST - ) - _group_id = sco_groups.createGroup( - context, partition_id, default=True, REQUEST=REQUEST + context, + formsemestre_id, + default=True, + redirect=0, ) + _group_id = sco_groups.createGroup(context, partition_id, default=True) # news if "titre" not in args: diff --git a/app/scodoc/sco_formsemestre_edit.py b/app/scodoc/sco_formsemestre_edit.py index b5325555f..cb9db6a41 100644 --- a/app/scodoc/sco_formsemestre_edit.py +++ b/app/scodoc/sco_formsemestre_edit.py @@ -68,8 +68,6 @@ def formsemestre_createwithmodules(context, REQUEST=None): """Page création d'un semestre""" H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Création d'un semestre", javascripts=["libjs/AutoSuggest.js"], cssstyles=["css/autosuggest_inquisitor.css"], @@ -552,9 +550,7 @@ def do_formsemestre_createwithmodules(context, REQUEST=None, edit=False): else: disabled = "" fcg = '

    ") else: H.append('

    Aucun groupe dans cette partition') - if sco_groups.sco_permissions_check.can_change_groups( - context, REQUEST, formsemestre_id - ): + if sco_groups.sco_permissions_check.can_change_groups(formsemestre_id): H.append( ' (créer)' % partition["partition_id"] ) H.append("

    ") - if sco_groups.sco_permissions_check.can_change_groups( - context, REQUEST, formsemestre_id - ): + if sco_groups.sco_permissions_check.can_change_groups(formsemestre_id): H.append( f"""

    ', formsemestre_status_head( context, formsemestre_id=formsemestre_id, page_title="Tableau de bord" diff --git a/app/scodoc/sco_formsemestre_validation.py b/app/scodoc/sco_formsemestre_validation.py index f2a71aab2..55877b0e8 100644 --- a/app/scodoc/sco_formsemestre_validation.py +++ b/app/scodoc/sco_formsemestre_validation.py @@ -106,8 +106,6 @@ def formsemestre_validation_etud_form( H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Parcours %(nomprenom)s" % etud, javascripts=["js/recap_parcours.js"], ) @@ -921,7 +919,7 @@ def do_formsemestre_validation_auto(context, formsemestre_id, REQUEST): "do_formsemestre_validation_auto: %d validations, %d conflicts" % (nb_valid, len(conflicts)) ) - H = [html_sco_header.sco_header(context, REQUEST, page_title="Saisie automatique")] + H = [html_sco_header.sco_header(page_title="Saisie automatique")] H.append( """

    Saisie automatique des décisions du semestre %s

    Opération effectuée.

    @@ -992,8 +990,6 @@ def formsemestre_validate_previous_ue(context, formsemestre_id, etudid, REQUEST= H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Validation UE", javascripts=["js/validate_previous_ue.js"], ), diff --git a/app/scodoc/sco_groups.py b/app/scodoc/sco_groups.py index b9653d21e..db4d4a99b 100644 --- a/app/scodoc/sco_groups.py +++ b/app/scodoc/sco_groups.py @@ -33,15 +33,18 @@ Optimisation possible: et éviter ainsi l'appel ulterieur à get_etud_groups() dans _make_table_notes """ -import time import collections -import re import operator +import re +import time -import xml.dom.minidom from xml.etree import ElementTree from xml.etree.ElementTree import Element +import flask +from flask import g +from flask import url_for + import app.scodoc.sco_utils as scu import app.scodoc.notesdb as ndb from app.scodoc.notes_log import log @@ -171,8 +174,9 @@ def get_partition_groups(context, partition): ) -def get_default_group(context, formsemestre_id, fix_if_missing=False, REQUEST=None): +def get_default_group(formsemestre_id, fix_if_missing=False): """Returns group_id for default ('tous') group""" + context = None # #context r = ndb.SimpleDictFetch( "SELECT gd.group_id FROM group_descr gd, partition p WHERE p.formsemestre_id=%(formsemestre_id)s AND p.partition_name is NULL AND p.partition_id = gd.partition_id", {"formsemestre_id": formsemestre_id}, @@ -190,10 +194,8 @@ def get_default_group(context, formsemestre_id, fix_if_missing=False, REQUEST=No ] except ScoException: log("creating default partition for %s" % formsemestre_id) - partition_id = partition_create( - context, formsemestre_id, default=True, REQUEST=REQUEST - ) - group_id = createGroup(context, partition_id, default=True, REQUEST=REQUEST) + partition_id = partition_create(context, formsemestre_id, default=True) + group_id = createGroup(context, partition_id, default=True) return group_id # debug check if len(r) != 1: @@ -330,7 +332,7 @@ def get_etud_main_group(context, etudid, sem): if groups: return groups[0] else: - return get_group(context, get_default_group(context, sem["formsemestre_id"])) + return get_group(context, get_default_group(sem["formsemestre_id"])) def formsemestre_get_main_partition(context, formsemestre_id): @@ -600,7 +602,7 @@ def setGroups( partition = get_partition(context, partition_id) formsemestre_id = partition["formsemestre_id"] - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") log("***setGroups: partition_id=%s" % partition_id) log("groupsLists=%s" % groupsLists) @@ -669,7 +671,7 @@ def setGroups( # group_name = six.text_type(group_name, "utf-8").encode( # scu.SCO_ENCODING # ) # #py3 #sco8 - group_id = createGroup(context, partition_id, group_name, REQUEST=REQUEST) + group_id = createGroup(context, partition_id, group_name) # Place dans ce groupe les etudiants indiqués: for etudid in fs[1:-1]: change_etud_group_in_partition( @@ -682,15 +684,11 @@ def setGroups( ) -def createGroup(context, partition_id, group_name="", default=False, REQUEST=None): - """Create a new group in this partition - (called from JS) - """ +def createGroup(context, partition_id, group_name="", default=False): + """Create a new group in this partition""" partition = get_partition(context, partition_id) formsemestre_id = partition["formsemestre_id"] - if REQUEST and not sco_permissions_check.can_change_groups( - context, REQUEST, formsemestre_id - ): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") # if group_name: @@ -727,9 +725,7 @@ def suppressGroup(context, group_id, partition_id=None, REQUEST=None): else: partition_id = group["partition_id"] partition = get_partition(context, partition_id) - if not sco_permissions_check.can_change_groups( - context, REQUEST, partition["formsemestre_id"] - ): + if not sco_permissions_check.can_change_groups(partition["formsemestre_id"]): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") log( "suppressGroup: group_id=%s group_name=%s partition_name=%s" @@ -744,13 +740,10 @@ def partition_create( partition_name="", default=False, numero=None, - REQUEST=None, redirect=1, ): """Create a new partition""" - if REQUEST and not sco_permissions_check.can_change_groups( - context, REQUEST, formsemestre_id - ): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") if partition_name: partition_name = partition_name.strip() @@ -774,8 +767,12 @@ def partition_create( log("createPartition: created partition_id=%s" % partition_id) # if redirect: - return REQUEST.RESPONSE.redirect( - "editPartitionForm?formsemestre_id=" + formsemestre_id + return flask.redirect( + url_for( + "scolar.editPartitionForm", + scodoc_dept=g.scodoc_dept, + formsemestre_id=formsemestre_id, + ) ) else: return partition_id @@ -794,7 +791,7 @@ def getArrowIconsTags(context, REQUEST): def editPartitionForm(context, formsemestre_id=None, REQUEST=None): """Form to create/suppress partitions""" # ad-hoc form - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") partitions = get_partitions_list(context, formsemestre_id) arrow_up, arrow_down, arrow_none = getArrowIconsTags(context, REQUEST) @@ -804,8 +801,6 @@ def editPartitionForm(context, formsemestre_id=None, REQUEST=None): # H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Partitions...", javascripts=["js/editPartitionForm.js"], ), @@ -924,7 +919,7 @@ def partition_set_attr(context, partition_id, attr, value, REQUEST=None): partition = get_partition(context, partition_id) formsemestre_id = partition["formsemestre_id"] - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") log("partition_set_attr(%s, %s, %s)" % (partition_id, attr, value)) @@ -947,7 +942,7 @@ def partition_delete( default partition cannot be suppressed (unless force)""" partition = get_partition(context, partition_id) formsemestre_id = partition["formsemestre_id"] - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") if not partition["partition_name"] and not force: @@ -991,7 +986,7 @@ def partition_move(context, partition_id, after=0, REQUEST=None, redirect=1): """Move before/after previous one (decrement/increment numero)""" partition = get_partition(context, partition_id) formsemestre_id = partition["formsemestre_id"] - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") # redirect = int(redirect) @@ -1026,7 +1021,7 @@ def partition_rename(context, partition_id, REQUEST=None): """Form to rename a partition""" partition = get_partition(context, partition_id) formsemestre_id = partition["formsemestre_id"] - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") H = ["

    Renommer une partition

    "] tf = TrivialFormulator( @@ -1049,7 +1044,7 @@ def partition_rename(context, partition_id, REQUEST=None): ) if tf[0] == 0: return ( - html_sco_header.sco_header(context, REQUEST) + html_sco_header.sco_header() + "\n".join(H) + "\n" + tf[1] @@ -1086,7 +1081,7 @@ def partition_set_name(context, partition_id, partition_name, REQUEST=None, redi "Partition %s déjà existante dans ce semestre !" % partition_name ) - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") redirect = int(redirect) cnx = ndb.GetDBConnexion() @@ -1111,7 +1106,7 @@ def group_set_name(context, group_id, group_name, REQUEST=None, redirect=1): if group["group_name"] is None: raise ValueError("can't set a name to default group") formsemestre_id = group["formsemestre_id"] - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") redirect = int(redirect) cnx = ndb.GetDBConnexion() @@ -1128,7 +1123,7 @@ def group_rename(context, group_id, REQUEST=None): """Form to rename a group""" group = get_group(context, group_id) formsemestre_id = group["formsemestre_id"] - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") H = ["

    Renommer un groupe de %s

    " % group["partition_name"]] tf = TrivialFormulator( @@ -1151,7 +1146,7 @@ def group_rename(context, group_id, REQUEST=None): ) if tf[0] == 0: return ( - html_sco_header.sco_header(context, REQUEST) + html_sco_header.sco_header() + "\n".join(H) + "\n" + tf[1] @@ -1178,7 +1173,7 @@ def groups_auto_repartition(context, partition_id=None, REQUEST=None): formsemestre_id = partition["formsemestre_id"] # renvoie sur page édition groupes dest_url = "affectGroups?partition_id=%s" % partition_id - if not sco_permissions_check.can_change_groups(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) @@ -1196,9 +1191,7 @@ def groups_auto_repartition(context, partition_id=None, REQUEST=None): ] H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Répartition des groupes" - ), + html_sco_header.sco_header(page_title="Répartition des groupes"), "

    Répartition des groupes de %s

    " % partition["partition_name"], "

    Semestre %s

    " % sem["titreannee"], """

    Les groupes existants seront effacés et remplacés par @@ -1240,9 +1233,7 @@ def groups_auto_repartition(context, partition_id=None, REQUEST=None): # except: # H.append('

    Nom de groupe invalide: %s

    '%group_name) # return '\n'.join(H) + tf[1] + html_sco_header.sco_footer( REQUEST) - group_ids.append( - createGroup(context, partition_id, group_name, REQUEST=REQUEST) - ) + group_ids.append(createGroup(context, partition_id, group_name)) # nt = sco_cache.NotesTableCache.get(formsemestre_id) # > identdict identdict = nt.identdict diff --git a/app/scodoc/sco_groups_edit.py b/app/scodoc/sco_groups_edit.py index 20758e3f7..2a25050f4 100644 --- a/app/scodoc/sco_groups_edit.py +++ b/app/scodoc/sco_groups_edit.py @@ -40,15 +40,11 @@ def affectGroups(context, partition_id, REQUEST=None): # Ported from DTML and adapted to new group management (nov 2009) partition = sco_groups.get_partition(context, partition_id) formsemestre_id = partition["formsemestre_id"] - if not sco_groups.sco_permissions_check.can_change_groups( - context, REQUEST, formsemestre_id - ): + if not sco_groups.sco_permissions_check.can_change_groups(formsemestre_id): raise AccessDenied("vous n'avez pas la permission d'effectuer cette opération") H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Affectation aux groupes", javascripts=["js/groupmgr.js"], cssstyles=["css/groups.css"], diff --git a/app/scodoc/sco_groups_view.py b/app/scodoc/sco_groups_view.py index ec683d4e7..033cb0e41 100644 --- a/app/scodoc/sco_groups_view.py +++ b/app/scodoc/sco_groups_view.py @@ -105,8 +105,6 @@ def groups_view( H = [ html_sco_header.sco_header( - context, - REQUEST, javascripts=JAVASCRIPTS, cssstyles=CSSSTYLES, init_qtip=True, @@ -170,9 +168,7 @@ def form_groups_choice( Si submit_on_change, ajoute une classe "submit_on_change" qui est utilisee en JS """ - default_group_id = sco_groups.get_default_group( - context, groups_infos.formsemestre_id - ) + default_group_id = sco_groups.get_default_group(groups_infos.formsemestre_id) H = [ """
    @@ -202,9 +198,7 @@ def menu_groups_choice(context, groups_infos, submit_on_change=False): et doit comporter au moins un élément, sauf si formsemestre_id est spécifié. (utilisé pour retrouver le semestre et proposer la liste des autres groupes) """ - default_group_id = sco_groups.get_default_group( - context, groups_infos.formsemestre_id - ) + default_group_id = sco_groups.get_default_group(groups_infos.formsemestre_id) if submit_on_change: klass = "submit_on_change" @@ -334,7 +328,7 @@ class DisplayedGroupsInfos(object): if not formsemestre_id: raise Exception("missing parameter formsemestre_id or group_ids") if select_all_when_unspecified: - group_ids = [sco_groups.get_default_group(context, formsemestre_id)] + group_ids = [sco_groups.get_default_group(formsemestre_id)] else: # selectionne le premier groupe trouvé, s'il y en a un partition = sco_groups.get_partitions_list( @@ -344,7 +338,7 @@ class DisplayedGroupsInfos(object): if groups: group_ids = [groups[0]["group_id"]] else: - group_ids = [sco_groups.get_default_group(context, formsemestre_id)] + group_ids = [sco_groups.get_default_group(formsemestre_id)] gq = [] for group_id in group_ids: @@ -861,7 +855,7 @@ def tab_absences_html(context, groups_infos, etat=None, REQUEST=None): H.append('

    Opérations diverses

      ') # Lien pour verif codes INE/NIP # (pour tous les etudiants du semestre) - group_id = sco_groups.get_default_group(context, groups_infos.formsemestre_id) + group_id = sco_groups.get_default_group(groups_infos.formsemestre_id) if authuser.has_permission(Permission.ScoEtudInscrit): H.append( '
    • Vérifier codes Apogée (de tous les groupes)
      • ' diff --git a/app/scodoc/sco_import_etuds.py b/app/scodoc/sco_import_etuds.py index fa884ec21..d5db5eab7 100644 --- a/app/scodoc/sco_import_etuds.py +++ b/app/scodoc/sco_import_etuds.py @@ -241,9 +241,7 @@ def students_import_excel( dest = "formsemestre_status?formsemestre_id=%s" % formsemestre_id else: dest = scu.NotesURL() - H = [ - html_sco_header.sco_header(context, REQUEST, page_title="Import etudiants") - ] + H = [html_sco_header.sco_header(page_title="Import etudiants")] H.append("
        ") for d in diag: H.append("
      • %s
        • " % d) @@ -503,11 +501,7 @@ def students_import_admission( type_admission=type_admission, ) if REQUEST: - H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Import données admissions" - ) - ] + H = [html_sco_header.sco_header(page_title="Import données admissions")] H.append("

        Import terminé !

        ") H.append( '

        Continuer

        ' @@ -618,7 +612,7 @@ def scolars_import_admission( log("scolars_import_admission: formsemestre_id=%s" % formsemestre_id) members = sco_groups.get_group_members( - context, sco_groups.get_default_group(context, formsemestre_id) + context, sco_groups.get_default_group(formsemestre_id) ) etuds_by_nomprenom = {} # { nomprenom : etud } diag = [] diff --git a/app/scodoc/sco_inscr_passage.py b/app/scodoc/sco_inscr_passage.py index 670ff4cdd..0e86b3601 100644 --- a/app/scodoc/sco_inscr_passage.py +++ b/app/scodoc/sco_inscr_passage.py @@ -287,9 +287,7 @@ def formsemestre_inscr_passage( # -- check lock if sem["etat"] != "1": raise ScoValueError("opération impossible: semestre verrouille") - header = html_sco_header.sco_header( - context, REQUEST, page_title="Passage des étudiants" - ) + header = html_sco_header.sco_header(page_title="Passage des étudiants") footer = html_sco_header.sco_footer() H = [header] if type(etuds) == type(""): diff --git a/app/scodoc/sco_liste_notes.py b/app/scodoc/sco_liste_notes.py index ea055dfa2..986f49af3 100644 --- a/app/scodoc/sco_liste_notes.py +++ b/app/scodoc/sco_liste_notes.py @@ -246,7 +246,7 @@ def _make_table_notes( keep_numeric = False # Si pas de groupe, affiche tout if not group_ids: - group_ids = [sco_groups.get_default_group(context, M["formsemestre_id"])] + group_ids = [sco_groups.get_default_group(M["formsemestre_id"])] groups = sco_groups.listgroups(context, group_ids) gr_title = sco_groups.listgroups_abbrev(groups) diff --git a/app/scodoc/sco_lycee.py b/app/scodoc/sco_lycee.py index 7ca65a947..60e87dbdc 100644 --- a/app/scodoc/sco_lycee.py +++ b/app/scodoc/sco_lycee.py @@ -84,8 +84,6 @@ def scodoc_table_etuds_lycees(context, format="html", REQUEST=None): return t H = [ html_sco_header.sco_header( - context, - REQUEST, page_title=tab.page_title, init_google_maps=True, init_qtip=True, @@ -199,8 +197,6 @@ def formsemestre_etuds_lycees( ] H = [ html_sco_header.sco_header( - context, - REQUEST, page_title=tab.page_title, init_google_maps=True, init_qtip=True, diff --git a/app/scodoc/sco_moduleimpl_inscriptions.py b/app/scodoc/sco_moduleimpl_inscriptions.py index 36a893d81..8c7b53995 100644 --- a/app/scodoc/sco_moduleimpl_inscriptions.py +++ b/app/scodoc/sco_moduleimpl_inscriptions.py @@ -71,8 +71,6 @@ def moduleimpl_inscriptions_edit( if sem["etat"] != "1": raise ScoValueError("opération impossible: semestre verrouille") header = html_sco_header.sco_header( - context, - REQUEST, page_title="Inscription au module", init_qtip=True, javascripts=["js/etud_info.js"], diff --git a/app/scodoc/sco_moduleimpl_status.py b/app/scodoc/sco_moduleimpl_status.py index 707aabd6a..ad284e6f2 100644 --- a/app/scodoc/sco_moduleimpl_status.py +++ b/app/scodoc/sco_moduleimpl_status.py @@ -60,7 +60,7 @@ def moduleimpl_evaluation_menu(context, evaluation_id, nbnotes=0, REQUEST=None): context, moduleimpl_id=E["moduleimpl_id"] )[0] - group_id = sco_groups.get_default_group(context, modimpl["formsemestre_id"]) + group_id = sco_groups.get_default_group(modimpl["formsemestre_id"]) if ( sco_permissions_check.can_edit_notes( @@ -185,9 +185,7 @@ def moduleimpl_status(context, moduleimpl_id=None, partition_id=None, REQUEST=No arrow_up, arrow_down, arrow_none = sco_groups.getArrowIconsTags(context, REQUEST) # H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Module %(titre)s" % Mod - ), + html_sco_header.sco_header(page_title="Module %(titre)s" % Mod), """

        Module %(code)s %(titre)s

        """ % Mod, # XXX """caneditevals=%s caneditnotes=%s""" % (caneditevals,caneditnotes), """
        @@ -285,7 +283,7 @@ def moduleimpl_status(context, moduleimpl_id=None, partition_id=None, REQUEST=No Permission.ScoAbsChange ) and sco_formsemestre.sem_est_courant(context, sem): datelundi = sco_abs.ddmmyyyy(time.strftime("%d/%m/%Y")).prev_monday() - group_id = sco_groups.get_default_group(context, formsemestre_id) + group_id = sco_groups.get_default_group(formsemestre_id) H.append( f""" ajouter une ligne @@ -478,8 +478,6 @@ def ficheEtud(context, etudid=None, REQUEST=None):
        """ header = html_sco_header.sco_header( - context, - REQUEST, page_title="Fiche étudiant %(prenom)s %(nom)s" % info, cssstyles=["libjs/jQuery-tagEditor/jquery.tag-editor.css"], javascripts=[ @@ -547,12 +545,7 @@ def etud_info_html(context, etudid, with_photo="1", REQUEST=None, debug=False): context = context except: pass - # log('etud_info_html: %s' % REQUEST.QUERY_STRING) - formsemestre_id = sco_formsemestre_status.retreive_formsemestre_from_request( - context, REQUEST - ) - # log('etud_info_html: formsemestre_id=%s' % formsemestre_id) - + formsemestre_id = sco_formsemestre_status.retreive_formsemestre_from_request() with_photo = int(with_photo) etud = sco_etud.get_etud_info(filled=1, REQUEST=REQUEST)[0] photo_html = sco_photos.etud_photo_html( diff --git a/app/scodoc/sco_permissions_check.py b/app/scodoc/sco_permissions_check.py index 9dc2fc6f5..6a4010d60 100644 --- a/app/scodoc/sco_permissions_check.py +++ b/app/scodoc/sco_permissions_check.py @@ -62,7 +62,6 @@ def can_edit_evaluation(moduleimpl_id=None): # was _evaluation_check_write_access # AccessDenied("Modification évaluation impossible pour %s" % (uid,)) from app.scodoc import sco_formsemestre - from app.scodoc import sco_moduleimpl # acces pour resp. moduleimpl et resp. form semestre (dir etud) if moduleimpl_id is None: @@ -85,7 +84,7 @@ def can_edit_evaluation(moduleimpl_id=None): return False -def can_suppress_annotation(context, annotation_id, REQUEST): +def can_suppress_annotation(annotation_id): """True if current user can suppress this annotation Seuls l'auteur de l'annotation et le chef de dept peuvent supprimer une annotation. @@ -95,51 +94,48 @@ def can_suppress_annotation(context, annotation_id, REQUEST): if len(annos) != 1: raise sco_exceptions.ScoValueError("annotation inexistante !") anno = annos[0] - authuser = REQUEST.AUTHENTICATED_USER return ( - str(authuser) == anno["zope_authenticated_user"] - ) or authuser.has_permission(Permission.ScoEtudAddAnnotations) + current_user.user_name == anno["zope_authenticated_user"] + ) or current_user.has_permission(Permission.ScoEtudAddAnnotations) -def can_edit_suivi(context, REQUEST=None): +def can_edit_suivi(): """Vrai si l'utilisateur peut modifier les informations de suivi sur la page etud" """ - authuser = REQUEST.AUTHENTICATED_USER - return authuser.has_permission(Permission.ScoEtudChangeAdr) + return current_user.has_permission(Permission.ScoEtudChangeAdr) -def can_validate_sem(context, REQUEST, formsemestre_id): +def can_validate_sem(formsemestre_id): "Vrai si utilisateur peut saisir decision de jury dans ce semestre" from app.scodoc import sco_formsemestre + context = None # XXX #context sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) if sem["etat"] != "1": return False # semestre verrouillé - return is_chef_or_diretud(context, REQUEST, sem) + return is_chef_or_diretud(sem) -def can_edit_pv(context, REQUEST, formsemestre_id): +def can_edit_pv(formsemestre_id): "Vrai si utilisateur peut editer un PV de jury de ce semestre" from app.scodoc import sco_formsemestre + context = None # XXX #context sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) - if is_chef_or_diretud(context, REQUEST, sem): + if is_chef_or_diretud(sem): return True # Autorise les secrétariats, repérés via la permission ScoEtudChangeAdr # (ceci nous évite d'ajouter une permission Zope aux installations existantes) - authuser = REQUEST.AUTHENTICATED_USER - return authuser.has_permission(Permission.ScoEtudChangeAdr) + return current_user.has_permission(Permission.ScoEtudChangeAdr) -def is_chef_or_diretud(context, REQUEST, sem): +def is_chef_or_diretud(sem): "Vrai si utilisateur est admin, chef dept ou responsable du semestre" - authuser = REQUEST.AUTHENTICATED_USER - if authuser.has_permission(Permission.ScoImplement): - return True # admin, chef dept - uid = str(authuser) - if uid in sem["responsables"]: + if ( + current_user.has_permission(Permission.ScoImplement) + or current_user.user_name in sem["responsables"] + ): return True - return False @@ -151,21 +147,20 @@ def check_access_diretud( """ from app.scodoc import sco_formsemestre - authuser = REQUEST.AUTHENTICATED_USER sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) header = html_sco_header.sco_header( context, page_title="Accès interdit", REQUEST=REQUEST ) footer = html_sco_header.sco_footer() - if (str(authuser) not in sem["responsables"]) and not authuser.has_permission( - required_permission - ): + if ( + current_user.user_name not in sem["responsables"] + ) and not current_user.has_permission(required_permission): return ( False, "\n".join( [ header, - "

        Opération non autorisée pour %s

        " % authuser, + "

        Opération non autorisée pour %s

        " % current_user, "

        Responsable de ce semestre : %s

        " % ", ".join(sem["responsables"]), footer, @@ -176,18 +171,17 @@ def check_access_diretud( return True, "" -def can_change_groups(context, REQUEST, formsemestre_id): +def can_change_groups(formsemestre_id): "Vrai si l'utilisateur peut changer les groupes dans ce semestre" from app.scodoc import sco_formsemestre + context = None # XXX #context sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) if sem["etat"] != "1": return False # semestre verrouillé - authuser = REQUEST.AUTHENTICATED_USER - if authuser.has_permission(Permission.ScoEtudChangeGroups): + if current_user.has_permission(Permission.ScoEtudChangeGroups): return True # admin, chef dept - uid = str(authuser) - if uid in sem["responsables"]: + if current_user.user_name in sem["responsables"]: return True return False diff --git a/app/scodoc/sco_placement.py b/app/scodoc/sco_placement.py index 49f04ae30..b03eb6430 100644 --- a/app/scodoc/sco_placement.py +++ b/app/scodoc/sco_placement.py @@ -412,7 +412,7 @@ def placement_eval_selectetuds(context, evaluation_id, REQUEST=None): page_title = 'Placement "%s"' % theeval["description"] else: page_title = "Placement des étudiants" - H = [html_sco_header.sco_header(context, REQUEST, page_title=page_title)] + H = [html_sco_header.sco_header(page_title=page_title)] formid = "placementfile" if not REQUEST.form.get("%s-submitted" % formid, False): diff --git a/app/scodoc/sco_poursuite_dut.py b/app/scodoc/sco_poursuite_dut.py index 42788d386..065b03d8f 100644 --- a/app/scodoc/sco_poursuite_dut.py +++ b/app/scodoc/sco_poursuite_dut.py @@ -170,7 +170,7 @@ def formsemestre_poursuite_report( """Table avec informations "poursuite" """ sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) etuds = _getEtudInfoGroupes( - context, [sco_groups.get_default_group(context, formsemestre_id)] + context, [sco_groups.get_default_group(formsemestre_id)] ) infos = [] diff --git a/app/scodoc/sco_preferences.py b/app/scodoc/sco_preferences.py index 5379a084a..615e72ad1 100644 --- a/app/scodoc/sco_preferences.py +++ b/app/scodoc/sco_preferences.py @@ -1958,7 +1958,7 @@ class BasePreferences(object): context = None # XXX TO REMOVE #context H = [ - html_sco_header.sco_header(context, REQUEST, page_title="Préférences"), + html_sco_header.sco_header(page_title="Préférences"), "

        Préférences globales pour %s

        " % scu.ScoURL(), """

        Ces paramètres s'appliquent par défaut à tous les semestres, sauf si ceux-ci définissent des valeurs spécifiques.

        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 bff3a1e4a..f4a288aa8 100644 --- a/app/scodoc/sco_pvjury.py +++ b/app/scodoc/sco_pvjury.py @@ -516,7 +516,7 @@ def formsemestre_pvjury( if not dpv: if format == "html": return ( - html_sco_header.sco_header(context, REQUEST) + html_sco_header.sco_header() + "

        Aucune information disponible !

        " + footer ) @@ -634,7 +634,7 @@ def formsemestre_pvjury_pdf( etuddescr = "" if not group_ids: # tous les inscrits du semestre - group_ids = [sco_groups.get_default_group(context, formsemestre_id)] + group_ids = [sco_groups.get_default_group(formsemestre_id)] groups_infos = sco_groups_view.DisplayedGroupsInfos( context, group_ids, formsemestre_id=formsemestre_id, REQUEST=REQUEST @@ -818,7 +818,7 @@ def formsemestre_lettres_individuelles( sem = sco_formsemestre.get_formsemestre(context, formsemestre_id) if not group_ids: # tous les inscrits du semestre - group_ids = [sco_groups.get_default_group(context, formsemestre_id)] + group_ids = [sco_groups.get_default_group(formsemestre_id)] groups_infos = sco_groups_view.DisplayedGroupsInfos( context, group_ids, formsemestre_id=formsemestre_id, REQUEST=REQUEST ) diff --git a/app/scodoc/sco_recapcomplet.py b/app/scodoc/sco_recapcomplet.py index 8b3e73c3f..a160029e5 100644 --- a/app/scodoc/sco_recapcomplet.py +++ b/app/scodoc/sco_recapcomplet.py @@ -95,8 +95,6 @@ def formsemestre_recapcomplet( if not isFile: H += [ html_sco_header.sco_header( - context, - REQUEST, page_title="Récapitulatif", no_side_bar=True, init_qtip=True, @@ -173,7 +171,7 @@ def formsemestre_recapcomplet( """

        Voir les décisions du jury

        """ % formsemestre_id ) - if sco_permissions_check.can_validate_sem(context, REQUEST, formsemestre_id): + if sco_permissions_check.can_validate_sem(formsemestre_id): H.append("

        ") if modejury: H.append( diff --git a/app/scodoc/sco_report.py b/app/scodoc/sco_report.py index 3164d66de..6d1bcb7c9 100644 --- a/app/scodoc/sco_report.py +++ b/app/scodoc/sco_report.py @@ -365,7 +365,7 @@ def formsemestre_report_counts( if format != "html": return t H = [ - html_sco_header.sco_header(context, REQUEST, page_title=title), + html_sco_header.sco_header(page_title=title), t, "\n".join(F), """

        Le tableau affiche le nombre d'étudiants de ce semestre dans chacun @@ -756,7 +756,7 @@ def formsemestre_suivi_cohorte( ) H = [ - html_sco_header.sco_header(context, REQUEST, page_title=tab.page_title), + html_sco_header.sco_header(page_title=tab.page_title), """

        Suivi cohorte: devenir des étudiants de ce semestre

        """, _gen_form_selectetuds( formsemestre_id, @@ -1229,8 +1229,6 @@ def formsemestre_suivi_parcours( H = [ html_sco_header.sco_header( - context, - REQUEST, page_title=tab.page_title, init_qtip=True, javascripts=["js/etud_info.js"], @@ -1559,8 +1557,6 @@ def formsemestre_graph_parcours( H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Parcours étudiants de %(titreannee)s" % sem, no_side_bar=True, ), diff --git a/app/scodoc/sco_saisie_notes.py b/app/scodoc/sco_saisie_notes.py index 2d51147c0..6c89e8fb5 100644 --- a/app/scodoc/sco_saisie_notes.py +++ b/app/scodoc/sco_saisie_notes.py @@ -311,7 +311,7 @@ def do_evaluation_set_missing( diag = "Valeur %s invalide" % value if diag: return ( - html_sco_header.sco_header(context, REQUEST) + html_sco_header.sco_header() + '

        %s

        Recommencer' % (diag, evaluation_id) + html_sco_header.sco_footer() @@ -348,7 +348,7 @@ def do_evaluation_set_missing( url=mod["url"], ) return ( - html_sco_header.sco_header(context, REQUEST) + html_sco_header.sco_header() + """

        %d notes changées

    " @@ -1971,7 +1961,7 @@ def formsemestre_validation_etud_manu( REQUEST=None, ): "Enregistre choix jury pour un étudiant" - if not sco_permissions_check.can_validate_sem(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_validate_sem(formsemestre_id): return scu.confirm_dialog( context, message="

    Opération non autorisée pour %s" @@ -2001,7 +1991,7 @@ def formsemestre_validate_previous_ue( context, formsemestre_id, etudid=None, REQUEST=None ): "Form. saisie UE validée hors ScoDoc " - if not sco_permissions_check.can_validate_sem(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_validate_sem(formsemestre_id): return scu.confirm_dialog( context, message="

    Opération non autorisée pour %s" @@ -2029,7 +2019,7 @@ def formsemestre_ext_edit_ue_validations( context, formsemestre_id, etudid=None, REQUEST=None ): "Form. edition UE semestre extérieur" - if not sco_permissions_check.can_validate_sem(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_validate_sem(formsemestre_id): return scu.confirm_dialog( context, message="

    Opération non autorisée pour %s" @@ -2054,7 +2044,7 @@ sco_publish( @scodoc7func(context) def etud_ue_suppress_validation(context, etudid, formsemestre_id, ue_id, REQUEST=None): """Suppress a validation (ue_id, etudid) and redirect to formsemestre""" - if not sco_permissions_check.can_validate_sem(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_validate_sem(formsemestre_id): return scu.confirm_dialog( context, message="

    Opération non autorisée pour %s" @@ -2072,7 +2062,7 @@ def etud_ue_suppress_validation(context, etudid, formsemestre_id, ue_id, REQUEST @scodoc7func(context) def formsemestre_validation_auto(context, formsemestre_id, REQUEST): "Formulaire saisie automatisee des decisions d'un semestre" - if not sco_permissions_check.can_validate_sem(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_validate_sem(formsemestre_id): return scu.confirm_dialog( context, message="

    Opération non autorisée pour %s" @@ -2091,7 +2081,7 @@ def formsemestre_validation_auto(context, formsemestre_id, REQUEST): @scodoc7func(context) def do_formsemestre_validation_auto(context, formsemestre_id, REQUEST): "Formulaire saisie automatisee des decisions d'un semestre" - if not sco_permissions_check.can_validate_sem(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_validate_sem(formsemestre_id): return scu.confirm_dialog( context, message="

    Opération non autorisée pour %s" @@ -2112,7 +2102,7 @@ def formsemestre_validation_suppress_etud( context, formsemestre_id, etudid, REQUEST=None, dialog_confirmed=False ): """Suppression des decisions de jury pour un etudiant.""" - if not sco_permissions_check.can_validate_sem(context, REQUEST, formsemestre_id): + if not sco_permissions_check.can_validate_sem(formsemestre_id): return scu.confirm_dialog( context, message="

    Opération non autorisée pour %s" diff --git a/app/views/scolar.py b/app/views/scolar.py index 54179ffa7..741fe9991 100644 --- a/app/views/scolar.py +++ b/app/views/scolar.py @@ -155,10 +155,7 @@ def about(context, REQUEST): ) d = "" return ( - html_sco_header.sco_header(context, REQUEST) - + "\n".join(H) - + d - + html_sco_header.sco_footer() + html_sco_header.sco_header() + "\n".join(H) + d + html_sco_header.sco_footer() ) @@ -519,9 +516,7 @@ def doAddAnnotation(context, etudid, comment, REQUEST): @scodoc7func(context) def doSuppressAnnotation(context, etudid, annotation_id, REQUEST): """Suppression annotation.""" - if not sco_permissions_check.can_suppress_annotation( - context, annotation_id, REQUEST - ): + if not sco_permissions_check.can_suppress_annotation(annotation_id): raise AccessDenied("Vous n'avez pas le droit d'effectuer cette opération !") cnx = ndb.GetDBConnexion() @@ -560,7 +555,7 @@ def formChangeCoordonnees(context, etudid, REQUEST): % etud ] header = html_sco_header.sco_header( - context, REQUEST, page_title="Changement adresse de %(nomprenom)s" % etud + page_title="Changement adresse de %(nomprenom)s" % etud ) tf = TrivialFormulator( @@ -722,7 +717,7 @@ def etud_photo_orig_page(context, etudid=None, REQUEST=None): "Page with photo in orig. size" etud = sco_etud.get_etud_info(etudid=etudid, filled=1, REQUEST=REQUEST)[0] H = [ - html_sco_header.sco_header(context, REQUEST, page_title=etud["nomprenom"]), + html_sco_header.sco_header(page_title=etud["nomprenom"]), "

    %s

    " % etud["nomprenom"], '
    ' % url_for("scolar.ficheEtud", scodoc_dept=g.scodoc_dept, etudid=etudid), @@ -744,7 +739,7 @@ def formChangePhoto(context, etudid=None, REQUEST=None): else: etud["photoloc"] = "externe" H = [ - html_sco_header.sco_header(context, REQUEST, page_title="Changement de photo"), + html_sco_header.sco_header(page_title="Changement de photo"), """

    Changement de la photo de %(nomprenom)s

    Photo actuelle (%(photoloc)s): """ @@ -868,8 +863,6 @@ def _formDem_of_Def( etud["operation_name"] = operation_name # header = html_sco_header.sco_header( - context, - REQUEST, page_title="%(operation_name)s de %(nomprenom)s (du semestre %(semtitre)s)" % etud, ) @@ -1101,7 +1094,7 @@ def etudident_edit_form(context, REQUEST=None): def _etudident_create_or_edit_form(context, REQUEST, edit): "Le formulaire HTML" - H = [html_sco_header.sco_header(context, REQUEST, init_jquery_ui=True)] + H = [html_sco_header.sco_header(init_jquery_ui=True)] F = html_sco_header.sco_footer() etudid = REQUEST.form.get("etudid", None) cnx = ndb.GetDBConnexion() @@ -1740,7 +1733,7 @@ def form_students_import_excel(context, REQUEST, formsemestre_id=None): if sem and sem["etat"] != "1": raise ScoValueError("Modification impossible: semestre verrouille") H = [ - html_sco_header.sco_header(context, REQUEST, page_title="Import etudiants"), + html_sco_header.sco_header(page_title="Import etudiants"), """

    Téléchargement d\'une nouvelle liste d\'etudiants

    A utiliser pour importer de nouveaux étudiants (typiquement au @@ -1886,9 +1879,7 @@ def import_generate_excel_sample(context, REQUEST, with_codesemestre="1"): @scodoc7func(context) def import_generate_admission_sample(context, REQUEST, formsemestre_id): "une feuille excel pour importation données admissions" - group = sco_groups.get_group( - context, sco_groups.get_default_group(context, formsemestre_id) - ) + group = sco_groups.get_group(context, sco_groups.get_default_group(formsemestre_id)) fmt = sco_import_etuds.sco_import_format() data = sco_import_etuds.sco_import_generate_excel_sample( fmt, @@ -1913,8 +1904,6 @@ def form_students_import_infos_admissions(context, REQUEST, formsemestre_id=None # autorise juste l'export H = [ html_sco_header.sco_header( - context, - REQUEST, page_title="Export données admissions (Parcoursup ou autre)", ), """

    Téléchargement des informations sur l'admission des étudiants

    @@ -1929,9 +1918,7 @@ def form_students_import_infos_admissions(context, REQUEST, formsemestre_id=None # On a le droit d'importer: H = [ - html_sco_header.sco_header( - context, REQUEST, page_title="Import données admissions Parcoursup" - ), + html_sco_header.sco_header(page_title="Import données admissions Parcoursup"), """

    Téléchargement des informations sur l'admission des étudiants depuis feuilles import Parcoursup

    A utiliser pour renseigner les informations sur l'origine des étudiants (lycées, bac, etc). Ces informations sont facultatives mais souvent utiles pour mieux connaitre les étudiants et aussi pour effectuer des statistiques (résultats suivant le type de bac...). Les données sont affichées sur les fiches individuelles des étudiants.

    diff --git a/app/views/users.py b/app/views/users.py index e9cc3478c..4df336d72 100644 --- a/app/views/users.py +++ b/app/views/users.py @@ -98,7 +98,7 @@ def create_user_form(context, REQUEST, user_name=None, edit=0): auth_dept = current_user.dept initvalues = {} edit = int(edit) - H = [html_sco_header.sco_header(context, REQUEST, bodyOnLoad="init_tf_form('')")] + H = [html_sco_header.sco_header(bodyOnLoad="init_tf_form('')")] F = html_sco_header.sco_footer() if edit: if not user_name: @@ -503,7 +503,7 @@ def form_change_password(REQUEST, user_name=None): u = current_user else: u = User.query.filter_by(user_name=user_name).first() - H = [html_sco_header.sco_header(context, REQUEST, user_check=False)] + H = [html_sco_header.sco_header(user_check=False)] F = html_sco_header.sco_footer() # check access if not can_handle_passwd(u): @@ -586,7 +586,7 @@ def change_password(user_name, password, password2, REQUEST): + '    Continuer' % scu.ScoURL() ) - return html_sco_header.sco_header(context, REQUEST) + "\n".join(H) + F + return html_sco_header.sco_header() + "\n".join(H) + F @bp.route("/delete_user_form", methods=["GET", "POST"]) @@ -599,7 +599,7 @@ def delete_user_form(REQUEST, user_name, dialog_confirmed=False): if not can_handle_passwd(u): # access denied (or non existent user) return ( - html_sco_header.sco_header(context, REQUEST, user_check=False) + html_sco_header.sco_header(user_check=False) + "

    Vous n'avez pas la permission de supprimer cet utilisateur

    " + html_sco_header.sco_footer() ) diff --git a/scotests/sco_fake_gen.py b/scotests/sco_fake_gen.py index 9bd12cab8..e36bd08d0 100644 --- a/scotests/sco_fake_gen.py +++ b/scotests/sco_fake_gen.py @@ -233,7 +233,7 @@ class ScoFake(object): etapes=None, responsables=["bach"], ): - oid = sco_formsemestre.do_formsemestre_create(self.context, locals(), REQUEST) + oid = sco_formsemestre.do_formsemestre_create(locals()) # oids = self.context.do_formsemestre_list(args={"formsemestre_id": oid}) oids = sco_formsemestre.do_formsemestre_list( self.context, args={"formsemestre_id": oid} diff --git a/scotests/test_absence.py b/scotests/test_absence.py index 9913cc462..72de4efc8 100644 --- a/scotests/test_absence.py +++ b/scotests/test_absence.py @@ -203,9 +203,7 @@ _ = sco_groups.partition_create( REQUEST=REQUEST, ) li1 = sco_groups.get_partitions_list(context.Scolarite, sem["formsemestre_id"]) -_ = sco_groups.createGroup( - context.Scolarite, li1[0]["partition_id"], "Groupe 1", REQUEST=REQUEST -) +_ = sco_groups.createGroup(context.Scolarite, li1[0]["partition_id"], "Groupe 1") # --- Affectation des élèves dans des groupes diff --git a/scotests/test_absence2.py b/scotests/test_absence2.py index d141b0f41..0fbeb5fd7 100644 --- a/scotests/test_absence2.py +++ b/scotests/test_absence2.py @@ -90,9 +90,7 @@ _ = sco_groups.partition_create( REQUEST=REQUEST, ) li1 = sco_groups.get_partitions_list(context.Scolarite, sem["formsemestre_id"]) -_ = sco_groups.createGroup( - context.Scolarite, li1[0]["partition_id"], "Groupe 1", REQUEST=REQUEST -) +_ = sco_groups.createGroup(context.Scolarite, li1[0]["partition_id"], "Groupe 1") # --- Affectation des élèves dans des groupes diff --git a/scotests/test_api7doc.py b/scotests/test_api7doc.py index a1d6133ac..0a3596e92 100644 --- a/scotests/test_api7doc.py +++ b/scotests/test_api7doc.py @@ -1,6 +1,7 @@ import sco_formations import json import random + # La variable context est définie par le script de lancement # l'affecte ainsi pour évietr les warnins pylint: context = context # pylint: disable=undefined-variable @@ -21,7 +22,7 @@ file.close() f = sco_formations.formation_import_xml(REQUEST=REQUEST, doc=doc, context=context.Notes) -# --- Création des semestres +# --- Création des semestres sem1 = G.create_formsemestre( formation_id=f[0], @@ -56,26 +57,26 @@ sem4 = G.create_formsemestre( li_module = context.Notes.do_module_list() mods_imp = [] -for mod in li_module : - if mod["semestre_id"] == 1 : +for mod in li_module: + if mod["semestre_id"] == 1: formsemestre_id = sem1["formsemestre_id"] - elif mod["semestre_id"] == 2 : + elif mod["semestre_id"] == 2: formsemestre_id = sem2["formsemestre_id"] - elif mod["semestre_id"] == 3 : + elif mod["semestre_id"] == 3: formsemestre_id = sem3["formsemestre_id"] - else : + else: formsemestre_id = sem4["formsemestre_id"] mi = G.create_moduleimpl( - module_id=mod["module_id"], - formsemestre_id=formsemestre_id, - responsable_id="bach", + module_id=mod["module_id"], + formsemestre_id=formsemestre_id, + responsable_id="bach", ) mods_imp.append(mi) # --- Création des étudiants -etuds=[] +etuds = [] for nom, prenom in [ ("Semestre11", "EtudiantNumero1"), ("Semestre12", "EtudiantNumero2"), @@ -84,8 +85,8 @@ for nom, prenom in [ ("Semestre35", "EtudiantNumero5"), ("Semestre36", "EtudiantNumero6"), ("Semestre47", "EtudiantNumero7"), - ("Semestre48", "EtudiantNumero8") -] : + ("Semestre48", "EtudiantNumero8"), +]: etud = G.create_etud( nom=nom, prenom=prenom, @@ -108,16 +109,24 @@ for etud in etuds[6:]: # --- Création d'une évaluation pour chaque UE -lim_sem1 = sco_moduleimpl.do_moduleimpl_list(context.Notes, formsemestre_id=sem1["formsemestre_id"], REQUEST=REQUEST) +lim_sem1 = sco_moduleimpl.do_moduleimpl_list( + context.Notes, formsemestre_id=sem1["formsemestre_id"], REQUEST=REQUEST +) load_lim_sem1 = json.loads(lim_sem1) -lim_sem2 = sco_moduleimpl.do_moduleimpl_list(context.Notes, formsemestre_id=sem2["formsemestre_id"], REQUEST=REQUEST) +lim_sem2 = sco_moduleimpl.do_moduleimpl_list( + context.Notes, formsemestre_id=sem2["formsemestre_id"], REQUEST=REQUEST +) load_lim_sem2 = json.loads(lim_sem2) -lim_sem3 = sco_moduleimpl.do_moduleimpl_list(context.Notes, formsemestre_id=sem3["formsemestre_id"], REQUEST=REQUEST) +lim_sem3 = sco_moduleimpl.do_moduleimpl_list( + context.Notes, formsemestre_id=sem3["formsemestre_id"], REQUEST=REQUEST +) load_lim_sem3 = json.loads(lim_sem3) -lim_sem4 = sco_moduleimpl.do_moduleimpl_list(context.Notes, formsemestre_id=sem4["formsemestre_id"], REQUEST=REQUEST) +lim_sem4 = sco_moduleimpl.do_moduleimpl_list( + context.Notes, formsemestre_id=sem4["formsemestre_id"], REQUEST=REQUEST +) load_lim_sem4 = json.loads(lim_sem4) @@ -132,45 +141,74 @@ for moduleimpl_id, jour, description, coefficient in [ (load_lim_sem4[3]["moduleimpl_id"], "03/02/2021", "InterroTestSemestre4", 1.0), (load_lim_sem4[9]["moduleimpl_id"], "04/02/2021", "InterroTestSemestre4", 1.0), (load_lim_sem4[13]["moduleimpl_id"], "05/02/2021", "InterroTestSemestre4", 1.0), -] : - e = G.create_evaluation(moduleimpl_id=moduleimpl_id, jour=jour, description=description, coefficient=coefficient) +]: + e = G.create_evaluation( + moduleimpl_id=moduleimpl_id, + jour=jour, + description=description, + coefficient=coefficient, + ) # --- Saisie des notes des étudiants (un élève a 12, un autre a 7 pour chaque semestre) -lie1 = context.Notes.do_evaluation_list_in_formsemestre(formsemestre_id=sem1["formsemestre_id"]) -lie2 = context.Notes.do_evaluation_list_in_formsemestre(formsemestre_id=sem2["formsemestre_id"]) -lie3 = context.Notes.do_evaluation_list_in_formsemestre(formsemestre_id=sem3["formsemestre_id"]) -lie4 = context.Notes.do_evaluation_list_in_formsemestre(formsemestre_id=sem4["formsemestre_id"]) +lie1 = context.Notes.do_evaluation_list_in_formsemestre( + formsemestre_id=sem1["formsemestre_id"] +) +lie2 = context.Notes.do_evaluation_list_in_formsemestre( + formsemestre_id=sem2["formsemestre_id"] +) +lie3 = context.Notes.do_evaluation_list_in_formsemestre( + formsemestre_id=sem3["formsemestre_id"] +) +lie4 = context.Notes.do_evaluation_list_in_formsemestre( + formsemestre_id=sem4["formsemestre_id"] +) -for eval in lie1 : - for etud in etuds[:2] : - if etud == etuds[0] : - nb_changed, nb_suppress, existing_decisions = G.create_note(evaluation=eval, etud=etud, note=12.0) - else : - nb_changed, nb_suppress, existing_decisions = G.create_note(evaluation=eval, etud=etud, note=7.0) +for eval in lie1: + for etud in etuds[:2]: + if etud == etuds[0]: + nb_changed, nb_suppress, existing_decisions = G.create_note( + evaluation=eval, etud=etud, note=12.0 + ) + else: + nb_changed, nb_suppress, existing_decisions = G.create_note( + evaluation=eval, etud=etud, note=7.0 + ) -for eval in lie2 : - for etud in etuds[2:4] : - if etud == etuds[2] : - nb_changed, nb_suppress, existing_decisions = G.create_note(evaluation=eval, etud=etud, note=12.0) - else : - nb_changed, nb_suppress, existing_decisions = G.create_note(evaluation=eval, etud=etud, note=7.0) +for eval in lie2: + for etud in etuds[2:4]: + if etud == etuds[2]: + nb_changed, nb_suppress, existing_decisions = G.create_note( + evaluation=eval, etud=etud, note=12.0 + ) + else: + nb_changed, nb_suppress, existing_decisions = G.create_note( + evaluation=eval, etud=etud, note=7.0 + ) -for eval in lie3 : - for etud in etuds[4:6] : - if etud == etuds[4] : - nb_changed, nb_suppress, existing_decisions = G.create_note(evaluation=eval, etud=etud, note=12.0) - else : - nb_changed, nb_suppress, existing_decisions = G.create_note(evaluation=eval, etud=etud, note=7.0) +for eval in lie3: + for etud in etuds[4:6]: + if etud == etuds[4]: + nb_changed, nb_suppress, existing_decisions = G.create_note( + evaluation=eval, etud=etud, note=12.0 + ) + else: + nb_changed, nb_suppress, existing_decisions = G.create_note( + evaluation=eval, etud=etud, note=7.0 + ) -for eval in lie4 : - for etud in etuds[6:] : - if etud == etuds[6] : - nb_changed, nb_suppress, existing_decisions = G.create_note(evaluation=eval, etud=etud, note=12.0) - else : - nb_changed, nb_suppress, existing_decisions = G.create_note(evaluation=eval, etud=etud, note=7.0) +for eval in lie4: + for etud in etuds[6:]: + if etud == etuds[6]: + nb_changed, nb_suppress, existing_decisions = G.create_note( + evaluation=eval, etud=etud, note=12.0 + ) + else: + nb_changed, nb_suppress, existing_decisions = G.create_note( + evaluation=eval, etud=etud, note=7.0 + ) # --- Département @@ -184,45 +222,68 @@ load_lif = json.loads(lif) assert len(load_lif) == 1 assert load_lif[0]["formation_id"] == f[0] -exp = sco_formations.formation_export(context.Notes, formation_id=f[0], format="json", REQUEST=REQUEST) +exp = sco_formations.formation_export( + context.Notes, formation_id=f[0], format="json", REQUEST=REQUEST +) load_exp = json.loads(exp) assert load_exp["acronyme"] == "DUT Info" assert load_exp["titre_officiel"] == "DUT Informatique" assert load_exp["formation_code"] == "FCOD2" -# --- Semestre +# --- Semestre -li_sem = context.Notes.formsemestre_list(formation_id=f[0], format = "json", REQUEST=REQUEST) +li_sem = context.Notes.formsemestre_list( + formation_id=f[0], format="json", REQUEST=REQUEST +) load_li_sem = json.loads(li_sem) assert len(load_li_sem) == 4 # --- Création des groupes -_ = sco_groups.partition_create(context.Scolarite, formsemestre_id=sem1["formsemestre_id"], partition_name="Eleve 1ere annee", REQUEST=REQUEST) +_ = sco_groups.partition_create( + context.Scolarite, + formsemestre_id=sem1["formsemestre_id"], + partition_name="Eleve 1ere annee", + REQUEST=REQUEST, +) li1 = sco_groups.get_partitions_list(context.Scolarite, sem1["formsemestre_id"]) -_ = sco_groups.createGroup(context.Scolarite, li1[0]["partition_id"], "Groupe S1A", REQUEST=REQUEST) +_ = sco_groups.createGroup(context.Scolarite, li1[0]["partition_id"], "Groupe S1A") li_grp1 = sco_groups.get_partition_groups(context.Scolarite, li1[0]) -for etud in etuds[0:2] : +for etud in etuds[0:2]: sco_groups.set_group(context.Scolarite, etud["etudid"], li_grp1[0]["group_id"]) # --- Groupes -li_part = sco_groups.formsemestre_partition_list(context.Scolarite, formsemestre_id=sem1["formsemestre_id"], format = "json", REQUEST=REQUEST) +li_part = sco_groups.formsemestre_partition_list( + context.Scolarite, + formsemestre_id=sem1["formsemestre_id"], + format="json", + REQUEST=REQUEST, +) load_li_part = json.loads(li_part) -assert len(load_li_part) == 2 # 2 partition (defaut et eleve 1ere annee) +assert len(load_li_part) == 2 # 2 partition (defaut et eleve 1ere annee) assert load_li_part[0]["formsemestre_id"] == sem1["formsemestre_id"] -assert len(load_li_part[0]["group"]) == 1 # 2 groupes S1A +assert len(load_li_part[0]["group"]) == 1 # 2 groupes S1A assert load_li_part[0]["group"][0]["group_name"] == "Groupe S1A" assert load_li_part[0]["group"][0]["group_id"] == li_grp1[0]["group_id"] -assert load_li_part[0]["partition_id"] == load_li_part[0]["group"][0]["partition_id"] == li1[0]["partition_id"] +assert ( + load_li_part[0]["partition_id"] + == load_li_part[0]["group"][0]["partition_id"] + == li1[0]["partition_id"] +) assert load_li_part[0]["partition_name"] == "Eleve 1ere annee" -vue_g1 = sco_groups_view.groups_view(context.Scolarite, group_ids = [li_grp1[0]["group_id"]], format = "json", REQUEST=REQUEST) +vue_g1 = sco_groups_view.groups_view( + context.Scolarite, + group_ids=[li_grp1[0]["group_id"]], + format="json", + REQUEST=REQUEST, +) load_vue_g1 = json.loads(vue_g1) assert len(load_vue_g1) == 2 assert load_vue_g1[0][li1[0]["partition_id"]] == li_grp1[0]["group_name"] @@ -230,32 +291,49 @@ assert load_vue_g1[0][li1[0]["partition_id"]] == li_grp1[0]["group_name"] # --- Etudiant -etudi = context.Scolarite.etud_info(etudid=etuds[0]["etudid"], format="json", REQUEST=REQUEST) +etudi = context.Scolarite.etud_info( + etudid=etuds[0]["etudid"], format="json", REQUEST=REQUEST +) load_etudi = json.loads(etudi) assert load_etudi["prenom"] == "Etudiantnumero1" assert load_etudi["nom"] == "SEMESTRE11" assert load_etudi["etudid"] == etuds[0]["etudid"] -# --- Absences +# --- Absences for debut, fin, demijournee in [ ("18/01/2021", "18/01/2021", 1), ("19/01/2021", "19/01/2021", 0), -] : - sco_abs_views.doSignaleAbsence(context.Absences, datedebut=debut, datefin=fin, demijournee=demijournee, etudid=etuds[0]["etudid"], REQUEST=REQUEST) +]: + sco_abs_views.doSignaleAbsence( + context.Absences, + datedebut=debut, + datefin=fin, + demijournee=demijournee, + etudid=etuds[0]["etudid"], + REQUEST=REQUEST, + ) -nb_abs = context.Absences.CountAbs(etuds[0]["etudid"], debut="09/01/2020", fin="02/01/2021") +nb_abs = context.Absences.CountAbs( + etuds[0]["etudid"], debut="09/01/2020", fin="02/01/2021" +) assert nb_abs == 2 -liste_abs = sco_abs_views.ListeAbsEtud(context.Absences, etuds[0]["etudid"], format='json', REQUEST=REQUEST) +liste_abs = sco_abs_views.ListeAbsEtud( + context.Absences, etuds[0]["etudid"], format="json", REQUEST=REQUEST +) load_liste_abs = json.loads(liste_abs) assert len(load_liste_abs) == 2 -_ = sco_abs_views.doAnnuleAbsence(context.Absences, "15/01/2021", "15/01/2021", 1, etudid=etuds[0]["etudid"], REQUEST=REQUEST) +_ = sco_abs_views.doAnnuleAbsence( + context.Absences, + "15/01/2021", + "15/01/2021", + 1, + etudid=etuds[0]["etudid"], + REQUEST=REQUEST, +) # --- Module -#Voir test dans test_formation. - - - +# Voir test dans test_formation. diff --git a/scotests/test_groupes.py b/scotests/test_groupes.py index 745890f01..62c59a861 100644 --- a/scotests/test_groupes.py +++ b/scotests/test_groupes.py @@ -71,18 +71,18 @@ assert len(etuds) == 20 # --- Création d'une formation f = G.create_formation( - acronyme="DUTI", - titre="DUT Info", - titre_officiel="DUT Informatique", - ) + acronyme="DUTI", + titre="DUT Info", + titre_officiel="DUT Informatique", +) + +assert f["acronyme"] == "DUTI" +assert f["titre"] == "DUT Info" +assert f["titre_officiel"] == "DUT Informatique" -assert f["acronyme"]=="DUTI" -assert f["titre"]=="DUT Info" -assert f["titre_officiel"]=="DUT Informatique" - # --- Création d'UE, matière, module pour les premieres années - + ue1 = G.create_ue(formation_id=f["formation_id"], acronyme="UE11", titre="UE1S1") mat1 = G.create_matiere(ue_id=ue1["ue_id"], titre="mat1") @@ -96,16 +96,16 @@ mod1 = G.create_module( ) assert ue1["formation_id"] == f["formation_id"] -assert ue1["acronyme"]=="UE11" -assert ue1["titre"]=="UE1S1" +assert ue1["acronyme"] == "UE11" +assert ue1["titre"] == "UE1S1" -assert mod1["matiere_id"]==mat1["matiere_id"] -assert mod1["code"]=="M1S1" -assert mod1["titre"]=="mod1" -assert mod1["ue_id"]==ue1["ue_id"] -assert mod1["formation_id"]==f["formation_id"] +assert mod1["matiere_id"] == mat1["matiere_id"] +assert mod1["code"] == "M1S1" +assert mod1["titre"] == "mod1" +assert mod1["ue_id"] == ue1["ue_id"] +assert mod1["formation_id"] == f["formation_id"] -# --- Création d'UE, matière, module pour les deuxieme années +# --- Création d'UE, matière, module pour les deuxieme années ue2 = G.create_ue(formation_id=f["formation_id"], acronyme="UE31", titre="UE1S1") @@ -128,10 +128,10 @@ sem1 = G.create_formsemestre( date_fin="01/02/2021", ) -assert sem1["formation_id"]==f["formation_id"] -assert sem1["semestre_id"]==1 -assert sem1["date_debut"]=="01/09/2020" -assert sem1["date_fin"]=="01/02/2021" +assert sem1["formation_id"] == f["formation_id"] +assert sem1["semestre_id"] == 1 +assert sem1["date_debut"] == "01/09/2020" +assert sem1["date_fin"] == "01/02/2021" sem2 = G.create_formsemestre( formation_id=f["formation_id"], @@ -147,9 +147,9 @@ mi1 = G.create_moduleimpl( responsable_id="bach", ) -assert mi1["module_id" ]==mod1["module_id"] -assert mi1["formsemestre_id"]==sem1["formsemestre_id"] -assert mi1["responsable_id"]=="bach" +assert mi1["module_id"] == mod1["module_id"] +assert mi1["formsemestre_id"] == sem1["formsemestre_id"] +assert mi1["responsable_id"] == "bach" mi2 = G.create_moduleimpl( module_id=mod2["module_id"], @@ -160,16 +160,26 @@ mi2 = G.create_moduleimpl( # --- Inscription des étudiants -for etud in etuds[:10] : +for etud in etuds[:10]: G.inscrit_etudiant(sem1, etud) - -for etud in etuds [10:] : + +for etud in etuds[10:]: G.inscrit_etudiant(sem2, etud) - + # --- Création de 2 partitions -_ = sco_groups.partition_create(context.Scolarite, formsemestre_id=sem1["formsemestre_id"], partition_name="Eleve 1ere annee", REQUEST=REQUEST) -_ = sco_groups.partition_create(context.Scolarite, formsemestre_id=sem2["formsemestre_id"], partition_name="Eleve 2eme annee", REQUEST=REQUEST) +_ = sco_groups.partition_create( + context.Scolarite, + formsemestre_id=sem1["formsemestre_id"], + partition_name="Eleve 1ere annee", + REQUEST=REQUEST, +) +_ = sco_groups.partition_create( + context.Scolarite, + formsemestre_id=sem2["formsemestre_id"], + partition_name="Eleve 2eme annee", + REQUEST=REQUEST, +) li1 = sco_groups.get_partitions_list(context.Scolarite, sem1["formsemestre_id"]) @@ -177,54 +187,64 @@ li2 = sco_groups.get_partitions_list(context.Scolarite, sem2["formsemestre_id"]) # --- Création des groupes - -_ = sco_groups.createGroup(context.Scolarite, li1[0]["partition_id"], "Groupe S1A", REQUEST=REQUEST) -_ = sco_groups.createGroup(context.Scolarite, li1[0]["partition_id"], "Groupe S1B", REQUEST=REQUEST) -_ = sco_groups.createGroup(context.Scolarite, li2[0]["partition_id"], "Groupe S3A", REQUEST=REQUEST) -_ = sco_groups.createGroup(context.Scolarite, li2[0]["partition_id"], "Groupe S3B", REQUEST=REQUEST) -_ = sco_groups.createGroup(context.Scolarite, li2[0]["partition_id"], "Groupe TEST", REQUEST=REQUEST) + +_ = sco_groups.createGroup(context.Scolarite, li1[0]["partition_id"], "Groupe S1A") +_ = sco_groups.createGroup(context.Scolarite, li1[0]["partition_id"], "Groupe S1B") +_ = sco_groups.createGroup(context.Scolarite, li2[0]["partition_id"], "Groupe S3A") +_ = sco_groups.createGroup(context.Scolarite, li2[0]["partition_id"], "Groupe S3B") +_ = sco_groups.createGroup(context.Scolarite, li2[0]["partition_id"], "Groupe TEST") li_grp1 = sco_groups.get_partition_groups(context.Scolarite, li1[0]) li_grp2 = sco_groups.get_partition_groups(context.Scolarite, li2[0]) -li_grp3 = sco_groups.get_partition_groups(context.Scolarite, li1[1]) #liste groupe defaut +li_grp3 = sco_groups.get_partition_groups( + context.Scolarite, li1[1] +) # liste groupe defaut -assert len(li_grp1) == 2 #test de get_partition_groups # 2 -assert len(li_grp2) == 3 #test de get_partition_groups # 3 +assert len(li_grp1) == 2 # test de get_partition_groups # 2 +assert len(li_grp2) == 3 # test de get_partition_groups # 3 assert li_grp1[0]["group_name"] == "Groupe S1A" # --- Affectation des élèves dans les groupes -for etud in etuds[:5] : +for etud in etuds[:5]: sco_groups.set_group(context.Scolarite, etud["etudid"], li_grp1[0]["group_id"]) - -for etud in etuds[5:10] : + +for etud in etuds[5:10]: sco_groups.set_group(context.Scolarite, etud["etudid"], li_grp1[1]["group_id"]) - -for etud in etuds[10:15] : + +for etud in etuds[10:15]: sco_groups.set_group(context.Scolarite, etud["etudid"], li_grp2[0]["group_id"]) -for etud in etuds[15:] : +for etud in etuds[15:]: sco_groups.set_group(context.Scolarite, etud["etudid"], li_grp2[1]["group_id"]) - + # --- Vérification si les élèves sont bien dans les bons groupes -for etud in etuds[:5] : - grp = sco_groups.get_etud_groups(context.Scolarite, etud["etudid"], sem1, exclude_default=True) +for etud in etuds[:5]: + grp = sco_groups.get_etud_groups( + context.Scolarite, etud["etudid"], sem1, exclude_default=True + ) assert grp[0]["group_name"] == "Groupe S1A" -for etud in etuds[5:10] : - grp = sco_groups.get_etud_groups(context.Scolarite, etud["etudid"], sem1, exclude_default=True) - assert grp[0]["group_name"] == "Groupe S1B" +for etud in etuds[5:10]: + grp = sco_groups.get_etud_groups( + context.Scolarite, etud["etudid"], sem1, exclude_default=True + ) + assert grp[0]["group_name"] == "Groupe S1B" -for etud in etuds[10:15] : - grp = sco_groups.get_etud_groups(context.Scolarite, etud["etudid"], sem2, exclude_default=True) +for etud in etuds[10:15]: + grp = sco_groups.get_etud_groups( + context.Scolarite, etud["etudid"], sem2, exclude_default=True + ) assert grp[0]["group_name"] == "Groupe S3A" -for etud in etuds[15:] : - grp = sco_groups.get_etud_groups(context.Scolarite, etud["etudid"], sem2, exclude_default=True) +for etud in etuds[15:]: + grp = sco_groups.get_etud_groups( + context.Scolarite, etud["etudid"], sem2, exclude_default=True + ) assert grp[0]["group_name"] == "Groupe S3B" - + # --- Création d'une évaluation e1 = G.create_evaluation( @@ -243,132 +263,181 @@ e2 = G.create_evaluation( # --- Saisie des notes -for etud in etuds[10:] : +for etud in etuds[10:]: nb_changed, nb_suppress, existing_decisions = G.create_note( evaluation=e1, etud=etud, note=float(random.randint(0, 20)) ) - - -for etud in etuds[:10] : + + +for etud in etuds[:10]: nb_changed, nb_suppress, existing_decisions = G.create_note( evaluation=e2, etud=etud, note=float(random.randint(0, 20)) ) - -# --- Liste des étudiants inscrits aux evaluations -lie1 = sco_groups.do_evaluation_listeetuds_groups(context.Scolarite, e1["evaluation_id"], groups = li_grp1) -lie2 = sco_groups.do_evaluation_listeetuds_groups(context.Scolarite, e2["evaluation_id"], groups = li_grp2) +# --- Liste des étudiants inscrits aux evaluations -for etud in etuds[:10] : - assert etud["etudid"] in lie1 # test de do_evaluation_listeetuds_groups +lie1 = sco_groups.do_evaluation_listeetuds_groups( + context.Scolarite, e1["evaluation_id"], groups=li_grp1 +) +lie2 = sco_groups.do_evaluation_listeetuds_groups( + context.Scolarite, e2["evaluation_id"], groups=li_grp2 +) + +for etud in etuds[:10]: + assert etud["etudid"] in lie1 # test de do_evaluation_listeetuds_groups + +for etud in etuds[10:]: + assert etud["etudid"] in lie2 # test de do_evaluation_listeetuds_groups -for etud in etuds[10:] : - assert etud["etudid"] in lie2 # test de do_evaluation_listeetuds_groups - # --- Liste des groupes présents aux évaluation -lig1 = sco_groups.do_evaluation_listegroupes(context.Scolarite, e1["evaluation_id"], include_default=False) -lig2 = sco_groups.do_evaluation_listegroupes(context.Scolarite, e2["evaluation_id"], include_default=False) +lig1 = sco_groups.do_evaluation_listegroupes( + context.Scolarite, e1["evaluation_id"], include_default=False +) +lig2 = sco_groups.do_evaluation_listegroupes( + context.Scolarite, e2["evaluation_id"], include_default=False +) + - assert len(lig1) == 2 assert len(lig2) == 2 assert li_grp1[0] and li_grp1[1] in lig1 # test do_evaluation_listegroupes assert li_grp2[0] and li_grp2[1] in lig2 # test do_evaluation_listegroupes - # --- Changement de groupe d'un élève -grp1 = sco_groups.get_etud_groups(context.Scolarite, etuds[0]["etudid"], sem1, exclude_default = True) -sco_groups.change_etud_group_in_partition(context.Scolarite, etuds[0]["etudid"], li_grp1[1]["group_id"], li1[0], REQUEST=REQUEST) -grp2 = sco_groups.get_etud_groups(context.Scolarite, etuds[0]["etudid"], sem1, exclude_default = True) +grp1 = sco_groups.get_etud_groups( + context.Scolarite, etuds[0]["etudid"], sem1, exclude_default=True +) +sco_groups.change_etud_group_in_partition( + context.Scolarite, + etuds[0]["etudid"], + li_grp1[1]["group_id"], + li1[0], + REQUEST=REQUEST, +) +grp2 = sco_groups.get_etud_groups( + context.Scolarite, etuds[0]["etudid"], sem1, exclude_default=True +) assert grp1 != grp2 -assert grp1[0] == li_grp1[0] # test get_etud_groups -assert grp2[0]["group_name"] == "Groupe S1B" #test du changement de groupe +assert grp1[0] == li_grp1[0] # test get_etud_groups +assert grp2[0]["group_name"] == "Groupe S1B" # test du changement de groupe -# --- Liste des partitions en format json +# --- Liste des partitions en format json -lijson_s1 = sco_groups.formsemestre_partition_list(context.Scolarite, formsemestre_id=sem1["formsemestre_id"], format = "json", REQUEST=REQUEST) +lijson_s1 = sco_groups.formsemestre_partition_list( + context.Scolarite, + formsemestre_id=sem1["formsemestre_id"], + format="json", + REQUEST=REQUEST, +) load_lijson_s1 = json.loads(lijson_s1) -assert len(load_lijson_s1) == 2 # 2 partition (defaut et eleve 1ere annee) +assert len(load_lijson_s1) == 2 # 2 partition (defaut et eleve 1ere annee) assert load_lijson_s1[0]["formsemestre_id"] == sem1["formsemestre_id"] -assert len(load_lijson_s1[0]["group"]) == 2 # 2 groupes S1A et S1B +assert len(load_lijson_s1[0]["group"]) == 2 # 2 groupes S1A et S1B assert load_lijson_s1[0]["group"][0]["group_name"] == "Groupe S1A" assert load_lijson_s1[0]["group"][0]["group_id"] == li_grp1[0]["group_id"] -assert load_lijson_s1[0]["partition_id"] == load_lijson_s1[0]["group"][0]["partition_id"] == li1[0]["partition_id"] +assert ( + load_lijson_s1[0]["partition_id"] + == load_lijson_s1[0]["group"][0]["partition_id"] + == li1[0]["partition_id"] +) assert load_lijson_s1[0]["partition_name"] == "Eleve 1ere annee" - # --- Vue d'un groupes (liste d'élève en format json) -vue_g1 = sco_groups_view.groups_view(context.Scolarite, group_ids = [li_grp1[0]["group_id"]], format = "json", REQUEST=REQUEST) +vue_g1 = sco_groups_view.groups_view( + context.Scolarite, + group_ids=[li_grp1[0]["group_id"]], + format="json", + REQUEST=REQUEST, +) load_vue_g1 = json.loads(vue_g1) assert len(load_vue_g1) == 4 assert load_vue_g1[0][li1[0]["partition_id"]] == li_grp1[0]["group_name"] -vue_sem = sco_groups_view.groups_view(context.Scolarite, formsemestre_id=sem1["formsemestre_id"], format = "json", REQUEST=REQUEST) +vue_sem = sco_groups_view.groups_view( + context.Scolarite, + formsemestre_id=sem1["formsemestre_id"], + format="json", + REQUEST=REQUEST, +) load_vue_sem = json.loads(vue_sem) assert len(load_vue_sem) == 10 -tab=[] -val=False -for etud in etuds[:10] : - for i in range(len(load_vue_sem)) : - if etud["prenom"] == load_vue_sem[i]["prenom"] and etud["nom_disp"]==load_vue_sem[i]["nom_disp"] : +tab = [] +val = False +for etud in etuds[:10]: + for i in range(len(load_vue_sem)): + if ( + etud["prenom"] == load_vue_sem[i]["prenom"] + and etud["nom_disp"] == load_vue_sem[i]["nom_disp"] + ): val = True tab.append(val) -assert not False in tab #tout mes etudiants sont present dans vue_sem. +assert not False in tab # tout mes etudiants sont present dans vue_sem. # --- Test des fonctions dans sco_groups -assert li_grp1[0] == sco_groups.get_group(context.Scolarite, li_grp1[0]["group_id"]) #test get_group +assert li_grp1[0] == sco_groups.get_group( + context.Scolarite, li_grp1[0]["group_id"] +) # test get_group assert len(li_grp2) == 3 sco_groups.group_delete(context.Scolarite, li_grp2[2]) -#assert len(li_grp2) == 2 #TEST DE group_delete, aucun changement sur la console mais se supprime sur scodoc web - # mais pas dans la console comme pour countAbs() - -assert sco_groups.get_partition(context.Scolarite, li1[0]["partition_id"])== li1[0] # test de get_partition -assert sco_groups.get_partition(context.Scolarite, li2[0]["partition_id"])== li2[0] # test de get_partition +# assert len(li_grp2) == 2 #TEST DE group_delete, aucun changement sur la console mais se supprime sur scodoc web +# mais pas dans la console comme pour countAbs() +assert ( + sco_groups.get_partition(context.Scolarite, li1[0]["partition_id"]) == li1[0] +) # test de get_partition +assert ( + sco_groups.get_partition(context.Scolarite, li2[0]["partition_id"]) == li2[0] +) # test de get_partition li1 = sco_groups.get_partitions_list(context.Scolarite, sem1["formsemestre_id"]) -#assert p1 in li1 #test de get_partitions_list -assert len(li1) == 2 #eleve de 1ere annee + la partition defaut +# assert p1 in li1 #test de get_partitions_list +assert len(li1) == 2 # eleve de 1ere annee + la partition defaut li2 = sco_groups.get_partitions_list(context.Scolarite, sem2["formsemestre_id"]) -#assert p2 in li2 #test de get_partitions_list -assert len(li2) == 2 #eleve de 2eme annee + la partition defaut +# assert p2 in li2 #test de get_partitions_list +assert len(li2) == 2 # eleve de 2eme annee + la partition defaut dp1 = sco_groups.get_default_partition(context.Scolarite, sem1["formsemestre_id"]) dp2 = sco_groups.get_default_partition(context.Scolarite, sem2["formsemestre_id"]) -assert dp1 in li1 # test si dp1 est bien dans li1 et par consequent teste la fonction get_default_partition -assert dp2 in li2 # test si dp2 est bien dans li1 et par consequent teste la fonction get_default_partition +assert ( + dp1 in li1 +) # test si dp1 est bien dans li1 et par consequent teste la fonction get_default_partition +assert ( + dp2 in li2 +) # test si dp2 est bien dans li1 et par consequent teste la fonction get_default_partition -dg1 = sco_groups.get_default_group(context.Scolarite, sem1["formsemestre_id"], REQUEST=REQUEST) -assert li_grp3[0]["group_id"] == dg1 #test de get_default_group +dg1 = sco_groups.get_default_group(sem1["formsemestre_id"]) +assert li_grp3[0]["group_id"] == dg1 # test de get_default_group sg = sco_groups.get_sem_groups(context.Scolarite, sem1["formsemestre_id"]) -assert len(sg) == 3 #test de get_sem_groups +assert len(sg) == 3 # test de get_sem_groups assert li_grp1[0] and li_grp1[1] in sg -assert li_grp3[0] in sg # test de get_sem_groups +assert li_grp3[0] in sg # test de get_sem_groups limembre = sco_groups.get_group_members(context.Scolarite, li_grp1[0]["group_id"]) -assert len(limembre) == 4 # car on a changé de groupe un etudiant de ce groupe donc 5-1=4 +assert ( + len(limembre) == 4 +) # car on a changé de groupe un etudiant de ce groupe donc 5-1=4 """ Commentaire : @@ -377,19 +446,3 @@ Meme probleme que pour les groupes, lorsque l'on supprime un groupe il est toujo mais pas dans scodoc web. """ - - - - - - - - - - - - - - - - diff --git a/tests/unit/test_caches.py b/tests/unit/test_caches.py index 6d7785b1d..32475e50f 100644 --- a/tests/unit/test_caches.py +++ b/tests/unit/test_caches.py @@ -53,9 +53,10 @@ def test_cache_evaluations(test_client): raise Exception("no evaluations") # evaluation_id = sem_evals[0]["evaluation_id"] - sco_evaluations.do_evaluation_get_all_notes(evaluation_id) - # should have been be cached: - assert sco_cache.EvaluationCache.get(evaluation_id) + eval_notes = sco_evaluations.do_evaluation_get_all_notes(evaluation_id) + # should have been be cached, except if empty + if eval_notes: + assert sco_cache.EvaluationCache.get(evaluation_id) sco_cache.invalidate_formsemestre(sem["formsemestre_id"]) # should have been erased from cache: assert not sco_cache.EvaluationCache.get(evaluation_id)