Completed
Pull Request — master (#1)
by Jacob
15:50
created
collector/blog/PostCollector.class.inc.php 2 patches
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -17,6 +17,9 @@  discard block
 block discarded – undo
17 17
 		return self::run_query($query);
18 18
 	}
19 19
 
20
+	/**
21
+	 * @param integer $total
22
+	 */
20 23
 	public static function getMainList($total, $offset = 0)
21 24
 	{
22 25
 		$query = "SELECT `post`.`id` AS `id`, `title`, `path`, `date`, `body`, `category` FROM `jpemeric_blog`.`post` WHERE `display` = '1' ORDER BY `date` DESC LIMIT {$offset}, {$total}";
@@ -29,6 +32,9 @@  discard block
 block discarded – undo
29 32
 		return self::get_count($query);
30 33
 	}
31 34
 
35
+	/**
36
+	 * @param integer $total
37
+	 */
32 38
 	public static function getPostsForTag($tag_id, $total, $offset = 0)
33 39
 	{
34 40
 		$query = "SELECT `post`.`id` AS `id`, `title`, `path`, `date`, `body`, `category` FROM `jpemeric_blog`.`post`,`jpemeric_blog`.`ptlink` WHERE `display` = '1' && `post`.`id` = `ptlink`.`post_id` && `ptlink`.`tag_id` = '{$tag_id}' ORDER BY `date` DESC LIMIT {$offset}, {$total}";
@@ -41,6 +47,9 @@  discard block
 block discarded – undo
41 47
 		return self::get_count($query);
42 48
 	}
43 49
 
50
+	/**
51
+	 * @param integer $total
52
+	 */
44 53
 	public static function getPostsForCategory($category, $total, $offset = 0)
