Completed
Push — master ( 8fa960...c81f91 )
by Jacob
03:35
created
controller/home/ContactController.class.inc.php 1 patch
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -33,17 +33,21 @@
 block discarded – undo
33 33
 
34 34
 	private function process_form()
35 35
 	{
36
-		if(!Request::hasPost() || Request::getPost('submit') != 'Send Message!')
37
-			return (object) array('display' => 'normal');
36
+		if(!Request::hasPost() || Request::getPost('submit') != 'Send Message!') {
37
+					return (object) array('display' => 'normal');
38
+		}
38 39
 		
39 40
 		Loader::load('utility', 'Validate');
40 41
 		$error_result = array();
41
-		if(!Validate::checkRequest('post', 'name', 'string'))
42
-			$error_result['name'] = 'please enter your name';
43
-		if(!Validate::checkRequest('post', 'email', 'string'))
44
-			$error_result['email'] = 'please enter a valid email';
45
-		if(!Validate::checkRequest('post', 'message', 'string'))
46
-			$error_result['message'] = 'please write a message';
42
+		if(!Validate::checkRequest('post', 'name', 'string')) {
43
+					$error_result['name'] = 'please enter your name';
44
+		}
45
+		if(!Validate::checkRequest('post', 'email', 'string')) {
46
+					$error_result['email'] = 'please enter a valid email';
47
+		}
48
+		if(!Validate::checkRequest('post', 'message', 'string')) {
49
+					$error_result['message'] = 'please write a message';
50
+		}
47 51
 		
48 52
 		$values = (object) array(
49 53
 			'name' => Request::getPost('name'),
Please login to merge, or discard this patch.
controller/lifestream/PostController.class.inc.php 1 patch
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -20,15 +20,18 @@
 block discarded – undo
20 20
 		parent::__construct();
21 21
 		
22 22
 		$id = URLDecode::getPiece(2);
23
-		if(!$id || !is_numeric($id))
24
-			$this->eject();
23
+		if(!$id || !is_numeric($id)) {
24
+					$this->eject();
25
+		}
25 26
 		
26 27
 		$post = $this->postRepository->getPostById($id);
27
-		if(!$post)
28
-			$this->eject();
28
+		if(!$post) {
29
+					$this->eject();
30
+		}
29 31
 		
30
-		if(URLDecode::getPiece(1) != $post['type'])
31
-			$this->eject();
32
+		if(URLDecode::getPiece(1) != $post['type']) {
33
+					$this->eject();
34
+		}
32 35
 		
33 36
 		$this->post = $post;
34 37
 	}
Please login to merge, or discard this patch.
utility/Header.class.inc.php 1 patch
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -133,8 +133,9 @@  discard block
 block discarded – undo
133 133
 
134 134
 	private static function send($array, $gzip = true)
135 135
 	{
136
-		if($gzip)
137
-			self::start_gzipping();
136
+		if($gzip) {
137
+					self::start_gzipping();
138
+		}
138 139
 		
139 140
 		foreach($array as $row)
140 141
 		{
@@ -144,15 +145,17 @@  discard block
 block discarded – undo
144 145
 
145 146
 	private static function get_date($timestamp = false)
146 147
 	{
147
-		if($timestamp == 0)
148
-			$timestamp = time();
148
+		if($timestamp == 0) {
149
+					$timestamp = time();
150
+		}
149 151
 		return gmdate('D, d M Y H:i:s \G\M\T', $timestamp);
150 152
 	}
151 153
 
152 154
 	private static function start_gzipping()
153 155
 	{
154
-		if(!ob_start('ob_gzhandler'))
155
-            ob_start();
156
+		if(!ob_start('ob_gzhandler')) {
157
+		            ob_start();
158
+		}
156 159
 	}
157 160
 
158 161
 }
Please login to merge, or discard this patch.
controller/lifestream/HomeController.class.inc.php 1 patch
Braces   +15 added lines, -10 removed lines patch added patch discarded remove patch
@@ -38,8 +38,9 @@  discard block
 block discarded – undo
38 38
 	protected function get_page_number()
39 39
 	{
40 40
 		$page = URLDecode::getPiece(2);
41
-		if(isset($page) && is_numeric($page))
42
-			return $page;
41
+		if(isset($page) && is_numeric($page)) {
42
+					return $page;
43
+		}
43 44
 		return 1;
44 45
 	}
45 46
 
@@ -58,25 +59,29 @@  discard block
 block discarded – undo
58 59
 
59 60
 	protected function get_list_next_link()
60 61
 	{
61
-		if($this->page == 1)
62
-			return;
63
-		if($this->page == 2)
64
-			return Loader::getRootUrl('lifestream');
62
+		if($this->page == 1) {
63
+					return;
64
+		}
65
+		if($this->page == 2) {
66
+					return Loader::getRootUrl('lifestream');
67
+		}
65 68
 		return Loader::getRootUrl('lifestream') . 'page/' . ($this->page - 1) . '/';
66 69
 	}
67 70
 
68 71
 	protected function get_list_prev_link()
69 72
 	{
70
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
71
-			return;
73
+		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count()) {
74
+					return;
75
+		}
72 76
 		return Loader::getRootUrl('lifestream') . 'page/' . ($this->page + 1) . '/';
73 77
 	}
74 78
 
75 79
 	private $total_post_count;
76 80
 	protected function get_total_post_count()
77 81
 	{
78
-		if(!isset($this->total_post_count))
79
-			$this->total_post_count = $this->activityRepository->getActivitiesCount();
82
+		if(!isset($this->total_post_count)) {
83
+					$this->total_post_count = $this->activityRepository->getActivitiesCount();
84
+		}
80 85
 		return $this->total_post_count;
81 86
 	}
82 87
 
Please login to merge, or discard this patch.
controller/lifestream/TagController.class.inc.php 1 patch
Braces   +15 added lines, -10 removed lines patch added patch discarded remove patch
@@ -97,8 +97,9 @@  discard block
 block discarded – undo
97 97
 	protected function get_page_number()
98 98
 	{
99 99
 		$page = URLDecode::getPiece(3);
100
-		if(isset($page) && is_numeric($page))
101
-			return $page;
100
+		if(isset($page) && is_numeric($page)) {
101
+					return $page;
102
+		}
102 103
 		return 1;
103 104
 	}
104 105
 
@@ -117,25 +118,29 @@  discard block
 block discarded – undo
117 118
 
118 119
 	protected function get_list_next_link()
119 120
 	{
120
-		if($this->page == 1)
121
-			return;
122
-		if($this->page == 2)
123
-			return Loader::getRootUrl('lifestream') . $this->tag . '/';
121
+		if($this->page == 1) {
122
+					return;
123
+		}
124
+		if($this->page == 2) {
125
+					return Loader::getRootUrl('lifestream') . $this->tag . '/';
126
+		}
124 127
 		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page - 1) . '/';
125 128
 	}
