fjbender / magento-1

lib/Payone/Log4php/LoggerAppender.php

Indentation +229 added lines, -229 removed lines

@@ -26,262 +26,262 @@
  */
 abstract class Payone_Log4php_LoggerAppender extends Payone_Log4php_LoggerConfigurable {
 	
-	/**
-	 * Set to true when the appender is closed. A closed appender will not 
-	 * accept any logging requests. 
-	 * @var boolean 
-	 */
-	protected $closed = false;
+    /**
+     * Set to true when the appender is closed. A closed appender will not 
+     * accept any logging requests. 
+     * @var boolean 
+     */
+    protected $closed = false;
 	
-	/**
-	 * The first filter in the filter chain.
-	 * @var LoggerFilter
-	 */
-	protected $filter;
+    /**
+     * The first filter in the filter chain.
+     * @var LoggerFilter
+     */
+    protected $filter;
 			
-	/**
-	 * The appender's layout. Can be null if the appender does not use 
-	 * a layout.
-	 * @var Payone_Log4php_LoggerLayout
-	 */
-	protected $layout; 
+    /**
+     * The appender's layout. Can be null if the appender does not use 
+     * a layout.
+     * @var Payone_Log4php_LoggerLayout
+     */
+    protected $layout; 
 	
-	/**
-	 * Appender name. Used by other components to identify this appender.
-	 * @var string
-	 */
-	protected $name;
+    /**
+     * Appender name. Used by other components to identify this appender.
+     * @var string
+     */
+    protected $name;
 	
-	/**
-	 * Appender threshold level. Events whose level is below the threshold 
-	 * will not be logged.
-	 * @var Payone_Log4php_LoggerLevel
-	 */
-	protected $threshold;
+    /**
+     * Appender threshold level. Events whose level is below the threshold 
+     * will not be logged.
+     * @var Payone_Log4php_LoggerLevel
+     */
+    protected $threshold;
 	
-	/**
-	 * Set to true if the appender requires a layout.
-	 * 
-	 * True by default, appenders which do not use a layout should override 
-	 * this property to false.
-	 * 
-	 * @var boolean
-	 */
-	protected $requiresLayout = true;
+    /**
+     * Set to true if the appender requires a layout.
+     * 
+     * True by default, appenders which do not use a layout should override 
+     * this property to false.
+     * 
+     * @var boolean
+     */
+    protected $requiresLayout = true;
 	
-	/**
-	 * Default constructor.
-	 * @param string $name Appender name
-	 */
-	public function __construct($name = '') {
-		$this->name = $name;
+    /**
+     * Default constructor.
+     * @param string $name Appender name
+     */
+    public function __construct($name = '') {
+        $this->name = $name;
 		
-		// Closes the appender on shutdown. Better than a destructor because
-		// it will be called even if a fatal error occurs (destructor won't).
-		register_shutdown_function(array($this, 'close'));
+        // Closes the appender on shutdown. Better than a destructor because
+        // it will be called even if a fatal error occurs (destructor won't).
+        register_shutdown_function(array($this, 'close'));
 		
-		if ($this->requiresLayout) {
-			$this->layout = $this->getDefaultLayout();
-		}
-	}
+        if ($this->requiresLayout) {
+            $this->layout = $this->getDefaultLayout();
+        }
+    }
 	
-	/**
-	 * Returns the default layout for this appender. Can be overriden by 
-	 * derived appenders.
-	 * 
-	 * @return Payone_Log4php_LoggerLayout
-	 */
-	public function getDefaultLayout()
-	{
-		return new Payone_Log4php_LoggerLayoutSimple();
-	}
+    /**
+     * Returns the default layout for this appender. Can be overriden by 
+     * derived appenders.
+     * 
+     * @return Payone_Log4php_LoggerLayout
+     */
+    public function getDefaultLayout()
+    {
+        return new Payone_Log4php_LoggerLayoutSimple();
+    }
 	
-	/**
-	 * Adds a filter to the end of the filter chain.
-	 * @param LoggerFilter $filter add a new LoggerFilter
-	 */
-	public function addFilter($filter) {
-		if($this->filter === null) {
-			$this->filter = $filter;
-		} else {
-			$this->filter->addNext($filter);
-		}
-	}
+    /**
+     * Adds a filter to the end of the filter chain.
+     * @param LoggerFilter $filter add a new LoggerFilter
+     */
+    public function addFilter($filter) {
+        if($this->filter === null) {
+            $this->filter = $filter;
+        } else {
+            $this->filter->addNext($filter);
+        }
+    }
 	
-	/**
-	 * Clears the filter chain by removing all the filters in it.
-	 */
-	public function clearFilters() {
-		$this->filter = null;
-	}
+    /**
+     * Clears the filter chain by removing all the filters in it.
+     */
+    public function clearFilters() {
+        $this->filter = null;
+    }
 
-	/**
-	 * Returns the first filter in the filter chain. 
-	 * The return value may be <i>null</i> if no is filter is set.
-	 * @return LoggerFilter
-	 */
-	public function getFilter() {
-		return $this->filter;
-	} 
+    /**
+     * Returns the first filter in the filter chain. 
+     * The return value may be <i>null</i> if no is filter is set.
+     * @return LoggerFilter
+     */
+    public function getFilter() {
+        return $this->filter;
+    } 
 	
-	/** 
-	 * Returns the first filter in the filter chain. 
-	 * The return value may be <i>null</i> if no is filter is set.
-	 * @return LoggerFilter
-	 */
-	public function getFirstFilter() {
-		return $this->filter;
-	}
+    /** 
+     * Returns the first filter in the filter chain. 
+     * The return value may be <i>null</i> if no is filter is set.
+     * @return LoggerFilter
+     */
+    public function getFirstFilter() {
+        return $this->filter;
+    }
 	
-	/**
-	 * Performs threshold checks and invokes filters before delegating logging 
-	 * to the subclass' specific <i>append()</i> method.
-	 * @see Payone_Log4php_LoggerAppender::append()
-	 * @param Payone_Log4php_LoggerLoggingEvent $event
-	 */
-	public function doAppend(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->closed) {
-			return;
-		}
+    /**
+     * Performs threshold checks and invokes filters before delegating logging 
+     * to the subclass' specific <i>append()</i> method.
+     * @see Payone_Log4php_LoggerAppender::append()
+     * @param Payone_Log4php_LoggerLoggingEvent $event
+     */
+    public function doAppend(Payone_Log4php_LoggerLoggingEvent $event) {
+        if($this->closed) {
+            return;
+        }
 		
-		if(!$this->isAsSevereAsThreshold($event->getLevel())) {
-			return;
-		}
+        if(!$this->isAsSevereAsThreshold($event->getLevel())) {
+            return;
+        }
 
-		$f = $this->getFirstFilter();
-		while($f !== null) {
-			switch ($f->decide($event)) {
-				case Payone_Log4php_LoggerFilter::DENY: return;
-				case Payone_Log4php_LoggerFilter::ACCEPT: return $this->append($event);
-				case Payone_Log4php_LoggerFilter::NEUTRAL: $f = $f->getNext();
-			}
-		}
-		$this->append($event);
-	}	 
+        $f = $this->getFirstFilter();
+        while($f !== null) {
+            switch ($f->decide($event)) {
+                case Payone_Log4php_LoggerFilter::DENY: return;
+                case Payone_Log4php_LoggerFilter::ACCEPT: return $this->append($event);
+                case Payone_Log4php_LoggerFilter::NEUTRAL: $f = $f->getNext();
+            }
+        }
+        $this->append($event);
+    }	 
 
-	/**
-	 * Sets the appender layout.
-	 * @param Payone_Log4php_LoggerLayout $layout
-	 */
-	public function setLayout($layout) {
-		if($this->requiresLayout()) {
-			$this->layout = $layout;
-		}
-	} 
+    /**
+     * Sets the appender layout.
+     * @param Payone_Log4php_LoggerLayout $layout
+     */
+    public function setLayout($layout) {
+        if($this->requiresLayout()) {
+            $this->layout = $layout;
+        }
+    } 
 	
