Completed
Push — master ( 1cc257...44b189 )
by Jacob
03:32
created
router/WaterfallRouter.class.inc.php 1 patch
Indentation   +62 added lines, -62 removed lines patch added patch discarded remove patch
@@ -1,8 +1,8 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3 3
 Loader::load('collector', array(
4
-    'waterfall/LogCollector',
5
-    'waterfall/WaterfallCollector'));
4
+	'waterfall/LogCollector',
5
+	'waterfall/WaterfallCollector'));
6 6
 Loader::load('router', 'Router');
7 7
 
8 8
 class WaterfallRouter extends Router
@@ -26,15 +26,15 @@  discard block
 block discarded – undo
26 26
 			(object) array(
27 27
 				'pattern' => '@^/falls/results(/?)$@',
28 28
 				'replace' => '/falls/'),
29
-            (object) array(
30
-                'pattern' => '@^/log(/?)$@',
31
-                'replace' => '/journal/'),
32
-            (object) array(
33
-                'pattern' => '@^/log/(\d+)(/?)$@',
34
-                'replace' => '/journal/$1'),
35
-            (object) array(
36
-                'pattern' => '@^/map/([^/]+)(/?)$@',
37
-                'replace' => '/map/'),
29
+			(object) array(
30
+				'pattern' => '@^/log(/?)$@',
31
+				'replace' => '/journal/'),
32
+			(object) array(
33
+				'pattern' => '@^/log/(\d+)(/?)$@',
34
+				'replace' => '/journal/$1'),
35
+			(object) array(
36
+				'pattern' => '@^/map/([^/]+)(/?)$@',
37
+				'replace' => '/map/'),
38 38
 			(object) array(
39 39
 				'pattern' => '@^/about/([a-z]*)(/?)$@',
40 40
 				'replace' => '/about/'),
@@ -73,57 +73,57 @@  discard block
 block discarded – undo
73 73
 
74 74
 	protected function check_for_special_redirect($uri)
