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/22 19:32:49 UTC
svn commit: r777630 -
/incubator/log4php/trunk/src/main/php/LoggerHierarchy.php
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);