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);