75 75
 	{
76
-        if (preg_match('@^/falls/([a-z\'-]+)(/?)$@', $uri, $matches)) {
77
-            $alias = $matches[1];
78
-            $alias = str_replace("'", '', $alias);
79
-            $alias .= '-falls';
80
-            $result = WaterfallCollector::getByOldAlias($alias);
81
-            if ($result !== null) {
82
-                return "/{$result->watercourse_alias}/{$result->alias}/";
83
-            } else {
84
-                Loader::loadNew('controller', '/Error404Controller')->activate();
85
-            }
86
-        }
76
+		if (preg_match('@^/falls/([a-z\'-]+)(/?)$@', $uri, $matches)) {
77
+			$alias = $matches[1];
78
+			$alias = str_replace("'", '', $alias);
79
+			$alias .= '-falls';
80
+			$result = WaterfallCollector::getByOldAlias($alias);
81
+			if ($result !== null) {
82
+				return "/{$result->watercourse_alias}/{$result->alias}/";
83
+			} else {
84
+				Loader::loadNew('controller', '/Error404Controller')->activate();
85
+			}
86
+		}
87 87
         
88
-        if (preg_match('@^/photos/([a-z\'-]+)-([^/]+)(/?)$@', $uri, $matches)) {
89
-            $alias = $matches[1];
90
-            $alias = explode('-', $alias);
91
-            array_pop($alias);
92
-            $alias = implode('-', $alias);
93
-            $alias = str_replace("'", '', $alias);
94
-            $alias .= '-falls';
95
-            $result = WaterfallCollector::getByOldAlias($alias);
96
-            if ($result !== null) {
97
-                return "/{$result->watercourse_alias}/{$result->alias}/";
98
-            } else {
99
-                Loader::loadNew('controller', '/Error404Controller')->activate();
100
-            }
101
-        }
88
+		if (preg_match('@^/photos/([a-z\'-]+)-([^/]+)(/?)$@', $uri, $matches)) {
89
+			$alias = $matches[1];
90
+			$alias = explode('-', $alias);
91
+			array_pop($alias);
92
+			$alias = implode('-', $alias);
93
+			$alias = str_replace("'", '', $alias);
94
+			$alias .= '-falls';
95
+			$result = WaterfallCollector::getByOldAlias($alias);
96
+			if ($result !== null) {
97
+				return "/{$result->watercourse_alias}/{$result->alias}/";
98
+			} else {
99
+				Loader::loadNew('controller', '/Error404Controller')->activate();
100
+			}
101
+		}
102 102
         
103
-        if (preg_match('@/log/([a-z]+-\d{2}-\d{4})(/?)$@', $uri, $matches)) {
104
-            $date = $matches[1];
105
-            $date = explode('-', $date);
106
-            $date = mktime(0, 0, 0, date('n', strtotime($date[0])), $date[1], $date[2]);
107
-            $date = date('Y-m-d', $date);
108
-            $result = LogCollector::getByDate($date);
109
-            if ($result !== null) {
110
-                return "/journal/{$result->alias}/";
111
-            } else {
112
-                Loader::loadNew('controller', '/Error404Controller')->activate();
113
-            }
114
-        }
103
+		if (preg_match('@/log/([a-z]+-\d{2}-\d{4})(/?)$@', $uri, $matches)) {
104
+			$date = $matches[1];
105
+			$date = explode('-', $date);
106
+			$date = mktime(0, 0, 0, date('n', strtotime($date[0])), $date[1], $date[2]);
107
+			$date = date('Y-m-d', $date);
108
+			$result = LogCollector::getByDate($date);
109
+			if ($result !== null) {
110
+				return "/journal/{$result->alias}/";
111
+			} else {
112
+				Loader::loadNew('controller', '/Error404Controller')->activate();
113
+			}
114
+		}
115 115
         
116
-        if (preg_match('@/map/([a-z\'-]+)(/?)$@', $uri, $matches)) {
117
-            $alias = $matches[1];
118
-            $alias = str_replace("'", '', $alias);
119
-            $alias .= '-falls';
120
-            $result = WaterfallCollector::getByOldAlias($alias);
121
-            if ($result !== null) {
122
-                return "/map/#{$result->watercourse_alias}/{$result->alias}";
123
-            } else {
124
-                Loader::loadNew('controller', '/Error404Controller')->activate();
125
-            }
126
-        }
116
+		if (preg_match('@/map/([a-z\'-]+)(/?)$@', $uri, $matches)) {
117
+			$alias = $matches[1];
118
+			$alias = str_replace("'", '', $alias);
119
+			$alias .= '-falls';
120
+			$result = WaterfallCollector::getByOldAlias($alias);
121
+			if ($result !== null) {
122
+				return "/map/#{$result->watercourse_alias}/{$result->alias}";
123
+			} else {
124
+				Loader::loadNew('controller', '/Error404Controller')->activate();
125
+			}
126
+		}
127 127
         
128 128
 		return $uri;
129 129
 	}
@@ -164,9 +164,9 @@  discard block
 block discarded – undo
164 164
 			(object) array(
165 165
 				'match' => '/map/',
166 166
 				'controller' => 'MapController'),
167
-            (object) array(
168
-                'match' => '/map/#[a-z-/]+',
169
-                'controller' => 'MapController'),
167
+			(object) array(
168
+				'match' => '/map/#[a-z-/]+',
169
+				'controller' => 'MapController'),
170 170
 			(object) array(
171 171
 				'match' => '/journal/',
172 172
 				'controller' => 'LogListController'),
Please login to merge, or discard this patch.
router/BlogRouter.class.inc.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -45,11 +45,11 @@  discard block
 block discarded – undo
45 45
 
46 46
 	protected function check_for_special_redirect($uri)