126 129
 
127 130
 	protected function get_list_prev_link()
128 131
 	{
129
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
130
-			return;
132
+		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count()) {
133
+					return;
134
+		}
131 135
 		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page + 1) . '/';
132 136
 	}
133 137
 
134 138
 	private $total_post_count;
135 139
 	protected function get_total_post_count()
136 140
 	{
137
-		if(!isset($this->total_post_count))
138
-			$this->total_post_count = $this->activityRepository->getActivitiesByTypeCount($this->tag);
141
+		if(!isset($this->total_post_count)) {
142
+					$this->total_post_count = $this->activityRepository->getActivitiesByTypeCount($this->tag);
143
+		}
139 144
 		return $this->total_post_count;
140 145
 	}
141 146
 
Please login to merge, or discard this patch.
controller/PageController.class.inc.php 1 patch
Braces   +40 added lines, -29 removed lines patch added patch discarded remove patch
@@ -102,10 +102,11 @@  discard block
 block discarded – undo
102 102
 		Loader::load('view', '/Head', $this->data_array['head']);
103 103
 		foreach($this->body_view_array as $view)
104 104
 		{
105
-			if(substr($view, 0, 1) == '/')
106
-				Loader::load('view', $view, $this->data_array['body']);
107
-			else
108
-				Loader::load('view', URLDecode::getSite() . '/' . $view, $this->data_array['body']);
105
+			if(substr($view, 0, 1) == '/') {
106
+							Loader::load('view', $view, $this->data_array['body']);
107
+			} else {
108
+							Loader::load('view', URLDecode::getSite() . '/' . $view, $this->data_array['body']);
109
+			}
109 110
 		}
110 111
         
111 112
         if (URLDecode::getSite() == 'waterfalls') {
@@ -114,8 +115,9 @@  discard block
 block discarded – undo
114 115
             Loader::load('view', '/Foot', array('tracking_code' => self::$TRACKING_CODE));
115 116
         }
116 117
 		
117
-		if($view == '/404' || $view == '/503')
118
-			exit;
118
+		if($view == '/404' || $view == '/503') {
119
+					exit;
120
+		}
119 121
 	}
120 122
 
121 123
 	protected function set_header_method($method)
@@ -195,14 +197,16 @@  discard block
 block discarded – undo
195 197
 
196 198
 	protected function eject()
197 199
 	{
198
-		if(get_class($this) !== 'Error404Controller')
199
-			Loader::loadNew('controller', '/Error404Controller')->activate();
200
+		if(get_class($this) !== 'Error404Controller') {
201
+					Loader::loadNew('controller', '/Error404Controller')->activate();
202
+		}
200 203
 	}
