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


Reply via email to