47 47
 	{
48
-		if(preg_match('@^/post_([0-9]{4}-[0-9]{2}-[0-9]{2})_([a-z0-9-]+)(/?)$@', $uri, $matches))
48
+		if (preg_match('@^/post_([0-9]{4}-[0-9]{2}-[0-9]{2})_([a-z0-9-]+)(/?)$@', $uri, $matches))
49 49
 		{
50 50
 			Loader::load('collector', 'blog/PostCollector');
51 51
 			$post = PostCollector::getPostByURI($matches[2]);
52
-			if(!$post)
52
+			if (!$post)
53 53
 			{
54 54
 				Loader::loadNew('controller', '/Error404Controller')
55 55
 					->activate();
@@ -61,21 +61,21 @@  discard block
 block discarded – undo
61 61
 		else
62 62
 		{
63 63
 			$post_uri = URLDecode::getPiece(1);
64
-			if($post_uri !== null)
64
+			if ($post_uri !== null)
65 65
 			{
66 66
 				Loader::load('collector', 'blog/PostCollector');
67 67
 				$post = PostCollector::getPostByURI($post_uri);
68 68
 				
69
-				if($post != false)
69
+				if ($post != false)
70 70
 				{
71 71
 					Loader::load('utility', 'Content');
72 72
 					$uri = Content::instance('URLSafe', "/{$post->category}/{$post->title_url}/")->activate();
73 73
 				}
74 74
 			}
75 75
 		}
76
-		if($uri == '/search/')
76
+		if ($uri == '/search/')
77 77
 		{
78
-			if(Request::getGet('submit') == 'Submit Search' && Request::getGet('search'))
78
+			if (Request::getGet('submit') == 'Submit Search' && Request::getGet('search'))
79 79
 			{
80 80
 				$uri .= Request::getGet('search');
81 81
 				$uri .= '/';
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,8 +57,7 @@
 block discarded – undo
57 57
 			
58 58
 			Loader::load('utility', 'Content');
59 59
 			$uri = Content::instance('URLSafe', "/{$post->category}/{$post->title_url}/")->activate();
60
-		}
61
-		else
60
+		} else
62 61
 		{
63 62
 			$post_uri = URLDecode::getPiece(1);
64 63
 			if($post_uri !== null)
Please login to merge, or discard this patch.
router/PortfolioRouter.class.inc.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,12 +27,12 @@
 block discarded – undo
27 27
 
28 28
 	protected function check_for_special_redirect($uri)
29 29
 	{
30
-		if(preg_match('@^/piece/([a-z0-9-]+)(/?)$@', $uri, $matches))
30
+		if (preg_match('@^/piece/([a-z0-9-]+)(/?)$@', $uri, $matches))
31 31
 		{
32 32
 			Loader::load('collector', 'portfolio/PortfolioCollector');
33 33
 			$piece = PortfolioCollector::getPieceByURI($matches[1]);
34 34
 			
35
-			if($piece === null)
35
+			if ($piece === null)
36 36
 			{
37 37
 				Loader::loadNew('controller', '/Error404Controller')
38 38
 					->activate();
Please login to merge, or discard this patch.
router/Router.class.inc.php 3 patches
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -96,9 +96,9 @@  discard block
 block discarded – undo
96 96
 		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
97 97
 			$redirect_uri .= '/';
98 98
 		
99
-        if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
100
-            $redirect_uri = 'http://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
101
-        }
99
+		if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
100
+			$redirect_uri = 'http://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
101
+		}
102 102
         
103 103
 		if($redirect_uri == URLDecode::getURI())
104 104
 			return;
@@ -129,8 +129,8 @@  discard block
 block discarded – undo
129 129
 			}
130 130
 
131 131
 			$image = new ImageOld($file);
132
-        	        if(!$image->isValid()) {
133
-                	        Loader::loadNew('controller', '/Error404Controller')->activate();
132
+					if(!$image->isValid()) {
133
+							Loader::loadNew('controller', '/Error404Controller')->activate();
134 134
 				exit();
135 135
 			}
136 136
 		}
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
 			URLDecode::getExtension() != 'jpg' &&
202 202
 			URLDecode::getExtension() != 'png' &&
203 203
 			URLDecode::getExtension() != 'ico' &&
204
-            strstr(URLDecode::getURI(), '#') === false);
204
+			strstr(URLDecode::getURI(), '#') === false);
205 205
 	}
206 206
 
207 207
 }
Please login to merge, or discard this patch.
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -19,18 +19,18 @@  discard block
 block discarded – undo
19 19
 
20 20
 	private static function get_router_name()
