Description

Implement the factory method for the class Zend_Log_Writer_Mail in order to configure all writers of Zend_Log with a Zend_Config.

Comments

Posted by Renan de Lima (renanbr) on 2010-06-19T18:17:54.000+0000

my personal implementation patch. Benoit, i didn't stopped writing tests, if you want you can assing to me.

Posted by Renan de Lima (renanbr) on 2010-08-10T18:31:42.000+0000

attached my local changes that implements this issue and its tests, all green ;-) i will wait until next week before i assign this issue to me and commit to trunk

Posted by Benoît Durand (intiilapa) on 2010-08-12T12:27:27.000+0000

It's a generic approach to allow every setters ^^
It lacks a configuration setting for the method setSubjectPreprendText() for me. I think this feature should allow multiple recipients. This should be possible by providing an array for the header To, and the fact that we can define an array in an application configuration.

I am surprised that the unit tests contain any assertion.

Posted by Renan de Lima (renanbr) on 2010-08-12T12:48:23.000+0000

yes Benoit, it's a generic setter

about writer configuration you are right, current solution allow us to configure mail object, but it doesn't allow to configure log writer object, it needs to be fixed

about multiple recipients i think you are right again, it should allow set multiple recipients

forget the current solution, let's try solve this starting from usage, could this piece of application.ini file bellow a nice way to represent your wishes?