jacobemerick / web

controller/portfolio/DefaultPageController.class.inc.php

Spacing +5 added lines, -5 removed lines

@@ -39,12 +39,12 @@ 
 				'name' => 'Contact',

 				'link' => Loader::getRootURL() . 'contact/'));

 		

-		if(!URLDecode::getPiece(1))

+		if (!URLDecode::getPiece(1))

 			$active_page = 'About';

 		else

 			$active_page = ucfirst(URLDecode::getPiece(1));

 		

-		foreach($menu as $menu_item)

+		foreach ($menu as $menu_item)

 		{

 			$menu_item = (object) $menu_item;

 			$menu_item->active = ($menu_item->name == $active_page);

@@ -59,11 +59,11 @@ 
 		Loader::load('collector', 'portfolio/PortfolioCollector');

 		$portfolio_result = PortfolioCollector::getPiecesForCategory($category_id);

 		

-		foreach($portfolio_result as $piece)

+		foreach ($portfolio_result as $piece)

 		{

 			$piece_obj = new stdclass();

 			$piece_obj->title = $piece->title;

-			if("{$piece->category}" == 1)

+			if ("{$piece->category}" == 1)

 				$piece_obj->url = Loader::getRootURL() . "web/{$piece->title_url}/";

 			else

 				$piece_obj->url = Loader::getRootURL() . "print/{$piece->title_url}/";

@@ -71,7 +71,7 @@ 
 			$piece_array[] = $piece_obj;

 		}

 		

-		switch($category_id)

+		switch ($category_id)

 		{

 			case 1 :

 				$title = "Sites I've worked on";

Braces +10 added lines, -8 removed lines

@@ -39,10 +39,11 @@ 
 				'name' => 'Contact',

 				'link' => Loader::getRootURL() . 'contact/'));

 		

-		if(!URLDecode::getPiece(1))

-			$active_page = 'About';

-		else

-			$active_page = ucfirst(URLDecode::getPiece(1));

+		if(!URLDecode::getPiece(1)) {

+					$active_page = 'About';

+		} else {

+					$active_page = ucfirst(URLDecode::getPiece(1));

+		}

 		

 		foreach($menu as $menu_item)

 		{

@@ -63,10 +64,11 @@ 
 		{

 			$piece_obj = new stdclass();

 			$piece_obj->title = $piece->title;

-			if("{$piece->category}" == 1)

-				$piece_obj->url = Loader::getRootURL() . "web/{$piece->title_url}/";

-			else

-				$piece_obj->url = Loader::getRootURL() . "print/{$piece->title_url}/";

+			if("{$piece->category}" == 1) {

+							$piece_obj->url = Loader::getRootURL() . "web/{$piece->title_url}/";

+			} else {

+							$piece_obj->url = Loader::getRootURL() . "print/{$piece->title_url}/";

+			}

 			$piece_obj->image = "/{$piece->image}";

 			$piece_array[] = $piece_obj;

 		}

controller/portfolio/PieceController.class.inc.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 		Loader::load('collector', 'portfolio/PortfolioCollector');

 		$portfolio_result = PortfolioCollector::getPieceByURI($this->title_url);

 		

-		if($portfolio_result === null)

+		if ($portfolio_result === null)

 			$this->eject();

 		

 		$portfolio_image_result = PortfolioCollector::getImagesForPiece($portfolio_result->id, 2);

@@ -51,7 +51,7 @@ 
 		$main_image->width = $dimensions[0];

 		$main_image->height = $dimensions[1];

 		

-		foreach($portfolio_image_result as $portfolio_image)

+		foreach ($portfolio_image_result as $portfolio_image)

 		{

 			$thumb = $portfolio_image->name;

 			$thumb_array = explode('.', $thumb);

@@ -72,7 +72,7 @@ 
 		

 		$portfolio_tag_result = PortfolioCollector::getTagsForPiece($portfolio_result->id);

 		

-		foreach($portfolio_tag_result as $portfolio_tag)

+		foreach ($portfolio_tag_result as $portfolio_tag)

 		{

 			$tag_array[] = $portfolio_tag->name;

 		}

Braces +3 added lines, -2 removed lines

@@ -36,8 +36,9 @@
 		Loader::load('collector', 'portfolio/PortfolioCollector');

 		$portfolio_result = PortfolioCollector::getPieceByURI($this->title_url);

 		

-		if($portfolio_result === null)

-			$this->eject();

+		if($portfolio_result === null) {

+					$this->eject();

+		}

 		

 		$portfolio_image_result = PortfolioCollector::getImagesForPiece($portfolio_result->id, 2);

 		$image = new ImageOld("portfolio/{$portfolio_image_result[0]->name}");

controller/robot/WaterfallsRobotController.class.inc.php

Indentation +4 added lines, -4 removed lines

@@ -5,9 +5,9 @@
 class WaterfallsRobotController extends RobotController

 {

 

-    protected function set_data()

-    {

-        $this->disallow_for_all_robots('/search/');

-    }

+	protected function set_data()

+	{

+		$this->disallow_for_all_robots('/search/');

+	}

 

 }

controller/rss/BlogPostRSSController.class.inc.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	{

 		$post_result = PostCollector::getMainList(250);

 		

-		foreach($post_result as $post)

+		foreach ($post_result as $post)

 		{

 			$this->addItem(

 				$post->title,

controller/site/ChangelogController.class.inc.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 		Loader::load('collector', 'data/ChangelogCollector');

 		$changelog_result = ChangelogCollector::getLast20Changes();

 		

-		foreach($changelog_result as $change)

+		foreach ($changelog_result as $change)

 		{

 			$changelog[] = (object) array(

 				'date' => (object) array(

controller/site/ContactController.class.inc.php

Spacing +5 added lines, -5 removed lines

@@ -29,17 +29,17 @@
 

 	private function get_form_results()

 	{

-		if(!Request::hasPost())

+		if (!Request::hasPost())

 			return array();

 		

-		if(!Validate::checkRequest('post', 'name', 'string'))

+		if (!Validate::checkRequest('post', 'name', 'string'))

 			$error_message['name'] = 'Please enter a value for your name.';

-		if(!Validate::checkRequest('post', 'email', 'string'))

+		if (!Validate::checkRequest('post', 'email', 'string'))

 			$error_message['email'] = 'Please enter a valid email address.';

-		if(!Validate::checkRequest('post', 'message', 'string'))

+		if (!Validate::checkRequest('post', 'message', 'string'))

 			$error_message['message'] = 'Please enter a message.';

 		

-		if(!empty($error_message))

+		if (!empty($error_message))

 		{

 			return array(

 				'error_message' => $error_message,

Braces +12 added lines, -8 removed lines

@@ -29,15 +29,19 @@
 

 	private function get_form_results()

 	{

-		if(!Request::hasPost())

-			return array();

+		if(!Request::hasPost()) {

+					return array();

+		}

 		

-		if(!Validate::checkRequest('post', 'name', 'string'))

-			$error_message['name'] = 'Please enter a value for your name.';

-		if(!Validate::checkRequest('post', 'email', 'string'))

-			$error_message['email'] = 'Please enter a valid email address.';

-		if(!Validate::checkRequest('post', 'message', 'string'))

-			$error_message['message'] = 'Please enter a message.';

+		if(!Validate::checkRequest('post', 'name', 'string')) {

+					$error_message['name'] = 'Please enter a value for your name.';

+		}

+		if(!Validate::checkRequest('post', 'email', 'string')) {

+					$error_message['email'] = 'Please enter a valid email address.';

+		}

+		if(!Validate::checkRequest('post', 'message', 'string')) {

+					$error_message['message'] = 'Please enter a message.';

+		}

 		

 		if(!empty($error_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}>";

controller/site/DefaultPageController.class.inc.php

Spacing +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
 		$active_page = $this->get_active_page();

 		

 		$menu = array();

-		foreach(self::$MENU as $item)

+		foreach (self::$MENU as $item)

 		{

 			$menu[] = (object) array(

 				'name' => $item['name'],

@@ -56,13 +56,13 @@ 
 	{

 		$current_uri = URLDecode::getPiece(1);

 		

-		if(!isset($current_uri))

+		if (!isset($current_uri))

 			return 'About';

 		

 		$menu = self::$MENU;

-		foreach($menu as $link)

+		foreach ($menu as $link)

 		{

-			if($link['uri'] == $current_uri)

+			if ($link['uri'] == $current_uri)

 				return $link['name'];

 		}

 		

Braces +6 added lines, -4 removed lines

@@ -56,14 +56,16 @@
 	{

 		$current_uri = URLDecode::getPiece(1);

 		

-		if(!isset($current_uri))

-			return 'About';

+		if(!isset($current_uri)) {

+					return 'About';

+		}

 		

 		$menu = self::$MENU;

 		foreach($menu as $link)

 		{

-			if($link['uri'] == $current_uri)

-				return $link['name'];

+			if($link['uri'] == $current_uri) {

+							return $link['name'];

+			}

 		}

 		

 		return 'About';

controller/sitemap/BlogSitemapController.class.inc.php

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);

 		}

Doc Comments +3 added lines

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

 	}

 

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

 	{

 		switch($changefreq)

controller/sitemap/PortfolioSitemapController.class.inc.php

Spacing +2 added lines, -2 removed lines

@@ -16,9 +16,9 @@
 		

 		$pieces = PortfolioCollector::getAllPieces();

 		

-		foreach($pieces as $piece)

+		foreach ($pieces as $piece)

 		{

-			if($piece->category == 1)

+			if ($piece->category == 1)

 				$this->addURL("web/{$piece->title_url}/", date('Y-01-01'), 'yearly', .7);

 			else

 				$this->addURL("print/{$piece->title_url}/", date('Y-01-01'), 'yearly', .7);

Braces +5 added lines, -4 removed lines

@@ -18,10 +18,11 @@
 		

 		foreach($pieces as $piece)

 		{

-			if($piece->category == 1)

-				$this->addURL("web/{$piece->title_url}/", date('Y-01-01'), 'yearly', .7);

-			else

-				$this->addURL("print/{$piece->title_url}/", date('Y-01-01'), 'yearly', .7);

+			if($piece->category == 1) {

+							$this->addURL("web/{$piece->title_url}/", date('Y-01-01'), 'yearly', .7);

+			} else {

+							$this->addURL("print/{$piece->title_url}/", date('Y-01-01'), 'yearly', .7);

+			}

 		}

 	}