21 21
 	{
22
-		if(Request::isAJAX())
22
+		if (Request::isAJAX())
23 23
 			return 'AJAXRouter';
24
-		if(URLDecode::getURI() == '/sitemap.xml')
24
+		if (URLDecode::getURI() == '/sitemap.xml')
25 25
 			return 'SitemapRouter';
26
-		if(URLDecode::getURI() == '/rss/')
26
+		if (URLDecode::getURI() == '/rss/')
27 27
 			return 'RSSRouter';
28
-		if(URLDecode::getExtension() == 'jpg')
28
+		if (URLDecode::getExtension() == 'jpg')
29 29
 			return 'ImageRouter';
30
-		if(URLDecode::getExtension() == 'png')
30
+		if (URLDecode::getExtension() == 'png')
31 31
 			return 'ImageRouter';
32 32
 		
33
-		switch(URLDecode::getSite())
33
+		switch (URLDecode::getSite())
34 34
 		{
35 35
 			case 'ajax' :
36 36
 				return 'AjaxRouter';
@@ -78,37 +78,37 @@  discard block
 block discarded – undo
78 78
 
79 79
 	final protected function check_for_redirect($redirect_uri)
80 80
 	{
81
-		foreach($this->get_redirect_array() as $check)
81
+		foreach ($this->get_redirect_array() as $check)
82 82
 		{
83 83
 			$redirect_uri = preg_replace($check->pattern, $check->replace, $redirect_uri);
84 84
 		}
85 85
 		
86 86
 		$redirect_uri = $this->check_for_special_redirect($redirect_uri);
87 87
 		
88
-		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
88
+		if ($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
89 89
 			$redirect_uri .= '/';
90 90
 		
91 91
         if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
92 92
             $redirect_uri = 'http://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
93 93
         }
94 94
         
95
-		if($redirect_uri == URLDecode::getURI())
95
+		if ($redirect_uri == URLDecode::getURI())
96 96
 			return;
97 97
 		
98 98
 		$controller_check = $redirect_uri;
99
-		if(substr($redirect_uri, 0, 4) == 'http')
99
+		if (substr($redirect_uri, 0, 4) == 'http')
100 100
 			$controller_check = preg_replace('@^http://([a-z\.]+)@', '', $redirect_uri);
101 101
 		
102 102
 		$controller = $this->get_controller($controller_check);
103
-		if($controller == '/Error404Controller')
103
+		if ($controller == '/Error404Controller')
104 104
 		{
105 105
 			Loader::loadNew('controller', '/Error404Controller')
106 106
 				->activate();
107 107
 			exit;
108 108
 		}
109 109
 
110
-		if($this->get_primary_folder() == 'images') {
111
-			$file = $controller_check;//URLDecode::getURI();
110
+		if ($this->get_primary_folder() == 'images') {
111
+			$file = $controller_check; //URLDecode::getURI();
112 112
 
113 113
 			if (
114 114
 				URLDecode::getSite() == 'images' ||
@@ -121,13 +121,13 @@  discard block
 block discarded – undo
121 121
 			}
122 122
 
123 123
 			$image = new ImageOld($file);
124
-        	        if(!$image->isValid()) {
124
+        	        if (!$image->isValid()) {
125 125
                 	        Loader::loadNew('controller', '/Error404Controller')->activate();
126 126
 				exit();
127 127
 			}
128 128
 		}
129 129
 		
130
-		if(substr($redirect_uri, 0, 4) != 'http')
130
+		if (substr($redirect_uri, 0, 4) != 'http')
131 131
 		{
132 132
 			$redirect_uri = substr($redirect_uri, 1);
133 133
 			$redirect_uri = URLDecode::getBase() . $redirect_uri;
@@ -144,12 +144,12 @@  discard block
 block discarded – undo
144 144
 
145 145
 	final private function get_controller($uri)
146 146
 	{
147
-		foreach($this->get_direct_array() as $check)
147
+		foreach ($this->get_direct_array() as $check)
148 148
 		{
149
-			if($uri == $check->match)
149
+			if ($uri == $check->match)
150 150
 				return "{$this->get_primary_folder()}/{$check->controller}";
151 151
 			
152
-			if(preg_match("@^{$check->match}$@", $uri))
152
+			if (preg_match("@^{$check->match}$@", $uri))
153 153
 				return "{$this->get_primary_folder()}/{$check->controller}";
154 154
 		}
155 155
 		
@@ -158,13 +158,13 @@  discard block
 block discarded – undo
158 158
 
159 159
 	final private function get_primary_folder()
160 160
 	{
161
-		if(Request::isAjax())
161
+		if (Request::isAjax())
162 162
 			return 'ajax';
163
-		if(URLDecode::getURI() == '/sitemap.xml')
163
+		if (URLDecode::getURI() == '/sitemap.xml')
164 164
 			return 'sitemap';
165
-		if(URLDecode::getURI() == '/rss/')
165
+		if (URLDecode::getURI() == '/rss/')
166 166
 			return 'rss';
167
-		if(
167
+		if (
168 168
 			URLDecode::getExtension() == 'jpg' ||
169 169
 			URLDecode::getExtension() == 'png')
170 170
 			return 'images';
Please login to merge, or discard this patch.
Braces   +42 added lines, -28 removed lines patch added patch discarded remove patch
@@ -19,16 +19,21 @@  discard block
 block discarded – undo
19 19
 
20 20
 	private static function get_router_name()
21 21
 	{
22
-		if(Request::isAJAX())
23
-			return 'AJAXRouter';
24
-		if(URLDecode::getURI() == '/sitemap.xml')
25
-			return 'SitemapRouter';
26
-		if(URLDecode::getURI() == '/rss/')
27
-			return 'RSSRouter';
28
-		if(URLDecode::getExtension() == 'jpg')
29
-			return 'ImageRouter';
30
-		if(URLDecode::getExtension() == 'png')
31
-			return 'ImageRouter';
22
+		if(Request::isAJAX()) {
23
+					return 'AJAXRouter';
24
+		}
25
+		if(URLDecode::getURI() == '/sitemap.xml') {
26
+					return 'SitemapRouter';
27
+		}
28
+		if(URLDecode::getURI() == '/rss/') {
29
+					return 'RSSRouter';
30
+		}
31
+		if(URLDecode::getExtension() == 'jpg') {
32
+					return 'ImageRouter';
33
+		}
34
+		if(URLDecode::getExtension() == 'png') {
35
+					return 'ImageRouter';
36
+		}
32 37
 		
33 38
 		switch(URLDecode::getSite())
34 39
 		{
@@ -85,19 +90,22 @@  discard block
 block discarded – undo
85 90
 		
86 91
 		$redirect_uri = $this->check_for_special_redirect($redirect_uri);
87 92
 		
88
-		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
89
-			$redirect_uri .= '/';
93
+		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/') {
94
+					$redirect_uri .= '/';
95
+		}
90 96
 		
91 97
         if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
92 98
             $redirect_uri = 'http://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
93 99
         }
94 100
         
95
-		if($redirect_uri == URLDecode::getURI())
96
-			return;
101
+		if($redirect_uri == URLDecode::getURI()) {
102
+					return;
103
+		}
97 104
 		
98 105
 		$controller_check = $redirect_uri;
99
-		if(substr($redirect_uri, 0, 4) == 'http')
100
-			$controller_check = preg_replace('@^http://([a-z\.]+)@', '', $redirect_uri);
106
+		if(substr($redirect_uri, 0, 4) == 'http') {
107
+					$controller_check = preg_replace('@^http://([a-z\.]+)@', '', $redirect_uri);
108
+		}
101 109
 		
102 110
 		$controller = $this->get_controller($controller_check);
103 111
 		if($controller == '/Error404Controller')
@@ -146,11 +154,13 @@  discard block
 block discarded – undo
146 154
 	{
147 155
 		foreach($this->get_direct_array() as $check)
148 156
 		{
149
-			if($uri == $check->match)
150
-				return "{$this->get_primary_folder()}/{$check->controller}";
157
+			if($uri == $check->match) {
158
+							return "{$this->get_primary_folder()}/{$check->controller}";
159
+			}
151 160
 			
152
-			if(preg_match("@^{$check->match}$@", $uri))
153
-				return "{$this->get_primary_folder()}/{$check->controller}";
161
+			if(preg_match("@^{$check->match}$@", $uri)) {
162
+							return "{$this->get_primary_folder()}/{$check->controller}";
163
+			}
154 164
 		}
155 165
 		
156 166
 		return '/Error404Controller';
@@ -158,16 +168,20 @@  discard block
 block discarded – undo
158 168
 
159 169
 	final private function get_primary_folder()
160 170
 	{
161
-		if(Request::isAjax())
162
-			return 'ajax';
163
-		if(URLDecode::getURI() == '/sitemap.xml')
164
-			return 'sitemap';
165
-		if(URLDecode::getURI() == '/rss/')
166
-			return 'rss';
171
+		if(Request::isAjax()) {
172
+					return 'ajax';
173
+		}
174
+		if(URLDecode::getURI() == '/sitemap.xml') {
175
+					return 'sitemap';
176
+		}
177
+		if(URLDecode::getURI() == '/rss/') {
178
+					return 'rss';
179
+		}
167 180
 		if(
168 181
 			URLDecode::getExtension() == 'jpg' ||
169
-			URLDecode::getExtension() == 'png')
170
-			return 'images';
182
+			URLDecode::getExtension() == 'png') {
183
+					return 'images';
184
+		}
171 185
 		
172 186
 		return URLDecode::getSite();
173 187
 	}
Please login to merge, or discard this patch.
router/SitemapRouter.class.inc.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -16,9 +16,9 @@
 block discarded – undo
16 16
 	protected function get_direct_array()
17 17
 	{
18 18
 		$site = URLDecode::getSite();
19
-        if ($site == 'waterfalls') {
20
-            $site = 'waterfall';
21
-        }
19
+		if ($site == 'waterfalls') {
20
+			$site = 'waterfall';
21
+		}
22 22
 		$site = ucwords($site);
23 23
 		
24 24
 		return array(
Please login to merge, or discard this patch.
collector/waterfall/WatercourseCollector.class.inc.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -153,9 +153,9 @@
 block discarded – undo
153 153
 		return self::run_query($query);
154 154
 	}
155 155
 
156
-    public static function getParentWatercourse($watercourse)
157
-    {
158
-    }
156
+	public static function getParentWatercourse($watercourse)
157
+	{
158
+	}
159 159
 
160 160
 	public static function getLogCountForWatercourse($watercourse)
161 161
 	{
Please login to merge, or discard this patch.
controller/AJAXController.class.inc.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
 
29 29
 	protected function set_response($message, $type = 'internal')
30 30
 	{
31
-		switch($type)
31
+		switch ($type)
32 32
 		{
33 33
 			case 'internal' :
34 34
 				$this->response['internal'] = $message;
Please login to merge, or discard this patch.
controller/rss/BlogPostRSSController.class.inc.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
 	{
27 27
 		$post_result = PostCollector::getMainList(250);
28 28
 		
29
-		foreach($post_result as $post)
29
+		foreach ($post_result as $post)
30 30
 		{
31 31
 			$this->addItem(
32 32
 				$post->title,
Please login to merge, or discard this patch.
controller/lifestream/TagController.class.inc.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 
47 47
 	private function get_title()
48 48
 	{
49
-		switch($this->tag)
49
+		switch ($this->tag)
50 50
 		{
51 51
 			case 'blog' :
52 52
 				return 'Jacob has a Blog';
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 
72 72
 	private function get_description()
73 73
 	{
74
-		switch($this->tag)
74
+		switch ($this->tag)
75 75
 		{
76 76
 			case 'blog' :
77 77
 				return 'Yeah, Jacob has a blog. Check out his posting activity.';
@@ -97,7 +97,7 @@  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))
100
+		if (isset($page) && is_numeric($page))
101 101
 			return $page;
102 102
 		return 1;
103 103
 	}
@@ -117,16 +117,16 @@  discard block
 block discarded – undo
117 117
 
118 118
 	protected function get_list_next_link()
119 119
 	{
120
-		if($this->page == 1)
120
+		if ($this->page == 1)
121 121
 			return;
122
-		if($this->page == 2)
122
+		if ($this->page == 2)
123 123
 			return Loader::getRootUrl('lifestream') . $this->tag . '/';
124 124
 		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page - 1) . '/';
125 125
 	}
126 126
 
127 127
 	protected function get_list_prev_link()
128 128
 	{
129
-		if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
129
+		if (($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count())
130 130
 			return;
131 131
 		return Loader::getRootUrl('lifestream') . $this->tag . '/page/' . ($this->page + 1) . '/';
132 132
 	}
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 	private $total_post_count;
135 135
 	protected function get_total_post_count()
136 136
 	{
137
-		if(!isset($this->total_post_count))
137
+		if (!isset($this->total_post_count))
138 138
 			$this->total_post_count = ActivityCollector::getCountForTag($this->tag);
139 139
 		return $this->total_post_count;
140 140
 	}
Please login to merge, or discard this 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->postRepository->getPostsByTypeCount($this->tag);
141
+		if(!isset($this->total_post_count)) {
142
+					$this->total_post_count = $this->postRepository->getPostsByTypeCount($this->tag);
143
+		}
139 144
 		return $this->total_post_count;
140 145
 	}
141 146
 
Please login to merge, or discard this patch.