Long (and unnecessary?) description of behaviour... short solution. Patch says all.
Description:
If debugging is turned on, loads of cruft is printed to (in my case) the browser... and... $mail->Send() still returns TRUE.
That TRUE value should be passed on to the caller, because now you get two watchdog entries (at least sometimes, depending on the state of your :
1) "PHPMailer debug message: 1 - " **
And sometimes
2) 'Failed queue processing for @message' (as per messaging_store.class.inc: queue_process_step() )
The 2nd message is confusing, because queue processing does not fail.
Fix: see patch.
-------
** This from messaging_phpmailer.module line 261.
Note: $mail->ErrorInfo is empty in my case, and $result doesn't contain anything useful either, so the watchdog message doesn't say much for me, because all the verbose info was logged to my browser.
But I'm not bothered with that, here. (Maybe it differs per PHP setup / PHPMailer version?)
I'd just like the 2nd error message to go away.
| Comment | File | Size | Author |
|---|---|---|---|
| messaging-phpmailer-debug.patch | 1.02 KB | roderik |
Comments
Comment #1
jose reyero commentedCommitted, thanks.