Completed
Push — master ( 7e42fe...9ab714 )
by Jacob
04:00
created
controller/images/HomeController.class.inc.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -16,13 +16,13 @@  discard block
 block discarded – undo
16 16
 	{
17 17
 		$file = URLDecode::getURI();
18 18
 		
19
-		if(
19
+		if (
20 20
 			URLDecode::getSite() == 'images' ||
21 21
 			URLDecode::getExtension() == 'ico')
22 22
 			$file = "/css/{$file}";
23
-		else if(URLDecode::getSite() == 'portfolio')
23
+		else if (URLDecode::getSite() == 'portfolio')
24 24
 			$file = "/portfolio/{$file}";
25
-		else if(substr($file, 0, 7) == '/photo/')
25
+		else if (substr($file, 0, 7) == '/photo/')
26 26
 			$file = '/photo/processed/' . substr($file, 7);
27 27
 		
28 28
 		$this->use_old_image_logic($file);
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 
33 33
 	private function send_headers($extension, $last_modified)
34 34
 	{
35
-		switch($extension)
35
+		switch ($extension)
36 36
 		{
37 37
 			case 'gif' :
38 38
 				Header::sendGIF($last_modified);
@@ -52,11 +52,11 @@  discard block
 block discarded – undo
52 52
 	private function use_old_image_logic($file)
53 53
 	{
54 54
 		$image = new ImageOld($file);
55
-		if(!$image->isValid()) {
55
+		if (!$image->isValid()) {
56 56
 			$this->eject();
57 57
 		}
58 58
 		
59
-		switch($image->getExtension())
59
+		switch ($image->getExtension())
60 60
 		{
61 61
 			case 'gif' :
62 62
 				Header::sendGIF($image->getLastModified());
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 
79 79
 	protected function eject()
80 80
 	{
81
-		if(get_class($this) !== 'Error404Controller')
81
+		if (get_class($this) !== 'Error404Controller')
82 82
 			Loader::loadNew('controller', '/Error404Controller')->activate();
83 83
 	}
84 84
 
Please login to merge, or discard this patch.
Braces   +10 added lines, -8 removed lines patch added patch discarded remove patch
@@ -18,12 +18,13 @@  discard block
 block discarded – undo
18 18
 		
19 19
 		if(
20 20
 			URLDecode::getSite() == 'images' ||
21
-			URLDecode::getExtension() == 'ico')
22
-			$file = "/css/{$file}";
23
-		else if(URLDecode::getSite() == 'portfolio')
24
-			$file = "/portfolio/{$file}";
25
-		else if(substr($file, 0, 7) == '/photo/')
26
-			$file = '/photo/processed/' . substr($file, 7);
21
+			URLDecode::getExtension() == 'ico') {
22
+					$file = "/css/{$file}";
23
+		} else if(URLDecode::getSite() == 'portfolio') {
24
+					$file = "/portfolio/{$file}";
25
+		} else if(substr($file, 0, 7) == '/photo/') {
26
+					$file = '/photo/processed/' . substr($file, 7);
27
+		}
27 28
 		
28 29
 		$this->use_old_image_logic($file);
29 30
 		
@@ -78,8 +79,9 @@  discard block
 block discarded – undo
78 79
 
79 80
 	protected function eject()
80 81
 	{
81
-		if(get_class($this) !== 'Error404Controller')
82
-			Loader::loadNew('controller', '/Error404Controller')->activate();
82
+		if(get_class($this) !== 'Error404Controller') {
83
+					Loader::loadNew('controller', '/Error404Controller')->activate();
84
+		}
83 85
 	}
84 86
 
85 87
 }
Please login to merge, or discard this patch.
controller/portfolio/DefaultPageController.class.inc.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -39,12 +39,12 @@  discard block
 block discarded – undo
39 39
 				'name' => 'Contact',
40 40
 				'link' => Loader::getRootURL() . 'contact/'));
41 41
 		
42
-		if(!URLDecode::getPiece(1))
42
+		if (!URLDecode::getPiece(1))
43 43
 			$active_page = 'About';
44 44
 		else
45 45
 			$active_page = ucfirst(URLDecode::getPiece(1));
46 46
 		
47
-		foreach($menu as $menu_item)
47
+		foreach ($menu as $menu_item)
48 48
 		{
49 49
 			$menu_item = (object) $menu_item;
50 50
 			$menu_item->active = ($menu_item->name == $active_page);
@@ -59,11 +59,11 @@  discard block
 block discarded – undo
59 59
 		Loader::load('collector', 'portfolio/PortfolioCollector');
60 60
 		$portfolio_result = PortfolioCollector::getPiecesForCategory($category_id);
61 61
 		
62
-		foreach($portfolio_result as $piece)
62
+		foreach ($portfolio_result as $piece)
63 63
 		{
64 64
 			$piece_obj = new stdclass();
65 65
 			$piece_obj->title = $piece->title;
66
-			if("{$piece->category}" == 1)
66
+			if ("{$piece->category}" == 1)
67 67
 				$piece_obj->url = Loader::getRootURL() . "web/{$piece->title_url}/";
68 68
 			else
69 69
 				$piece_obj->url = Loader::getRootURL() . "print/{$piece->title_url}/";
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 			$piece_array[] = $piece_obj;
72 72
 		}
73 73
 		
74
-		switch($category_id)
74
+		switch ($category_id)
75 75
 		{
76 76
 			case 1 :
77 77
 				$title = "Sites I've worked on";
Please login to merge, or discard this patch.
Braces   +10 added lines, -8 removed lines patch added patch discarded remove patch
@@ -39,10 +39,11 @@  discard block
 block discarded – undo
39 39
 				'name' => 'Contact',
40 40
 				'link' => Loader::getRootURL() . 'contact/'));
41 41
 		
42
-		if(!URLDecode::getPiece(1))
43
-			$active_page = 'About';
44
-		else
45
-			$active_page = ucfirst(URLDecode::getPiece(1));
42
+		if(!URLDecode::getPiece(1)) {
43
+					$active_page = 'About';
44
+		} else {
45
+					$active_page = ucfirst(URLDecode::getPiece(1));
46
+		}
46 47
 		
47 48
 		foreach($menu as $menu_item)
48 49
 		{
@@ -63,10 +64,11 @@  discard block
 block discarded – undo
63 64
 		{
64 65
 			$piece_obj = new stdclass();
65 66
 			$piece_obj->title = $piece->title;
66
-			if("{$piece->category}" == 1)
67
-				$piece_obj->url = Loader::getRootURL() . "web/{$piece->title_url}/";
68
-			else
69
-				$piece_obj->url = Loader::getRootURL() . "print/{$piece->title_url}/";
67
+			if("{$piece->category}" == 1) {
68
+							$piece_obj->url = Loader::getRootURL() . "web/{$piece->title_url}/";
69
+			} else {
70
+							$piece_obj->url = Loader::getRootURL() . "print/{$piece->title_url}/";
71
+			}
70 72
 			$piece_obj->image = "/{$piece->image}";
71 73
 			$piece_array[] = $piece_obj;
72 74
 		}
Please login to merge, or discard this patch.
controller/portfolio/ContactController.class.inc.php 3 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@
 block discarded – undo
47 47
 		
48 48
 		else
49 49
 		{
50
-            global $config;
50
+			global $config;
51 51
 			$mail = new Mail();
52 52
 			$mail->setToAddress($config->admin_email, 'Jacob Emerick');
53 53
 			$mail->setSubject('Portfolio Contact');
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -28,17 +28,17 @@  discard block
 block discarded – undo
28 28
 
29 29
 	private function process_form_data()
30 30
 	{
31
-		if(!Request::hasPost())
31
+		if (!Request::hasPost())
32 32
 			return array();
33 33
 		
34
-		if(!Validate::checkRequest('post', 'name', 'string'))
34
+		if (!Validate::checkRequest('post', 'name', 'string'))
35 35
 			$error_message['name'] = 'Please enter a value for your name.';
36
-		if(!Validate::checkRequest('post', 'email', 'string'))
36
+		if (!Validate::checkRequest('post', 'email', 'string'))
37 37
 			$error_message['email'] = 'Please enter a valid email address.';
38
-		if(!Validate::checkRequest('post', 'message', 'string'))
38
+		if (!Validate::checkRequest('post', 'message', 'string'))
39 39
 			$error_message['message'] = 'Please enter a message.';
40 40
 		
41
-		if(!empty($error_message))
41
+		if (!empty($error_message))
42 42
 		{
43 43
 			return array(
44 44
 				'error_message' => $error_message,
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 			$mail->send();
59 59
 			
60 60
 			return array(
61
-				'success_message' => "Thank you for your message, ".Request::getPost('name')."! I'll get back to you as soon as possible.");
61
+				'success_message' => "Thank you for your message, " . Request::getPost('name') . "! I'll get back to you as soon as possible.");
62 62
 		}
63 63
 	}
64 64
 
Please login to merge, or discard this patch.
Braces   +13 added lines, -11 removed lines patch added patch discarded remove patch
@@ -28,24 +28,26 @@
 block discarded – undo
28 28
 
29 29
 	private function process_form_data()
30 30
 	{
31
-		if(!Request::hasPost())
32
-			return array();
31
+		if(!Request::hasPost()) {
32
+					return array();
33
+		}
33 34
 		
34
-		if(!Validate::checkRequest('post', 'name', 'string'))
35
-			$error_message['name'] = 'Please enter a value for your name.';
36
-		if(!Validate::checkRequest('post', 'email', 'string'))
37
-			$error_message['email'] = 'Please enter a valid email address.';
38
-		if(!Validate::checkRequest('post', 'message', 'string'))
39
-			$error_message['message'] = 'Please enter a message.';
35
+		if(!Validate::checkRequest('post', 'name', 'string')) {
36
+					$error_message['name'] = 'Please enter a value for your name.';
37
+		}
38
+		if(!Validate::checkRequest('post', 'email', 'string')) {
39
+					$error_message['email'] = 'Please enter a valid email address.';
40
+		}
41
+		if(!Validate::checkRequest('post', 'message', 'string')) {
42
+					$error_message['message'] = 'Please enter a message.';
43
+		}
40 44
 		
41 45
 		if(!empty($error_message))
42 46
 		{
43 47
 			return array(
44 48
 				'error_message' => $error_message,
45 49
 				'value' => Request::getPost());
46
-		}
47
-		
48
-		else
50
+		} else
49 51
 		{
50 52
             global $config;
51 53
 			$mail = new Mail();
Please login to merge, or discard this patch.
controller/portfolio/PieceController.class.inc.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 		Loader::load('collector', 'portfolio/PortfolioCollector');
37 37
 		$portfolio_result = PortfolioCollector::getPieceByURI($this->title_url);
38 38
 		
39
-		if($portfolio_result === null)
39
+		if ($portfolio_result === null)
40 40
 			$this->eject();
41 41
 		
42 42
 		$portfolio_image_result = PortfolioCollector::getImagesForPiece($portfolio_result->id, 2);
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 		$main_image->width = $dimensions[0];
52 52
 		$main_image->height = $dimensions[1];
53 53
 		
54
-		foreach($portfolio_image_result as $portfolio_image)
54
+		foreach ($portfolio_image_result as $portfolio_image)
55 55
 		{
56 56
 			$thumb = $portfolio_image->name;
57 57
 			$thumb_array = explode('.', $thumb);
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
 		
73 73
 		$portfolio_tag_result = PortfolioCollector::getTagsForPiece($portfolio_result->id);
74 74
 		
75
-		foreach($portfolio_tag_result as $portfolio_tag)
75
+		foreach ($portfolio_tag_result as $portfolio_tag)
76 76
 		{
77 77
 			$tag_array[] = $portfolio_tag->name;
78 78
 		}
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,8 +36,9 @@
 block discarded – undo
36 36
 		Loader::load('collector', 'portfolio/PortfolioCollector');
37 37
 		$portfolio_result = PortfolioCollector::getPieceByURI($this->title_url);
38 38
 		
39
-		if($portfolio_result === null)
40
-			$this->eject();
39
+		if($portfolio_result === null) {
40
+					$this->eject();
41
+		}
41 42
 		
42 43
 		$portfolio_image_result = PortfolioCollector::getImagesForPiece($portfolio_result->id, 2);
43 44
 		$image = new ImageOld("portfolio/{$portfolio_image_result[0]->name}");
Please login to merge, or discard this patch.
module/form/CommentSubmitModule.class.inc.php 3 patches
Indentation   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -172,24 +172,24 @@
 block discarded – undo
172 172
 			}
173 173
 		}
174 174
         
175
-        $site = URLDecode::getSite();
175
+		$site = URLDecode::getSite();
176 176
         
177
-        if ($site == 'blog') {
178
-            $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 [email protected] 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!";
179
-        } else if ($site == 'waterfalls') {
180
-            $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 [email protected] 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!";
181
-        }
177
+		if ($site == 'blog') {
178
+			$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 [email protected] 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!";
179
+		} else if ($site == 'waterfalls') {
180
+			$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 [email protected] 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!";
181
+		}
182 182
         
183 183
 		foreach($email_recipient_array as $email_recipient)
184 184
 		{
185 185
 			$mail = new Mail();
186 186
 			$mail->setToAddress($email_recipient['email'], $email_recipient['name']);
187 187
             
188
-            if ($site == 'blog') {
189
-                $mail->setSubject("New Comment on Jacob Emerick's Blog");
190
-            } else if ($site == 'waterfalls') {
191
-                $mail->setSubject("New Comment on Waterfalls of the Keweenaw");
192
-            }
188
+			if ($site == 'blog') {
189
+				$mail->setSubject("New Comment on Jacob Emerick's Blog");
190
+			} else if ($site == 'waterfalls') {
191
+				$mail->setSubject("New Comment on Waterfalls of the Keweenaw");
192
+			}
193 193
             
194 194
 			$mail->setMessage($message);
195 195
 			$mail->send();
Please login to merge, or discard this patch.
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -26,15 +26,15 @@  discard block
 block discarded – undo
26 26
 
27 27
 	public function activate()
28 28
 	{
29
-		if(!Request::hasPost())
29
+		if (!Request::hasPost())
30 30
 			return false;
31
-		if(!Request::getPost('submit') == 'Submit Comment')
31
+		if (!Request::getPost('submit') == 'Submit Comment')
32 32
 			return false;
33 33
 		
34 34
 		$errors = $this->fetch_errors();
35
-		if(count($errors) > 0)
35
+		if (count($errors) > 0)
36 36
 			return $errors;
37
-		if(Request::getPost('catch') !== '')
37
+		if (Request::getPost('catch') !== '')
38 38
 			return false;
39 39
 		
40 40
 		$page_id = $this->save_comment_page();
@@ -50,17 +50,17 @@  discard block
 block discarded – undo
50 50
 	private function fetch_errors()
51 51
 	{
52 52
 		$errors = array();
53
-		if(!Validate::checkRequest('post', 'name', 'name'))
53
+		if (!Validate::checkRequest('post', 'name', 'name'))
54 54
 			$errors['name'] = 'You must include a valid name';
55
-		if(!Validate::checkRequest('post', 'email', 'email'))
55
+		if (!Validate::checkRequest('post', 'email', 'email'))
56 56
 			$errors['email'] = 'You must include a valid email';
57
-		if(Request::getPost('website') && !Validate::checkRequest('post', 'website', 'url'))
57
+		if (Request::getPost('website') && !Validate::checkRequest('post', 'website', 'url'))
58 58
 			$errors['website'] = 'Please enter a valid website';
59
-		if(!Validate::checkRequest('post', 'comment', 'string'))
59
+		if (!Validate::checkRequest('post', 'comment', 'string'))
60 60
 			$errors['comment'] = 'You must enter a comment';
61
-		if(Request::getPost('notify') && Request::getPost('notify') != 'check')
61
+		if (Request::getPost('notify') && Request::getPost('notify') != 'check')
62 62
 			$errors['notify'] = 'You entered an invalid notify request';
63
-		if(Request::getPost('reply') && !Validate::checkRequest('post', 'reply', 'integer'))
63
+		if (Request::getPost('reply') && !Validate::checkRequest('post', 'reply', 'integer'))
64 64
 			$errors['reply'] = 'You entered an invalid reply request';
65 65
 		
66 66
 		return $errors;
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 	private function save_comment_page()
70 70
 	{
71 71
 		$page_result = CommentCollector::getCommentPageByURL($this->path, $this->site);
72
-		if($page_result !== null)
72
+		if ($page_result !== null)
73 73
 			return $page_result->id;
74 74
 		
75 75
 		$query = "INSERT INTO `jpemeric_comment`.`comment_page` (`site`, `path`) VALUES ('%d', '%s')";
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 			'name' => Request::getPost('name'),
86 86
 			'email' => Request::getPost('email'));
87 87
 		
88
-		if(Request::getPost('website') != '')
88
+		if (Request::getPost('website') != '')
89 89
 			$cookie_value['website'] = Request::getPost('website');
90 90
 		
91 91
 		$cookie_value = json_encode($cookie_value);
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 			->save();
96 96
 		
97 97
 		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
98
-		if($commenter_result !== null)
98
+		if ($commenter_result !== null)
99 99
 			return $commenter_result->id;
100 100
 		
101 101
 		$query = "INSERT INTO `jpemeric_comment`.`commenter` (`name`,`email`,`url`) VALUES ('%s','%s','%s')";
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
 	private function save_comment()
113 113
 	{
114 114
 		$comment_result = CommentCollector::getCommentByBody(Request::getPost('comment'));
115
-		if($comment_result !== null)
115
+		if ($comment_result !== null)
116 116
 			return $comment_result->id;
117 117
 		
118 118
 		$query = "INSERT INTO `jpemeric_comment`.`comment` (`body`, `body_format`) VALUES ('%s', '%s')";
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
 		$query = "INSERT INTO `jpemeric_comment`.`comment_meta` (`commenter`,`comment`,`reply`,`notify`,`comment_page`,`date`,`display`) VALUES ('%d','%d','%d','%d','%d','%s','%d')";
134 134
 		
135 135
 		$reply = Database::escape(Request::getPost('type'));
136
-		if($reply == 'new')
136
+		if ($reply == 'new')
137 137
 			$reply = 0;
138 138
 		// else check to make sure value is legit
139 139
 		
@@ -157,13 +157,13 @@  discard block
 block discarded – undo
157 157
 		
158 158
 		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
159 159
 		
160
-		if($commenter_result->trusted == 1)
160
+		if ($commenter_result->trusted == 1)
161 161
 		{
162 162
 			$notification_result = CommentCollector::getNotificationForPage($page);
163 163
 			
164
-			foreach($notification_result as $notification_row)
164
+			foreach ($notification_result as $notification_row)
165 165
 			{
166
-				if($notification_row->email == Request::getPost('email'))
166
+				if ($notification_row->email == Request::getPost('email'))
167 167
 					continue;
168 168
 				
169 169
 				$email_recipient_array[$notification_row->email] = array(
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
             $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 [email protected] 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!";
181 181
         }
182 182
         
183
-		foreach($email_recipient_array as $email_recipient)
183
+		foreach ($email_recipient_array as $email_recipient)
184 184
 		{
185 185
 			$mail = new Mail();
186 186
 			$mail->setToAddress($email_recipient['email'], $email_recipient['name']);
Please login to merge, or discard this patch.
Braces   +48 added lines, -32 removed lines patch added patch discarded remove patch
@@ -26,16 +26,20 @@  discard block
 block discarded – undo
26 26
 
27 27
 	public function activate()
28 28
 	{
29
-		if(!Request::hasPost())
30
-			return false;
31
-		if(!Request::getPost('submit') == 'Submit Comment')
32
-			return false;
29
+		if(!Request::hasPost()) {
30
+					return false;
31
+		}
32
+		if(!Request::getPost('submit') == 'Submit Comment') {
33
+					return false;
34
+		}
33 35
 		
34 36
 		$errors = $this->fetch_errors();
35
-		if(count($errors) > 0)
36
-			return $errors;
37
-		if(Request::getPost('catch') !== '')
38
-			return false;
37
+		if(count($errors) > 0) {
38
+					return $errors;
39
+		}
40
+		if(Request::getPost('catch') !== '') {
41
+					return false;
42
+		}
39 43
 		
40 44
 		$page_id = $this->save_comment_page();
41 45
 		$commenter_id = $this->save_commenter();
@@ -50,18 +54,24 @@  discard block
 block discarded – undo
50 54
 	private function fetch_errors()
51 55
 	{
52 56
 		$errors = array();
53
-		if(!Validate::checkRequest('post', 'name', 'name'))
54
-			$errors['name'] = 'You must include a valid name';
55
-		if(!Validate::checkRequest('post', 'email', 'email'))
56
-			$errors['email'] = 'You must include a valid email';
57
-		if(Request::getPost('website') && !Validate::checkRequest('post', 'website', 'url'))
58
-			$errors['website'] = 'Please enter a valid website';
59
-		if(!Validate::checkRequest('post', 'comment', 'string'))
60
-			$errors['comment'] = 'You must enter a comment';
61
-		if(Request::getPost('notify') && Request::getPost('notify') != 'check')
62
-			$errors['notify'] = 'You entered an invalid notify request';
63
-		if(Request::getPost('reply') && !Validate::checkRequest('post', 'reply', 'integer'))
64
-			$errors['reply'] = 'You entered an invalid reply request';
57
+		if(!Validate::checkRequest('post', 'name', 'name')) {
58
+					$errors['name'] = 'You must include a valid name';
59
+		}
60
+		if(!Validate::checkRequest('post', 'email', 'email')) {
61
+					$errors['email'] = 'You must include a valid email';
62
+		}
63
+		if(Request::getPost('website') && !Validate::checkRequest('post', 'website', 'url')) {
64
+					$errors['website'] = 'Please enter a valid website';
65
+		}
66
+		if(!Validate::checkRequest('post', 'comment', 'string')) {
67
+					$errors['comment'] = 'You must enter a comment';
68
+		}
69
+		if(Request::getPost('notify') && Request::getPost('notify') != 'check') {
70
+					$errors['notify'] = 'You entered an invalid notify request';
71
+		}
72
+		if(Request::getPost('reply') && !Validate::checkRequest('post', 'reply', 'integer')) {
73
+					$errors['reply'] = 'You entered an invalid reply request';
74
+		}
65 75
 		
66 76
 		return $errors;
67 77
 	}
@@ -69,8 +79,9 @@  discard block
 block discarded – undo
69 79
 	private function save_comment_page()
70 80
 	{
71 81
 		$page_result = CommentCollector::getCommentPageByURL($this->path, $this->site);
72
-		if($page_result !== null)
73
-			return $page_result->id;
82
+		if($page_result !== null) {
83
+					return $page_result->id;
84
+		}
74 85
 		
75 86
 		$query = "INSERT INTO `jpemeric_comment`.`comment_page` (`site`, `path`) VALUES ('%d', '%s')";
76 87
 		$query = sprintf($query, $this->site, $this->path);
@@ -85,8 +96,9 @@  discard block
 block discarded – undo
85 96
 			'name' => Request::getPost('name'),
86 97
 			'email' => Request::getPost('email'));
87 98
 		
88
-		if(Request::getPost('website') != '')
89
-			$cookie_value['website'] = Request::getPost('website');
99
+		if(Request::getPost('website') != '') {
100
+					$cookie_value['website'] = Request::getPost('website');
101
+		}
90 102
 		
91 103
 		$cookie_value = json_encode($cookie_value);
92 104
 		
@@ -95,8 +107,9 @@  discard block
 block discarded – undo
95 107
 			->save();
96 108
 		
97 109
 		$commenter_result = CommentCollector::getCommenterByFields(Request::getPost('name'), Request::getPost('email'), Request::getPost('website'));
98
-		if($commenter_result !== null)
99
-			return $commenter_result->id;
110
+		if($commenter_result !== null) {
111
+					return $commenter_result->id;
112
+		}
100 113
 		
101 114
 		$query = "INSERT INTO `jpemeric_comment`.`commenter` (`name`,`email`,`url`) VALUES ('%s','%s','%s')";
102 115
 		
@@ -112,8 +125,9 @@  discard block
 block discarded – undo
112 125
 	private function save_comment()
113 126
 	{
114 127
 		$comment_result = CommentCollector::getCommentByBody(Request::getPost('comment'));
115
-		if($comment_result !== null)
116
-			return $comment_result->id;
128
+		if($comment_result !== null) {
129
+					return $comment_result->id;
130
+		}
117 131
 		
118 132
 		$query = "INSERT INTO `jpemeric_comment`.`comment` (`body`, `body_format`) VALUES ('%s', '%s')";
119 133
 		
@@ -133,8 +147,9 @@  discard block
 block discarded – undo
133 147
 		$query = "INSERT INTO `jpemeric_comment`.`comment_meta` (`commenter`,`comment`,`reply`,`notify`,`comment_page`,`date`,`display`) VALUES ('%d','%d','%d','%d','%d','%s','%d')";
134 148
 		
135 149
 		$reply = Database::escape(Request::getPost('type'));
136
-		if($reply == 'new')
137
-			$reply = 0;
150
+		if($reply == 'new') {
151
+					$reply = 0;
152
+		}
138 153
 		// else check to make sure value is legit
139 154
 		
140 155
 		$notify = (Request::getPost('notify') == 'check') ? 1 : 0;
@@ -163,8 +178,9 @@  discard block
 block discarded – undo
163 178
 			
164 179
 			foreach($notification_result as $notification_row)
165 180
 			{
166
-				if($notification_row->email == Request::getPost('email'))
167
-					continue;
181
+				if($notification_row->email == Request::getPost('email')) {
182
+									continue;
183
+				}
168 184
 				
169 185
 				$email_recipient_array[$notification_row->email] = array(
170 186
 					'email' => $notification_row->email,
Please login to merge, or discard this patch.
utility/Environment.class.inc.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
 
10 10
 	public function __construct($user_agent = null)
11 11
 	{
12
-		if(!isset($user_agent))
12
+		if (!isset($user_agent))
13 13
 			$user_agent = Request::getServer('HTTP_USER_AGENT');
14 14
 		$this->user_agent = $user_agent;
15 15
 	}
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -9,8 +9,9 @@
 block discarded – undo
9 9
 
10 10
 	public function __construct($user_agent = null)
11 11
 	{
12
-		if(!isset($user_agent))
13
-			$user_agent = Request::getServer('HTTP_USER_AGENT');
12
+		if(!isset($user_agent)) {
13
+					$user_agent = Request::getServer('HTTP_USER_AGENT');
14
+		}
14 15
 		$this->user_agent = $user_agent;
15 16
 	}
16 17
 
Please login to merge, or discard this patch.
utility/Database.class.inc.php 3 patches
Indentation   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -16,17 +16,17 @@  discard block
 block discarded – undo
16 16
 
17 17
 	private function __construct($write_params, $read_params)
18 18
 	{
19
-        $this->write_connection = $this->connect(
20
-            $write_params->host,
21
-            $write_params->user,
22
-            $write_params->password
23
-        );
24
-
25
-        $this->read_connection = $this->connect(
26
-            $read_params->host,
27
-            $read_params->user,
28
-            $read_params->password
29
-        );
19
+		$this->write_connection = $this->connect(
20
+			$write_params->host,
21
+			$write_params->user,
22
+			$write_params->password
23
+		);
24
+
25
+		$this->read_connection = $this->connect(
26
+			$read_params->host,
27
+			$read_params->user,
28
+			$read_params->password
29
+		);
30 30
 
31 31
 		return $this;
32 32
 	}
@@ -45,12 +45,12 @@  discard block
 block discarded – undo
45 45
 	public static function instance()
46 46
 	{
47 47
 		if(!isset(self::$instance)) {
48
-            global $config;
49
-            self::$instance = new Database(
50
-                $config->database->master,
51
-                $config->database->slave
52
-            );
53
-        }
48
+			global $config;
49
+			self::$instance = new Database(
50
+				$config->database->master,
51
+				$config->database->slave
52
+			);
53
+		}
54 54
 
55 55
 		return self::$instance;
56 56
 	}
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 		$mysqli = new mysqli($host, $username, $password);
37 37
 		
38 38
 		$has_connection_error = $mysqli->connect_error;
39
-		if(isset($has_connection_error))
39
+		if (isset($has_connection_error))
40 40
 			$this->has_connection_error = true;
41 41
 		
42 42
 		return $mysqli;
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 
45 45
 	public static function instance()
46 46
 	{
47
-		if(!isset(self::$instance)) {
47
+		if (!isset(self::$instance)) {
48 48
             global $config;
49 49
             self::$instance = new Database(
50 50
                 $config->database->master,
@@ -63,11 +63,11 @@  discard block
 block discarded – undo
63 63
 	public static function select($query)
64 64
 	{
65 65
 		$start = microtime(true);
66
-		if($result = self::instance()->read_connection->query($query))
66
+		if ($result = self::instance()->read_connection->query($query))
67 67
 		{
68 68
 			self::instance()->log_query($query, $start);
69 69
 			$array = array();
70
-			while($row = $result->fetch_object())
70
+			while ($row = $result->fetch_object())
71 71
 				$array[] = $row;
72 72
 			$result->close();
73 73
 			return $array;
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 	public static function selectRow($query)
80 80
 	{
81 81
 		$result = self::select($query);
82
-		if(is_array($result))
82
+		if (is_array($result))
83 83
 			return array_pop($result);
84 84
 		return false;
85 85
 	}
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 	public static function execute($query)
88 88
 	{
89 89
 		$start = microtime(true);
90
-		if(self::instance()->write_connection->query($query))
90
+		if (self::instance()->write_connection->query($query))
91 91
 		{
92 92
 			self::instance()->log_query($query, $start);
93 93
 			return true;
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
 	public static function lastInsertID()
100 100
 	{
101 101
 		$id = self::instance()->write_connection->insert_id;
102
-		if($id == 0)
102
+		if ($id == 0)
103 103
 			return false;
104 104
 		return $id;
105 105
 	}
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 	private function gather_query_data()
120 120
 	{
121 121
 		$query_data = array();
122
-		foreach($this->query_log as $query)
122
+		foreach ($this->query_log as $query)
123 123
 		{
124 124
 			$query = self::explain($query);
125 125
 			$query_data[] = $query;
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
 	{
132 132
 		$sql = 'EXPLAIN ' . $query['sql'];
133 133
 		
134
-		if($result = self::instance()->read_connection->query($sql))
134
+		if ($result = self::instance()->read_connection->query($sql))
135 135
 		{
136 136
 			$row = $result->fetch_assoc();
137 137
 			$query['explain'] = $row;
Please login to merge, or discard this patch.
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -36,8 +36,9 @@  discard block
 block discarded – undo
36 36
 		$mysqli = new mysqli($host, $username, $password);
37 37
 		
38 38
 		$has_connection_error = $mysqli->connect_error;
39
-		if(isset($has_connection_error))
40
-			$this->has_connection_error = true;
39
+		if(isset($has_connection_error)) {
40
+					$this->has_connection_error = true;
41
+		}
41 42
 		
42 43
 		return $mysqli;
43 44
 	}
@@ -67,8 +68,9 @@  discard block
 block discarded – undo
67 68
 		{
68 69
 			self::instance()->log_query($query, $start);
69 70
 			$array = array();
70
-			while($row = $result->fetch_object())
71
-				$array[] = $row;
71
+			while($row = $result->fetch_object()) {
72
+							$array[] = $row;
73
+			}
72 74
 			$result->close();
73 75
 			return $array;
74 76
 		}
@@ -79,8 +81,9 @@  discard block
 block discarded – undo
79 81
 	public static function selectRow($query)
80 82
 	{
81 83
 		$result = self::select($query);
82
-		if(is_array($result))
83
-			return array_pop($result);
84
+		if(is_array($result)) {
85
+					return array_pop($result);
86
+		}
84 87
 		return false;
85 88
 	}
86 89
 
@@ -99,8 +102,9 @@  discard block
 block discarded – undo
99 102
 	public static function lastInsertID()
100 103
 	{
101 104
 		$id = self::instance()->write_connection->insert_id;
102
-		if($id == 0)
103
-			return false;
105
+		if($id == 0) {
106
+					return false;
107
+		}
104 108
 		return $id;
105 109
 	}
106 110
 
Please login to merge, or discard this patch.
utility/cookie/CommenterCookie.class.inc.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -22,10 +22,10 @@
 block discarded – undo
22 22
 
23 23
 	protected function getDomain()
24 24
 	{
25
-        $site = URLDecode::getSite();
26
-        if ($site == 'waterfalls' && Loader::isLive()) {
27
-            return 'waterfallsofthekeweenaw.com';
28
-        }
25
+		$site = URLDecode::getSite();
26
+		if ($site == 'waterfalls' && Loader::isLive()) {
27
+			return 'waterfallsofthekeweenaw.com';
28
+		}
29 29
 		return self::$COOKIE_DOMAIN;
30 30
 	}
31 31
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -6,8 +6,8 @@
 block discarded – undo
6 6
 {
7 7
 
8 8
 	private static $COOKIE_NAME			= 'commenter';
9
-	private static $COOKIE_DURATION		= 31536000; // one year
10
-	private static $COOKIE_DOMAIN		= 'jacobemerick.com';
9
+	private static $COOKIE_DURATION = 31536000; // one year
10
+	private static $COOKIE_DOMAIN = 'jacobemerick.com';
11 11
 	private static $COOKIE_PATH			= '/';
12 12
 
13 13
 	protected function getName()
Please login to merge, or discard this patch.
utility/Visitor.class.inc.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 
27 27
 	public static function instance()
28 28
 	{
29
-		if(!isset(self::$instance))
29
+		if (!isset(self::$instance))
30 30
 			self::$instance = new Visitor();
31 31
 		return self::$instance;
32 32
 	}
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
 	public static function getBrowser()
36 36
 	{
37 37
 		Loader::load('utility', 'environment/BrowserEnvironment');
38
-		if(!isset(self::instance()->browser))
38
+		if (!isset(self::instance()->browser))
39 39
 			self::instance()->browser = new BrowserEnvironment();
40 40
 		return self::instance()->browser;
41 41
 	}
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 	public static function getPlatform()
45 45
 	{
46 46
 		Loader::load('utility', 'environment/PlatformEnvironment');
47
-		if(!isset(self::instance()->platform))
47
+		if (!isset(self::instance()->platform))
48 48
 			self::instance()->platform = new PlatformEnvironment();
49 49
 		return self::instance()->platform;
50 50
 	}
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 	public static function getIP()
54 54
 	{
55 55
 		Loader::load('utility', 'Request');
56
-		if(!isset(self::instance()->ip_address))
56
+		if (!isset(self::instance()->ip_address))
57 57
 			self::instance()->ip_address = Request::getServer('REMOTE_ADDR');
58 58
 		return self::instance()->ip_address;
59 59
 	}
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 		Loader::loadInstance('utility', 'Database');
80 80
 		
81 81
 		$query = $this->get_tracking_query();
82
-		if(Database::execute($query) == true)
82
+		if (Database::execute($query) == true)
83 83
 		{
84 84
 			$this->raw_visit_log_id = Database::lastInsertID();
85 85
 			return true;
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 		
95 95
 		$columns = '`';
96 96
 		$values = "'";
97
-		foreach(self::$tracking_field_array as $field_column => $server_array_key)
97
+		foreach (self::$tracking_field_array as $field_column => $server_array_key)
98 98
 		{
99 99
 			$value = Request::getServer($server_array_key);
100 100
 			$value = Database::escape($value);
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
 			$values .= "{$value}','";
104 104
 		}
105 105
 		
106
-		if(Request::hasPost())
106
+		if (Request::hasPost())
107 107
 		{
108 108
 			$value = Request::getPost();
109 109
 			$value = serialize($value);
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 
122 122
 	private function track_response_error($code)
123 123
 	{
124
-		if(isset($this->raw_visit_log_id) && $this->raw_visit_log_id > 0)
124
+		if (isset($this->raw_visit_log_id) && $this->raw_visit_log_id > 0)
125 125
 		{
126 126
 			$query = "UPDATE `jpemeric_stat`.`raw_visit_log` SET `code` = '{$code}' WHERE `id` = '{$this->raw_visit_log_id}' LIMIT 1";
127 127
 			return Database::execute($query);
Please login to merge, or discard this patch.
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -26,8 +26,9 @@  discard block
 block discarded – undo
26 26
 
27 27
 	public static function instance()
28 28
 	{
29
-		if(!isset(self::$instance))
30
-			self::$instance = new Visitor();
29
+		if(!isset(self::$instance)) {
30
+					self::$instance = new Visitor();
31
+		}
31 32
 		return self::$instance;
32 33
 	}
33 34
 
@@ -35,8 +36,9 @@  discard block
 block discarded – undo
35 36
 	public static function getBrowser()
36 37
 	{
37 38
 		Loader::load('utility', 'environment/BrowserEnvironment');
38
-		if(!isset(self::instance()->browser))
39
-			self::instance()->browser = new BrowserEnvironment();
39
+		if(!isset(self::instance()->browser)) {
40
+					self::instance()->browser = new BrowserEnvironment();
41
+		}
40 42
 		return self::instance()->browser;
41 43
 	}
42 44
 
@@ -44,8 +46,9 @@  discard block
 block discarded – undo
44 46
 	public static function getPlatform()
45 47
 	{
46 48
 		Loader::load('utility', 'environment/PlatformEnvironment');
47
-		if(!isset(self::instance()->platform))
48
-			self::instance()->platform = new PlatformEnvironment();
49
+		if(!isset(self::instance()->platform)) {
50
+					self::instance()->platform = new PlatformEnvironment();
51
+		}
49 52
 		return self::instance()->platform;
50 53
 	}
51 54
 
@@ -53,8 +56,9 @@  discard block
 block discarded – undo
53 56
 	public static function getIP()
54 57
 	{
55 58
 		Loader::load('utility', 'Request');
56
-		if(!isset(self::instance()->ip_address))
57
-			self::instance()->ip_address = Request::getServer('REMOTE_ADDR');
59
+		if(!isset(self::instance()->ip_address)) {
60
+					self::instance()->ip_address = Request::getServer('REMOTE_ADDR');
61
+		}
58 62
 		return self::instance()->ip_address;
59 63
 	}
60 64
 
Please login to merge, or discard this patch.