Skip to content

Commit 0677321

Browse files
committed
[CLEANUP] Use of isset instead of @
1 parent 166c44f commit 0677321

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

public/include/classes/mail.class.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,11 @@ public function sendMail($template, $aData, $throttle=false) {
9090
->setSender($senderEmail)
9191
->setReturnPath($senderEmail)
9292
->setBody($this->smarty->fetch(BASEPATH . 'templates/mail/' . $template . '.tpl'), 'text/html');
93-
if (strlen(@$aData['senderName']) > 0 && strlen(@$aData['senderEmail']) > 0 && filter_var(@$aData['senderEmail'], FILTER_VALIDATE_EMAIL))
93+
if (isset($aData['senderName']) &&
94+
isset($aData['senderEmail']) &&
95+
strlen($aData['senderName']) > 0 &&
96+
strlen($aData['senderEmail']) > 0 &&
97+
filter_var($aData['senderEmail'], FILTER_VALIDATE_EMAIL))
9498
$message->setReplyTo(array($aData['senderEmail'] => $aData['senderName']));
9599

96100
// Send message out with configured transport

0 commit comments

Comments
 (0)