@@ -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 | |