Removes incorrect implementation of 'Notifies the sender when a moderated message...
[platal.git] / modules / lists.php
index 11c5d3f..bb467b1 100644 (file)
@@ -131,7 +131,7 @@ class ListsModule extends PLModule
 
     function handler_ajax(&$page, $list = null)
     {
-        header('Content-Type: text/html; charset="UTF-8"');
+        pl_content_headers("text/html");
         $domain = $this->prepare_client($page);
         $page->changeTpl('lists/liste.inc.tpl', NO_SKIN);
         S::assert_xsrf_token();
@@ -234,7 +234,7 @@ class ListsModule extends PLModule
         $liste = Post::v('liste');
 
         if (empty($liste)) {
-            $page->trigError('Le champ «adresse souhaitée» est vide.');
+            $page->trigError('Le champ « adresse souhaitée » est vide.');
         }
         if (!preg_match("/^[a-zA-Z0-9\-]*$/", $liste)) {
             $page->trigError('Le nom de la liste ne doit contenir que des lettres non accentuées, chiffres et tirets.');
@@ -245,17 +245,17 @@ class ListsModule extends PLModule
             $domain = $promo . '.' . $globals->mail->domain;
 
             if (($promo < 1921) || ($promo > date('Y'))) {
-                $page->trigError('La promotion est mal renseignée, elle doit être du type : 2004.');
+                $page->trigError('La promotion est mal renseignée, elle doit être du type&nbsp;: 2004.');
             }
 
             $new = $liste . '@' . $domain;
-            $res = XDB::query('SELECT COUNT(*) FROM x4dat.virtual WHERE alias={?}', $new);
+            $res = XDB::query('SELECT COUNT(*) FROM virtual WHERE alias={?}', $new);
 
         } else {
             if ($asso == "groupex") {
                 $groupex_name = Post::v('groupex_name');
 
-                $res_groupe = XDB::query('SELECT mail_domain FROM groupex.asso WHERE nom={?}', $groupex_name);
+                $res_groupe = XDB::query('SELECT mail_domain FROM #groupex#.asso WHERE nom={?}', $groupex_name);
                 $domain = $res_groupe->fetchOneCell();
 
                 if (!$domain) {
@@ -263,7 +263,7 @@ class ListsModule extends PLModule
                 }
 
                 $new = $liste . '@' . $domain;
-                $res = XDB::query('SELECT COUNT(*) FROM x4dat.virtual WHERE alias={?}', $new);
+                $res = XDB::query('SELECT COUNT(*) FROM virtual WHERE alias={?}', $new);
             } else {
                 $res = XDB::query("SELECT COUNT(*) FROM aliases WHERE alias={?}", $liste);
                 $domain = $globals->mail->domain;
@@ -273,7 +273,7 @@ class ListsModule extends PLModule
         $n = $res->fetchOneCell();
 
         if ($n) {
-            $page->trigError('L\'«adresse souhaitée» est déjà prise.');
+            $page->trigError("L'«&nbsp;adresse souhaitée&nbsp;» est déjà prise.");
         }
 
         if (!Post::v('desc')) {
@@ -289,6 +289,7 @@ class ListsModule extends PLModule
         }
 
         if (!$page->nb_errs()) {
+            $page->trigSuccess('Demande de création envoyée&nbsp;!');
             $page->assign('created', true);
             require_once 'validations.inc.php';
             $req = new ListeReq(S::user(), $asso, $liste, $domain,
@@ -334,7 +335,7 @@ class ListsModule extends PLModule
             $page->assign_by_ref('owners',  $moderos);
             $page->assign('nb_m',  count($mem));
         } else {
-            $page->kill("La liste n'existe pas ou tu n'as pas le droit d'en voir les détails");
+            $page->kill("La liste n'existe pas ou tu n'as pas le droit d'en voir les détails.");
         }
     }
 
@@ -346,9 +347,7 @@ class ListsModule extends PLModule
         $this->prepare_client($page);
         $members = $this->client->get_members($liste);
         $list = list_fetch_names(list_extract_members($members[1]));
-        header('Content-Type: text/x-csv; charset=utf-8;');
-        header('Pragma: ');
-        header('Cache-Control: ');
+        pl_content_headers("text/x-csv");
 
         echo "email,nom,prenom,promo\n";
         foreach ($list as $member) {
@@ -378,7 +377,7 @@ class ListsModule extends PLModule
 
         $owners = $this->client->get_owners($liste);
         if (!is_array($owners)) {
-            $page->kill("La liste n'existe pas ou tu n'as pas le droit d'en voir les détails");
+            $page->kill("La liste n'existe pas ou tu n'as pas le droit d'en voir les détails.");
         }
 
         global $platal;
@@ -532,29 +531,15 @@ class ListsModule extends PLModule
             }
         } elseif (Env::has('mid')) {
             if (Get::has('mid') && !Env::has('mok') && !Env::has('mdel')) {
-                $page->changeTpl('lists/moderate_mail.tpl');
-                require_once('banana/moderate.inc.php');
-                $params = array('listname' => $liste, 'domain' => $domain,
-                                'artid' => Get::i('mid'), 'part' => Get::v('part'), 'action' => Get::v('action'));
-                $params['client'] = $this->client;
-                run_banana($page, 'ModerationBanana', $params);
+                require_once 'banana/moderate.inc.php';
 
-                $msg = file_get_contents('/etc/mailman/fr/refuse.txt');
-                $msg = str_replace("%(adminaddr)s", "$liste-owner@{$domain}", $msg);
-                $msg = str_replace("%(request)s",   "<< SUJET DU MAIL >>",    $msg);
-                $msg = str_replace("%(reason)s",    "<< TON EXPLICATION >>",  $msg);
-                $msg = str_replace("%(listname)s",  $liste, $msg);
-                $page->assign('msg', $msg);
-                return;
-            } elseif (Get::has('mid') && Env::has('mok')) {
                 $page->changeTpl('lists/moderate_mail.tpl');
-                require_once('banana/moderate.inc.php');
                 $params = array('listname' => $liste, 'domain' => $domain,
                                 'artid' => Get::i('mid'), 'part' => Get::v('part'), 'action' => Get::v('action'));
                 $params['client'] = $this->client;
                 run_banana($page, 'ModerationBanana', $params);
 
-                $msg = file_get_contents('/etc/mailman/fr/accept.txt');
+                $msg = file_get_contents('/etc/mailman/fr/refuse.txt');
                 $msg = str_replace("%(adminaddr)s", "$liste-owner@{$domain}", $msg);
                 $msg = str_replace("%(request)s",   "<< SUJET DU MAIL >>",    $msg);
                 $msg = str_replace("%(reason)s",    "<< TON EXPLICATION >>",  $msg);
@@ -563,7 +548,7 @@ class ListsModule extends PLModule
                 return;
             }
 
-            $mail = $this->moderate_mail($domain, $liste, Env::i('mid'));
+            $this->moderate_mail($domain, $liste, Env::i('mid'));
         } elseif (Env::has('sid')) {
             if (list($subs,$mails) = $this->get_pending_ops($domain, $liste)) {
                 foreach($subs as $user) {
@@ -589,7 +574,7 @@ class ListsModule extends PLModule
             $page->assign_by_ref('subs', $subs);
             $page->assign_by_ref('mails', $mails);
         } else {
-            $page->kill("La liste n'existe pas ou tu n'as pas le droit de la modérer");
+            $page->kill("La liste n'existe pas ou tu n'as pas le droit de la modérer.");
         }
     }
 
@@ -660,7 +645,7 @@ class ListsModule extends PLModule
             S::assert_xsrf_token();
 
             $members = User::getBulkForlifeEmails(Env::v('add_member'),
-                                                  false,
+                                                  true,
                                                   array('ListsModule', 'no_login_callback'));
             $arr = $this->client->mass_subscribe($liste, $members);
             if (is_array($arr)) {
@@ -678,7 +663,7 @@ class ListsModule extends PLModule
                 $page->trigError('Une erreur s\'est produite lors du téléchargement du fichier');
             } else {
                 $members = User::getBulkForlifeEmails($upload->getContents(),
-                                                      false,
+                                                      true,
                                                       array('ListsModule', 'no_login_callback'));
                 $arr = $this->client->mass_subscribe($liste, $members);
                 if (is_array($arr)) {
@@ -740,8 +725,8 @@ class ListsModule extends PLModule
 
         } else {
             $page->kill("La liste n'existe pas ou tu n'as pas le droit de l'administrer.<br />"
-                       ." Si tu penses qu'il s'agit d'une erreur, "
-                       ."<a href='mailto:support@polytechnique.org'>contact le support</a>");
+                      " Si tu penses qu'il s'agit d'une erreur, "
+                      . "<a href='mailto:support@polytechnique.org'>contact le support</a>.");
         }
     }
 
@@ -841,11 +826,11 @@ class ListsModule extends PLModule
                                  $type, $liste.$app.$domain);
                 }
                 $page->assign('deleted', true);
