jacobemerick / web

collector/data/DistanceCollector.class.inc.php

Doc Comments +3 added lines

@@ -17,6 +17,9 @@
 		return self::run_row_query($query);

 	}

 

+	/**
+	 * @param string $date
+	 */
 	public static function getTweetByFields($date, $text)

 	{

 		$query = "SELECT * FROM `jpemeric_stream`.`twitter` WHERE `date` = '{$date}' && `text` = '{$text}' LIMIT 1";

collector/data/TwitterCollector.class.inc.php

Doc Comments +3 added lines

@@ -17,6 +17,9 @@
 		return self::run_row_query($query);

 	}

 

+	/**
+	 * @param string $date
+	 */
 	public static function getTweetByFields($date, $text)

 	{

 		$query = "SELECT * FROM `jpemeric_stream`.`twitter` WHERE `date` = '{$date}' && `text` = '{$text}' LIMIT 1";

collector/stream/DistanceCollector.class.inc.php

Doc Comments +3 added lines

@@ -17,6 +17,9 @@
 		return self::run_row_query($query);

 	}

 

+	/**
+	 * @param string $date
+	 */
 	public static function getTweetByFields($date, $text)

 	{

 		$query = "SELECT * FROM `jpemeric_stream`.`twitter` WHERE `date` = '{$date}' && `text` = '{$text}' LIMIT 1";

controller/sitemap/BlogSitemapController.class.inc.php

Doc Comments +3 added lines

@@ -64,6 +64,9 @@
 		}

 	}

 

