fjbender / magento-1

lib/Payone/Log4php/LoggerAppenderMail.php

Indentation +45 added lines, -46 removed lines

@@ -47,67 +47,66 @@
  *      Tue Sep  8 21:51:04 2009,120 [5485] FATAL root - Some critical message!
  *      Tue Sep  8 21:51:06 2009,120 [5485] FATAL root - Some more critical message!
  * </pre>
-
  * @version $Revision$
  * @package log4php
  * @subpackage appenders
  */
 class Payone_Log4php_LoggerAppenderMail extends Payone_Log4php_LoggerAppender {
 
-	/** @var string 'from' field */
-	protected $from = null;
+    /** @var string 'from' field */
+    protected $from = null;
 
-	/** @var string 'subject' field */
-	protected $subject = 'Log4php Report';
+    /** @var string 'subject' field */
+    protected $subject = 'Log4php Report';
 	
-	/** @var string 'to' field */
-	protected $to = null;
+    /** @var string 'to' field */
+    protected $to = null;
 
-	/** @var indiciates if this appender should run in dry mode */
-	protected $dry = false;
+    /** @var indiciates if this appender should run in dry mode */
+    protected $dry = false;
 
-	/** @var string used to create mail body */
-	protected $body = '';
+    /** @var string used to create mail body */
+    protected $body = '';
 	
-	public function close() {
-		if($this->closed != true) {
-			$from = $this->from;
-			$to = $this->to;
+    public function close() {
+        if($this->closed != true) {
+            $from = $this->from;
+            $to = $this->to;
 	
-			if(!empty($this->body) and $from !== null and $to !== null and $this->layout !== null) {
-				$subject = $this->subject;
-				if(!$this->dry) {
-					mail(
-						$to, $subject, 
-						$this->layout->getHeader() . $this->body . $this->layout->getFooter(),
-						"From: {$from}\r\n");
-				} else {
-				    echo "DRY MODE OF MAIL APP.: Send mail to: ".$to." with content: ".$this->body;
-				}
-			}
-			$this->closed = true;
-		}
-	}
+            if(!empty($this->body) and $from !== null and $to !== null and $this->layout !== null) {
+                $subject = $this->subject;
+                if(!$this->dry) {
+                    mail(
+                        $to, $subject, 
+                        $this->layout->getHeader() . $this->body . $this->layout->getFooter(),
+                        "From: {$from}\r\n");
+                } else {
+                    echo "DRY MODE OF MAIL APP.: Send mail to: ".$to." with content: ".$this->body;
+                }
+            }
+            $this->closed = true;
+        }
+    }
 	
-	public function setSubject($subject) {
-		$this->setString('subject', $subject);
-	}
+    public function setSubject($subject) {
+        $this->setString('subject', $subject);
+    }
 	
-	public function setTo($to) {
-		$this->setString('to', $to);
-	}
+    public function setTo($to) {
+        $this->setString('to', $to);
+    }
 
-	public function setFrom($from) {
-		$this->setString('from', $from);
-	}
+    public function setFrom($from) {
+        $this->setString('from', $from);
+    }
 
-	public function setDry($dry) {
-		$this->setBoolean('dry', $dry);
-	}
+    public function setDry($dry) {
+        $this->setBoolean('dry', $dry);
+    }
 	
-	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->layout !== null) {
-			$this->body .= $this->layout->format($event);
-		}
-	}
+    public function append(Payone_Log4php_LoggerLoggingEvent $event) {
+        if($this->layout !== null) {
+            $this->body .= $this->layout->format($event);
+        }
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -70,19 +70,19 @@ 
 	protected $body = '';
 	
 	public function close() {
-		if($this->closed != true) {
+		if ($this->closed != true) {
 			$from = $this->from;
 			$to = $this->to;
 	
-			if(!empty($this->body) and $from !== null and $to !== null and $this->layout !== null) {
+			if (!empty($this->body) and $from !== null and $to !== null and $this->layout !== null) {
 				$subject = $this->subject;
-				if(!$this->dry) {
+				if (!$this->dry) {
 					mail(
 						$to, $subject, 
 						$this->layout->getHeader() . $this->body . $this->layout->getFooter(),
 						"From: {$from}\r\n");
 				} else {
-				    echo "DRY MODE OF MAIL APP.: Send mail to: ".$to." with content: ".$this->body;
+				    echo "DRY MODE OF MAIL APP.: Send mail to: " . $to . " with content: " . $this->body;
 				}
 			}
 			$this->closed = true;
@@ -106,7 +106,7 @@ 
 	}
 	
 	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->layout !== null) {
+		if ($this->layout !== null) {
 			$this->body .= $this->layout->format($event);
 		}
 	}

lib/Payone/Log4php/LoggerAppenderSocket.php

Indentation +82 added lines, -82 removed lines

@@ -36,87 +36,87 @@
  */ 
 class Payone_Log4php_LoggerAppenderSocket extends Payone_Log4php_LoggerAppender {
 	
-	/** 
-	 * Target host.
-	 * @see http://php.net/manual/en/function.fsockopen.php 
-	 */
-	protected $remoteHost;
-	
-	/** Target port */
-	protected $port = 4446;
-	
-	/** Connection timeout in ms. */
-	protected $timeout;
-	
-	// ******************************************
-	// *** Appender methods                   ***
-	// ******************************************
-	
-	/** Override the default layout to use serialized. */
-	public function getDefaultLayout() {
-		return new Payone_Log4php_LoggerLayoutSerialized();
-	}
-	
-	public function activateOptions() {
-		if (empty($this->remoteHost)) {
-			$this->warn("Required parameter [remoteHost] not set. Closing appender.");
-			$this->closed = true;
-			return;
-		}
-	
-		if (empty($this->timeout)) {
-			$this->timeout = ini_get("default_socket_timeout");
-		}
-	
-		$this->closed = false;
-	}
-	
-	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
-		$socket = fsockopen($this->remoteHost, $this->port, $errno, $errstr, $this->timeout);
-		if ($socket === false) {
-			$this->warn("Could not open socket to {$this->remoteHost}:{$this->port}. Closing appender.");
-			$this->closed = true;
-			return;
-		}
-	
-		if (false === fwrite($socket, $this->layout->format($event))) {
-			$this->warn("Error writing to socket. Closing appender.");
-			$this->closed = true;
-		}
-		fclose($socket);
-	}
-	
-	// ******************************************
-	// *** Accessor methods                   ***
-	// ******************************************
-	
-	/** Sets the target host. */
-	public function setRemoteHost($hostname) {
-		$this->setString('remoteHost', $hostname);
-	}
-	
-	/** Sets the target port */
-	public function setPort($port) {
-		$this->setPositiveInteger('port', $port);
-	}
+    /** 
+     * Target host.
+     * @see http://php.net/manual/en/function.fsockopen.php 
+     */
+    protected $remoteHost;
+	
+    /** Target port */
+    protected $port = 4446;
+	
+    /** Connection timeout in ms. */
+    protected $timeout;
+	
+    // ******************************************
+    // *** Appender methods                   ***
+    // ******************************************
+	
+    /** Override the default layout to use serialized. */
+    public function getDefaultLayout() {
+        return new Payone_Log4php_LoggerLayoutSerialized();
+    }
+	
+    public function activateOptions() {
+        if (empty($this->remoteHost)) {
+            $this->warn("Required parameter [remoteHost] not set. Closing appender.");
+            $this->closed = true;
+            return;
+        }
+	
+        if (empty($this->timeout)) {
+            $this->timeout = ini_get("default_socket_timeout");
+        }
+	
+        $this->closed = false;
+    }
+	
+    public function append(Payone_Log4php_LoggerLoggingEvent $event) {
+        $socket = fsockopen($this->remoteHost, $this->port, $errno, $errstr, $this->timeout);
+        if ($socket === false) {
+            $this->warn("Could not open socket to {$this->remoteHost}:{$this->port}. Closing appender.");
+            $this->closed = true;
+            return;
+        }
+	
+        if (false === fwrite($socket, $this->layout->format($event))) {
+            $this->warn("Error writing to socket. Closing appender.");
+            $this->closed = true;
+        }
+        fclose($socket);
+    }
+	
+    // ******************************************
+    // *** Accessor methods                   ***
+    // ******************************************
+	
+    /** Sets the target host. */
+    public function setRemoteHost($hostname) {
+        $this->setString('remoteHost', $hostname);
+    }
+	
+    /** Sets the target port */
+    public function setPort($port) {
+        $this->setPositiveInteger('port', $port);
+    }
 	 
-	/** Sets the timeout. */
-	public function setTimeout($timeout) {
-		$this->setPositiveInteger('timeout', $timeout);
-	}
-	
-	/** Returns the target host. */
-	public function getRemoteHost() {
-		return $this->getRemoteHost();
-	}
-	
-	/** Returns the target port. */
-	public function getPort() {
-		return $this->port;
-	}
-	
-	/** Returns the timeout */
-	public function getTimeout() {
-		return $this->timeout;
-	}
+    /** Sets the timeout. */
+    public function setTimeout($timeout) {
+        $this->setPositiveInteger('timeout', $timeout);
+    }
+	
+    /** Returns the target host. */
+    public function getRemoteHost() {
+        return $this->getRemoteHost();
+    }
+	
+    /** Returns the target port. */
+    public function getPort() {
+        return $this->port;
+    }
+	
+    /** Returns the timeout */
+    public function getTimeout() {
+        return $this->timeout;
+    }
 }

lib/Payone/Log4php/LoggerAppenderDailyFile.php

Indentation +50 added lines, -50 removed lines

@@ -52,59 +52,59 @@
  */
 class Payone_Log4php_LoggerAppenderDailyFile extends Payone_Log4php_LoggerAppenderFile {
 
-	/**
-	 * Format date. 
-	 * It follows the {@link PHP_MANUAL#date()} formatting rules and <b>should always be set before {@link $file} param</b>.
-	 * @var string
-	 */
-	protected $datePattern = "Ymd";
+    /**
+     * Format date. 
+     * It follows the {@link PHP_MANUAL#date()} formatting rules and <b>should always be set before {@link $file} param</b>.
+     * @var string
+     */
+    protected $datePattern = "Ymd";
 	
-	/**
-	 * Sets date format for the file name.
-	 * @param string $datePattern a regular date() string format
-	 */
-	public function setDatePattern($datePattern) {
-		$this->setString('datePattern', $datePattern);
-	}
+    /**
+     * Sets date format for the file name.
+     * @param string $datePattern a regular date() string format
+     */
+    public function setDatePattern($datePattern) {
+        $this->setString('datePattern', $datePattern);
+    }
 	
-	/**
-	 * @return string returns date format for the filename
-	 */
-	public function getDatePattern() {
-		return $this->datePattern;
-	}
+    /**
+     * @return string returns date format for the filename
+     */
+    public function getDatePattern() {
+        return $this->datePattern;
+    }
 	
-	/** 
-	 * Similar to parent method, but but replaces "%s" in the file name with 
-	 * the current date in format specified by the 'datePattern' parameter.
-	 */ 
-	public function activateOptions() {
-		$fileName = $this->getFile();
-		$date = date($this->getDatePattern());
-		$fileName = sprintf($fileName, $date);
+    /** 
+     * Similar to parent method, but but replaces "%s" in the file name with 
+     * the current date in format specified by the 'datePattern' parameter.
+     */ 
+    public function activateOptions() {
+        $fileName = $this->getFile();
+        $date = date($this->getDatePattern());
+        $fileName = sprintf($fileName, $date);
 		
-		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;
+        }
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -83,17 +83,17 @@
 		$date = date($this->getDatePattern());
 		$fileName = sprintf($fileName, $date);
 		
-		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());

lib/Payone/Log4php/LoggerPatternConverterLevel.php

Indentation +3 added lines, -3 removed lines

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

lib/Payone/Log4php/LoggerPatternConverterRequest.php

Indentation +1 added lines, -1 removed lines

@@ -29,5 +29,5 @@
  * @subpackage pattern
  */
 class Payone_Log4php_LoggerPatternConverterRequest extends Payone_Log4php_LoggerPatternConverterSuperglobal {
-	protected $name = '_REQUEST';
+    protected $name = '_REQUEST';
 }
\ No newline at end of file

lib/Payone/Log4php/LoggerThrowableInformation.php

Indentation +39 added lines, -39 removed lines

@@ -26,51 +26,51 @@
  */
 class Payone_Log4php_LoggerThrowableInformation {
 	
-	/** @var Exception Throwable to log */
-	private $throwable;
+    /** @var Exception Throwable to log */
+    private $throwable;
 	
-	/** @var array Array of throwable messages */
-	private $throwableArray;
+    /** @var array Array of throwable messages */
+    private $throwableArray;
 	
-	/** @var Logger reference */
-	private $logger;
+    /** @var Logger reference */
+    private $logger;
 	
-	/**
-	 * Create a new instance
-	 * 
-	 * @param $throwable - a throwable as a exception
-	 * @param $logger - Logger reference
-	 */
-	public function __construct(Exception $throwable)  {
-		$this->throwable = $throwable;
-	}
+    /**
+     * Create a new instance
+     * 
+     * @param $throwable - a throwable as a exception
+     * @param $logger - Logger reference
+     */
+    public function __construct(Exception $throwable)  {
+        $this->throwable = $throwable;
+    }
 	
-	/**
-	* Return source exception
-	* 
-	* @return Exception
-	*/
-	public function getThrowable() {
-		return $this->throwable;
-	}
+    /**
+     * Return source exception
+     * 
+     * @return Exception
+     */
+    public function getThrowable() {
+        return $this->throwable;
+    }
 	
-	/**
-	 * @desc Returns string representation of throwable
-	 * 
-	 * @return array 
-	 */
-	public function getStringRepresentation() {
-		if (!is_array($this->throwableArray)) {
-			$renderer = Payone_Log4php_Logger::getHierarchy()->getRendererMap()->getByClassName(get_class($this->throwable));
+    /**
+     * @desc Returns string representation of throwable
+     * 
+     * @return array 
+     */
+    public function getStringRepresentation() {
+        if (!is_array($this->throwableArray)) {
+            $renderer = Payone_Log4php_Logger::getHierarchy()->getRendererMap()->getByClassName(get_class($this->throwable));
 			
-			// TODO: why this?
-			if ($renderer instanceof Payone_Log4php_LoggerRendererDefault) {
-				$renderer = new Payone_Log4php_LoggerRendererException();
-			}
-			$this->throwableArray = explode("\n", $renderer->render($this->throwable));
-		}
+            // TODO: why this?
+            if ($renderer instanceof Payone_Log4php_LoggerRendererDefault) {
+                $renderer = new Payone_Log4php_LoggerRendererException();
+            }
+            $this->throwableArray = explode("\n", $renderer->render($this->throwable));
+        }
 		
-		return $this->throwableArray;
-	}
+        return $this->throwableArray;
+    }
 }
 ?>
\ No newline at end of file

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 	 * @param $throwable - a throwable as a exception
 	 * @param $logger - Logger reference
 	 */
-	public function __construct(Exception $throwable)  {
+	public function __construct(Exception $throwable) {
 		$this->throwable = $throwable;
 	}
 	

lib/Payone/Log4php/LoggerPatternConverterMessage.php

Indentation +3 added lines, -3 removed lines

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

lib/Payone/Log4php/LoggerAppenderEcho.php

Indentation +34 added lines, -34 removed lines

@@ -41,45 +41,45 @@
  * @subpackage appenders
  */
 class Payone_Log4php_LoggerAppenderEcho extends Payone_Log4php_LoggerAppender {
-	/** boolean used internally to mark first append */
-	protected $firstAppend = true;
+    /** boolean used internally to mark first append */
+    protected $firstAppend = true;
 	
-	/** 
-	 * If set to true, a <br /> element will be inserted before each line
-	 * break in the logged message. Default value is false. @var boolean 
-	 */
-	protected $htmlLineBreaks = false;
+    /** 
+     * If set to true, a <br /> element will be inserted before each line
+     * break in the logged message. Default value is false. @var boolean 
+     */
+    protected $htmlLineBreaks = false;
 	
-	public function close() {
-		if($this->closed != true) {
-			if(!$this->firstAppend) {
-				echo $this->layout->getFooter();
-			}
-		}
-		$this->closed = true;
-	}
+    public function close() {
+        if($this->closed != true) {
+            if(!$this->firstAppend) {
+                echo $this->layout->getFooter();
+            }
+        }
+        $this->closed = true;
+    }
 
-	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->layout !== null) {
-			if($this->firstAppend) {
-				echo $this->layout->getHeader();
-				$this->firstAppend = false;
-			}
-			$text = $this->layout->format($event);
+    public function append(Payone_Log4php_LoggerLoggingEvent $event) {
+        if($this->layout !== null) {
+            if($this->firstAppend) {
+                echo $this->layout->getHeader();
+                $this->firstAppend = false;
+            }
+            $text = $this->layout->format($event);
 			
-			if ($this->htmlLineBreaks) {
-				$text = nl2br($text);
-			}
-			echo $text;
-		} 
-	}
+            if ($this->htmlLineBreaks) {
+                $text = nl2br($text);
+            }
+            echo $text;
+        } 
+    }
 	
-	public function setHtmlLineBreaks($value) {
-		$this->setBoolean('htmlLineBreaks', $value);
-	}
+    public function setHtmlLineBreaks($value) {
+        $this->setBoolean('htmlLineBreaks', $value);
+    }
 
-	public function getHtmlLineBreaks() {
-		return $this->htmlLineBreaks;
-	}
+    public function getHtmlLineBreaks() {
+        return $this->htmlLineBreaks;
+    }
 }
 

Spacing +4 added lines, -4 removed lines

@@ -51,8 +51,8 @@ 
 	protected $htmlLineBreaks = false;
 	
 	public function close() {
-		if($this->closed != true) {
-			if(!$this->firstAppend) {
+		if ($this->closed != true) {
+			if (!$this->firstAppend) {
 				echo $this->layout->getFooter();
 			}
 		}
@@ -60,8 +60,8 @@ 
 	}
 
 	public function append(Payone_Log4php_LoggerLoggingEvent $event) {
-		if($this->layout !== null) {
-			if($this->firstAppend) {
+		if ($this->layout !== null) {
+			if ($this->firstAppend) {
 				echo $this->layout->getHeader();
 				$this->firstAppend = false;
 			}

lib/Payone/Log4php/LoggerFilterDenyAll.php

Indentation +10 added lines, -10 removed lines

@@ -43,14 +43,14 @@
  */
 class Payone_Log4php_LoggerFilterDenyAll extends Payone_Log4php_LoggerFilter {
 
-	/**
-	 * Always returns the integer constant {@link LoggerFilter::DENY}
-	 * regardless of the {@link LoggerLoggingEvent} parameter.
-	 * 
-	 * @param Payone_Log4php_LoggerLoggingEvent $event The {@link LoggerLoggingEvent} to filter.
-	 * @return LoggerFilter::DENY Always returns {@link LoggerFilter::DENY}
-	 */
-	public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
-		return Payone_Log4php_LoggerFilter::DENY;
-	}
+    /**
+     * Always returns the integer constant {@link LoggerFilter::DENY}
+     * regardless of the {@link LoggerLoggingEvent} parameter.
+     * 
+     * @param Payone_Log4php_LoggerLoggingEvent $event The {@link LoggerLoggingEvent} to filter.
+     * @return LoggerFilter::DENY Always returns {@link LoggerFilter::DENY}
+     */
+    public function decide(Payone_Log4php_LoggerLoggingEvent $event) {
+        return Payone_Log4php_LoggerFilter::DENY;
+    }
 }