[jira] [Resolved] (LOG4PHP-160) Appeneders should use a default layout is no layout is specified in configuration

2011-12-05 Thread Ivan Habunek (Resolved) (JIRA)
. > Appeneders should use a default layout is no layout is specified in > configuration > - > > Key: LOG4PHP-160 > URL: https://issues.apache.org/jira/

[jira] [Commented] (LOG4PHP-160) Appeneders should use a default layout is no layout is specified in configuration

2011-12-04 Thread Florian Semm (Commented) (JIRA)
[ https://issues.apache.org/jira/browse/LOG4PHP-160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13162426#comment-13162426 ] Florian Semm commented on LOG4PHP-160: -- well i don't know which default-

[jira] [Commented] (LOG4PHP-160) Appeneders should use a default layout is no layout is specified in configuration

2011-12-04 Thread Ivan Habunek (Commented) (JIRA)
tly looks good", but lost "patch" along the way. :) > Appeneders should use a default layout is no layout is specified in > configuration > - > > Key: L

[jira] [Commented] (LOG4PHP-160) Appeneders should use a default layout is no layout is specified in configuration

2011-12-04 Thread Ivan Habunek (Commented) (JIRA)
e as default layout. For example, the socket appender which should default to either XML or serialized layout (once LOG4PHP-154 is finished). I'm not sure why you removed the check "if empty($class)" from LoggerConfigurator and the tests. I think it's still useful. I have skip

[jira] [Commented] (LOG4PHP-160) Appeneders should use a default layout is no layout is specified in configuration

2011-11-29 Thread Ivan Habunek (Commented) (JIRA)
apply it to trunk when I get a bit of free time. > Appeneders should use a default layout is no layout is specified in > configuration > - > > Key: LOG4PHP-160 >

[jira] [Updated] (LOG4PHP-160) Appeneders should use a default layout is no layout is specified in configuration

2011-11-28 Thread Florian Semm (Updated) (JIRA)
[ https://issues.apache.org/jira/browse/LOG4PHP-160?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Florian Semm updated LOG4PHP-160: - Attachment: defaultLayout.patch - a default-layout will be set in the constructor - add tests

Re: Default-Layout

2011-11-27 Thread Florian Semm
ayout() method in the LoggerConfigurator is a better place. You can call there a method like getDefaultLayout() or a static variable. I can see the problem. I wanted to keep the solution within the appender class because, if it's defined in the configurator, then the default layout will not be ap

Re: Default-Layout

2011-11-27 Thread Ivan Habunek
Configurator is a better place. You can call there a method like > getDefaultLayout() or a static variable. I can see the problem. I wanted to keep the solution within the appender class because, if it's defined in the configurator, then the default layout will not be applied when configur

Default-Layout

2011-11-26 Thread Florian Semm
Am 25.11.2011 22:20, schrieb Ivan Habunek: The idea I had was to implement it in the activateOptions() method of the base class LoggerAppender. If no layout is present then set a default layout. You just have to make sure the parent::activateOptions() is called in Appenders which override this

[jira] [Created] (LOG4PHP-160) Appeneders should use a default layout is no layout is specified in configuration

2011-10-19 Thread Ivan Habunek (Created) (JIRA)
Appeneders should use a default layout is no layout is specified in configuration - Key: LOG4PHP-160 URL: https://issues.apache.org/jira/browse/LOG4PHP-160 Project