diff options
author | Kenneth Katzgrau <kkatzgrau@mailservice.hangoutapp.com> | 2010-12-07 16:43:43 +0000 |
---|---|---|
committer | Kenneth Katzgrau <kkatzgrau@mailservice.hangoutapp.com> | 2010-12-07 16:43:43 +0000 |
commit | c00a85677152ebd0c8e1f3bdd33cccbea463e37c (patch) | |
tree | fb154a2a145468d810646869e5b44ca49bb31a6b | |
parent | cb8b85365f39d58dd4179ec0ff3aed2b25202677 (diff) | |
download | KLogger-c00a85677152ebd0c8e1f3bdd33cccbea463e37c.zip KLogger-c00a85677152ebd0c8e1f3bdd33cccbea463e37c.tar.gz KLogger-c00a85677152ebd0c8e1f3bdd33cccbea463e37c.tar.bz2 |
Fixed a major issue with the checking of the severity level. Special thanks to Tim Kinnane for pointing it out
-rw-r--r-- | README.markdown | 3 | ||||
-rwxr-xr-x | example/example.php | 15 | ||||
-rwxr-xr-x | src/KLogger.php | 8 |
3 files changed, 21 insertions, 5 deletions
diff --git a/README.markdown b/README.markdown index 1eac80d..41b6c84 100644 --- a/README.markdown +++ b/README.markdown @@ -56,8 +56,9 @@ Additionally, it's been used in numerous projects, both commercial and personal. ## Special Thanks -Special thanks to all contributors, which right now includes one person: +Special thanks to all contributors, which right now includes two people: +[Tim Kinnane](http://twitter.com/etherealtim) [Brian Fenton](http://github.com/fentie) ## License diff --git a/example/example.php b/example/example.php new file mode 100755 index 0000000..efbc317 --- /dev/null +++ b/example/example.php @@ -0,0 +1,15 @@ +<?php + +# Should log to the same directory as this file +require dirname(__FILE__) . '/../src/KLogger.php'; + +$log = KLogger::instance(dirname(__FILE__), KLogger::DEBUG); + +$log->logInfo('Info Test'); +$log->logNotice('Notice Test'); +$log->logWarn('Warn Test'); +$log->logError('Error Test'); +$log->logFatal('Fatal Test'); +$log->logAlert('Alert Test'); +$log->logCrit('Crit test'); +$log->logEmerg('Emerg Test'); diff --git a/src/KLogger.php b/src/KLogger.php index 3760771..b50b970 100755 --- a/src/KLogger.php +++ b/src/KLogger.php @@ -321,7 +321,7 @@ class KLogger */
public function logCrit($line)
{
- $this->log($line, self::FATAL);
+ $this->log($line, self::CRIT);
}
/**
@@ -343,7 +343,7 @@ class KLogger */
public function log($line, $severity)
{
- if ($this->_severityThreshold <= $severity) {
+ if ($this->_severityThreshold >= $severity) {
$status = $this->_getTimeLine($severity);
$this->writeFreeFormLine("$status $line \n");
}
@@ -376,6 +376,8 @@ class KLogger return "$time - ALERT -->";
case self::CRIT:
return "$time - CRIT -->";
+ case self::FATAL: # FATAL is an alias of CRIT
+ return "$time - FATAL -->";
case self::NOTICE:
return "$time - NOTICE -->";
case self::INFO:
@@ -386,8 +388,6 @@ class KLogger return "$time - DEBUG -->";
case self::ERR:
return "$time - ERROR -->";
- case self::FATAL:
- return "$time - FATAL -->";
default:
return "$time - LOG -->";
}
|