View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0016869||phplist application||Message Send Process||public||06-09-13 22:34||09-09-13 13:53|
|Target Version||3.0.X||Fixed in Version||3.0.4|
|Summary||0016869: Multiple records written to the event log|
|Description||When a message fails when sending to an email address three records are written to the event log each with the same information.|
in class phplistmailer the method compatSent()writes a record to the event log,
then when it returns to function sendemail() in sendemaillib.php that function then calls output() which also writes to the event log, and then explicitly writes to the event log.
In that order these are the records in the event log:
Error sending email to Error sending email to firstname.lastname@example.org
Error sending message 5 (6/19) to email@example.com (firstname.lastname@example.org)
Error sending message 5 to email@example.com (firstname.lastname@example.org)
One of these is sufficient.
I suggest removing the event logging from compatSent(), as the calling function can itself decide what to do. Then either call output() with the second parameter set to false so that it does not log, or remove the explicit call of logEvent().
|Tags||No tags attached.|