-	/**
-	 * Returns the appender layout.
-	 * @return Payone_Log4php_LoggerLayout
-	 */
-	public function getLayout() {
-		return $this->layout;
-	}
+    /**
+     * Returns the appender layout.
+     * @return Payone_Log4php_LoggerLayout
+     */
+    public function getLayout() {
+        return $this->layout;
+    }
 	
-	/**
-	 * Configurators call this method to determine if the appender
-	 * requires a layout. 
-	 *
-	 * <p>If this method returns <i>true</i>, meaning that layout is required, 
-	 * then the configurator will configure a layout using the configuration 
-	 * information at its disposal.	 If this method returns <i>false</i>, 
-	 * meaning that a layout is not required, then layout configuration will be
-	 * skipped even if there is available layout configuration
-	 * information at the disposal of the configurator.</p>
-	 *
-	 * <p>In the rather exceptional case, where the appender
-	 * implementation admits a layout but can also work without it, then
-	 * the appender should return <i>true</i>.</p>
-	 * 
-	 * @return boolean
-	 */
-	public function requiresLayout() {
-		return $this->requiresLayout;
-	}
+    /**
+     * Configurators call this method to determine if the appender
+     * requires a layout. 
+     *
+     * <p>If this method returns <i>true</i>, meaning that layout is required, 
+     * then the configurator will configure a layout using the configuration 
+     * information at its disposal.	 If this method returns <i>false</i>, 
+     * meaning that a layout is not required, then layout configuration will be
+     * skipped even if there is available layout configuration
+     * information at the disposal of the configurator.</p>
+     *
+     * <p>In the rather exceptional case, where the appender
+     * implementation admits a layout but can also work without it, then
+     * the appender should return <i>true</i>.</p>
+     * 
+     * @return boolean
+     */
+    public function requiresLayout() {
+        return $this->requiresLayout;
+    }
 	
-	/**
-	 * Retruns the appender name.
-	 * @return string
-	 */
-	public function getName() {
-		return $this->name;
-	}
+    /**
+     * Retruns the appender name.
+     * @return string
+     */
+    public function getName() {
+        return $this->name;
+    }
 	
-	/**
-	 * Sets the appender name.
-	 * @param string $name
-	 */
-	public function setName($name) {
-		$this->name = $name;	
-	}
+    /**
+     * Sets the appender name.
+     * @param string $name
+     */
+    public function setName($name) {
+        $this->name = $name;	
+    }
 	
-	/**
-	 * Returns the appender's threshold level. 
-	 * @return Payone_Log4php_LoggerLevel
-	 */
-	public function getThreshold() { 
-		return $this->threshold;
-	}
+    /**
+     * Returns the appender's threshold level. 
+     * @return Payone_Log4php_LoggerLevel
+     */
+    public function getThreshold() { 
+        return $this->threshold;
+    }
 	
-	/**
-	 * Sets the appender threshold.
-	 * 
-	 * @param Payone_Log4php_LoggerLevel|string $threshold Either a {@link LoggerLevel}
-	 *   object or a string equivalent.
-	 * @see LoggerOptionConverter::toLevel()
-	 */
-	public function setThreshold($threshold) {
-		$this->setLevel('threshold', $threshold);
-	}
+    /**
+     * Sets the appender threshold.
+     * 
+     * @param Payone_Log4php_LoggerLevel|string $threshold Either a {@link LoggerLevel}
+     *   object or a string equivalent.
+     * @see LoggerOptionConverter::toLevel()
+     */
+    public function setThreshold($threshold) {
+        $this->setLevel('threshold', $threshold);
+    }
 	
-	/**
-	 * Checks whether the message level is below the appender's threshold. 
-	 *
-	 * If there is no threshold set, then the return value is always <i>true</i>.
-	 * 
-	 * @param Payone_Log4php_LoggerLevel $level
-	 * @return boolean Returns true if level is greater or equal than 
-	 *   threshold, or if the threshold is not set. Otherwise returns false.
-	 */
-	public function isAsSevereAsThreshold($level) {
-		if($this->threshold === null) {
-			return true;
-		}
-		return $level->isGreaterOrEqual($this->getThreshold());
-	}
+    /**
+     * Checks whether the message level is below the appender's threshold. 
+     *
+     * If there is no threshold set, then the return value is always <i>true</i>.
+     * 
+     * @param Payone_Log4php_LoggerLevel $level
+     * @return boolean Returns true if level is greater or equal than 
+     *   threshold, or if the threshold is not set. Otherwise returns false.
+     */
+    public function isAsSevereAsThreshold($level) {
+        if($this->threshold === null) {
+            return true;
+        }
+        return $level->isGreaterOrEqual($this->getThreshold());
+    }
 
-	/**
-	 * Prepares the appender for logging.
-	 * 
-	 * Derived appenders should override this method if option structure
-	 * requires it.
-	 */
-	public function activateOptions() {
-		$this->closed = false;
-	}
+    /**
+     * Prepares the appender for logging.
+     * 
+     * Derived appenders should override this method if option structure
+     * requires it.
+     */
+    public function activateOptions() {
+        $this->closed = false;
+    }
 	
-	/**
-	 * Forwards the logging event to the destination.
-	 * 
-	 * Derived appenders should implement this method to perform actual logging.
-	 * 
-	 * @param Payone_Log4php_LoggerLoggingEvent $event
-	 */
-	abstract protected function append(Payone_Log4php_LoggerLoggingEvent $event);
+    /**
+     * Forwards the logging event to the destination.
+     * 
+     * Derived appenders should implement this method to perform actual logging.
+     * 
+     * @param Payone_Log4php_LoggerLoggingEvent $event
+     */
+    abstract protected function append(Payone_Log4php_LoggerLoggingEvent $event);
 
-	/**
-	 * Releases any resources allocated by the appender.
-	 * 
-	 * Derived appenders should override this method to perform proper closing
-	 * procedures.
-	 */
-	public function close() {
-		$this->closed = true;
-	}
+    /**
+     * Releases any resources allocated by the appender.
+     * 
+     * Derived appenders should override this method to perform proper closing
+     * procedures.
+     */
+    public function close() {
+        $this->closed = true;
+    }
 	
-	/** Triggers a warning for this logger with the given message. */
-	protected function warn($message) {
-		$id = get_class($this) . (empty($this->name) ? '' : ":{$this->name}");
-		trigger_error("log4php: [$id]: $message", E_USER_WARNING);
-	}
+    /** Triggers a warning for this logger with the given message. */
+    protected function warn($message) {
+        $id = get_class($this) . (empty($this->name) ? '' : ":{$this->name}");
+        trigger_error("log4php: [$id]: $message", E_USER_WARNING);
+    }
 	
 }

Spacing +6 added lines, -6 removed lines