-                $page->trigSuccess('La liste a été détruite !');
+                $page->trigSuccess('La liste a été détruite&nbsp;!');
             } else {
                 $page->kill('Une erreur est survenue lors de la suppression de la liste.<br />'
                          . 'Contact les administrateurs du site pour régler le problème : '
-                         . '<a href="mailto:support@polytechnique.org">support@polytechnique.org</a>');
+                         . '<a href="mailto:support@polytechnique.org">support@polytechnique.org</a>.');
             }
         } elseif (list($details,$options) = $this->client->get_owner_options($liste)) {
             if (!$details['own']) {
@@ -855,7 +840,7 @@ class ListsModule extends PLModule
             $page->assign_by_ref('options', $options);
             $page->assign('bogo_level', $this->client->get_bogo_level($liste));
         } else {
-            $page->kill("La liste n'existe pas ou tu n'as pas le droit de l'administrer");
+            $page->kill("La liste n'existe pas ou tu n'as pas le droit de l'administrer.");
         }
     }
 
@@ -884,7 +869,7 @@ class ListsModule extends PLModule
             $page->assign_by_ref('details', $details);
             $page->assign_by_ref('options', $options);
         } else {
-            $page->kill("La liste n'existe pas");
+            $page->kill("La liste n'existe pas.");
         }
     }
 
@@ -907,7 +892,7 @@ class ListsModule extends PLModule
             $page->assign_by_ref('details', $details);
             $page->assign_by_ref('options', $options);
         } else {
-            $page->kill("La liste n'existe pas");
+            $page->kill("La liste n'existe pas.");
         }
     }