jacobemerick / web

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/URLDecode.class.inc.php

Doc Comments +6 added lines

@@ -38,6 +38,9 @@ 
 		self::$array['pieces'] = (array) $uri_array;

 	}

 

+	/**
+	 * @return string
+	 */
 	static function getSite()

 	{

 		return self::$array['site'];

@@ -66,6 +69,9 @@ 
 		return substr($file, strrpos($file, '.') + 1);;

 	}

 

+	/**
+	 * @param integer $piece
+	 */
 	static function getPiece($piece = null)

 	{

 		if(!$piece)

Spacing +8 added lines, -8 removed lines

@@ -15,11 +15,11 @@ 
 	static private function form_url_array($host, $uri)
 	{
 		$uri = substr($uri, 1);
-		if(strpos($uri, '?'))
+		if (strpos($uri, '?'))
 			$uri = substr($uri, 0, strpos($uri, '?'));
 		$uri_array = explode('/', $uri);
 		
-		if(!Loader::isLive())
+		if (!Loader::isLive())
 			$host = substr($host, strpos($host, '.') + 1);
 		
 		self::$array['host'] = $host;
@@ -33,7 +33,7 @@ 
 		self::$array['base'] = 'http://' . (!Loader::isLive() ? 'dev.' : '') . $host . '/';
 		self::$array['uri'] = '/' . implode('/', $uri_array);
 		
-		if(end($uri_array) == '')
+		if (end($uri_array) == '')
 			$uri_array = array_slice($uri_array, 0, count($uri_array) - 1);
 		self::$array['pieces'] = (array) $uri_array;
 	}
@@ -61,21 +61,21 @@ 
 	static function getExtension()
 	{
 		$file = self::getPiece(-1);
-		if(substr($file, -1) == '/')
+		if (substr($file, -1) == '/')
 			return false;
-		return substr($file, strrpos($file, '.') + 1);;
+		return substr($file, strrpos($file, '.') + 1); ;
 	}
 
 	static function getPiece($piece = null)
 	{
-		if(!$piece)
+		if (!$piece)
 			return self::$array['pieces'];
 		
-		if($piece == -1)
+		if ($piece == -1)
 			return end(self::$array['pieces']);
 		
 		$piece = $piece - 1;
-		if(array_key_exists($piece, self::$array['pieces']))
+		if (array_key_exists($piece, self::$array['pieces']))
 			return self::$array['pieces'][$piece];
 		return;
 	}

Braces +21 added lines, -14 removed lines

@@ -15,12 +15,14 @@ 
 	static private function form_url_array($host, $uri)
 	{
 		$uri = substr($uri, 1);
-		if(strpos($uri, '?'))
-			$uri = substr($uri, 0, strpos($uri, '?'));
+		if(strpos($uri, '?')) {

+					$uri = substr($uri, 0, strpos($uri, '?'));

+		}

 		$uri_array = explode('/', $uri);
 		
-		if(!Loader::isLive())
-			$host = substr($host, strpos($host, '.') + 1);
+		if(!Loader::isLive()) {

+					$host = substr($host, strpos($host, '.') + 1);

+		}

 		
 		self::$array['host'] = $host;
 		
@@ -33,8 +35,9 @@ 
 		self::$array['base'] = 'http://' . (!Loader::isLive() ? 'dev.' : '') . $host . '/';
 		self::$array['uri'] = '/' . implode('/', $uri_array);
 		
-		if(end($uri_array) == '')
-			$uri_array = array_slice($uri_array, 0, count($uri_array) - 1);
+		if(end($uri_array) == '') {

+					$uri_array = array_slice($uri_array, 0, count($uri_array) - 1);

+		}

 		self::$array['pieces'] = (array) $uri_array;
 	}
 
@@ -61,22 +64,26 @@ 
 	static function getExtension()
 	{
 		$file = self::getPiece(-1);
-		if(substr($file, -1) == '/')
-			return false;
+		if(substr($file, -1) == '/') {

+					return false;

+		}

 		return substr($file, strrpos($file, '.') + 1);;
 	}
 
 	static function getPiece($piece = null)
 	{
-		if(!$piece)
-			return self::$array['pieces'];
+		if(!$piece) {

+					return self::$array['pieces'];

+		}

 		
-		if($piece == -1)
-			return end(self::$array['pieces']);
+		if($piece == -1) {

+					return end(self::$array['pieces']);

+		}

 		
 		$piece = $piece - 1;
-		if(array_key_exists($piece, self::$array['pieces']))
-			return self::$array['pieces'][$piece];
+		if(array_key_exists($piece, self::$array['pieces'])) {

+					return self::$array['pieces'][$piece];

+		}

 		return;
 	}
 

Indentation +10 added lines, -10 removed lines

@@ -24,21 +24,21 @@
 		
 		self::$array['host'] = $host;
 		
-    if (
-      $host == 'www.waterfallsofthekeweenaw.com' ||
-      $host == 'waterfallsofthekeweenaw.com'
-    ) {
+	if (
+	  $host == 'www.waterfallsofthekeweenaw.com' ||
+	  $host == 'waterfallsofthekeweenaw.com'
+	) {
 			self::$array['site'] = 'waterfalls';
 		} else {
 			self::$array['site'] = substr($host, 0, strpos($host, '.'));
 		}
 
-    $base = '';
-    $base .= (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') ? 'https' : 'http';
-    $base .= '://';
-    $base .= (!Loader::isLive()) ? 'dev' : '';
-    $base .= $host;
-    $base .= '/';
+	$base = '';
+	$base .= (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') ? 'https' : 'http';
+	$base .= '://';
+	$base .= (!Loader::isLive()) ? 'dev' : '';
+	$base .= $host;
+	$base .= '/';
 
 		self::$array['base'] = $base;
 		self::$array['uri'] = '/' . implode('/', $uri_array);

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' :

src/Domain/Blog/Post/MysqlPostRepository.php

Doc Comments +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
     protected $connections;
 
     /**
-     * @param Aura\Sql\ConnectionLocator $connections
+     * @param ConnectionLocator $connections
      */
     public function __construct(ConnectionLocator $connections)
     {
@@ -22,7 +22,7 @@ 
      * @param string $category
      * @param string $path
      *
-     * @return array|false
+     * @return boolean
      */
     public function findByPath($category, $path)
     {

Indentation +136 added lines, -136 removed lines

@@ -7,165 +7,165 @@ 
 class MysqlPostRepository implements PostRepository
 {
 
-    /** @var  Aura\Sql\ConnectionLocator */
-    protected $connections;
-
-    /**
-     * @param Aura\Sql\ConnectionLocator $connections
-     */
-    public function __construct(ConnectionLocator $connections)
-    {
-        $this->connections = $connections;
-    }
-
-    /**
-     * @param string $category
-     * @param string $path
-     *
-     * @return array|false
-     */
-    public function findPostByPath($category, $path)
-    {
-        $query = "
+	/** @var  Aura\Sql\ConnectionLocator */
+	protected $connections;
+
+	/**
+	 * @param Aura\Sql\ConnectionLocator $connections
+	 */
+	public function __construct(ConnectionLocator $connections)
+	{
+		$this->connections = $connections;
+	}
+
+	/**
+	 * @param string $category
+	 * @param string $path
+	 *
+	 * @return array|false
+	 */
+	public function findPostByPath($category, $path)
+	{
+		$query = "
             SELECT `id`, `title`, `path`, `date`, `body`, `category`
             FROM `jpemeric_blog`.`post`
             WHERE `path` = :path AND `category` = :category AND `display` = :is_active
             LIMIT 1";
-        $bindings = [
-            'path'      => $path,
-            'category'  => $category,
-            'is_active' => 1,
-        ];
-
-        return $this
-            ->connections
-            ->getRead()
-            ->fetchOne($query, $bindings);
-    }
-
-    public function getActivePosts($limit = null, $offset = 0)
-    {
-        $query = "
+		$bindings = [
+			'path'      => $path,
+			'category'  => $category,
+			'is_active' => 1,
+		];
+
+		return $this
+			->connections
+			->getRead()
+			->fetchOne($query, $bindings);
+	}
+
+	public function getActivePosts($limit = null, $offset = 0)
+	{
+		$query = "
             SELECT `id`, `title`, `path`, `date`, `body`, `category`
             FROM `jpemeric_blog`.`post`
             WHERE `display` = :is_active
             ORDER BY `date` DESC";
-        if ($limit != null) {
-            $query .= "
+		if ($limit != null) {
+			$query .= "
             LIMIT {$offset}, {$limit}";
-        }
+		}
 
-        $bindings = [
-            'is_active' => 1,
-        ];
+		$bindings = [
+			'is_active' => 1,
+		];
 
-        return $this
-            ->connections
-            ->getRead()
-            ->fetchAll($query, $bindings);
-    }
+		return $this
+			->connections
+			->getRead()
+			->fetchAll($query, $bindings);
+	}
 
-    public function getActivePostsCount()
-    {
-        $query = "
+	public function getActivePostsCount()
+	{
+		$query = "
             SELECT COUNT(1) AS `count`
             FROM `jpemeric_blog`.`post`
             WHERE `display` = :is_active";
-        $bindings = [
-            'is_active' => 1,
-        ];
-
-        return $this
-            ->connections
-            ->getRead()
-            ->fetchValue($query, $bindings);
-    }
-
-    public function getActivePostsByTag($tag, $limit = null, $offset = 0)
-    {
-        $query = "
+		$bindings = [
+			'is_active' => 1,
+		];
+
+		return $this
+			->connections
+			->getRead()
+			->fetchValue($query, $bindings);
+	}
+
+	public function getActivePostsByTag($tag, $limit = null, $offset = 0)
+	{
+		$query = "
             SELECT `id`, `title`, `path`, `date`, `body`, `category`
             FROM `jpemeric_blog`.`post`
             INNER JOIN `jpemeric_blog`.`ptlink` ON `ptlink`.`post_id` = `post`.`id` AND
                                                    `ptlink`.`tag_id` = :tag_id
             WHERE `display` = :is_active";
-        if ($limit != null) {
-            $query .= "
+		if ($limit != null) {
+			$query .= "
             LIMIT {$offset}, {$limit}";
-        }
-
-        $bindings = [
-            'tag_id'    => $tag,
-            'is_active' => 1,
-        ];
-
-        return $this
-            ->connections
-            ->getRead()
-            ->fetchAll($query, $bindings);
-    }
-
-    public function getActivePostsCountByTag($tag)
-    {
-        $query = "
+		}
+
+		$bindings = [
+			'tag_id'    => $tag,
+			'is_active' => 1,
+		];
+
+		return $this
+			->connections
+			->getRead()
+			->fetchAll($query, $bindings);
+	}
+
+	public function getActivePostsCountByTag($tag)
+	{
+		$query = "
             SELECT COUNT(1) AS `count`
             FROM `jpemeric_blog`.`post`
             INNER JOIN `jpemeric_blog`.`ptlink` ON `ptlink`.`post_id` = `post`.`id` AND
                                                    `ptlink`.`tag_id` = :tag_id
             WHERE `display` = :is_active";
-        $bindings = [
-            'tag_id'    => $tag,
-            'is_active' => 1,
-        ];
-
-        return $this
-            ->connections
-            ->getRead()
-            ->fetchValue($query, $bindings);
-    }
-
-    public function getActivePostsByCategory($category, $limit = null, $offset = 0)
-    {
-        $query = "
+		$bindings = [
+			'tag_id'    => $tag,
+			'is_active' => 1,
+		];
+
+		return $this
+			->connections
+			->getRead()
+			->fetchValue($query, $bindings);
+	}
+
+	public function getActivePostsByCategory($category, $limit = null, $offset = 0)
+	{
+		$query = "
             SELECT `id`, `title`, `path`, `date`, `body`, `category`
             FROM `jpemeric_blog`.`post`
             WHERE `category` = :category AND `display` = :is_active";
-        if ($limit != null) {
-            $query .= "
+		if ($limit != null) {
+			$query .= "
             LIMIT {$offset}, {$limit}";
-        }
-
-        $bindings = [
-            'category'  => $category,
-            'is_active' => 1,
-        ];
-
-        return $this
-            ->connections
-            ->getRead()
-            ->fetchAll($query, $bindings);
-    }
-
-    public function getActivePostsCountByCategory($category)
-    {
-        $query = "
+		}
+
+		$bindings = [
+			'category'  => $category,
+			'is_active' => 1,
+		];
+
+		return $this
+			->connections
+			->getRead()
+			->fetchAll($query, $bindings);
+	}
+
+	public function getActivePostsCountByCategory($category)
+	{
+		$query = "
             SELECT COUNT(1) AS `count`
             FROM `jpemeric_blog`.`post`
             WHERE `category` = :category AND `display` = :is_active";
-        $bindings = [
-            'category'  => $category,
-            'is_active' => 1,
-        ];
-
-        return $this
-            ->connections
-            ->getRead()
-            ->fetchValue($query, $bindings);
-    }
-
-    public function getActivePostsByRelatedTags($post, $limit = 4)
-    {
-        $query = "
+		$bindings = [
+			'category'  => $category,
+			'is_active' => 1,
+		];
+
+		return $this
+			->connections
+			->getRead()
+			->fetchValue($query, $bindings);
+	}
+
+	public function getActivePostsByRelatedTags($post, $limit = 4)
+	{
+		$query = "
             SELECT `id`, `title`, `path`, `date`, `body`, `category`, COUNT(1) AS `count`
             FROM `jpemeric_blog`.`post`
             INNER JOIN `jpemeric_blog`.`ptlink` ON `ptlink`.`post_id` = `post`.`id` AND
@@ -185,14 +185,14 @@ 
             GROUP BY `id`
             ORDER BY `count` DESC
             LIMIT {$limit}";
-        $bindings = [
-            'post'      => $post,
-            'is_active' => 1,
-        ];
-
-        return $this
-            ->connections
-            ->getRead()
-            ->fetchAll($query, $bindings);
-    }
+		$bindings = [
+			'post'      => $post,
+			'is_active' => 1,
+		];
+
+		return $this
+			->connections
+			->getRead()
+			->fetchAll($query, $bindings);
+	}
 }

src/Domain/Blog/Post/PostRepositoryInterface.php

Indentation +8 added lines, -8 removed lines

@@ -4,12 +4,12 @@
 
 interface PostRepository
 {
-    public function findPostByPath($category, $path);
-    public function getActivePosts($limit = null, $offset= 0);
-    public function getActivePostsCount();
-    public function getActivePostsByTag($tag, $limit = null, $offset = 0);
-    public function getActivePostsCountByTag($tag);
-    public function getActivePostsByCategory($category, $limit = null, $offset = 0);
-    public function getActivePostsCountByCategory($category);
-    public function getActivePostsByRelatedTags($post, $limit = 4);
+	public function findPostByPath($category, $path);
+	public function getActivePosts($limit = null, $offset= 0);
+	public function getActivePostsCount();
+	public function getActivePostsByTag($tag, $limit = null, $offset = 0);
+	public function getActivePostsCountByTag($tag);
+	public function getActivePostsByCategory($category, $limit = null, $offset = 0);
+	public function getActivePostsCountByCategory($category);
+	public function getActivePostsByRelatedTags($post, $limit = 4);
 }

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 interface PostRepository
 {
     public function findPostByPath($category, $path);
-    public function getActivePosts($limit = null, $offset= 0);
+    public function getActivePosts($limit = null, $offset = 0);
     public function getActivePostsCount();
     public function getActivePostsByTag($tag, $limit = null, $offset = 0);
     public function getActivePostsCountByTag($tag);