Closed (fixed)
Project:
Messaging
Version:
5.x-1.0-beta1
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Apr 2008 at 22:25 UTC
Updated:
15 Apr 2009 at 12:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
m3avrck commentedhere's a proposed patch which adds a debug option to turn this on/off...
Comment #2
m3avrck commentedComment #3
jose reyero commentedThe patch looks good.
However there's a messaging_debug module already which provides a debug option so I don't think we need that one duplicated. But I see the current 'debug' concept (not sending out messages at all) doesn't match this one (extra logging).
So what I think we should do here is extending the debug options in messaging_debug to:
0. None
1. Extended logging
2. Do not send out messages (just log)
With a better wording maybe. Does this make sense?
Comment #4
m3avrck commentedJose, that makes sense for sure.
PHPMailer is slightly screwy in the fact that you have to explicitly set an option in the mailing class to capture debug info and even then mixed results. I'm not super attached to my patch but want to show more as a proof of concept.
I agree a better and *single* debug option would be best. Will mull over some ideas, hmm...
Comment #5
jose reyero commentedAdded the option to the module, see the new settings form