Author: ihabunek
Date: Sun Oct 23 07:56:21 2011
New Revision: 1187849
URL: http://svn.apache.org/viewvc?rev=1187849&view=rev
Log:
Minor cleanup of LoggerAppender. Changed warn() method to include appedner
class in warning.
Modified:
logging/log4php/trunk/src/main/php/LoggerAppender.php
Modified: logging/log4php/trunk/src/main/php/LoggerAppender.php
URL:
http://svn.apache.org/viewvc/logging/log4php/trunk/src/main/php/LoggerAppender.php?rev=1187849&r1=1187848&r2=1187849&view=diff
==============================================================================
--- logging/log4php/trunk/src/main/php/LoggerAppender.php (original)
+++ logging/log4php/trunk/src/main/php/LoggerAppender.php Sun Oct 23 07:56:21
2011
@@ -37,14 +37,14 @@ abstract class LoggerAppender {
* The first filter in the filter chain.
* @var LoggerFilter
*/
- protected $filter = null;
+ protected $filter;
/**
* The appender's layout. Can be null if the appender does not use
* a layout.
* @var LoggerLayout
*/
- protected $layout = null;
+ protected $layout;
/**
* Appender name. Used by other components to identify this appender.
@@ -57,7 +57,7 @@ abstract class LoggerAppender {
* will not be logged.
* @var LoggerLevel
*/
- protected $threshold = null;
+ protected $threshold;
/**
* Set to true if the appender requires a layout.
@@ -101,7 +101,6 @@ abstract class LoggerAppender {
*/
public function clearFilters() {
unset($this->filter);
- $this->filter = null;
}
/**
@@ -273,7 +272,8 @@ abstract class LoggerAppender {
/** Triggers a warning for this logger with the given message. */
protected function warn($message) {
- trigger_error("log4php: appender [{$this->name}]: $message",
E_USER_WARNING);
+ $id = get_class($this) . (empty($this->name) ? '' :
":{$this->name}");
+ trigger_error("log4php: [$id]: $message", E_USER_WARNING);
}
}