Author: grobmeier Date: Fri May 22 17:32:49 2009 New Revision: 777630 URL: http://svn.apache.org/viewvc?rev=777630&view=rev Log: getLoggerbyFactory should not be public - use public getLogger instead
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=777630&r1=777629&r2=777630&view=diff ============================================================================== --- incubator/log4php/trunk/src/main/php/LoggerHierarchy.php (original) +++ incubator/log4php/trunk/src/main/php/LoggerHierarchy.php Fri May 22 17:32:49 2009 @@ -76,9 +76,8 @@ */ protected function __construct(LoggerRoot $root) { $this->root = $root; - // Enable all level levels by default. - $this->setThreshold(LoggerLevel::getLevelAll()); $this->root->setHierarchy($this); + $this->setThreshold(LoggerLevel::getLevelAll()); $this->rendererMap = new LoggerRendererMap(); $this->defaultFactory = new LoggerDefaultCategoryFactory(); } @@ -138,7 +137,7 @@ * @return Logger * @todo merge with {...@link getLogger()} */ - public function getLoggerByFactory($name, $factory) { + private function getLoggerByFactory($name, $factory) { if(!isset($this->ht[$name])) { $this->ht[$name] = $factory->makeNewLoggerInstance($name); $this->ht[$name]->setHierarchy($this);