@@ -101,7 +101,7 @@ 
 	 * @param LoggerFilter $filter add a new LoggerFilter
 	 */
 	public function addFilter($filter) {
-		if($this->filter === null) {
+		if ($this->filter === null) {
 			$this->filter = $filter;
 		} else {
 			$this->filter->addNext($filter);
@@ -140,16 +140,16 @@ 
 	 * @param Payone_Log4php_LoggerLoggingEvent $event
 	 */
 	public function doAppend(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->closed) {
+		if ($this->closed) {
 			return;
 		}
 		
-		if(!$this->isAsSevereAsThreshold($event->getLevel())) {
+		if (!$this->isAsSevereAsThreshold($event->getLevel())) {
 			return;
 		}
 
 		$f = $this->getFirstFilter();
-		while($f !== null) {
+		while ($f !== null) {
 			switch ($f->decide($event)) {
 				case Payone_Log4php_LoggerFilter::DENY: return;
 				case Payone_Log4php_LoggerFilter::ACCEPT: return $this->append($event);
@@ -164,7 +164,7 @@ 
 	 * @param Payone_Log4php_LoggerLayout $layout
 	 */
 	public function setLayout($layout) {
-		if($this->requiresLayout()) {
+		if ($this->requiresLayout()) {
 			$this->layout = $layout;
 		}
 	} 
@@ -243,7 +243,7 @@ 
 	 *   threshold, or if the threshold is not set. Otherwise returns false.
 	 */
 	public function isAsSevereAsThreshold($level) {
-		if($this->threshold === null) {
+		if ($this->threshold === null) {
 			return true;
 		}
 		return $level->isGreaterOrEqual($this->getThreshold());

lib/Payone/Log4php/LoggerAppenderFile.php

Indentation +101 added lines, -101 removed lines

@@ -42,121 +42,121 @@
  */
 class Payone_Log4php_LoggerAppenderFile extends Payone_Log4php_LoggerAppender {
 
-	/**
-	 * @var boolean if {@link $file} exists, appends events.
-	 */
-	protected $append = true;
+    /**
+     * @var boolean if {@link $file} exists, appends events.
+     */
+    protected $append = true;
 	
-	/**
-	 * @var string the file name used to append events
-	 */
-	protected $file;
+    /**
+     * @var string the file name used to append events
+     */
+    protected $file;
 
-	/**
-	 * @var mixed file resource
-	 */
-	protected $fp = false;
+    /**
+     * @var mixed file resource
+     */
+    protected $fp = false;
 	
-	public function activateOptions() {
-		$fileName = $this->getFile();
+    public function activateOptions() {
+        $fileName = $this->getFile();
 
-		if (empty($fileName)) {
-			$this->warn("Required parameter 'fileName' not set. Closing appender.");
-			$this->closed = true;
-			return;
-		}
+        if (empty($fileName)) {
+            $this->warn("Required parameter 'fileName' not set. Closing appender.");
+            $this->closed = true;
+            return;
+        }
 		
-		if(!is_file($fileName)) {
-			$dir = dirname($fileName);
-			if(!is_dir($dir)) {
-				mkdir($dir, 0777, true);
-			}
-		}
+        if(!is_file($fileName)) {
+            $dir = dirname($fileName);
+            if(!is_dir($dir)) {
+                mkdir($dir, 0777, true);
+            }
+        }
 
-		$this->fp = fopen($fileName, ($this->getAppend()? 'a':'w'));
-		if($this->fp) {
-			if(flock($this->fp, LOCK_EX)) {
-				if($this->getAppend()) {
-					fseek($this->fp, 0, SEEK_END);
-				}
-				fwrite($this->fp, $this->layout->getHeader());
-				flock($this->fp, LOCK_UN);
-				$this->closed = false;
-			} else {
-				// TODO: should we take some action in this case?
-				$this->closed = true;
-			}		 
-		} else {
-			$this->closed = true;
-		}
-	}
+        $this->fp = fopen($fileName, ($this->getAppend()? 'a':'w'));
+        if($this->fp) {
+            if(flock($this->fp, LOCK_EX)) {
+                if($this->getAppend()) {
+                    fseek($this->fp, 0, SEEK_END);
+                }
+                fwrite($this->fp, $this->layout->getHeader());
+                flock($this->fp, LOCK_UN);
+                $this->closed = false;
+            } else {
+                // TODO: should we take some action in this case?
+                $this->closed = true;
+            }		 
+        } else {
+            $this->closed = true;
+        }
+    }
 	
-	public function close() {
-		if($this->closed != true) {
-			if($this->fp and $this->layout !== null) {
-				if(flock($this->fp, LOCK_EX)) {
-					fwrite($this->fp, $this->layout->getFooter());
-					flock($this->fp, LOCK_UN);
-				}
-				fclose($this->fp);
-			}
-			$this->closed = true;
-		}
-	}
+    public function close() {
+        if($this->closed != true) {
+            if($this->fp and $this->layout !== null) {
+                if(flock($this->fp, LOCK_EX)) {
+                    fwrite($this->fp, $this->layout->getFooter());
+                    flock($this->fp, LOCK_UN);
+                }
+                fclose($this->fp);
+            }
+            $this->closed = true;
+        }
+    }
 
-	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->fp and $this->layout !== null) {
-			if(flock($this->fp, LOCK_EX)) {
-				fwrite($this->fp, $this->layout->format($event));
-				flock($this->fp, LOCK_UN);
-			} else {
-				$this->closed = true;
-			}
-		} 
-	}
+    public function append(Payone_Log4php_LoggerLoggingEvent $event) {
+        if($this->fp and $this->layout !== null) {
+            if(flock($this->fp, LOCK_EX)) {
+                fwrite($this->fp, $this->layout->format($event));
+                flock($this->fp, LOCK_UN);
+            } else {
+                $this->closed = true;
+            }
+        } 
+    }
 	
-	/**
-	 * Sets the file where the log output will go.
-	 * @param string $file
-	 */
-	public function setFile($file) {
-		$this->setString('file', $file);
-	}
+    /**
+     * Sets the file where the log output will go.
+     * @param string $file
+     */
+    public function setFile($file) {
+        $this->setString('file', $file);
+    }
 	
-	/**
-	 * @return string
-	 */
-	public function getFile() {
-		return $this->file;
-	}
+    /**
+     * @return string
+     */
+    public function getFile() {
+        return $this->file;
+    }
 	
-	/**
-	 * @return boolean
-	 */
-	public function getAppend() {
-		return $this->append;
-	}
+    /**
+     * @return boolean
+     */
+    public function getAppend() {
+        return $this->append;
+    }
 
-	public function setAppend($append) {
-		$this->setBoolean('append', $append);
-	}
+    public function setAppend($append) {
+        $this->setBoolean('append', $append);
+    }
 
-	/**
-	 * Sets the file where the log output will go.
-	 * @param string $fileName
-	 * @deprecated Use setFile() instead.
-	 */
-	public function setFileName($fileName) {
-		$this->setFile($fileName);
-	}
+    /**
+     * Sets the file where the log output will go.
+     * @param string $fileName
+     * @deprecated Use setFile() instead.
+     */
+    public function setFileName($fileName) {
+        $this->setFile($fileName);
+    }
 	
-	/**
-	 * @return string
-	 * @deprecated Use getFile() instead.
-	 */
-	public function getFileName() {
-		return $this->getFile();
-	}
+    /**
+     * @return string
+     * @deprecated Use getFile() instead.
+     */
+    public function getFileName() {
+        return $this->getFile();
+    }
 	
 	 
 }

Spacing +11 added lines, -11 removed lines

@@ -66,17 +66,17 @@ 
 			return;
 		}
 		
-		if(!is_file($fileName)) {
+		if (!is_file($fileName)) {
 			$dir = dirname($fileName);
-			if(!is_dir($dir)) {
+			if (!is_dir($dir)) {
 				mkdir($dir, 0777, true);
 			}
 		}
 
-		$this->fp = fopen($fileName, ($this->getAppend()? 'a':'w'));
-		if($this->fp) {
-			if(flock($this->fp, LOCK_EX)) {
-				if($this->getAppend()) {
+		$this->fp = fopen($fileName, ($this->getAppend() ? 'a' : 'w'));
+		if ($this->fp) {
+			if (flock($this->fp, LOCK_EX)) {
+				if ($this->getAppend()) {
 					fseek($this->fp, 0, SEEK_END);
 				}
 				fwrite($this->fp, $this->layout->getHeader());
@@ -92,9 +92,9 @@ 
 	}
 	
 	public function close() {
-		if($this->closed != true) {
-			if($this->fp and $this->layout !== null) {
-				if(flock($this->fp, LOCK_EX)) {
+		if ($this->closed != true) {
+			if ($this->fp and $this->layout !== null) {
+				if (flock($this->fp, LOCK_EX)) {
 					fwrite($this->fp, $this->layout->getFooter());
 					flock($this->fp, LOCK_UN);
 				}
@@ -105,8 +105,8 @@ 
 	}
 
 	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->fp and $this->layout !== null) {
-			if(flock($this->fp, LOCK_EX)) {
+		if ($this->fp and $this->layout !== null) {
+			if (flock($this->fp, LOCK_EX)) {
 				fwrite($this->fp, $this->layout->format($event));
 				flock($this->fp, LOCK_UN);
 			} else {

lib/Payone/Log4php/LoggerPatternConverterFile.php

Indentation +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@
  */
 class Payone_Log4php_LoggerPatternConverterFile extends Payone_Log4php_LoggerPatternConverter {
 
-	public function convert(Payone_Log4php_LoggerLoggingEvent $event) {
-		return $event->getLocationInformation()->getFileName();
-	}
+    public function convert(Payone_Log4php_LoggerLoggingEvent $event) {
+        return $event->getLocationInformation()->getFileName();
+    }
 }

lib/Payone/Log4php/LoggerAppenderMongoDB.php

Indentation +229 added lines, -229 removed lines

@@ -33,282 +33,282 @@
  */
 class Payone_Log4php_LoggerAppenderMongoDB extends Payone_Log4php_LoggerAppender {
 	
-	// ******************************************
-	// ** Constants                            **
-	// ******************************************
+    // ******************************************
+    // ** Constants                            **
+    // ******************************************
 	
-	/** Default prefix for the {@link $host}. */	
-	const DEFAULT_MONGO_URL_PREFIX = 'mongodb://';
+    /** Default prefix for the {@link $host}. */	
+    const DEFAULT_MONGO_URL_PREFIX = 'mongodb://';
 	
-	/** Default value for {@link $host}, without a prefix. */
-	const DEFAULT_MONGO_HOST = 'localhost';
+    /** Default value for {@link $host}, without a prefix. */
+    const DEFAULT_MONGO_HOST = 'localhost';
 	
-	/** Default value for {@link $port} */
-	const DEFAULT_MONGO_PORT = 27017;
+    /** Default value for {@link $port} */
+    const DEFAULT_MONGO_PORT = 27017;
 	
-	/** Default value for {@link $databaseName} */
-	const DEFAULT_DB_NAME = 'log4php_mongodb';
+    /** Default value for {@link $databaseName} */
+    const DEFAULT_DB_NAME = 'log4php_mongodb';
 	
-	/** Default value for {@link $collectionName} */
-	const DEFAULT_COLLECTION_NAME = 'logs';
+    /** Default value for {@link $collectionName} */
+    const DEFAULT_COLLECTION_NAME = 'logs';
 	
-	/** Default value for {@link $timeout} */
-	const DEFAULT_TIMEOUT_VALUE = 3000;
+    /** Default value for {@link $timeout} */
+    const DEFAULT_TIMEOUT_VALUE = 3000;
 	
-	// ******************************************
-	// ** Configurable parameters              **
-	// ******************************************
+    // ******************************************
+    // ** Configurable parameters              **
+    // ******************************************
 	
-	/** Server on which mongodb instance is located. */
-	protected $host;
+    /** Server on which mongodb instance is located. */
+    protected $host;
 	
-	/** Port on which the instance is bound. */
-	protected $port;
+    /** Port on which the instance is bound. */
+    protected $port;
 	
-	/** Name of the database to which to log. */
-	protected $databaseName;
+    /** Name of the database to which to log. */
+    protected $databaseName;
 	
-	/** Name of the collection within the given database. */
-	protected $collectionName;
+    /** Name of the collection within the given database. */
+    protected $collectionName;
 			
-	/** Username used to connect to the database. */
-	protected $userName;
+    /** Username used to connect to the database. */
+    protected $userName;
 	
-	/** Password used to connect to the database. */
-	protected $password;
+    /** Password used to connect to the database. */
+    protected $password;
 	
-	/** Timeout value used when connecting to the database (in milliseconds). */
-	protected $timeout;
+    /** Timeout value used when connecting to the database (in milliseconds). */
+    protected $timeout;
 	
-	// ******************************************
-	// ** Member variables                     **
-	// ******************************************
+    // ******************************************
+    // ** Member variables                     **
+    // ******************************************
 
-	/** 
-	 * Connection to the MongoDB instance.
-	 * @var Mongo
-	 */
-	protected $connection;
+    /** 
+     * Connection to the MongoDB instance.
+     * @var Mongo
+     */
+    protected $connection;
 	
-	/** 
-	 * The collection to which log is written. 
-	 * @var MongoCollection
-	 */
-	protected $collection;
+    /** 
+     * The collection to which log is written. 
+     * @var MongoCollection
+     */
+    protected $collection;
 
-	/** 
-	 * Set to true if the appender can append. 
-	 * @todo Maybe we should use $closed here instead? 
-	 */
-	protected $canAppend = false;
+    /** 
+     * Set to true if the appender can append. 
+     * @todo Maybe we should use $closed here instead? 
+     */
+    protected $canAppend = false;
 	
-	/** Appender does not require a layout. */
-	protected $requiresLayout = false;
+    /** Appender does not require a layout. */
+    protected $requiresLayout = false;
 		
-	public function __construct($name = '') {
-		parent::__construct($name);
-		$this->host = self::DEFAULT_MONGO_URL_PREFIX . self::DEFAULT_MONGO_HOST;
-		$this->port = self::DEFAULT_MONGO_PORT;
-		$this->databaseName = self::DEFAULT_DB_NAME;
-		$this->collectionName = self::DEFAULT_COLLECTION_NAME;
-		$this->timeout = self::DEFAULT_TIMEOUT_VALUE;
-	}
+    public function __construct($name = '') {
+        parent::__construct($name);
+        $this->host = self::DEFAULT_MONGO_URL_PREFIX . self::DEFAULT_MONGO_HOST;
+        $this->port = self::DEFAULT_MONGO_PORT;
+        $this->databaseName = self::DEFAULT_DB_NAME;
+        $this->collectionName = self::DEFAULT_COLLECTION_NAME;
+        $this->timeout = self::DEFAULT_TIMEOUT_VALUE;
+    }
 	
-	/**
-	 * Setup db connection.
-	 * Based on defined options, this method connects to the database and 
-	 * creates a {@link $collection}. 
-	 *  
-	 * @throws Exception if the attempt to connect to the requested database fails.
-	 */
-	public function activateOptions() {
-		try {
-			$this->connection = new Mongo(sprintf('%s:%d', $this->host, $this->port), array("timeout" => $this->timeout));
-			$db	= $this->connection->selectDB($this->databaseName);
-			if ($this->userName !== null && $this->password !== null) {
-				$authResult = $db->authenticate($this->userName, $this->password);
-				if ($authResult['ok'] == floatval(0)) {
-					throw new Exception($authResult['errmsg'], $authResult['ok']);
-				}
-			}
+    /**
+     * Setup db connection.
+     * Based on defined options, this method connects to the database and 
+     * creates a {@link $collection}. 
+     *  
+     * @throws Exception if the attempt to connect to the requested database fails.
+     */
+    public function activateOptions() {
+        try {
+            $this->connection = new Mongo(sprintf('%s:%d', $this->host, $this->port), array("timeout" => $this->timeout));
+            $db	= $this->connection->selectDB($this->databaseName);
+            if ($this->userName !== null && $this->password !== null) {
+                $authResult = $db->authenticate($this->userName, $this->password);
+                if ($authResult['ok'] == floatval(0)) {
+                    throw new Exception($authResult['errmsg'], $authResult['ok']);
+                }
+            }
 			
-			$this->collection = $db->selectCollection($this->collectionName);												 
-		} catch (Exception $ex) {
-			$this->canAppend = false;
-			throw new Payone_Log4php_LoggerException($ex);
-		} 
+            $this->collection = $db->selectCollection($this->collectionName);												 
+        } catch (Exception $ex) {
+            $this->canAppend = false;
+            throw new Payone_Log4php_LoggerException($ex);
+        } 
 			
-		$this->canAppend = true;
-	}		 
+        $this->canAppend = true;
+    }		 
 	
-	/**
-	 * Appends a new event to the mongo database.
-	 * 
-	 * @throws LoggerException If the pattern conversion or the INSERT statement fails.
-	 */
-	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
-		if ($this->canAppend == true && $this->collection != null) {
-			$document = $this->format($event);
-			$this->collection->insert($document);			
-		}
-	}
+    /**
+     * Appends a new event to the mongo database.
+     * 
+     * @throws LoggerException If the pattern conversion or the INSERT statement fails.
+     */
+    public function append(Payone_Log4php_LoggerLoggingEvent $event) {
+        if ($this->canAppend == true && $this->collection != null) {
+            $document = $this->format($event);
+            $this->collection->insert($document);			
+        }
+    }
 	
-	/**
-	 * Converts the logging event into an array which can be logged to mongodb.
-	 * 
-	 * @param Payone_Log4php_LoggerLoggingEvent $event
-	 * @return array The array representation of the logging event.
-	 */
-	protected function format(Payone_Log4php_LoggerLoggingEvent $event) {
-		$timestampSec  = (int) $event->getTimestamp();
-		$timestampUsec = (int) (($event->getTimestamp() - $timestampSec) * 1000000);
+    /**
+     * Converts the logging event into an array which can be logged to mongodb.
+     * 
+     * @param Payone_Log4php_LoggerLoggingEvent $event
+     * @return array The array representation of the logging event.
+     */
+    protected function format(Payone_Log4php_LoggerLoggingEvent $event) {
+        $timestampSec  = (int) $event->getTimestamp();
+        $timestampUsec = (int) (($event->getTimestamp() - $timestampSec) * 1000000);
 
-		$document = array(
-			'timestamp'  => new MongoDate($timestampSec, $timestampUsec),
-			'level'      => $event->getLevel()->toString(),
-			'thread'     => (int) $event->getThreadName(),
-			'message'    => $event->getMessage(),
-			'loggerName' => $event->getLoggerName() 
-		);	
+        $document = array(
+            'timestamp'  => new MongoDate($timestampSec, $timestampUsec),
+            'level'      => $event->getLevel()->toString(),
+            'thread'     => (int) $event->getThreadName(),
+            'message'    => $event->getMessage(),
+            'loggerName' => $event->getLoggerName() 
+        );	
 
-		$locationInfo = $event->getLocationInformation();
-		if ($locationInfo != null) {
-			$document['fileName']   = $locationInfo->getFileName();
-			$document['method']     = $locationInfo->getMethodName();
-			$document['lineNumber'] = ($locationInfo->getLineNumber() == 'NA') ? 'NA' : (int) $locationInfo->getLineNumber();
-			$document['className']  = $locationInfo->getClassName();
-		}	
+        $locationInfo = $event->getLocationInformation();
+        if ($locationInfo != null) {
+            $document['fileName']   = $locationInfo->getFileName();
+            $document['method']     = $locationInfo->getMethodName();
+            $document['lineNumber'] = ($locationInfo->getLineNumber() == 'NA') ? 'NA' : (int) $locationInfo->getLineNumber();
+            $document['className']  = $locationInfo->getClassName();
+        }	
 
-		$throwableInfo = $event->getThrowableInformation();
-		if ($throwableInfo != null) {
-			$document['exception'] = $this->formatThrowable($throwableInfo->getThrowable());
-		}
+        $throwableInfo = $event->getThrowableInformation();
+        if ($throwableInfo != null) {
+            $document['exception'] = $this->formatThrowable($throwableInfo->getThrowable());
+        }
 		
-		return $document;
-	}
+        return $document;
+    }
 	
-	/**
-	 * Converts an Exception into an array which can be logged to mongodb.
-	 * 
-	 * Supports innner exceptions (PHP >= 5.3)
-	 * 
-	 * @param Exception $ex
-	 * @return array
-	 */
-	protected function formatThrowable(Exception $ex) {
-		$array = array(				
-			'message'    => $ex->getMessage(),
-			'code'       => $ex->getCode(),
-			'stackTrace' => $ex->getTraceAsString(),
-		);
+    /**
+     * Converts an Exception into an array which can be logged to mongodb.
+     * 
+     * Supports innner exceptions (PHP >= 5.3)
+     * 
+     * @param Exception $ex
+     * @return array
+     */
+    protected function formatThrowable(Exception $ex) {
+        $array = array(				
+            'message'    => $ex->getMessage(),
+            'code'       => $ex->getCode(),
+            'stackTrace' => $ex->getTraceAsString(),
+        );
                         
-		if (method_exists($ex, 'getPrevious') && $ex->getPrevious() !== null) {
-			$array['innerException'] = $this->formatThrowable($ex->getPrevious());
-		}
+        if (method_exists($ex, 'getPrevious') && $ex->getPrevious() !== null) {
+            $array['innerException'] = $this->formatThrowable($ex->getPrevious());
+        }
 			
-		return $array;
-	}
+        return $array;
+    }
 		
-	/**
-	 * Closes the connection to the logging database
-	 */
-	public function close() {
-		if($this->closed != true) {
-			$this->collection = null;
-			if ($this->connection !== null) {
-				$this->connection->close();
-				$this->connection = null;	
-			}					
-			$this->closed = true;
-		}
-	}
+    /**
+     * Closes the connection to the logging database
+     */
+    public function close() {
+        if($this->closed != true) {
+            $this->collection = null;
+            if ($this->connection !== null) {
+                $this->connection->close();
+                $this->connection = null;	
+            }					
+            $this->closed = true;
+        }
+    }
 	
-	/** Sets the value of {@link $host} parameter. */
-	public function setHost($host) {
-		if (!preg_match('/^mongodb\:\/\//', $host)) {
-			$host = self::DEFAULT_MONGO_URL_PREFIX . $host;
-		}
-		$this->host = $host;
-	}
+    /** Sets the value of {@link $host} parameter. */
+    public function setHost($host) {
+        if (!preg_match('/^mongodb\:\/\//', $host)) {
+            $host = self::DEFAULT_MONGO_URL_PREFIX . $host;
+        }
+        $this->host = $host;
+    }
 		
-	/** Returns the value of {@link $host} parameter. */
-	public function getHost() {
-		return $this->host;
-	}
+    /** Returns the value of {@link $host} parameter. */
+    public function getHost() {
+        return $this->host;
+    }
 		
-	/** Sets the value of {@link $port} parameter. */
-	public function setPort($port) {
-		$this->setPositiveInteger('port', $port);
-	}
+    /** Sets the value of {@link $port} parameter. */
+    public function setPort($port) {
+        $this->setPositiveInteger('port', $port);
+    }
 		
-	/** Returns the value of {@link $port} parameter. */
-	public function getPort() {
-		return $this->port;
-	}
+    /** Returns the value of {@link $port} parameter. */
+    public function getPort() {
+        return $this->port;
+    }
 		
-	/** Sets the value of {@link $databaseName} parameter. */
-	public function setDatabaseName($databaseName) {
-		$this->setString('databaseName', $databaseName);
-	}
+    /** Sets the value of {@link $databaseName} parameter. */
+    public function setDatabaseName($databaseName) {
+        $this->setString('databaseName', $databaseName);
+    }
 		
-	/** Returns the value of {@link $databaseName} parameter. */
-	public function getDatabaseName() {
-		return $this->databaseName;
-	}
+    /** Returns the value of {@link $databaseName} parameter. */
+    public function getDatabaseName() {
+        return $this->databaseName;
+    }
 	
-	/** Sets the value of {@link $collectionName} parameter. */
-	public function setCollectionName($collectionName) {
-		$this->setString('collectionName', $collectionName);
-	}
+    /** Sets the value of {@link $collectionName} parameter. */
+    public function setCollectionName($collectionName) {
+        $this->setString('collectionName', $collectionName);
+    }
 		
-	/** Returns the value of {@link $collectionName} parameter. */
-	public function getCollectionName() {
-		return $this->collectionName;
-	}
+    /** Returns the value of {@link $collectionName} parameter. */
+    public function getCollectionName() {
+        return $this->collectionName;
+    }
 		
-	/** Sets the value of {@link $userName} parameter. */
-	public function setUserName($userName) {
-		$this->setString('userName', $userName, true);
-	}
+    /** Sets the value of {@link $userName} parameter. */
+    public function setUserName($userName) {
+        $this->setString('userName', $userName, true);
+    }
 	
-	/** Returns the value of {@link $userName} parameter. */
-	public function getUserName() {
-		return $this->userName;
-	}
+    /** Returns the value of {@link $userName} parameter. */
+    public function getUserName() {
+        return $this->userName;
+    }
 		
-	/** Sets the value of {@link $password} parameter. */
-	public function setPassword($password) {
-		$this->setString('password', $password, true);
-	}
+    /** Sets the value of {@link $password} parameter. */
+    public function setPassword($password) {
+        $this->setString('password', $password, true);
+    }
 		
-	/** Returns the value of {@link $password} parameter. */
-	public function getPassword() {
-		return $this->password;
-	}
+    /** Returns the value of {@link $password} parameter. */
+    public function getPassword() {
+        return $this->password;
+    }
 	
-	/** Sets the value of {@link $timeout} parameter. */
-	public function setTimeout($timeout) {
-		$this->setPositiveInteger('timeout', $timeout);
-	}
+    /** Sets the value of {@link $timeout} parameter. */
+    public function setTimeout($timeout) {
+        $this->setPositiveInteger('timeout', $timeout);
+    }
 
-	/** Returns the value of {@link $timeout} parameter. */
-	public function getTimeout() {
-		return $this->timeout;
-	}
-	/** 
-	 * Returns the mongodb connection.
-	 * @return Mongo
-	 */
-	public function getConnection() {
-		return $this->connection;
-	}
+    /** Returns the value of {@link $timeout} parameter. */
+    public function getTimeout() {
+        return $this->timeout;
+    }
+    /** 
+     * Returns the mongodb connection.
+     * @return Mongo
+     */
+    public function getConnection() {
+        return $this->connection;
+    }
 	
-	/** 
-	 * Returns the active mongodb collection.
-	 * @return MongoCollection
-	 */
-	public function getCollection() {
-		return $this->collection;
-	}
+    /** 
+     * Returns the active mongodb collection.
+     * @return MongoCollection
+     */
+    public function getCollection() {
+        return $this->collection;
+    }
 }
 ?>
\ No newline at end of file

Spacing +6 added lines, -6 removed lines

@@ -124,7 +124,7 @@ 
 	public function activateOptions() {
 		try {
 			$this->connection = new Mongo(sprintf('%s:%d', $this->host, $this->port), array("timeout" => $this->timeout));
-			$db	= $this->connection->selectDB($this->databaseName);
+			$db = $this->connection->selectDB($this->databaseName);
 			if ($this->userName !== null && $this->password !== null) {
 				$authResult = $db->authenticate($this->userName, $this->password);
 				if ($authResult['ok'] == floatval(0)) {
@@ -160,13 +160,13 @@ 
 	 * @return array The array representation of the logging event.
 	 */
 	protected function format(Payone_Log4php_LoggerLoggingEvent $event) {
-		$timestampSec  = (int) $event->getTimestamp();
-		$timestampUsec = (int) (($event->getTimestamp() - $timestampSec) * 1000000);
+		$timestampSec  = (int)$event->getTimestamp();
+		$timestampUsec = (int)(($event->getTimestamp() - $timestampSec) * 1000000);
 
 		$document = array(
 			'timestamp'  => new MongoDate($timestampSec, $timestampUsec),
 			'level'      => $event->getLevel()->toString(),
-			'thread'     => (int) $event->getThreadName(),
+			'thread'     => (int)$event->getThreadName(),
 			'message'    => $event->getMessage(),
 			'loggerName' => $event->getLoggerName() 
 		);	
@@ -175,7 +175,7 @@ 
 		if ($locationInfo != null) {
 			$document['fileName']   = $locationInfo->getFileName();
 			$document['method']     = $locationInfo->getMethodName();
-			$document['lineNumber'] = ($locationInfo->getLineNumber() == 'NA') ? 'NA' : (int) $locationInfo->getLineNumber();
+			$document['lineNumber'] = ($locationInfo->getLineNumber() == 'NA') ? 'NA' : (int)$locationInfo->getLineNumber();
 			$document['className']  = $locationInfo->getClassName();
 		}	
 
@@ -213,7 +213,7 @@ 
 	 * Closes the connection to the logging database
 	 */
 	public function close() {
-		if($this->closed != true) {
+		if ($this->closed != true) {
 			$this->collection = null;
 			if ($this->connection !== null) {
 				$this->connection->close();

lib/Payone/Log4php/LoggerRendererException.php

Indentation +8 added lines, -8 removed lines

@@ -27,15 +27,15 @@
  */
 class Payone_Log4php_LoggerRendererException implements Payone_Log4php_LoggerRendererObject {
 
-	public function render($o) {
-		$strRep  = 'Throwable('.get_class($o).'): '.$o->getMessage().' in '.$o->getFile().' on line '.$o->getLine();
-		$strRep .= PHP_EOL.$o->getTraceAsString();
+    public function render($o) {
+        $strRep  = 'Throwable('.get_class($o).'): '.$o->getMessage().' in '.$o->getFile().' on line '.$o->getLine();
+        $strRep .= PHP_EOL.$o->getTraceAsString();
 		
-		if (method_exists($o, 'getPrevious') && $o->getPrevious() !== null) {
-			$strRep .= PHP_EOL.'Caused by: '.$this->render($o->getPrevious());
-		}
+        if (method_exists($o, 'getPrevious') && $o->getPrevious() !== null) {
+            $strRep .= PHP_EOL.'Caused by: '.$this->render($o->getPrevious());
+        }
 		
-		return $strRep;		
-	}
+        return $strRep;		
+    }
 }
 ?>
\ No newline at end of file

Spacing +3 added lines, -3 removed lines

@@ -28,11 +28,11 @@
 class Payone_Log4php_LoggerRendererException implements Payone_Log4php_LoggerRendererObject {
 
 	public function render($o) {
-		$strRep  = 'Throwable('.get_class($o).'): '.$o->getMessage().' in '.$o->getFile().' on line '.$o->getLine();
-		$strRep .= PHP_EOL.$o->getTraceAsString();
+		$strRep  = 'Throwable(' . get_class($o) . '): ' . $o->getMessage() . ' in ' . $o->getFile() . ' on line ' . $o->getLine();
+		$strRep .= PHP_EOL . $o->getTraceAsString();
 		
 		if (method_exists($o, 'getPrevious') && $o->getPrevious() !== null) {
-			$strRep .= PHP_EOL.'Caused by: '.$this->render($o->getPrevious());
+			$strRep .= PHP_EOL . 'Caused by: ' . $this->render($o->getPrevious());
 		}
 		
 		return $strRep;		

lib/Payone/Log4php/LoggerPatternConverterMethod.php

Indentation +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@
  */
 class Payone_Log4php_LoggerPatternConverterMethod extends Payone_Log4php_LoggerPatternConverter {
 
-	public function convert(Payone_Log4php_LoggerLoggingEvent $event) {
-		return $event->getLocationInformation()->getMethodName();
-	}
+    public function convert(Payone_Log4php_LoggerLoggingEvent $event) {
+        return $event->getLocationInformation()->getMethodName();
+    }
 }

lib/Payone/Log4php/LoggerAppenderPDO.php

Indentation +201 added lines, -201 removed lines

@@ -49,246 +49,246 @@
  */
 class Payone_Log4php_LoggerAppenderPDO extends Payone_Log4php_LoggerAppender {
 
-	// ******************************************
-	// *** Configurable parameters            ***
-	// ******************************************
+    // ******************************************
+    // *** Configurable parameters            ***
+    // ******************************************
 	
-	/** 
-	 * DSN string used to connect to the database.
-	 * @see http://www.php.net/manual/en/pdo.construct.php
-	 */
-	protected $dsn;
+    /** 
+     * DSN string used to connect to the database.
+     * @see http://www.php.net/manual/en/pdo.construct.php
+     */
+    protected $dsn;
 
-	/** Database user name. */
-	protected $user;
+    /** Database user name. */
+    protected $user;
 	
-	/** Database password. */
-	protected $password;
+    /** Database password. */
+    protected $password;
 	
-	/** 
-	 * The insert query.
-	 * 
-	 * The __TABLE__ placeholder will be replaced by the table name from 
-	 * {@link $table}.
-	 *  
-	 * The questionmarks are part of the prepared statement, and they must 
-	 * match the number of conversion specifiers in {@link insertPattern}.
-	 */
-	protected $insertSQL = "INSERT INTO __TABLE__ (timestamp, logger, level, message, thread, file, line) VALUES (?, ?, ?, ?, ?, ?, ?)";
+    /** 
+     * The insert query.
+     * 
+     * The __TABLE__ placeholder will be replaced by the table name from 
+     * {@link $table}.
+     *  
+     * The questionmarks are part of the prepared statement, and they must 
+     * match the number of conversion specifiers in {@link insertPattern}.
+     */
+    protected $insertSQL = "INSERT INTO __TABLE__ (timestamp, logger, level, message, thread, file, line) VALUES (?, ?, ?, ?, ?, ?, ?)";
 
-	/** 
-	 * A comma separated list of {@link Payone_Log4php_LoggerPatternLayout} format strings
-	 * which replace the "?" in {@link $insertSQL}.
-	 * 
-	 * Must contain the same number of comma separated conversion patterns as 
-	 * there are question marks in {@link insertSQL}.
- 	 * 
- 	 * @see Payone_Log4php_LoggerPatternLayout For conversion patterns.
-	 */
-	protected $insertPattern = "%date{Y-m-d H:i:s},%logger,%level,%message,%pid,%file,%line";
+    /** 
+     * A comma separated list of {@link Payone_Log4php_LoggerPatternLayout} format strings
+     * which replace the "?" in {@link $insertSQL}.
+     * 
+     * Must contain the same number of comma separated conversion patterns as 
+     * there are question marks in {@link insertSQL}.
+     * 
+     * @see Payone_Log4php_LoggerPatternLayout For conversion patterns.
+     */
+    protected $insertPattern = "%date{Y-m-d H:i:s},%logger,%level,%message,%pid,%file,%line";
 
-	/** Name of the table to which to append log events. */
-	protected $table = 'log4php_log';
+    /** Name of the table to which to append log events. */
+    protected $table = 'log4php_log';
 	
-	/** The number of recconect attempts to make on failed append. */
-	protected $reconnectAttempts = 3;
+    /** The number of recconect attempts to make on failed append. */
+    protected $reconnectAttempts = 3;
 	
 	
-	// ******************************************
-	// *** Private memebers                   ***
-	// ******************************************
+    // ******************************************
+    // *** Private memebers                   ***
+    // ******************************************
 	
-	/** 
-	 * The PDO instance.
-	 * @var PDO 
-	 */
-	protected $db;
+    /** 
+     * The PDO instance.
+     * @var PDO 
+     */
+    protected $db;
 	
-	/** 
-	 * Prepared statement for the insert query.
-	 * @var PDOStatement 
-	 */
-	protected $preparedInsert;
+    /** 
+     * Prepared statement for the insert query.
+     * @var PDOStatement 
+     */
+    protected $preparedInsert;
 	
-	/** This appender does not require a layout. */
-	protected $requiresLayout = false;
+    /** This appender does not require a layout. */
+    protected $requiresLayout = false;
 	
 
-	// ******************************************
-	// *** Appender methods                   ***
-	// ******************************************
+    // ******************************************
+    // *** Appender methods                   ***
+    // ******************************************
 	
-	/**
-	 * Acquires a database connection based on parameters.
-	 * Parses the insert pattern to create a chain of converters which will be
-	 * used in forming query parameters from logging events.
-	 */
-	public function activateOptions() {
-		try {
-			$this->establishConnection();
-		} catch (PDOException $e) {
-			$this->warn("Failed connecting to database: " . $e->getMessage());
-			$this->close();
-			return;
-		}
+    /**
+     * Acquires a database connection based on parameters.
+     * Parses the insert pattern to create a chain of converters which will be
+     * used in forming query parameters from logging events.
+     */
+    public function activateOptions() {
+        try {
+            $this->establishConnection();
+        } catch (PDOException $e) {
+            $this->warn("Failed connecting to database: " . $e->getMessage());
+            $this->close();
+            return;
+        }
 
-		// Parse the insert patterns; pattern parts are comma delimited
-		$pieces = explode(',', $this->insertPattern);
-		$converterMap = Payone_Log4php_LoggerLayoutPattern::getDefaultConverterMap();
-		foreach($pieces as $pattern) {
-			$parser = new Payone_Log4php_LoggerPatternParser($pattern, $converterMap);
-			$this->converters[] = $parser->parse(); 
-		}
+        // Parse the insert patterns; pattern parts are comma delimited
+        $pieces = explode(',', $this->insertPattern);
+        $converterMap = Payone_Log4php_LoggerLayoutPattern::getDefaultConverterMap();
+        foreach($pieces as $pattern) {
+            $parser = new Payone_Log4php_LoggerPatternParser($pattern, $converterMap);
+            $this->converters[] = $parser->parse(); 
+        }
 		
-		$this->closed = false;
-	}
+        $this->closed = false;
+    }
 	
-	/** 
-	 * Connects to the database, and prepares the insert query.
-	 * @throws PDOException If connect or prepare fails.  
-	 */
-	protected function establishConnection() {
-		// Acquire database connection
-		$this->db = new PDO($this->dsn, $this->user, $this->password);
-		$this->db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
+    /** 
+     * Connects to the database, and prepares the insert query.
+     * @throws PDOException If connect or prepare fails.  
+     */
+    protected function establishConnection() {
+        // Acquire database connection
+        $this->db = new PDO($this->dsn, $this->user, $this->password);
+        $this->db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 		
-		// Prepare the insert statement
-		$insertSQL = str_replace('__TABLE__', $this->table, $this->insertSQL);
-		$this->preparedInsert = $this->db->prepare($insertSQL);
-	}
+        // Prepare the insert statement
+        $insertSQL = str_replace('__TABLE__', $this->table, $this->insertSQL);
+        $this->preparedInsert = $this->db->prepare($insertSQL);
+    }
 	
-	/**
-	 * Appends a new event to the database.
-	 * 
-	 * If writing to database fails, it will retry by re-establishing the 
-	 * connection up to $reconnectAttempts times. If writing still fails, 
-	 * the appender will close.
-	 */
-	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
+    /**
+     * Appends a new event to the database.
+     * 
+     * If writing to database fails, it will retry by re-establishing the 
+     * connection up to $reconnectAttempts times. If writing still fails, 
+     * the appender will close.
+     */
+    public function append(Payone_Log4php_LoggerLoggingEvent $event) {
 		
-		for ($attempt = 1; $attempt <= $this->reconnectAttempts; $attempt++) {
-			try {
-				// If it's a retry, reestablish the connection
-				if ($attempt > 1) {
-					$this->establishConnection();
-				}
+        for ($attempt = 1; $attempt <= $this->reconnectAttempts; $attempt++) {
+            try {
+                // If it's a retry, reestablish the connection
+                if ($attempt > 1) {
+                    $this->establishConnection();
+                }
 				
-				// Write to database
-				@$this->preparedInsert->execute($this->format($event));
-				@$this->preparedInsert->closeCursor();
-				break;
-			} catch (PDOException $e) {
-				$this->warn("Failed writing to database: ". $e->getMessage());
+                // Write to database
+                @$this->preparedInsert->execute($this->format($event));
+                @$this->preparedInsert->closeCursor();
+                break;
+            } catch (PDOException $e) {
+                $this->warn("Failed writing to database: ". $e->getMessage());
 				
-				// Close the appender if it's the last attempt
-				if ($attempt == $this->reconnectAttempts) {
-					$this->warn("Failed after {$this->reconnectAttempts} attempts. Closing appender.");
-					$this->close();
-				} else {
-					$this->warn("Attempting a reconnect (attempt $attempt of {$this->reconnectAttempts}).");
-				}
-			}
-		}
-	}
+                // Close the appender if it's the last attempt
+                if ($attempt == $this->reconnectAttempts) {
+                    $this->warn("Failed after {$this->reconnectAttempts} attempts. Closing appender.");
+                    $this->close();
+                } else {
+                    $this->warn("Attempting a reconnect (attempt $attempt of {$this->reconnectAttempts}).");
+                }
+            }
+        }
+    }
 	
-	/**
-	 * Converts the logging event to a series of database parameters by using 
-	 * the converter chain which was set up on activation. 
-	 */
-	protected function format(Payone_Log4php_LoggerLoggingEvent $event) {
-		$params = array();
-		foreach($this->converters as $converter) {
-			$buffer = '';
-			while ($converter !== null) {
-				$converter->format($buffer, $event);
-				$converter = $converter->next;
-			}
-			$params[] = $buffer;
-		}
-		return $params;
-	}
+    /**
+     * Converts the logging event to a series of database parameters by using 
+     * the converter chain which was set up on activation. 
+     */
+    protected function format(Payone_Log4php_LoggerLoggingEvent $event) {
+        $params = array();
+        foreach($this->converters as $converter) {
+            $buffer = '';
+            while ($converter !== null) {
+                $converter->format($buffer, $event);
+                $converter = $converter->next;
+            }
+            $params[] = $buffer;
+        }
+        return $params;
+    }
 	
-	/**
-	 * Closes the connection to the logging database
-	 */
-	public function close() {
-		// Close the connection (if any)
-		$this->db = null;
+    /**
+     * Closes the connection to the logging database
+     */
+    public function close() {
+        // Close the connection (if any)
+        $this->db = null;
 		
-		// Close the appender
-		$this->closed = true;
-	}
+        // Close the appender
+        $this->closed = true;
+    }
 	
-	// ******************************************
-	// *** Accessor methods                   ***
-	// ******************************************
+    // ******************************************
+    // *** Accessor methods                   ***
+    // ******************************************
 	
-	/**
-	 * Returns the active database handle or null if not established.
-	 * @return PDO
-	 */
-	public function getDatabaseHandle() {
-		return $this->db;
-	}
+    /**
+     * Returns the active database handle or null if not established.
+     * @return PDO
+     */
+    public function getDatabaseHandle() {
+        return $this->db;
+    }
 	
-	/** Sets the username. */
-	public function setUser($user) {
-		$this->setString('user', $user);
-	}
+    /** Sets the username. */
+    public function setUser($user) {
+        $this->setString('user', $user);
+    }
 	
-	/** Returns the username. */
-	public function getUser($user) {
-		return $this->user;
-	}
+    /** Returns the username. */
+    public function getUser($user) {
+        return $this->user;
+    }
 	
-	/** Sets the password. */
-	public function setPassword($password) {
-		$this->setString('password', $password);
-	}
+    /** Sets the password. */
+    public function setPassword($password) {
+        $this->setString('password', $password);
+    }
 	
-	/** Returns the password. */
-	public function getPassword($password) {
-		return $this->password;
-	}
+    /** Returns the password. */
+    public function getPassword($password) {
+        return $this->password;
+    }
 	
-	/** Sets the insert SQL. */
-	public function setInsertSQL($sql) {
-		$this->setString('insertSQL', $sql);
-	}
+    /** Sets the insert SQL. */
+    public function setInsertSQL($sql) {
+        $this->setString('insertSQL', $sql);
+    }
 	
-	/** Returns the insert SQL. */
-	public function getInsertSQL($sql) {
-		return $this->insertSQL;
-	}
+    /** Returns the insert SQL. */
+    public function getInsertSQL($sql) {
+        return $this->insertSQL;
+    }
 
-	/** Sets the insert pattern. */
-	public function setInsertPattern($pattern) {
-		$this->setString('insertPattern', $pattern);
-	}
+    /** Sets the insert pattern. */
+    public function setInsertPattern($pattern) {
+        $this->setString('insertPattern', $pattern);
+    }
 	
-	/** Returns the insert pattern. */
-	public function getInsertPattern($pattern) {
-		return $this->insertPattern;
-	}
+    /** Returns the insert pattern. */
+    public function getInsertPattern($pattern) {
+        return $this->insertPattern;
+    }
 
-	/** Sets the table name. */
-	public function setTable($table) {
-		$this->setString('table', $table);
-	}
+    /** Sets the table name. */
+    public function setTable($table) {
+        $this->setString('table', $table);
+    }
 	
-	/** Returns the table name. */
-	public function getTable($table) {
-		return $this->table;
-	}
+    /** Returns the table name. */
+    public function getTable($table) {
+        return $this->table;
+    }
 	
-	/** Sets the DSN string. */
-	public function setDSN($dsn) {
-		$this->setString('dsn', $dsn);
-	}
+    /** Sets the DSN string. */
+    public function setDSN($dsn) {
+        $this->setString('dsn', $dsn);
+    }
 	
-	/** Returns the DSN string. */
-	public function getDSN($dsn) {
-		return $this->setString('dsn', $dsn);
-	}	
+    /** Returns the DSN string. */
+    public function getDSN($dsn) {
+        return $this->setString('dsn', $dsn);
+    }	
 }
 

Spacing +3 added lines, -3 removed lines

@@ -135,7 +135,7 @@ 
 		// Parse the insert patterns; pattern parts are comma delimited
 		$pieces = explode(',', $this->insertPattern);
 		$converterMap = Payone_Log4php_LoggerLayoutPattern::getDefaultConverterMap();
-		foreach($pieces as $pattern) {
+		foreach ($pieces as $pattern) {
 			$parser = new Payone_Log4php_LoggerPatternParser($pattern, $converterMap);
 			$this->converters[] = $parser->parse(); 
 		}
@@ -178,7 +178,7 @@ 
 				@$this->preparedInsert->closeCursor();
 				break;
 			} catch (PDOException $e) {
-				$this->warn("Failed writing to database: ". $e->getMessage());
+				$this->warn("Failed writing to database: " . $e->getMessage());
 				
 				// Close the appender if it's the last attempt
 				if ($attempt == $this->reconnectAttempts) {
@@ -197,7 +197,7 @@ 
 	 */
 	protected function format(Payone_Log4php_LoggerLoggingEvent $event) {
 		$params = array();
-		foreach($this->converters as $converter) {
+		foreach ($this->converters as $converter) {
 			$buffer = '';
 			while ($converter !== null) {
 				$converter->format($buffer, $event);

lib/Payone/Log4php/LoggerPatternConverterLiteral.php

Indentation +7 added lines, -7 removed lines

@@ -26,13 +26,13 @@
  */
 class Payone_Log4php_LoggerPatternConverterLiteral extends Payone_Log4php_LoggerPatternConverter {
 
-	private $literalValue;
+    private $literalValue;
 	
-	public function __construct($literalValue) {
-		$this->literalValue = $literalValue;
-	}
+    public function __construct($literalValue) {
+        $this->literalValue = $literalValue;
+    }
 	
-	public function convert(Payone_Log4php_LoggerLoggingEvent $event) {
-		return $this->literalValue;
-	}
+    public function convert(Payone_Log4php_LoggerLoggingEvent $event) {
+        return $this->literalValue;
+    }
 }

lib/Payone/Log4php/LoggerPatternConverterNDC.php

Indentation +4 added lines, -4 removed lines

@@ -26,8 +26,8 @@
  */
 class Payone_Log4php_LoggerPatternConverterNDC extends Payone_Log4php_LoggerPatternConverter {
 
-	public function convert(Payone_Log4php_LoggerLoggingEvent $event) {
-		return $event->getNDC();
-	}
+    public function convert(Payone_Log4php_LoggerLoggingEvent $event) {
+        return $event->getNDC();
+    }
 }
- 
\ No newline at end of file
+    
\ No newline at end of file