jacobemerick / web

utility/Debugger.class.inc.php

Doc Comments +3 added lines

@@ -28,6 +28,9 @@
 		return self::$instance;

 	}

 

+	/**
+	 * @param string $message
+	 */
 	public static function log($message)

 	{

 		self::instance()->console->logError('Gah, this is using Debugger::log()!');

Indentation +25 added lines, -25 removed lines

@@ -18,8 +18,8 @@ 
 	private function __construct()
 	{
 		$this->console = new Console();
-    $this->profiler = new PhpQuickProfiler();
-    $this->profiler->setConsole($this->console);
+	$this->profiler = new PhpQuickProfiler();
+	$this->profiler->setConsole($this->console);
 	}
 
 	public static function instance()
@@ -82,29 +82,29 @@ 
 
 	public static function display()
 	{
-    if ($_COOKIE['debugger'] == 'display' && self::instance()->display) {
-      $pdo = '';
-      global $container;
-      if (!empty($container) && !empty($container['db_connection_locator'])) {
-        $pdo = $container['db_connection_locator']->getRead();
-      }
-      if (!empty($pdo)) {
-        $profiles = $pdo->getProfiler()->getProfiles();
-        $profiles = array_filter($profiles, function ($profile) {
-            return $profile['function'] == 'perform';
-        });
-        $profiles = array_map(function ($profile) {
-            return array(
-                'sql' => trim(preg_replace('/\s+/', ' ', $profile['statement'])),
-                'parameters' => $profile['bind_values'],
-                'time' => $profile['duration']
-            );
-        }, $profiles);
-        self::instance()->profiler->setProfiledQueries($profiles);
-      }
-      self::instance()->profiler->setDisplay(new Particletree\Pqp\Display());
-      self::instance()->profiler->display($pdo);
-    }
+	if ($_COOKIE['debugger'] == 'display' && self::instance()->display) {
+	  $pdo = '';
+	  global $container;
+	  if (!empty($container) && !empty($container['db_connection_locator'])) {
+		$pdo = $container['db_connection_locator']->getRead();
+	  }
+	  if (!empty($pdo)) {
+		$profiles = $pdo->getProfiler()->getProfiles();
+		$profiles = array_filter($profiles, function ($profile) {
+			return $profile['function'] == 'perform';
+		});
+		$profiles = array_map(function ($profile) {
+			return array(
+				'sql' => trim(preg_replace('/\s+/', ' ', $profile['statement'])),
+				'parameters' => $profile['bind_values'],
+				'time' => $profile['duration']
+			);
+		}, $profiles);
+		self::instance()->profiler->setProfiledQueries($profiles);
+	  }
+	  self::instance()->profiler->setDisplay(new Particletree\Pqp\Display());
+	  self::instance()->profiler->display($pdo);
+	}
 	}
 
 }

Spacing +8 added lines, -8 removed lines

@@ -8,7 +8,7 @@ 
 
 	private static $UNKNOWN_ERROR_FILE		= 'Unknown File';
 	private static $UNKNOWN_ERROR_LINE		= 'Unknown Line';
-	private static $UNKNOWN_ERROR_CONTEXT	= 'Unknown Context';
+	private static $UNKNOWN_ERROR_CONTEXT = 'Unknown Context';
 
 	private $profiler;
 	private $display = true;
@@ -24,7 +24,7 @@ 
 
 	public static function instance()
 	{
-		if(!isset(self::$instance))
+		if (!isset(self::$instance))
 			self::$instance = new Debugger();
 		return self::$instance;
 	}
