optimized etud search box

html_sidebar.py

@@ -88,7 +88,7 @@ def sidebar(context, REQUEST=None):
 
     H.append(
         """<div class="box-chercheetud">Chercher étudiant:<br/>
-<form id="form-chercheetud" action="%(ScoURL)s/search_etud_in_dept">
+<form method="get" id="form-chercheetud" action="%(ScoURL)s/search_etud_in_dept">
 <div><input type="text" size="12" id="in-expnom" name="expnom" spellcheck="false"></input></div>
 </form></div>
 <div class="etud-insidebar">

sco_find_etud.py

@@ -96,73 +96,52 @@ def form_search_etud(
         return "\n".join(H)
 
 
-# was chercheEtud()
-def search_etud_in_dept(
-    context,
-    expnom=None,
-    dest_url="ficheEtud",
-    parameters={},
-    parameters_keys="",
-    add_headers=True,  # complete page
-    title=None,
-    REQUEST=None,
-):
-    """Page recherche d'un etudiant
-    expnom est un regexp sur le nom ou un code_nip ou un etudid
-    dest_url est la page sur laquelle on sera redirigé après choix
-    parameters spécifie des arguments additionnels à passer à l'URL (en plus de etudid)
+def search_etud_in_dept(context, expnom="", REQUEST=None):
+    """Page recherche d'un etudiant.
+
+    Affiche la fiche de l'étudiant, ou, si la recherche donne plusieurs résultats, la liste des étudianst
+    correspondants.
+    Appelée par boite de recherche barre latérale gauche.
+
+    Args:
+        expnom: string, regexp sur le nom ou un code_nip ou un etudid
     """
-    if type(expnom) == ListType:
-        expnom = expnom[0]
-    q = []
-    if parameters:
-        for param in parameters.keys():
-            q.append("%s=%s" % (param, parameters[param]))
-    elif parameters_keys:
-        for key in parameters_keys.split(","):
-            v = REQUEST.form.get(key, False)
-            if v:
-                q.append("%s=%s" % (key, v))
-    query_string = "&".join(q)
-
-    no_side_bar = True
-    H = []
-    if title:
-        H.append("<h2>%s</h2>" % title)
-
-    if scu.is_valid_code_nip(expnom):
-        etuds = search_etuds_infos(context, code_nip=expnom, REQUEST=REQUEST)
-    elif expnom:
-        etuds = search_etuds_infos(context, expnom=expnom, REQUEST=REQUEST)
-    if expnom and not etuds:
+    dest_url = "ficheEtud"
+    if len(expnom) > 1:
         etuds = context.getEtudInfo(filled=1, etudid=expnom, REQUEST=REQUEST)
         if len(etuds) != 1:
-            etuds = []
+            if scu.is_valid_code_nip(expnom):
+                etuds = search_etuds_infos(context, code_nip=expnom, REQUEST=REQUEST)
+            else:
+                etuds = search_etuds_infos(context, expnom=expnom, REQUEST=REQUEST)
+    else:
+        etuds = []  # si expnom est trop court, n'affiche rien
+
     if len(etuds) == 1:
         # va directement a la destination
-        return REQUEST.RESPONSE.redirect(
-            dest_url + "?etudid=%s&amp;" % etuds[0]["etudid"] + query_string
-        )
+        return context.ficheEtud(etudid=etuds[0]["etudid"], REQUEST=REQUEST)
 
+    H = [
+        context.sco_header(
+            page_title="Recherche d'un étudiant",
+            no_side_bar=True,
+            init_qtip=True,
+            javascripts=["js/etud_info.js"],
+            REQUEST=REQUEST,
+        ),
+        """<h2>%d résultats pour "%s": choisissez un étudiant:</h2>"""
+        % (len(etuds), expnom),
+        form_search_etud(
+            context,
+            dest_url=dest_url,
+            REQUEST=REQUEST,
+            title="Autre recherche",
+        ),
+    ]
     if len(etuds) > 0:
         # Choix dans la liste des résultats:
-        H.append(
-            """<h2>%d résultats pour "%s": choisissez un étudiant:</h2>"""
-            % (len(etuds), expnom)
-        )
-        H.append(
-            form_search_etud(
-                context,
-                dest_url=dest_url,
-                parameters=parameters,
-                parameters_keys=parameters_keys,
-                REQUEST=REQUEST,
-                title="Autre recherche",
-            )
-        )
-
         for e in etuds:
-            target = dest_url + "?etudid=%s&amp;" % e["etudid"] + query_string
+            target = dest_url + "?etudid=%s&amp;" % e["etudid"]
             e["_nomprenom_target"] = target
             e["inscription_target"] = target
             e["_nomprenom_td_attrs"] = 'id="%s" class="etudinfo"' % (e["etudid"])
@@ -187,34 +166,16 @@ def search_etud_in_dept(
                 form_search_etud(
                     context,
                     dest_url=dest_url,
-                    parameters=parameters,
-                    parameters_keys=parameters_keys,
                     REQUEST=REQUEST,
                     title="Autre recherche",
                 )
             )
-
     else:
         H.append('<h2 style="color: red;">Aucun résultat pour "%s".</h2>' % expnom)
-        add_headers = True
-        no_side_bar = False
     H.append(
         """<p class="help">La recherche porte sur tout ou partie du NOM ou du NIP de l'étudiant</p>"""
     )
-    if add_headers:
-        return (
-            context.sco_header(
-                REQUEST,
-                page_title="Choix d'un étudiant",
-                init_qtip=True,
-                javascripts=["js/etud_info.js"],
-                no_side_bar=no_side_bar,
-            )
-            + "\n".join(H)
-            + context.sco_footer(REQUEST)
-        )
-    else:
-        return "\n".join(H)
+    return "\n".join(H) + context.sco_footer(REQUEST)
 
 
 # Was chercheEtudsInfo()