@@ -60,8 +60,9 @@ |
||
60 | 60 | { |
61 | 61 | $post_array[] = $this->format_post($post, true); |
62 | 62 | } |
63 | - if(count($post_array) < 1 && URLDecode::getPiece(1) !== 'search') |
|
64 | - $this->eject(); |
|
63 | + if(count($post_array) < 1 && URLDecode::getPiece(1) !== 'search') { |
|
64 | + $this->eject(); |
|
65 | + } |
|
65 | 66 | return $post_array; |
66 | 67 | } |
67 | 68 |
@@ -56,14 +56,16 @@ |
||
56 | 56 | { |
57 | 57 | $current_uri = URLDecode::getPiece(1); |
58 | 58 | |
59 | - if(!isset($current_uri)) |
|
60 | - return 'About'; |
|
59 | + if(!isset($current_uri)) { |
|
60 | + return 'About'; |
|
61 | + } |
|
61 | 62 | |
62 | 63 | $menu = self::$MENU; |
63 | 64 | foreach($menu as $link) |
64 | 65 | { |
65 | - if($link['uri'] == $current_uri) |
|
66 | - return $link['name']; |
|
66 | + if($link['uri'] == $current_uri) { |
|
67 | + return $link['name']; |
|
68 | + } |
|
67 | 69 | } |
68 | 70 | |
69 | 71 | return 'About'; |
@@ -29,15 +29,19 @@ |
||
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 | { |
@@ -39,10 +39,11 @@ discard block |
||
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 |
||
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 | } |
@@ -28,24 +28,26 @@ |
||
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(); |
@@ -36,8 +36,9 @@ |
||
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}"); |
@@ -26,16 +26,20 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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, |
@@ -36,8 +36,9 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -15,12 +15,14 @@ discard block |
||
15 | 15 | static private function form_url_array($host, $uri) |
16 | 16 | { |
17 | 17 | $uri = substr($uri, 1); |
18 | - if(strpos($uri, '?')) |
|
19 | - $uri = substr($uri, 0, strpos($uri, '?')); |
|
18 | + if(strpos($uri, '?')) { |
|
19 | + $uri = substr($uri, 0, strpos($uri, '?')); |
|
20 | + } |
|
20 | 21 | $uri_array = explode('/', $uri); |
21 | 22 | |
22 | - if(!Loader::isLive()) |
|
23 | - $host = substr($host, strpos($host, '.') + 1); |
|
23 | + if(!Loader::isLive()) { |
|
24 | + $host = substr($host, strpos($host, '.') + 1); |
|
25 | + } |
|
24 | 26 | |
25 | 27 | self::$array['host'] = $host; |
26 | 28 | |
@@ -33,8 +35,9 @@ discard block |
||
33 | 35 | self::$array['base'] = 'http://' . (!Loader::isLive() ? 'dev.' : '') . $host . '/'; |
34 | 36 | self::$array['uri'] = '/' . implode('/', $uri_array); |
35 | 37 | |
36 | - if(end($uri_array) == '') |
|
37 | - $uri_array = array_slice($uri_array, 0, count($uri_array) - 1); |
|
38 | + if(end($uri_array) == '') { |
|
39 | + $uri_array = array_slice($uri_array, 0, count($uri_array) - 1); |
|
40 | + } |
|
38 | 41 | self::$array['pieces'] = (array) $uri_array; |
39 | 42 | } |
40 | 43 | |
@@ -61,22 +64,26 @@ discard block |
||
61 | 64 | static function getExtension() |
62 | 65 | { |
63 | 66 | $file = self::getPiece(-1); |
64 | - if(substr($file, -1) == '/') |
|
65 | - return false; |
|
67 | + if(substr($file, -1) == '/') { |
|
68 | + return false; |
|
69 | + } |
|
66 | 70 | return substr($file, strrpos($file, '.') + 1);; |
67 | 71 | } |
68 | 72 | |
69 | 73 | static function getPiece($piece = null) |
70 | 74 | { |
71 | - if(!$piece) |
|
72 | - return self::$array['pieces']; |
|
75 | + if(!$piece) { |
|
76 | + return self::$array['pieces']; |
|
77 | + } |
|
73 | 78 | |
74 | - if($piece == -1) |
|
75 | - return end(self::$array['pieces']); |
|
79 | + if($piece == -1) { |
|
80 | + return end(self::$array['pieces']); |
|
81 | + } |
|
76 | 82 | |
77 | 83 | $piece = $piece - 1; |
78 | - if(array_key_exists($piece, self::$array['pieces'])) |
|
79 | - return self::$array['pieces'][$piece]; |
|
84 | + if(array_key_exists($piece, self::$array['pieces'])) { |
|
85 | + return self::$array['pieces'][$piece]; |
|
86 | + } |
|
80 | 87 | return; |
81 | 88 | } |
82 | 89 |