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/23 15:30:21 UTC

svn commit: r777917 - /incubator/log4php/trunk/src/main/php/LoggerHierarchy.php

Author: grobmeier
Date: Sat May 23 13:30:21 2009
New Revision: 777917

URL: http://svn.apache.org/viewvc?rev=777917&view=rev
Log:
put constructor to public - no need for forcing singleton atm

Modified:
    incubator/log4php/trunk/src/main/php/LoggerHierarchy.php

Modified: incubator/log4php/trunk/src/main/php/LoggerHierarchy.php
URL: http://svn.apache.org/viewvc/incubator/log4php/trunk/src/main/php/LoggerHierarchy.php?rev=777917&r1=777916&r2=777917&view=diff
==============================================================================
--- incubator/log4php/trunk/src/main/php/LoggerHierarchy.php (original)
+++ incubator/log4php/trunk/src/main/php/LoggerHierarchy.php Sat May 23 13:30:21 2009
@@ -83,7 +83,7 @@
 	 * Create a new logger hierarchy.
 	 * @param object $root the root logger
 	 */
-	protected function __construct(LoggerRoot $root) {
+	public function __construct(LoggerRoot $root) {
 		$this->root = $root;
 		$this->root->setHierarchy($this);
 		$this->setThreshold(LoggerLevel::getLevelAll());
@@ -152,11 +152,16 @@
 			$this->ht[$name]->setHierarchy($this);
 			$nodes = explode('.', $name);
 			$firstNode = array_shift($nodes);
+			
+			// if name is not a first node but another first node is their
 			if($firstNode != $name and isset($this->ht[$firstNode])) {
 				$this->ht[$name]->setParent($this->ht[$firstNode]);
 			} else {
+				// if there is no father, set root logger as father
 				$this->ht[$name]->setParent($this->root);
 			} 
+			
+			// if there are more nodes than one
 			if(count($nodes) > 0) {
 				// find parent node
 				foreach($nodes as $node) {