From 1f17ab93163aacbfd5e1aa08728691270bb5c642 Mon Sep 17 00:00:00 2001
From: x2003bruneau <x2003bruneau@839d8a87-29fc-0310-9880-83ba4fa771e5>
Date: Thu, 10 Aug 2006 23:27:51 +0000
Subject: [PATCH] Search UI is more coherent : * advanced and quick search have
 the same look * no [link] lost alone on a line

&& fix advanced search


git-svn-id: svn+ssh://murphy/home/svn/platal/trunk@800 839d8a87-29fc-0310-9880-83ba4fa771e5
---
 modules/search.php              |  1 +
 templates/search/adv.form.tpl   | 41 ++++++++++++++++++++++++-----------------
 templates/search/adv.links.tpl  |  2 +-
 templates/search/index.tpl      | 11 ++++-------
 templates/search/quick.form.tpl | 27 +++++++++++++++++++--------
 5 files changed, 49 insertions(+), 33 deletions(-)

diff --git a/modules/search.php b/modules/search.php
index eb7c9f1..0db2640 100644
--- a/modules/search.php
+++ b/modules/search.php
@@ -145,6 +145,7 @@ class SearchModule extends PLModule
                                                    AND w.uid='.S::v('uid').')
                 '.$globals->search->result_where_statement."
                     $where
+                 GROUP BY  u.user_id
                  ORDER BY  ".($order?($order.', '):'')
                 .implode(',',array_filter(array($fields->get_order_statement(),
                                                 'promo DESC, NomSortKey, prenom'))).'
diff --git a/templates/search/adv.form.tpl b/templates/search/adv.form.tpl
index 2ac5780..f35bf4e 100644
--- a/templates/search/adv.form.tpl
+++ b/templates/search/adv.form.tpl
@@ -20,13 +20,23 @@
 {*                                                                        *}
 {**************************************************************************}
 
-<h1>Recherche avancée</h1>
-
-<p>[<a href="search">Recherche simple</a>]</p>
+<h1>Recherche dans l'annuaire</h1>
 
 <form id="recherche" action="search/adv" method="get">
+  <script type="text/javascript">{literal}
+    function launch_form(url) {
+      var f = document.getElementById('recherche');
+      f.action = url;
+      f.submit();
+    }
+  {/literal}</script>
   <table class="bicol" cellpadding="3" summary="Recherche">
     <tr>
+      <th colspan="2">
+        Recherche avancée [<a href="search">&lt;&lt;&lt;&nbsp;Recherche simple</a>]
+      </th>
+    </tr>
+    <tr>
       <td>Nom</td>
       <td>
     	<input type="hidden" name="rechercher" value="Chercher"/>
@@ -300,24 +310,21 @@
       <td>Commentaire contient</td>
       <td><input type="text" name="free" size="32" value="{$smarty.request.free}" /></td>
     </tr>
+    <tr>
+      <td colspan="2" style="padding-top: 1.5em">
+        <div style="float: right">
+          <input type="button" value="Chercher" onclick="launch_form('search/adv')"/>
+        </div>
+        {if $smarty.session.auth ge AUTH_COOKIE}
+          <input type='checkbox' name='order' value='date_mod' {if $smarty.request.order eq "date_mod"}checked='checked'{/if} />
+          mettre les fiches modifiées récemment en premier
+        {/if}
+      </td>
+    </tr>
   </table>
-  {if $smarty.session.auth ge AUTH_COOKIE}
-  <p>
-    <input type='checkbox' name='order' value='date_mod' {if $smarty.request.order eq "date_mod"}checked='checked'{/if} />
-    mettre les fiches modifiées récemment en premier
-  </p>
-  {/if}
 </form>
 <p class="center">
-  <script type="text/javascript">{literal}
-    function launch_form(url) {
-      var f = document.getElementById('recherche');
-      f.action = url;
-      f.submit();
-    }
-  {/literal}</script>
   <input type="button" value="Voir les résultats sur une carte" onclick="launch_form('geoloc/')"/>
