Maj d'ID pour suivre la convention, changemement d'ID en Class pour l'unicité sur une meme page

Reviewed-on: https://scodoc.org/git/viennet/ScoDoc/pulls/83

TrivialFormulator.py

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

html_sidebar.py

@@ -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_find_etud.py

@@ -171,8 +171,8 @@ def search_etud_in_dept(context, expnom="", REQUEST=None):
                 )
             )
     else:
-        H.append('<h2 style="color: red;">Aucun résultat pour "%s".</h2>' % expnom)
+        H.append('<h2 id="title-no-result" 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"]
+        f["_titre_id"] = "titre-%s" % f["acronyme"].lower().replace(" ","-")
         # 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(), f["formation_id"])
+                % (f["acronyme"].lower().replace(" ", "-"), 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"], f["formation_id"], editicon)
+                % (f["acronyme"].lower().replace(" ","-"), f["formation_id"], editicon)
             )
         else:
             but_edit = '<span class="but_placeholder"></span>'

sco_formsemestre_inscriptions.py

@@ -149,12 +149,7 @@ 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 = "formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s&check=1" % (
+    adr = (
-        formsemestre_id,
+        "formsemestre_validation_etud_form?formsemestre_id=%s&etudid=%s&check=1"
-        etudid,
+        % (formsemestre_id, 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"],
+                    ue["acronyme"].lower().replace(" ","-"),
                     " ".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 class="redboldtext">""",
+<td class="fichetitre2">Responsable: </td><td id="ens-responsable" 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>""")
+    H.append("""</td><td class="ens-in-module">""")
     H.append(
         ", ".join([context.Users.user_info(m["ens_id"])["nomprenom"] for m in M["ens"]])
     )
@@ -360,18 +360,12 @@ 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(
+        H.append("&nbsp;&nbsp;&nbsp; <em id=""%(description)s"">%(description)s</em>" % eval)
-            "&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(
+            H.append("<div id='hidden-id' style='visibility:hidden'>%(evaluation_id)s</div>")
-                "<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,9 +26,7 @@
 ##############################################################################
 
 """ScoDoc ficheEtud
-
    Fiche description d'un étudiant et de son parcours
-
 """
 
 import sco_utils as scu
@@ -415,12 +413,10 @@ 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>
@@ -428,8 +424,6 @@ 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">
 <table><tr>
@@ -440,21 +434,15 @@ def ficheEtud(context, etudid=None, REQUEST=None):
 </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>
@@ -472,9 +460,7 @@ 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,8 +361,7 @@ 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>"""
+                    message="""<h3 class="title-error">Code étudiant (%s) dupliqué !</h3>""" % code_name,
-                    % 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)
@@ -373,10 +372,7 @@ def _check_duplicate_code(cnx, args, code_name, context, edit=True, REQUEST=None
                     REQUEST=REQUEST,
                 )
             else:
-                err_page = (
+                err_page = """<h3 class="title-error">Code étudiant (%s) dupliqué !</h3>""" % code_name
-                    """<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)