45 54
 	{
46 55
 		$category = self::escape($category);
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@
 block discarded – undo
80 80
 		$tag_clause = "'{$tag_clause}'";
81 81
 		
82 82
 		$exclude_clause = '';
83
-		if(count($exclude_posts) > 0)
83
+		if (count($exclude_posts) > 0)
84 84
 		{
85 85
 			$exclude_clause .= " && `ptlink`.`post_id` NOT IN ('";
86 86
 			$exclude_clause .= implode("', '", $exclude_posts);
Please login to merge, or discard this patch.
collector/comment/CommentCollector.class.inc.php 4 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -83,6 +83,9 @@
 block discarded – undo
83 83
 		return self::run_query($query);
84 84
 	}
85 85
 
86
+	/**
87
+	 * @param integer $count
88
+	 */
86 89
 	public static function getRecentBlogComments($count)
87 90
 	{
88 91
 		$query = "
Please login to merge, or discard this patch.
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -114,9 +114,9 @@  discard block
 block discarded – undo
114 114
 		return self::run_query($query);
115 115
 	}
116 116
 
117
-    public static function getRecentWaterfallComments($count = 5)
118
-    {
119
-        $query = "
117
+	public static function getRecentWaterfallComments($count = 5)
118
+	{
119
+		$query = "
120 120
             SELECT
121 121
                 `comment_meta`.`id`,
122 122
                 CONCAT('journal/', `log`.`alias`) AS `log_path`,
@@ -156,8 +156,8 @@  discard block
 block discarded – undo
156 156
                 `comment_meta`.`date` DESC
157 157
             LIMIT {$count}";
158 158
         
159
-        return self::run_query($query);
160
-    }
159
+		return self::run_query($query);
160
+	}
161 161
 
162 162
 	public static function getCommentsForURL($site, $path, $commenter = 0)
163 163
 	{
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -163,7 +163,7 @@
 block discarded – undo
163 163
 	{
164 164
 		$path = self::escape($path);
165 165
 		
166
-		if($commenter != 0)
166
+		if ($commenter != 0)
167 167
 			$trusted_commenter_clause = "(`commenter`.`trusted` = '1' || `commenter`.`id` = '{$commenter}')";
168 168
 		else
169 169
 			$trusted_commenter_clause = "`commenter`.`trusted` = '1'";
Please login to merge, or discard this patch.
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -163,10 +163,11 @@
 block discarded – undo
163 163
 	{
164 164
 		$path = self::escape($path);
165 165
 		
166
-		if($commenter != 0)
167
-			$trusted_commenter_clause = "(`commenter`.`trusted` = '1' || `commenter`.`id` = '{$commenter}')";
168
-		else
169
-			$trusted_commenter_clause = "`commenter`.`trusted` = '1'";
166
+		if($commenter != 0) {
167
+					$trusted_commenter_clause = "(`commenter`.`trusted` = '1' || `commenter`.`id` = '{$commenter}')";
168
+		} else {
169
+					$trusted_commenter_clause = "`commenter`.`trusted` = '1'";
170
+		}
170 171
 		
171 172
 		$query = "
172 173
 			SELECT
Please login to merge, or discard this patch.
collector/portfolio/PortfolioCollector.class.inc.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -19,6 +19,9 @@
 block discarded – undo
19 19
 		return self::run_row_query($query);
20 20
 	}
21 21
 
22
+	/**
23
+	 * @param integer $type
24
+	 */
22 25
 	public static function getImagesForPiece($piece_id, $type)
23 26
 	{
24 27
 		$piece_id = self::escape($piece_id);
Please login to merge, or discard this patch.
collector/waterfall/LogCollector.class.inc.php 2 patches
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -34,6 +34,9 @@  discard block
 block discarded – undo
34 34
 		return self::run_query($query);
35 35
 	}
36 36
 
37
+    /**
38
+     * @param integer $total
39
+     */
37 40
     public static function getRecentList($total)
38 41
     {
39 42
         $query = "
@@ -178,6 +181,9 @@  discard block
 block discarded – undo
178 181
 		return self::run_row_query($query);
179 182
 	}
180 183
 
184
+    /**
185
+     * @param string $date
186
+     */
181 187
     public static function getByDate($date)
182 188
     {
183 189
         $date = self::escape($date);
Please login to merge, or discard this patch.
Indentation   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -34,9 +34,9 @@  discard block
 block discarded – undo
34 34
 		return self::run_query($query);
35 35
 	}
36 36
 
37
-    public static function getRecentList($total)
38
-    {
39
-        $query = "
37
+	public static function getRecentList($total)
38
+	{
39
+		$query = "
40 40
             SELECT
41 41
                 `log`.`title`,
42 42
                 `log`.`alias`,
@@ -59,8 +59,8 @@  discard block
 block discarded – undo
59 59
                 `log`.`date` DESC
60 60
             LIMIT
61 61
                 {$total}";
62
-        return self::run_query($query);
63
-    }
62
+		return self::run_query($query);
63
+	}
64 64
 
65 65
 	public static function getListCount()
66 66
 	{
@@ -178,11 +178,11 @@  discard block
 block discarded – undo
178 178
 		return self::run_row_query($query);
179 179
 	}
180 180
 
181
-    public static function getByDate($date)
182
-    {
183
-        $date = self::escape($date);
181
+	public static function getByDate($date)
182
+	{
183
+		$date = self::escape($date);
184 184
         
185
-        $query = "
185
+		$query = "
186 186
             SELECT
187 187
                 `log`.`alias`
188 188
             FROM
@@ -192,8 +192,8 @@  discard block
 block discarded – undo
192 192
                 `log`.`is_public` = '1'
193 193
             LIMIT 1";
194 194
         
195
-        return self::run_row_query($query);
196
-    }
195
+		return self::run_row_query($query);
196
+	}
197 197
 	
198 198
 	public static function getById($log)
199 199
 	{
Please login to merge, or discard this patch.
collector/waterfall/WaterfallCollector.class.inc.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -117,6 +117,9 @@
 block discarded – undo
117 117
         return self::run_query($query);
118 118
     }
119 119
 
120
+    /**
121
+     * @param string $alias
122
+     */
120 123
     public static function getByOldAlias($alias)
121 124
     {
122 125
         $alias = self::escape($alias);
Please login to merge, or discard this patch.
Indentation   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -43,9 +43,9 @@  discard block
 block discarded – undo
43 43
 		return self::run_query($query);
44 44
 	}
45 45
 
46
-    public static function getMapList()
47
-    {
48
-        $query = "
46
+	public static function getMapList()
47
+	{
48
+		$query = "
49 49
             SELECT
50 50
                 `waterfall`.`name`,
51 51
                 `waterfall`.`alias`,
@@ -71,8 +71,8 @@  discard block
 block discarded – undo
71 71
             ORDER BY
72 72
                 `waterfall`.`name`";
73 73
         
74
-        return self::run_query($query);
75
-    }
74
+		return self::run_query($query);
75
+	}
76 76
 
77 77
 	public static function getListCount()
78 78
 	{
@@ -92,9 +92,9 @@  discard block
 block discarded – undo
92 92
 		return self::get_count($query);
93 93
 	}
94 94
 
95
-    public static function getNearbyList($waterfall)
96
-    {
97
-        $query = "
95
+	public static function getNearbyList($waterfall)
96
+	{
97
+		$query = "
98 98
             SELECT
99 99
                 `waterfall`.`name`,
100 100
                 `waterfall`.`alias` AS `alias`,
@@ -114,14 +114,14 @@  discard block
 block discarded – undo
114 114
                 `waterfall`.`name`,
115 115
                 `watercourse`.`name`";
116 116
         
117
-        return self::run_query($query);
118
-    }
117
+		return self::run_query($query);
118
+	}
119 119
 
120
-    public static function getByOldAlias($alias)
121
-    {
122
-        $alias = self::escape($alias);
120
+	public static function getByOldAlias($alias)
121
+	{
122
+		$alias = self::escape($alias);
123 123
         
124
-        $query = "
124
+		$query = "
125 125
             SELECT
126 126
                 `waterfall`.`alias`,
127 127
                 `watercourse`.`alias` AS `watercourse_alias`
@@ -134,8 +134,8 @@  discard block
 block discarded – undo
134 134
                 `waterfall`.`is_public` = '1'
135 135
             LIMIT 1";
136 136
         
137
-        return self::run_row_query($query);
138
-    }
137
+		return self::run_row_query($query);
138
+	}
139 139
 
140 140
 	public static function getByAlias($watercourse, $waterfall)
141 141
 	{
Please login to merge, or discard this patch.
controller/blog/DefaultPageController.class.inc.php 4 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -97,6 +97,9 @@
 block discarded – undo
97 97
 		return $tag_array;
98 98
 	}
99 99
 
100
+	/**
101
+	 * @param boolean $trim
102
+	 */
100 103
 	final private function get_body_for_post($post, $trim)
101 104
 	{
102 105
 		$body = $post->body;
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 		Loader::load('collector', 'image/PhotoCollector');
38 38
 		$photo_result = PhotoCollector::getRow($id);
39 39
 		
40
-		if($photo_result == null)
40
+		if ($photo_result == null)
41 41
 			return;
42 42
 		
43 43
 		$name = $photo_result->name;
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 		
88 88
 		$tag_result = TagCollector::getTagsForPost($post->id);
89 89
         $tag_array = array();
90
-		foreach($tag_result as $tag)
90
+		foreach ($tag_result as $tag)
91 91
 		{
92 92
 			$tag_object = new stdclass();
93 93
 			$tag_object->name = $tag->tag;
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 	{
102 102
 		$body = $post->body;
103 103
 		
104
-		if($trim)
104
+		if ($trim)
105 105
 			$body = Content::instance('SmartTrim', $body)->activate(self::$LENGTH_OF_TRIMMED_POST);
106 106
 		
107 107
 		$body = Content::instance('FixPhoto', $body)->activate(false, 'standard');
@@ -126,9 +126,9 @@  discard block
 block discarded – undo
126 126
 		$maximum_tag_count = $this->get_maximum_tag_count($tag_result);
127 127
 		
128 128
 		$cloud_array = array();
129
-		foreach($tag_result as $tag)
129
+		foreach ($tag_result as $tag)
130 130
 		{
131
-			if($tag->tag_count < self::$MINIMUM_TAG_COUNT)
131
+			if ($tag->tag_count < self::$MINIMUM_TAG_COUNT)
132 132
 				continue;
133 133
 			
134 134
 			$tag_object = new stdclass();
@@ -145,9 +145,9 @@  discard block
 block discarded – undo
145 145
 	{
146 146
 		$maximum = 1;
147 147
 		
148
-		foreach($tag_result as $tag)
148
+		foreach ($tag_result as $tag)
149 149
 		{
150
-			if($tag->tag_count > $maximum)
150
+			if ($tag->tag_count > $maximum)
151 151
 				$maximum = $tag->tag_count;
152 152
 		}
153 153
 		return $maximum;
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
 		$comment_array = CommentCollector::getRecentBlogComments(self::$RECENT_COMMENT_COUNT);
159 159
 		
160 160
 		$array = array();
161
-		foreach($comment_array as $comment)
161
+		foreach ($comment_array as $comment)
162 162
 		{
163 163
 			$body = $comment->body;
164 164
 			$body = strip_tags($body);
Please login to merge, or discard this patch.
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -37,8 +37,9 @@  discard block
 block discarded – undo
37 37
 		Loader::load('collector', 'image/PhotoCollector');
38 38
 		$photo_result = PhotoCollector::getRow($id);
39 39
 		
40
-		if($photo_result == null)
41
-			return;
40
+		if($photo_result == null) {
41
+					return;
42
+		}
42 43
 		
43 44
 		$name = $photo_result->name;
44 45
 		$category = $photo_result->category;
@@ -101,8 +102,9 @@  discard block
 block discarded – undo
101 102
 	{
102 103
 		$body = $post->body;
103 104
 		
104
-		if($trim)
105
-			$body = Content::instance('SmartTrim', $body)->activate(self::$LENGTH_OF_TRIMMED_POST);
105
+		if($trim) {
106
+					$body = Content::instance('SmartTrim', $body)->activate(self::$LENGTH_OF_TRIMMED_POST);
107
+		}
106 108
 		
107 109
 		$body = Content::instance('FixPhoto', $body)->activate(false, 'standard');
108 110
 		$body = Content::instance('MarkupCode', $body)->activate();
@@ -128,8 +130,9 @@  discard block
 block discarded – undo
128 130
 		$cloud_array = array();
129 131
 		foreach($tag_result as $tag)
130 132
 		{
131
-			if($tag->tag_count < self::$MINIMUM_TAG_COUNT)
132
-				continue;
133
+			if($tag->tag_count < self::$MINIMUM_TAG_COUNT) {
134
+							continue;
135
+			}
133 136
 			
134 137
 			$tag_object = new stdclass();
135 138
 			$tag_object->name = $tag->tag;
@@ -147,8 +150,9 @@  discard block
 block discarded – undo
147 150
 		
148 151
 		foreach($tag_result as $tag)
149 152
 		{
150
-			if($tag->tag_count > $maximum)
151
-				$maximum = $tag->tag_count;
153
+			if($tag->tag_count > $maximum) {
154
+							$maximum = $tag->tag_count;
155
+			}
152 156
 		}
153 157
 		return $maximum;
154 158
 	}
Please login to merge, or discard this patch.
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -20,14 +20,14 @@  discard block
 block discarded – undo
20 20
 
21 21
 	protected function set_head_data()
22 22
 	{
23
-    $this->set_head('rss_link', [
24
-      'title' => 'Jacob Emerick Blog Feed',
25
-      'url' => '/rss.xml'
26
-    ]);
27
-    $this->set_head('rss_comment_link', [
28
-      'title' => 'Jacob Emerick Blog Comment Feed',
29
-      'url' => '/rss-comments.xml'
30
-    ]);
23
+	$this->set_head('rss_link', [
24
+	  'title' => 'Jacob Emerick Blog Feed',
25
+	  'url' => '/rss.xml'
26
+	]);
27
+	$this->set_head('rss_comment_link', [
28
+	  'title' => 'Jacob Emerick Blog Comment Feed',
29
+	  'url' => '/rss-comments.xml'
30
+	]);
31 31
 		
32 32
 		$this->add_css('normalize');
33 33
 		$this->add_css('blog');
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 		Loader::load('collector', 'blog/TagCollector');
93 93
 		
94 94
 		$tag_result = TagCollector::getTagsForPost($post->id);
95
-        $tag_array = array();
95
+		$tag_array = array();
96 96
 		foreach($tag_result as $tag)
97 97
 		{
98 98
 			$tag_object = new stdclass();
Please login to merge, or discard this patch.
controller/PageController.class.inc.php 4 patches
Doc Comments   +28 added lines patch added patch discarded remove patch
@@ -204,6 +204,9 @@  discard block
 block discarded – undo
204 204
 			exit;
205 205
 	}
206 206
 
207
+	/**
208
+	 * @param string $method
209
+	 */
207 210
 	protected function set_header_method($method)
208 211
 	{
209 212
 		$this->headers = $method;
@@ -214,6 +217,9 @@  discard block
 block discarded – undo
214 217
 		$this->set_head('title', $value);
215 218
 	}
216 219
 
220
+	/**
221
+	 * @param string $value
222
+	 */
217 223
 	protected function set_author($value)
218 224
 	{
219 225
 		$this->set_head('author', $value);
@@ -229,26 +235,41 @@  discard block
 block discarded – undo
229 235
 		$this->set_head('keywords', implode(', ', $array));
230 236
 	}
231 237
 	
238
+	/**
239
+	 * @param string $url
240
+	 */
232 241
 	protected function set_canonical($url)
233 242
 	{
234 243
 		$this->set_head('canonical', $url);
235 244
 	}
236 245
 
246
+	/**
247
+	 * @param string $set
248
+	 */
237 249
 	protected function set_head($set, $value)
238 250
 	{
239 251
 		$this->data_array['head'][$set] = $value;
240 252
 	}
241 253
 
254
+	/**
255
+	 * @param string $set
256
+	 */
242 257
 	protected function set_body($set, $value)
243 258
 	{
244 259
 		$this->data_array['body'][$set] = $value;
245 260
 	}
246 261
 
262
+	/**
263
+	 * @param string $file
264
+	 */
247 265
 	protected function add_css($file)
248 266
 	{
249 267
 		$this->css_array[] = $file;
250 268
 	}
251 269
 
270
+	/**
271
+	 * @param string $file
272
+	 */
252 273
 	protected function add_js($file)
253 274
 	{
254 275
 		$this->js_array[] = $file;
@@ -271,6 +292,9 @@  discard block
 block discarded – undo
271 292
 		$this->set_head('js_link_array', $js_array);
272 293
 	}
273 294
 
295
+	/**
296
+	 * @param string $view
297
+	 */
274 298
 	protected function set_body_view($view)
275 299
 	{
276 300
 		$this->body_view_array[] = $view;
@@ -315,6 +339,10 @@  discard block
 block discarded – undo
315 339
 	}
316 340
 
317 341
 	private $comment_errors;
342
+
343
+	/**
344
+	 * @param string $redirect_url
345
+	 */
318 346
 	protected function handle_comment_submit($site_id, $path, $redirect_url, $page_title)
319 347
 	{
320 348
 		if(Request::hasPost() && Request::getPost('submit') == 'Submit Comment')
Please login to merge, or discard this patch.
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
 			'date' => date('Y')));
39 39
 		
40 40
 		Loader::loadInstance('utility', 'Database');
41
-		if(Database::isConnected() === false)
41
+		if (Database::isConnected() === false)
42 42
 			$this->unavailable();
43 43
 	}
44 44
 
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
         $post_result = $activityRepository->getActivities(5);
64 64
 
65 65
         $post_array = array();
66
-        foreach($post_result as $row) {
66
+        foreach ($post_result as $row) {
67 67
             array_push($post_array, $this->expand_post($row));
68 68
         }
69 69
 
@@ -99,9 +99,9 @@  discard block
 block discarded – undo
99 99
 		$headers = $this->headers;
100 100
 		Header::$headers();
101 101
 		Loader::load('view', '/Head', $this->data_array['head']);
102
-		foreach($this->body_view_array as $view)
102
+		foreach ($this->body_view_array as $view)
103 103
 		{
104
-			if(substr($view, 0, 1) == '/')
104
+			if (substr($view, 0, 1) == '/')
105 105
 				Loader::load('view', $view, $this->data_array['body']);
106 106
 			else
107 107
 				Loader::load('view', URLDecode::getSite() . '/' . $view, $this->data_array['body']);
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
             Loader::load('view', '/Foot', array('tracking_code' => self::$TRACKING_CODE));
114 114
         }
115 115
 		
116
-		if($view == '/404' || $view == '/503')
116
+		if ($view == '/404' || $view == '/503')
117 117
 			exit;
118 118
 	}
119 119
 
@@ -169,10 +169,10 @@  discard block
 block discarded – undo
169 169
 
170 170
 	private function load_assets()
171 171
 	{
172
-    $css_array = array_map(function ($stylesheet) {
172
+    $css_array = array_map(function($stylesheet) {
173 173
       return "/css/{$stylesheet}.css";
174 174
     }, $this->css_array);
175
-    $js_array = array_map(function ($script) {
175
+    $js_array = array_map(function($script) {
176 176
       if (substr($script, 0, 4) == 'http') {
177 177
         return $script;
178 178
       }
@@ -190,26 +190,26 @@  discard block
 block discarded – undo
190 190
 
191 191
 	protected function eject()
192 192
 	{
193
-		if(get_class($this) !== 'Error404Controller')
193
+		if (get_class($this) !== 'Error404Controller')
194 194
 			Loader::loadNew('controller', '/Error404Controller')->activate();
195 195
 	}
196 196
 
197 197
 	protected function unavailable()
198 198
 	{
199
-		if(get_class($this) !== 'Error503Controller')
199
+		if (get_class($this) !== 'Error503Controller')
200 200
 			Loader::loadNew('controller', '/Error503Controller')->activate();
201 201
 	}
202 202
 
203 203
 	protected function redirect($uri, $method = 301)
204 204
 	{
205
-		switch($method)
205
+		switch ($method)
206 206
 		{
207 207
 			case 301 :
208
-				if(get_class($this) !== 'Error301Controller')
208
+				if (get_class($this) !== 'Error301Controller')
209 209
 					Loader::loadNew('controller', '/Error301Controller', array($uri))->activate();
210 210
 				break;
211 211
 			case 303 :
212
-				if(get_class($this) !== 'Error303Controller')
212
+				if (get_class($this) !== 'Error303Controller')
213 213
 					Loader::loadNew('controller', '/Error303Controller', array($uri))->activate();
214 214
 				break;
215 215
 		}
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
229 229
 	private $comment_errors;
230 230
 	protected function handle_comment_submit($site_id, $path, $redirect_url, $page_title)
231 231
 	{
232
-		if(Request::hasPost() && Request::getPost('submit') == 'Submit Comment')
232
+		if (Request::hasPost() && Request::getPost('submit') == 'Submit Comment')
233 233
 		{
234 234
 			$parameters = array($site_id, $path, $redirect_url, $page_title);
235 235
 			$this->comment_errors = Loader::loadNew('module', 'form/CommentSubmitModule', $parameters)->activate();
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
 		$comment_array = array();
248 248
 		$comment_result = CommentCollector::getCommentsForURL($site, $path, $commenter->id);
249 249
 		
250
-		foreach($comment_result as $comment)
250
+		foreach ($comment_result as $comment)
251 251
 		{
252 252
 			$comment_object = new stdclass();
253 253
 			$comment_object->id = $comment->id;
@@ -257,12 +257,12 @@  discard block
 block discarded – undo
257 257
 			$comment_object->url = $comment->url;
258 258
 			$comment_object->trusted = $comment->trusted;
259 259
 			
260
-			if($comment->reply == 0 && Request::getPost('type') == $comment->id)
260
+			if ($comment->reply == 0 && Request::getPost('type') == $comment->id)
261 261
 				$comment_object->errors = $this->comment_errors;
262 262
 			else
263 263
 				$comment_object->errors = array();
264 264
 			
265
-			if($comment->reply == 0)
265
+			if ($comment->reply == 0)
266 266
 			{
267 267
 				$comment_object->replies = array();
268 268
 				$comment_array[$comment->id] = $comment_object;
@@ -293,21 +293,21 @@  discard block
 block discarded – undo
293 293
 		$commenter->website = '';
294 294
 		
295 295
 		$commenter_cookie = Cookie::instance('Commenter');
296
-		if(!$commenter_cookie->exists())
296
+		if (!$commenter_cookie->exists())
297 297
 			return $commenter;
298 298
 		
299 299
 		$commenter_cookie_value = $commenter_cookie->getValue();
300 300
 		$commenter_cookie_value = json_decode($commenter_cookie_value);
301 301
 		
302
-		if($commenter_cookie_value === NULL)
302
+		if ($commenter_cookie_value === NULL)
303 303
 			return $commenter;
304 304
 		
305
-		if(!isset($commenter_cookie_value->name) || !isset($commenter_cookie_value->email))
305
+		if (!isset($commenter_cookie_value->name) || !isset($commenter_cookie_value->email))
306 306
 			return $commenter;
307 307
 		
308 308
 		$commenter_object = CommentCollector::getCommenterByFields($commenter_cookie_value->name, $commenter_cookie_value->email, (isset($commenter_cookie_value->website) ? $commenter_cookie_value->website : ''));
309 309
 		
310
-		if($commenter_object === NULL)
310
+		if ($commenter_object === NULL)
311 311
 			return $commenter;
312 312
 		
313 313
 		$commenter->id = $commenter_object->id;
Please login to merge, or discard this patch.
Braces   +43 added lines, -31 removed lines patch added patch discarded remove patch
@@ -38,8 +38,9 @@  discard block
 block discarded – undo
38 38
 			'date' => date('Y')));
39 39
 		
40 40
 		Loader::loadInstance('utility', 'Database');
41
-		if(Database::isConnected() === false)
42
-			$this->unavailable();
41
+		if(Database::isConnected() === false) {
42
+					$this->unavailable();
43
+		}
43 44
 	}
44 45
 
45 46
 	protected function get_domain_container()
@@ -101,10 +102,11 @@  discard block
 block discarded – undo
101 102
 		Loader::load('view', '/Head', $this->data_array['head']);
102 103
 		foreach($this->body_view_array as $view)
103 104
 		{
104
-			if(substr($view, 0, 1) == '/')
105
-				Loader::load('view', $view, $this->data_array['body']);
106
-			else
107
-				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
+			}
108 110
 		}
109 111
         
110 112
         if (URLDecode::getSite() == 'waterfalls') {
@@ -113,8 +115,9 @@  discard block
 block discarded – undo
113 115
             Loader::load('view', '/Foot', array('tracking_code' => self::$TRACKING_CODE));
114 116
         }
115 117
 		
116
-		if($view == '/404' || $view == '/503')
117
-			exit;
118
+		if($view == '/404' || $view == '/503') {
119
+					exit;
120
+		}
118 121
 	}
119 122
 
120 123
 	protected function set_header_method($method)
@@ -190,14 +193,16 @@  discard block
 block discarded – undo
190 193
 
191 194
 	protected function eject()
192 195
 	{
193
-		if(get_class($this) !== 'Error404Controller')
194
-			Loader::loadNew('controller', '/Error404Controller')->activate();
196
+		if(get_class($this) !== 'Error404Controller') {
197
+					Loader::loadNew('controller', '/Error404Controller')->activate();
198
+		}
195 199
 	}
196 200
 
197 201
 	protected function unavailable()
198 202
 	{
199
-		if(get_class($this) !== 'Error503Controller')
200
-			Loader::loadNew('controller', '/Error503Controller')->activate();
203
+		if(get_class($this) !== 'Error503Controller') {
204
+					Loader::loadNew('controller', '/Error503Controller')->activate();
205
+		}
201 206
 	}
202 207
 
203 208
 	protected function redirect($uri, $method = 301)
@@ -205,12 +210,14 @@  discard block
 block discarded – undo
205 210
 		switch($method)
206 211
 		{
207 212
 			case 301 :
208
-				if(get_class($this) !== 'Error301Controller')
209
-					Loader::loadNew('controller', '/Error301Controller', array($uri))->activate();
213
+				if(get_class($this) !== 'Error301Controller') {
214
+									Loader::loadNew('controller', '/Error301Controller', array($uri))->activate();
215
+				}
210 216
 				break;
211 217
 			case 303 :
212
-				if(get_class($this) !== 'Error303Controller')
213
-					Loader::loadNew('controller', '/Error303Controller', array($uri))->activate();
218
+				if(get_class($this) !== 'Error303Controller') {
219
+									Loader::loadNew('controller', '/Error303Controller', array($uri))->activate();
220
+				}
214 221
 				break;
215 222
 		}
216 223
 	}
@@ -257,18 +264,19 @@  discard block
 block discarded – undo
257 264
 			$comment_object->url = $comment->url;
258 265
 			$comment_object->trusted = $comment->trusted;
259 266
 			
260
-			if($comment->reply == 0 && Request::getPost('type') == $comment->id)
261
-				$comment_object->errors = $this->comment_errors;
262
-			else
263
-				$comment_object->errors = array();
267
+			if($comment->reply == 0 && Request::getPost('type') == $comment->id) {
268
+							$comment_object->errors = $this->comment_errors;
269
+			} else {
270
+							$comment_object->errors = array();
271
+			}
264 272
 			
265 273
 			if($comment->reply == 0)
266 274
 			{
267 275
 				$comment_object->replies = array();
268 276
 				$comment_array[$comment->id] = $comment_object;
269
-			}
270
-			else
271
-				$comment_array[$comment->reply]->replies[$comment->id] = $comment_object;
277
+			} else {
278
+							$comment_array[$comment->reply]->replies[$comment->id] = $comment_object;
279
+			}
272 280
 		}
273 281
 		
274 282
 		$comment_count = CommentCollector::getCommentCountForURL($site, $path);
@@ -293,22 +301,26 @@  discard block
 block discarded – undo
293 301
 		$commenter->website = '';
294 302
 		
295 303
 		$commenter_cookie = Cookie::instance('Commenter');
296
-		if(!$commenter_cookie->exists())
297
-			return $commenter;
304
+		if(!$commenter_cookie->exists()) {
305
+					return $commenter;
306
+		}
298 307
 		
299 308
 		$commenter_cookie_value = $commenter_cookie->getValue();
300 309
 		$commenter_cookie_value = json_decode($commenter_cookie_value);
301 310
 		
302
-		if($commenter_cookie_value === NULL)
303
-			return $commenter;
311
+		if($commenter_cookie_value === NULL) {
312
+					return $commenter;
313
+		}
304 314
 		
305
-		if(!isset($commenter_cookie_value->name) || !isset($commenter_cookie_value->email))
306
-			return $commenter;
315
+		if(!isset($commenter_cookie_value->name) || !isset($commenter_cookie_value->email)) {
316
+					return $commenter;
317
+		}
307 318
 		
308 319
 		$commenter_object = CommentCollector::getCommenterByFields($commenter_cookie_value->name, $commenter_cookie_value->email, (isset($commenter_cookie_value->website) ? $commenter_cookie_value->website : ''));
309 320
 		
310
-		if($commenter_object === NULL)
311
-			return $commenter;
321
+		if($commenter_object === NULL) {
322
+					return $commenter;
323
+		}
312 324
 		
313 325
 		$commenter->id = $commenter_object->id;
314 326
 		$commenter->name = $commenter_object->name;
Please login to merge, or discard this patch.
Indentation   +49 added lines, -49 removed lines patch added patch discarded remove patch
@@ -56,37 +56,37 @@  discard block
 block discarded – undo
56 56
 		return $domain_container;
57 57
 	}
58 58
 
59
-    protected function get_recent_activity()
60
-    {
61
-        global $container;
62
-        $activityRepository = new Jacobemerick\Web\Domain\Stream\Activity\MysqlActivityRepository($container['db_connection_locator']);
63
-        $post_result = $activityRepository->getActivities(5);
64
-
65
-        $post_array = array();
66
-        foreach($post_result as $row) {
67
-            array_push($post_array, $this->expand_post($row));
68
-        }
69
-
70
-        return $post_array;
71
-    }
72
-
73
-    protected function expand_post($raw_post, $format = 'short')
74
-    {
75
-        $post = [
76
-            'type' => $raw_post['type'],
77
-            'title' => ($format == 'short') ? $raw_post['message'] : $raw_post['message_long'],
78
-            'date' => $this->get_parsed_date($raw_post['datetime']),
79
-        ];
80
-
81
-        if ($format != 'short') {
82
-            $post['url'] = Loader::getRootUrl('lifestream') . "{$raw_post['type']}/{$raw_post['id']}/";
83
-
84
-            $metadata = json_decode($raw_post['metadata'], true);
85
-            $post = array_merge($post, $metadata);
86
-        }
87
-
88
-        return (object) $post;
89
-    }
59
+	protected function get_recent_activity()
60
+	{
61
+		global $container;
62
+		$activityRepository = new Jacobemerick\Web\Domain\Stream\Activity\MysqlActivityRepository($container['db_connection_locator']);
63
+		$post_result = $activityRepository->getActivities(5);
64
+
65
+		$post_array = array();
66
+		foreach($post_result as $row) {
67
+			array_push($post_array, $this->expand_post($row));
68
+		}
69
+
70
+		return $post_array;
71
+	}
72
+
73
+	protected function expand_post($raw_post, $format = 'short')
74
+	{
75
+		$post = [
76
+			'type' => $raw_post['type'],
77
+			'title' => ($format == 'short') ? $raw_post['message'] : $raw_post['message_long'],
78
+			'date' => $this->get_parsed_date($raw_post['datetime']),
79
+		];
80
+
81
+		if ($format != 'short') {
82
+			$post['url'] = Loader::getRootUrl('lifestream') . "{$raw_post['type']}/{$raw_post['id']}/";
83
+
84
+			$metadata = json_decode($raw_post['metadata'], true);
85
+			$post = array_merge($post, $metadata);
86
+		}
87
+
88
+		return (object) $post;
89
+	}
90 90
 
91 91
 	public function activate()
92 92
 	{
@@ -107,11 +107,11 @@  discard block
 block discarded – undo
107 107
 				Loader::load('view', URLDecode::getSite() . '/' . $view, $this->data_array['body']);
108 108
 		}
109 109
         
110
-        if (URLDecode::getSite() == 'waterfalls') {
111
-            Loader::load('view', '/WaterfallFoot');
112
-        } else {
113
-            Loader::load('view', '/Foot', array('tracking_code' => self::$TRACKING_CODE));
114
-        }
110
+		if (URLDecode::getSite() == 'waterfalls') {
111
+			Loader::load('view', '/WaterfallFoot');
112
+		} else {
113
+			Loader::load('view', '/Foot', array('tracking_code' => self::$TRACKING_CODE));
114
+		}
115 115
 		
116 116
 		if($view == '/404' || $view == '/503')
117 117
 			exit;
@@ -169,19 +169,19 @@  discard block
 block discarded – undo
169 169
 
170 170
 	private function load_assets()
171 171
 	{
172
-    $css_array = array_map(function ($stylesheet) {
173
-      $path = "/css/{$stylesheet[0]}.css";
174
-      if ($stylesheet[1] > 1) {
175
-        $path .= "?v={$stylesheet[1]}";
176
-      }
177
-      return $path;
178
-    }, $this->css_array);
179
-    $js_array = array_map(function ($script) {
180
-      if (substr($script, 0, 4) == 'http') {
181
-        return $script;
182
-      }
183
-      return "/js/{$script}.min.js";
184
-    }, $this->js_array);
172
+	$css_array = array_map(function ($stylesheet) {
173
+	  $path = "/css/{$stylesheet[0]}.css";
174
+	  if ($stylesheet[1] > 1) {
175
+		$path .= "?v={$stylesheet[1]}";
176
+	  }
177
+	  return $path;
178
+	}, $this->css_array);
179
+	$js_array = array_map(function ($script) {
180
+	  if (substr($script, 0, 4) == 'http') {
181
+		return $script;
182
+	  }
183
+	  return "/js/{$script}.min.js";
184
+	}, $this->js_array);
185 185
 		
186 186
 		$this->set_head('css_link_array', $css_array);
187 187
 		$this->set_head('js_link_array', $js_array);
Please login to merge, or discard this patch.
utility/content/MarkupCodeContent.class.inc.php 3 patches
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -25,6 +25,9 @@  discard block
 block discarded – undo
25 25
 		return;
26 26
 	}
27 27
 
28
+	/**
29
+	 * @param string $content
30
+	 */
28 31
 	private function wrap_in_list($content)
29 32
 	{
30 33
 		$content_array = explode("\n", $content);
@@ -44,6 +47,9 @@  discard block
 block discarded – undo
44 47
 		return $content;
45 48
 	}
46 49
 
50
+	/**
51
+	 * @param string $type
52
+	 */
47 53
 	private function highlight_code($content, $type)
48 54
 	{
49 55
 		switch($type)
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -10,10 +10,10 @@  discard block
 block discarded – undo
10 10
 	protected function execute($title = '')
11 11
 	{
12 12
 		preg_match_all(self::$MARKUP_DELIMITER, $this->content, $matches);
13
-		if(count($matches[1]) == 0)
13
+		if (count($matches[1]) == 0)
14 14
 			return;
15 15
 		
16
-		foreach($matches[3] as $key => $match)
16
+		foreach ($matches[3] as $key => $match)
17 17
 		{
18 18
 			$replacement = $match;
19 19
 			$replacement = $this->wrap_in_list($replacement);
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 	{
30 30
 		$content_array = explode("\n", $content);
31 31
 		
32
-		foreach($content_array as $key => $row)
32
+		foreach ($content_array as $key => $row)
33 33
 		{
34 34
 			$new_row = ($key % 2 == 0) ? '<li class="even">' : '<li class="odd">';
35 35
 			$new_row .= '<p>';
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 
47 47
 	private function highlight_code($content, $type)
48 48
 	{
49
-		switch($type)
49
+		switch ($type)
50 50
 		{
51 51
 			default :
52 52
 				return $content;
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -10,8 +10,9 @@
 block discarded – undo
10 10
 	protected function execute($title = '')
11 11
 	{
12 12
 		preg_match_all(self::$MARKUP_DELIMITER, $this->content, $matches);
13
-		if(count($matches[1]) == 0)
14
-			return;
13
+		if(count($matches[1]) == 0) {
14
+					return;
15
+		}
15 16
 		
16 17
 		foreach($matches[3] as $key => $match)
17 18
 		{
Please login to merge, or discard this patch.
utility/content/SmartTrimContent.class.inc.php 4 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -62,6 +62,9 @@
 block discarded – undo
62 62
 		$this->is_trimmed = true;
63 63
 	}
64 64
 
65
+	/**
66
+	 * @param string $content
67
+	 */
65 68
 	private function trim_html_string($content, $length)
66 69
 	{
67 70
 		$content = preg_replace(self::$PHOTO_PLACEHOLDER_MATCH, '', $content);
Please login to merge, or discard this patch.
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,8 +42,8 @@
 block discarded – undo
42 42
 		
43 43
 		if($length < strlen($this->content))
44 44
 			$this->trim_string($length);
45
-        else
46
-            $etc = '';
45
+		else
46
+			$etc = '';
47 47
 		$this->check_exclude_tags();
48 48
 		$this->close_tags($etc);
49 49
 	}
Please login to merge, or discard this patch.
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -27,20 +27,20 @@  discard block
 block discarded – undo
27 27
 	protected function execute()
28 28
 	{
29 29
 		$args = func_get_args();
30
-		if(count($args) < 1)
30
+		if (count($args) < 1)
31 31
 		{
32 32
 			trigger_error('Someone called SmartTrimContent w/o defining a length... bad!');
33 33
 			return;
34 34
 		}
35 35
 		
36
-		if(count($args) == 2)
36
+		if (count($args) == 2)
37 37
 			$etc = $args[1];
38 38
 		else
39 39
 			$etc = self::$ETC;
40 40
 		
41 41
 		$length = $args[0];
42 42
 		
43
-		if($length < strlen($this->content))
43
+		if ($length < strlen($this->content))
44 44
 			$this->trim_string($length);
45 45
         else
46 46
             $etc = '';
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 		
55 55
 		$last_right_bracket_position = strripos($content, self::$RIGHT_BRACKET);
56 56
 		$last_left_bracket_position = strripos($content, self::$LEFT_BRACKET);
57
-		if($last_left_bracket_position > $last_right_bracket_position)
57
+		if ($last_left_bracket_position > $last_right_bracket_position)
58 58
 			$content = substr($content, 0, $last_left_bracket_position);
59 59
 		$content = trim($content);
60 60
 		
@@ -73,16 +73,16 @@  discard block
 block discarded – undo
73 73
 		$content = current($content);
74 74
 		
75 75
 		$max_length = $length;
76
-		foreach($matches[0] as $match)
76
+		foreach ($matches[0] as $match)
77 77
 		{
78 78
 			$max_length += strlen($match[0]);
79
-			if($max_length <= $match[1])
79
+			if ($max_length <= $match[1])
80 80
 				break;
81 81
 			
82 82
 			$content = substr($content, 0, $match[1]) . $match[0] . substr($content, $match[1]);
83 83
 		}
84 84
 		
85
-		if(substr($content, -7) == '</p><p>')
85
+		if (substr($content, -7) == '</p><p>')
86 86
 			$content = substr($content, 0, -7);
87 87
 		
88 88
 		return $content;
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 		$tags_preg = $this->get_tags_preg(self::$EXCLUDE_TAGS);
95 95
 		preg_match_all($tags_preg, $content, $matches, PREG_OFFSET_CAPTURE);
96 96
 		
97
-		if(count($matches[1]) % 2 == 1)
97
+		if (count($matches[1]) % 2 == 1)
98 98
 		{
99 99
 			$cut_tag = end($matches[1]);
100 100
 			$cut_tag_position = $cut_tag[1] - 1;
@@ -112,9 +112,9 @@  discard block
 block discarded – undo
112 112
 		preg_match_all($tags_preg, $content, $matches);
113 113
 		$open_tags = array();
114 114
 		
115
-		foreach($matches[1] as $tag)
115
+		foreach ($matches[1] as $tag)
116 116
 		{
117
-			if(in_array($tag, $open_tags))
117
+			if (in_array($tag, $open_tags))
118 118
 			{
119 119
 				$key = array_search($tag, $open_tags);
120 120
 				unset($open_tags[$key]);
@@ -124,11 +124,11 @@  discard block
 block discarded – undo
124 124
 		}
125 125
 		
126 126
 		$open_tags = array_reverse($open_tags);
127
-		if(count($open_tags) > 0)
127
+		if (count($open_tags) > 0)
128 128
 		{
129
-			foreach($open_tags as $key => $open_tag)
129
+			foreach ($open_tags as $key => $open_tag)
130 130
 			{
131
-				if($key == count($open_tags) - 1)
131
+				if ($key == count($open_tags) - 1)
132 132
 					$content .= $etc;
133 133
 				$content .= "</{$open_tag}>";
134 134
 			}
Please login to merge, or discard this patch.
Braces   +28 added lines, -22 removed lines patch added patch discarded remove patch
@@ -33,17 +33,19 @@  discard block
 block discarded – undo
33 33
 			return;
34 34
 		}
35 35
 		
36
-		if(count($args) == 2)
37
-			$etc = $args[1];
38
-		else
39
-			$etc = self::$ETC;
36
+		if(count($args) == 2) {
37
+					$etc = $args[1];
38
+		} else {
39
+					$etc = self::$ETC;
40
+		}
40 41
 		
41 42
 		$length = $args[0];
42 43
 		
43
-		if($length < strlen($this->content))
44
-			$this->trim_string($length);
45
-        else
46
-            $etc = '';
44
+		if($length < strlen($this->content)) {
45
+					$this->trim_string($length);
46
+		} else {
47
+                    $etc = '';
48
+        }
47 49
 		$this->check_exclude_tags();
48 50
 		$this->close_tags($etc);
49 51
 	}
@@ -54,8 +56,9 @@  discard block
 block discarded – undo
54 56
 		
55 57
 		$last_right_bracket_position = strripos($content, self::$RIGHT_BRACKET);
56 58
 		$last_left_bracket_position = strripos($content, self::$LEFT_BRACKET);
57
-		if($last_left_bracket_position > $last_right_bracket_position)
58
-			$content = substr($content, 0, $last_left_bracket_position);
59
+		if($last_left_bracket_position > $last_right_bracket_position) {
60
+					$content = substr($content, 0, $last_left_bracket_position);
61
+		}
59 62
 		$content = trim($content);
60 63
 		
61 64
 		$this->content = $content;
@@ -76,14 +79,16 @@  discard block
 block discarded – undo
76 79
 		foreach($matches[0] as $match)
77 80
 		{
78 81
 			$max_length += strlen($match[0]);
79
-			if($max_length <= $match[1])
80
-				break;
82
+			if($max_length <= $match[1]) {
83
+							break;
84
+			}
81 85
 			
82 86
 			$content = substr($content, 0, $match[1]) . $match[0] . substr($content, $match[1]);
83 87
 		}
84 88
 		
85
-		if(substr($content, -7) == '</p><p>')
86
-			$content = substr($content, 0, -7);
89
+		if(substr($content, -7) == '</p><p>') {
90
+					$content = substr($content, 0, -7);
91
+		}
87 92
 		
88 93
 		return $content;
89 94
 	}
@@ -118,9 +123,9 @@  discard block
 block discarded – undo
118 123
 			{
119 124
 				$key = array_search($tag, $open_tags);
120 125
 				unset($open_tags[$key]);
121
-			}
122
-			else
123
-				$open_tags[] = $tag;
126
+			} else {
127
+							$open_tags[] = $tag;
128
+			}
124 129
 		}
125 130
 		
126 131
 		$open_tags = array_reverse($open_tags);
@@ -128,13 +133,14 @@  discard block
 block discarded – undo
128 133
 		{
129 134
 			foreach($open_tags as $key => $open_tag)
130 135
 			{
131
-				if($key == count($open_tags) - 1)
132
-					$content .= $etc;
136
+				if($key == count($open_tags) - 1) {
137
+									$content .= $etc;
138
+				}
133 139
 				$content .= "</{$open_tag}>";
134 140
 			}
135
-		}
136
-		else
137
-			$content .= $etc;
141
+		} else {
142
+					$content .= $etc;
143
+		}
138 144
 		
139 145
 		$this->content = $content;
140 146
 	}
Please login to merge, or discard this patch.