minor corrections

Hi,

I was testing your dSendMail2 class, starting with the MassMail example, and found a couple of things need to be changed in order for it to work correctly:

example-MassMailer.php, lines 30-31:
var $logFolder = './log/';
var $logFile = "send-".date('Y-m-d H-i-s').".dat";

I had to change these lines to:
$m->logFolder = './log'; // note: removed trailing slash
$m->logFile = "send-".date('Y-m-d H-i-s').".dat";

dSendMail2.inc.php, line 537:
if(!$this->logFile)

I had to remove the "!" (negation) for this to work; otherwise the logfile is never opened!

I also added to the end of the fopen() call: 'or die("Cannot open log file.");'

Thanks for providing the class! I'll let you know if I find anything else that needs fixing. :-)

-MH

OK, so it's two years later, and I just realized that line 537 of dSendMail2.inc.php was correct in its original form.

The problem of the logfile not being opened occurred because I based my "mass mailer" code on the example code in example-MassMailer.php, which attempts to set the name of the logfile with the following code:

var $logFile   = "send-".date('Y-m-d H-i-s').".dat";

Here, $logFile is being assigned a string value, but dSendMail2.inc.php expects $logFile to be the handle of an open file.

Since $logFile wasn't empty, dSendMail2.inc.php didn't open a new logfile, but when it tried to write to the file, it couldn't.

So I commented out the above line and restored line 537 to:
if(!$this->logFile)

which works better because it doesn't create a new logfile every time the _log() function is called. :-)