-  <input type="button" value="Chercher" onclick="launch_form('search/adv')"/>
 </p>
 <p>
   <strong>N.B.</strong> Le caractère joker * peut remplacer une ou plusieurs lettres dans les recherches.
diff --git a/templates/search/adv.links.tpl b/templates/search/adv.links.tpl
index c8bc4a8..3f6a69b 100644
--- a/templates/search/adv.links.tpl
+++ b/templates/search/adv.links.tpl
@@ -20,7 +20,7 @@
 {*                                                                        *}
 {**************************************************************************}
 
-{if $do_title}<h1>Recherche avancée</h1>{/if}
+{if $do_title}<h1>Recherche dans l'annuaire</h1>{/if}
 
 <ul>
   {if $with_soundex && ($smarty.request.firstname || $smarty.request.name)}
diff --git a/templates/search/index.tpl b/templates/search/index.tpl
index 1404886..c6a0c20 100644
--- a/templates/search/index.tpl
+++ b/templates/search/index.tpl
@@ -31,14 +31,11 @@
   {/if}
 
   <h1 class='right'>
-    {if $search_results_nb==0}Aucune{else}{$search_results_nb}{/if} réponse{if $search_results_nb>1}s{/if}.
+    {if $search_results_nb==0}Aucune{else}{$search_results_nb}{/if} réponse{if $search_results_nb>1}s{/if}
+    {if $search_results_nb and $advanced and !$simple}
+    &nbsp;(<a href='geoloc/?{$search_vars}'>Voir sur une carte</a>)
+    {/if}
   </h1>
-
-  {if $search_results_nb and $advanced and !$simple}
-  <p>
-    [<a href='geoloc/?{$search_vars}'>Voir la recherche sur une carte</a>].
-  </p>
-  {/if}
   
   {if $search_results_nb > 1}
   <div>
diff --git a/templates/search/quick.form.tpl b/templates/search/quick.form.tpl
index dc95c51..13046a8 100644
--- a/templates/search/quick.form.tpl
+++ b/templates/search/quick.form.tpl
@@ -20,26 +20,37 @@
 {*                                                                        *}
 {**************************************************************************}
 
-<h1>Recherche simple dans l'annuaire</h1>
+<h1>Recherche dans l'annuaire</h1>
 
 <form action="search" method="get"{if $show_js} id="quick_form"{/if}>
   <table class="bicol" cellspacing="0" cellpadding="4">
+    {if $smarty.session.auth ge AUTH_COOKIE}
+    <tr>
+      <th colspan="2">
+        Recherche simple [<a href="search/adv">&gt;&gt;&gt;&nbsp;Recherche&nbsp;avancée</a>]
+      </th>
+    </tr>
+    <tr>
+      <td colspan="2">
+        <input type='text' name="quick" value="{$smarty.request.quick}" style="width: 98%" /><br />
+      </td>
+    </tr>
     <tr>
       <td style="width: 78%">
-        <input type='text' name="quick" value="{$smarty.request.quick}" style="width: 100%" /><br />
-        {if $smarty.session.auth ge AUTH_COOKIE}
         <input type='checkbox' name='order' value='date_mod' {if $smarty.request.order eq "date_mod"}checked='checked'{/if} />
         mettre les fiches modifiées récemment en premier
         {if $smarty.request.nonins}
         <br /><input type='checkbox' name='nonins' readonly="readonly" checked='checked' value='1' /> Chercher uniquement des non inscrits
         {/if}
-        {/if}
       </td>
-      <td>
+    {else}
+    <tr>
+      <td style="width: 78%">
+        <input type='text' name="quick" value="{$smarty.request.quick}" style="width: 98%" /><br />
+      </td>
+    {/if}
+      <td class="right">
         <input type="submit" value="Chercher" />
-        {if $smarty.session.auth ge AUTH_COOKIE}
-        <br /><a class='smaller' href="search/adv">Recherche&nbsp;avancée</a>
-        {/if}
       </td>
     </tr>
   </table>
-- 
2.1.4