+	/**
+	 * @param string $changefreq
+	 */
 	private function add_paginated_pages($posts, $base_url, $changefreq, $rank)

 	{

 		switch($changefreq)

Spacing +13 added lines, -13 removed lines

@@ -9,19 +9,19 @@ 
 final class BlogSitemapController extends SitemapController
 {
 
-	private static $HOME_PAGE_RANK			= .9;
-	private static $CATEGORY_PAGE_RANK		= .3;
-	private static $TAG_PAGE_RANK			= .2;
-	private static $PAGINATED_PAGE_RANK		= .1;
-	private static $POST_PAGE_RANK			= .8;
+	private static $HOME_PAGE_RANK = .9;
+	private static $CATEGORY_PAGE_RANK = .3;
+	private static $TAG_PAGE_RANK = .2;
+	private static $PAGINATED_PAGE_RANK = .1;
+	private static $POST_PAGE_RANK = .8;
 
 	private static $HOME_PAGE_CHANGEFREQ		= 'daily';
-	private static $CATEGORY_PAGE_CHANGEFREQ	= 'weekly';
+	private static $CATEGORY_PAGE_CHANGEFREQ = 'weekly';
 	private static $TAG_PAGE_CHANGEFREQ			= 'monthly';
-	private static $PAGINATED_PAGE_CHANGEFREQ	= 'monthly';
+	private static $PAGINATED_PAGE_CHANGEFREQ = 'monthly';
 	private static $POST_PAGE_CHANGEFREQ		= 'weekly';
 
-	private static $POSTS_PER_PAGE		= 10;
+	private static $POSTS_PER_PAGE = 10;
 
 	private static $CATEGORY_ARRAY = array(
 		'hiking',
@@ -50,7 +50,7 @@ 
 	{
 		$categories = self::$CATEGORY_ARRAY;
 		
-		foreach($categories as $category)
+		foreach ($categories as $category)
 		{
 			$post_count = PostCollector::getPostCountForCategory($category);
 			
@@ -63,7 +63,7 @@ 
 	{
 		$tags = TagCollector::getAllTags();
 		
-		foreach($tags as $tag)
+		foreach ($tags as $tag)
 		{
 			$post_count = PostCollector::getPostCountForTag($tag->id);
 			
@@ -76,7 +76,7 @@ 
 	{
 		$posts = PostCollector::getMainList(500);
 		
-		foreach($posts as $post)
+		foreach ($posts as $post)
 		{
 			$base_url = "{$post->category}/{$post->path}/";
 			$this->addURL($base_url, date('Y-m-d', strtotime('last Monday')), self::$POST_PAGE_CHANGEFREQ, self::$POST_PAGE_RANK);
@@ -85,7 +85,7 @@ 
 
 	private function add_paginated_pages($post_count, $base_url, $changefreq, $rank)
 	{
-		switch($changefreq)
+		switch ($changefreq)
 		{
 			case 'daily' :
 				$lastmod = date('Y-m-d');
@@ -99,7 +99,7 @@ 
 		}
 		
 		$this->addURL($base_url, $lastmod, $changefreq, $rank);
-		for($i = 2; (($i - 1) * self::$POSTS_PER_PAGE) < $post_count; $i++)
+		for ($i = 2; (($i - 1) * self::$POSTS_PER_PAGE) < $post_count; $i++)
 		{
 			$this->addURL("{$base_url}{$i}/", date('Y-m-01'), self::$PAGINATED_PAGE_CHANGEFREQ, self::$PAGINATED_PAGE_RANK);
 		}

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

Doc Comments +3 added lines

@@ -49,6 +49,9 @@
 		Database::execute($query);

 	}

 

+	/**
+	 * @return string
+	 */
 	private function get_changeset_to_check()

 	{

 		$changeset = ChangelogCollector::getMostRecentChange();

Spacing +11 added lines, -11 removed lines

@@ -10,11 +10,11 @@ 
 
 	protected function execute()
 	{
-		if(stristr($this->content, 'http') === false)
+		if (stristr($this->content, 'http') === false)
 			return;
 		
 		$parameters = func_get_args();
-		if(count($parameters) > 1 || (count($parameters) == 1 && !is_array($parameters[0])))
+		if (count($parameters) > 1 || (count($parameters) == 1 && !is_array($parameters[0])))
 		{
 			trigger_error('Unexpected parameters passed into AutolinkTwitterURL!');
 			return;
@@ -22,10 +22,10 @@ 
 		$parameters = array_shift($parameters);
 		
 		$found_link = preg_match_all(self::$LINK_PATTERN, $this->content, $matches, PREG_SET_ORDER);
-		if($found_link === false || $found_link === 0)
+		if ($found_link === false || $found_link === 0)
 			return;
 		
-		foreach($matches as $match)
+		foreach ($matches as $match)
 		{
 			$parameters['href'] = $match[0];
 			$display_url = $this->get_display_url($match);
@@ -41,7 +41,7 @@ 
 	{
 		$display_url = $this->check_for_redirect($match[0]);
 		
-		if(strlen($display_url) > (self::$DISPLAY_URL_LENGTH - 2))
+		if (strlen($display_url) > (self::$DISPLAY_URL_LENGTH - 2))
 		{
 			$display_url = substr($display_url, 0, self::$DISPLAY_URL_LENGTH);
 			$display_url .= '...';
@@ -53,12 +53,12 @@ 
 	private function get_replacement_link($parameters, $display_url)
 	{
 		$link = '<a';
-		foreach($parameters as $attribute => $value)
+		foreach ($parameters as $attribute => $value)
 		{
 			$link .= ' ';
 			$link .= $attribute;
 			$link .= '=';
-			$link .= '"' . $value .'"';
+			$link .= '"' . $value . '"';
 		}
 		$link .= '>';
 		$link .= $display_url;
@@ -70,14 +70,14 @@ 
 	private function check_for_redirect($url)
 	{
 		$headers = @get_headers($url);
-		if($headers === false)
+		if ($headers === false)
 			$headers = $this->get_headers($url);
 		
-		if(stristr($headers[0], '301') !== false || stristr($headers[0], '302') !== false)
+		if (stristr($headers[0], '301') !== false || stristr($headers[0], '302') !== false)
 		{
-			foreach($headers as $header)
+			foreach ($headers as $header)
 			{
-				if(substr($header, 0, 9) == 'Location:')
+				if (substr($header, 0, 9) == 'Location:')
 					break;
 			}
 			

Braces +12 added lines, -8 removed lines

@@ -10,8 +10,9 @@ 
 
 	protected function execute()
 	{
-		if(stristr($this->content, 'http') === false)
-			return;
+		if(stristr($this->content, 'http') === false) {

+					return;

+		}

 		
 		$parameters = func_get_args();
 		if(count($parameters) > 1 || (count($parameters) == 1 && !is_array($parameters[0])))
@@ -22,8 +23,9 @@ 
 		$parameters = array_shift($parameters);
 		
 		$found_link = preg_match_all(self::$LINK_PATTERN, $this->content, $matches, PREG_SET_ORDER);
-		if($found_link === false || $found_link === 0)
-			return;
+		if($found_link === false || $found_link === 0) {

+					return;

+		}

 		
 		foreach($matches as $match)
 		{
@@ -70,15 +72,17 @@ 
 	private function check_for_redirect($url)
 	{
 		$headers = @get_headers($url);
-		if($headers === false)
-			$headers = $this->get_headers($url);
+		if($headers === false) {

+					$headers = $this->get_headers($url);

+		}

 		
 		if(stristr($headers[0], '301') !== false || stristr($headers[0], '302') !== false)
 		{
 			foreach($headers as $header)
 			{
-				if(substr($header, 0, 9) == 'Location:')
-					break;
+				if(substr($header, 0, 9) == 'Location:') {

+									break;

+				}

 			}
 			
 			$url = substr($header, 10);

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

utility/content/FixPhotoContent.class.inc.php

Doc Comments +4 added lines

@@ -26,6 +26,10 @@
 		return Loader::getImagePath('photo', $path);

 	}

 

+	/**
+	 * @param string $size
+	 * @param boolean $is_absolute
+	 */
 	private function get_photo_content($string, $size, $is_absolute)

 	{

 		list($category, $file_name) = explode('/', $string);

Indentation +3 added lines, -3 removed lines

@@ -44,9 +44,9 @@
 		$width = $file_size[0];
 		$description = $photo_result->description;
         
-        if ($description == '') {
-            Debugger::logMessage("No description for {$category}/{$photo}");
-        }
+		if ($description == '') {
+			Debugger::logMessage("No description for {$category}/{$photo}");
+		}
 		
 		$domain = '/';
 		if($is_absolute)

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 	protected function execute($is_absolute = false, $size = 'medium')
 	{
 		preg_match_all(self::$PHOTO_PLACEHOLDER_MATCH, $this->content, $matches);
-		foreach($matches[1] as $key => $match)
+		foreach ($matches[1] as $key => $match)
 		{
 			$photo_content = $this->get_photo_content($match, $size, $is_absolute);
 			$this->content = str_replace($matches[0][$key], $photo_content, $this->content);
@@ -37,7 +37,7 @@ 
 		
 		Loader::load('collector', 'image/PhotoCollector');
 		$photo_result = PhotoCollector::fetchRow($category, $photo);
-		if($photo_result == false)
+		if ($photo_result == false)
 			return '';
 		
 		$height = $file_size[1];
@@ -49,7 +49,7 @@ 
         }
 		
 		$domain = '/';
-		if($is_absolute)
+		if ($is_absolute)
 			$domain = Loader::getRootUrl('blog');
 		
 		return sprintf(self::$PHOTO_CONTENT, $domain, $category, $photo, $size, $height, $width, $description, $description);

Braces +6 added lines, -4 removed lines

@@ -37,8 +37,9 @@ 
 		
 		Loader::load('collector', 'image/PhotoCollector');
 		$photo_result = PhotoCollector::fetchRow($category, $photo);
-		if($photo_result == false)
-			return '';
+		if($photo_result == false) {

+					return '';

+		}

 		
 		$height = $file_size[1];
 		$width = $file_size[0];
@@ -49,8 +50,9 @@ 
         }
 		
 		$domain = '/';
-		if($is_absolute)
-			$domain = Loader::getRootUrl('blog');
+		if($is_absolute) {

+					$domain = Loader::getRootUrl('blog');

+		}

 		
 		return sprintf(self::$PHOTO_CONTENT, $domain, $category, $photo, $size, $height, $width, $description, $description);
 	}