Actions
Bug #9185
closedMailreport - Cant sending repports if multiple emails addresses in smtpnotifyemailaddress
Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Mail report
Target version:
-
Start date:
12/09/2018
Due date:
% Done:
0%
Estimated time:
Plus Target Version:
Affected Version:
2.4.x
Affected Plus Version:
Affected Architecture:
All
Description
In the field smtpnotifyemailaddress on the system_advanced_notifications.php we can put pultiple mail addresses with a coma separator.
The system handle them correctly.
But the mailreport packages does not, because in mail_reports.inc it only consider the field smtpnotifyemailaddress contains only one address.
Can I suggest a correction in the source code by pushing request on github ?
Here is the modification i suggest :
$addresses = explode(",", $config['notifications']['smtp']['notifyemailaddress']);
foreach ($addresses as $address){
$recipient=explode("@", $address);
$recipient=str_replace(array("_",".")," ",$recipient0);
$mail->AddAddress($address, ucwords(strtolower($recipient));
}
Actions