From f9db8aab3baa12d7ae8f43fb34c2658441e75d25 Mon Sep 17 00:00:00 2001 From: aurelien Date: Fri, 25 Jun 2021 01:07:09 +0200 Subject: [PATCH] =?UTF-8?q?Changement=20des=20ids=20pour=20respecter=20les?= =?UTF-8?q?=20conventions,=20changements=20de=20certains=20id=20en=20class?= =?UTF-8?q?=20pour=20=C3=A9viter=20de=20les=20retrouver=20plusieurs=20fois?= =?UTF-8?q?=20sur=20la=20meme=20page?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- TrivialFormulator.py | 2 +- VERSION.py | 2 +- ZNotes.py | 2 +- ZScoDoc.py | 6 ++++-- ZScoUsers.py | 2 +- html_sidebar.py | 25 +++++++++++++------------ sco_apogee_csv.py | 8 ++++---- sco_dept.py | 2 +- sco_edit_module.py | 6 +++--- sco_edit_ue.py | 4 ++-- sco_find_etud.py | 2 +- sco_formations.py | 14 +++++++------- sco_formsemestre_inscriptions.py | 9 +++++++-- sco_formsemestre_validation.py | 19 ++++++++++++------- sco_moduleimpl_status.py | 21 +++++++++++++++++---- sco_page_etud.py | 22 ++++++++-------------- scolars.py | 8 ++++++-- 17 files changed, 89 insertions(+), 65 deletions(-) diff --git a/TrivialFormulator.py b/TrivialFormulator.py index 5fd84c529..54f2249b5 100644 --- a/TrivialFormulator.py +++ b/TrivialFormulator.py @@ -760,6 +760,6 @@ def tf_error_message(msg): if type(msg) == StringType: msg = [msg] return ( - '' + '' % '
  • '.join(msg) ) diff --git a/VERSION.py b/VERSION.py index 8ad330ea2..90a125dc2 100644 --- a/VERSION.py +++ b/VERSION.py @@ -1,7 +1,7 @@ # -*- mode: python -*- # -*- coding: utf-8 -*- -SCOVERSION = "7.25m" +SCOVERSION = "7.26m" SCONAME = "ScoDoc" diff --git a/ZNotes.py b/ZNotes.py index f454d2e04..59106cbfe 100644 --- a/ZNotes.py +++ b/ZNotes.py @@ -376,7 +376,7 @@ class ZNotes(ObjectManager, PropertyManager, RoleManager, Item, Persistent, Impl if editable: H.append( - """

    Créer une formation

    + """

    Créer une formation

    Importer une formation (xml)

    Une "formation" est un programme pédagogique structuré en UE, matières et modules. Chaque semestre se réfère à une formation. La modification d'une formation affecte tous les semestres qui s'y réfèrent.

    """ diff --git a/ZScoDoc.py b/ZScoDoc.py index 3812a004b..eb099f82b 100644 --- a/ZScoDoc.py +++ b/ZScoDoc.py @@ -898,7 +898,9 @@ subversion: %(svn_version)s # aucun departement defini: aide utilisateur H.append("

    Aucun département à ajouter !

    ") else: - H.append("""
    """ + ) for deptId in addableDepts: H.append("""""" % (deptId, deptId)) H.append( @@ -915,7 +917,7 @@ subversion: %(svn_version)s (le site peut donc être recréé sans perte de données).

    - """ ) for deptFolder in self._list_depts(): diff --git a/ZScoUsers.py b/ZScoUsers.py index 43ed81625..121c6420a 100644 --- a/ZScoUsers.py +++ b/ZScoUsers.py @@ -143,7 +143,7 @@ class ZScoUsers( if authuser.has_permission(ScoUsersAdmin, self): H.append( - '

    Ajouter un utilisateur' + '

    Ajouter un utilisateur' ) H.append( '   Importer des utilisateurs

    ' diff --git a/html_sidebar.py b/html_sidebar.py index fa874c289..8967acbf9 100644 --- a/html_sidebar.py +++ b/html_sidebar.py @@ -55,9 +55,9 @@ def sidebar_common(context, REQUEST=None): % params, context.sidebar_dept(REQUEST), """

    Scolarité

    - Semestres
    - Programmes
    - Absences
    + Semestres
    + Programmes
    + Absences
    """ % params, ] @@ -66,12 +66,13 @@ def sidebar_common(context, REQUEST=None): ScoUsersView, context ): H.append( - """Utilisateurs
    """ % params + """Utilisateurs
    """ + % params ) if authuser.has_permission(ScoChangePreferences, context): H.append( - """Paramétrage
    """ + """Paramétrage
    """ % params ) @@ -104,7 +105,7 @@ def sidebar(context, REQUEST=None): """

    %(civilite_str)s %(nom_disp)s

    -

    %(etudid)s

    +

    %(etudid)s

    Absences""" % params ) @@ -124,21 +125,21 @@ def sidebar(context, REQUEST=None): if REQUEST.AUTHENTICATED_USER.has_permission(ScoAbsChange, context): H.append( """ -
  • Ajouter
    • -
  • Justifier
    • -
  • Supprimer
    • +
  • Ajouter
    • +
  • Justifier
    • +
  • Supprimer
    • """ % params ) if context.get_preference("handle_billets_abs"): H.append( - """
  • Billets
    • """ + """
  • Billets
    • """ % params ) H.append( """ -
  • Calendrier
    • -
  • Liste
    • +
  • Calendrier
    • +
  • Liste
    • """ % params diff --git a/sco_apogee_csv.py b/sco_apogee_csv.py index 55ccc145f..59382af5c 100644 --- a/sco_apogee_csv.py +++ b/sco_apogee_csv.py @@ -422,7 +422,7 @@ class ApoEtud(dict): # Elements UE decisions_ue = nt.get_etud_decision_ues(etudid) for ue in nt.get_ues(): - if ue["code_apogee"] == code: + if code in ue["code_apogee"].split(","): if self.export_res_ues: if decisions_ue and ue["ue_id"] in decisions_ue: ue_status = nt.get_etud_ue_status(etudid, ue["ue_id"]) @@ -443,7 +443,7 @@ class ApoEtud(dict): modimpls = nt.get_modimpls() module_code_found = False for modimpl in modimpls: - if modimpl["module"]["code_apogee"] == code: + if code in modimpl["module"]["code_apogee"].split(","): n = nt.get_etud_mod_moy(modimpl["moduleimpl_id"], etudid) if n != "NI" and self.export_res_modules: return dict(N=_apo_fmt_note(n), B=20, J="", R="") @@ -972,13 +972,13 @@ class ApoData: self.context, sem["formsemestre_id"] ) for ue in nt.get_ues(): - if ue["code_apogee"] == code: + if code in ue["code_apogee"].split(","): s.add(code) continue # associé à un module: modimpls = nt.get_modimpls() for modimpl in modimpls: - if modimpl["module"]["code_apogee"] == code: + if code in modimpl["module"]["code_apogee"].split(","): s.add(code) continue # log('codes_by_sem=%s' % pprint.pformat(codes_by_sem)) diff --git a/sco_dept.py b/sco_dept.py index e90539c8b..e9f435daa 100644 --- a/sco_dept.py +++ b/sco_dept.py @@ -143,7 +143,7 @@ Chercher étape courante:

    Gestion des étudiants

      -
    • créer un nouvel étudiant
      • +
    • créer un nouvel étudiant
    • importer de nouveaux étudiants (ne pas utiliser sauf cas particulier, utilisez plutôt le lien dans le tableau de bord semestre si vous souhaitez inscrire les étudiants importés à un semestre)
      • diff --git a/sco_edit_module.py b/sco_edit_module.py index 1b15984e1..28e5d152c 100644 --- a/sco_edit_module.py +++ b/sco_edit_module.py @@ -153,8 +153,8 @@ def module_create(context, matiere_id=None, REQUEST=None): "code_apogee", { "title": "Code Apogée", - "size": 15, - "explanation": "code élément pédagogique Apogée (optionnel)", + "size": 25, + "explanation": "code élément pédagogique Apogée (optionnel) ou liste de codes ELP séparés par une virgule", }, ), ( @@ -355,7 +355,7 @@ def module_edit(context, module_id=None, REQUEST=None): { "title": "Code Apogée", "size": 15, - "explanation": "code élément pédagogique Apogée (optionnel)", + "explanation": "code élément pédagogique Apogée (optionnel) ou liste de codes ELP séparés par une virgule", }, ), ( diff --git a/sco_edit_ue.py b/sco_edit_ue.py index 452b12a6d..247562c0f 100644 --- a/sco_edit_ue.py +++ b/sco_edit_ue.py @@ -146,8 +146,8 @@ def ue_edit(context, ue_id=None, create=False, formation_id=None, REQUEST=None): "code_apogee", { "title": "Code Apogée", - "size": 15, - "explanation": "code élément pédagogique Apogée (optionnel)", + "size": 25, + "explanation": "code élément pédagogique Apogée (optionnel) ou liste de codes ELP séparés par une virgule", }, ), ] diff --git a/sco_find_etud.py b/sco_find_etud.py index 151b811c6..91f8d4d88 100644 --- a/sco_find_etud.py +++ b/sco_find_etud.py @@ -58,7 +58,7 @@ def form_search_etud( H.append( """ %s - +
      (entrer une partie du nom) """ diff --git a/sco_formations.py b/sco_formations.py index 0faa82d38..d51e12091 100644 --- a/sco_formations.py +++ b/sco_formations.py @@ -229,7 +229,7 @@ def formation_list_table(context, formation_id=None, args={}, REQUEST=None): f["parcours_name"] = "" f["_titre_target"] = "ue_list?formation_id=%(formation_id)s" % f f["_titre_link_class"] = "stdlink" - f["_titre_id"] = "titre_%s" % f["acronyme"] + f["_titre_id"] = "titre-%s" % f["acronyme"] # Ajoute les semestres associés à chaque formation: f["sems"] = sco_formsemestre.do_formsemestre_list( context, args={"formation_id": f["formation_id"]} @@ -242,8 +242,8 @@ def formation_list_table(context, formation_id=None, args={}, REQUEST=None): for s in f["sems"] ] + [ - 'ajouter' - % (f["acronyme"],f["formation_id"]) + 'ajouter' + % (f["acronyme"].lower(), f["formation_id"]) ] ) if f["sems"]: @@ -260,15 +260,15 @@ def formation_list_table(context, formation_id=None, args={}, REQUEST=None): but_locked = '' if editable and not locked: but_suppr = ( - '%s' - % (f["acronyme"],f["formation_id"], suppricon) + '%s' + % (f["acronyme"], f["formation_id"], suppricon) ) else: but_suppr = '' if editable: but_edit = ( - '%s' - % (f["acronyme"],f["formation_id"], editicon) + '%s' + % (f["acronyme"], f["formation_id"], editicon) ) else: but_edit = '' diff --git a/sco_formsemestre_inscriptions.py b/sco_formsemestre_inscriptions.py index 3bf300509..f281e8cd7 100644 --- a/sco_formsemestre_inscriptions.py +++ b/sco_formsemestre_inscriptions.py @@ -147,9 +147,14 @@ def formsemestre_inscription_with_modules_form( if (not only_ext) or (sem["modalite"] == "EXT"): H.append( """ -
    • %s +
    • %s """ - % (sem["formsemestre_id"],etudid, sem["formsemestre_id"], sem["titremois"]) + % ( + sem["formsemestre_id"], + etudid, + sem["formsemestre_id"], + sem["titremois"], + ) ) H.append("
    ") else: diff --git a/sco_formsemestre_validation.py b/sco_formsemestre_validation.py index cc9065eca..2656f0a52 100644 --- a/sco_formsemestre_validation.py +++ b/sco_formsemestre_validation.py @@ -401,9 +401,9 @@ def formsemestre_validation_etud_manu( def _redirect_valid_choice( formsemestre_id, etudid, Se, choice, desturl, sortcol, REQUEST ): - adr = ( - "formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s&check=1" - % (formsemestre_id, etudid) + adr = "formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s&check=1" % ( + formsemestre_id, + etudid, ) if sortcol: adr += "&sortcol=" + sortcol @@ -454,8 +454,8 @@ def decisions_possible_rows(Se, assiduite, subtitle="", trclass=""): H.append("Code %sDevenir" % TitleCur) for ch in choices: H.append( - """""" - % (trclass, ch.rule_id, ch.codechoice,ch.rule_id) + """""" + % (trclass, ch.rule_id, ch.codechoice, ch.rule_id) ) H.append("%s " % ch.explication) if Se.prev: @@ -640,8 +640,13 @@ def formsemestre_recap_parcours_table( # log('') H.append( - '%s' - % (class_ue,ue["acronyme"], " ".join(explanation_ue), scu.fmt_note(moy_ue)) + '%s' + % ( + class_ue, + ue["acronyme"], + " ".join(explanation_ue), + scu.fmt_note(moy_ue), + ) ) if len(ues) < Se.nb_max_ue: H.append('' % (Se.nb_max_ue - len(ues))) diff --git a/sco_moduleimpl_status.py b/sco_moduleimpl_status.py index 1050cd220..d99ffeb93 100644 --- a/sco_moduleimpl_status.py +++ b/sco_moduleimpl_status.py @@ -321,7 +321,7 @@ def moduleimpl_status(context, moduleimpl_id=None, partition_id=None, REQUEST=No top_table_links = "" if sem["etat"] == "1": # non verrouillé top_table_links = ( - """Créer nouvelle évaluation + """Créer nouvelle évaluation Trier par date """ % M @@ -360,12 +360,18 @@ def moduleimpl_status(context, moduleimpl_id=None, partition_id=None, REQUEST=No """Evaluation sans date""" % eval ) - H.append("    %(description)s" % eval) + H.append( + "    %(description)s" % eval + ) if eval["evaluation_type"] == EVALUATION_RATTRAPAGE: H.append( """rattrapage""" ) - H.append("") + H.append( + "" + ) elif eval["evaluation_type"] == EVALUATION_SESSION2: H.append( """session 2""" @@ -454,7 +460,14 @@ def moduleimpl_status(context, moduleimpl_id=None, partition_id=None, REQUEST=No """""" % eval ) - H.append(scu.icontag("delete_img", alt="supprimer", title="Supprimer", id="delete_buttion_%s" %eval["evaluation_id"])) + H.append( + scu.icontag( + "delete_img", + alt="supprimer", + title="Supprimer", + id="button-delete-%s" % eval["evaluation_id"], + ) + ) if caneditevals: H.append("""""") elif etat["evalcomplete"]: diff --git a/sco_page_etud.py b/sco_page_etud.py index dd589b5f9..d32dfcc41 100644 --- a/sco_page_etud.py +++ b/sco_page_etud.py @@ -74,27 +74,21 @@ def _menuScolarite(context, authuser, sem, etudid): if ins["etat"] != "D": dem_title = "Démission" - dem_url = ( - "formDem?etudid=%(etudid)s&formsemestre_id=%(formsemestre_id)s" % args - ) + dem_url = "formDem?etudid=%(etudid)s&formsemestre_id=%(formsemestre_id)s" % args else: dem_title = "Annuler la démission" dem_url = ( - "doCancelDem?etudid=%(etudid)s&formsemestre_id=%(formsemestre_id)s" - % args + "doCancelDem?etudid=%(etudid)s&formsemestre_id=%(formsemestre_id)s" % args ) # Note: seul un etudiant inscrit (I) peut devenir défaillant. if ins["etat"] != sco_codes_parcours.DEF: def_title = "Déclarer défaillance" - def_url = ( - "formDef?etudid=%(etudid)s&formsemestre_id=%(formsemestre_id)s" % args - ) + def_url = "formDef?etudid=%(etudid)s&formsemestre_id=%(formsemestre_id)s" % args elif ins["etat"] == sco_codes_parcours.DEF: def_title = "Annuler la défaillance" def_url = ( - "doCancelDef?etudid=%(etudid)s&formsemestre_id=%(formsemestre_id)s" - % args + "doCancelDef?etudid=%(etudid)s&formsemestre_id=%(formsemestre_id)s" % args ) def_enabled = ( (ins["etat"] != "D") @@ -430,16 +424,16 @@ def ficheEtud(context, etudid=None, REQUEST=None):
    - + %(groupes_row)s - +
    Situation :%(situation)s
    Situation :%(situation)s
    Né%(ne)s le :%(info_naissance)s
    Né%(ne)s le :%(info_naissance)s
    -
    +
    -
    Adresse :Adresse : %(domicile)s %(codepostaldomicile)s %(villedomicile)s %(paysdomicile)s %(modifadresse)s %(telephones)s
    diff --git a/scolars.py b/scolars.py index 8095e8b5d..119204da5 100644 --- a/scolars.py +++ b/scolars.py @@ -361,7 +361,8 @@ def _check_duplicate_code(cnx, args, code_name, context, edit=True, REQUEST=None parameters = {} if context: err_page = context.confirmDialog( - message="""

    Code étudiant (%s) dupliqué !

    """ % code_name, + message="""

    Code étudiant (%s) dupliqué !

    """ + % code_name, helpmsg="""Le %s %s est déjà utilisé: un seul étudiant peut avoir ce code. Vérifier votre valeur ou supprimer l'autre étudiant avec cette valeur.

    • """ % (code_name, args[code_name]) + "
    • ".join(listh) @@ -372,7 +373,10 @@ def _check_duplicate_code(cnx, args, code_name, context, edit=True, REQUEST=None REQUEST=REQUEST, ) else: - err_page = """

      Code étudiant (%s) dupliqué !

      """ % code_name + err_page = ( + """

      Code étudiant (%s) dupliqué !

      """ + % code_name + ) log("*** error: code %s duplique: %s" % (code_name, args[code_name])) raise ScoGenError(err_page)