jacobemerick / web

router/PortfolioRouter.class.inc.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
             ],
         ];
 
-        return array_map(function ($row) {
+        return array_map(function($row) {
             return (object) $row;
         }, $paths);
     }
@@ -48,7 +48,7 @@ 
             ],
         ];
 
-        return array_map(function ($row) {
+        return array_map(function($row) {
             return (object) $row;
         }, $paths);
     }

Braces +4 added lines, -6 removed lines

@@ -2,11 +2,10 @@ 
 
 Loader::load('router', 'Router');
 
-class PortfolioRouter extends Router
-{
+class PortfolioRouter extends Router

+{

 
-    protected function get_redirect_array()
-    {
+    protected function get_redirect_array() {

         $paths = [
             [
                 'pattern' => '@/index.(html|htm|php)$@',
@@ -31,8 +30,7 @@ 
         }, $paths);
     }
 
-    protected function get_direct_array()
-    {
+    protected function get_direct_array() {

         $paths = [
             [
                 'match' => '/',

router/Router.class.inc.php

Spacing +15 added lines, -15 removed lines

@@ -18,10 +18,10 @@ 
 
 	private static function get_router_name()
 	{
-		if(Request::isAJAX())
+		if (Request::isAJAX())
 			return 'AJAXRouter';
 		
-		switch(URLDecode::getSite())
+		switch (URLDecode::getSite())
 		{
 			case 'ajax' :
 				return 'AjaxRouter';
@@ -65,42 +65,42 @@ 
 
 	final protected function check_for_redirect($redirect_uri)
 	{
-		foreach($this->get_redirect_array() as $check)
+		foreach ($this->get_redirect_array() as $check)
 		{
 			$redirect_uri = preg_replace($check->pattern, $check->replace, $redirect_uri);
 		}
 		
 		$redirect_uri = $this->check_for_special_redirect($redirect_uri);
 		
-		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
+		if ($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
 			$redirect_uri .= '/';
 		
         if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
             $protocol = (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') ? 'https' : 'http';
-            $redirect_uri = $protocol . '://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
+            $redirect_uri = $protocol.'://'.(!Loader::isLive() ? 'dev' : 'www').'.waterfallsofthekeweenaw.com'.$redirect_uri;
         }
         
-		if($redirect_uri == URLDecode::getURI())
+		if ($redirect_uri == URLDecode::getURI())
 			return;
 		
 		$controller_check = $redirect_uri;
-		if(substr($redirect_uri, 0, 4) == 'http') {
+		if (substr($redirect_uri, 0, 4) == 'http') {
       $protocol = (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') ? 'https' : 'http';
-			$controller_check = preg_replace('@^' . $protocol . '://([a-z\.]+)@', '', $redirect_uri);
+			$controller_check = preg_replace('@^'.$protocol.'://([a-z\.]+)@', '', $redirect_uri);
     }
 		
 		$controller = $this->get_controller($controller_check);
-		if($controller == '/Error404Controller')
+		if ($controller == '/Error404Controller')
 		{
 			Loader::loadNew('controller', '/Error404Controller')
 				->activate();
 			exit;
 		}
 
-		if(substr($redirect_uri, 0, 4) != 'http')
+		if (substr($redirect_uri, 0, 4) != 'http')
 		{
 			$redirect_uri = substr($redirect_uri, 1);
-			$redirect_uri = URLDecode::getBase() . $redirect_uri;
+			$redirect_uri = URLDecode::getBase().$redirect_uri;
 		}
 		
 		Loader::loadNew('controller', '/Error301Controller', (array) $redirect_uri)
@@ -114,12 +114,12 @@ 
 
 	final private function get_controller($uri)
 	{
-		foreach($this->get_direct_array() as $check)
+		foreach ($this->get_direct_array() as $check)
 		{
-			if($uri == $check->match)
+			if ($uri == $check->match)
 				return "{$this->get_primary_folder()}/{$check->controller}";
 			
-			if(preg_match("@^{$check->match}$@", $uri))
+			if (preg_match("@^{$check->match}$@", $uri))
 				return "{$this->get_primary_folder()}/{$check->controller}";
 		}
 		
@@ -128,7 +128,7 @@ 
 
 	final private function get_primary_folder()
 	{
-		if(Request::isAjax())
+		if (Request::isAjax())
 			return 'ajax';
 		
 		return URLDecode::getSite();

Indentation +105 added lines, -105 removed lines

@@ -1,144 +1,144 @@
 <?
 
 Loader::load('utility', array(
-	'Request',
-	'URLDecode'));
+    'Request',
+    'URLDecode'));
 
 abstract class Router
 {
 
-	public function __construct() {}
+    public function __construct() {}
 
-	public static function instance()
-	{
-		$router_name = self::get_router_name();
-		$router = Loader::loadNew('router', $router_name);
-		$router->route();
-	}
+    public static function instance()
+    {
+        $router_name = self::get_router_name();
+        $router = Loader::loadNew('router', $router_name);
+        $router->route();
+    }
 
-	private static function get_router_name()
-	{
-		if(Request::isAJAX())
-			return 'AJAXRouter';
+    private static function get_router_name()
+    {
+        if(Request::isAJAX())
+            return 'AJAXRouter';
 		
-		switch(URLDecode::getSite())
-		{
-			case 'ajax' :
-				return 'AjaxRouter';
-			break;
-			case 'blog' :
-				return 'BlogRouter';
-			break;
-			case 'home' :
-				return 'HomeRouter';
-			break;
-			case 'lifestream' :
-				return 'LifestreamRouter';
-			break;
-			case 'portfolio' :
-				return 'PortfolioRouter';
-			break;
-			case 'site' :
-				return 'SiteRouter';
-			break;
-			case 'waterfalls' :
-				return 'WaterfallRouter';
-			break;
-		}
+        switch(URLDecode::getSite())
+        {
+            case 'ajax' :
+                return 'AjaxRouter';
+            break;
+            case 'blog' :
+                return 'BlogRouter';
+            break;
+            case 'home' :
+                return 'HomeRouter';
+            break;
+            case 'lifestream' :
+                return 'LifestreamRouter';
+            break;
+            case 'portfolio' :
+                return 'PortfolioRouter';
+            break;
+            case 'site' :
+                return 'SiteRouter';
+            break;
+            case 'waterfalls' :
+                return 'WaterfallRouter';
+            break;
+        }
 		
-		Loader::loadNew('controller', '/Error404Controller')->activate();
-	}
+        Loader::loadNew('controller', '/Error404Controller')->activate();
+    }
 
-	protected function route()
-	{
-		$uri = URLDecode::getURI();
+    protected function route()
+    {
+        $uri = URLDecode::getURI();
 		
-		$this->check_for_redirect($uri);
+        $this->check_for_redirect($uri);
 		
-		$controller = $this->get_controller($uri);
-		Loader::loadNew('controller', $controller)
-			->activate();
-	}
+        $controller = $this->get_controller($uri);
+        Loader::loadNew('controller', $controller)
+            ->activate();
+    }
 
-	abstract protected function get_redirect_array();
-	abstract protected function get_direct_array();
+    abstract protected function get_redirect_array();
+    abstract protected function get_direct_array();
 
-	final protected function check_for_redirect($redirect_uri)
-	{
-		foreach($this->get_redirect_array() as $check)
-		{
-			$redirect_uri = preg_replace($check->pattern, $check->replace, $redirect_uri);
-		}
+    final protected function check_for_redirect($redirect_uri)
+    {
+        foreach($this->get_redirect_array() as $check)
+        {
+            $redirect_uri = preg_replace($check->pattern, $check->replace, $redirect_uri);
+        }
 		
-		$redirect_uri = $this->check_for_special_redirect($redirect_uri);
+        $redirect_uri = $this->check_for_special_redirect($redirect_uri);
 		
-		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
-			$redirect_uri .= '/';
+        if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
+            $redirect_uri .= '/';
 		
         if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
             $protocol = (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') ? 'https' : 'http';
             $redirect_uri = $protocol . '://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
         }
         
-		if($redirect_uri == URLDecode::getURI())
-			return;
+        if($redirect_uri == URLDecode::getURI())
+            return;
 		
-		$controller_check = $redirect_uri;
-		if(substr($redirect_uri, 0, 4) == 'http') {
-      $protocol = (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') ? 'https' : 'http';
-			$controller_check = preg_replace('@^' . $protocol . '://([a-z\.]+)@', '', $redirect_uri);
+        $controller_check = $redirect_uri;
+        if(substr($redirect_uri, 0, 4) == 'http') {
+        $protocol = (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') ? 'https' : 'http';
+            $controller_check = preg_replace('@^' . $protocol . '://([a-z\.]+)@', '', $redirect_uri);
     }
 		
-		$controller = $this->get_controller($controller_check);
-		if($controller == '/Error404Controller')
-		{
-			Loader::loadNew('controller', '/Error404Controller')
-				->activate();
-			exit;
-		}
+        $controller = $this->get_controller($controller_check);
+        if($controller == '/Error404Controller')
+        {
+            Loader::loadNew('controller', '/Error404Controller')
+                ->activate();
+            exit;
+        }
 
-		if(substr($redirect_uri, 0, 4) != 'http')
-		{
-			$redirect_uri = substr($redirect_uri, 1);
-			$redirect_uri = URLDecode::getBase() . $redirect_uri;
-		}
+        if(substr($redirect_uri, 0, 4) != 'http')
+        {
+            $redirect_uri = substr($redirect_uri, 1);
+            $redirect_uri = URLDecode::getBase() . $redirect_uri;
+        }
 		
-		Loader::loadNew('controller', '/Error301Controller', (array) $redirect_uri)
-			->activate();
-	}
+        Loader::loadNew('controller', '/Error301Controller', (array) $redirect_uri)
+            ->activate();
+    }
 
-	protected function check_for_special_redirect($uri)
-	{
-		return $uri;
-	}
+    protected function check_for_special_redirect($uri)
+    {
+        return $uri;
+    }
 
-	final private function get_controller($uri)
-	{
-		foreach($this->get_direct_array() as $check)
-		{
-			if($uri == $check->match)
-				return "{$this->get_primary_folder()}/{$check->controller}";
+    final private function get_controller($uri)
+    {
+        foreach($this->get_direct_array() as $check)
+        {
+            if($uri == $check->match)
+                return "{$this->get_primary_folder()}/{$check->controller}";
 			
-			if(preg_match("@^{$check->match}$@", $uri))
-				return "{$this->get_primary_folder()}/{$check->controller}";
-		}
+            if(preg_match("@^{$check->match}$@", $uri))
+                return "{$this->get_primary_folder()}/{$check->controller}";
+        }
 		
-		return '/Error404Controller';
-	}
+        return '/Error404Controller';
+    }
 
-	final private function get_primary_folder()
-	{
-		if(Request::isAjax())
-			return 'ajax';
+    final private function get_primary_folder()
+    {
+        if(Request::isAjax())
+            return 'ajax';
 		
-		return URLDecode::getSite();
-	}
+        return URLDecode::getSite();
+    }
 
-	private function requires_trailing_slash()
-	{
-		return (
-			URLDecode::getExtension() != 'json' &&
+    private function requires_trailing_slash()
+    {
+        return (
+            URLDecode::getExtension() != 'json' &&
             strstr(URLDecode::getURI(), '#') === false);
-	}
+    }
 
 }

Braces +28 added lines, -30 removed lines

@@ -4,22 +4,21 @@ 
 	'Request',
 	'URLDecode'));
 
-abstract class Router
-{
+abstract class Router

+{

 
 	public function __construct() {}
 
-	public static function instance()
-	{
+	public static function instance() {

 		$router_name = self::get_router_name();
 		$router = Loader::loadNew('router', $router_name);
 		$router->route();
 	}
 
-	private static function get_router_name()
-	{
-		if(Request::isAJAX())
-			return 'AJAXRouter';
+	private static function get_router_name() {

+		if(Request::isAJAX()) {

+					return 'AJAXRouter';

+		}

 		
 		switch(URLDecode::getSite())
 		{
@@ -49,8 +48,7 @@ 
 		Loader::loadNew('controller', '/Error404Controller')->activate();
 	}
 
-	protected function route()
-	{
+	protected function route() {

 		$uri = URLDecode::getURI();
 		
 		$this->check_for_redirect($uri);
@@ -63,8 +61,7 @@ 
 	abstract protected function get_redirect_array();
 	abstract protected function get_direct_array();
 
-	final protected function check_for_redirect($redirect_uri)
-	{
+	final protected function check_for_redirect($redirect_uri) {

 		foreach($this->get_redirect_array() as $check)
 		{
 			$redirect_uri = preg_replace($check->pattern, $check->replace, $redirect_uri);
@@ -72,16 +69,18 @@ 
 		
 		$redirect_uri = $this->check_for_special_redirect($redirect_uri);
 		
-		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
-			$redirect_uri .= '/';
+		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/') {

+					$redirect_uri .= '/';

+		}

 		
         if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
             $protocol = (!empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] !== 'off') ? 'https' : 'http';
             $redirect_uri = $protocol . '://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
         }
         
-		if($redirect_uri == URLDecode::getURI())
-			return;
+		if($redirect_uri == URLDecode::getURI()) {

+					return;

+		}

 		
 		$controller_check = $redirect_uri;
 		if(substr($redirect_uri, 0, 4) == 'http') {
@@ -107,35 +106,34 @@ 
 			->activate();
 	}
 
-	protected function check_for_special_redirect($uri)
-	{
+	protected function check_for_special_redirect($uri) {

 		return $uri;
 	}
 
-	final private function get_controller($uri)
-	{
+	final private function get_controller($uri) {

 		foreach($this->get_direct_array() as $check)
 		{
-			if($uri == $check->match)
-				return "{$this->get_primary_folder()}/{$check->controller}";
+			if($uri == $check->match) {

+							return "{$this->get_primary_folder()}/{$check->controller}";

+			}

 			
-			if(preg_match("@^{$check->match}$@", $uri))
-				return "{$this->get_primary_folder()}/{$check->controller}";
+			if(preg_match("@^{$check->match}$@", $uri)) {

+							return "{$this->get_primary_folder()}/{$check->controller}";

+			}

 		}
 		
 		return '/Error404Controller';
 	}
 
-	final private function get_primary_folder()
-	{
-		if(Request::isAjax())
-			return 'ajax';
+	final private function get_primary_folder() {

+		if(Request::isAjax()) {

+					return 'ajax';

+		}

 		
 		return URLDecode::getSite();
 	}
 
-	private function requires_trailing_slash()
-	{
+	private function requires_trailing_slash() {

 		return (
 			URLDecode::getExtension() != 'json' &&
             strstr(URLDecode::getURI(), '#') === false);

public/portfolio/index.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 
 $namespace = 'portfolio';
-require_once __DIR__ . '/../../bootstrap.php';
+require_once __DIR__.'/../../bootstrap.php';
 
 // route
 Loader::loadInstance('router', 'Router');

utility/content/ImperialUnitContent.class.inc.php

Spacing +5 added lines, -5 removed lines

@@ -26,16 +26,16 @@ 
         
         switch ($type) {
             case 'full miles' :
-                $this->content = number_format(round($number / (12 * 3 * 1760))) . ' miles';
+                $this->content = number_format(round($number / (12 * 3 * 1760))).' miles';
                 break;
             case 'tenth miles' :
-                $this->content = round($number / (12 * 3 * 1760), 1) . ' miles';
+                $this->content = round($number / (12 * 3 * 1760), 1).' miles';
                 break;
             case 'yards' :
-                $this->content = number_format(round($number / (12 * 3))) . ' yards';
+                $this->content = number_format(round($number / (12 * 3))).' yards';
                 break;
             case 'feet' :
-                $this->content = number_format(round($number / 12)) . "'";
+                $this->content = number_format(round($number / 12))."'";
                 break;
             case 'inches' :
                 $feet = floor($number / 12);
@@ -49,7 +49,7 @@ 
                 }
                 
                 $this->content = '';
-                $this->content .= number_format($feet) . "'";
+                $this->content .= number_format($feet)."'";
                 if (isset($inches) && $inches > 0) {
                     $this->content .= " {$inches}\"";
                 }

Braces +3 added lines, -4 removed lines

@@ -2,11 +2,10 @@
 
 Loader::load('utility', 'Content');
 
-final class ImperialUnitContent extends Content
-{
+final class ImperialUnitContent extends Content

+{

 
-    protected function execute($type = '')
-    {
+    protected function execute($type = '') {

         $number = floatval($this->content);
         $number *= 39.37; // convert to inches
         

module/form/CommentSubmitModule.class.inc.php

Spacing +21 added lines, -21 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
 		
@@ -152,13 +152,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(
@@ -171,15 +171,15 @@ 
         
         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!";
+            $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!";
+            $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;
 
-		foreach($email_recipient_array as $email_recipient)
+		foreach ($email_recipient_array as $email_recipient)
 		{
       $sent = $container['mail']
         ->addTo($email_recipient['email'], $email_recipient['name'])

Indentation +148 added lines, -148 removed lines

@@ -3,165 +3,165 @@ 
 Loader::load('collector', 'comment/CommentCollector');
 
 Loader::load('utility', array(
-	'Content',
-	'Request',
-	'Validate'));
+    'Content',
+    'Request',
+    'Validate'));
 
 final class CommentSubmitModule
 {
 
-	private $site;
-	private $path;
-	private $full_path;
-	private $page_title;
-
-	public function __construct($site, $path, $full_path, $page_title)
-	{
-		$this->site = $site;
-		$this->path = $path;
-		$this->full_path = $full_path;
-		$this->page_title = $page_title;
-	}
-
-	public function activate()
-	{
-		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;
-		
-		$page_id = $this->save_comment_page();
-		$commenter_id = $this->save_commenter();
-		$comment_id = $this->save_comment();
-		
-		$comment_meta_id = $this->save_comment_meta($commenter_id, $comment_id, $page_id);
+    private $site;
+    private $path;
+    private $full_path;
+    private $page_title;
+
+    public function __construct($site, $path, $full_path, $page_title)
+    {
+        $this->site = $site;
+        $this->path = $path;
+        $this->full_path = $full_path;
+        $this->page_title = $page_title;
+    }
+
+    public function activate()
+    {
+        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;
+		
+        $page_id = $this->save_comment_page();
+        $commenter_id = $this->save_commenter();
+        $comment_id = $this->save_comment();
+		
+        $comment_meta_id = $this->save_comment_meta($commenter_id, $comment_id, $page_id);
     $comment_service_id = $this->save_to_comment_service(Request::getPost());
 		
-		$this->send_notifications($page_id);
-		$this->redirect_to_comment($comment_meta_id);
-	}
-
-	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';
-		
-		return $errors;
-	}
-
-	private function save_comment_page()
-	{
-		$page_result = CommentCollector::getCommentPageByURL($this->path, $this->site);
-		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);
-		Database::execute($query);
-		
-		return Database::lastInsertID();
-	}
-
-	private function save_commenter()
-	{
-		$cookie_value = array(
-			'name' => Request::getPost('name'),
-			'email' => Request::getPost('email'));
-		
-		if(Request::getPost('website') != '')
-			$cookie_value['website'] = Request::getPost('website');
+        $this->send_notifications($page_id);
+        $this->redirect_to_comment($comment_meta_id);
+    }
+
+    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';
+		
+        return $errors;
+    }
+
+    private function save_comment_page()
+    {
+        $page_result = CommentCollector::getCommentPageByURL($this->path, $this->site);
+        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);
+        Database::execute($query);
+		
+        return Database::lastInsertID();
+    }
+
+    private function save_commenter()
+    {
+        $cookie_value = array(
+            'name' => Request::getPost('name'),
+            'email' => Request::getPost('email'));
+		
+        if(Request::getPost('website') != '')
+            $cookie_value['website'] = Request::getPost('website');
 
     setcookie('commenter', json_encode($cookie_value), time() + 31536000, '/', 'jacobemerick.com');
 
-		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
-		if($commenter_result !== null)
-			return $commenter_result->id;
+        $commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
+        if($commenter_result !== null)
+            return $commenter_result->id;
 		
-		$query = "INSERT INTO `jpemeric_comment`.`commenter` (`name`,`email`,`url`) VALUES ('%s','%s','%s')";
+        $query = "INSERT INTO `jpemeric_comment`.`commenter` (`name`,`email`,`url`) VALUES ('%s','%s','%s')";
 		
-		$name = Database::escape(Request::getPost('name'));
-		$email = Database::escape(Request::getPost('email'));
-		$website = Database::escape(Request::getPost('website'));
+        $name = Database::escape(Request::getPost('name'));
+        $email = Database::escape(Request::getPost('email'));
+        $website = Database::escape(Request::getPost('website'));
 		
-		$query = sprintf($query, $name, $email, $website);
-		Database::execute($query);
-		return Database::lastInsertID();
-	}
+        $query = sprintf($query, $name, $email, $website);
+        Database::execute($query);
+        return Database::lastInsertID();
+    }
 
-	private function save_comment()
-	{
-		$comment_result = CommentCollector::getCommentByBody(Request::getPost('comment'));
-		if($comment_result !== null)
-			return $comment_result->id;
+    private function save_comment()
+    {
+        $comment_result = CommentCollector::getCommentByBody(Request::getPost('comment'));
+        if($comment_result !== null)
+            return $comment_result->id;
 		
-		$query = "INSERT INTO `jpemeric_comment`.`comment` (`body`, `body_format`) VALUES ('%s', '%s')";
+        $query = "INSERT INTO `jpemeric_comment`.`comment` (`body`, `body_format`) VALUES ('%s', '%s')";
 		
-		$body = Database::escape(Request::getPost('comment'));
+        $body = Database::escape(Request::getPost('comment'));
 		
-		$body_format = Request::getPost('comment');
-		$body_format = Content::instance('CleanComment', $body_format)->activate();
-		$body_format = Database::escape($body_format);
+        $body_format = Request::getPost('comment');
+        $body_format = Content::instance('CleanComment', $body_format)->activate();
+        $body_format = Database::escape($body_format);
 		
-		$query = sprintf($query, $body, $body_format);
-		Database::execute($query);
-		return Database::lastInsertID();
-	}
+        $query = sprintf($query, $body, $body_format);
+        Database::execute($query);
+        return Database::lastInsertID();
+    }
 
-	private function save_comment_meta($commenter, $comment, $page)
-	{
-		$query = "INSERT INTO `jpemeric_comment`.`comment_meta` (`commenter`,`comment`,`reply`,`notify`,`comment_page`,`date`,`display`) VALUES ('%d','%d','%d','%d','%d','%s','%d')";
+    private function save_comment_meta($commenter, $comment, $page)
+    {
+        $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;
-		// else check to make sure value is legit
+        $reply = Database::escape(Request::getPost('type'));
+        if($reply == 'new')
+            $reply = 0;
+        // else check to make sure value is legit
 		
-		$notify = (Request::getPost('notify') == 'check') ? 1 : 0;
-		$date = date('Y-m-d H:i:s');
-		$display = 1;
+        $notify = (Request::getPost('notify') == 'check') ? 1 : 0;
+        $date = date('Y-m-d H:i:s');
+        $display = 1;
 		
-		$query = sprintf($query, $commenter, $comment, $reply, $notify, $page, $date, $display);
-		Database::execute($query);
-		return Database::lastInsertID();
-	}
+        $query = sprintf($query, $commenter, $comment, $reply, $notify, $page, $date, $display);
+        Database::execute($query);
+        return Database::lastInsertID();
+    }
 
-	private function send_notifications($page)
-	{
-		$email_recipient_array = array();
+    private function send_notifications($page)
+    {
+        $email_recipient_array = array();
 		
-		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
+        $commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
 		
-		if($commenter_result->trusted == 1)
-		{
-			$notification_result = CommentCollector::getNotificationForPage($page);
+        if($commenter_result->trusted == 1)
+        {
+            $notification_result = CommentCollector::getNotificationForPage($page);
 			
-			foreach($notification_result as $notification_row)
-			{
-				if($notification_row->email == Request::getPost('email'))
-					continue;
+            foreach($notification_result as $notification_row)
+            {
+                if($notification_row->email == Request::getPost('email'))
+                    continue;
 				
-				$email_recipient_array[$notification_row->email] = array(
-					'email' => $notification_row->email,
-					'name' => $notification_row->name);
-			}
-		}
+                $email_recipient_array[$notification_row->email] = array(
+                    'email' => $notification_row->email,
+                    'name' => $notification_row->name);
+            }
+        }
         
         $site = URLDecode::getSite();
         
@@ -175,26 +175,26 @@ 
 
     global $container;
 
-		foreach($email_recipient_array as $email_recipient)
-		{
-      $sent = $container['mail']
+        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();		
-		}
-	}
-
-	private function redirect_to_comment($comment_id)
-	{
-		$url = '';
-		$url .= $this->full_path;
-		$url .= "#comment-{$comment_id}";
-		
-		Loader::loadNew('controller', 'Error303Controller', array($url))->activate();
-		exit;
-	}
+        }
+    }
+
+    private function redirect_to_comment($comment_id)
+    {
+        $url = '';
+        $url .= $this->full_path;
+        $url .= "#comment-{$comment_id}";
+		
+        Loader::loadNew('controller', 'Error303Controller', array($url))->activate();
+        exit;
+    }
 
     private function save_to_comment_service(array $data)
     {

Braces +60 added lines, -54 removed lines

@@ -7,34 +7,36 @@ 
 	'Request',
 	'Validate'));
 
-final class CommentSubmitModule
-{
+final class CommentSubmitModule

+{

 
 	private $site;
 	private $path;
 	private $full_path;
 	private $page_title;
 
-	public function __construct($site, $path, $full_path, $page_title)
-	{
+	public function __construct($site, $path, $full_path, $page_title) {

 		$this->site = $site;
 		$this->path = $path;
 		$this->full_path = $full_path;
 		$this->page_title = $page_title;
 	}
 
-	public function activate()
-	{
-		if(!Request::hasPost())
-			return false;
-		if(!Request::getPost('submit') == 'Submit Comment')
-			return false;
+	public function activate() {

+		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();
@@ -47,30 +49,35 @@ 
 		$this->redirect_to_comment($comment_meta_id);
 	}
 
-	private function fetch_errors()
-	{
+	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;
 	}
 
-	private function save_comment_page()
-	{
+	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);
@@ -79,20 +86,21 @@ 
 		return Database::lastInsertID();
 	}
 
-	private function save_commenter()
-	{
+	private function save_commenter() {

 		$cookie_value = array(
 			'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');

+		}

 
     setcookie('commenter', json_encode($cookie_value), time() + 31536000, '/', 'jacobemerick.com');
 
 		$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')";
 		
@@ -105,11 +113,11 @@ 
 		return Database::lastInsertID();
 	}
 
-	private function save_comment()
-	{
+	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')";
 		
@@ -124,13 +132,13 @@ 
 		return Database::lastInsertID();
 	}
 
-	private function save_comment_meta($commenter, $comment, $page)
-	{
+	private function save_comment_meta($commenter, $comment, $page) {

 		$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;
@@ -142,8 +150,7 @@ 
 		return Database::lastInsertID();
 	}
 
-	private function send_notifications($page)
-	{
+	private function send_notifications($page) {

 		$email_recipient_array = array();
 		
 		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
@@ -154,8 +161,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,
@@ -186,8 +194,7 @@ 
 		}
 	}
 
-	private function redirect_to_comment($comment_id)
-	{
+	private function redirect_to_comment($comment_id) {

 		$url = '';
 		$url .= $this->full_path;
 		$url .= "#comment-{$comment_id}";
@@ -196,8 +203,7 @@ 
 		exit;
 	}
 
-    private function save_to_comment_service(array $data)
-    {
+    private function save_to_comment_service(array $data) {

         $path = $_SERVER['REQUEST_URI'];
         $path = explode('/', $path);
         $path = array_filter($path);

controller/lifestream/TagController.class.inc.php

Spacing +10 added lines, -10 removed lines

@@ -46,7 +46,7 @@ 
 
 	private function get_title()
 	{
-		switch($this->tag)
+		switch ($this->tag)
 		{
 			case 'blog' :
 				return 'Jacob has a Blog';
@@ -74,7 +74,7 @@ 
 
 	private function get_description()
 	{
-		switch($this->tag)
+		switch ($this->tag)
 		{
 			case 'blog' :
 				return 'Yeah, Jacob has a blog. Check out his posting activity.';
@@ -103,7 +103,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;
 	}
@@ -123,24 +123,24 @@ 
 
 	protected function get_list_next_link()
 	{
-		if($this->page == 1)
+		if ($this->page == 1)
 			return;
-		if($this->page == 2)
-			return Loader::getRootUrl('lifestream') . $this->tag . '/';
-		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page - 1) . '/';
+		if ($this->page == 2)
+			return Loader::getRootUrl('lifestream').$this->tag.'/';
+		return Loader::getRootUrl('lifestream').$this->tag.'/page/'.($this->page - 1).'/';
 	}
 
 	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 Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page + 1) . '/';
+		return Loader::getRootUrl('lifestream').$this->tag.'/page/'.($this->page + 1).'/';
 	}
 
 	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 = $this->activityRepository->getActivitiesByTypeCount($this->tag);
 		return $this->total_post_count;
 	}

Indentation +126 added lines, -126 removed lines

@@ -5,144 +5,144 @@
 final class TagController extends DefaultListController
 {
 
-	private static $TITLE = "%s Activity | Jacob Emerick's Lifestream";
-	private static $DESCRIPTION = "%s activity on Jacob Emerick's lifestream, all nice and paginated in one awesome website.";
+    private static $TITLE = "%s Activity | Jacob Emerick's Lifestream";
+    private static $DESCRIPTION = "%s activity on Jacob Emerick's lifestream, all nice and paginated in one awesome website.";
 
-	private static $KEYWORD_ARRAY = array(
-		'lifestream',
-		'activity stream',
-		'Jacob Emerick');
+    private static $KEYWORD_ARRAY = array(
+        'lifestream',
+        'activity stream',
+        'Jacob Emerick');
 
-	private static $LIST_DESCRIPTION = 'viewing %d - %d of %d %s activities';
+    private static $LIST_DESCRIPTION = 'viewing %d - %d of %d %s activities';
 
-	private $tag;
+    private $tag;
 
-	public function __construct()
-	{
-		parent::__construct();
+    public function __construct()
+    {
+        parent::__construct();
 		
-		$this->tag = URLDecode::getPiece(1);
-	}
+        $this->tag = URLDecode::getPiece(1);
+    }
 
-	protected function set_head_data()
-	{
-		$this->set_title(sprintf(self::$TITLE, ucwords($this->tag)));
-		$this->set_description(sprintf(self::$DESCRIPTION, ucwords($this->tag)));
+    protected function set_head_data()
+    {
+        $this->set_title(sprintf(self::$TITLE, ucwords($this->tag)));
+        $this->set_description(sprintf(self::$DESCRIPTION, ucwords($this->tag)));
 		
-		$keyword_array = self::$KEYWORD_ARRAY;
-		array_unshift($keyword_array, $this->tag);
-		$this->set_keywords($keyword_array);
+        $keyword_array = self::$KEYWORD_ARRAY;
+        array_unshift($keyword_array, $this->tag);
+        $this->set_keywords($keyword_array);
 		
-		parent::set_head_data();
-	}
+        parent::set_head_data();
+    }
 
-	protected function set_body_data()
-	{
-		$this->set_body('title', $this->get_title());
-		$this->set_body('description', $this->get_description());
+    protected function set_body_data()
+    {
+        $this->set_body('title', $this->get_title());
+        $this->set_body('description', $this->get_description());
 		
-		parent::set_body_data();
-	}
-
-	private function get_title()
-	{
-		switch($this->tag)
-		{
-			case 'blog' :
-				return 'Jacob has a Blog';
-			break;
-			case 'book' :
-				return 'Jacob reads Books';
-			break;
-			case 'distance' :
-				return 'Run, Jacob, and Hike';
-			break;
-      case 'github' :
+        parent::set_body_data();
+    }
+
+    private function get_title()
+    {
+        switch($this->tag)
+        {
+            case 'blog' :
+                return 'Jacob has a Blog';
+            break;
+            case 'book' :
+                return 'Jacob reads Books';
+            break;
+            case 'distance' :
+                return 'Run, Jacob, and Hike';
+            break;
+        case 'github' :
         return 'Jacob, Code Monkey';
-      break;
-			case 'hulu' :
-				return 'Jacob watches Hulu';
-			break;
-			case 'twitter' :
-				return 'Jacob likes to Tweet';
-			break;
-			case 'youtube' :
-				return 'Videos Jacob Likes';
-			break;
-		}
-	}
-
-	private function get_description()
-	{
-		switch($this->tag)
-		{
-			case 'blog' :
-				return 'Yeah, Jacob has a blog. Check out his posting activity.';
-			break;
-			case 'book' :
-				return "Outside of building websites, hiking, and <em>being awesome</em>, Jacob reads books. Check out what he's been reading.";
-			break;
-			case 'distance' :
-				return 'All the cool kids like to be in shape. Jacob goes running and hiking.';
-			break;
-      case 'github' :
+        break;
+            case 'hulu' :
+                return 'Jacob watches Hulu';
+            break;
+            case 'twitter' :
+                return 'Jacob likes to Tweet';
+            break;
+            case 'youtube' :
+                return 'Videos Jacob Likes';
+            break;
+        }
+    }
+
+    private function get_description()
+    {
+        switch($this->tag)
+        {
+            case 'blog' :
+                return 'Yeah, Jacob has a blog. Check out his posting activity.';
+            break;
+            case 'book' :
+                return "Outside of building websites, hiking, and <em>being awesome</em>, Jacob reads books. Check out what he's been reading.";
+            break;
+            case 'distance' :
+                return 'All the cool kids like to be in shape. Jacob goes running and hiking.';
+            break;
+        case 'github' :
         return 'Since he is trying to be a developer, Jacob codes. Here is some code that he pushed around on the Githubs.';
-      break;
-			case 'hulu' :
-				return 'Occasionally Jacob chills and watches some Hulu. Well, used to, anyways.';
-			break;
-			case 'twitter' :
-				return 'All the cool kids are on Twitter. The cool kids and Jacob. Check his tweets, yo.';
-			break;
-			case 'youtube' :
-				return 'Jacob watches videos on YouTube. Some videos he likes more than others.';
-			break;
-		}
-	}
-
-	protected function get_page_number()
-	{
-		$page = URLDecode::getPiece(3);
-		if(isset($page) && is_numeric($page))
-			return $page;
-		return 1;
-	}
-
-	protected function get_list_results()
-	{
-		return $this->activityRepository->getActivitiesByType($this->tag, self::$POSTS_PER_PAGE, $this->offset);
-	}
-
-	protected function get_list_description()
-	{
-		$start = $this->offset + 1;
-		$end = min($this->offset + self::$POSTS_PER_PAGE, $this->get_total_post_count());
+        break;
+            case 'hulu' :
+                return 'Occasionally Jacob chills and watches some Hulu. Well, used to, anyways.';
+            break;
+            case 'twitter' :
+                return 'All the cool kids are on Twitter. The cool kids and Jacob. Check his tweets, yo.';
+            break;
+            case 'youtube' :
+                return 'Jacob watches videos on YouTube. Some videos he likes more than others.';
+            break;
+        }
+    }
+
+    protected function get_page_number()
+    {
+        $page = URLDecode::getPiece(3);
+        if(isset($page) && is_numeric($page))
+            return $page;
+        return 1;
+    }
+
+    protected function get_list_results()
+    {
+        return $this->activityRepository->getActivitiesByType($this->tag, self::$POSTS_PER_PAGE, $this->offset);
+    }
+
+    protected function get_list_description()
+    {
+        $start = $this->offset + 1;
+        $end = min($this->offset + self::$POSTS_PER_PAGE, $this->get_total_post_count());
 		
-		return sprintf(self::$LIST_DESCRIPTION, $start, $end, $this->get_total_post_count(), $this->tag);
-	}
-
-	protected function get_list_next_link()
-	{
-		if($this->page == 1)
-			return;
-		if($this->page == 2)
-			return Loader::getRootUrl('lifestream') . $this->tag . '/';
-		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page - 1) . '/';
-	}
-
-	protected function get_list_prev_link()
-	{
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
-			return;
-		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page + 1) . '/';
-	}
-
-	private $total_post_count;
-	protected function get_total_post_count()
-	{
-		if(!isset($this->total_post_count))
-			$this->total_post_count = $this->activityRepository->getActivitiesByTypeCount($this->tag);
-		return $this->total_post_count;
-	}
+        return sprintf(self::$LIST_DESCRIPTION, $start, $end, $this->get_total_post_count(), $this->tag);
+    }
+
+    protected function get_list_next_link()
+    {
+        if($this->page == 1)
+            return;
+        if($this->page == 2)
+            return Loader::getRootUrl('lifestream') . $this->tag . '/';
+        return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page - 1) . '/';
+    }
+
+    protected function get_list_prev_link()
+    {
+        if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
+            return;
+        return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page + 1) . '/';
+    }
+
+    private $total_post_count;
+    protected function get_total_post_count()
+    {
+        if(!isset($this->total_post_count))
+            $this->total_post_count = $this->activityRepository->getActivitiesByTypeCount($this->tag);
+        return $this->total_post_count;
+    }
 
 }

Braces +28 added lines, -34 removed lines

@@ -2,8 +2,8 @@ 
 
 Loader::load('controller', 'lifestream/DefaultListController');
 
-final class TagController extends DefaultListController
-{
+final class TagController extends DefaultListController

+{

 
 	private static $TITLE = "%s Activity | Jacob Emerick's Lifestream";
 	private static $DESCRIPTION = "%s activity on Jacob Emerick's lifestream, all nice and paginated in one awesome website.";
@@ -17,15 +17,13 @@ 
 
 	private $tag;
 
-	public function __construct()
-	{
+	public function __construct() {

 		parent::__construct();
 		
 		$this->tag = URLDecode::getPiece(1);
 	}
 
-	protected function set_head_data()
-	{
+	protected function set_head_data() {

 		$this->set_title(sprintf(self::$TITLE, ucwords($this->tag)));
 		$this->set_description(sprintf(self::$DESCRIPTION, ucwords($this->tag)));
 		
@@ -36,16 +34,14 @@ 
 		parent::set_head_data();
 	}
 
-	protected function set_body_data()
-	{
+	protected function set_body_data() {

 		$this->set_body('title', $this->get_title());
 		$this->set_body('description', $this->get_description());
 		
 		parent::set_body_data();
 	}
 
-	private function get_title()
-	{
+	private function get_title() {

 		switch($this->tag)
 		{
 			case 'blog' :
@@ -72,8 +68,7 @@ 
 		}
 	}
 
-	private function get_description()
-	{
+	private function get_description() {

 		switch($this->tag)
 		{
 			case 'blog' :
@@ -100,48 +95,47 @@ 
 		}
 	}
 
-	protected function get_page_number()
-	{
+	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;
 	}
 
-	protected function get_list_results()
-	{
+	protected function get_list_results() {

 		return $this->activityRepository->getActivitiesByType($this->tag, self::$POSTS_PER_PAGE, $this->offset);
 	}
 
-	protected function get_list_description()
-	{
+	protected function get_list_description() {

 		$start = $this->offset + 1;
 		$end = min($this->offset + self::$POSTS_PER_PAGE, $this->get_total_post_count());
 		
 		return sprintf(self::$LIST_DESCRIPTION, $start, $end, $this->get_total_post_count(), $this->tag);
 	}
 
-	protected function get_list_next_link()
-	{
-		if($this->page == 1)
-			return;
-		if($this->page == 2)
-			return Loader::getRootUrl('lifestream') . $this->tag . '/';
+	protected function get_list_next_link() {

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

+					return;

+		}

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

+					return Loader::getRootUrl('lifestream') . $this->tag . '/';

+		}

 		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page - 1) . '/';
 	}
 
-	protected function get_list_prev_link()
-	{
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
-			return;
+	protected function get_list_prev_link() {

+		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count()) {

+					return;

+		}

 		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page + 1) . '/';
 	}
 
 	private $total_post_count;
-	protected function get_total_post_count()
-	{
-		if(!isset($this->total_post_count))
-			$this->total_post_count = $this->activityRepository->getActivitiesByTypeCount($this->tag);
+	protected function get_total_post_count() {

+		if(!isset($this->total_post_count)) {

+					$this->total_post_count = $this->activityRepository->getActivitiesByTypeCount($this->tag);

+		}

 		return $this->total_post_count;
 	}
 

controller/lifestream/HomeController.class.inc.php

Spacing +7 added lines, -7 removed lines

@@ -38,7 +38,7 @@ 
 	protected function get_page_number()
 	{
 		$page = URLDecode::getPiece(2);
-		if(isset($page) && is_numeric($page))
+		if (isset($page) && is_numeric($page))
 			return $page;
 		return 1;
 	}
@@ -58,24 +58,24 @@ 
 
 	protected function get_list_next_link()
 	{
-		if($this->page == 1)
+		if ($this->page == 1)
 			return;
-		if($this->page == 2)
+		if ($this->page == 2)
 			return Loader::getRootUrl('lifestream');
-		return Loader::getRootUrl('lifestream') . 'page/' . ($this->page - 1) . '/';
+		return Loader::getRootUrl('lifestream').'page/'.($this->page - 1).'/';
 	}
 
 	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 Loader::getRootUrl('lifestream') . 'page/' . ($this->page + 1) . '/';
+		return Loader::getRootUrl('lifestream').'page/'.($this->page + 1).'/';
 	}
 
 	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 = $this->activityRepository->getActivitiesCount();
 		return $this->total_post_count;
 	}

Indentation +60 added lines, -60 removed lines

@@ -5,79 +5,79 @@
 final class HomeController extends DefaultListController
 {
 
-	private static $TITLE = "Jacob Emerick's Lifestream";
-	private static $DESCRIPTION = 'Lifestream for Jacob Emerick, a combination of his latest twitter, youtube, running, reading, and blogging activity.';
+    private static $TITLE = "Jacob Emerick's Lifestream";
+    private static $DESCRIPTION = 'Lifestream for Jacob Emerick, a combination of his latest twitter, youtube, running, reading, and blogging activity.';
 
-	private static $KEYWORD_ARRAY = array(
-		'lifestream',
-		'activity stream',
-		'Jacob Emerick',
-		'jacobemerick',
-		'jpemeric',
-		'twitter');
+    private static $KEYWORD_ARRAY = array(
+        'lifestream',
+        'activity stream',
+        'Jacob Emerick',
+        'jacobemerick',
+        'jpemeric',
+        'twitter');
 
-	private static $LIST_DESCRIPTION = 'viewing %d - %d of %d total activities';
+    private static $LIST_DESCRIPTION = 'viewing %d - %d of %d total activities';
 
-	protected function set_head_data()
-	{
-		$this->set_title(self::$TITLE);
-		$this->set_description(self::$DESCRIPTION);
-		$this->set_keywords(self::$KEYWORD_ARRAY);
+    protected function set_head_data()
+    {
+        $this->set_title(self::$TITLE);
+        $this->set_description(self::$DESCRIPTION);
+        $this->set_keywords(self::$KEYWORD_ARRAY);
 		
-		parent::set_head_data();
-	}
+        parent::set_head_data();
+    }
 
-	protected function set_body_data()
-	{
-		$this->set_body('title', "What's Jacob Up To?");
-		$this->set_body('description', "A combination of Jacob Emerick's tweets, runs, reads, blogs, and YouTubes all in <em>one awesome lifestream</em>.");
+    protected function set_body_data()
+    {
+        $this->set_body('title', "What's Jacob Up To?");
+        $this->set_body('description', "A combination of Jacob Emerick's tweets, runs, reads, blogs, and YouTubes all in <em>one awesome lifestream</em>.");
 		
-		parent::set_body_data();
-	}
+        parent::set_body_data();
+    }
 
-	protected function get_page_number()
-	{
-		$page = URLDecode::getPiece(2);
-		if(isset($page) && is_numeric($page))
-			return $page;
-		return 1;
-	}
+    protected function get_page_number()
+    {
+        $page = URLDecode::getPiece(2);
+        if(isset($page) && is_numeric($page))
+            return $page;
+        return 1;
+    }
 
-	protected function get_list_results()
-	{
+    protected function get_list_results()
+    {
     return $this->activityRepository->getActivities(self::$POSTS_PER_PAGE, $this->offset);
-	}
+    }
 
-	protected function get_list_description()
-	{
-		$start = $this->offset + 1;
-		$end = min($this->offset + self::$POSTS_PER_PAGE, $this->get_total_post_count());
+    protected function get_list_description()
+    {
+        $start = $this->offset + 1;
+        $end = min($this->offset + self::$POSTS_PER_PAGE, $this->get_total_post_count());
 		
-		return sprintf(self::$LIST_DESCRIPTION, $start, $end, $this->get_total_post_count());
-	}
+        return sprintf(self::$LIST_DESCRIPTION, $start, $end, $this->get_total_post_count());
+    }
 
-	protected function get_list_next_link()
-	{
-		if($this->page == 1)
-			return;
-		if($this->page == 2)
-			return Loader::getRootUrl('lifestream');
-		return Loader::getRootUrl('lifestream') . 'page/' . ($this->page - 1) . '/';
-	}
+    protected function get_list_next_link()
+    {
+        if($this->page == 1)
+            return;
+        if($this->page == 2)
+            return Loader::getRootUrl('lifestream');
+        return Loader::getRootUrl('lifestream') . 'page/' . ($this->page - 1) . '/';
+    }
 
-	protected function get_list_prev_link()
-	{
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
-			return;
-		return Loader::getRootUrl('lifestream') . 'page/' . ($this->page + 1) . '/';
-	}
+    protected function get_list_prev_link()
+    {
+        if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
+            return;
+        return Loader::getRootUrl('lifestream') . 'page/' . ($this->page + 1) . '/';
+    }
 
-	private $total_post_count;
-	protected function get_total_post_count()
-	{
-		if(!isset($this->total_post_count))
-			$this->total_post_count = $this->activityRepository->getActivitiesCount();
-		return $this->total_post_count;
-	}
+    private $total_post_count;
+    protected function get_total_post_count()
+    {
+        if(!isset($this->total_post_count))
+            $this->total_post_count = $this->activityRepository->getActivitiesCount();
+        return $this->total_post_count;
+    }
 
 }

Braces +25 added lines, -28 removed lines

@@ -2,8 +2,8 @@ 
 
 Loader::load('controller', 'lifestream/DefaultListController');
 
-final class HomeController extends DefaultListController
-{
+final class HomeController extends DefaultListController

+{

 
 	private static $TITLE = "Jacob Emerick's Lifestream";
 	private static $DESCRIPTION = 'Lifestream for Jacob Emerick, a combination of his latest twitter, youtube, running, reading, and blogging activity.';
@@ -18,8 +18,7 @@ 
 
 	private static $LIST_DESCRIPTION = 'viewing %d - %d of %d total activities';
 
-	protected function set_head_data()
-	{
+	protected function set_head_data() {

 		$this->set_title(self::$TITLE);
 		$this->set_description(self::$DESCRIPTION);
 		$this->set_keywords(self::$KEYWORD_ARRAY);
@@ -27,56 +26,54 @@ 
 		parent::set_head_data();
 	}
 
-	protected function set_body_data()
-	{
+	protected function set_body_data() {

 		$this->set_body('title', "What's Jacob Up To?");
 		$this->set_body('description', "A combination of Jacob Emerick's tweets, runs, reads, blogs, and YouTubes all in <em>one awesome lifestream</em>.");
 		
 		parent::set_body_data();
 	}
 
-	protected function get_page_number()
-	{
+	protected function get_page_number() {

 		$page = URLDecode::getPiece(2);
-		if(isset($page) && is_numeric($page))
-			return $page;
+		if(isset($page) && is_numeric($page)) {

+					return $page;

+		}

 		return 1;
 	}
 
-	protected function get_list_results()
-	{
+	protected function get_list_results() {

     return $this->activityRepository->getActivities(self::$POSTS_PER_PAGE, $this->offset);
 	}
 
-	protected function get_list_description()
-	{
+	protected function get_list_description() {

 		$start = $this->offset + 1;
 		$end = min($this->offset + self::$POSTS_PER_PAGE, $this->get_total_post_count());
 		
 		return sprintf(self::$LIST_DESCRIPTION, $start, $end, $this->get_total_post_count());
 	}
 
-	protected function get_list_next_link()
-	{
-		if($this->page == 1)
-			return;
-		if($this->page == 2)
-			return Loader::getRootUrl('lifestream');
+	protected function get_list_next_link() {

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

+					return;

+		}

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

+					return Loader::getRootUrl('lifestream');

+		}

 		return Loader::getRootUrl('lifestream') . 'page/' . ($this->page - 1) . '/';
 	}
 
-	protected function get_list_prev_link()
-	{
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
-			return;
+	protected function get_list_prev_link() {

+		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count()) {

+					return;

+		}

 		return Loader::getRootUrl('lifestream') . 'page/' . ($this->page + 1) . '/';
 	}
 
 	private $total_post_count;
-	protected function get_total_post_count()
-	{
-		if(!isset($this->total_post_count))
-			$this->total_post_count = $this->activityRepository->getActivitiesCount();
+	protected function get_total_post_count() {

+		if(!isset($this->total_post_count)) {

+					$this->total_post_count = $this->activityRepository->getActivitiesCount();

+		}

 		return $this->total_post_count;
 	}
 

controller/site/ContactController.class.inc.php

Spacing +9 added lines, -9 removed lines

@@ -28,17 +28,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,
@@ -50,14 +50,14 @@ 
       ->addTo($container['config']->admin_email)
       ->setSubject('Site Contact')
       ->setPlainMessage(
-        'Name: ' . Request::getPost('name') . "\n" .
-        'Email: ' . Request::getPost('email') . "\n" .
-        'Message: ' . Request::getPost('message')
+        '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.");
+			'success_message' => "Thank you for your message, ".Request::getPost('name')."! I'll get back to you as soon as possible.");
 	}
 
 }

Indentation +40 added lines, -40 removed lines

@@ -2,62 +2,62 @@
 
 Loader::load('controller', 'site/DefaultPageController');
 Loader::load('utility', array(
-	'Validate'));
+    'Validate'));
 
 final class ContactController extends DefaultPageController
 {
 
-	protected function set_head_data()
-	{
-		$this->set_title("Contact Form for Jacob Emerick's Sites");
-		$this->set_description("Want to reach out to Jacob Emerick with questions or concerns? Well, then here's a handy contact form for you that will connect you with the man himself.");
-		$this->set_keywords(array('contact', 'webmaster', 'support', 'help', 'Jacob Emerick'));
+    protected function set_head_data()
+    {
+        $this->set_title("Contact Form for Jacob Emerick's Sites");
+        $this->set_description("Want to reach out to Jacob Emerick with questions or concerns? Well, then here's a handy contact form for you that will connect you with the man himself.");
+        $this->set_keywords(array('contact', 'webmaster', 'support', 'help', 'Jacob Emerick'));
 		
-		parent::set_head_data();
-	}
+        parent::set_head_data();
+    }
 
-	protected function set_body_data()
-	{
-		parent::set_body_data();
+    protected function set_body_data()
+    {
+        parent::set_body_data();
 		
-		$this->set_body('top_data', array('title' => 'Contact Me'));
+        $this->set_body('top_data', array('title' => 'Contact Me'));
 		
-		$this->set_body('body_data', $this->get_form_results());
-		$this->set_body('body_view', 'Contact');
-	}
-
-	private function get_form_results()
-	{
-		if(!Request::hasPost())
-			return array();
+        $this->set_body('body_data', $this->get_form_results());
+        $this->set_body('body_view', 'Contact');
+    }
+
+    private function get_form_results()
+    {
+        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))
-		{
-			return array(
-				'error_message' => $error_message,
-				'value' => Request::getPost());
-		}
+        if(!empty($error_message))
+        {
+            return array(
+                'error_message' => $error_message,
+                'value' => Request::getPost());
+        }
 
     global $container;
     $sent = $container['mail']
-      ->addTo($container['config']->admin_email)
-      ->setSubject('Site Contact')
-      ->setPlainMessage(
+        ->addTo($container['config']->admin_email)
+        ->setSubject('Site Contact')
+        ->setPlainMessage(
         'Name: ' . Request::getPost('name') . "\n" .
         'Email: ' . Request::getPost('email') . "\n" .
         'Message: ' . Request::getPost('message')
-      )
-      ->send();
+        )
+        ->send();
 
-		return array(
-			'success_message' => "Thank you for your message, " . Request::getPost('name') . "! I'll get back to you as soon as possible.");
-	}
+        return array(
+            'success_message' => "Thank you for your message, " . Request::getPost('name') . "! I'll get back to you as soon as possible.");
+    }
 
 }

Braces +17 added lines, -16 removed lines

@@ -4,11 +4,10 @@ 
 Loader::load('utility', array(
 	'Validate'));
 
-final class ContactController extends DefaultPageController
-{
+final class ContactController extends DefaultPageController

+{

 
-	protected function set_head_data()
-	{
+	protected function set_head_data() {

 		$this->set_title("Contact Form for Jacob Emerick's Sites");
 		$this->set_description("Want to reach out to Jacob Emerick with questions or concerns? Well, then here's a handy contact form for you that will connect you with the man himself.");
 		$this->set_keywords(array('contact', 'webmaster', 'support', 'help', 'Jacob Emerick'));
@@ -16,8 +15,7 @@ 
 		parent::set_head_data();
 	}
 
-	protected function set_body_data()
-	{
+	protected function set_body_data() {

 		parent::set_body_data();
 		
 		$this->set_body('top_data', array('title' => 'Contact Me'));
@@ -26,17 +24,20 @@ 
 		$this->set_body('body_view', 'Contact');
 	}
 
-	private function get_form_results()
-	{
-		if(!Request::hasPost())
-			return array();
+	private function get_form_results() {

+		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))
 		{

controller/waterfalls/MapController.class.inc.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     {
         parent::set_head_data();
         
-        $this->set_title(self::$TITLE . ' | ' . self::$WEBSITE_TITLE);
+        $this->set_title(self::$TITLE.' | '.self::$WEBSITE_TITLE);
         $this->set_description(self::$DESCRIPTION);
         $this->set_keywords(self::$KEYWORD_ARRAY);
     }

Braces +7 added lines, -12 removed lines

@@ -3,8 +3,8 @@ 
 Loader::load('controller', 'waterfalls/DefaultPageController');
 Loader::load('collector', 'waterfall/WaterfallCollector');
 
-final class MapController extends DefaultPageController
-{
+final class MapController extends DefaultPageController

+{

 
     private static $TITLE = 'Map of Upper Peninsula Waterfalls';
     private static $DESCRIPTION = '';
@@ -17,16 +17,14 @@ 
         'Upper Tahquamenon Falls',
     );
 
-    public function __construct()
-    {
+    public function __construct() {

         parent::__construct();
 
         $this->add_js('https://maps.googleapis.com/maps/api/js?sensor=false');
         $this->add_waterfall_js();
     }
 
-    protected function set_head_data()
-    {
+    protected function set_head_data() {

         parent::set_head_data();
         
         $this->set_title(self::$TITLE . ' | ' . self::$WEBSITE_TITLE);
@@ -34,8 +32,7 @@ 
         $this->set_keywords(self::$KEYWORD_ARRAY);
     }
 
-    protected function set_body_data($page_type = 'wide')
-    {
+    protected function set_body_data($page_type = 'wide') {

         parent::set_body_data($page_type);
         
         $this->set_head('full_page_map', true);
@@ -43,15 +40,13 @@ 
         $this->set_body('view', 'Map');
     }
 
-    private function fetch_waterfall_list()
-    {
+    private function fetch_waterfall_list() {

         $list = WaterfallCollector::getMapList();
         array_walk($list, array($this, 'parse_waterfall'));
         return $list;
     }
 
-    private function parse_waterfall(&$waterfall)
-    {
+    private function parse_waterfall(&$waterfall) {

         $waterfall = (object) array(
             'name'  => $waterfall->name,
             'latitude' => $waterfall->latitude,