From 0677321259851e4b7d81d8fcfedd3512bbefba40 Mon Sep 17 00:00:00 2001 From: Sebastian Grewe Date: Mon, 7 Apr 2014 09:55:02 +0200 Subject: [PATCH] [CLEANUP] Use of isset instead of @ --- public/include/classes/mail.class.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/public/include/classes/mail.class.php b/public/include/classes/mail.class.php index 3c016e9c..3a26b089 100644 --- a/public/include/classes/mail.class.php +++ b/public/include/classes/mail.class.php @@ -90,7 +90,11 @@ class Mail extends Base { ->setSender($senderEmail) ->setReturnPath($senderEmail) ->setBody($this->smarty->fetch(BASEPATH . 'templates/mail/' . $template . '.tpl'), 'text/html'); - if (strlen(@$aData['senderName']) > 0 && strlen(@$aData['senderEmail']) > 0 && filter_var(@$aData['senderEmail'], FILTER_VALIDATE_EMAIL)) + if (isset($aData['senderName']) && + isset($aData['senderEmail']) && + strlen($aData['senderName']) > 0 && + strlen($aData['senderEmail']) > 0 && + filter_var($aData['senderEmail'], FILTER_VALIDATE_EMAIL)) $message->setReplyTo(array($aData['senderEmail'] => $aData['senderName'])); // Send message out with configured transport