@@ -52,11 +52,11 @@ 
 
 	public static function internal_error($code, $string, $file = null, $line = null, $context = null)
 	{
-		if($file == null)
+		if ($file == null)
 			$file = self::$UNKNOWN_ERROR_FILE;
-		if($line == null)
+		if ($line == null)
 			$line = self::$UNKNOWN_ERROR_LINE;
-		if($context == null)
+		if ($context == null)
 			$context = self::$UNKNOWN_ERROR_CONTEXT;
 		
 		self::instance()->console->logError(new Exception($string), "{$string}... TYPE: {$code}");
@@ -68,7 +68,7 @@ 
 	{
 		$error = error_get_last();
 		
-		if(isset($error))
+		if (isset($error))
 			self::internal_error($error['type'], $error['message'], $error['file'], $error['line']);
 		
 		self::display();
@@ -90,10 +90,10 @@ 
       }
       if (!empty($pdo)) {
         $profiles = $pdo->getProfiler()->getProfiles();
-        $profiles = array_filter($profiles, function ($profile) {
+        $profiles = array_filter($profiles, function($profile) {
             return $profile['function'] == 'perform';
         });
-        $profiles = array_map(function ($profile) {
+        $profiles = array_map(function($profile) {
             return array(
                 'sql' => trim(preg_replace('/\s+/', ' ', $profile['statement'])),
                 'parameters' => $profile['bind_values'],

Braces +15 added lines, -10 removed lines

@@ -24,8 +24,9 @@ 
 
 	public static function instance()
 	{
-		if(!isset(self::$instance))
-			self::$instance = new Debugger();
+		if(!isset(self::$instance)) {

+					self::$instance = new Debugger();

+		}

 		return self::$instance;
 	}
 
@@ -52,12 +53,15 @@ 
 
 	public static function internal_error($code, $string, $file = null, $line = null, $context = null)
 	{
-		if($file == null)
-			$file = self::$UNKNOWN_ERROR_FILE;
-		if($line == null)
-			$line = self::$UNKNOWN_ERROR_LINE;
-		if($context == null)
-			$context = self::$UNKNOWN_ERROR_CONTEXT;
+		if($file == null) {

+					$file = self::$UNKNOWN_ERROR_FILE;

+		}

+		if($line == null) {

+					$line = self::$UNKNOWN_ERROR_LINE;

+		}

+		if($context == null) {

+					$context = self::$UNKNOWN_ERROR_CONTEXT;

+		}

 		
 		self::instance()->console->logError(new Exception($string), "{$string}... TYPE: {$code}");
 		
@@ -68,8 +72,9 @@ 
 	{
 		$error = error_get_last();
 		
-		if(isset($error))
-			self::internal_error($error['type'], $error['message'], $error['file'], $error['line']);
+		if(isset($error)) {

+					self::internal_error($error['type'], $error['message'], $error['file'], $error['line']);

+		}

 		
 		self::display();
 		return true;

utility/Loader.class.inc.php

Doc Comments +10 added lines

@@ -69,6 +69,9 @@ 
 		return $extension;

 	}

 

+	/**
+	 * @param string $type
+	 */
 	public static function getImagePath($type, $file)

 	{

 		if($type == 'photo')

@@ -142,6 +145,10 @@ 
 		return new ReflectionClass($class_name);

 	}

 

+	/**
+	 * @param string $type
+	 * @param string $file
+	 */
 	public static function loadInstance($type, $file)

 	{

 		self::load($type, $file);

@@ -157,6 +164,9 @@ 
 		trigger_error("Requested class cannot be instance'd: {$type}, {$file}");

 	}

 

+	/**
+	 * @param string $type
+	 */
 	public static function loadNew($type, $file, $data = array())

 	{

 		self::load($type, $file);

Indentation +11 added lines, -11 removed lines

@@ -179,16 +179,16 @@
 		return self::instance()->is_live;
 	}
 
-    public static function getRootURL($site = '')
-    {
-        if (strlen($site) > 0) {
-            if ($site == 'waterfalls' && self::instance()->is_live) {
-                return 'http://www.waterfallsofthekeweenaw.com/';
-            } else {
-                return 'http://' . (self::instance()->is_live ? '' : 'dev.') . $site . '.jacobemerick.com/';
-            }
-        }
-        return '/';
-    }
+	public static function getRootURL($site = '')
+	{
+		if (strlen($site) > 0) {
+			if ($site == 'waterfalls' && self::instance()->is_live) {
+				return 'http://www.waterfallsofthekeweenaw.com/';
+			} else {
+				return 'http://' . (self::instance()->is_live ? '' : 'dev.') . $site . '.jacobemerick.com/';
+			}
+		}
+		return '/';
+	}
 
 }

Spacing +9 added lines, -9 removed lines

@@ -17,14 +17,14 @@ 
 
 	public static function instance()
 	{
-		if(!isset(self::$instance))
+		if (!isset(self::$instance))
 			self::$instance = new Loader();
 		return self::$instance;
 	}
 
 	private function get_root()
 	{
-		if(!isset($this->root))
+		if (!isset($this->root))
 		{
 			$current_directory = dirname(__FILE__);
 			$current_directory = substr($current_directory, 0, -7);
@@ -52,7 +52,7 @@ 
 
 	private static function get_extension($type)
 	{
-		switch($type)
+		switch ($type)
 		{
 			case 'collector' :
 			case 'controller' :
@@ -104,18 +104,18 @@ 
 
 	public static function load($type, $files, $data = array())
 	{
-		foreach((array) $files as $file)
+		foreach ((array) $files as $file)
 		{
 			$file_path = self::instance()->get_path($type, $file);
-			if(in_array($file_path, self::instance()->get_included_files()) && $type !== 'view')
+			if (in_array($file_path, self::instance()->get_included_files()) && $type !== 'view')
 				continue;
 			
-			if(!file_exists($file_path))
+			if (!file_exists($file_path))
 				Debugger::logMessage("Requested file does not exist: {$type}, {$file}");
 			
 			self::instance()->add_included_file($file_path);
 			
-			switch($type)
+			switch ($type)
 			{
 				case 'images' :
 				case 'scripts' :
@@ -145,7 +145,7 @@ 
 		
 		$reflectionObject = self::create_reflection_class($file);
 		
-		if(
+		if (
 			$reflectionObject->hasMethod('instance') &&
 			$reflectionObject->getMethod('instance')->isStatic())
 		{
@@ -160,7 +160,7 @@ 
 		
 		$reflectionObject = self::create_reflection_class($file);
 		
-		if($reflectionObject->hasMethod('__construct'))
+		if ($reflectionObject->hasMethod('__construct'))
 			return $reflectionObject->newInstanceArgs($data);
 		else
 			return $reflectionObject->newInstance();

Braces +14 added lines, -10 removed lines

@@ -17,8 +17,9 @@ 
 
 	public static function instance()
 	{
-		if(!isset(self::$instance))
-			self::$instance = new Loader();
+		if(!isset(self::$instance)) {

+					self::$instance = new Loader();

+		}

 		return self::$instance;
 	}
 
@@ -107,11 +108,13 @@ 
 		foreach((array) $files as $file)
 		{
 			$file_path = self::instance()->get_path($type, $file);
-			if(in_array($file_path, self::instance()->get_included_files()) && $type !== 'view')
-				continue;
+			if(in_array($file_path, self::instance()->get_included_files()) && $type !== 'view') {

+							continue;

+			}

 			
-			if(!file_exists($file_path))
-				Debugger::logMessage("Requested file does not exist: {$type}, {$file}");
+			if(!file_exists($file_path)) {

+							Debugger::logMessage("Requested file does not exist: {$type}, {$file}");

+			}

 			
 			self::instance()->add_included_file($file_path);
 			
@@ -160,10 +163,11 @@ 
 		
 		$reflectionObject = self::create_reflection_class($file);
 		
-		if($reflectionObject->hasMethod('__construct'))
-			return $reflectionObject->newInstanceArgs($data);
-		else
-			return $reflectionObject->newInstance();
+		if($reflectionObject->hasMethod('__construct')) {

+					return $reflectionObject->newInstanceArgs($data);

+		} else {

+					return $reflectionObject->newInstance();

+		}

 	}
 
 	public static function getRoot()

utility/Mail.class.inc.php

Doc Comments +9 added lines

@@ -18,6 +18,9 @@ 
 		$this->headers['Bcc'] = "Jacob <{$config->admin_email}>";

 	}

 

+	/**
+	 * @param string $person
+	 */
 	public function setToAddress($email, $person = null)

 	{

 		if($person)

@@ -27,12 +30,18 @@ 
 		return $this;

 	}

 

+	/**
+	 * @param string $subject
+	 */
 	public function setSubject($subject)

 	{

 		$this->subject = $subject;

 		return $this;

 	}

 

+	/**
+	 * @param string $message
+	 */
 	public function setMessage($message)

 	{

 		$this->message = $message;

Indentation +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 
 	public function __construct()
 	{
-        global $config;
+		global $config;
 		$this->headers['From'] = self::$FROM;
 		$this->headers['Reply-To'] = "Jacob <{$config->admin_email}>";
 		$this->headers['Bcc'] = "Jacob <{$config->admin_email}>";

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
 
 	public function setToAddress($email, $person = null)
 	{
-		if($person)
+		if ($person)
 			$this->to[] = "{$person} <{$email}>";
 		else
 			$this->to[] = "{$email}";
@@ -41,7 +41,7 @@ 
 
 	public function send()
 	{
-		if(!Loader::isLive())
+		if (!Loader::isLive())
 		{
 			Debugger::logMessage("Email Failed:\n\n{$this->message}");
 			return;
@@ -51,14 +51,14 @@ 
 		$subject = $this->subject;
 		$message = $this->message;
 		
-		if(empty($to) || empty($subject) || empty($message))
+		if (empty($to) || empty($subject) || empty($message))
 		{
 			Debugger::logMessage("Attempted to send an email without all of the required fields.");
 			return;
 		}
 		
 		$header = '';
-		foreach($this->headers as $key => $value)
+		foreach ($this->headers as $key => $value)
 		{
 			$header .= "{$key}: {$value}\r\n";
 		}

Braces +5 added lines, -4 removed lines

@@ -20,10 +20,11 @@
 
 	public function setToAddress($email, $person = null)
 	{
-		if($person)
-			$this->to[] = "{$person} <{$email}>";
-		else
-			$this->to[] = "{$email}";
+		if($person) {

+					$this->to[] = "{$person} <{$email}>";

+		} else {

+					$this->to[] = "{$email}";

+		}

 		return $this;
 	}
 

utility/Request.class.inc.php

Doc Comments +3 added lines

@@ -34,6 +34,9 @@
 		return false;

 	}

 

+	/**
+	 * @param string $key
+	 */
 	static function getGet($key = null)

 	{

 		if($key)

Spacing +7 added lines, -7 removed lines

@@ -18,9 +18,9 @@ 
 
 	static function getServer($key = null)
 	{
-		if($key)
+		if ($key)
 		{
-			if(isset(self::$server[$key]))
+			if (isset(self::$server[$key]))
 				return self::$server[$key];
 			return false;
 		}
@@ -29,16 +29,16 @@ 
 
 	static function isAjax()
 	{
-		if(self::getServer(self::$AJAX_REQUEST))
+		if (self::getServer(self::$AJAX_REQUEST))
 			return true;
 		return false;
 	}
 
 	static function getGet($key = null)
 	{
-		if($key)
+		if ($key)
 		{
-			if(isset(self::$get[$key]))
+			if (isset(self::$get[$key]))
 				return self::$get[$key];
 			return false;
 		}
@@ -47,9 +47,9 @@ 
 
 	static function getPost($key = null)
 	{
-		if($key)
+		if ($key)
 		{
-			if(isset(self::$post[$key]))
+			if (isset(self::$post[$key]))
 				return self::$post[$key];
 			return false;
 		}

Braces +12 added lines, -8 removed lines

@@ -20,8 +20,9 @@ 
 	{
 		if($key)
 		{
-			if(isset(self::$server[$key]))
-				return self::$server[$key];
+			if(isset(self::$server[$key])) {

+							return self::$server[$key];

+			}

 			return false;
 		}
 		return self::$server;
@@ -29,8 +30,9 @@ 
 
 	static function isAjax()
 	{
-		if(self::getServer(self::$AJAX_REQUEST))
-			return true;
+		if(self::getServer(self::$AJAX_REQUEST)) {

+					return true;

+		}

 		return false;
 	}
 
@@ -38,8 +40,9 @@ 
 	{
 		if($key)
 		{
-			if(isset(self::$get[$key]))
-				return self::$get[$key];
+			if(isset(self::$get[$key])) {

+							return self::$get[$key];

+			}

 			return false;
 		}
 		return self::$get;
@@ -49,8 +52,9 @@ 
 	{
 		if($key)
 		{
-			if(isset(self::$post[$key]))
-				return self::$post[$key];
+			if(isset(self::$post[$key])) {

+							return self::$post[$key];

+			}

 			return false;
 		}
 		return self::$post;

utility/Validate.class.inc.php

Doc Comments +5 added lines

@@ -72,6 +72,11 @@
 		return $matches[0] == $string;

 	}

 

+	/**
+	 * @param string $type
+	 * @param string $key
+	 * @param string $validation
+	 */
 	public static function checkRequest($type, $key, $validation, $strict = false)

 	{

 		switch($type)

Spacing +11 added lines, -11 removed lines

@@ -11,39 +11,39 @@ 
 
 	public static function isBoolean($value, $strict = false)
 	{
-		if($strict && ($value === true || $value === false))
+		if ($strict && ($value === true || $value === false))
 			return true;
-		if(!$strict && ((bool) $value === true || (bool) $value === false))
+		if (!$strict && ((bool) $value === true || (bool) $value === false))
 			return true;
 		return false;
 	}
 
 	public static function isDate($value)
 	{
-		if(strtotime($value) !== -1)
+		if (strtotime($value) !== -1)
 			return true;
-		if(date('y', $value) !== false)
+		if (date('y', $value) !== false)
 			return true;
 		return false;
 	}
 
 	public static function isInteger($value, $strict = false)
 	{
-		if($strict)
+		if ($strict)
 			return is_int($value);
 		return (int) $value == $value;
 	}
 
 	public static function isIP($value)
 	{
-		if(self::isInteger(ip2long($value)))
+		if (self::isInteger(ip2long($value)))
 			return true;
 		return false;
 	}
 
 	public static function isString($value, $strict = false)
 	{
-		if($strict)
+		if ($strict)
 			return is_string($value);
 		return (string) $value == $value;
 	}
@@ -67,14 +67,14 @@ 
 	private static function check_value($pattern, $string)
 	{
 		preg_match($pattern, $string, $matches);
-		if(empty($matches))
+		if (empty($matches))
 			return false;
 		return $matches[0] == $string;
 	}
 
 	public static function checkRequest($type, $key, $validation, $strict = false)
 	{
-		switch($type)
+		switch ($type)
 		{
 			case 'server':
 				$value = Request::getServer($key);
@@ -84,10 +84,10 @@ 
 			break;
 		}
 		
-		if($value == false)
+		if ($value == false)
 			return false;
 		
-		switch($validation)
+		switch ($validation)
 		{
 			case 'boolean':
 				return self::isBoolean($value, $strict);

Braces +27 added lines, -18 removed lines

@@ -11,40 +11,47 @@ 
 
 	public static function isBoolean($value, $strict = false)
 	{
-		if($strict && ($value === true || $value === false))
-			return true;
-		if(!$strict && ((bool) $value === true || (bool) $value === false))
-			return true;
+		if($strict && ($value === true || $value === false)) {

+					return true;

+		}

+		if(!$strict && ((bool) $value === true || (bool) $value === false)) {

+					return true;

+		}

 		return false;
 	}
 
 	public static function isDate($value)
 	{
-		if(strtotime($value) !== -1)
-			return true;
-		if(date('y', $value) !== false)
-			return true;
+		if(strtotime($value) !== -1) {

+					return true;

+		}

+		if(date('y', $value) !== false) {

+					return true;

+		}

 		return false;
 	}
 
 	public static function isInteger($value, $strict = false)
 	{
-		if($strict)
-			return is_int($value);
+		if($strict) {

+					return is_int($value);

+		}

 		return (int) $value == $value;
 	}
 
 	public static function isIP($value)
 	{
-		if(self::isInteger(ip2long($value)))
-			return true;
+		if(self::isInteger(ip2long($value))) {

+					return true;

+		}

 		return false;
 	}
 
 	public static function isString($value, $strict = false)
 	{
-		if($strict)
-			return is_string($value);
+		if($strict) {

+					return is_string($value);

+		}

 		return (string) $value == $value;
 	}
 
@@ -67,8 +74,9 @@ 
 	private static function check_value($pattern, $string)
 	{
 		preg_match($pattern, $string, $matches);
-		if(empty($matches))
-			return false;
+		if(empty($matches)) {

+					return false;

+		}

 		return $matches[0] == $string;
 	}
 
@@ -84,8 +92,9 @@ 
 			break;
 		}
 		
-		if($value == false)
-			return false;
+		if($value == false) {

+					return false;

+		}

 		
 		switch($validation)
 		{

collector/blog/IntroductionCollector.class.inc.php

Doc Comments +3 added lines

@@ -239,6 +239,9 @@
 		return $text;

 	}

 

+	/**
+	 * @param string $type
+	 */
 	private function get_replace_text($entity_type, $entity, $type)

 	{

 		switch($entity_type)

tests/unit/domain/blog/MysqlPostRepositoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     public function __construct()
     {
         $extendedPdo = $this->newExtendedPdo();
-        $this->connections = new ConnectionLocator(function () use ($extendedPdo) {
+        $this->connections = new ConnectionLocator(function() use ($extendedPdo) {
             return $extendedPdo;
         });
     }

Indentation +87 added lines, -87 removed lines

@@ -7,21 +7,21 @@ 
 class MysqlPostRepositoryTest extends PHPUnit_Framework_TestCase
 {
 
-    protected $connections;
-
-    public function __construct()
-    {
-        $extendedPdo = $this->newExtendedPdo();
-        $this->connections = new ConnectionLocator(function () use ($extendedPdo) {
-            return $extendedPdo;
-        });
-    }
-
-    protected function newExtendedPdo()
-    {
-        $extendedPdo = new ExtendedPdo('sqlite::memory:');
-        $extendedPdo->exec('ATTACH DATABASE `jpemeric_blog.db` AS `jpemeric_blog`');
-        $extendedPdo->exec("
+	protected $connections;
+
+	public function __construct()
+	{
+		$extendedPdo = $this->newExtendedPdo();
+		$this->connections = new ConnectionLocator(function () use ($extendedPdo) {
+			return $extendedPdo;
+		});
+	}
+
+	protected function newExtendedPdo()
+	{
+		$extendedPdo = new ExtendedPdo('sqlite::memory:');
+		$extendedPdo->exec('ATTACH DATABASE `jpemeric_blog.db` AS `jpemeric_blog`');
+		$extendedPdo->exec("
             CREATE TABLE IF NOT EXISTS `jpemeric_blog`.`post` (
               `id` integer PRIMARY KEY AUTOINCREMENT,
               `title` varchar(60) NOT NULL,
@@ -31,88 +31,88 @@ 
               `body` text NOT NULL,
               `display` integer NOT NULL
             )"
-        );
-        return $extendedPdo;
-    }
-
-    public function newMysqlPostRepository()
-    {
-        return new MysqlPostRepository($this->connections);
-    }
-
-    public function testConstructSetsConnections()
-    {
-        $this->assertAttributeEquals(
-            $this->connections,
-            'connections',
-            $this->newMysqlPostRepository()
-        );
-    }
-
-    public function testFindPostByPath()
-    {
-        $test_active_post = array(
-            'title'     => 'test findByPath active',
-            'path'      => 'test-findbypath-active',
-            'category'  => 'test-category',
-            'date'      => (new DateTime())->format('Y-m-d H:i:s'),
-            'body'      => 'test content',
-            'display'   => 1
-        );
-
-        $this->connections->getDefault()->perform("
+		);
+		return $extendedPdo;
+	}
+
+	public function newMysqlPostRepository()
+	{
+		return new MysqlPostRepository($this->connections);
+	}
+
+	public function testConstructSetsConnections()
+	{
+		$this->assertAttributeEquals(
+			$this->connections,
+			'connections',
+			$this->newMysqlPostRepository()
+		);
+	}
+
+	public function testFindPostByPath()
+	{
+		$test_active_post = array(
+			'title'     => 'test findByPath active',
+			'path'      => 'test-findbypath-active',
+			'category'  => 'test-category',
+			'date'      => (new DateTime())->format('Y-m-d H:i:s'),
+			'body'      => 'test content',
+			'display'   => 1
+		);
+
+		$this->connections->getDefault()->perform("
             INSERT INTO jpemeric_blog.post
                 (title, path, category, date, body, display)
             VALUES
                 (:title, :path, :category, :date, :body, :display)",
-            $test_active_post);
-
-        $active_post = $this->newMysqlPostRepository()->findPostByPath(
-            $test_active_post['category'],
-            $test_active_post['path']
-        );
-        $this->assertSame($test_active_post['title'], $active_post['title']);
-
-        $test_inactive_post = array(
-            'title'     => 'test findByPath inactive',
-            'path'      => 'test-findbypath-inactive',
-            'category'  => 'test-category',
-            'date'      => (new DateTime())->format('Y-m-d H:i:s'),
-            'body'      => 'test content',
-            'display'   => 0
-        );
-
-        $this->connections->getDefault()->perform("
+			$test_active_post);
+
+		$active_post = $this->newMysqlPostRepository()->findPostByPath(
+			$test_active_post['category'],
+			$test_active_post['path']
+		);
+		$this->assertSame($test_active_post['title'], $active_post['title']);
+
+		$test_inactive_post = array(
+			'title'     => 'test findByPath inactive',
+			'path'      => 'test-findbypath-inactive',
+			'category'  => 'test-category',
+			'date'      => (new DateTime())->format('Y-m-d H:i:s'),
+			'body'      => 'test content',
+			'display'   => 0
+		);
+
+		$this->connections->getDefault()->perform("
             INSERT INTO jpemeric_blog.post
                 (title, path, category, date, body, display)
             VALUES
                 (:title, :path, :category, :date, :body, :display)",
-            $test_inactive_post);
-
-        $inactive_post = $this->newMysqlPostRepository()->findPostByPath(
-            $test_inactive_post['category'],
-            $test_inactive_post['path']
-        );
-        $this->assertFalse($inactive_post);
-
-        $nonexistant_post = $this->newMysqlPostRepository()->findPostByPath(
-            'test-category',
-            'test-findbypath-nonexistant'
-        );
-        $this->assertFalse($nonexistant_post);
+			$test_inactive_post);
+
+		$inactive_post = $this->newMysqlPostRepository()->findPostByPath(
+			$test_inactive_post['category'],
+			$test_inactive_post['path']
+		);
+		$this->assertFalse($inactive_post);
+
+		$nonexistant_post = $this->newMysqlPostRepository()->findPostByPath(
+			'test-category',
+			'test-findbypath-nonexistant'
+		);
+		$this->assertFalse($nonexistant_post);
    }
 
-    public function testIsInstanceOfPostRepository()
-    {
-        $this->assertInstanceOf(
-            'Jacobemerick\Web\Domain\Blog\Post\PostRepositoryInterface',
-            $this->newMysqlPostRepository()
-        );
-    }
+	public function testIsInstanceOfPostRepository()
+	{
+		$this->assertInstanceOf(
+			'Jacobemerick\Web\Domain\Blog\Post\PostRepositoryInterface',
+			$this->newMysqlPostRepository()
+		);
+	}
 
-    public static function tearDownAfterClass()
-    {
+	public static function tearDownAfterClass()
+	{
 //        $this->connections->getDefault()->disconnect();
-        unlink('jpemeric_blog.db');
-    }
+		unlink('jpemeric_blog.db');
+	}
 }

utility/content/CleanCommentContent.class.inc.php

Doc Comments +8 added lines

@@ -36,6 +36,10 @@ 
 		$this->replace_element_patterns();

 	}

 

+	/**
+	 * @param string $pattern
+	 * @param string $replace
+	 */
 	private function process_element($pattern, $replace)

 	{

 		$match_count = preg_match_all($pattern, $this->content, $matches, PREG_SET_ORDER);

@@ -75,6 +79,10 @@ 
 		$this->content = strip_tags($this->content);

 	}

 

+	/**
+	 * @param string $pattern
+	 * @param string $replace
+	 */
 	private function link_unlinked_urls($pattern, $replace)

 	{

 		$match_count = preg_match_all($pattern, $this->content, $matches, PREG_SET_ORDER);

Spacing +5 added lines, -5 removed lines

@@ -40,10 +40,10 @@ 
 	{
 		$match_count = preg_match_all($pattern, $this->content, $matches, PREG_SET_ORDER);
 		
-		if($match_count < 1)
+		if ($match_count < 1)
 			return;
 		
-		foreach($matches as $match)
+		foreach ($matches as $match)
 		{
 			$full_match = array_shift($match);
 			$placeholder = $this->create_placeholder($full_match);
@@ -79,10 +79,10 @@ 
 	{
 		$match_count = preg_match_all($pattern, $this->content, $matches, PREG_SET_ORDER);
 		
-		if($match_count < 1)
+		if ($match_count < 1)
 			return;
 		
-		foreach($matches as $match)
+		foreach ($matches as $match)
 		{
 			$full_match = array_shift($match);
 			$full_match_pattern = $this->create_full_match_pattern($full_match);
@@ -99,7 +99,7 @@ 
 
 	private function replace_element_patterns()
 	{
-		foreach($this->replacement_array as $key => $replace)
+		foreach ($this->replacement_array as $key => $replace)
 		{
 			$this->content = str_replace($key, $replace, $this->content);
 		}

Braces +6 added lines, -4 removed lines

@@ -40,8 +40,9 @@ 
 	{
 		$match_count = preg_match_all($pattern, $this->content, $matches, PREG_SET_ORDER);
 		
-		if($match_count < 1)
-			return;
+		if($match_count < 1) {

+					return;

+		}

 		
 		foreach($matches as $match)
 		{
@@ -79,8 +80,9 @@ 
 	{
 		$match_count = preg_match_all($pattern, $this->content, $matches, PREG_SET_ORDER);
 		
-		if($match_count < 1)
-			return;
+		if($match_count < 1) {

+					return;

+		}

 		
 		foreach($matches as $match)
 		{

utility/content/FixInternalLinkContent.class.inc.php

Doc Comments +3 added lines

@@ -23,6 +23,9 @@
 		return;

 	}

 

+	/**
+	 * @param boolean $is_absolute
+	 */
 	private function get_link($string, $is_absolute, $anchor = '')

 	{

 		list($type, $uri) = explode('/', $string, 2);

Indentation +26 added lines, -26 removed lines

@@ -72,38 +72,38 @@
 				
 				break;
 			case 'falls' :
-                $pieces = explode('/', $uri);
-                if (count($pieces) == 1) {
-                    Loader::load('collector', 'waterfall/WatercourseCollector');
-                    list ($watercourse_alias) = $pieces;
-                    $watercourse = WatercourseCollector::getByAlias($watercourse_alias);
+				$pieces = explode('/', $uri);
+				if (count($pieces) == 1) {
+					Loader::load('collector', 'waterfall/WatercourseCollector');
+					list ($watercourse_alias) = $pieces;
+					$watercourse = WatercourseCollector::getByAlias($watercourse_alias);
                     
-                    if ($watercourse == null) {
-                        return;
-                    }
+					if ($watercourse == null) {
+						return;
+					}
                     
-                    $link .= ($is_absolute) ? Loader::getRootURL('waterfalls') : '/';
-                    $link .= "{$watercourse->alias}/";
+					$link .= ($is_absolute) ? Loader::getRootURL('waterfalls') : '/';
+					$link .= "{$watercourse->alias}/";
                     
-                    if ($anchor == '') {
-                        $anchor = $watercourse->name;
-                    }
-                } else if (count($pieces) == 2) {
-                    Loader::load('collector', 'waterfall/WaterfallCollector');
-                    list ($watercourse_alias, $waterfall_alias) = $pieces;
-                    $waterfall = WaterfallCollector::getByAlias($watercourse_alias, $waterfall_alias);
+					if ($anchor == '') {
+						$anchor = $watercourse->name;
+					}
+				} else if (count($pieces) == 2) {
+					Loader::load('collector', 'waterfall/WaterfallCollector');
+					list ($watercourse_alias, $waterfall_alias) = $pieces;
+					$waterfall = WaterfallCollector::getByAlias($watercourse_alias, $waterfall_alias);
                     
-                    if ($waterfall == null) {
-                        return;
-                    }
+					if ($waterfall == null) {
+						return;
+					}
                     
-                    $link .= ($is_absolute) ? Loader::getRootURL('waterfalls') : '/';
-                    $link .= "{$waterfall->watercourse_alias}/{$waterfall->alias}/";
+					$link .= ($is_absolute) ? Loader::getRootURL('waterfalls') : '/';
+					$link .= "{$waterfall->watercourse_alias}/{$waterfall->alias}/";
                     
-                    if ($anchor == '') {
-                        $anchor = $waterfall->name;
-                    }
-                }
+					if ($anchor == '') {
+						$anchor = $waterfall->name;
+					}
+				}
 				break;
 			default :
 				break;

Spacing +8 added lines, -8 removed lines

@@ -12,9 +12,9 @@ 
 	protected function execute($is_absolute = true)
 	{
 		preg_match_all(self::$LINK_PLACEHOLDER_MATCH, $this->content, $matches);
-		foreach($matches[1] as $key => $match)
+		foreach ($matches[1] as $key => $match)
 		{
-			if(isset($matches[3][$key]))
+			if (isset($matches[3][$key]))
 				$link_content = $this->get_link($match, $is_absolute, $matches[3][$key]);
 			else
 				$link_content = $this->get_link($match, $is_absolute);
@@ -29,19 +29,19 @@ 
 		
 		$link = '';
 		
-		switch($type)
+		switch ($type)
 		{
 			case 'blog' :
 				Loader::load('collector', 'blog/PostCollector');
 				$post = PostCollector::getPostByURI($uri);
 				
-				if($post === NULL)
+				if ($post === NULL)
 					return;
 				
 				$link .= ($is_absolute) ? Loader::getRootURL('blog') : '/';
 				$link .= "{$post->category}/{$post->path}/";
 				
-				if($anchor == '')
+				if ($anchor == '')
 					$anchor = $post->title;
 				
 				break;
@@ -49,7 +49,7 @@ 
 				$link .= ($is_absolute) ? Loader::getRootURL('blog') : '/';
 				$link .= "tag/{$uri}/";
 				
-				if($anchor == '')
+				if ($anchor == '')
 				{
 					$anchor = $uri;
 					$anchor = str_replace('-', ' ', $anchor);
@@ -61,13 +61,13 @@ 
 				Loader::load('collector', 'waterfall/LogCollector');
 				$log = LogCollector::getByAlias($uri);
 				
-				if($log === NULL)
+				if ($log === NULL)
 					return;
 				
 				$link .= ($is_absolute) ? Loader::getRootURL('waterfalls') : '/';
 				$link .= "journal/{$log->alias}/";
 				
-				if($anchor == '')
+				if ($anchor == '')
 					$anchor = $log->title;
 				
 				break;

Braces +17 added lines, -12 removed lines

@@ -14,10 +14,11 @@ 
 		preg_match_all(self::$LINK_PLACEHOLDER_MATCH, $this->content, $matches);
 		foreach($matches[1] as $key => $match)
 		{
-			if(isset($matches[3][$key]))
-				$link_content = $this->get_link($match, $is_absolute, $matches[3][$key]);
-			else
-				$link_content = $this->get_link($match, $is_absolute);
+			if(isset($matches[3][$key])) {

+							$link_content = $this->get_link($match, $is_absolute, $matches[3][$key]);

+			} else {

+							$link_content = $this->get_link($match, $is_absolute);

+			}

 			$this->content = str_replace($matches[0][$key], $link_content, $this->content);
 		}
 		return;
@@ -35,14 +36,16 @@ 
 				Loader::load('collector', 'blog/PostCollector');
 				$post = PostCollector::getPostByURI($uri);
 				
-				if($post === NULL)
-					return;
+				if($post === NULL) {

+									return;

+				}

 				
 				$link .= ($is_absolute) ? Loader::getRootURL('blog') : '/';
 				$link .= "{$post->category}/{$post->path}/";
 				
-				if($anchor == '')
-					$anchor = $post->title;
+				if($anchor == '') {

+									$anchor = $post->title;

+				}

 				
 				break;
 			case 'blog-tag' :
@@ -61,14 +64,16 @@ 
 				Loader::load('collector', 'waterfall/LogCollector');
 				$log = LogCollector::getByAlias($uri);
 				
-				if($log === NULL)
-					return;
+				if($log === NULL) {

+									return;

+				}

 				
 				$link .= ($is_absolute) ? Loader::getRootURL('waterfalls') : '/';
 				$link .= "journal/{$log->alias}/";
 				
-				if($anchor == '')
-					$anchor = $log->title;
+				if($anchor == '') {

+									$anchor = $log->title;

+				}

 				
 				break;
 			case 'falls' :