201 204
 
202 205
 	protected function unavailable()
203 206
 	{
204
-		if(get_class($this) !== 'Error503Controller')
205
-			Loader::loadNew('controller', '/Error503Controller')->activate();
207
+		if(get_class($this) !== 'Error503Controller') {
208
+					Loader::loadNew('controller', '/Error503Controller')->activate();
209
+		}
206 210
 	}
207 211
 
208 212
 	protected function redirect($uri, $method = 301)
@@ -210,12 +214,14 @@  discard block
 block discarded – undo
210 214
 		switch($method)
211 215
 		{
212 216
 			case 301 :
213
-				if(get_class($this) !== 'Error301Controller')
214
-					Loader::loadNew('controller', '/Error301Controller', array($uri))->activate();
217
+				if(get_class($this) !== 'Error301Controller') {
218
+									Loader::loadNew('controller', '/Error301Controller', array($uri))->activate();
219
+				}
215 220
 				break;
216 221
 			case 303 :
217
-				if(get_class($this) !== 'Error303Controller')
218
-					Loader::loadNew('controller', '/Error303Controller', array($uri))->activate();
222
+				if(get_class($this) !== 'Error303Controller') {
223
+									Loader::loadNew('controller', '/Error303Controller', array($uri))->activate();
224
+				}
219 225
 				break;
220 226
 		}
221 227
 	}
@@ -262,18 +268,19 @@  discard block
 block discarded – undo
262 268
 			$comment_object->url = $comment->url;
263 269
 			$comment_object->trusted = $comment->trusted;
264 270
 			
265
-			if($comment->reply == 0 && Request::getPost('type') == $comment->id)
266
-				$comment_object->errors = $this->comment_errors;
267
-			else
268
-				$comment_object->errors = array();
271
+			if($comment->reply == 0 && Request::getPost('type') == $comment->id) {
272
+							$comment_object->errors = $this->comment_errors;
273
+			} else {
274
+							$comment_object->errors = array();
275
+			}
269 276
 			
270 277
 			if($comment->reply == 0)
271 278
 			{
272 279
 				$comment_object->replies = array();
273 280
 				$comment_array[$comment->id] = $comment_object;
274
-			}
275
-			else
276
-				$comment_array[$comment->reply]->replies[$comment->id] = $comment_object;
281
+			} else {
282
+							$comment_array[$comment->reply]->replies[$comment->id] = $comment_object;
283
+			}
277 284
 		}
278 285
 		
279 286
 		$comment_count = CommentCollector::getCommentCountForURL($site, $path);
@@ -298,22 +305,26 @@  discard block
 block discarded – undo
298 305
 		$commenter->website = '';
299 306
 		
300 307
 		$commenter_cookie = Cookie::instance('Commenter');
301
-		if(!$commenter_cookie->exists())
302
-			return $commenter;
308
+		if(!$commenter_cookie->exists()) {
309
+					return $commenter;
310
+		}
303 311
 		
304 312
 		$commenter_cookie_value = $commenter_cookie->getValue();
305 313
 		$commenter_cookie_value = json_decode($commenter_cookie_value);
306 314
 		
307
-		if($commenter_cookie_value === NULL)
308
-			return $commenter;
315
+		if($commenter_cookie_value === NULL) {
316
+					return $commenter;
317
+		}
309 318
 		
310
-		if(!isset($commenter_cookie_value->name) || !isset($commenter_cookie_value->email))
311
-			return $commenter;
319
+		if(!isset($commenter_cookie_value->name) || !isset($commenter_cookie_value->email)) {
320
+					return $commenter;
321
+		}
312 322
 		
313 323
 		$commenter_object = CommentCollector::getCommenterByFields($commenter_cookie_value->name, $commenter_cookie_value->email, (isset($commenter_cookie_value->website) ? $commenter_cookie_value->website : ''));
314 324
 		
315
-		if($commenter_object === NULL)
316
-			return $commenter;
325
+		if($commenter_object === NULL) {
326
+					return $commenter;
327
+		}
317 328
 		
318 329
 		$commenter->id = $commenter_object->id;
319 330
 		$commenter->name = $commenter_object->name;
Please login to merge, or discard this patch.
utility/Loader.class.inc.php 1 patch
Braces   +11 added lines, -8 removed lines patch added patch discarded remove patch
@@ -17,8 +17,9 @@  discard block
 block discarded – undo
17 17
 
18 18
 	public static function instance()
19 19
 	{
20
-		if(!isset(self::$instance))
21
-			self::$instance = new Loader();
20
+		if(!isset(self::$instance)) {
21
+					self::$instance = new Loader();
22
+		}
22 23
 		return self::$instance;
23 24
 	}
