You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4php-dev@logging.apache.org by gr...@apache.org on 2009/05/05 06:33:00 UTC

svn commit: r771557 - /incubator/log4php/trunk/src/main/php/appenders/LoggerAppenderEcho.php

Author: grobmeier
Date: Tue May  5 04:32:59 2009
New Revision: 771557

URL: http://svn.apache.org/viewvc?rev=771557&view=rev
Log:
#LOG4PHP-23: syntax clean up

Modified:
    incubator/log4php/trunk/src/main/php/appenders/LoggerAppenderEcho.php

Modified: incubator/log4php/trunk/src/main/php/appenders/LoggerAppenderEcho.php
URL: http://svn.apache.org/viewvc/incubator/log4php/trunk/src/main/php/appenders/LoggerAppenderEcho.php?rev=771557&r1=771556&r2=771557&view=diff
==============================================================================
--- incubator/log4php/trunk/src/main/php/appenders/LoggerAppenderEcho.php (original)
+++ incubator/log4php/trunk/src/main/php/appenders/LoggerAppenderEcho.php Tue May  5 04:32:59 2009
@@ -30,31 +30,27 @@
  * @subpackage appenders
  */
 class LoggerAppenderEcho extends LoggerAppender {
-
-	/**
-	 * @access private 
-	 */
-	var $requiresLayout = true;
-
-	/**
-	 * @var boolean used internally to mark first append 
-	 * @access private 
-	 */
-	var $firstAppend	= true;
+	/** boolean used internally to mark first append */
+	private $firstAppend = true;
+	
+	public function __construct($name) {
+	    parent::__construct($name);
+	    $this->requiresLayout = true;
+	    $this->firstAppend = true;
+	}
 	
-	function activateOptions() {
+	public function activateOptions() {
 		$this->closed = false;
-		return;
 	}
 	
-	function close() {
+	public function close() {
 		if(!$this->firstAppend) {
 			echo $this->layout->getFooter();
 		}
 		$this->closed = true;	 
 	}
 
-	function append($event) {
+	public function append($event) {
 		if($this->layout !== null) {
 			if($this->firstAppend) {
 				echo $this->layout->getHeader();