bugs 396, 416 et 417 : interface de gestion des spams dans les ML + admin peuvent...

git-svn-id: svn+ssh://murphy/home/svn/platal/trunk@331 839d8a87-29fc-0310-9880-83ba4fa771e5

diff --git a/ChangeLog b/ChangeLog
index fc7bf8e..91a01a9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -12,6 +12,7 @@ New :
 
     * Lists:
         - Remove ugly hack for admin lists (staff left to a X.net group)    -MC
+
     * Notifications :
         - iCal calendar for birthdays.                                      -Car
 
@@ -32,6 +33,7 @@ Bug/Wish :
 
     * Admin :
         - #408: Admin can change sex of users without root intervention.    -Car
+        - #417: Activate or unactivate emails directly from admin page.     -Car
 
     * Carnet :
         - #414: Link to users' fiche go to private fiche and not public.    -Car
@@ -41,6 +43,9 @@ Bug/Wish :
         - #391: Go back to top link at end of each event.                   -Car
         - #411: Count chars in events proposals to make them smaller.       -Car
 
+    * Lists :
+        - #396, #416: Making moderating ML messages easier.                 -Car
+
     * Profile :
         - #400: Several phones for each address.                            -Car
         - #406: Direct link in owner's fiche to edit it.                    -Car

diff --git a/htdocs/admin/utilisateurs.php b/htdocs/admin/utilisateurs.php
index 9ea8941..8c4011d 100644 (file)
--- a/htdocs/admin/utilisateurs.php
+++ b/htdocs/admin/utilisateurs.php
@@ -90,7 +90,16 @@ if ($login) {
                     $page->trig($val." a été supprimé");
                 }
                break;
-
+        case "activate_fwd":
+        if (!empty($val)) {
+            $redirect->modify_one_email($val, true);
+        }
+        break;
+        case "deactivate_fwd":
+        if (!empty($val)) {
+            $redirect->modify_one_email($val, false);
+        }
+        break;
            case "add_alias":
                $globals->xdb->execute("INSERT INTO  aliases (id,alias,type) VALUES  ({?}, {?}, 'alias')",
                         $mr['user_id'], Env::get('email'));

diff --git a/include/emails.inc.php b/include/emails.inc.php
index d095ed9..c28b74f 100644 (file)
--- a/include/emails.inc.php
+++ b/include/emails.inc.php
@@ -274,6 +274,17 @@ class Redirect
         }
     }
 
+    function modify_one_email($email, $activate) {
+        foreach ($this->emails as $i=>$mail) {
+            if ($mail->email == $email) {
+                if ($activate)
+                    $this->emails[$i]->activate($this->uid);
+                else
+                    $this->emails[$i]->deactivate($this->uid);
+            }
+        }
+    }
+
     // }}}
 }
 

diff --git a/templates/admin/utilisateurs.tpl b/templates/admin/utilisateurs.tpl
index 16e6bec..83528e1 100644 (file)
--- a/templates/admin/utilisateurs.tpl
+++ b/templates/admin/utilisateurs.tpl
@@ -98,6 +98,13 @@ function del_fwd(fwd) {
   document.forms.fwds.del_fwd.value = fwd;
   document.forms.fwds.submit();
 }
+function act_fwd(fwd, activate) {
+  if (activate)
+    document.forms.fwds.activate_fwd.value = fwd;
+  else
+    document.forms.fwds.deactivate_fwd.value = fwd;
+  document.forms.fwds.submit();
+}
 // ]]>
 </script>
 {/literal}
@@ -255,10 +262,10 @@ Pour ceci changer ses permissions en 'disabled'.
 <strong>* à ne modifier qu'avec l'accord express de l'utilisateur !!!</strong>
 </p>
 
-<form id="fwds" method="post" action="{$smarty.server.PHP_SELF}">
+<form id="fwds" method="post" action="{$smarty.server.PHP_SELF}#fwds">
   <table class="bicol" cellpadding="2" cellspacing="0">
     <tr>
-      <th colspan="3">
+      <th colspan="4">
         Redirections
       </th>
     </tr>
@@ -268,6 +275,9 @@ Pour ceci changer ses permissions en 'disabled'.
         {if $mail->active}active{/if}
       </td>
       <td>
+        <span class="smaller"><a href="javascript:act_fwd('{$mail->email}',{if $mail->active}false{else}true{/if})">{if $mail->active}des{/if}activer</a></span>
+      </td>
+      <td>
         {$mail->email}
       </td>
       <td class="action">
@@ -276,7 +286,7 @@ Pour ceci changer ses permissions en 'disabled'.
     </tr>
     {/foreach}
     <tr>
-      <td class="titre">
+      <td class="titre" colspan="2">
         Ajouter un email
       </td>
       <td>
@@ -285,6 +295,8 @@ Pour ceci changer ses permissions en 'disabled'.
       <td class="action">
         <input type="hidden" name="user_id" value="{$mr.user_id}" />
         <input type="hidden" name="del_fwd" value="" />
+        <input type="hidden" name="activate_fwd" value="" />
+        <input type="hidden" name="deactivate_fwd" value="" />
         <input type="submit" name="add_fwd" value="Ajouter" />
       </td>
     </tr>

diff --git a/templates/listes/moderate.tpl b/templates/listes/moderate.tpl
index fdfef09..2569168 100644 (file)
--- a/templates/listes/moderate.tpl
+++ b/templates/listes/moderate.tpl
@@ -69,7 +69,9 @@
   </li>
   <li>
   <strong>détruire:</strong> le mail est effacé sans autre forme de procès.
-  N'utiliser <strong>QUE</strong> pour les virus et les courriers indésirables.
+  N'utiliser <strong>QUE</strong> pour les virus et les courriers indésirables. <br/>
+  S'il y a trop d'indésirables, il est probablement plus rapide pour la suite de les
+  <a href="listes-options.php?liste={$smarty.request.liste}#antispam">jeter directement</a> et non de les modérer. 
   </li>
 </ul>
 
@@ -88,9 +90,8 @@
     <td class='right'>{$m.size}o</td>
     <td class='right'>{$m.stamp|date_format:"%X<br />%x"}</td>
     <td class='action'>
-      <a href='?liste={$smarty.request.liste}&amp;mid={$m.id}'>voir</a>
-      <a href='?liste={$smarty.request.liste}&amp;mid={$m.id}&amp;mok=1'>accepter</a><br />
-      <a href='?liste={$smarty.request.liste}&amp;mid={$m.id}&amp;mdel=1'>détruire</a></td>
+      <a href='?liste={$smarty.request.liste}&amp;mid={$m.id}'>voir</a><br/>
+      <a href='?liste={$smarty.request.liste}&amp;mid={$m.id}&amp;mok=1'>accepter</a>&nbsp;<a href='?liste={$smarty.request.liste}&amp;mid={$m.id}&amp;mdel=1'>détruire</a></td>
   </tr>
   {/foreach}
 </table>

diff --git a/templates/listes/options.tpl b/templates/listes/options.tpl
index 00532f4..28a2c45 100644 (file)
--- a/templates/listes/options.tpl
+++ b/templates/listes/options.tpl
@@ -128,7 +128,7 @@ Tu n'es pas administrateur de la liste, mais du site.
         <span class='smaller'>détermine la politique de l'antispam sur cette liste.</span>
       </td>
       <td>
-        <em>que faire des mails marqués « [spam probable] » ?</em><br />
+        <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<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} /> les supprimer