fjbender / magento-1

lib/Payone/Log4php/LoggerFilterLevelMatch.php

Doc Comments -1 removed lines

@@ -67,7 +67,6 @@
 	}
 	
 	/**
-	 * @param string $l the level to match
 	 */
 	public function setLevelToMatch($level) {
 		$this->setLevel('levelToMatch', $level);

Indentation +45 added lines, -45 removed lines

@@ -47,54 +47,54 @@
  */
 class Payone_Log4php_LoggerFilterLevelMatch extends Payone_Log4php_LoggerFilter {
   
-	/** 
-	 * Indicates if this event should be accepted or denied on match
-	 * @var boolean
-	 */
-	protected $acceptOnMatch = true;
+    /** 
+     * Indicates if this event should be accepted or denied on match
+     * @var boolean
+     */
+    protected $acceptOnMatch = true;
 
-	/**
-	 * The level, when to match
-	 * @var Payone_Log4php_LoggerLevel
-	 */
-	protected $levelToMatch;
+    /**
+     * The level, when to match
+     * @var Payone_Log4php_LoggerLevel
+     */
+    protected $levelToMatch;
   
-	/**
-	 * @param boolean $acceptOnMatch
-	 */
-	public function setAcceptOnMatch($acceptOnMatch) {
-		$this->setBoolean('acceptOnMatch', $acceptOnMatch);
-	}
+    /**
+     * @param boolean $acceptOnMatch
+     */
+    public function setAcceptOnMatch($acceptOnMatch) {
+        $this->setBoolean('acceptOnMatch', $acceptOnMatch);
+    }
 	
-	/**
-	 * @param string $l the level to match
-	 */
-	public function setLevelToMatch($level) {
-		$this->setLevel('levelToMatch', $level);
-	}
+    /**
+     * @param string $l the level to match
+     */
+    public function setLevelToMatch($level) {
+        $this->setLevel('levelToMatch', $level);
+    }
 
-	/**
-	 * Return the decision of this filter.
-	 * 
-	 * Returns {@link LoggerFilter::NEUTRAL} if the <b><var>LevelToMatch</var></b>
-	 * option is not set or if there is not match.	Otherwise, if there is a
-	 * match, then the returned decision is {@link LoggerFilter::ACCEPT} if the
-	 * <b><var>AcceptOnMatch</var></b> property is set to <i>true</i>. The
-	 * returned decision is {@link LoggerFilter::DENY} if the
-	 * <b><var>AcceptOnMatch</var></b> property is set to <i>false</i>.
-	 *
-	 * @param Payone_Log4php_LoggerLoggingEvent $event
-	 * @return integer
-	 */
-	public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->levelToMatch === null) {
-			return Payone_Log4php_LoggerFilter::NEUTRAL;
-		}
+    /**
+     * Return the decision of this filter.
+     * 
+     * Returns {@link LoggerFilter::NEUTRAL} if the <b><var>LevelToMatch</var></b>
+     * option is not set or if there is not match.	Otherwise, if there is a
+     * match, then the returned decision is {@link LoggerFilter::ACCEPT} if the
+     * <b><var>AcceptOnMatch</var></b> property is set to <i>true</i>. The
+     * returned decision is {@link LoggerFilter::DENY} if the
+     * <b><var>AcceptOnMatch</var></b> property is set to <i>false</i>.
+     *
+     * @param Payone_Log4php_LoggerLoggingEvent $event
+     * @return integer
+     */
+    public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
+        if($this->levelToMatch === null) {
+            return Payone_Log4php_LoggerFilter::NEUTRAL;
+        }
 		
-		if($this->levelToMatch->equals($event->getLevel())) {	
-			return $this->acceptOnMatch ? Payone_Log4php_LoggerFilter::ACCEPT : Payone_Log4php_LoggerFilter::DENY;
-		} else {
-			return Payone_Log4php_LoggerFilter::NEUTRAL;
-		}
-	}
+        if($this->levelToMatch->equals($event->getLevel())) {	
+            return $this->acceptOnMatch ? Payone_Log4php_LoggerFilter::ACCEPT : Payone_Log4php_LoggerFilter::DENY;
+        } else {
+            return Payone_Log4php_LoggerFilter::NEUTRAL;
+        }
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -87,11 +87,11 @@
 	 * @return integer
 	 */
 	public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->levelToMatch === null) {
+		if ($this->levelToMatch === null) {
 			return Payone_Log4php_LoggerFilter::NEUTRAL;
 		}
 		
-		if($this->levelToMatch->equals($event->getLevel())) {	
+		if ($this->levelToMatch->equals($event->getLevel())) {	
 			return $this->acceptOnMatch ? Payone_Log4php_LoggerFilter::ACCEPT : Payone_Log4php_LoggerFilter::DENY;
 		} else {
 			return Payone_Log4php_LoggerFilter::NEUTRAL;

lib/Payone/Log4php/LoggerFilterLevelRange.php

Doc Comments -2 removed lines

@@ -87,14 +87,12 @@
 	}
 	
 	/**
-	 * @param string $l the level min to match
 	 */
 	public function setLevelMin($level) {
 		$this->setLevel('levelMin', $level);
 	}
 
 	/**
-	 * @param string $l the level max to match
 	 */
 	public function setLevelMax($level) {
 		$this->setLevel('levelMax', $level);

Indentation +62 added lines, -62 removed lines

@@ -64,75 +64,75 @@
  */
 class Payone_Log4php_LoggerFilterLevelRange extends Payone_Log4php_LoggerFilter {
 
-	/**
-	 * @var boolean
-	 */
-	protected $acceptOnMatch = true;
+    /**
+     * @var boolean
+     */
+    protected $acceptOnMatch = true;
 
-	/**
-	 * @var Payone_Log4php_LoggerLevel
-	 */
-	protected $levelMin;
+    /**
+     * @var Payone_Log4php_LoggerLevel
+     */
+    protected $levelMin;
   
-	/**
-	 * @var Payone_Log4php_LoggerLevel
-	 */
-	protected $levelMax;
+    /**
+     * @var Payone_Log4php_LoggerLevel
+     */
+    protected $levelMax;
 
-	/**
-	 * @param boolean $acceptOnMatch
-	 */
-	public function setAcceptOnMatch($acceptOnMatch) {
-		$this->setBoolean('acceptOnMatch', $acceptOnMatch); 
-	}
+    /**
+     * @param boolean $acceptOnMatch
+     */
+    public function setAcceptOnMatch($acceptOnMatch) {
+        $this->setBoolean('acceptOnMatch', $acceptOnMatch); 
+    }
 	
-	/**
-	 * @param string $l the level min to match
-	 */
-	public function setLevelMin($level) {
-		$this->setLevel('levelMin', $level);
-	}
+    /**
+     * @param string $l the level min to match
+     */
+    public function setLevelMin($level) {
+        $this->setLevel('levelMin', $level);
+    }
 
-	/**
-	 * @param string $l the level max to match
-	 */
-	public function setLevelMax($level) {
-		$this->setLevel('levelMax', $level);
-	}
+    /**
+     * @param string $l the level max to match
+     */
+    public function setLevelMax($level) {
+        $this->setLevel('levelMax', $level);
+    }
 
-	/**
-	 * Return the decision of this filter.
-	 *
-	 * @param Payone_Log4php_LoggerLoggingEvent $event
-	 * @return integer
-	 */
-	public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
-		$level = $event->getLevel();
+    /**
+     * Return the decision of this filter.
+     *
+     * @param Payone_Log4php_LoggerLoggingEvent $event
+     * @return integer
+     */
+    public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
+        $level = $event->getLevel();
 		
-		if($this->levelMin !== null) {
-			if($level->isGreaterOrEqual($this->levelMin) == false) {
-				// level of event is less than minimum
-				return Payone_Log4php_LoggerFilter::DENY;
-			}
-		}
+        if($this->levelMin !== null) {
+            if($level->isGreaterOrEqual($this->levelMin) == false) {
+                // level of event is less than minimum
+                return Payone_Log4php_LoggerFilter::DENY;
+            }
+        }
 
-		if($this->levelMax !== null) {
-			if($level->toInt() > $this->levelMax->toInt()) {
-				// level of event is greater than maximum
-				// Alas, there is no Level.isGreater method. and using
-				// a combo of isGreaterOrEqual && !Equal seems worse than
-				// checking the int values of the level objects..
-				return Payone_Log4php_LoggerFilter::DENY;
-			}
-		}
+        if($this->levelMax !== null) {
+            if($level->toInt() > $this->levelMax->toInt()) {
+                // level of event is greater than maximum
+                // Alas, there is no Level.isGreater method. and using
+                // a combo of isGreaterOrEqual && !Equal seems worse than
+                // checking the int values of the level objects..
+                return Payone_Log4php_LoggerFilter::DENY;
+            }
+        }
 
-		if($this->acceptOnMatch) {
-			// this filter set up to bypass later filters and always return
-			// accept if level in range
-			return Payone_Log4php_LoggerFilter::ACCEPT;
-		} else {
-			// event is ok for this filter; allow later filters to have a look..
-			return Payone_Log4php_LoggerFilter::NEUTRAL;
-		}
-	}
+        if($this->acceptOnMatch) {
+            // this filter set up to bypass later filters and always return
+            // accept if level in range
+            return Payone_Log4php_LoggerFilter::ACCEPT;
+        } else {
+            // event is ok for this filter; allow later filters to have a look..
+            return Payone_Log4php_LoggerFilter::NEUTRAL;
+        }
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -109,15 +109,15 @@ 
 	public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
 		$level = $event->getLevel();
 		
-		if($this->levelMin !== null) {
-			if($level->isGreaterOrEqual($this->levelMin) == false) {
+		if ($this->levelMin !== null) {
+			if ($level->isGreaterOrEqual($this->levelMin) == false) {
 				// level of event is less than minimum
 				return Payone_Log4php_LoggerFilter::DENY;
 			}
 		}
 
-		if($this->levelMax !== null) {
-			if($level->toInt() > $this->levelMax->toInt()) {
+		if ($this->levelMax !== null) {
+			if ($level->toInt() > $this->levelMax->toInt()) {
 				// level of event is greater than maximum
 				// Alas, there is no Level.isGreater method. and using
 				// a combo of isGreaterOrEqual && !Equal seems worse than
@@ -126,7 +126,7 @@ 
 			}
 		}
 
-		if($this->acceptOnMatch) {
+		if ($this->acceptOnMatch) {
 			// this filter set up to bypass later filters and always return
 			// accept if level in range
 			return Payone_Log4php_LoggerFilter::ACCEPT;

lib/Payone/Log4php/LoggerFilterStringMatch.php

Doc Comments -1 removed lines

@@ -65,7 +65,6 @@
 	}
 	
 	/**
-	 * @param string $s the string to match
 	 */
 	public function setStringToMatch($string) {
 		$this->setString('stringToMatch', $string);

Indentation +33 added lines, -33 removed lines

@@ -47,43 +47,43 @@
  */
 class Payone_Log4php_LoggerFilterStringMatch extends Payone_Log4php_LoggerFilter {
 
-	/**
-	 * @var boolean
-	 */
-	protected $acceptOnMatch = true;
+    /**
+     * @var boolean
+     */
+    protected $acceptOnMatch = true;
 
-	/**
-	 * @var string
-	 */
-	protected $stringToMatch;
+    /**
+     * @var string
+     */
+    protected $stringToMatch;
 
-	/**
-	 * @param mixed $acceptOnMatch a boolean or a string ('true' or 'false')
-	 */
-	public function setAcceptOnMatch($acceptOnMatch) {
-		$this->setBoolean('acceptOnMatch', $acceptOnMatch);
-	}
+    /**
+     * @param mixed $acceptOnMatch a boolean or a string ('true' or 'false')
+     */
+    public function setAcceptOnMatch($acceptOnMatch) {
+        $this->setBoolean('acceptOnMatch', $acceptOnMatch);
+    }
 	
-	/**
-	 * @param string $s the string to match
-	 */
-	public function setStringToMatch($string) {
-		$this->setString('stringToMatch', $string);
-	}
+    /**
+     * @param string $s the string to match
+     */
+    public function setStringToMatch($string) {
+        $this->setString('stringToMatch', $string);
+    }
 
-	/**
-	 * @return integer a {@link LOGGER_FILTER_NEUTRAL} is there is no string match.
-	 */
-	public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
-		$msg = $event->getRenderedMessage();
+    /**
+     * @return integer a {@link LOGGER_FILTER_NEUTRAL} is there is no string match.
+     */
+    public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
+        $msg = $event->getRenderedMessage();
 		
-		if($msg === null or $this->stringToMatch === null) {
-			return Payone_Log4php_LoggerFilter::NEUTRAL;
-		}
+        if($msg === null or $this->stringToMatch === null) {
+            return Payone_Log4php_LoggerFilter::NEUTRAL;
+        }
 		
-		if(strpos($msg, $this->stringToMatch) !== false ) {
-			return ($this->acceptOnMatch) ? Payone_Log4php_LoggerFilter::ACCEPT : Payone_Log4php_LoggerFilter::DENY;
-		}
-		return Payone_Log4php_LoggerFilter::NEUTRAL;
-	}
+        if(strpos($msg, $this->stringToMatch) !== false ) {
+            return ($this->acceptOnMatch) ? Payone_Log4php_LoggerFilter::ACCEPT : Payone_Log4php_LoggerFilter::DENY;
+        }
+        return Payone_Log4php_LoggerFilter::NEUTRAL;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -77,11 +77,11 @@
 	public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
 		$msg = $event->getRenderedMessage();
 		
-		if($msg === null or $this->stringToMatch === null) {
+		if ($msg === null or $this->stringToMatch === null) {
 			return Payone_Log4php_LoggerFilter::NEUTRAL;
 		}
 		
-		if(strpos($msg, $this->stringToMatch) !== false ) {
+		if (strpos($msg, $this->stringToMatch) !== false) {
 			return ($this->acceptOnMatch) ? Payone_Log4php_LoggerFilter::ACCEPT : Payone_Log4php_LoggerFilter::DENY;
 		}
 		return Payone_Log4php_LoggerFilter::NEUTRAL;

lib/Payone/Log4php/LoggerHierarchy.php

Doc Comments -1 removed lines

@@ -213,7 +213,6 @@
 	
 	/**
 	 * Sets the main threshold level.
-	 * @param Payone_Log4php_LoggerLevel $l
 	 */
 	public function setThreshold(Payone_Log4php_LoggerLevel $threshold) {
 		$this->threshold = $threshold;

Indentation +164 added lines, -164 removed lines

@@ -49,192 +49,192 @@
  */
 class Payone_Log4php_LoggerHierarchy {
 	
-	/** Array holding all Logger instances. */
-	protected $loggers = array();
+    /** Array holding all Logger instances. */
+    protected $loggers = array();
 	
-	/** 
-	 * The root logger.
-	 * @var RootLogger 
-	 */
-	protected $root = null;
+    /** 
+     * The root logger.
+     * @var RootLogger 
+     */
+    protected $root = null;
 	
-	/** 
-	 * The logger renderer map.
-	 * @var Payone_Log4php_LoggerRendererMap
-	 */
-	protected $rendererMap;
+    /** 
+     * The logger renderer map.
+     * @var Payone_Log4php_LoggerRendererMap
+     */
+    protected $rendererMap;
 
-	/** 
-	 * Main level threshold. Events with lower level will not be logged by any 
-	 * logger, regardless of it's configuration.
-	 * @var Payone_Log4php_LoggerLevel
-	 */
-	protected $threshold;
+    /** 
+     * Main level threshold. Events with lower level will not be logged by any 
+     * logger, regardless of it's configuration.
+     * @var Payone_Log4php_LoggerLevel
+     */
+    protected $threshold;
 	
-	/**
-	 * Creates a new logger hierarchy.
-	 * @param Payone_Log4php_LoggerRoot $root The root logger.
-	 */
-	public function __construct(Payone_Log4php_LoggerRoot $root) {
-		$this->root = $root;
-		$this->setThreshold(Payone_Log4php_LoggerLevel::getLevelAll());
-		$this->rendererMap = new Payone_Log4php_LoggerRendererMap();
-	}
+    /**
+     * Creates a new logger hierarchy.
+     * @param Payone_Log4php_LoggerRoot $root The root logger.
+     */
+    public function __construct(Payone_Log4php_LoggerRoot $root) {
+        $this->root = $root;
+        $this->setThreshold(Payone_Log4php_LoggerLevel::getLevelAll());
+        $this->rendererMap = new Payone_Log4php_LoggerRendererMap();
+    }
 	 
-	/**
-	 * Clears all loggers.
-	 */
-	public function clear() {
-		$this->loggers = array();
-	}
+    /**
+     * Clears all loggers.
+     */
+    public function clear() {
+        $this->loggers = array();
+    }
 	
-	/**
-	 * Check if the named logger exists in the hierarchy.
-	 * @param string $name
-	 * @return boolean
-	 */
-	public function exists($name) {
-		return isset($this->loggers[$name]);
-	}
+    /**
+     * Check if the named logger exists in the hierarchy.
+     * @param string $name
+     * @return boolean
+     */
+    public function exists($name) {
+        return isset($this->loggers[$name]);
+    }
 
-	/**
-	 * Returns all the currently defined loggers in this hierarchy as an array.
-	 * @return array
-	 */	 
-	public function getCurrentLoggers() {
-		return array_values($this->loggers);
-	}
+    /**
+     * Returns all the currently defined loggers in this hierarchy as an array.
+     * @return array
+     */	 
+    public function getCurrentLoggers() {
+        return array_values($this->loggers);
+    }
 	
-	/**
-	 * Returns a named logger instance logger. If it doesn't exist, one is created.
-	 * 
-	 * @param string $name Logger name
-	 * @return Logger Logger instance.
-	 */
-	public function getLogger($name) {
-		if(!isset($this->loggers[$name])) {
-			$logger = new Payone_Log4php_Logger($name);
+    /**
+     * Returns a named logger instance logger. If it doesn't exist, one is created.
+     * 
+     * @param string $name Logger name
+     * @return Logger Logger instance.
+     */
+    public function getLogger($name) {
+        if(!isset($this->loggers[$name])) {
+            $logger = new Payone_Log4php_Logger($name);
 
-			$nodes = explode('.', $name);
-			$firstNode = array_shift($nodes);
+            $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->loggers[$firstNode])) {
-				$logger->setParent($this->loggers[$firstNode]);
-			} else {
-				// if there is no father, set root logger as father
-				$logger->setParent($this->root);
-			} 
+            // if name is not a first node but another first node is their
+            if($firstNode != $name and isset($this->loggers[$firstNode])) {
+                $logger->setParent($this->loggers[$firstNode]);
+            } else {
+                // if there is no father, set root logger as father
+                $logger->setParent($this->root);
+            } 
 		
-			// if there are more nodes than one
-			if(count($nodes) > 0) {
-				// find parent node
-				foreach($nodes as $node) {
-					$parentNode = "$firstNode.$node";
-					if(isset($this->loggers[$parentNode]) and $parentNode != $name) {
-						$logger->setParent($this->loggers[$parentNode]);
-					}
-					$firstNode .= ".$node";
-				}
-			}
+            // if there are more nodes than one
+            if(count($nodes) > 0) {
+                // find parent node
+                foreach($nodes as $node) {
+                    $parentNode = "$firstNode.$node";
+                    if(isset($this->loggers[$parentNode]) and $parentNode != $name) {
+                        $logger->setParent($this->loggers[$parentNode]);
+                    }
+                    $firstNode .= ".$node";
+                }
+            }
 			
-			$this->loggers[$name] = $logger;
-		}		
+            $this->loggers[$name] = $logger;
+        }		
 		
-		return $this->loggers[$name];
-	} 
+        return $this->loggers[$name];
+    } 
 	
-	/**
-	 * Returns the logger renderer map.
-	 * @return Payone_Log4php_LoggerRendererMap
-	 */
-	public function getRendererMap() {
-		return $this->rendererMap;
-	}
+    /**
+     * Returns the logger renderer map.
+     * @return Payone_Log4php_LoggerRendererMap
+     */
+    public function getRendererMap() {
+        return $this->rendererMap;
+    }
 	
-	/**
-	 * Returns the root logger.
-	 * @return Payone_Log4php_LoggerRoot
-	 */ 
-	public function getRootLogger() {
-		if(!isset($this->root) or $this->root == null) {
-			$this->root = new Payone_Log4php_LoggerRoot();
-		}
-		return $this->root;
-	}
+    /**
+     * Returns the root logger.
+     * @return Payone_Log4php_LoggerRoot
+     */ 
+    public function getRootLogger() {
+        if(!isset($this->root) or $this->root == null) {
+            $this->root = new Payone_Log4php_LoggerRoot();
+        }
+        return $this->root;
+    }
 	 
-	/**
-	 * Returns the main threshold level.
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public function getThreshold() {
-		return $this->threshold;
-	} 
+    /**
+     * Returns the main threshold level.
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public function getThreshold() {
+        return $this->threshold;
+    } 
 
-	/**
-	 * Returns true if the hierarchy is disabled for given log level and false
-	 * otherwise.
-	 * @return boolean
-	 */
-	public function isDisabled(Payone_Log4php_LoggerLevel $level) {
-		return ($this->threshold->toInt() > $level->toInt());
-	}
+    /**
+     * Returns true if the hierarchy is disabled for given log level and false
+     * otherwise.
+     * @return boolean
+     */
+    public function isDisabled(Payone_Log4php_LoggerLevel $level) {
+        return ($this->threshold->toInt() > $level->toInt());
+    }
 	
-	/**
-	 * Reset all values contained in this hierarchy instance to their
-	 * default. 
-	 *
-	 * This removes all appenders from all loggers, sets
-	 * the level of all non-root loggers to <i>null</i>,
-	 * sets their additivity flag to <i>true</i> and sets the level
-	 * of the root logger to {@link LOGGER_LEVEL_DEBUG}.
-	 * 
-	 * <p>Existing loggers are not removed. They are just reset.
-	 *
-	 * <p>This method should be used sparingly and with care as it will
-	 * block all logging until it is completed.</p>
-	 */
-	public function resetConfiguration() {
-		$root = $this->getRootLogger();
+    /**
+     * Reset all values contained in this hierarchy instance to their
+     * default. 
+     *
+     * This removes all appenders from all loggers, sets
+     * the level of all non-root loggers to <i>null</i>,
+     * sets their additivity flag to <i>true</i> and sets the level
+     * of the root logger to {@link LOGGER_LEVEL_DEBUG}.
+     * 
+     * <p>Existing loggers are not removed. They are just reset.
+     *
+     * <p>This method should be used sparingly and with care as it will
+     * block all logging until it is completed.</p>
+     */
+    public function resetConfiguration() {
+        $root = $this->getRootLogger();
 		
-		$root->setLevel(Payone_Log4php_LoggerLevel::getLevelDebug());
-		$this->setThreshold(Payone_Log4php_LoggerLevel::getLevelAll());
-		$this->shutDown();
+        $root->setLevel(Payone_Log4php_LoggerLevel::getLevelDebug());
+        $this->setThreshold(Payone_Log4php_LoggerLevel::getLevelAll());
+        $this->shutDown();
 		
-		foreach($this->loggers as $logger) {
-			$logger->setLevel(null);
-			$logger->setAdditivity(true);
-			$logger->removeAllAppenders();
-		}
+        foreach($this->loggers as $logger) {
+            $logger->setLevel(null);
+            $logger->setAdditivity(true);
+            $logger->removeAllAppenders();
+        }
 		
-		$this->rendererMap->clear();
-		Payone_Log4php_LoggerAppenderPool::clear();
-	}
+        $this->rendererMap->clear();
+        Payone_Log4php_LoggerAppenderPool::clear();
+    }
 	
-	/**
-	 * Sets the main threshold level.
-	 * @param Payone_Log4php_LoggerLevel $l
-	 */
-	public function setThreshold(Payone_Log4php_LoggerLevel $threshold) {
-		$this->threshold = $threshold;
-	}
+    /**
+     * Sets the main threshold level.
+     * @param Payone_Log4php_LoggerLevel $l
+     */
+    public function setThreshold(Payone_Log4php_LoggerLevel $threshold) {
+        $this->threshold = $threshold;
+    }
 	
-	/**
-	 * Shutting down a hierarchy will <i>safely</i> close and remove
-	 * all appenders in all loggers including the root logger.
-	 * 
-	 * The shutdown method is careful to close nested
-	 * appenders before closing regular appenders. This is allows
-	 * configurations where a regular appender is attached to a logger
-	 * and again to a nested appender.
-	 * 
-	 * @todo Check if the last paragraph is correct.
-	 */
-	public function shutdown() {
-		$this->root->removeAllAppenders();
+    /**
+     * Shutting down a hierarchy will <i>safely</i> close and remove
+     * all appenders in all loggers including the root logger.
+     * 
+     * The shutdown method is careful to close nested
+     * appenders before closing regular appenders. This is allows
+     * configurations where a regular appender is attached to a logger
+     * and again to a nested appender.
+     * 
+     * @todo Check if the last paragraph is correct.
+     */
+    public function shutdown() {
+        $this->root->removeAllAppenders();
 		
-		foreach($this->loggers as $logger) {
-			$logger->removeAllAppenders();
-		}
-	}
+        foreach($this->loggers as $logger) {
+            $logger->removeAllAppenders();
+        }
+    }
 } 

Spacing +8 added lines, -8 removed lines

@@ -112,14 +112,14 @@ 
 	 * @return Logger Logger instance.
 	 */
 	public function getLogger($name) {
-		if(!isset($this->loggers[$name])) {
+		if (!isset($this->loggers[$name])) {
 			$logger = new Payone_Log4php_Logger($name);
 
 			$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->loggers[$firstNode])) {
+			if ($firstNode != $name and isset($this->loggers[$firstNode])) {
 				$logger->setParent($this->loggers[$firstNode]);
 			} else {
 				// if there is no father, set root logger as father
@@ -127,11 +127,11 @@ 
 			} 
 		
 			// if there are more nodes than one
-			if(count($nodes) > 0) {
+			if (count($nodes) > 0) {
 				// find parent node
-				foreach($nodes as $node) {
+				foreach ($nodes as $node) {
 					$parentNode = "$firstNode.$node";
-					if(isset($this->loggers[$parentNode]) and $parentNode != $name) {
+					if (isset($this->loggers[$parentNode]) and $parentNode != $name) {
 						$logger->setParent($this->loggers[$parentNode]);
 					}
 					$firstNode .= ".$node";
@@ -157,7 +157,7 @@ 
 	 * @return Payone_Log4php_LoggerRoot
 	 */ 
 	public function getRootLogger() {
-		if(!isset($this->root) or $this->root == null) {
+		if (!isset($this->root) or $this->root == null) {
 			$this->root = new Payone_Log4php_LoggerRoot();
 		}
 		return $this->root;
@@ -201,7 +201,7 @@ 
 		$this->setThreshold(Payone_Log4php_LoggerLevel::getLevelAll());
 		$this->shutDown();
 		
-		foreach($this->loggers as $logger) {
+		foreach ($this->loggers as $logger) {
 			$logger->setLevel(null);
 			$logger->setAdditivity(true);
 			$logger->removeAllAppenders();
@@ -233,7 +233,7 @@ 
 	public function shutdown() {
 		$this->root->removeAllAppenders();
 		
-		foreach($this->loggers as $logger) {
+		foreach ($this->loggers as $logger) {
 			$logger->removeAllAppenders();
 		}
 	}

lib/Payone/Log4php/LoggerLevel.php

Doc Comments +3 added lines, -3 removed lines

@@ -73,8 +73,8 @@ 
 	/**
 	 * Compares two logger levels.
 	 *
-	 * @param LoggerLevels $other
-	 * @return boolean 
+	 * @param Payone_Log4php_LoggerLevel $other
+	 * @return boolean|null 
 	 */
 	public function equals($other) {
 		if($other instanceof Payone_Log4php_LoggerLevel) {
@@ -223,7 +223,7 @@ 
 	 * this method returns the provided default level.
 	 *
 	 * @param mixed $arg The value to convert to level.
-	 * @param Payone_Log4php_LoggerLevel $default Value to return if conversion is not possible.
+	 * @param Payone_Log4php_LoggerLevel $defaultLevel Value to return if conversion is not possible.
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function toLevel($arg, $defaultLevel = null) {

Indentation +200 added lines, -200 removed lines

@@ -33,224 +33,224 @@
  */
 class Payone_Log4php_LoggerLevel {
 	
-	const OFF = 2147483647;
-	const FATAL = 50000;
-	const ERROR = 40000;
-	const WARN = 30000;
-	const INFO = 20000;
-	const DEBUG = 10000;
-	const TRACE = 5000;
-	const ALL = -2147483647;
+    const OFF = 2147483647;
+    const FATAL = 50000;
+    const ERROR = 40000;
+    const WARN = 30000;
+    const INFO = 20000;
+    const DEBUG = 10000;
+    const TRACE = 5000;
+    const ALL = -2147483647;
 
-	/** Integer level value. */
-	private $level;
+    /** Integer level value. */
+    private $level;
 	
-	/** Contains a list of instantiated levels. */
-	private static $levelMap;
+    /** Contains a list of instantiated levels. */
+    private static $levelMap;
 
-	/** String representation of the level. */
-	private $levelStr;
+    /** String representation of the level. */
+    private $levelStr;
 
-	/** 
-	 * Equivalent syslog level.
-	 * @var integer
-	 */
-	private $syslogEquivalent;
+    /** 
+     * Equivalent syslog level.
+     * @var integer
+     */
+    private $syslogEquivalent;
 
-	/**
-	 * Constructor
-	 *
-	 * @param integer $level
-	 * @param string $levelStr
-	 * @param integer $syslogEquivalent
-	 */
-	private function __construct($level, $levelStr, $syslogEquivalent) {
-		$this->level = $level;
-		$this->levelStr = $levelStr;
-		$this->syslogEquivalent = $syslogEquivalent;
-	}
+    /**
+     * Constructor
+     *
+     * @param integer $level
+     * @param string $levelStr
+     * @param integer $syslogEquivalent
+     */
+    private function __construct($level, $levelStr, $syslogEquivalent) {
+        $this->level = $level;
+        $this->levelStr = $levelStr;
+        $this->syslogEquivalent = $syslogEquivalent;
+    }
 
-	/**
-	 * Compares two logger levels.
-	 *
-	 * @param LoggerLevels $other
-	 * @return boolean 
-	 */
-	public function equals($other) {
-		if($other instanceof Payone_Log4php_LoggerLevel) {
-			if($this->level == $other->level) {
-				return true;
-			}
-		} else {
-			return false;
-		}
-	}
+    /**
+     * Compares two logger levels.
+     *
+     * @param LoggerLevels $other
+     * @return boolean 
+     */
+    public function equals($other) {
+        if($other instanceof Payone_Log4php_LoggerLevel) {
+            if($this->level == $other->level) {
+                return true;
+            }
+        } else {
+            return false;
+        }
+    }
 	
-	/**
-	 * Returns an Off Level
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function getLevelOff() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::OFF])) {
-			self::$levelMap[Payone_Log4php_LoggerLevel::OFF] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::OFF, 'OFF', LOG_ALERT);
-		}
-		return self::$levelMap[Payone_Log4php_LoggerLevel::OFF];
-	}
+    /**
+     * Returns an Off Level
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function getLevelOff() {
+        if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::OFF])) {
+            self::$levelMap[Payone_Log4php_LoggerLevel::OFF] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::OFF, 'OFF', LOG_ALERT);
+        }
+        return self::$levelMap[Payone_Log4php_LoggerLevel::OFF];
+    }
 
-	/**
-	 * Returns a Fatal Level
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function getLevelFatal() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::FATAL])) {
-			self::$levelMap[Payone_Log4php_LoggerLevel::FATAL] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::FATAL, 'FATAL', LOG_ALERT);
-		}
-		return self::$levelMap[Payone_Log4php_LoggerLevel::FATAL];
-	}
+    /**
+     * Returns a Fatal Level
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function getLevelFatal() {
+        if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::FATAL])) {
+            self::$levelMap[Payone_Log4php_LoggerLevel::FATAL] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::FATAL, 'FATAL', LOG_ALERT);
+        }
+        return self::$levelMap[Payone_Log4php_LoggerLevel::FATAL];
+    }
 	
-	/**
-	 * Returns an Error Level
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function getLevelError() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::ERROR])) {
-			self::$levelMap[Payone_Log4php_LoggerLevel::ERROR] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::ERROR, 'ERROR', LOG_ERR);
-		}
-		return self::$levelMap[Payone_Log4php_LoggerLevel::ERROR];
-	}
+    /**
+     * Returns an Error Level
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function getLevelError() {
+        if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::ERROR])) {
+            self::$levelMap[Payone_Log4php_LoggerLevel::ERROR] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::ERROR, 'ERROR', LOG_ERR);
+        }
+        return self::$levelMap[Payone_Log4php_LoggerLevel::ERROR];
+    }
 	
-	/**
-	 * Returns a Warn Level
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function getLevelWarn() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::WARN])) {
-			self::$levelMap[Payone_Log4php_LoggerLevel::WARN] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::WARN, 'WARN', LOG_WARNING);
-		}
-		return self::$levelMap[Payone_Log4php_LoggerLevel::WARN];
-	}
+    /**
+     * Returns a Warn Level
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function getLevelWarn() {
+        if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::WARN])) {
+            self::$levelMap[Payone_Log4php_LoggerLevel::WARN] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::WARN, 'WARN', LOG_WARNING);
+        }
+        return self::$levelMap[Payone_Log4php_LoggerLevel::WARN];
+    }
 
-	/**
-	 * Returns an Info Level
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function getLevelInfo() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::INFO])) {
-			self::$levelMap[Payone_Log4php_LoggerLevel::INFO] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::INFO, 'INFO', LOG_INFO);
-		}
-		return self::$levelMap[Payone_Log4php_LoggerLevel::INFO];
-	}
+    /**
+     * Returns an Info Level
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function getLevelInfo() {
+        if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::INFO])) {
+            self::$levelMap[Payone_Log4php_LoggerLevel::INFO] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::INFO, 'INFO', LOG_INFO);
+        }
+        return self::$levelMap[Payone_Log4php_LoggerLevel::INFO];
+    }
 
-	/**
-	 * Returns a Debug Level
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function getLevelDebug() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG])) {
-			self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::DEBUG, 'DEBUG', LOG_DEBUG);
-		}
-		return self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG];
-	}
+    /**
+     * Returns a Debug Level
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function getLevelDebug() {
+        if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG])) {
+            self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::DEBUG, 'DEBUG', LOG_DEBUG);
+        }
+        return self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG];
+    }
 	
-	/**
-	 * Returns a Trace Level
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function getLevelTrace() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::TRACE])) {
-			self::$levelMap[Payone_Log4php_LoggerLevel::TRACE] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::TRACE, 'TRACE', LOG_DEBUG);
-		}
-		return self::$levelMap[Payone_Log4php_LoggerLevel::TRACE];
-	}	
+    /**
+     * Returns a Trace Level
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function getLevelTrace() {
+        if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::TRACE])) {
+            self::$levelMap[Payone_Log4php_LoggerLevel::TRACE] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::TRACE, 'TRACE', LOG_DEBUG);
+        }
+        return self::$levelMap[Payone_Log4php_LoggerLevel::TRACE];
+    }	
 
-	/**
-	 * Returns an All Level
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function getLevelAll() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::ALL])) {
-			self::$levelMap[Payone_Log4php_LoggerLevel::ALL] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::ALL, 'ALL', LOG_DEBUG);
-		}
-		return self::$levelMap[Payone_Log4php_LoggerLevel::ALL];
-	}
+    /**
+     * Returns an All Level
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function getLevelAll() {
+        if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::ALL])) {
+            self::$levelMap[Payone_Log4php_LoggerLevel::ALL] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::ALL, 'ALL', LOG_DEBUG);
+        }
+        return self::$levelMap[Payone_Log4php_LoggerLevel::ALL];
+    }
 	
-	/**
-	 * Return the syslog equivalent of this priority as an integer.
-	 * @return integer
-	 */
-	public function getSyslogEquivalent() {
-		return $this->syslogEquivalent;
-	}
+    /**
+     * Return the syslog equivalent of this priority as an integer.
+     * @return integer
+     */
+    public function getSyslogEquivalent() {
+        return $this->syslogEquivalent;
+    }
 
-	/**
-	 * Returns <i>true</i> if this level has a higher or equal
-	 * level than the level passed as argument, <i>false</i>
-	 * otherwise.
-	 *
-	 * @param Payone_Log4php_LoggerLevel $other
-	 * @return boolean
-	 */
-	public function isGreaterOrEqual($other) {
-		return $this->level >= $other->level;
-	}
+    /**
+     * Returns <i>true</i> if this level has a higher or equal
+     * level than the level passed as argument, <i>false</i>
+     * otherwise.
+     *
+     * @param Payone_Log4php_LoggerLevel $other
+     * @return boolean
+     */
+    public function isGreaterOrEqual($other) {
+        return $this->level >= $other->level;
+    }
 
-	/**
-	 * Returns the string representation of this level.
-	 * @return string
-	 */
-	public function toString() {
-		return $this->levelStr;
-	}
+    /**
+     * Returns the string representation of this level.
+     * @return string
+     */
+    public function toString() {
+        return $this->levelStr;
+    }
 	
-	/**
-	 * Returns the string representation of this level.
-	 * @return string
-	 */
-	public function __toString() {
-		return $this->toString();
-	}
+    /**
+     * Returns the string representation of this level.
+     * @return string
+     */
+    public function __toString() {
+        return $this->toString();
+    }
 
-	/**
-	 * Returns the integer representation of this level.
-	 * @return integer
-	 */
-	public function toInt() {
-		return $this->level;
-	}
+    /**
+     * Returns the integer representation of this level.
+     * @return integer
+     */
+    public function toInt() {
+        return $this->level;
+    }
 
-	/**
-	 * Convert the input argument to a level. If the conversion fails, then 
-	 * this method returns the provided default level.
-	 *
-	 * @param mixed $arg The value to convert to level.
-	 * @param Payone_Log4php_LoggerLevel $default Value to return if conversion is not possible.
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public static function toLevel($arg, $defaultLevel = null) {
-		if(is_int($arg)) {
-			switch($arg) {
-				case self::ALL:	return self::getLevelAll();
-				case self::TRACE: return self::getLevelTrace();
-				case self::DEBUG: return self::getLevelDebug();
-				case self::INFO: return self::getLevelInfo();
-				case self::WARN: return self::getLevelWarn();
-				case self::ERROR: return self::getLevelError();
-				case self::FATAL: return self::getLevelFatal();
-				case self::OFF:	return self::getLevelOff();
-				default: return $defaultLevel;
-			}
-		} else {
-			switch(strtoupper($arg)) {
-				case 'ALL':	return self::getLevelAll();
-				case 'TRACE': return self::getLevelTrace();
-				case 'DEBUG': return self::getLevelDebug();
-				case 'INFO': return self::getLevelInfo();
-				case 'WARN': return self::getLevelWarn();
-				case 'ERROR': return self::getLevelError();
-				case 'FATAL': return self::getLevelFatal();
-				case 'OFF':	return self::getLevelOff();
-				default: return $defaultLevel;
-			}
-		}
-	}
+    /**
+     * Convert the input argument to a level. If the conversion fails, then 
+     * this method returns the provided default level.
+     *
+     * @param mixed $arg The value to convert to level.
+     * @param Payone_Log4php_LoggerLevel $default Value to return if conversion is not possible.
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public static function toLevel($arg, $defaultLevel = null) {
+        if(is_int($arg)) {
+            switch($arg) {
+                case self::ALL:	return self::getLevelAll();
+                case self::TRACE: return self::getLevelTrace();
+                case self::DEBUG: return self::getLevelDebug();
+                case self::INFO: return self::getLevelInfo();
+                case self::WARN: return self::getLevelWarn();
+                case self::ERROR: return self::getLevelError();
+                case self::FATAL: return self::getLevelFatal();
+                case self::OFF:	return self::getLevelOff();
+                default: return $defaultLevel;
+            }
+        } else {
+            switch(strtoupper($arg)) {
+                case 'ALL':	return self::getLevelAll();
+                case 'TRACE': return self::getLevelTrace();
+                case 'DEBUG': return self::getLevelDebug();
+                case 'INFO': return self::getLevelInfo();
+                case 'WARN': return self::getLevelWarn();
+                case 'ERROR': return self::getLevelError();
+                case 'FATAL': return self::getLevelFatal();
+                case 'OFF':	return self::getLevelOff();
+                default: return $defaultLevel;
+            }
+        }
+    }
 }

Spacing +13 added lines, -13 removed lines

@@ -77,8 +77,8 @@ 
 	 * @return boolean 
 	 */
 	public function equals($other) {
-		if($other instanceof Payone_Log4php_LoggerLevel) {
-			if($this->level == $other->level) {
+		if ($other instanceof Payone_Log4php_LoggerLevel) {
+			if ($this->level == $other->level) {
 				return true;
 			}
 		} else {
@@ -91,7 +91,7 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function getLevelOff() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::OFF])) {
+		if (!isset(self::$levelMap[Payone_Log4php_LoggerLevel::OFF])) {
 			self::$levelMap[Payone_Log4php_LoggerLevel::OFF] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::OFF, 'OFF', LOG_ALERT);
 		}
 		return self::$levelMap[Payone_Log4php_LoggerLevel::OFF];
@@ -102,7 +102,7 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function getLevelFatal() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::FATAL])) {
+		if (!isset(self::$levelMap[Payone_Log4php_LoggerLevel::FATAL])) {
 			self::$levelMap[Payone_Log4php_LoggerLevel::FATAL] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::FATAL, 'FATAL', LOG_ALERT);
 		}
 		return self::$levelMap[Payone_Log4php_LoggerLevel::FATAL];
@@ -113,7 +113,7 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function getLevelError() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::ERROR])) {
+		if (!isset(self::$levelMap[Payone_Log4php_LoggerLevel::ERROR])) {
 			self::$levelMap[Payone_Log4php_LoggerLevel::ERROR] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::ERROR, 'ERROR', LOG_ERR);
 		}
 		return self::$levelMap[Payone_Log4php_LoggerLevel::ERROR];
@@ -124,7 +124,7 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function getLevelWarn() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::WARN])) {
+		if (!isset(self::$levelMap[Payone_Log4php_LoggerLevel::WARN])) {
 			self::$levelMap[Payone_Log4php_LoggerLevel::WARN] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::WARN, 'WARN', LOG_WARNING);
 		}
 		return self::$levelMap[Payone_Log4php_LoggerLevel::WARN];
@@ -135,7 +135,7 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function getLevelInfo() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::INFO])) {
+		if (!isset(self::$levelMap[Payone_Log4php_LoggerLevel::INFO])) {
 			self::$levelMap[Payone_Log4php_LoggerLevel::INFO] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::INFO, 'INFO', LOG_INFO);
 		}
 		return self::$levelMap[Payone_Log4php_LoggerLevel::INFO];
@@ -146,7 +146,7 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function getLevelDebug() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG])) {
+		if (!isset(self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG])) {
 			self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::DEBUG, 'DEBUG', LOG_DEBUG);
 		}
 		return self::$levelMap[Payone_Log4php_LoggerLevel::DEBUG];
@@ -157,7 +157,7 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function getLevelTrace() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::TRACE])) {
+		if (!isset(self::$levelMap[Payone_Log4php_LoggerLevel::TRACE])) {
 			self::$levelMap[Payone_Log4php_LoggerLevel::TRACE] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::TRACE, 'TRACE', LOG_DEBUG);
 		}
 		return self::$levelMap[Payone_Log4php_LoggerLevel::TRACE];
@@ -168,7 +168,7 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function getLevelAll() {
-		if(!isset(self::$levelMap[Payone_Log4php_LoggerLevel::ALL])) {
+		if (!isset(self::$levelMap[Payone_Log4php_LoggerLevel::ALL])) {
 			self::$levelMap[Payone_Log4php_LoggerLevel::ALL] = new Payone_Log4php_LoggerLevel(Payone_Log4php_LoggerLevel::ALL, 'ALL', LOG_DEBUG);
 		}
 		return self::$levelMap[Payone_Log4php_LoggerLevel::ALL];
@@ -227,8 +227,8 @@ 
 	 * @return Payone_Log4php_LoggerLevel
 	 */
 	public static function toLevel($arg, $defaultLevel = null) {
-		if(is_int($arg)) {
-			switch($arg) {
+		if (is_int($arg)) {
+			switch ($arg) {
 				case self::ALL:	return self::getLevelAll();
 				case self::TRACE: return self::getLevelTrace();
 				case self::DEBUG: return self::getLevelDebug();
@@ -240,7 +240,7 @@ 
 				default: return $defaultLevel;
 			}
 		} else {
-			switch(strtoupper($arg)) {
+			switch (strtoupper($arg)) {
 				case 'ALL':	return self::getLevelAll();
 				case 'TRACE': return self::getLevelTrace();
 				case 'DEBUG': return self::getLevelDebug();

lib/Payone/Log4php/LoggerLocationInfo.php

Doc Comments +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 	 * Instantiate location information based on a {@link PHP_MANUAL#debug_backtrace}.
 	 *
 	 * @param array $trace
-	 * @param mixed $caller
+	 * @param string $fqcn
 	 */
 	public function __construct($trace, $fqcn = null) {
 		$this->lineNumber = isset($trace['line']) ? $trace['line'] : null;

Indentation +69 added lines, -69 removed lines

@@ -26,85 +26,85 @@
  * @since 0.3
  */
 class Payone_Log4php_LoggerLocationInfo {
-	/**
-	 * When location information is not available the constant
-	 * <i>NA</i> is returned. Current value of this string
-	 * constant is <b>?</b>.  
-	 */
-	const LOCATION_INFO_NA = 'NA';
+    /**
+     * When location information is not available the constant
+     * <i>NA</i> is returned. Current value of this string
+     * constant is <b>?</b>.  
+     */
+    const LOCATION_INFO_NA = 'NA';
 	
-	/**
-	* @var string Caller's line number.
-	*/
-	protected $lineNumber = null;
+    /**
+     * @var string Caller's line number.
+     */
+    protected $lineNumber = null;
 	
-	/**
-	* @var string Caller's file name.
-	*/
-	protected $fileName = null;
+    /**
+     * @var string Caller's file name.
+     */
+    protected $fileName = null;
 	
-	/**
-	* @var string Caller's fully qualified class name.
-	*/
-	protected $className = null;
+    /**
+     * @var string Caller's fully qualified class name.
+     */
+    protected $className = null;
 	
-	/**
-	* @var string Caller's method name.
-	*/
-	protected $methodName = null;
+    /**
+     * @var string Caller's method name.
+     */
+    protected $methodName = null;
 	
-	/**
-	* @var string 
-	*/
-	protected $fullInfo = null;
+    /**
+     * @var string 
+     */
+    protected $fullInfo = null;
 
-	/**
-	 * Instantiate location information based on a {@link PHP_MANUAL#debug_backtrace}.
-	 *
-	 * @param array $trace
-	 * @param mixed $caller
-	 */
-	public function __construct($trace, $fqcn = null) {
-		$this->lineNumber = isset($trace['line']) ? $trace['line'] : null;
-		$this->fileName = isset($trace['file']) ? $trace['file'] : null;
-		$this->className = isset($trace['class']) ? $trace['class'] : null;
-		$this->methodName = isset($trace['function']) ? $trace['function'] : null;
-		$this->fullInfo = $this->getClassName() . '.' . $this->getMethodName() . 
-			'(' . $this->getFileName() . ':' . $this->getLineNumber() . ')';
-	}
+    /**
+     * Instantiate location information based on a {@link PHP_MANUAL#debug_backtrace}.
+     *
+     * @param array $trace
+     * @param mixed $caller
+     */
+    public function __construct($trace, $fqcn = null) {
+        $this->lineNumber = isset($trace['line']) ? $trace['line'] : null;
+        $this->fileName = isset($trace['file']) ? $trace['file'] : null;
+        $this->className = isset($trace['class']) ? $trace['class'] : null;
+        $this->methodName = isset($trace['function']) ? $trace['function'] : null;
+        $this->fullInfo = $this->getClassName() . '.' . $this->getMethodName() . 
+            '(' . $this->getFileName() . ':' . $this->getLineNumber() . ')';
+    }
 
-	public function getClassName() {
-		return ($this->className === null) ? self::LOCATION_INFO_NA : $this->className; 
-	}
+    public function getClassName() {
+        return ($this->className === null) ? self::LOCATION_INFO_NA : $this->className; 
+    }
 
-	/**
-	 *	Return the file name of the caller.
-	 *	<p>This information is not always available.
-	 */
-	public function getFileName() {
-		return ($this->fileName === null) ? self::LOCATION_INFO_NA : $this->fileName; 
-	}
+    /**
+     *	Return the file name of the caller.
+     *	<p>This information is not always available.
+     */
+    public function getFileName() {
+        return ($this->fileName === null) ? self::LOCATION_INFO_NA : $this->fileName; 
+    }
 
-	/**
-	 *	Returns the line number of the caller.
-	 *	<p>This information is not always available.
-	 */
-	public function getLineNumber() {
-		return ($this->lineNumber === null) ? self::LOCATION_INFO_NA : $this->lineNumber; 
-	}
+    /**
+     *	Returns the line number of the caller.
+     *	<p>This information is not always available.
+     */
+    public function getLineNumber() {
+        return ($this->lineNumber === null) ? self::LOCATION_INFO_NA : $this->lineNumber; 
+    }
 
-	/**
-	 *	Returns the method name of the caller.
-	 */
-	public function getMethodName() {
-		return ($this->methodName === null) ? self::LOCATION_INFO_NA : $this->methodName; 
-	}
+    /**
+     *	Returns the method name of the caller.
+     */
+    public function getMethodName() {
+        return ($this->methodName === null) ? self::LOCATION_INFO_NA : $this->methodName; 
+    }
 
-	/**
-	 *	Returns the full information of the caller.
-	 */
-	public function getFullInfo() {
-		return ($this->fullInfo === null) ? self::LOCATION_INFO_NA : $this->fullInfo;
-	}
+    /**
+     *	Returns the full information of the caller.
+     */
+    public function getFullInfo() {
+        return ($this->fullInfo === null) ? self::LOCATION_INFO_NA : $this->fullInfo;
+    }
 
 }

lib/Payone/Log4php/LoggerLoggingEvent.php

Doc Comments +3 added lines, -3 removed lines

@@ -119,7 +119,7 @@ 
 	* LoggerLoggingEvent are filled when actually needed.
 	*
 	* @param string $fqcn name of the caller class.
-	* @param mixed $logger The {@link Logger} category of this event or the logger name.
+	* @param Payone_Log4php_Logger $logger The {@link Logger} category of this event or the logger name.
 	* @param Payone_Log4php_LoggerLevel $priority The level of this event.
 	* @param mixed $message The message of this event.
 	* @param integer $timeStamp the timestamp of this logging event.
@@ -314,7 +314,7 @@ 
 	
 	/**
 	 * Calculates the time of this event.
-	 * @return the time after event starttime when this event has occured
+	 * @return string time after event starttime when this event has occured
 	 */
 	public function getTime() {
 		$eventTime = $this->getTimeStamp();
@@ -333,7 +333,7 @@ 
 	}
 
 	/**
-	 * @return mixed LoggerThrowableInformation
+	 * @return Payone_Log4php_LoggerThrowableInformation LoggerThrowableInformation
 	 */
 	public function getThrowableInformation() {
 		return $this->throwableInfo;

Indentation +303 added lines, -303 removed lines

@@ -26,344 +26,344 @@
  */
 class Payone_Log4php_LoggerLoggingEvent {
 
-	private static $startTime;
+    private static $startTime;
 
-	/** 
-	* @var string Fully Qualified Class Name of the calling category class.
-	*/
-	private $fqcn;
+    /** 
+     * @var string Fully Qualified Class Name of the calling category class.
+     */
+    private $fqcn;
 	
-	/**
-	* @var Logger reference
-	*/
-	private $logger = null;
+    /**
+     * @var Logger reference
+     */
+    private $logger = null;
 	
-	/** 
-	 * The category (logger) name.
-	 * This field will be marked as private in future
-	 * releases. Please do not access it directly. 
-	 * Use the {@link getLoggerName()} method instead.
-	 * @deprecated 
-	 */
-	private $categoryName;
+    /** 
+     * The category (logger) name.
+     * This field will be marked as private in future
+     * releases. Please do not access it directly. 
+     * Use the {@link getLoggerName()} method instead.
+     * @deprecated 
+     */
+    private $categoryName;
 	
-	/** 
-	 * Level of the logging event.
-	 * @var Payone_Log4php_LoggerLevel
-	 */
-	protected $level;
+    /** 
+     * Level of the logging event.
+     * @var Payone_Log4php_LoggerLevel
+     */
+    protected $level;
 	
-	/** 
-	 * The nested diagnostic context (NDC) of logging event.
-	 * @var string  
-	 */
-	private $ndc;
+    /** 
+     * The nested diagnostic context (NDC) of logging event.
+     * @var string  
+     */
+    private $ndc;
 	
-	/** 
-	 * Have we tried to do an NDC lookup? If we did, there is no need
-	 * to do it again.	Note that its value is always false when
-	 * serialized. Thus, a receiving SocketNode will never use it's own
-	 * (incorrect) NDC. See also writeObject method.
-	 * @var boolean
-	 */
-	private $ndcLookupRequired = true;
+    /** 
+     * Have we tried to do an NDC lookup? If we did, there is no need
+     * to do it again.	Note that its value is always false when
+     * serialized. Thus, a receiving SocketNode will never use it's own
+     * (incorrect) NDC. See also writeObject method.
+     * @var boolean
+     */
+    private $ndcLookupRequired = true;
 	
-	/** 
-	 * Have we tried to do an MDC lookup? If we did, there is no need
-	 * to do it again.	Note that its value is always false when
-	 * serialized. See also the getMDC and getMDCCopy methods.
-	 * @var boolean	 
-	 */
-	private $mdcCopyLookupRequired = true;
+    /** 
+     * Have we tried to do an MDC lookup? If we did, there is no need
+     * to do it again.	Note that its value is always false when
+     * serialized. See also the getMDC and getMDCCopy methods.
+     * @var boolean	 
+     */
+    private $mdcCopyLookupRequired = true;
 	
-	/** 
-	 * @var mixed The application supplied message of logging event. 
-	 */
-	private $message;
+    /** 
+     * @var mixed The application supplied message of logging event. 
+     */
+    private $message;
 	
-	/** 
-	 * The application supplied message rendered through the log4php
-	 * objet rendering mechanism. At present renderedMessage == message.
-	 * @var string
-	 */
-	private $renderedMessage = null;
+    /** 
+     * The application supplied message rendered through the log4php
+     * objet rendering mechanism. At present renderedMessage == message.
+     * @var string
+     */
+    private $renderedMessage = null;
 	
-	/** 
-	 * The name of thread in which this logging event was generated.
-	 * log4php saves here the process id via {@link PHP_MANUAL#getmypid getmypid()} 
-	 * @var mixed
-	 */
-	private $threadName = null;
+    /** 
+     * The name of thread in which this logging event was generated.
+     * log4php saves here the process id via {@link PHP_MANUAL#getmypid getmypid()} 
+     * @var mixed
+     */
+    private $threadName = null;
 	
-	/** 
-	* The number of seconds elapsed from 1/1/1970 until logging event
-	* was created plus microseconds if available.
-	* @var float
-	*/
-	public $timeStamp;
+    /** 
+     * The number of seconds elapsed from 1/1/1970 until logging event
+     * was created plus microseconds if available.
+     * @var float
+     */
+    public $timeStamp;
 	
-	/** 
-	* @var Payone_Log4php_LoggerLocationInfo Location information for the caller.
-	*/
-	private $locationInfo = null;
+    /** 
+     * @var Payone_Log4php_LoggerLocationInfo Location information for the caller.
+     */
+    private $locationInfo = null;
 	
-	/**
-	 * @var Payone_Log4php_LoggerThrowableInformation log4php internal representation of throwable
-	 */
-	private $throwableInfo = null;
+    /**
+     * @var Payone_Log4php_LoggerThrowableInformation log4php internal representation of throwable
+     */
+    private $throwableInfo = null;
 	
-	/**
-	* Instantiate a LoggingEvent from the supplied parameters.
-	*
-	* Except {@link $timeStamp} all the other fields of
-	* LoggerLoggingEvent are filled when actually needed.
-	*
-	* @param string $fqcn name of the caller class.
-	* @param mixed $logger The {@link Logger} category of this event or the logger name.
-	* @param Payone_Log4php_LoggerLevel $priority The level of this event.
-	* @param mixed $message The message of this event.
-	* @param integer $timeStamp the timestamp of this logging event.
-	* @param Exception $throwable The throwable associated with logging event
-	*/
-	public function __construct($fqcn, $logger, $priority, $message, $timeStamp = null, Exception $throwable = null) {
-		$this->fqcn = $fqcn;
-		if($logger instanceof Payone_Log4php_Logger) {
-			$this->logger = $logger;
-			$this->categoryName = $logger->getName();
-		} else {
-			$this->categoryName = strval($logger);
-		}
-		$this->level = $priority;
-		$this->message = $message;
-		if($timeStamp !== null && is_float($timeStamp)) {
-			$this->timeStamp = $timeStamp;
-		} else {
-			$this->timeStamp = microtime(true);
-		}
+    /**
+     * Instantiate a LoggingEvent from the supplied parameters.
+     *
+     * Except {@link $timeStamp} all the other fields of
+     * LoggerLoggingEvent are filled when actually needed.
+     *
+     * @param string $fqcn name of the caller class.
+     * @param mixed $logger The {@link Logger} category of this event or the logger name.
+     * @param Payone_Log4php_LoggerLevel $priority The level of this event.
+     * @param mixed $message The message of this event.
+     * @param integer $timeStamp the timestamp of this logging event.
+     * @param Exception $throwable The throwable associated with logging event
+     */
+    public function __construct($fqcn, $logger, $priority, $message, $timeStamp = null, Exception $throwable = null) {
+        $this->fqcn = $fqcn;
+        if($logger instanceof Payone_Log4php_Logger) {
+            $this->logger = $logger;
+            $this->categoryName = $logger->getName();
+        } else {
+            $this->categoryName = strval($logger);
+        }
+        $this->level = $priority;
+        $this->message = $message;
+        if($timeStamp !== null && is_float($timeStamp)) {
+            $this->timeStamp = $timeStamp;
+        } else {
+            $this->timeStamp = microtime(true);
+        }
 		
-		if ($throwable !== null && $throwable instanceof Exception) {
-			$this->throwableInfo = new Payone_Log4php_LoggerThrowableInformation($throwable);
-		}
-	}
+        if ($throwable !== null && $throwable instanceof Exception) {
+            $this->throwableInfo = new Payone_Log4php_LoggerThrowableInformation($throwable);
+        }
+    }
 
-	/**
-	 * Returns the full qualified classname.
-	 * TODO: PHP does contain namespaces in 5.3. Those should be returned too, 
-	 */
-	 public function getFullQualifiedClassname() {
-		 return $this->fqcn;
-	 }
+    /**
+     * Returns the full qualified classname.
+     * TODO: PHP does contain namespaces in 5.3. Those should be returned too, 
+     */
+        public function getFullQualifiedClassname() {
+            return $this->fqcn;
+        }
 	 
-	/**
-	 * Set the location information for this logging event. The collected
-	 * information is cached for future use.
-	 *
-	 * <p>This method uses {@link PHP_MANUAL#debug_backtrace debug_backtrace()} function (if exists)
-	 * to collect informations about caller.</p>
-	 * <p>It only recognize informations generated by {@link Logger} and its subclasses.</p>
-	 * @return Payone_Log4php_LoggerLocationInfo
-	 */
-	public function getLocationInformation() {
-		if($this->locationInfo === null) {
+    /**
+     * Set the location information for this logging event. The collected
+     * information is cached for future use.
+     *
+     * <p>This method uses {@link PHP_MANUAL#debug_backtrace debug_backtrace()} function (if exists)
+     * to collect informations about caller.</p>
+     * <p>It only recognize informations generated by {@link Logger} and its subclasses.</p>
+     * @return Payone_Log4php_LoggerLocationInfo
+     */
+    public function getLocationInformation() {
+        if($this->locationInfo === null) {
 
-			$locationInfo = array();
-			$trace = debug_backtrace();
-			$prevHop = null;
-			// make a downsearch to identify the caller
-			$hop = array_pop($trace);
-			while($hop !== null) {
-				if(isset($hop['class'])) {
-					// we are sometimes in functions = no class available: avoid php warning here
-					$className = strtolower($hop['class']);
-					if(!empty($className) and ($className == 'logger' or 
-						strtolower(get_parent_class($className)) == 'logger')) {
-						$locationInfo['line'] = $hop['line'];
-						$locationInfo['file'] = $hop['file'];
-						break;
-					}
-				}
-				$prevHop = $hop;
-				$hop = array_pop($trace);
-			}
-			$locationInfo['class'] = isset($prevHop['class']) ? $prevHop['class'] : 'main';
-			if(isset($prevHop['function']) and
-				$prevHop['function'] !== 'include' and
-				$prevHop['function'] !== 'include_once' and
-				$prevHop['function'] !== 'require' and
-				$prevHop['function'] !== 'require_once') {
+            $locationInfo = array();
+            $trace = debug_backtrace();
+            $prevHop = null;
+            // make a downsearch to identify the caller
+            $hop = array_pop($trace);
+            while($hop !== null) {
+                if(isset($hop['class'])) {
+                    // we are sometimes in functions = no class available: avoid php warning here
+                    $className = strtolower($hop['class']);
+                    if(!empty($className) and ($className == 'logger' or 
+                        strtolower(get_parent_class($className)) == 'logger')) {
+                        $locationInfo['line'] = $hop['line'];
+                        $locationInfo['file'] = $hop['file'];
+                        break;
+                    }
+                }
+                $prevHop = $hop;
+                $hop = array_pop($trace);
+            }
+            $locationInfo['class'] = isset($prevHop['class']) ? $prevHop['class'] : 'main';
+            if(isset($prevHop['function']) and
+                $prevHop['function'] !== 'include' and
+                $prevHop['function'] !== 'include_once' and
+                $prevHop['function'] !== 'require' and
+                $prevHop['function'] !== 'require_once') {
 
-				$locationInfo['function'] = $prevHop['function'];
-			} else {
-				$locationInfo['function'] = 'main';
-			}
+                $locationInfo['function'] = $prevHop['function'];
+            } else {
+                $locationInfo['function'] = 'main';
+            }
 					 
-			$this->locationInfo = new Payone_Log4php_LoggerLocationInfo($locationInfo, $this->fqcn);
-		}
-		return $this->locationInfo;
-	}
+            $this->locationInfo = new Payone_Log4php_LoggerLocationInfo($locationInfo, $this->fqcn);
+        }
+        return $this->locationInfo;
+    }
 
-	/**
-	 * Return the level of this event. Use this form instead of directly
-	 * accessing the {@link $level} field.
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public function getLevel() {
-		return $this->level;
-	}
+    /**
+     * Return the level of this event. Use this form instead of directly
+     * accessing the {@link $level} field.
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public function getLevel() {
+        return $this->level;
+    }
 
-	/**
-	 * Returns the logger which created the event.
-	 * @return Logger
-	 */
-	public function getLogger() {
-		return $this->logger;
-	}
+    /**
+     * Returns the logger which created the event.
+     * @return Logger
+     */
+    public function getLogger() {
+        return $this->logger;
+    }
 	
-	/**
-	 * Return the name of the logger. Use this form instead of directly
-	 * accessing the {@link $categoryName} field.
-	 * @return string  
-	 */
-	public function getLoggerName() {
-		return $this->categoryName;
-	}
+    /**
+     * Return the name of the logger. Use this form instead of directly
+     * accessing the {@link $categoryName} field.
+     * @return string  
+     */
+    public function getLoggerName() {
+        return $this->categoryName;
+    }
 
-	/**
-	 * Return the message for this logging event.
-	 *
-	 * <p>Before serialization, the returned object is the message
-	 * passed by the user to generate the logging event. After
-	 * serialization, the returned value equals the String form of the
-	 * message possibly after object rendering.
-	 * @return mixed
-	 */
-	public function getMessage() {
-		if($this->message !== null) {
-			return $this->message;
-		} else {
-			return $this->getRenderedMessage();
-		}
-	}
+    /**
+     * Return the message for this logging event.
+     *
+     * <p>Before serialization, the returned object is the message
+     * passed by the user to generate the logging event. After
+     * serialization, the returned value equals the String form of the
+     * message possibly after object rendering.
+     * @return mixed
+     */
+    public function getMessage() {
+        if($this->message !== null) {
+            return $this->message;
+        } else {
+            return $this->getRenderedMessage();
+        }
+    }
 
-	/**
-	 * This method returns the NDC for this event. It will return the
-	 * correct content even if the event was generated in a different
-	 * thread or even on a different machine. The {@link LoggerNDC::get()} method
-	 * should <b>never</b> be called directly.
-	 * @return string  
-	 */
-	public function getNDC() {
-		if($this->ndcLookupRequired) {
-			$this->ndcLookupRequired = false;
-			$this->ndc = Payone_Log4php_LoggerNDC::get();
-		}
-		return $this->ndc;
-	}
+    /**
+     * This method returns the NDC for this event. It will return the
+     * correct content even if the event was generated in a different
+     * thread or even on a different machine. The {@link LoggerNDC::get()} method
+     * should <b>never</b> be called directly.
+     * @return string  
+     */
+    public function getNDC() {
+        if($this->ndcLookupRequired) {
+            $this->ndcLookupRequired = false;
+            $this->ndc = Payone_Log4php_LoggerNDC::get();
+        }
+        return $this->ndc;
+    }
 
-	/**
-	 * Returns the the context corresponding to the <code>key</code>
-	 * parameter.
-	 * @return string
-	 */
-	public function getMDC($key) {
-		return Payone_Log4php_LoggerMDC::get($key);
-	}
+    /**
+     * Returns the the context corresponding to the <code>key</code>
+     * parameter.
+     * @return string
+     */
+    public function getMDC($key) {
+        return Payone_Log4php_LoggerMDC::get($key);
+    }
 	
-	/**
-	 * Returns the entire MDC context.
-	 * @return array
-	 */
-	public function getMDCMap () {
-		return Payone_Log4php_LoggerMDC::getMap();
-	}
+    /**
+     * Returns the entire MDC context.
+     * @return array
+     */
+    public function getMDCMap () {
+        return Payone_Log4php_LoggerMDC::getMap();
+    }
 
-	/**
-	 * Render message.
-	 * @return string
-	 */
-	public function getRenderedMessage() {
-		if($this->renderedMessage === null and $this->message !== null) {
-			if(is_string($this->message)) {
-				$this->renderedMessage = $this->message;
-			} else {
-				$rendererMap = Payone_Log4php_Logger::getHierarchy()->getRendererMap();
-				$this->renderedMessage= $rendererMap->findAndRender($this->message);
-			}
-		}
-		return $this->renderedMessage;
-	}
+    /**
+     * Render message.
+     * @return string
+     */
+    public function getRenderedMessage() {
+        if($this->renderedMessage === null and $this->message !== null) {
+            if(is_string($this->message)) {
+                $this->renderedMessage = $this->message;
+            } else {
+                $rendererMap = Payone_Log4php_Logger::getHierarchy()->getRendererMap();
+                $this->renderedMessage= $rendererMap->findAndRender($this->message);
+            }
+        }
+        return $this->renderedMessage;
+    }
 
-	/**
-	 * Returns the time when the application started, as a UNIX timestamp 
-	 * with microseconds.
-	 * @return float
-	 */
-	public static function getStartTime() {
-		if(!isset(self::$startTime)) {
-			self::$startTime = microtime(true);
-		}
-		return self::$startTime; 
-	}
+    /**
+     * Returns the time when the application started, as a UNIX timestamp 
+     * with microseconds.
+     * @return float
+     */
+    public static function getStartTime() {
+        if(!isset(self::$startTime)) {
+            self::$startTime = microtime(true);
+        }
+        return self::$startTime; 
+    }
 
-	/**
-	 * @return float
-	 */
-	public function getTimeStamp() {
-		return $this->timeStamp;
-	}
+    /**
+     * @return float
+     */
+    public function getTimeStamp() {
+        return $this->timeStamp;
+    }
 	
-	/**
-	 * Calculates the time of this event.
-	 * @return the time after event starttime when this event has occured
-	 */
-	public function getTime() {
-		$eventTime = $this->getTimeStamp();
-		$eventStartTime = Payone_Log4php_LoggerLoggingEvent::getStartTime();
-		return number_format(($eventTime - $eventStartTime) * 1000, 0, '', '');
-	}
+    /**
+     * Calculates the time of this event.
+     * @return the time after event starttime when this event has occured
+     */
+    public function getTime() {
+        $eventTime = $this->getTimeStamp();
+        $eventStartTime = Payone_Log4php_LoggerLoggingEvent::getStartTime();
+        return number_format(($eventTime - $eventStartTime) * 1000, 0, '', '');
+    }
 	
-	/**
-	 * @return mixed
-	 */
-	public function getThreadName() {
-		if ($this->threadName === null) {
-			$this->threadName = (string)getmypid();
-		}
-		return $this->threadName;
-	}
+    /**
+     * @return mixed
+     */
+    public function getThreadName() {
+        if ($this->threadName === null) {
+            $this->threadName = (string)getmypid();
+        }
+        return $this->threadName;
+    }
 
-	/**
-	 * @return mixed LoggerThrowableInformation
-	 */
-	public function getThrowableInformation() {
-		return $this->throwableInfo;
-	}
+    /**
+     * @return mixed LoggerThrowableInformation
+     */
+    public function getThrowableInformation() {
+        return $this->throwableInfo;
+    }
 	
-	/**
-	 * Serialize this object
-	 * @return string
-	 */
-	public function toString() {
-		serialize($this);
-	}
+    /**
+     * Serialize this object
+     * @return string
+     */
+    public function toString() {
+        serialize($this);
+    }
 	
-	/**
-	 * Avoid serialization of the {@link $logger} object
-	 */
-	public function __sleep() {
-		return array(
-			'fqcn',
-			'categoryName',
-			'level',
-			'ndc',
-			'ndcLookupRequired',
-			'message',
-			'renderedMessage',
-			'threadName',
-			'timeStamp',
-			'locationInfo',
-		);
-	}
+    /**
+     * Avoid serialization of the {@link $logger} object
+     */
+    public function __sleep() {
+        return array(
+            'fqcn',
+            'categoryName',
+            'level',
+            'ndc',
+            'ndcLookupRequired',
+            'message',
+            'renderedMessage',
+            'threadName',
+            'timeStamp',
+            'locationInfo',
+        );
+    }
 
 }
 

Spacing +14 added lines, -14 removed lines

@@ -127,7 +127,7 @@ 
 	*/
 	public function __construct($fqcn, $logger, $priority, $message, $timeStamp = null, Exception $throwable = null) {
 		$this->fqcn = $fqcn;
-		if($logger instanceof Payone_Log4php_Logger) {
+		if ($logger instanceof Payone_Log4php_Logger) {
 			$this->logger = $logger;
 			$this->categoryName = $logger->getName();
 		} else {
@@ -135,7 +135,7 @@ 
 		}
 		$this->level = $priority;
 		$this->message = $message;
-		if($timeStamp !== null && is_float($timeStamp)) {
+		if ($timeStamp !== null && is_float($timeStamp)) {
 			$this->timeStamp = $timeStamp;
 		} else {
 			$this->timeStamp = microtime(true);
@@ -164,18 +164,18 @@ 
 	 * @return Payone_Log4php_LoggerLocationInfo
 	 */
 	public function getLocationInformation() {
-		if($this->locationInfo === null) {
+		if ($this->locationInfo === null) {
 
 			$locationInfo = array();
 			$trace = debug_backtrace();
 			$prevHop = null;
 			// make a downsearch to identify the caller
 			$hop = array_pop($trace);
-			while($hop !== null) {
-				if(isset($hop['class'])) {
+			while ($hop !== null) {
+				if (isset($hop['class'])) {
 					// we are sometimes in functions = no class available: avoid php warning here
 					$className = strtolower($hop['class']);
-					if(!empty($className) and ($className == 'logger' or 
+					if (!empty($className) and ($className == 'logger' or 
 						strtolower(get_parent_class($className)) == 'logger')) {
 						$locationInfo['line'] = $hop['line'];
 						$locationInfo['file'] = $hop['file'];
@@ -186,7 +186,7 @@ 
 				$hop = array_pop($trace);
 			}
 			$locationInfo['class'] = isset($prevHop['class']) ? $prevHop['class'] : 'main';
-			if(isset($prevHop['function']) and
+			if (isset($prevHop['function']) and
 				$prevHop['function'] !== 'include' and
 				$prevHop['function'] !== 'include_once' and
 				$prevHop['function'] !== 'require' and
@@ -238,7 +238,7 @@ 
 	 * @return mixed
 	 */
 	public function getMessage() {
-		if($this->message !== null) {
+		if ($this->message !== null) {
 			return $this->message;
 		} else {
 			return $this->getRenderedMessage();
@@ -253,7 +253,7 @@ 
 	 * @return string  
 	 */
 	public function getNDC() {
-		if($this->ndcLookupRequired) {
+		if ($this->ndcLookupRequired) {
 			$this->ndcLookupRequired = false;
 			$this->ndc = Payone_Log4php_LoggerNDC::get();
 		}
@@ -273,7 +273,7 @@ 
 	 * Returns the entire MDC context.
 	 * @return array
 	 */
-	public function getMDCMap () {
+	public function getMDCMap() {
 		return Payone_Log4php_LoggerMDC::getMap();
 	}
 
@@ -282,12 +282,12 @@ 
 	 * @return string
 	 */
 	public function getRenderedMessage() {
-		if($this->renderedMessage === null and $this->message !== null) {
-			if(is_string($this->message)) {
+		if ($this->renderedMessage === null and $this->message !== null) {
+			if (is_string($this->message)) {
 				$this->renderedMessage = $this->message;
 			} else {
 				$rendererMap = Payone_Log4php_Logger::getHierarchy()->getRendererMap();
-				$this->renderedMessage= $rendererMap->findAndRender($this->message);
+				$this->renderedMessage = $rendererMap->findAndRender($this->message);
 			}
 		}
 		return $this->renderedMessage;
@@ -299,7 +299,7 @@ 
 	 * @return float
 	 */
 	public static function getStartTime() {
-		if(!isset(self::$startTime)) {
+		if (!isset(self::$startTime)) {
 			self::$startTime = microtime(true);
 		}
 		return self::$startTime; 

lib/Payone/Log4php/LoggerNDC.php

Doc Comments +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
 
 	/**
 	 * Never use this method directly, use the {@link LoggerLoggingEvent::getNDC()} method instead.
-	 * @return array
+	 * @return string
 	 */
 	public static function get() {
 		return implode(' ', self::$stack);

Indentation +97 added lines, -97 removed lines

@@ -95,109 +95,109 @@
  */
 class Payone_Log4php_LoggerNDC {
 	
-	/** This is the repository of NDC stack */
-	private static $stack = array();
+    /** This is the repository of NDC stack */
+    private static $stack = array();
 	
-	/**
-	 * Clear any nested diagnostic information if any. This method is
-	 * useful in cases where the same thread can be potentially used
-	 * over and over in different unrelated contexts.
-	 *
-	 * <p>This method is equivalent to calling the {@link setMaxDepth()}
-	 * method with a zero <var>maxDepth</var> argument.
-	 */
-	public static function clear() {
-		self::$stack = array();
-	}
+    /**
+     * Clear any nested diagnostic information if any. This method is
+     * useful in cases where the same thread can be potentially used
+     * over and over in different unrelated contexts.
+     *
+     * <p>This method is equivalent to calling the {@link setMaxDepth()}
+     * method with a zero <var>maxDepth</var> argument.
+     */
+    public static function clear() {
+        self::$stack = array();
+    }
 
-	/**
-	 * Never use this method directly, use the {@link LoggerLoggingEvent::getNDC()} method instead.
-	 * @return array
-	 */
-	public static function get() {
-		return implode(' ', self::$stack);
-	}
+    /**
+     * Never use this method directly, use the {@link LoggerLoggingEvent::getNDC()} method instead.
+     * @return array
+     */
+    public static function get() {
+        return implode(' ', self::$stack);
+    }
   
-	/**
-	 * Get the current nesting depth of this diagnostic context.
-	 *
-	 * @see setMaxDepth()
-	 * @return integer
-	 */
-	public static function getDepth() {
-		return count(self::$stack);
-	}
+    /**
+     * Get the current nesting depth of this diagnostic context.
+     *
+     * @see setMaxDepth()
+     * @return integer
+     */
+    public static function getDepth() {
+        return count(self::$stack);
+    }
 
-	/**
-	 * Clients should call this method before leaving a diagnostic
-	 * context.
-	 *
-	 * <p>The returned value is the value that was pushed last. If no
-	 * context is available, then the empty string "" is returned.</p>
-	 *
-	 * @return string The innermost diagnostic context.
-	 */
-	public static function pop() {
-		if(count(self::$stack) > 0) {
-			return array_pop(self::$stack);
-		} else {
-			return '';
-		}
-	}
+    /**
+     * Clients should call this method before leaving a diagnostic
+     * context.
+     *
+     * <p>The returned value is the value that was pushed last. If no
+     * context is available, then the empty string "" is returned.</p>
+     *
+     * @return string The innermost diagnostic context.
+     */
+    public static function pop() {
+        if(count(self::$stack) > 0) {
+            return array_pop(self::$stack);
+        } else {
+            return '';
+        }
+    }
 
-	/**
-	 * Looks at the last diagnostic context at the top of this NDC
-	 * without removing it.
-	 *
-	 * <p>The returned value is the value that was pushed last. If no
-	 * context is available, then the empty string "" is returned.</p>
-	 * @return string The innermost diagnostic context.
-	 */
-	public static function peek(){
-		if(count(self::$stack) > 0) {
-			return end(self::$stack);
-		} else {
-			return '';
-		}
-	}
+    /**
+     * Looks at the last diagnostic context at the top of this NDC
+     * without removing it.
+     *
+     * <p>The returned value is the value that was pushed last. If no
+     * context is available, then the empty string "" is returned.</p>
+     * @return string The innermost diagnostic context.
+     */
+    public static function peek(){
+        if(count(self::$stack) > 0) {
+            return end(self::$stack);
+        } else {
+            return '';
+        }
+    }
   
-	/**
-	 * Push new diagnostic context information for the current thread.
-	 *
-	 * <p>The contents of the <var>message</var> parameter is
-	 * determined solely by the client.
-	 *	
-	 * @param string $message The new diagnostic context information.
-	 */
-	public static function push($message) {
-		array_push(self::$stack, (string)$message);
-	}
+    /**
+     * Push new diagnostic context information for the current thread.
+     *
+     * <p>The contents of the <var>message</var> parameter is
+     * determined solely by the client.
+     *	
+     * @param string $message The new diagnostic context information.
+     */
+    public static function push($message) {
+        array_push(self::$stack, (string)$message);
+    }
 
-	/**
-	 * Remove the diagnostic context for this thread.
-	 */
-	public static function remove() {
-		Payone_Log4php_LoggerNDC::clear();
-	}
+    /**
+     * Remove the diagnostic context for this thread.
+     */
+    public static function remove() {
+        Payone_Log4php_LoggerNDC::clear();
+    }
 
-	/**
-	 * Set maximum depth of this diagnostic context. If the current
-	 * depth is smaller or equal to <var>maxDepth</var>, then no
-	 * action is taken.
-	 *
-	 * <p>This method is a convenient alternative to multiple 
-	 * {@link pop()} calls. Moreover, it is often the case that at 
-	 * the end of complex call sequences, the depth of the NDC is
-	 * unpredictable. The {@link setMaxDepth()} method circumvents
-	 * this problem.
-	 *
-	 * @param integer $maxDepth
-	 * @see getDepth()
-	 */
-	public static function setMaxDepth($maxDepth) {
-		$maxDepth = (int)$maxDepth;
-		if(Payone_Log4php_LoggerNDC::getDepth() > $maxDepth) {
-			self::$stack = array_slice(self::$stack, 0, $maxDepth);
-		}
-	}
+    /**
+     * Set maximum depth of this diagnostic context. If the current
+     * depth is smaller or equal to <var>maxDepth</var>, then no
+     * action is taken.
+     *
+     * <p>This method is a convenient alternative to multiple 
+     * {@link pop()} calls. Moreover, it is often the case that at 
+     * the end of complex call sequences, the depth of the NDC is
+     * unpredictable. The {@link setMaxDepth()} method circumvents
+     * this problem.
+     *
+     * @param integer $maxDepth
+     * @see getDepth()
+     */
+    public static function setMaxDepth($maxDepth) {
+        $maxDepth = (int)$maxDepth;
+        if(Payone_Log4php_LoggerNDC::getDepth() > $maxDepth) {
+            self::$stack = array_slice(self::$stack, 0, $maxDepth);
+        }
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -138,7 +138,7 @@ 
 	 * @return string The innermost diagnostic context.
 	 */
 	public static function pop() {
-		if(count(self::$stack) > 0) {
+		if (count(self::$stack) > 0) {
 			return array_pop(self::$stack);
 		} else {
 			return '';
@@ -153,8 +153,8 @@ 
 	 * context is available, then the empty string "" is returned.</p>
 	 * @return string The innermost diagnostic context.
 	 */
-	public static function peek(){
-		if(count(self::$stack) > 0) {
+	public static function peek() {
+		if (count(self::$stack) > 0) {
 			return end(self::$stack);
 		} else {
 			return '';
@@ -196,7 +196,7 @@ 
 	 */
 	public static function setMaxDepth($maxDepth) {
 		$maxDepth = (int)$maxDepth;
-		if(Payone_Log4php_LoggerNDC::getDepth() > $maxDepth) {
+		if (Payone_Log4php_LoggerNDC::getDepth() > $maxDepth) {
 			self::$stack = array_slice(self::$stack, 0, $maxDepth);
 		}
 	}

lib/Payone/Log4php/LoggerPatternConverter.php

Doc Comments +1 added lines

@@ -73,6 +73,7 @@
 	 * converters must implement this method.
 	 *
 	 * @param Payone_Log4php_LoggerLoggingEvent $event
+	 * @return string
 	 */
 	abstract public function convert(Payone_Log4php_LoggerLoggingEvent $event);
 

Indentation +82 added lines, -82 removed lines

@@ -33,99 +33,99 @@
  */
 abstract class Payone_Log4php_LoggerPatternConverter {
 	
-	/**
-	 * Next converter in the converter chain.
-	 * @var Payone_Log4php_LoggerPatternConverter
-	 */
-	public $next = null;
+    /**
+     * Next converter in the converter chain.
+     * @var Payone_Log4php_LoggerPatternConverter
+     */
+    public $next = null;
 	
-	/**
-	 * Formatting information, parsed from pattern modifiers. 
-	 * @var Payone_Log4php_LoggerFormattingInfo
-	 */
-	protected $formattingInfo;
+    /**
+     * Formatting information, parsed from pattern modifiers. 
+     * @var Payone_Log4php_LoggerFormattingInfo
+     */
+    protected $formattingInfo;
 	
-	/**
-	 * Converter-specific formatting options.
-	 * @var array
-	 */
-	protected $option;
+    /**
+     * Converter-specific formatting options.
+     * @var array
+     */
+    protected $option;
 
-	/**
-	 * Constructor 
-	 * @param Payone_Log4php_LoggerFormattingInfo $formattingInfo
-	 * @param array $option
-	 */
-	public function __construct(Payone_Log4php_LoggerFormattingInfo $formattingInfo = null, $option = null) {
-		$this->formattingInfo = $formattingInfo;
-		$this->option = $option;
-		$this->activateOptions();
-	}
+    /**
+     * Constructor 
+     * @param Payone_Log4php_LoggerFormattingInfo $formattingInfo
+     * @param array $option
+     */
+    public function __construct(Payone_Log4php_LoggerFormattingInfo $formattingInfo = null, $option = null) {
+        $this->formattingInfo = $formattingInfo;
+        $this->option = $option;
+        $this->activateOptions();
+    }
 	
-	/**
-	 * Called in constructor. Converters which need to process the options 
-	 * can override this method. 
-	 */
-	public function activateOptions() { }
+    /**
+     * Called in constructor. Converters which need to process the options 
+     * can override this method. 
+     */
+    public function activateOptions() { }
   
-	/**
-	 * Converts the logging event to the desired format. Derived pattern 
-	 * converters must implement this method.
-	 *
-	 * @param Payone_Log4php_LoggerLoggingEvent $event
-	 */
-	abstract public function convert(Payone_Log4php_LoggerLoggingEvent $event);
+    /**
+     * Converts the logging event to the desired format. Derived pattern 
+     * converters must implement this method.
+     *
+     * @param Payone_Log4php_LoggerLoggingEvent $event
+     */
+    abstract public function convert(Payone_Log4php_LoggerLoggingEvent $event);
 
-	/**
-	 * Converts the event and formats it according to setting in the 
-	 * Formatting information object.
-	 *
-	 * @param string &$sbuf string buffer to write to
-	 * @param Payone_Log4php_LoggerLoggingEvent $event Event to be formatted.
-	 */
-	public function format(&$sbuf, $event) {
-		$string = $this->convert($event);
+    /**
+     * Converts the event and formats it according to setting in the 
+     * Formatting information object.
+     *
+     * @param string &$sbuf string buffer to write to
+     * @param Payone_Log4php_LoggerLoggingEvent $event Event to be formatted.
+     */
+    public function format(&$sbuf, $event) {
+        $string = $this->convert($event);
 		
-		if (!isset($this->formattingInfo)) {
-			$sbuf .= $string;
-			return;	
-		}
+        if (!isset($this->formattingInfo)) {
+            $sbuf .= $string;
+            return;	
+        }
 		
-		$fi = $this->formattingInfo;
+        $fi = $this->formattingInfo;
 		
-		// Empty string
-		if($string === '' || is_null($string)) {
-			if($fi->min > 0) {
-				$sbuf .= str_repeat(' ', $fi->min);
-			}
-			return;
-		}
+        // Empty string
+        if($string === '' || is_null($string)) {
+            if($fi->min > 0) {
+                $sbuf .= str_repeat(' ', $fi->min);
+            }
+            return;
+        }
 		
-		$len = strlen($string);
+        $len = strlen($string);
 	
-		// Trim the string if needed
-		if($len > $fi->max) {
-			if ($fi->trimLeft) {
-				$sbuf .= substr($string, $len - $fi->max, $fi->max);
-			} else {
-				$sbuf .= substr($string , 0, $fi->max);
-			}
-		}
+        // Trim the string if needed
+        if($len > $fi->max) {
+            if ($fi->trimLeft) {
+                $sbuf .= substr($string, $len - $fi->max, $fi->max);
+            } else {
+                $sbuf .= substr($string , 0, $fi->max);
+            }
+        }
 		
-		// Add padding if needed
-		else if($len < $fi->min) {
-			if($fi->padLeft) {
-				$sbuf .= str_repeat(' ', $fi->min - $len);
-				$sbuf .= $string;
-			} else {
-				$sbuf .= $string;
-				$sbuf .= str_repeat(' ', $fi->min - $len);
-			}
-		}
+        // Add padding if needed
+        else if($len < $fi->min) {
+            if($fi->padLeft) {
+                $sbuf .= str_repeat(' ', $fi->min - $len);
+                $sbuf .= $string;
+            } else {
+                $sbuf .= $string;
+                $sbuf .= str_repeat(' ', $fi->min - $len);
+            }
+        }
 		
-		// No action needed
-		else {
-			$sbuf .= $string;
-		}
-	}
+        // No action needed
+        else {
+            $sbuf .= $string;
+        }
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -94,8 +94,8 @@ 
 		$fi = $this->formattingInfo;
 		
 		// Empty string
-		if($string === '' || is_null($string)) {
-			if($fi->min > 0) {
+		if ($string === '' || is_null($string)) {
+			if ($fi->min > 0) {
 				$sbuf .= str_repeat(' ', $fi->min);
 			}
 			return;
@@ -104,17 +104,17 @@ 
 		$len = strlen($string);
 	
 		// Trim the string if needed
-		if($len > $fi->max) {
+		if ($len > $fi->max) {
 			if ($fi->trimLeft) {
 				$sbuf .= substr($string, $len - $fi->max, $fi->max);
 			} else {
-				$sbuf .= substr($string , 0, $fi->max);
+				$sbuf .= substr($string, 0, $fi->max);
 			}
 		}
 		
 		// Add padding if needed
-		else if($len < $fi->min) {
-			if($fi->padLeft) {
+		else if ($len < $fi->min) {
+			if ($fi->padLeft) {
 				$sbuf .= str_repeat(' ', $fi->min - $len);
 				$sbuf .= $string;
 			} else {