admin/emails/broken: remove trailing spaces after email addresses
[platal.git] / modules / email.php
index a632e47..78cef2d 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /***************************************************************************
- *  Copyright (C) 2003-2010 Polytechnique.org                              *
+ *  Copyright (C) 2003-2014 Polytechnique.org                              *
  *  http://opensource.polytechnique.org/                                   *
  *                                                                         *
  *  This program is free software; you can redistribute it and/or modify   *
@@ -25,12 +25,12 @@ class EmailModule extends PLModule
     {
         return array(
             'emails'                  => $this->make_hook('emails',      AUTH_COOKIE, 'mail'),
-            'emails/alias'            => $this->make_hook('alias',       AUTH_MDP,    'mail'),
-            'emails/antispam'         => $this->make_hook('antispam',    AUTH_MDP,    'mail'),
-            'emails/broken'           => $this->make_hook('broken',      AUTH_COOKIE),
-            'emails/redirect'         => $this->make_hook('redirect',    AUTH_MDP,    'mail'),
-            'emails/send'             => $this->make_hook('send',        AUTH_MDP,    'mail'),
-            'emails/antispam/submit'  => $this->make_hook('submit',      AUTH_COOKIE),
+            'emails/alias'            => $this->make_hook('alias',       AUTH_PASSWD, 'mail'),
+            'emails/antispam'         => $this->make_hook('antispam',    AUTH_PASSWD, 'mail'),
+            'emails/broken'           => $this->make_hook('broken',      AUTH_COOKIE, 'user'),
+            'emails/redirect'         => $this->make_hook('redirect',    AUTH_PASSWD, 'mail'),
+            'emails/send'             => $this->make_hook('send',        AUTH_PASSWD, 'mail'),
+            'emails/antispam/submit'  => $this->make_hook('submit',      AUTH_COOKIE, 'user'),
             'emails/test'             => $this->make_hook('test',        AUTH_COOKIE, 'mail', NO_AUTH),
 
             'emails/rewrite/in'       => $this->make_hook('rewrite_in',  AUTH_PUBLIC),
@@ -38,14 +38,14 @@ class EmailModule extends PLModule
 
             'emails/imap/in'          => $this->make_hook('imap_in',     AUTH_PUBLIC),
 
-            'admin/emails/duplicated' => $this->make_hook('duplicated',  AUTH_MDP,    'admin'),
-            'admin/emails/watch'      => $this->make_hook('duplicated',  AUTH_MDP,    'admin'),
-            'admin/emails/lost'       => $this->make_hook('lost',        AUTH_MDP,    'admin'),
-            'admin/emails/broken'     => $this->make_hook('broken_addr', AUTH_MDP,    'admin'),
+            'admin/emails/duplicated' => $this->make_hook('duplicated',  AUTH_PASSWD, 'admin'),
+            'admin/emails/watch'      => $this->make_hook('duplicated',  AUTH_PASSWD, 'admin'),
+            'admin/emails/lost'       => $this->make_hook('lost',        AUTH_PASSWD, 'admin'),
+            'admin/emails/broken'     => $this->make_hook('broken_addr', AUTH_PASSWD, 'admin,edit_directory'),
         );
     }
 
-    function handler_emails(&$page, $action = null, $email = null)
+    function handler_emails($page, $action = null, $email = null)
     {
         global $globals;
         require_once 'emails.inc.php';
@@ -61,27 +61,67 @@ class EmailModule extends PLModule
                 return PL_FORBIDDEN;
             }
 
-            XDB::execute("UPDATE  aliases
+            // First delete the bestalias flag from all this user's emails.
+            XDB::execute("UPDATE  email_source_account
                              SET  flags = TRIM(BOTH ',' FROM REPLACE(CONCAT(',', flags, ','), ',bestalias,', ','))
                            WHERE  uid = {?}", $user->id());
-            XDB::execute("UPDATE  aliases
+            // Then gives the bestalias flag to the given email.
+            list($email, $domain) = explode('@', $email);
+            XDB::execute("UPDATE  email_source_account  AS s
+                      INNER JOIN  email_virtual_domains AS m ON (m.id = s.domain)
+                      INNER JOIN  email_virtual_domains AS d ON (d.aliasing = m.id)
                              SET  flags = CONCAT_WS(',', IF(flags = '', NULL, flags), 'bestalias')
-                           WHERE  uid = {?} AND alias = {?}", $user->id(), $email);
+                           WHERE  s.uid = {?} AND s.email = {?} AND d.name = {?}",
+                         $user->id(), $email, $domain);
+            XDB::execute('UPDATE  accounts              AS a
+                      INNER JOIN  email_virtual_domains AS d ON (d.name = {?})
+                      INNER JOIN  email_virtual_domains AS m ON (d.aliasing = m.id)
+                             SET  a.best_domain = d.id
+                           WHERE  a.uid = {?} AND m.name = {?}',
+                         $domain, $user->id(), $user->mainEmailDomain());
 
             // As having a non-null bestalias value is critical in
             // plat/al's code, we do an a posteriori check on the
             // validity of the bestalias.
             fix_bestalias($user);
+            // Then refetch the user to update its bestalias.
+            S::set('user', User::getWithUID(S::user()->id()));
         }
 
         // Fetch and display aliases.
-        $sql = "SELECT  alias, (type='a_vie') AS a_vie,
-                        (alias REGEXP '\\\\.[0-9]{2}$') AS cent_ans,
-                        FIND_IN_SET('bestalias',flags) AS best, expire
-                  FROM  aliases
-                 WHERE  uid = {?} AND type!='homonyme'
-              ORDER BY  LENGTH(alias)";
-        $page->assign('aliases', XDB::iterator($sql, $user->id()));
+        $aliases = XDB::iterator("SELECT  CONCAT(s.email, '@', d.name) AS email, (s.type = 'forlife') AS forlife,
+                                          (s.email REGEXP '\\\\.[0-9]{2}$') AS hundred_year, s.expire,
+                                          (FIND_IN_SET('bestalias', s.flags) AND a.best_domain = d.id) AS bestalias,
+                                          ((s.type = 'alias_aux') AND d.aliasing = d.id) AS alias
+                                    FROM  email_source_account  AS s
+                              INNER JOIN  accounts              AS a ON (s.uid = a.uid)
+                              INNER JOIN  email_virtual_domains AS m ON (s.domain = m.id)
+                              INNER JOIN  email_virtual_domains AS d ON (d.aliasing = m.id)
+                                   WHERE  s.uid = {?}
+                                ORDER BY  !alias, s.email, d.name",
+                                 $user->id());
+        $aliases_forlife = array();
+        $aliases_hundred = array();
+        $aliases_other = array();
+        while ($a = $aliases->next()) {
+            if ($a['forlife']) {
+                $aliases_forlife[] = $a;
+            } elseif ($a['hundred_year']) {
+                $aliases_hundred[] = $a;
+            } else {
+                $aliases_other[] = $a;
+            }
+        }
+        $page->assign('aliases_forlife', $aliases_forlife);
+        $page->assign('aliases_hundred', $aliases_hundred);
+        $page->assign('aliases_other', $aliases_other);
+
+        $alias = XDB::fetchOneCell('SELECT  COUNT(email)
+                                      FROM  email_source_account
+                                     WHERE  uid = {?} AND type = \'alias_aux\'',
+                                   $user->id());
+        $page->assign('alias', $alias);
+
 
         // Check for homonyms.
         $page->assign('homonyme', $user->homonyme);
@@ -90,21 +130,15 @@ class EmailModule extends PLModule
         $redirect = new Redirect($user);
         $page->assign('mails', $redirect->active_emails());
 
-        // Display, when available, the @alias_dom email alias.
-        $res = XDB::query(
-                "SELECT  alias
-                   FROM  virtual          AS v
-             INNER JOIN  virtual_redirect AS vr USING(vid)
-                  WHERE  (redirect={?} OR redirect={?})
-                         AND alias LIKE '%@{$globals->mail->alias_dom}'",
-                $user->forlifeEmail(),
-                // TODO: remove this über-ugly hack. The issue is that you need
-                // to remove all @m4x.org addresses in virtual_redirect first.
-                $user->login() . '@' . $globals->mail->domain2);
-        $page->assign('melix', $res->fetchOneCell());
+        // User's mail domains.
+        $mail_domains = array($user->alternateEmailDomain());
+        $mail_domains[] = User::$sub_mail_domains['all'] . $globals->mail->domain;
+        $mail_domains[] = User::$sub_mail_domains['all'] . $globals->mail->domain2;
+        $page->assign('main_email_domain', $user->mainEmailDomain());
+        $page->assign('mail_domains', $mail_domains);
     }
 
-    function handler_alias(&$page, $action = null, $value = null)
+    function handler_alias($page, $action = null, $value = null)
     {
         global $globals;
 
@@ -112,76 +146,77 @@ class EmailModule extends PLModule
         $page->setTitle('Alias melix.net');
 
         $user = S::user();
-        $page->assign('demande', AliasReq::get_request($user->id()));
+        $page->assign('request', AliasReq::get_request($user->id()));
 
         // Remove the email alias.
-        if ($action == 'delete' && $value) {
+        if ($action == 'delete') {
             S::assert_xsrf_token();
 
-            XDB::execute(
-                    "DELETE  virtual, virtual_redirect
-                       FROM  virtual
-                 INNER JOIN  virtual_redirect USING (vid)
-                      WHERE  alias = {?} AND (redirect = {?} OR redirect = {?})",
-                    $value, $user->forlifeEmail(), $user->m4xForlifeEmail());
+            XDB::execute('DELETE FROM  email_source_account
+                                WHERE  uid = {?} AND type = \'alias_aux\'',
+                         $user->id());
+
+            require_once 'emails.inc.php';
+            fix_bestalias($user);
         }
 
-        // Fetch existing @alias_dom aliases.
-        $alias = $user->emailAlias();
-        $visibility = $user->hasProfile() && $user->profile()->alias_pub;
-        $page->assign('actuel', $alias);
+        // Fetch existing auxiliary aliases.
+        list($alias, $old_alias) = XDB::fetchOneRow('SELECT  CONCAT(s.email, \'@\', d.name), s.email
+                                                       FROM  email_source_account  AS s
+                                                 INNER JOIN  email_virtual_domains AS d ON (s.domain = d.id)
+                                                      WHERE  s.uid = {?} AND s.type = \'alias_aux\'',
+                                                    $user->id());
+        $visibility = $user->hasProfile() && $user->profile()->isVisible($user->profile()->alias_pub);
+        $page->assign('current', $alias);
         $page->assign('user', $user);
         $page->assign('mail_public', $visibility);
 
-        if ($action == 'ask' && Env::has('alias') && Env::has('raison')) {
+        if ($action == 'ask' && Env::has('alias') && Env::has('reason')) {
             S::assert_xsrf_token();
 
-            //Si l'utilisateur vient de faire une damande
-            $alias  = Env::v('alias');
-            $raison = Env::v('raison');
-            $public = (Env::v('public', 'off') == 'on') ? 'public' : 'private';
+            // Retrieves user request.
+            $new_alias  = Env::v('alias');
+            $reason = Env::v('reason');
+            $public = (Env::v('public', 'off') == 'on') ? 'private' : 'hidden';
 
-            $page->assign('r_alias', $alias);
-            $page->assign('r_raison', $raison);
-            if ($public == 'public') {
+            $page->assign('r_alias', $new_alias);
+            $page->assign('r_reason', $reason);
+            if ($public == 'private') {
                 $page->assign('r_public', true);
             }
 
-            //Quelques vérifications sur l'alias (caractères spéciaux)
-            if (!preg_match("/^[a-zA-Z0-9\-.]{3,20}$/", $alias)) {
+            // Checks special charaters in alias.
+            if (!preg_match("/^[a-zA-Z0-9\-.]{2,19}[a-zA-Z0-9\-]$/", $new_alias)) {
                 $page->trigError("L'adresse demandée n'est pas valide."
                             . " Vérifie qu'elle comporte entre 3 et 20 caractères"
                             . " et qu'elle ne contient que des lettres non accentuées,"
-                            . " des chiffres ou les caractères - et .");
+                            . " des chiffres ou les caractères '-' et '.'. De plus, elle ne"
+                            . " peut pas se terminer par un point.");
                 return;
             } else {
-                $alias_mail = $alias.'@'.$globals->mail->alias_dom;
-
-                //vérifier que l'alias n'est pas déja pris
-                $res = XDB::query('SELECT  COUNT(*)
-                                     FROM  virtual
-                                    WHERE  alias={?}',
-                                  $alias_mail);
+                // Checks if the alias has already been given.
+                $res = XDB::query('SELECT  COUNT(email)
+                                     FROM  email_source_account
+                                    WHERE  email = {?} AND type = \'alias_aux\'',
+                                  $new_alias);
                 if ($res->fetchOneCell() > 0) {
-                    $page->trigError("L'alias $alias_mail a déja été attribué.
-                                      Tu ne peux donc pas l'obtenir.");
+                    $page->trigError("L'alias $new_alias a déja été attribué. Tu ne peux donc pas l'obtenir.");
                     return;
                 }
 
-                //vérifier que l'alias n'est pas déja en demande
+                // Checks if the alias has already been asked for.
                 $it = Validate::iterate('alias');
                 while($req = $it->next()) {
-                    if ($req->alias == $alias_mail) {
-                        $page->trigError("L'alias $alias_mail a déja été demandé.
-                                    Tu ne peux donc pas l'obtenir pour l'instant.");
-                        return ;
+                    if ($req->alias == $new_alias) {
+                        $page->trigError("L'alias $new_alias a déja été demandé. Tu ne peux donc pas l'obtenir pour l'instant.");
+                        return;
                     }
                 }
 
-                //Insertion de la demande dans la base, écrase les requêtes précédente
-                $myalias = new AliasReq($user, $alias, $raison, $public);
+                // Sends requests. This will erase any previous alias pending request.
+                $myalias = new AliasReq($user, $new_alias, $reason, $public, $old_alias);
                 $myalias->submit();
-                $page->assign('success',$alias);
+                $page->assign('success', $new_alias);
                 return;
             }
         } elseif ($action == 'set' && ($value == 'public' || $value == 'private')) {
@@ -190,24 +225,18 @@ class EmailModule extends PLModule
             }
 
             if ($user->hasProfile()) {
-                XDB::execute("UPDATE  profiles
+                XDB::execute('UPDATE  profiles
                                  SET  alias_pub = {?}
-                               WHERE  pid = {?}",
+                               WHERE  pid = {?}',
                             $value, $user->profile()->id());
             }
-            $visibility = ($value == 'public');
             exit;
         }
-
-        $page->assign('actuel', $alias);
-        $page->assign('user', $user);
-        $page->assign('mail_public', $visibility);
     }
 
-    function handler_redirect(&$page, $action = null, $email = null)
+    function handler_redirect($page, $action = null, $email = null, $rewrite = null)
     {
         global $globals;
-
         require_once 'emails.inc.php';
 
         $page->changeTpl('emails/redirect.tpl');
@@ -238,15 +267,13 @@ class EmailModule extends PLModule
         }
 
         if ($action == 'rewrite' && $email) {
-            $rewrite = @func_get_arg(3);
             $redirect->modify_one_email_redirect($email, $rewrite);
         }
 
         if (Env::has('emailop')) {
             S::assert_xsrf_token();
 
-            $actifs = Env::v('emails_actifs', Array());
-            print_r(Env::v('emails_rewrite'));
+            $actifs = Env::v('emails_actifs', array());
             if (Env::v('emailop') == "ajouter" && Env::has('email')) {
                 $error_email = false;
                 $new_email = Env::v('email');
@@ -263,7 +290,7 @@ class EmailModule extends PLModule
             } elseif (empty($actifs)) {
                 $result = ERROR_INACTIVE_REDIRECTION;
             } elseif (is_array($actifs)) {
-                $result = $redirect->modify_email($actifs, Env::v('emails_rewrite', Array()));
+                $result = $redirect->modify_email($actifs, Env::v('emails_rewrite', array()));
             }
         }
 
@@ -281,31 +308,17 @@ class EmailModule extends PLModule
                              . $globals->mail->domain2 . ' ni vers polytechnique.edu.');
             break;
         }
-
-        // Fetch the @alias_dom email alias, if any.
-        $res = XDB::query(
-                "SELECT  alias
-                   FROM  virtual
-             INNER JOIN  virtual_redirect USING(vid)
-                  WHERE  (redirect={?} OR redirect={?})
-                         AND alias LIKE '%@{$globals->mail->alias_dom}'",
-                $user->forlifeEmail(),
-                // TODO: remove this über-ugly hack. The issue is that you need
-                // to remove all @m4x.org addresses in virtual_redirect first.
-                $user->login() . '@' . $globals->mail->domain2);
-        $melix = $res->fetchOneCell();
-        if ($melix) {
-            list($melix) = explode('@', $melix);
-            $page->assign('melix',$melix);
-        }
-
         // Fetch existing email aliases.
-        $res = XDB::query(
-                "SELECT  alias,expire
-                   FROM  aliases
-                  WHERE  uid={?} AND (type='a_vie' OR type='alias')
-               ORDER BY  !FIND_IN_SET('usage',flags), LENGTH(alias)", $user->id());
-        $page->assign('alias', $res->fetchAllAssoc());
+        $alias = XDB::query('SELECT  CONCAT(s.email, \'@\', d.name) AS email, s.expire
+                               FROM  email_source_account  AS s
+                         INNER JOIN  email_virtual_domains AS m ON (s.domain = m.id)
+                         INNER JOIN  email_virtual_domains AS d ON (m.id = d.aliasing)
+                              WHERE  s.uid = {?}
+                           ORDER BY  NOT(s.type = \'alias_aux\'), s.email, d.name',
+                            $user->id());
+        $page->assign('alias', $alias->fetchAllAssoc());
+        $page->assign('best_email', $user->bestEmail());
+
         $page->assign('emails', $redirect->emails);
 
         // Display GoogleApps acount information.
@@ -313,10 +326,10 @@ class EmailModule extends PLModule
         $page->assign('googleapps', GoogleAppsAccount::account_status($user->id()));
 
         require_once 'emails.combobox.inc.php';
-        fill_email_combobox($page);
+        fill_email_combobox($page, array('job', 'stripped_directory'));
     }
 
-    function handler_antispam(&$page, $statut_filtre = null)
+    function handler_antispam($page, $filter_status = null, $redirection = null)
     {
         require_once 'emails.inc.php';
         $wp = new PlWikiPage('Xorg.Antispam');
@@ -326,13 +339,20 @@ class EmailModule extends PLModule
 
         $user = S::user();
         $bogo = new Bogo($user);
-        if (isset($statut_filtre)) {
-            $bogo->change($statut_filtre + 0);
+        if (!is_null($filter_status)) {
+            if (is_null($redirection)) {
+                $bogo->changeAll($filter_status);
+            } else {
+                $bogo->change($redirection, $filter_status);
+            }
         }
-        $page->assign('filtre', $bogo->level());
+        $page->assign('filter', $bogo->state);
+        $page->assign('single_state', $bogo->single_state);
+        $page->assign('single_redirection', $bogo->single_redirection);
+        $page->assign('redirections', $bogo->redirections);
     }
 
-    function handler_submit(&$page)
+    function handler_submit($page)
     {
         $wp = new PlWikiPage('Xorg.Mails');
         $wp->buildCache();
@@ -367,10 +387,9 @@ class EmailModule extends PLModule
         }
     }
 
-    function handler_send(&$page)
+    function handler_send($page)
     {
         $page->changeTpl('emails/send.tpl');
-        $page->addJsLink('ajax.js');
 
         $page->setTitle('Envoyer un email');
 
@@ -382,8 +401,8 @@ class EmailModule extends PLModule
 
             unset($_POST['save']);
             if (trim(preg_replace('/-- .*/', '', Post::v('contenu'))) != "") {
-                $_POST['to_contacts'] = explode(';', @$_POST['to_contacts']);
-                $_POST['cc_contacts'] = explode(';', @$_POST['cc_contacts']);
+                Post::set('to_contacts', explode(';', Post::s('to_contacts')));
+                Post::set('cc_contacts', explode(';', Post::s('cc_contacts')));
                 $data = serialize($_POST);
                 XDB::execute('INSERT INTO  email_send_save (uid, data)
                                    VALUES  ({?}, {?})
@@ -459,7 +478,7 @@ class EmailModule extends PLModule
                     foreach ($files as $name=>&$upload) {
                         $mymail->addUploadAttachment($upload, $name);
                     }
-                    if (Env::v('nowiki')) {
+                    if (Env::v('wiki') == 'text') {
                         $mymail->setTxtBody(wordwrap($txt, 78, "\n"));
                     } else {
                         $mymail->setWikiBody($txt);
@@ -494,9 +513,17 @@ class EmailModule extends PLModule
         $page->assign('contacts', $contacts);
         $page->assign('maxsize', ini_get('upload_max_filesize') . 'o');
         $page->assign('user', S::user());
+        $preferences = XDB::fetchOneAssoc('SELECT  from_email, from_format
+                                             FROM  accounts
+                                            WHERE  uid = {?}',
+                                          S::user()->id());
+        if ($preferences['from_email'] == '') {
+            $preferences['from_email'] = '"' . S::user()->fullName() . '" <' . S::user()->bestEmail() . '>';
+        }
+        $page->assign('preferences', $preferences);
     }
 
-    function handler_test(&$page, $hruid = null)
+    function handler_test($page, $hruid = null)
     {
         require_once 'emails.inc.php';
 
@@ -526,7 +553,7 @@ class EmailModule extends PLModule
         exit;
     }
 
-    function handler_rewrite_in(&$page, $mail, $hash)
+    function handler_rewrite_in($page, $mail, $hash)
     {
         $page->changeTpl('emails/rewrite.tpl');
         $page->assign('option', 'in');
@@ -538,23 +565,23 @@ class EmailModule extends PLModule
             return PL_NOT_FOUND;
         }
         $mail{$pos} = '@';
-        $res = XDB::query("SELECT  COUNT(*)
-                             FROM  emails
-                            WHERE  email = {?} AND hash = {?}",
+        $res = XDB::query('SELECT  COUNT(*)
+                             FROM  email_redirect_account
+                            WHERE  redirect = {?} AND hash = {?} AND type = \'smtp\'',
                           $mail, $hash);
         $count = intval($res->fetchOneCell());
         if ($count > 0) {
-            XDB::query("UPDATE  emails
+            XDB::query('UPDATE  email_redirect_account
                            SET  allow_rewrite = true, hash = NULL
-                         WHERE  email = {?} AND hash = {?}",
-                         $mail, $hash);
+                         WHERE  redirect = {?} AND hash = {?} AND type = \'smtp\'',
+                       $mail, $hash);
             $page->trigSuccess("Réécriture activée pour l'adresse " . $mail);
             return;
         }
         return PL_NOT_FOUND;
     }
 
-    function handler_rewrite_out(&$page, $mail, $hash)
+    function handler_rewrite_out($page, $mail, $hash)
     {
         $page->changeTpl('emails/rewrite.tpl');
         $page->assign('option', 'out');
@@ -566,28 +593,28 @@ class EmailModule extends PLModule
             return PL_NOT_FOUND;
         }
         $mail{$pos} = '@';
-        $res = XDB::query("SELECT  COUNT(*)
-                             FROM  emails
-                            WHERE  email = {?} AND hash = {?}",
+        $res = XDB::query('SELECT  COUNT(*)
+                             FROM  email_redirect_account
+                            WHERE  redirect = {?} AND hash = {?} AND type = \'smtp\'',
                           $mail, $hash);
         $count = intval($res->fetchOneCell());
         if ($count > 0) {
             global $globals;
-            $res = XDB::query("SELECT  e.email, e.rewrite, a.alias
-                                 FROM  emails AS e
-                           INNER JOIN  aliases AS a ON (a.uid = e.uid AND a.type = 'a_vie')
-                                WHERE  e.email = {?} AND e.hash = {?}",
+            $res = XDB::query('SELECT  e.redirect, e.rewrite, a.hruid
+                                 FROM  email_redirect_account AS e
+                           INNER JOIN  accounts               AS a ON (e.uid = a.uid)
+                                WHERE  e.redirect = {?} AND e.hash = {?}',
                               $mail, $hash);
-            XDB::query("UPDATE  emails
+            XDB::query('UPDATE  email_redirect_account
                            SET  allow_rewrite = false, hash = NULL
-                         WHERE  email = {?} AND hash = {?}",
-                        $mail, $hash);
-            list($mail, $rewrite, $forlife) = $res->fetchOneRow();
+                         WHERE  redirect = {?} AND hash = {?}',
+                       $mail, $hash);
+            list($mail, $rewrite, $hruid) = $res->fetchOneRow();
             $mail = new PlMailer();
             $mail->setFrom("webmaster@" . $globals->mail->domain);
             $mail->addTo("support@" .  $globals->mail->domain);
             $mail->setSubject("Tentative de détournement de correspondance via le rewrite");
-            $mail->setTxtBody("$forlife a tenté un rewrite de $mail vers $rewrite. Cette demande a été rejetée via le web");
+            $mail->setTxtBody("$hruid a tenté un rewrite de $mail vers $rewrite. Cette demande a été rejetée via le web");
             $mail->send();
             $page->trigWarning("Un mail d'alerte a été envoyé à l'équipe de " . $globals->core->sitename);
             return;
@@ -595,7 +622,7 @@ class EmailModule extends PLModule
         return PL_NOT_FOUND;
     }
 
-    function handler_imap_in(&$page, $hash = null, $login = null)
+    function handler_imap_in($page, $hash = null, $login = null)
     {
         $page->changeTpl('emails/imap_register.tpl');
         $user = null;
@@ -612,24 +639,22 @@ class EmailModule extends PLModule
             }
         }
 
-        require_once('emails.inc.php');
+        require_once 'emails.inc.php';
         $page->assign('ok', false);
         if (S::logged() && (is_null($user) || $user->id() == S::i('uid'))) {
-            $storage = new EmailStorage(S::user(), 'imap');
-            $storage->activate();
+            Email::activate_storage(S::user(), 'imap', Bogo::IMAP_DEFAULT);
             $page->assign('ok', true);
             $page->assign('yourself', S::user()->displayName());
             $page->assign('sexe', S::user()->isFemale());
         } else if (!S::logged() && $user) {
-            $storage = new EmailStorage($user, 'imap');
-            $storage->activate();
+            Email::activate_storage($user, 'imap', Bogo::IMAP_DEFAULT);
             $page->assign('ok', true);
             $page->assign('yourself', $user->displayName());
             $page->assign('sexe', $user->isFemale());
         }
     }
 
-    function handler_broken(&$page, $warn = null, $email = null)
+    function handler_broken($page, $warn = null, $email = null)
     {
         require_once 'emails.inc.php';
         $wp = new PlWikiPage('Xorg.PatteCassée');
@@ -642,13 +667,17 @@ class EmailModule extends PLModule
         if ($warn == 'warn' && $email) {
             S::assert_xsrf_token();
 
+            // Usual verifications.
             $email = valide_email($email);
-            // vérifications d'usage
-            $uid = XDB::fetchOneCell("SELECT  uid
-                                        FROM  emails
-                                       WHERE  email = {?}", $email);
+            $uid = XDB::fetchOneCell('SELECT  uid
+                                        FROM  email_redirect_account
+                                       WHERE  redirect = {?}', $email);
+
             if ($uid) {
                 $dest = User::getWithUID($uid);
+                $active = XDB::fetchOneCell('SELECT flags
+                                               FROM email_redirect_account
+                                              WHERE redirect = {?} AND uid = {?}', $email, $uid) == 'active';
 
                 $mail = new PlMailer('emails/broken-web.mail.tpl');
                 $mail->assign('email', $email);
@@ -659,43 +688,19 @@ class EmailModule extends PLModule
         } elseif (Post::has('email')) {
             S::assert_xsrf_token();
 
-            $email = valide_email(Post::v('email'));
+            $email = Post::t('email');
 
-            list(,$fqdn) = explode('@', $email);
-            $fqdn = strtolower($fqdn);
-            if ($fqdn == 'polytechnique.org' || $fqdn == 'melix.org' ||  $fqdn == 'm4x.org' || $fqdn == 'melix.net') {
+            if (!User::isForeignEmailAddress($email)) {
                 $page->assign('neuneu', true);
             } else {
-                $page->assign('email',$email);
-                $x = XDB::fetchOneAssoc("SELECT  e1.uid, e1.panne != 0 AS panne, a.hruid,
-                                                 (COUNT(e2.uid) + IF(FIND_IN_SET('googleapps', eo.storage), 1, 0)) AS nb_mails
-                                           FROM  emails        AS e1
-                                     INNER JOIN  email_options AS eo ON (eo.uid = e1.uid)
-                                     INNER JOIN  accounts      AS a  ON (e1.uid = a.uid)
-                                      LEFT JOIN  emails        AS e2 ON (e1.uid = e2.uid
-                                                                         AND FIND_IN_SET('active', e2.flags)
-                                                                         AND e1.email != e2.email)
-                                          WHERE  e1.email = {?}
-                                       GROUP BY  e1.uid", $email);
-                if ($x) {
-                    // on écrit dans la base que l'adresse est cassée
-                    if (!$x['panne']) {
-                        XDB::execute("UPDATE  emails
-                                         SET  panne=NOW(), last=NOW(), panne_level = 1
-                                       WHERE  email = {?}", $email);
-                    } else {
-                        XDB::execute("UPDATE  emails
-                                         SET  panne_level = 1
-                                       WHERE  email = {?} AND panne_level = 0", $email);
-                    }
-                    $x['user'] = User::getWithUID($x['uid']);
-                    $page->assign_by_ref('x', $x);
-                }
+                $user = mark_broken_email($email);
+                $page->assign('user', $user);
+                $page->assign('email', $email);
             }
         }
     }
 
-    function handler_duplicated(&$page, $action = 'list', $email = null)
+    function handler_duplicated($page, $action = 'list', $email = null)
     {
         $page->changeTpl('emails/duplicated.tpl');
 
@@ -734,12 +739,11 @@ class EmailModule extends PLModule
         $page->assign('action', $action);
 
         if ($action == 'list') {
-            $sql = "SELECT  w.email, w.detection, w.state, a.alias AS forlife
-                      FROM  email_watch  AS w
-                 LEFT JOIN  emails        AS e USING(email)
-                 LEFT JOIN  aliases       AS a ON (a.uid = e.uid AND a.type = 'a_vie')
-                  ORDER BY  w.state, w.email, a.alias";
-            $it = Xdb::iterRow($sql);
+            $it = XDB::iterRow('SELECT  w.email, w.detection, w.state, s.email AS forlife
+                                  FROM  email_watch            AS w
+                            INNER JOIN  email_redirect_account AS r ON (w.email = r.redirect)
+                            INNER JOIN  email_source_account   AS s ON (s.uid = r.uid AND s.type = \'forlife\')
+                              ORDER BY  w.state, w.email, s.email');
 
             $table = array();
             $props = array();
@@ -761,15 +765,15 @@ class EmailModule extends PLModule
             }
             $page->assign('table', $table);
         } elseif ($action == 'edit') {
-            $sql = "SELECT  w.detection, w.state, w.last, w.description,
-                            a1.alias AS edit, a2.alias AS forlife
-                      FROM  email_watch AS w
-                 LEFT JOIN  aliases      AS a1 ON (a1.uid = w.uid AND a1.type = 'a_vie')
-                 LEFT JOIN  emails       AS e  ON (w.email = e.email)
-                 LEFT JOIN  aliases      AS a2 ON (a2.uid = e.uid AND a2.type = 'a_vie')
-                     WHERE  w.email = {?}
-                  ORDER BY  a2.alias";
-            $it = Xdb::iterRow($sql, $email);
+            $it = XDB::iterRow('SELECT  w.detection, w.state, w.last, w.description,
+                                        a.hruid AS edit, s.email AS forlife
+                                  FROM  email_watch            AS w
+                            INNER JOIN  email_redirect_account AS r ON (w.email = r.redirect)
+                            INNER JOIN  email_source_account   AS s ON (s.uid = r.uid AND s.type = \'forlife\')
+                             LEFT JOIN  accounts               AS a ON (w.uid = a.uid)
+                                 WHERE  w.email = {?}
+                              ORDER BY  s.email',
+                               $email);
 
             $props = array();
             while (list($detection, $state, $last, $description, $edit, $forlife) = $it->next()) {
@@ -789,23 +793,25 @@ class EmailModule extends PLModule
         }
     }
 
-    function handler_lost(&$page, $action = 'list', $email = null)
+    function handler_lost($page, $action = 'list', $email = null)
     {
         $page->changeTpl('emails/lost.tpl');
 
         $page->assign('lost_emails',
                       XDB::iterator('SELECT  a.uid, a.hruid, pd.promo
-                                       FROM  accounts         AS a
-                                 INNER JOIN  email_options    AS eo ON (eo.uid = a.uid)
-                                  LEFT JOIN  emails           AS e  ON (a.uid = e.uid AND FIND_IN_SET(\'active\', e.flags))
-                                  LEFT JOIN  account_profiles AS ap ON (ap.uid = a.uid AND FIND_IN_SET(\'owner\', perms))
-                                  LEFT JOIN  profile_display  AS pd ON (ap.pid = pd.pid)
-                                      WHERE  e.uid IS NULL AND FIND_IN_SET(\'googleapps\', eo.storage) = 0
-                                             AND a.state = \'active\'
+                                       FROM  accounts               AS a
+                                 INNER JOIN  account_types          AS at ON (a.type = at.type)
+                                  LEFT JOIN  email_redirect_account AS er ON (er.uid = a.uid AND er.flags = \'active\' AND er.broken_level < 3
+                                                                              AND er.type != \'imap\' AND er.type != \'homonym\')
+                                  LEFT JOIN  account_profiles       AS ap ON (ap.uid = a.uid AND FIND_IN_SET(\'owner\', ap.perms))
+                                  LEFT JOIN  profile_display        AS pd ON (ap.pid = pd.pid)
+                                  LEFT JOIN  profiles               AS p ON (ap.pid = p.pid)
+                                      WHERE  a.state = \'active\' AND er.redirect IS NULL AND FIND_IN_SET(\'mail\', at.perms) AND p.deathdate IS NULL
+                                   GROUP BY  a.uid
                                    ORDER BY  pd.promo, a.hruid'));
     }
 
-    function handler_broken_addr(&$page)
+    function handler_broken_addr($page)
     {
         require_once 'emails.inc.php';
         $page->changeTpl('emails/broken_addr.tpl');
@@ -828,13 +834,13 @@ class EmailModule extends PLModule
                         if (empty($email) || $email == '@') {
                             $invalid_emails[] = trim($orig_email) . ': invalid email';
                         } elseif (!in_array($email, $valid_emails)) {
-                            $res = XDB::query('SELECT  COUNT(*)
-                                                 FROM  emails
-                                                WHERE  email = {?}', $email);
-                            if ($res->fetchOneCell() > 0) {
+                            $nb = XDB::fetchOneCell('SELECT  COUNT(*)
+                                                       FROM  email_redirect_account
+                                                      WHERE  redirect = {?}', $email);
+                            if ($nb > 0) {
                                 $valid_emails[] = $email;
                             } else {
-                                $invalid_emails[] = "$orig_email: no such redirection";
+                                $invalid_emails[] = $orig_email . ': no such redirection';
                             }
                         }
                     }
@@ -852,104 +858,89 @@ class EmailModule extends PLModule
             if ($list == '') {
                 $page->trigError('La liste est vide.');
             } else {
+                require_once 'notifs.inc.php';
+
                 $broken_user_list = array();
+                $broken_user_email_count = array();
+                $broken_user_profiles = array();
                 $broken_list = explode("\n", $list);
                 sort($broken_list);
-                foreach ($broken_list as $orig_email) {
-                    $email = valide_email(trim($orig_email));
-                    if (empty($email) || $email == '@') {
+
+                foreach ($broken_list as $email) {
+                    $email = trim($email);
+                    $userobj = null;
+                    if ($user = mark_broken_email($email, true)) {
+                        $userobj = User::getSilentWithUID($user['uid']);
+                    }
+                    if (is_null($userobj)) {
                         continue;
                     }
+                    $profile = $userobj->profile();
+                    if (is_null($profile)) {
+                        continue;
+                    }
+                    if ($user['nb_mails'] > 0 && $user['notify']) {
+                        $mail = new PlMailer('emails/broken.mail.tpl');
+                        $mail->setTo($userobj);
+                        $mail->assign('user', $user);
+                        $mail->assign('email', $email);
+                        $mail->send();
+                    } else {
+                        WatchProfileUpdate::register($profile, 'broken');
+                    }
 
-                    $sel = XDB::query(
-                        "SELECT  e1.uid, e1.panne != 0 AS panne, count(e2.uid) AS nb_mails,
-                                 acc.full_name, a.alias
-                           FROM  emails        AS e1
-                      LEFT JOIN  emails        AS e2 ON (e1.uid = e2.uid AND FIND_IN_SET('active', e2.flags)
-                                                         AND e1.email != e2.email)
-                     INNER JOIN  accounts      AS acc  ON (e1.uid = acc.uid)
-                     INNER JOIN  aliases       AS a  ON (acc.uid = a.uid AND FIND_IN_SET('bestalias', a.flags))
-                          WHERE  e1.email = {?}
-                       GROUP BY  e1.uid", $email);
-
-                    if ($x = $sel->fetchOneAssoc()) {
-                        if (!$x['panne']) {
-                            XDB::execute('UPDATE  emails
-                                             SET  panne=NOW(), last=NOW(), panne_level = 1
-                                           WHERE  email = {?}',
-                                          $email);
-                        } else {
-                            XDB::execute('UPDATE  emails
-                                             SET  last = CURDATE(), panne_level = panne_level + 1
-                                           WHERE  email = {?}
-                                                  AND DATE_ADD(last, INTERVAL 14 DAY) < CURDATE()',
-                                         $email);
-                        }
-
-                        if (!empty($x['nb_mails'])) {
-                            $mail = new PlMailer('emails/broken.mail.tpl');
-                            $mail->addTo("\"{$x['full_name']}\" <{$x['alias']}@"
-                                         . Platal::globals()->mail->domain . '>');
-                            $mail->assign('x', $x);
-                            $mail->assign('email', $email);
-                            $mail->send();
-                        }
-
-                        if (!isset($broken_user_list[$x['alias']])) {
-                            $broken_user_list[$x['alias']] = array($email);
-                        } else {
-                            $broken_user_list[$x['alias']][] = $email;
-                        }
+                    $pid = $profile->id();
+                    if (!isset($broken_user_list[$pid])) {
+                        $broken_user_list[$pid] = array($email);
+                    } else {
+                        $broken_user_list[$pid][] = $email;
                     }
+                    $broken_user_email_count[$pid] = $user['nb_mails'];
+                    $broken_user_profiles[$pid] = $profile;
                 }
 
-                XDB::execute("UPDATE  emails
-                                 SET  panne_level = panne_level - 1
-                               WHERE  flags = 'active' AND panne_level > 1
-                                      AND DATE_ADD(last, INTERVAL 1 MONTH) < CURDATE()");
-                XDB::execute("UPDATE  emails
-                                 SET  panne_level = 0
-                               WHERE  flags = 'active' AND panne_level = 1
-                                      AND DATE_ADD(last, INTERVAL 1 YEAR) < CURDATE()");
+                XDB::execute('UPDATE  email_redirect_account
+                                 SET  broken_level = broken_level - 1
+                               WHERE  flags = \'active\' AND broken_level > 1
+                                      AND DATE_ADD(last, INTERVAL 1 MONTH) < CURDATE()');
+                XDB::execute('UPDATE  email_redirect_account
+                                 SET  broken_level = 0
+                               WHERE  flags = \'active\' AND broken_level = 1
+                                      AND DATE_ADD(last, INTERVAL 1 YEAR) < CURDATE()');
+
+                // Sort $broken_user_list with (promo, sortname, pid)
+                $sortable_array = array();
+                foreach ($broken_user_list as $pid => $mails) {
+                    $profile = $broken_user_profiles[$pid];
+                    $sortable_array[$pid] = array($profile->promo(), $profile->sortName(), $pid);
+                }
+                asort($sortable_array);
 
                 // Output the list of users with recently broken addresses,
                 // along with the count of valid redirections.
-                require_once 'notifs.inc.php';
-                pl_content_headers("text/x-csv");
+                pl_cached_content_headers('text/x-csv', 1);
 
                 $csv = fopen('php://output', 'w');
-                fputcsv($csv, array('nom', 'promo', 'alias', 'bounce', 'nbmails', 'url', 'corps', 'job', 'networking'), ';');
-                foreach ($broken_user_list as $alias => $mails) {
-                    $sel = Xdb::query(
-                        "SELECT  acc.uid, count(DISTINCT(e.email)) AS nb_mails,
-                                 IFNULL(pd.public_name, acc.full_name) AS fullname,
-                                 IFNULL(pd.promo, 0) AS promo, IFNULL(pce.name, 'Aucun') AS corps,
-                                 IFNULL(pje.name, 'Aucun') AS job, GROUP_CONCAT(pn.address SEPARATOR ', ') AS networking
-                           FROM  aliases            AS a
-                     INNER JOIN  accounts           AS acc ON (a.uid = acc.uid)
-                      LEFT JOIN  emails             AS e   ON (e.uid = acc.uid
-                                                               AND FIND_IN_SET('active', e.flags) AND e.panne = 0)
-                      LEFT JOIN  account_profiles   AS ap  ON (acc.uid = ap.uid AND FIND_IN_SET('owner', ap.perms))
-                      LEFT JOIN  profile_display    AS pd  ON (pd.pid = ap.pid)
-                      LEFT JOIN  profile_corps      AS pc  ON (pc.pid = ap.pid)
-                      LEFT JOIN  profile_corps_enum AS pce ON (pc.current_corpsid = pce.id)
-                      LEFT JOIN  profile_job        AS pj  ON (pj.pid = ap.pid)
-                      LEFT JOIN  profile_job_enum   AS pje ON (pj.jobid = pje.id)
-                      LEFT JOIN  profile_networking AS pn  ON (pn.pid = ap.pid)
-                          WHERE  a.alias = {?}
-                       GROUP BY  acc.uid", $alias);
-
-                    if ($x = $sel->fetchOneAssoc()) {
-                        if ($x['nb_mails'] == 0) {
-                            $user = User::getSilentWithUID($x['uid']);
-                            $profile = $user->profile();
-                            WatchProfileUpdate::register($profile, 'broken');
-                        }
-                        fputcsv($csv, array($x['fullname'], $x['promo'], $alias,
-                                            join(',', $mails), $x['nb_mails'],
-                                            'https://www.polytechnique.org/marketing/broken/' . $alias,
-                                            $x['corps'], $x['job'], $x['networking']), ';');
+                fputcsv($csv, array('nom', 'promo', 'bounces', 'nbmails', 'url', 'corps', 'job', 'networking'), ';');
+                $corpsList = DirEnum::getOptions(DirEnum::CURRENTCORPS);
+                foreach (array_keys($sortable_array) as $pid) {
+                    $mails = $broken_user_list[$pid];
+                    $profile = $broken_user_profiles[$pid];
+                    $current_corps = $profile->getCorpsName();
+                    $jobs = $profile->getJobs();
+                    $companies = array();
+                    foreach ($jobs as $job) {
+                        $companies[] = $job->company->name;
+                    }
+                    $networkings = $profile->getNetworking(Profile::NETWORKING_ALL);
+                    $networking_list = array();
+                    foreach ($networkings as $networking) {
+                        $networking_list[] = $networking['address'];
                     }
+                    fputcsv($csv, array($profile->fullName(), $profile->promo(),
+                                        join(',', $mails), $broken_user_email_count[$pid],
+                                        'https://www.polytechnique.org/marketing/broken/' . $profile->hrid(),
+                                        $current_corps, implode(',', $companies), implode(',', $networking_list)), ';');
                 }
                 fclose($csv);
                 exit;
@@ -958,5 +949,5 @@ class EmailModule extends PLModule
     }
 }
 
-// vim:set et sw=4 sts=4 sws=4 foldmethod=marker enc=utf-8:
+// vim:set et sw=4 sts=4 sws=4 foldmethod=marker fenc=utf-8:
 ?>