[ 
https://issues.apache.org/jira/browse/LOG4PHP-110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13033683#comment-13033683
 ] 

Ivan Habunek commented on LOG4PHP-110:
--------------------------------------

Hi Vladimir,

I have decided to drop the LoggerLayoutBson class for now, since it does not 
fit in with the rest of the layouts which return strings. Other appenders would 
fail if they were used with this layout.

In it's place, I have added the LoggerLoggingEventBsonifier helper class, from 
your github repository, and updated the appender to use this helper instead of 
the layout class. I made minor changes to the helper - replaced the ArrayObject 
with standard array since I don't see why an ArrayObject was needed here.

Maybe we can let the users create their own "bsonifier" objects and control the 
data layout that way? What do you think?

Ivan




> mongo database appender for log4php
> -----------------------------------
>
>                 Key: LOG4PHP-110
>                 URL: https://issues.apache.org/jira/browse/LOG4PHP-110
>             Project: Log4php
>          Issue Type: New Feature
>          Components: Code, Documentation, Tests
>    Affects Versions: 2.0
>         Environment: PHP
>            Reporter: Vladimir Gorej
>            Assignee: Christian Grobmeier
>             Fix For: 2.1
>
>         Attachments: src.tgz, throwable.path
>
>
> mongo database appender for log4php with unit test coverage

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to