logging-log4php-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Florian Semm (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LOG4PHP-162) Warning for invalid appender threshold level never called
Date Tue, 15 Nov 2011 07:39:52 GMT

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

Florian Semm commented on LOG4PHP-162:
--------------------------------------

i think all warnings which related to invalid-levels could be removed.

for instance the root-logger config :

{code}
if (isset($config['level'])) {
			$level = LoggerLevel::toLevel($config['level']);
			if (isset($level)) {
				$logger->setLevel($level);
			} else {
				$default = $logger->getLevel();
				$this->warn("Invalid logger level [{$config['level']}] specified for logger [$loggerName].");
			}
		}
{code}

$level is always set.
                
> Warning for invalid appender threshold level never called
> ---------------------------------------------------------
>
>                 Key: LOG4PHP-162
>                 URL: https://issues.apache.org/jira/browse/LOG4PHP-162
>             Project: Log4php
>          Issue Type: Improvement
>          Components: Code
>    Affects Versions: 2.1
>            Reporter: Florian Semm
>            Priority: Minor
>             Fix For: 2.2
>
>
> appender config:
> <appender name="foo" class="LoggerAppenderConsole" threshold="FOO" />
> Expected warning: "Invalid threshold value [FOO]..."
> LoggerLevel::toLevel() return always level-debug if default-level is null. If the threshold
unknow the default-level (debug in this case) will be returned. There is no need for this
warning in line 266-267 LoggerConfigurator.

--
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

        

Mime
View raw message