From 4a71cf6706354c90259514915f71e01509fb2e8d Mon Sep 17 00:00:00 2001 From: =?utf8?q?St=C3=A9phane=20Jacob?= Date: Wed, 21 Jan 2009 13:41:05 +0100 Subject: [PATCH] Uses MailNotFound exception unknown axletters. --- modules/axletter.php | 24 ++++++++++++++---------- modules/axletter/axletter.inc.php | 3 +-- 2 files changed, 15 insertions(+), 12 deletions(-) diff --git a/modules/axletter.php b/modules/axletter.php index 96a5e38..805f238 100644 --- a/modules/axletter.php +++ b/modules/axletter.php @@ -268,16 +268,20 @@ class AXLetterModule extends PLModule $this->load('axletter.inc.php'); $page->changeTpl('axletter/show.tpl'); - $nl = new AXLetter($nid); - if (Get::has('text')) { - $nl->toText($page, S::v('prenom'), S::v('nom'), S::v('femme')); - } else { - $nl->toHtml($page, S::v('prenom'), S::v('nom'), S::v('femme')); - } - if (Post::has('send')) { - $nl->sendTo(S::user()->login(), S::user()->bestEmail(), - S::v('prenom'), S::v('nom'), - S::v('femme'), S::v('mail_fmt') != 'texte'); + try { + $nl = new AXLetter($nid); + if (Get::has('text')) { + $nl->toText($page, S::v('prenom'), S::v('nom'), S::v('femme')); + } else { + $nl->toHtml($page, S::v('prenom'), S::v('nom'), S::v('femme')); + } + if (Post::has('send')) { + $nl->sendTo(S::user()->login(), S::user()->bestEmail(), + S::v('prenom'), S::v('nom'), + S::v('femme'), S::v('mail_fmt') != 'texte'); + } + } catch (MailNotFound $e) { + return PL_NOT_FOUND; } } diff --git a/modules/axletter/axletter.inc.php b/modules/axletter/axletter.inc.php index f8d843f..385365e 100644 --- a/modules/axletter/axletter.inc.php +++ b/modules/axletter/axletter.inc.php @@ -48,8 +48,7 @@ class AXLetter extends MassMailer WHERE id = {?} OR short_name = {?}", $id, $id); } if (!$res->numRows()) { - $this->_id = null; - return; + throw new MailNotFound(); } $id = $res->fetchOneRow(); } -- 2.1.4