24 25
 
@@ -107,8 +108,9 @@  discard block
 block discarded – undo
107 108
 		foreach((array) $files as $file)
108 109
 		{
109 110
 			$file_path = self::instance()->get_path($type, $file);
110
-			if(in_array($file_path, self::instance()->get_included_files()) && $type !== 'view')
111
-				continue;
111
+			if(in_array($file_path, self::instance()->get_included_files()) && $type !== 'view') {
112
+							continue;
113
+			}
112 114
 			
113 115
 			// if(!file_exists($file_path))
114 116
 			// 	Debugger::logMessage("Requested file does not exist: {$type}, {$file}");
@@ -160,10 +162,11 @@  discard block
 block discarded – undo
160 162
 		
161 163
 		$reflectionObject = self::create_reflection_class($file);
162 164
 		
163
-		if($reflectionObject->hasMethod('__construct'))
164
-			return $reflectionObject->newInstanceArgs($data);
165
-		else
166
-			return $reflectionObject->newInstance();
165
+		if($reflectionObject->hasMethod('__construct')) {
166
+					return $reflectionObject->newInstanceArgs($data);
167
+		} else {
168
+					return $reflectionObject->newInstance();
169
+		}
167 170
 	}
168 171
 
169 172
 	public static function getRoot()
Please login to merge, or discard this patch.
controller/portfolio/ContactController.class.inc.php 1 patch
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -29,15 +29,19 @@
 block discarded – undo
29 29
 
30 30
 	private function get_form_results()
31 31
 	{
32
-		if(!Request::hasPost())
33
-			return array();
32
+		if(!Request::hasPost()) {
33
+					return array();
34
+		}
34 35
 		
35
-		if(!Validate::checkRequest('post', 'name', 'string'))
36
-			$error_message['name'] = 'Please enter a value for your name.';
37
-		if(!Validate::checkRequest('post', 'email', 'string'))
38
-			$error_message['email'] = 'Please enter a valid email address.';
39
-		if(!Validate::checkRequest('post', 'message', 'string'))
40
-			$error_message['message'] = 'Please enter a message.';
36
+		if(!Validate::checkRequest('post', 'name', 'string')) {
37
+					$error_message['name'] = 'Please enter a value for your name.';
38
+		}
39
+		if(!Validate::checkRequest('post', 'email', 'string')) {
40
+					$error_message['email'] = 'Please enter a valid email address.';
41
+		}
42
+		if(!Validate::checkRequest('post', 'message', 'string')) {
43
+					$error_message['message'] = 'Please enter a message.';
44
+		}
41 45
 		
42 46
 		if(!empty($error_message))
43 47
 		{
Please login to merge, or discard this patch.
utility/content/FetchFirstPhotoContent.class.inc.php 1 patch
Braces   +11 added lines, -8 removed lines patch added patch discarded remove patch
@@ -11,10 +11,11 @@  discard block
 block discarded – undo
11 11
 
12 12
 	protected function execute($is_absolute = false, $size = 'thumb')
13 13
 	{
14
-		if(preg_match(self::$PHOTO_PLACEHOLDER_MATCH, $this->content, $match) === 1)
15
-			$this->content = $this->get_thumb($match[1], $is_absolute, $size);
16
-		else
17
-			$this->content = self::$DEFAULT_RETURN;
14
+		if(preg_match(self::$PHOTO_PLACEHOLDER_MATCH, $this->content, $match) === 1) {
15
+					$this->content = $this->get_thumb($match[1], $is_absolute, $size);
16
+		} else {
17
+					$this->content = self::$DEFAULT_RETURN;
18
+		}
18 19
 		return;
19 20
 	}
20 21
 
@@ -34,16 +35,18 @@  discard block
 block discarded – undo
34 35
 		
35 36
 		Loader::load('collector', 'image/PhotoCollector');
36 37
 		$photo_result = PhotoCollector::fetchRow($category, $photo);
37
-		if($photo_result == false)
38
-			return '';
38
+		if($photo_result == false) {
39
+					return '';
40
+		}
39 41
 		
40 42
 		$height = $file_size[1];
41 43
 		$width = $file_size[0];
42 44
 		$description = $photo_result->description;
43 45
 		
44 46
 		$domain = '/';
45
-		if($is_absolute)
46
-			$domain = Loader::getRootURL(URLDecode::getSite());
47
+		if($is_absolute) {
48
+					$domain = Loader::getRootURL(URLDecode::getSite());
49
+		}
47 50
 
48 51
 		return sprintf(self::$IMAGE_NODE, $domain, $category, $photo, $size, $extension, $height, $width, $description);
49 52
 	}
Please login to merge, or discard this patch.