Uploaded image for project: 'Log4php'
  1. Log4php
  2. LOG4PHP-153

Merge LoggerAppenderEmail and LoggerAppenderEmailEvent into a single appender

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • 2.1.0
    • None
    • Code

    Description

      These two appenders are very similar. It would make sense to merge the functionality into a single logger which would have an option to buffer the output.

      For example, a "bufferSize" option can be added. If bufferSize is set to 10, then an email will be sent for every 10 logged messages. If bufferSize is set to 1, the appender will send an email for each logged message (like LoggerAppenderMailEvent) and if the bufferSize is set to NULL, then buffering would be unlimited (like LoggerAppenderMail).

      I propose to merge the whole funtionality into LoggerAppenderMail and make LoggerAppenderMailEvent deprecated.

      Attachments

        1. log4php.patch
          6 kB
          Michal Vanek
        2. phpunit-testresults.xml
          57 kB
          Michal Vanek

        Activity

          People

            juice Ivan Habunek
            juice Ivan Habunek
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: