jacobemerick / web

controller/waterfalls/MapController.class.inc.php

Indentation +48 added lines, -48 removed lines

@@ -6,62 +6,62 @@
 final class MapController extends DefaultPageController
 {
 
-    private static $TITLE = 'Map of Upper Peninsula Waterfalls';
-    private static $DESCRIPTION = '';
-    private static $KEYWORD_ARRAY = array();
+	private static $TITLE = 'Map of Upper Peninsula Waterfalls';
+	private static $DESCRIPTION = '';
+	private static $KEYWORD_ARRAY = array();
 
-    private static $HIDE_ON_INITIAL_LOAD_LIST = array(
-        'Lower Tahquamenon Falls',
-        'Rainbow Falls',
-        'Sandstone Falls',
-        'Upper Tahquamenon Falls',
-    );
+	private static $HIDE_ON_INITIAL_LOAD_LIST = array(
+		'Lower Tahquamenon Falls',
+		'Rainbow Falls',
+		'Sandstone Falls',
+		'Upper Tahquamenon Falls',
+	);
 
-    public function __construct()
-    {
-        parent::__construct();
+	public function __construct()
+	{
+		parent::__construct();
 
-        $this->add_js('https://maps.googleapis.com/maps/api/js?sensor=false');
-        $this->add_waterfall_js();
-    }
+		$this->add_js('https://maps.googleapis.com/maps/api/js?sensor=false');
+		$this->add_waterfall_js();
+	}
 
-    protected function set_head_data()
-    {
-        parent::set_head_data();
+	protected function set_head_data()
+	{
+		parent::set_head_data();
         
-        $this->set_title(self::$TITLE . ' | ' . self::$WEBSITE_TITLE);
-        $this->set_description(self::$DESCRIPTION);
-        $this->set_keywords(self::$KEYWORD_ARRAY);
-    }
+		$this->set_title(self::$TITLE . ' | ' . self::$WEBSITE_TITLE);
+		$this->set_description(self::$DESCRIPTION);
+		$this->set_keywords(self::$KEYWORD_ARRAY);
+	}
 
-    protected function set_body_data($page_type = 'wide')
-    {
-        parent::set_body_data($page_type);
+	protected function set_body_data($page_type = 'wide')
+	{
+		parent::set_body_data($page_type);
         
-        $this->set_head('full_page_map', true);
-        $this->set_body('waterfall_list', $this->fetch_waterfall_list());
-        $this->set_body('view', 'Map');
-    }
+		$this->set_head('full_page_map', true);
+		$this->set_body('waterfall_list', $this->fetch_waterfall_list());
+		$this->set_body('view', 'Map');
+	}
 
-    private function fetch_waterfall_list()
-    {
-        $list = WaterfallCollector::getMapList();
-        array_walk($list, array($this, 'parse_waterfall'));
-        return $list;
-    }
+	private function fetch_waterfall_list()
+	{
+		$list = WaterfallCollector::getMapList();
+		array_walk($list, array($this, 'parse_waterfall'));
+		return $list;
+	}
 
-    private function parse_waterfall(&$waterfall)
-    {
-        $waterfall = (object) array(
-            'name'  => $waterfall->name,
-            'latitude' => $waterfall->latitude,
-            'longitude' => $waterfall->longitude,
-            'link' => "/{$waterfall->watercourse_alias}/{$waterfall->alias}/",
-            'image' => "/photo/{$waterfall->photo_category}/{$waterfall->photo}-size-medium.jpg",
-            'image_alt' => $waterfall->photo_description,
-            'watercourse' => $waterfall->watercourse,
-            'hide_on_initial_load' => (in_array($waterfall->name, self::$HIDE_ON_INITIAL_LOAD_LIST)),
-        );
-    }
+	private function parse_waterfall(&$waterfall)
+	{
+		$waterfall = (object) array(
+			'name'  => $waterfall->name,
+			'latitude' => $waterfall->latitude,
+			'longitude' => $waterfall->longitude,
+			'link' => "/{$waterfall->watercourse_alias}/{$waterfall->alias}/",
+			'image' => "/photo/{$waterfall->photo_category}/{$waterfall->photo}-size-medium.jpg",
+			'image_alt' => $waterfall->photo_description,
+			'watercourse' => $waterfall->watercourse,
+			'hide_on_initial_load' => (in_array($waterfall->name, self::$HIDE_ON_INITIAL_LOAD_LIST)),
+		);
+	}
 
 }

controller/waterfalls/CompanionListController.class.inc.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 		$alias = URLDecode::getPiece(2);
 		$this->companion = CompanionCollector::getByAlias($alias);
 		
-		if(!$this->companion)
+		if (!$this->companion)
 			$this->eject();
 		
 		parent::__construct();

Braces +3 added lines, -2 removed lines

@@ -27,8 +27,9 @@
 		$alias = URLDecode::getPiece(2);
 		$this->companion = CompanionCollector::getByAlias($alias);
 		
-		if(!$this->companion)
-			$this->eject();
+		if(!$this->companion) {

+					$this->eject();

+		}

 		
 		parent::__construct();
 	}

controller/blog/AboutController.class.inc.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 		Loader::load('collector', 'blog/IntroductionCollector');
 		$introduction_result = IntroductionCollector::getRow('about');
 		
-		if($introduction_result !== null)
+		if ($introduction_result !== null)
 		{
 			$introduction = array();
 			$introduction['title'] = $introduction_result->title;

Indentation +3 added lines, -3 removed lines

@@ -33,9 +33,9 @@
 
 	protected function get_introduction()
 	{
-        global $container;
-        $repository = new Jacobemerick\Web\Domain\Blog\Introduction\MysqlIntroductionRepository($container['db_connection_locator']);
-        $introduction_result = $repository->findByType('about');
+		global $container;
+		$repository = new Jacobemerick\Web\Domain\Blog\Introduction\MysqlIntroductionRepository($container['db_connection_locator']);
+		$introduction_result = $repository->findByType('about');
 		
 		if($introduction_result !== null)
 		{

controller/blog/SearchController.class.inc.php

Spacing +9 added lines, -9 removed lines

@@ -45,7 +45,7 @@ 
 	{
 		parent::set_head_data();
 		
-		if($this->page == 1)
+		if ($this->page == 1)
 		{
 			$this->set_title(sprintf(self::$TITLE_MAIN, ucwords($this->query)));
 			$this->set_description(sprintf(self::$DESCRIPTION_MAIN, ucwords($this->query)));
@@ -63,10 +63,10 @@ 
 
 	protected function get_introduction()
 	{
-		if($this->total_pages > 1)
+		if ($this->total_pages > 1)
 			return array(
 				'title' => "Posts from search '{$this->query}', page {$this->page} of {$this->total_pages}.");
-		else if($this->total_pages == 1)
+		else if ($this->total_pages == 1)
 			return array(
 				'title' => "Posts from search '{$this->query}'.");
 		else
@@ -77,7 +77,7 @@ 
 	protected function get_page_number()
 	{
 		$page = URLDecode::getPiece(3);
-		if(isset($page) && is_numeric($page))
+		if (isset($page) && is_numeric($page))
 			return $page;
 		return 1;
 	}
@@ -85,7 +85,7 @@ 
 	private $search_result;
 	private function get_search_result()
 	{
-		if(!isset($this->search_result))
+		if (!isset($this->search_result))
 		{
 			$posts = PostCollector::getMainList(self::$MAXIMUM_SEARCH_POSTS);
 			
@@ -113,16 +113,16 @@ 
 
 	protected function get_list_next_link()
 	{
-		if($this->page == 1)
+		if ($this->page == 1)
 			return;
-		if($this->page == 2)
+		if ($this->page == 2)
 			return Content::instance('URLSafe', "/search/{$this->query}/")->activate();
 		return Content::instance('URLSafe', "/search/{$this->query}/" . ($this->page - 1) . '/')->activate();
 	}
 
 	protected function get_list_prev_link()
 	{
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
+		if (($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
 			return;
 		return Content::instance('URLSafe', "/search/{$this->query}/" . ($this->page + 1) . '/')->activate();
 	}
@@ -130,7 +130,7 @@ 
 	private $total_post_count;
 	protected function get_total_post_count()
 	{
-		if(!isset($this->total_post_count))
+		if (!isset($this->total_post_count))
 			$this->total_post_count = count($this->get_search_result());
 		return $this->total_post_count;
 	}

Braces +26 added lines, -21 removed lines

@@ -49,8 +49,7 @@ 
 		{
 			$this->set_title(sprintf(self::$TITLE_MAIN, ucwords($this->query)));
 			$this->set_description(sprintf(self::$DESCRIPTION_MAIN, ucwords($this->query)));
-		}
-		else
+		} else
 		{
 			$this->set_title(sprintf(self::$TITLE_PAGINATED, ucwords($this->query), $this->page, $this->total_pages));
 			$this->set_description(sprintf(self::$DESCRIPTION_PAGINATED, $this->page, $this->total_pages, ucwords($this->query)));
@@ -63,22 +62,24 @@ 
 
 	protected function get_introduction()
 	{
-		if($this->total_pages > 1)
-			return array(
-				'title' => "Posts from search '{$this->query}', page {$this->page} of {$this->total_pages}.");
-		else if($this->total_pages == 1)
-			return array(
-				'title' => "Posts from search '{$this->query}'.");
-		else
-			return array(
-				'title' => "Sorry, '{$this->query}' didn't return any posts.");
+		if($this->total_pages > 1) {

+					return array(
+				'title' => "Posts from search '{$this->query}', page {$this->page} of {$this->total_pages}.");

+		} else if($this->total_pages == 1) {

+					return array(
+				'title' => "Posts from search '{$this->query}'.");

+		} else {

+					return array(
+				'title' => "Sorry, '{$this->query}' didn't return any posts.");

+		}

 	}
 
 	protected function get_page_number()
 	{
 		$page = URLDecode::getPiece(3);
-		if(isset($page) && is_numeric($page))
-			return $page;
+		if(isset($page) && is_numeric($page)) {

+					return $page;

+		}

 		return 1;
 	}
 
@@ -113,25 +114,29 @@ 
 
 	protected function get_list_next_link()
 	{
-		if($this->page == 1)
-			return;
-		if($this->page == 2)
-			return Content::instance('URLSafe', "/search/{$this->query}/")->activate();
+		if($this->page == 1) {

+					return;

+		}

+		if($this->page == 2) {

+					return Content::instance('URLSafe', "/search/{$this->query}/")->activate();

+		}

 		return Content::instance('URLSafe', "/search/{$this->query}/" . ($this->page - 1) . '/')->activate();
 	}
 
 	protected function get_list_prev_link()
 	{
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
-			return;
+		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count()) {

+					return;

+		}

 		return Content::instance('URLSafe', "/search/{$this->query}/" . ($this->page + 1) . '/')->activate();
 	}
 
 	private $total_post_count;
 	protected function get_total_post_count()
 	{
-		if(!isset($this->total_post_count))
-			$this->total_post_count = count($this->get_search_result());
+		if(!isset($this->total_post_count)) {

+					$this->total_post_count = count($this->get_search_result());

+		}

 		return $this->total_post_count;
 	}
 

Indentation +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
 	{
 		$query = URLDecode::getPiece(2);
 		$query = urldecode($query);
-        $query = str_replace('-', ' ', $query);
+		$query = str_replace('-', ' ', $query);
 		
 		$this->query = $query;
 		
@@ -87,9 +87,9 @@ 
 	{
 		if(!isset($this->search_result))
 		{
-        global $container;
-        $repository = new Jacobemerick\Web\Domain\Blog\Post\MysqlPostRepository($container['db_connection_locator']);
-        $posts = $repository->getActivePosts();
+		global $container;
+		$repository = new Jacobemerick\Web\Domain\Blog\Post\MysqlPostRepository($container['db_connection_locator']);
+		$posts = $repository->getActivePosts();
 			
 			$this->search_result = Search::instance()
 				->setQuery($this->query)

controller/blog/DefaultListController.class.inc.php

Spacing +2 added lines, -2 removed lines

@@ -56,11 +56,11 @@
 	final private function get_list_posts()
 	{
 		$post_array = array();
-		foreach($this->get_list_results() as $post)
+		foreach ($this->get_list_results() as $post)
 		{
 			$post_array[] = $this->format_post($post, true);
 		}
-		if(count($post_array) < 1 && URLDecode::getPiece(1) !== 'search')
+		if (count($post_array) < 1 && URLDecode::getPiece(1) !== 'search')
 			$this->eject();
 		return $post_array;
 	}

Braces +3 added lines, -2 removed lines

@@ -60,8 +60,9 @@
 		{
 			$post_array[] = $this->format_post($post, true);
 		}
-		if(count($post_array) < 1 && URLDecode::getPiece(1) !== 'search')
-			$this->eject();
+		if(count($post_array) < 1 && URLDecode::getPiece(1) !== 'search') {

+					$this->eject();

+		}

 		return $post_array;
 	}
 

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/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 +10 added lines, -10 removed lines

@@ -46,16 +46,16 @@
 				'value' => Request::getPost());
 		}
 
-    global $container;
-    $sent = $container['mail']
-      ->addTo($container['config']->admin_email)
-      ->setSubject('Site Contact')
-      ->setPlainMessage(
-        'Name: ' . Request::getPost('name') . "\n" .
-        'Email: ' . Request::getPost('email') . "\n" .
-        'Message: ' . Request::getPost('message')
-      )
-      ->send();
+	global $container;
+	$sent = $container['mail']
+	  ->addTo($container['config']->admin_email)
+	  ->setSubject('Site Contact')
+	  ->setPlainMessage(
+		'Name: ' . Request::getPost('name') . "\n" .
+		'Email: ' . Request::getPost('email') . "\n" .
+		'Message: ' . Request::getPost('message')
+	  )
+	  ->send();
 
 		return array(
 			'success_message' => "Thank you for your message, " . Request::getPost('name') . "! I'll get back to you as soon as possible.");

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(

Indentation +3 added lines, -3 removed lines

@@ -28,9 +28,9 @@
 
 	private function get_changelog()
 	{
-    global $container;
-    $changelogRepository = new MysqlChangelogRepository($container['db_connection_locator']);
-    $changelog_result = $changelogRepository->getChanges(40);
+	global $container;
+	$changelogRepository = new MysqlChangelogRepository($container['db_connection_locator']);
+	$changelog_result = $changelogRepository->getChanges(40);
 		
 		foreach($changelog_result as $change)
 		{

module/form/CommentSubmitModule.class.inc.php

Spacing +19 added lines, -19 removed lines

@@ -26,15 +26,15 @@ 
 
 	public function activate()
 	{
-		if(!Request::hasPost())
+		if (!Request::hasPost())
 			return false;
-		if(!Request::getPost('submit') == 'Submit Comment')
+		if (!Request::getPost('submit') == 'Submit Comment')
 			return false;
 		
 		$errors = $this->fetch_errors();
-		if(count($errors) > 0)
+		if (count($errors) > 0)
 			return $errors;
-		if(Request::getPost('catch') !== '')
+		if (Request::getPost('catch') !== '')
 			return false;
 		
 		$page_id = $this->save_comment_page();
@@ -50,17 +50,17 @@ 
 	private function fetch_errors()
 	{
 		$errors = array();
-		if(!Validate::checkRequest('post', 'name', 'name'))
+		if (!Validate::checkRequest('post', 'name', 'name'))
 			$errors['name'] = 'You must include a valid name';
-		if(!Validate::checkRequest('post', 'email', 'email'))
+		if (!Validate::checkRequest('post', 'email', 'email'))
 			$errors['email'] = 'You must include a valid email';
-		if(Request::getPost('website') && !Validate::checkRequest('post', 'website', 'url'))
+		if (Request::getPost('website') && !Validate::checkRequest('post', 'website', 'url'))
 			$errors['website'] = 'Please enter a valid website';
-		if(!Validate::checkRequest('post', 'comment', 'string'))
+		if (!Validate::checkRequest('post', 'comment', 'string'))
 			$errors['comment'] = 'You must enter a comment';
-		if(Request::getPost('notify') && Request::getPost('notify') != 'check')
+		if (Request::getPost('notify') && Request::getPost('notify') != 'check')
 			$errors['notify'] = 'You entered an invalid notify request';
-		if(Request::getPost('reply') && !Validate::checkRequest('post', 'reply', 'integer'))
+		if (Request::getPost('reply') && !Validate::checkRequest('post', 'reply', 'integer'))
 			$errors['reply'] = 'You entered an invalid reply request';
 		
 		return $errors;
@@ -69,7 +69,7 @@ 
 	private function save_comment_page()
 	{
 		$page_result = CommentCollector::getCommentPageByURL($this->path, $this->site);
-		if($page_result !== null)
+		if ($page_result !== null)
 			return $page_result->id;
 		
 		$query = "INSERT INTO `jpemeric_comment`.`comment_page` (`site`, `path`) VALUES ('%d', '%s')";
@@ -85,7 +85,7 @@ 
 			'name' => Request::getPost('name'),
 			'email' => Request::getPost('email'));
 		
-		if(Request::getPost('website') != '')
+		if (Request::getPost('website') != '')
 			$cookie_value['website'] = Request::getPost('website');
 		
 		$cookie_value = json_encode($cookie_value);
@@ -95,7 +95,7 @@ 
 			->save();
 		
 		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
-		if($commenter_result !== null)
+		if ($commenter_result !== null)
 			return $commenter_result->id;
 		
 		$query = "INSERT INTO `jpemeric_comment`.`commenter` (`name`,`email`,`url`) VALUES ('%s','%s','%s')";
@@ -112,7 +112,7 @@ 
 	private function save_comment()
 	{
 		$comment_result = CommentCollector::getCommentByBody(Request::getPost('comment'));
-		if($comment_result !== null)
+		if ($comment_result !== null)
 			return $comment_result->id;
 		
 		$query = "INSERT INTO `jpemeric_comment`.`comment` (`body`, `body_format`) VALUES ('%s', '%s')";
@@ -133,7 +133,7 @@ 
 		$query = "INSERT INTO `jpemeric_comment`.`comment_meta` (`commenter`,`comment`,`reply`,`notify`,`comment_page`,`date`,`display`) VALUES ('%d','%d','%d','%d','%d','%s','%d')";
 		
 		$reply = Database::escape(Request::getPost('type'));
-		if($reply == 'new')
+		if ($reply == 'new')
 			$reply = 0;
 		// else check to make sure value is legit
 		
@@ -157,13 +157,13 @@ 
 		
 		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
 		
-		if($commenter_result->trusted == 1)
+		if ($commenter_result->trusted == 1)
 		{
 			$notification_result = CommentCollector::getNotificationForPage($page);
 			
-			foreach($notification_result as $notification_row)
+			foreach ($notification_result as $notification_row)
 			{
-				if($notification_row->email == Request::getPost('email'))
+				if ($notification_row->email == Request::getPost('email'))
 					continue;
 				
 				$email_recipient_array[$notification_row->email] = array(
@@ -180,7 +180,7 @@ 
             $message = "Hello!\nThere has been a new comment on the page '{$this->page_title}' at Waterfalls of the Keweenaw. You have chosen to be notified of it - please reply to jacob@jacobemerick.com if you would like to be removed from these notifications.\n\nOn " . date('F j, Y g:i a') . ", " . Request::getPost('name') . " commented...\n" . Request::getPost('comment') . "\n\nVisit {$this->full_path}#comments to see and reply to all the comments on this post.\nThank you!";
         }
         
-		foreach($email_recipient_array as $email_recipient)
+		foreach ($email_recipient_array as $email_recipient)
 		{
 			$mail = new Mail();
 			$mail->setToAddress($email_recipient['email'], $email_recipient['name']);

Braces +48 added lines, -32 removed lines

@@ -26,16 +26,20 @@ 
 
 	public function activate()
 	{
-		if(!Request::hasPost())
-			return false;
-		if(!Request::getPost('submit') == 'Submit Comment')
-			return false;
+		if(!Request::hasPost()) {

+					return false;

+		}

+		if(!Request::getPost('submit') == 'Submit Comment') {

+					return false;

+		}

 		
 		$errors = $this->fetch_errors();
-		if(count($errors) > 0)
-			return $errors;
-		if(Request::getPost('catch') !== '')
-			return false;
+		if(count($errors) > 0) {

+					return $errors;

+		}

+		if(Request::getPost('catch') !== '') {

+					return false;

+		}

 		
 		$page_id = $this->save_comment_page();
 		$commenter_id = $this->save_commenter();
@@ -50,18 +54,24 @@ 
 	private function fetch_errors()
 	{
 		$errors = array();
-		if(!Validate::checkRequest('post', 'name', 'name'))
-			$errors['name'] = 'You must include a valid name';
-		if(!Validate::checkRequest('post', 'email', 'email'))
-			$errors['email'] = 'You must include a valid email';
-		if(Request::getPost('website') && !Validate::checkRequest('post', 'website', 'url'))
-			$errors['website'] = 'Please enter a valid website';
-		if(!Validate::checkRequest('post', 'comment', 'string'))
-			$errors['comment'] = 'You must enter a comment';
-		if(Request::getPost('notify') && Request::getPost('notify') != 'check')
-			$errors['notify'] = 'You entered an invalid notify request';
-		if(Request::getPost('reply') && !Validate::checkRequest('post', 'reply', 'integer'))
-			$errors['reply'] = 'You entered an invalid reply request';
+		if(!Validate::checkRequest('post', 'name', 'name')) {

+					$errors['name'] = 'You must include a valid name';

+		}

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

+					$errors['email'] = 'You must include a valid email';

+		}

+		if(Request::getPost('website') && !Validate::checkRequest('post', 'website', 'url')) {

+					$errors['website'] = 'Please enter a valid website';

+		}

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

+					$errors['comment'] = 'You must enter a comment';

+		}

+		if(Request::getPost('notify') && Request::getPost('notify') != 'check') {

+					$errors['notify'] = 'You entered an invalid notify request';

+		}

+		if(Request::getPost('reply') && !Validate::checkRequest('post', 'reply', 'integer')) {

+					$errors['reply'] = 'You entered an invalid reply request';

+		}

 		
 		return $errors;
 	}
@@ -69,8 +79,9 @@ 
 	private function save_comment_page()
 	{
 		$page_result = CommentCollector::getCommentPageByURL($this->path, $this->site);
-		if($page_result !== null)
-			return $page_result->id;
+		if($page_result !== null) {

+					return $page_result->id;

+		}

 		
 		$query = "INSERT INTO `jpemeric_comment`.`comment_page` (`site`, `path`) VALUES ('%d', '%s')";
 		$query = sprintf($query, $this->site, $this->path);
@@ -85,8 +96,9 @@ 
 			'name' => Request::getPost('name'),
 			'email' => Request::getPost('email'));
 		
-		if(Request::getPost('website') != '')
-			$cookie_value['website'] = Request::getPost('website');
+		if(Request::getPost('website') != '') {

+					$cookie_value['website'] = Request::getPost('website');

+		}

 		
 		$cookie_value = json_encode($cookie_value);
 		
@@ -95,8 +107,9 @@ 
 			->save();
 		
 		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
-		if($commenter_result !== null)
-			return $commenter_result->id;
+		if($commenter_result !== null) {

+					return $commenter_result->id;

+		}

 		
 		$query = "INSERT INTO `jpemeric_comment`.`commenter` (`name`,`email`,`url`) VALUES ('%s','%s','%s')";
 		
@@ -112,8 +125,9 @@ 
 	private function save_comment()
 	{
 		$comment_result = CommentCollector::getCommentByBody(Request::getPost('comment'));
-		if($comment_result !== null)
-			return $comment_result->id;
+		if($comment_result !== null) {

+					return $comment_result->id;

+		}

 		
 		$query = "INSERT INTO `jpemeric_comment`.`comment` (`body`, `body_format`) VALUES ('%s', '%s')";
 		
@@ -133,8 +147,9 @@ 
 		$query = "INSERT INTO `jpemeric_comment`.`comment_meta` (`commenter`,`comment`,`reply`,`notify`,`comment_page`,`date`,`display`) VALUES ('%d','%d','%d','%d','%d','%s','%d')";
 		
 		$reply = Database::escape(Request::getPost('type'));
-		if($reply == 'new')
-			$reply = 0;
+		if($reply == 'new') {

+					$reply = 0;

+		}

 		// else check to make sure value is legit
 		
 		$notify = (Request::getPost('notify') == 'check') ? 1 : 0;
@@ -163,8 +178,9 @@ 
 			
 			foreach($notification_result as $notification_row)
 			{
-				if($notification_row->email == Request::getPost('email'))
-					continue;
+				if($notification_row->email == Request::getPost('email')) {

+									continue;

+				}

 				
 				$email_recipient_array[$notification_row->email] = array(
 					'email' => $notification_row->email,

Indentation +15 added lines, -15 removed lines

@@ -167,26 +167,26 @@
 			}
 		}
         
-        $site = URLDecode::getSite();
+		$site = URLDecode::getSite();
         
-        if ($site == 'blog') {
-            $subject = "New Comment on Jacob Emerick's Blog";
-            $message = "Hello!\nThere has been a new comment on the post '{$this->page_title}' at Jacob Emerick's Blog. You have chosen to be notified of it - please reply to jacob@jacobemerick.com if you would like to be removed from these notifications.\n\nOn " . date('F j, Y g:i a') . ", " . Request::getPost('name') . " commented...\n" . Request::getPost('comment') . "\n\nVisit {$this->full_path}#comments to see and reply to all the comments on this post.\nThank you!";
-        } else if ($site == 'waterfalls') {
-            $subject = "New Comment on Waterfalls of the Keweenaw";
-            $message = "Hello!\nThere has been a new comment on the page '{$this->page_title}' at Waterfalls of the Keweenaw. You have chosen to be notified of it - please reply to jacob@jacobemerick.com if you would like to be removed from these notifications.\n\nOn " . date('F j, Y g:i a') . ", " . Request::getPost('name') . " commented...\n" . Request::getPost('comment') . "\n\nVisit {$this->full_path}#comments to see and reply to all the comments on this post.\nThank you!";
-        }
+		if ($site == 'blog') {
+			$subject = "New Comment on Jacob Emerick's Blog";
+			$message = "Hello!\nThere has been a new comment on the post '{$this->page_title}' at Jacob Emerick's Blog. You have chosen to be notified of it - please reply to jacob@jacobemerick.com if you would like to be removed from these notifications.\n\nOn " . date('F j, Y g:i a') . ", " . Request::getPost('name') . " commented...\n" . Request::getPost('comment') . "\n\nVisit {$this->full_path}#comments to see and reply to all the comments on this post.\nThank you!";
+		} else if ($site == 'waterfalls') {
+			$subject = "New Comment on Waterfalls of the Keweenaw";
+			$message = "Hello!\nThere has been a new comment on the page '{$this->page_title}' at Waterfalls of the Keweenaw. You have chosen to be notified of it - please reply to jacob@jacobemerick.com if you would like to be removed from these notifications.\n\nOn " . date('F j, Y g:i a') . ", " . Request::getPost('name') . " commented...\n" . Request::getPost('comment') . "\n\nVisit {$this->full_path}#comments to see and reply to all the comments on this post.\nThank you!";
+		}
 
-    global $container;
+	global $container;
 
 		foreach($email_recipient_array as $email_recipient)
 		{
-      $sent = $container['mail']
-        ->addTo($email_recipient['email'], $email_recipient['name'])
-        ->addBCC($container['config']->admin_email)
-        ->setSubject($subject)
-        ->setPlainMessage($message)
-        ->send();		
+	  $sent = $container['mail']
+		->addTo($email_recipient['email'], $email_recipient['name'])
+		->addBCC($container['config']->admin_email)
+		->setSubject($subject)
+		->setPlainMessage($message)
+		->send();		
 		}
 	}