LoggerLayoutPattern fails tests
-------------------------------

                 Key: LOG4PHP-102
                 URL: https://issues.apache.org/jira/browse/LOG4PHP-102
             Project: Log4php
          Issue Type: Bug
          Components: Code
    Affects Versions: 2.0
            Reporter: Ivan Habunek
            Priority: Minor
         Attachments: LoggerLayoutPatternTest.patch.txt, 
LoggerPatternConverter.patch.quickfix.txt, 
LoggerPatternConverter.patch.rewrite.txt

In LoggerLayoutPatternTest class, the testWarnLayout() method is commented out 
because it fails with the following message:

1) testWarnLayout(LoggerLayoutPatternTest)
Undefined index:  log4php.LoggerPatternConverter.spaces
/Users/cgrobmeier/Documents/Development/workspace/log4php-trunk/src/main/php/helpers/LoggerPatternConverter.php:131
/Users/cgrobmeier/Documents/Development/workspace/log4php-trunk/src/main/php/helpers/LoggerPatternConverter.php:104
/Users/cgrobmeier/Documents/Development/workspace/log4php-trunk/src/main/php/layouts/LoggerPatternLayout.php:216
/Users/cgrobmeier/Documents/Development/workspace/log4php-trunk/src/test/php/layouts/LoggerLayoutPatternTest.php:45

This seems to be related to using a global variable 
$GLOBALS['log4php.LoggerPatternConverter.spaces'] in 
LoggerPatternConverter::spacePad(). I am unsure why the global variable is not 
accessible in this test, but using a global variable seems the wrong approach 
here anyway.

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


Reply via email to