From 116e335ab867e35a3bda08aea731cd61410d7e70 Mon Sep 17 00:00:00 2001
From: Olivier Le Floch <olivier.le-floch@polytechnique.org>
Date: Sun, 15 Jun 2008 17:12:39 +0200
Subject: [PATCH] Followup to c638d8c8103 [Can disable moderation of Unsures.]
 : Fix spelling in comments and web interface ; optimise javascript code ; add
 <label> tags to simplify selecting options

---
 bin/lists.rpc.py            |  2 +-
 templates/lists/options.tpl | 41 ++++++++++++++++++-----------------------
 2 files changed, 19 insertions(+), 24 deletions(-)

diff --git a/bin/lists.rpc.py b/bin/lists.rpc.py
index 2346d4b..ccf3654 100755
--- a/bin/lists.rpc.py
+++ b/bin/lists.rpc.py
@@ -722,7 +722,7 @@ def get_bogo_level(userdesc, perms, vhost, listname):
         return 0
 
 def set_bogo_level(userdesc, perms, vhost, listname, level):
-    """ set filter to the specify level. """
+    """ set filter to the specified level. """
     try:
         mlist = MailList.MailList(vhost+VHOST_SEP+listname.lower(), lock=0)
     except:
diff --git a/templates/lists/options.tpl b/templates/lists/options.tpl
index d71d2b8..e329b6f 100644
--- a/templates/lists/options.tpl
+++ b/templates/lists/options.tpl
@@ -132,34 +132,29 @@ Tu n'es pas administrateur de la liste, mais du site.
       <td>
         <div id="spamlevel">
           <em><a name='antispam' id='antispam'></a>que faire des mails marqués « [spam probable] » ?</em><br />
-          <input type='radio' name='bogo_level' value='0' {if !$bogo_level}checked='checked'{/if} /> les laisser passer&nbsp;;<br />
-          <input type='radio' name='bogo_level' value='1' {if $bogo_level eq 1}checked='checked'{/if} /> les envoyer aux modérateurs pour approbation...<br />
-          <input type='radio' name='bogo_level' value='2' {if $bogo_level eq 2}checked='checked'{/if} /> ... après suppression des
-        spams les plus probables*&nbsp;;<br />
-          <input type='radio' name='bogo_level' value='3' {if $bogo_level eq 3}checked='checked'{/if} /> tous les supprimer.
+          <label><input type='radio' name='bogo_level' value='0' {if !$bogo_level}checked='checked'{/if} /> les laisser passer&nbsp;;</label><br />
+          <label><input type='radio' name='bogo_level' value='1' {if $bogo_level eq 1}checked='checked'{/if} /> les envoyer aux modérateurs pour approbation...</label><br />
+          <label><input type='radio' name='bogo_level' value='2' {if $bogo_level eq 2}checked='checked'{/if} /> ... après suppression des
+        spams les plus probables*&nbsp;;</label><br />
+          <label><input type='radio' name='bogo_level' value='3' {if $bogo_level eq 3}checked='checked'{/if} /> tous les supprimer.</label>
         </div>
         <div id="unsurelevel">
           <em>que faire des mails dont le classement est indéterminé** ?</em><br />
-          <input type='radio' name='unsure_level' value='0' {if !$unsure_level}checked='checked'{/if} /> les laisser
-          passer&nbsp;;<br />
-          <input type='radio' name='unsure_level' value='1' {if $unsure_level eq 1}checked='checked'{/if} /> les modérer.
+          <label><input type='radio' name='unsure_level' value='0' {if !$unsure_level}checked='checked'{/if} /> les laisser
+          passer&nbsp;;</label><br />
+          <label><input type='radio' name='unsure_level' value='1' {if $unsure_level eq 1}checked='checked'{/if} /> les modérer.</label>
         </div>
         <script type="text/javascript">//<![CDATA[
           {literal}
-          function hideUnsure() {
-            if ($(":radio[@name=bogo_level]:checked").val() == 0) {
-              $("#unsurelevel").hide();
-            } else {
-              $("#unsurelevel").show();
-            }
-          }
-
-          $(document).ready(
-            function() {
-              hideUnsure();
-              $(":radio[@name=bogo_level]").change(function() { hideUnsure(); });
-            }
-          );
+          $(function() {
+            $(":radio[@name=bogo_level]").change(function() {
+              if ($(":radio[@name=bogo_level]:checked").val() == 0) {
+                $("#unsurelevel").hide();
+              } else {
+                $("#unsurelevel").show();
+              }
+            }).change();
+          });
           {/literal}
         // ]]></script>
       </td>
@@ -170,7 +165,7 @@ Tu n'es pas administrateur de la liste, mais du site.
         d'être des faux-positifs.<br />
         **Certains mails ne sont pas classables par l'antispam qui le signale en indiquant que le mail est "Unsure". Ces
         mails contiennent statistiquement autant de spams que de non-spams, mais ceci peut varier d'une adresse à l'autre.
-        Cette option the permet de choisir si tu préfères que les mails 'Unsures' soient modérés ou envoyés diretement
+        Cette option te permet de choisir si tu préfères que les mails 'Unsures' soient modérés ou envoyés directement
         à la liste.
       </td>
     </tr>
-- 
2.1.4