You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4php-dev@logging.apache.org by "Hiroaki Kawai (JIRA)" <ji...@apache.org> on 2008/10/03 05:38:46 UTC

[jira] Updated: (LOG4PHP-20) Custom layout for LoggerAppenderPhp

     [ https://issues.apache.org/jira/browse/LOG4PHP-20?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Hiroaki Kawai updated LOG4PHP-20:
---------------------------------

    Attachment: LOG4PHP-20.patch

I'd like to suggest another way (LOG4PHP-20.patch). I think LoggerAppender->requiresLayout should only be used to check it requires layout and not to check whether it accepts the layout.

> Custom layout for LoggerAppenderPhp
> -----------------------------------
>
>                 Key: LOG4PHP-20
>                 URL: https://issues.apache.org/jira/browse/LOG4PHP-20
>             Project: Log4php
>          Issue Type: Bug
>         Environment: PHP 5.2.6 (cli) (built: Sep 12 2008 10:59:28) 
> Mac OS X 10.4.11
>            Reporter: Corin Lawson
>            Priority: Trivial
>         Attachments: LOG4PHP-20.patch
>
>   Original Estimate: 0.17h
>  Remaining Estimate: 0.17h
>
> PROBLEM
> Using the following configuration file, all messages in php log remain as TTCC format.
> log4php.appender.default=LoggerAppenderPhp
> log4php.appender.default.layout=LoggerPatternLayout
> log4php.appender.default.layout.conversionPattern="%-5p [%t] %c: %m%n"
> log4php.rootLogger=DEBUG, default
> SOLUTION
> In LoggerAppenderPhp class:
> First I removed the setting of the layout in activateOptions. Second I added the requiresLayout (defaulting to true) instance member.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.