From 2c93e70e6c9b13cb4e83ea9c739675ae234e6360 Mon Sep 17 00:00:00 2001 From: Florent Bruneau Date: Tue, 30 Dec 2008 18:25:49 +0100 Subject: [PATCH] Port emails/broken. Move the text of the mail in its own template. Signed-off-by: Florent Bruneau --- core | 2 +- modules/email.php | 75 +++++++++++++----------------------- templates/emails/broken-web.mail.tpl | 47 ++++++++++++++++++++++ templates/emails/broken.tpl | 2 +- 4 files changed, 75 insertions(+), 51 deletions(-) create mode 100644 templates/emails/broken-web.mail.tpl diff --git a/core b/core index 697c955..933e1c2 160000 --- a/core +++ b/core @@ -1 +1 @@ -Subproject commit 697c955db0a0cb1c68826e033966dcb13fea4977 +Subproject commit 933e1c23f280a829faf7c112164341f47957c0fb diff --git a/modules/email.php b/modules/email.php index 6dbd1ef..5ab18c5 100644 --- a/modules/email.php +++ b/modules/email.php @@ -621,40 +621,20 @@ class EmailModule extends PLModule $page->changeTpl('emails/broken.tpl'); if ($warn == 'warn' && $email) { - S::assert_xsrf_token(); + //S::assert_xsrf_token(); $email = valide_email($email); // vérifications d'usage - $sel = XDB::query("SELECT uid FROM emails WHERE email = {?}", $email); - if (($uid = $sel->fetchOneCell())) { - $dest = User::getSilent($uid); - - // envoi du mail - $message = "Bonjour ! - -Cet email a été généré automatiquement par le service de patte cassée de -Polytechnique.org car un autre utilisateur, " . S::user()->fullName() . ", -nous a signalé qu'en t'envoyant un email, il avait reçu un message d'erreur -indiquant que ton adresse de redirection $email -ne fonctionnait plus ! - -Nous te suggérons de vérifier cette adresse, et le cas échéant de mettre -à jour sur le site <{$globals->baseurl}/emails> tes adresses -de redirection... - -Pour plus de renseignements sur le service de patte cassée, n'hésite pas à -consulter la page <{$globals->baseurl}/emails/broken>. - - -À bientôt sur Polytechnique.org ! -L'équipe d'administration mail->domain . '>'; - - $mail = new PlMailer(); - $mail->setFrom('"Polytechnique.org" mail->domain . '>'); - $mail->addTo($dest->bestEmail()); - $mail->setSubject("Une de tes adresse de redirection Polytechnique.org ne marche plus !!"); - $mail->setTxtBody($message); - $mail->send(); + $uid = XDB::fetchOneCell("SELECT uid + FROM emails + WHERE email = {?}", $email); + if ($uid) { + $dest = User::getWithUID($uid); + + $mail = new PlMailer('emails/broken-web.mail.tpl'); + $mail->assign('email', $email); + $mail->assign('request', S::user()); + $mail->sendTo($dest); $page->trigSuccess("Email envoyé !"); } } elseif (Post::has('email')) { @@ -668,30 +648,27 @@ L'équipe d'administration mail->domain . '>'; $page->assign('neuneu', true); } else { $page->assign('email',$email); - $sel = XDB::query( - "SELECT e1.uid, e1.panne != 0 AS panne, - (count(e2.uid) + IF(FIND_IN_SET('googleapps', u.mail_storage), 1, 0)) AS nb_mails, - u.nom, u.prenom, u.promo, u.hruid - FROM emails as e1 - LEFT JOIN emails as e2 ON(e1.uid = e2.uid + $x = XDB::fetchOneAssoc("SELECT e1.uid, e1.panne != 0 AS panne, + (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) + LEFT JOIN emails as e2 ON(e1.uid = e2.uid AND FIND_IN_SET('active', e2.flags) AND e1.email != e2.email) - INNER JOIN auth_user_md5 as u ON(e1.uid = u.user_id) - WHERE e1.email = {?} - GROUP BY e1.uid", $email); - if ($x = $sel->fetchOneAssoc()) { + 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); + 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); + 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); } } diff --git a/templates/emails/broken-web.mail.tpl b/templates/emails/broken-web.mail.tpl new file mode 100644 index 0000000..ceab44c --- /dev/null +++ b/templates/emails/broken-web.mail.tpl @@ -0,0 +1,47 @@ +{**************************************************************************} +{* *} +{* Copyright (C) 2003-2008 Polytechnique.org *} +{* http://opensource.polytechnique.org/ *} +{* *} +{* This program is free software; you can redistribute it and/or modify *} +{* it under the terms of the GNU General Public License as published by *} +{* the Free Software Foundation; either version 2 of the License, or *} +{* (at your option) any later version. *} +{* *} +{* This program is distributed in the hope that it will be useful, *} +{* but WITHOUT ANY WARRANTY; without even the implied warranty of *} +{* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *} +{* GNU General Public License for more details. *} +{* *} +{* You should have received a copy of the GNU General Public License *} +{* along with this program; if not, write to the Free Software *} +{* Foundation, Inc., *} +{* 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA *} +{* *} +{**************************************************************************} + +{config_load file="mails.conf" section="emails_broken"} +{if $mail_part eq 'head'} +{from full=#from#} +{subject text=#subject#} +{elseif $mail_part eq 'wiki'} +Bonjour ! + +Cet email a été généré automatiquement par le service de patte cassée de +Polytechnique.org car un autre utilisateur, {$request->fullName()}, +nous a signalé qu'en t'envoyant un email, il avait reçu un message d'erreur +indiquant que ton adresse de redirection {$email} +ne fonctionnait plus ! + +Nous te suggérons de vérifier cette adresse, et le cas échéant de mettre +à jour tes adresses de redirection [[{$globals->baseurl}/emails|sur le site]]. + +Pour plus de renseignements sur le service de patte cassée, n'hésite pas à +consulter [[{$globals->baseurl}/emails/broken|la documentation sur le site]]. + + +À bientôt sur Polytechnique.org !\\ +[[support@{$globals->mail->domain}|L'équipe d'administration]] +{/if} + +{* vim:set et sw=2 sts=2 sws=2 enc=utf-8: *} diff --git a/templates/emails/broken.tpl b/templates/emails/broken.tpl index c017119..dfd7b6e 100644 --- a/templates/emails/broken.tpl +++ b/templates/emails/broken.tpl @@ -49,7 +49,7 @@ correspondant si tu veux que nous puissions te répondre. {elseif $x}

Patte Cassée

- Désolé, mais ton correspondant, {$x.prenom} {$x.nom} (X{$x.promo}), + Désolé, mais ton correspondant, {$x.user->fullName()}, n'a actuellement aucune adresse email de redirection active autre que celle que tu viens de rentrer. Nous t'invitons à prendre contact avec lui autrement que par email, -- 2.1.4