Changement des ids pour respecter les conventions, changements de certains id en class pour éviter de les retrouver plusieurs fois sur la meme page

TrivialFormulator.py

@@ -760,6 +760,6 @@ def tf_error_message(msg):
     if type(msg) == StringType:
         msg = [msg]
     return (
-        '<ul class="tf-msg"><li class="tf-msg error-message">%s</li></ul>'
+        '<ul class="tf-msg"><li id="error-message" class="tf-msg">%s</li></ul>'
         % '</li><li class="tf-msg">'.join(msg)
     )

VERSION.py

@@ -1,7 +1,7 @@
 # -*- mode: python -*-
 # -*- coding: utf-8 -*-
 
-SCOVERSION = "7.26m"
+SCOVERSION = "7.25m"
 
 SCONAME = "ScoDoc"
 

html_sidebar.py

@@ -66,7 +66,7 @@ def sidebar_common(context, REQUEST=None):
         ScoUsersView, context
     ):
         H.append(
-            """<a id = "link-view-users"href="%(UsersURL)s" class="sidebar">Utilisateurs</a> <br/>"""
+            """<a id = "utilisateurs_Vue"href="%(UsersURL)s" class="sidebar">Utilisateurs</a> <br/>"""
             % params
         )
 
@@ -105,7 +105,7 @@ def sidebar(context, REQUEST=None):
             """<h2 id="insidebar-etud"><a href="%(ScoURL)s/ficheEtud?etudid=%(etudid)s" class="sidebar">
     <font color="#FF0000">%(civilite_str)s %(nom_disp)s</font></a>
     </h2>
-    <p id="euid">%(etudid)s</p>
+    <p id ="euid">%(etudid)s</p>
     <b>Absences</b>"""
             % params
         )

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 code in ue["code_apogee"].split(","):
+            if ue["code_apogee"] == code:
                 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 code in modimpl["module"]["code_apogee"].split(","):
+            if modimpl["module"]["code_apogee"] == code:
                 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 code in ue["code_apogee"].split(","):
+                    if ue["code_apogee"] == code:
                         s.add(code)
                         continue
                 # associé à un module:
                 modimpls = nt.get_modimpls()
                 for modimpl in modimpls:
-                    if code in modimpl["module"]["code_apogee"].split(","):
+                    if modimpl["module"]["code_apogee"] == code:
                         s.add(code)
                         continue
         # log('codes_by_sem=%s' % pprint.pformat(codes_by_sem))

sco_edit_module.py

@@ -153,8 +153,8 @@ def module_create(context, matiere_id=None, REQUEST=None):
                 "code_apogee",
                 {
                     "title": "Code Apogée",
-                    "size": 25,
+                    "size": 15,
-                    "explanation": "code élément pédagogique Apogée (optionnel) ou liste de codes ELP séparés par une virgule",
+                    "explanation": "code élément pédagogique Apogée (optionnel)",
                 },
             ),
             (
@@ -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) ou liste de codes ELP séparés par une virgule",
+                    "explanation": "code élément pédagogique Apogée (optionnel)",
                 },
             ),
             (

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": 25,
+                "size": 15,
-                "explanation": "code élément pédagogique Apogée (optionnel) ou liste de codes ELP séparés par une virgule",
+                "explanation": "code élément pédagogique Apogée (optionnel)",
             },
         ),
     ]

sco_find_etud.py

@@ -171,8 +171,8 @@ def search_etud_in_dept(context, expnom="", REQUEST=None):
                 )
             )
     else:
-        H.append('<h2 id="title-no-result" style="color: red;">Aucun résultat pour "%s".</h2>' % expnom)
+        H.append('<h2 style="color: red;">Aucun résultat pour "%s".</h2>' % expnom)
-        H.append(
+    H.append(
         """<p class="help">La recherche porte sur tout ou partie du NOM ou du NIP de l'étudiant</p>"""
     )
     return "\n".join(H) + context.sco_footer(REQUEST)

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"].lower().replace(" ","-")
+        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"]}
@@ -243,7 +243,7 @@ def formation_list_table(context, formation_id=None, args={}, REQUEST=None):
             ]
             + [
                 '<a class="stdlink" id="add-semestre-%s" href="formsemestre_createwithmodules?formation_id=%s&semestre_id=1">ajouter</a>'
-                % (f["acronyme"].lower().replace(" ", "-"), f["formation_id"])
+                % (f["acronyme"].lower(), f["formation_id"])
             ]
         )
         if f["sems"]:
@@ -261,14 +261,14 @@ def formation_list_table(context, formation_id=None, args={}, REQUEST=None):
         if editable and not locked:
             but_suppr = (
                 '<a class="stdlink" id="delete-formation-%s" href="formation_delete?formation_id=%s">%s</a>'
-                % (f["acronyme"],f["formation_id"], suppricon)
+                % (f["acronyme"], f["formation_id"], suppricon)
             )
         else:
             but_suppr = '<span class="but_placeholder"></span>'
         if editable:
             but_edit = (
                 '<a class="stdlink" id="edit-formation-%s" href="formation_edit?formation_id=%s">%s</a>'
-                % (f["acronyme"].lower().replace(" ","-"), f["formation_id"], editicon)
+                % (f["acronyme"], f["formation_id"], editicon)
             )
         else:
             but_edit = '<span class="but_placeholder"></span>'

sco_formsemestre_inscriptions.py

@@ -149,7 +149,12 @@ def formsemestre_inscription_with_modules_form(
                         """
                     <li><a class="stdlink" id="inscription-semestre-%s" href="formsemestre_inscription_with_modules?etudid=%s&formsemestre_id=%s">%s</a>
                     """
-                        % (sem["formsemestre_id"],etudid, sem["formsemestre_id"], sem["titremois"])
+                        % (
+                            sem["formsemestre_id"],
+                            etudid,
+                            sem["formsemestre_id"],
+                            sem["titremois"],
+                        )
                     )
         H.append("</ul>")
     else:

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 = (
+    adr = "formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s&check=1" % (
-        "formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s&check=1"
+        formsemestre_id,
-        % (formsemestre_id, etudid)
+        etudid,
     )
     if sortcol:
         adr += "&sortcol=" + sortcol
@@ -455,7 +455,7 @@ def decisions_possible_rows(Se, assiduite, subtitle="", trclass=""):
     for ch in choices:
         H.append(
             """<tr class="%s"><td title="règle %s"><input type="radio" name="codechoice" value="%s" id="input-choice-%s" onClick="document.getElementById('subut').disabled=false;">"""
-            % (trclass, ch.rule_id, ch.codechoice,ch.rule_id)
+            % (trclass, ch.rule_id, ch.codechoice, ch.rule_id)
         )
         H.append("%s </input></td>" % ch.explication)
         if Se.prev:
@@ -643,7 +643,7 @@ def formsemestre_recap_parcours_table(
                 '<td class="%s" id="ue-%s" title="%s">%s</td>'
                 % (
                     class_ue,
-                    ue["acronyme"].lower().replace(" ","-"),
+                    ue["acronyme"],
                     " ".join(explanation_ue),
                     scu.fmt_note(moy_ue),
                 )

sco_moduleimpl_status.py

@@ -169,7 +169,7 @@ def moduleimpl_status(context, moduleimpl_id=None, partition_id=None, REQUEST=No
 
 <table>
 <tr>
-<td class="fichetitre2">Responsable: </td><td id="ens-responsable" class="redboldtext">""",
+<td class="fichetitre2">Responsable: </td><td class="redboldtext">""",
         context.Users.user_info(M["responsable_id"])["nomprenom"],
         """<span class="blacktt">(%(responsable_id)s)</span>""" % M,
     ]
@@ -181,7 +181,7 @@ def moduleimpl_status(context, moduleimpl_id=None, partition_id=None, REQUEST=No
         )
     except:
         pass
-    H.append("""</td><td class="ens-in-module">""")
+    H.append("""</td><td>""")
     H.append(
         ", ".join([context.Users.user_info(m["ens_id"])["nomprenom"] for m in M["ens"]])
     )
@@ -360,12 +360,18 @@ def moduleimpl_status(context, moduleimpl_id=None, partition_id=None, REQUEST=No
                 """<a href="evaluation_edit?evaluation_id=%(evaluation_id)s" class="mievr_evalnodate">Evaluation sans date</a>"""
                 % eval
             )
-        H.append("&nbsp;&nbsp;&nbsp; <em id=""%(description)s"">%(description)s</em>" % eval)
+        H.append(
+            "&nbsp;&nbsp;&nbsp; <em id="
+            "%(description)s"
+            ">%(description)s</em>" % eval
+        )
         if eval["evaluation_type"] == EVALUATION_RATTRAPAGE:
             H.append(
                 """<span class="mievr_rattr" title="remplace si meilleure note">rattrapage</span>"""
             )
-            H.append("<div id='hidden-id' style='visibility:hidden'>%(evaluation_id)s</div>")
+            H.append(
+                "<div id='hidden-id' style='visibility:hidden'>%(evaluation_id)s</div>"
+            )
         elif eval["evaluation_type"] == EVALUATION_SESSION2:
             H.append(
                 """<span class="mievr_rattr" title="remplace autres notes">session 2</span>"""

sco_page_etud.py

@@ -26,7 +26,9 @@
 ##############################################################################
 
 """ScoDoc ficheEtud
+
    Fiche description d'un étudiant et de son parcours
+
 """
 
 import sco_utils as scu
@@ -413,10 +415,12 @@ def ficheEtud(context, etudid=None, REQUEST=None):
 <div class="ficheEtud" id="ficheEtud"><table>
 <tr><td>
 <h2>%(nomprenom)s (%(inscription)s)</h2>
+
 <span>%(emaillink)s</span> 
 </td><td class="photocell">
 <a href="etud_photo_orig_page?etudid=%(etudid)s">%(etudfoto)s</a>
 </td></tr></table>
+
 <div class="fichesituation">
 <div class="fichetablesitu">
 <table>
@@ -424,25 +428,33 @@ def ficheEtud(context, etudid=None, REQUEST=None):
 %(groupes_row)s
 <tr><td class="fichetitre2">Né%(ne)s le :</td><td id="student-birth">%(info_naissance)s</td></tr>
 </table>
+
+
 <!-- Adresse -->
-<div class="ficheadresse" id="fiche-adresse">
+<div class="ficheadresse" id="ficheadresse">
 <table><tr>
-<td class="fichetitre2">Adresse :</td><td id="student-address"> %(domicile)s %(codepostaldomicile)s %(villedomicile)s %(paysdomicile)s
+<td class="fichetitre2">Adresse :</td><td id="student-adress> %(domicile)s %(codepostaldomicile)s %(villedomicile)s %(paysdomicile)s
 %(modifadresse)s
 %(telephones)s
 </td></tr></table>
 </div>
 </div>
 </div>
+
 %(inscriptions_mkup)s
+
 <div class="ficheadmission">
 %(adm_data)s
+
 %(fichiers_archive_htm)s
 </div>
+
 %(debouche_html)s
+
 <div class="ficheannotations">
 %(tit_anno)s
 <table id="etudannotations">%(liste_annotations)s</table>
+
 <form action="doAddAnnotation" method="GET" class="noprint">
 <input type="hidden" name="etudid" value="%(etudid)s">
 <b>Ajouter une annotation sur %(nomprenom)s: </b>
@@ -460,7 +472,9 @@ def ficheEtud(context, etudid=None, REQUEST=None):
 </table>
 </form>
 </div>
+
 <div class="code_nip">code NIP: %(code_nip)s</div>
+
 </div>
         """
     header = context.sco_header(

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="""<h3 class="title-error">Code étudiant (%s) dupliqué !</h3>""" % code_name,
+                    message="""<h3 class="title-error">Code étudiant (%s) dupliqué !</h3>"""
+                    % 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.<p><ul><li>"""
                     % (code_name, args[code_name])
                     + "</li><li>".join(listh)
@@ -372,7 +373,10 @@ def _check_duplicate_code(cnx, args, code_name, context, edit=True, REQUEST=None
                     REQUEST=REQUEST,
                 )
             else:
-                err_page = """<h3 class="title-error">Code étudiant (%s) dupliqué !</h3>""" % code_name
+                err_page = (
+                    """<h3 class="title-error">Code étudiant (%s) dupliqué !</h3>"""
+                    % code_name
+                )
             log("*** error: code %s duplique: %s" % (code_name, args[code_name]))
             raise ScoGenError(err_page)