LOG4PHP-174:
* Using integer level constants for switching instead of strings.
* Removed call to console->debug() because the method does not exist.
* Removed call to console->trace() because it is not used for trace-level logging, but for outputting the call trace.
git-svn-id: https://svn.apache.org/repos/asf/logging/log4php/trunk@1306892 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/main/php/appenders/LoggerAppenderFirephp.php b/src/main/php/appenders/LoggerAppenderFirephp.php
index 89e9b8a..8527d87 100644
--- a/src/main/php/appenders/LoggerAppenderFirephp.php
+++ b/src/main/php/appenders/LoggerAppenderFirephp.php
@@ -75,28 +75,24 @@
public function append(LoggerLoggingEvent $event) {
$msg = $this->getLayout()->format($event);
- switch ($this->getLogLevel($event)) {
- case 'debug':
- $this->console->trace($msg); //includes backtrace
- break;
- case 'warn':
- $this->console->debug($msg);
- break;
- case 'error':
- $this->console->warn($msg);
- break;
- case 'fatal':
- $this->console->error($msg);
- break;
- default:
- $this->console->info($msg);
+ switch ($event->getLevel()->toInt()) {
+ case LoggerLevel::TRACE:
+ case LoggerLevel::DEBUG:
+ $this->console->log($msg);
+ break;
+ case LoggerLevel::INFO:
+ $this->console->info($msg);
+ break;
+ case LoggerLevel::WARN:
+ $this->console->warn($msg);
+ break;
+ case LoggerLevel::ERROR:
+ case LoggerLevel::FATAL:
+ $this->console->error($msg);
+ break;
}
}
- private function getLogLevel(LoggerLoggingEvent $event) {
- return strtolower($event->getLevel()->toString());
- }
-
/** Returns the target. */
public function getTarget() {
return $this->target;
