[jira] [Resolved] (LOG4PHP-158) LoggerAppenderFile throws wrong warning messages

["Ivan Habunek (Resolved) (JIRA)" <ji...@apache.org>]

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

Ivan Habunek resolved LOG4PHP-158.
----------------------------------

    Resolution: Fixed

Added a check to make sure filename is set. If not, warn and close the appender.
                
> LoggerAppenderFile throws wrong warning messages
> ------------------------------------------------
>
>                 Key: LOG4PHP-158
>                 URL: https://issues.apache.org/jira/browse/LOG4PHP-158
>             Project: Log4php
>          Issue Type: Bug
>          Components: Code
>    Affects Versions: 2.2
>            Reporter: Ivan Habunek
>            Assignee: Ivan Habunek
>             Fix For: 2.2
>
>
> Florian reported this issue on the experimental branch. It is an appender issue; to be fixed once the work on configurator has been merged to trunk.
> On 10.10.2011. 8:19, Florian Semm wrote:
> > when I have a LoggerAppenderFile config without the option 'file', I got 
> > a warning:
> > 
> > Warning: mkdir() [<a href='function.mkdir'>function.mkdir</a>]: No such 
> > file or directory in 
> > C:\xampp\htdocs\Symfony-experimental\vendor\log4php\src\main\php\appenders\LoggerAppenderFile.php 
> > line 74
> > 
> > The appender execute mkdir(..) with $this->fileName = null/empty string.
> > 
> > I think there is a not null check missing in line LoggerAppenderFile:69
> > 
> > if ($fileName == null) throw Exception();
> >
> > i work with the experimental branch

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira