@@ -27,8 +27,9 @@ |
||
| 27 | 27 | protected function get_page_number() |
| 28 | 28 | { |
| 29 | 29 | $page = URLDecode::getPiece(2); |
| 30 | - if(isset($page) && is_numeric($page)) |
|
| 31 | - return $page; |
|
| 30 | + if(isset($page) && is_numeric($page)) {
|
|
| 31 | + return $page; |
|
| 32 | + } |
|
| 32 | 33 | return 1; |
| 33 | 34 | } |
| 34 | 35 | |
@@ -33,17 +33,21 @@ |
||
| 33 | 33 | |
| 34 | 34 | private function process_form() |
| 35 | 35 | { |
| 36 | - if(!Request::hasPost() || Request::getPost('submit') != 'Send Message!') |
|
| 37 | - return (object) array('display' => 'normal'); |
|
| 36 | + if(!Request::hasPost() || Request::getPost('submit') != 'Send Message!') {
|
|
| 37 | + return (object) array('display' => 'normal');
|
|
| 38 | + } |
|
| 38 | 39 | |
| 39 | 40 | Loader::load('utility', 'Validate'); |
| 40 | 41 | $error_result = array(); |
| 41 | - if(!Validate::checkRequest('post', 'name', 'string')) |
|
| 42 | - $error_result['name'] = 'please enter your name'; |
|
| 43 | - if(!Validate::checkRequest('post', 'email', 'string')) |
|
| 44 | - $error_result['email'] = 'please enter a valid email'; |
|
| 45 | - if(!Validate::checkRequest('post', 'message', 'string')) |
|
| 46 | - $error_result['message'] = 'please write a message'; |
|
| 42 | + if(!Validate::checkRequest('post', 'name', 'string')) {
|
|
| 43 | + $error_result['name'] = 'please enter your name'; |
|
| 44 | + } |
|
| 45 | + if(!Validate::checkRequest('post', 'email', 'string')) {
|
|
| 46 | + $error_result['email'] = 'please enter a valid email'; |
|
| 47 | + } |
|
| 48 | + if(!Validate::checkRequest('post', 'message', 'string')) {
|
|
| 49 | + $error_result['message'] = 'please write a message'; |
|
| 50 | + } |
|
| 47 | 51 | |
| 48 | 52 | $values = (object) array( |
| 49 | 53 | 'name' => Request::getPost('name'), |
@@ -27,8 +27,9 @@ |
||
| 27 | 27 | $alias = URLDecode::getPiece(3); |
| 28 | 28 | $this->tag = LogTagCollector::getTag($alias); |
| 29 | 29 | |
| 30 | - if(!$this->tag) |
|
| 31 | - $this->eject(); |
|
| 30 | + if(!$this->tag) {
|
|
| 31 | + $this->eject(); |
|
| 32 | + } |
|
| 32 | 33 | |
| 33 | 34 | parent::__construct(); |
| 34 | 35 | } |
@@ -29,8 +29,9 @@ |
||
| 29 | 29 | $path = URLDecode::getPiece(1); |
| 30 | 30 | $this->watercourse = WatercourseCollector::getByAlias($path); |
| 31 | 31 | |
| 32 | - if(!$this->watercourse) |
|
| 33 | - $this->eject(); |
|
| 32 | + if(!$this->watercourse) {
|
|
| 33 | + $this->eject(); |
|
| 34 | + } |
|
| 34 | 35 | |
| 35 | 36 | parent::__construct(); |
| 36 | 37 | } |
@@ -20,15 +20,17 @@ discard block |
||
| 20 | 20 | $this->offset = ($this->page - 1) * $this->get_item_count_per_page(); |
| 21 | 21 | |
| 22 | 22 | $items = $this->get_items(); |
| 23 | - if(count($items) < 1) |
|
| 24 | - $this->eject(); |
|
| 23 | + if(count($items) < 1) {
|
|
| 24 | + $this->eject(); |
|
| 25 | + } |
|
| 25 | 26 | } |
| 26 | 27 | |
| 27 | 28 | final protected function get_page_number() |
| 28 | 29 | { |
| 29 | 30 | $page = $this->get_page_number_piece(); |
| 30 | - if(isset($page) && is_numeric($page)) |
|
| 31 | - return $page; |
|
| 31 | + if(isset($page) && is_numeric($page)) {
|
|
| 32 | + return $page; |
|
| 33 | + } |
|
| 32 | 34 | return 1; |
| 33 | 35 | } |
| 34 | 36 | |
@@ -41,8 +43,9 @@ discard block |
||
| 41 | 43 | |
| 42 | 44 | final private function get_introduction() |
| 43 | 45 | { |
| 44 | - if($this->page == 1) |
|
| 45 | - return $this->get_initial_introduction(); |
|
| 46 | + if($this->page == 1) {
|
|
| 47 | + return $this->get_initial_introduction(); |
|
| 48 | + } |
|
| 46 | 49 | |
| 47 | 50 | return $this->get_subsequent_introduction(); |
| 48 | 51 | } |
@@ -68,8 +71,9 @@ discard block |
||
| 68 | 71 | private $item_count; |
| 69 | 72 | final private function get_item_count() |
| 70 | 73 | { |
| 71 | - if(!isset($this->item_count)) |
|
| 72 | - $this->item_count = $this->get_item_count_result(); |
|
| 74 | + if(!isset($this->item_count)) {
|
|
| 75 | + $this->item_count = $this->get_item_count_result(); |
|
| 76 | + } |
|
| 73 | 77 | return $this->item_count; |
| 74 | 78 | } |
| 75 | 79 | |
@@ -82,10 +86,11 @@ discard block |
||
| 82 | 86 | $this->set_head('next_link', $this->get_list_next_link()); |
| 83 | 87 | $this->set_head('previous_link', $this->get_list_prev_link()); |
| 84 | 88 | |
| 85 | - if($this->page == 1) |
|
| 86 | - $meta_array = $this->get_initial_meta(); |
|
| 87 | - else |
|
| 88 | - $meta_array = $this->get_subsequent_meta(); |
|
| 89 | + if($this->page == 1) {
|
|
| 90 | + $meta_array = $this->get_initial_meta(); |
|
| 91 | + } else {
|
|
| 92 | + $meta_array = $this->get_subsequent_meta(); |
|
| 93 | + } |
|
| 89 | 94 | |
| 90 | 95 | $this->set_title($meta_array['title']); |
| 91 | 96 | $this->set_description($meta_array['description']); |
@@ -119,17 +124,20 @@ discard block |
||
| 119 | 124 | |
| 120 | 125 | final private function get_list_prev_link() |
| 121 | 126 | { |
| 122 | - if($this->page == 1) |
|
| 123 | - return; |
|
| 124 | - if($this->page == 2) |
|
| 125 | - return $this->get_list_link_root(); |
|
| 127 | + if($this->page == 1) {
|
|
| 128 | + return; |
|
| 129 | + } |
|
| 130 | + if($this->page == 2) {
|
|
| 131 | + return $this->get_list_link_root(); |
|
| 132 | + } |
|
| 126 | 133 | return $this->get_list_link_root() . ($this->page - 1) . '/'; |
| 127 | 134 | } |
| 128 | 135 | |
| 129 | 136 | final private function get_list_next_link() |
| 130 | 137 | { |
| 131 | - if(($this->page * $this->get_item_count_per_page()) >= $this->get_item_count()) |
|
| 132 | - return; |
|
| 138 | + if(($this->page * $this->get_item_count_per_page()) >= $this->get_item_count()) {
|
|
| 139 | + return; |
|
| 140 | + } |
|
| 133 | 141 | return $this->get_list_link_root() . ($this->page + 1) . '/'; |
| 134 | 142 | } |
| 135 | 143 | |
@@ -139,15 +147,17 @@ discard block |
||
| 139 | 147 | { |
| 140 | 148 | $link_array = array(); |
| 141 | 149 | |
| 142 | - if($this->get_item_count_per_page() >= $this->get_item_count()) |
|
| 143 | - return $link_array; |
|
| 150 | + if($this->get_item_count_per_page() >= $this->get_item_count()) {
|
|
| 151 | + return $link_array; |
|
| 152 | + } |
|
| 144 | 153 | |
| 145 | 154 | $link = new stdclass(); |
| 146 | 155 | $link->anchor = '« previous'; |
| 147 | - if($this->get_list_prev_link() !== null) |
|
| 148 | - $link->uri = $this->get_list_prev_link(); |
|
| 149 | - else |
|
| 150 | - $link->class = 'inactive'; |
|
| 156 | + if($this->get_list_prev_link() !== null) {
|
|
| 157 | + $link->uri = $this->get_list_prev_link(); |
|
| 158 | + } else {
|
|
| 159 | + $link->class = 'inactive'; |
|
| 160 | + } |
|
| 151 | 161 | $link_array[] = $link; |
| 152 | 162 | |
| 153 | 163 | for($i = 1; $i <= ceil($this->get_item_count() / $this->get_item_count_per_page()); $i++) |
@@ -155,14 +165,15 @@ discard block |
||
| 155 | 165 | $link = new stdclass(); |
| 156 | 166 | $link->anchor = $i; |
| 157 | 167 | |
| 158 | - if($i == $this->page) |
|
| 159 | - $link->class = 'current'; |
|
| 160 | - else |
|
| 168 | + if($i == $this->page) {
|
|
| 169 | + $link->class = 'current'; |
|
| 170 | + } else |
|
| 161 | 171 | { |
| 162 | - if($i == 1) |
|
| 163 | - $link->uri = $this->get_list_link_root(); |
|
| 164 | - else |
|
| 165 | - $link->uri = $this->get_list_link_root() . $i . '/'; |
|
| 172 | + if($i == 1) {
|
|
| 173 | + $link->uri = $this->get_list_link_root(); |
|
| 174 | + } else {
|
|
| 175 | + $link->uri = $this->get_list_link_root() . $i . '/'; |
|
| 176 | + } |
|
| 166 | 177 | } |
| 167 | 178 | |
| 168 | 179 | $link_array[] = $link; |
@@ -170,10 +181,11 @@ discard block |
||
| 170 | 181 | |
| 171 | 182 | $link = new stdclass(); |
| 172 | 183 | $link->anchor = 'next »'; |
| 173 | - if($this->get_list_next_link() !== null) |
|
| 174 | - $link->uri = $this->get_list_next_link(); |
|
| 175 | - else |
|
| 176 | - $link->class = 'inactive'; |
|
| 184 | + if($this->get_list_next_link() !== null) {
|
|
| 185 | + $link->uri = $this->get_list_next_link(); |
|
| 186 | + } else {
|
|
| 187 | + $link->class = 'inactive'; |
|
| 188 | + } |
|
| 177 | 189 | $link_array[] = $link; |
| 178 | 190 | |
| 179 | 191 | return $link_array; |
@@ -27,8 +27,9 @@ |
||
| 27 | 27 | $alias = URLDecode::getPiece(2); |
| 28 | 28 | $this->companion = CompanionCollector::getByAlias($alias); |
| 29 | 29 | |
| 30 | - if(!$this->companion) |
|
| 31 | - $this->eject(); |
|
| 30 | + if(!$this->companion) {
|
|
| 31 | + $this->eject(); |
|
| 32 | + } |
|
| 32 | 33 | |
| 33 | 34 | parent::__construct(); |
| 34 | 35 | } |
@@ -25,8 +25,9 @@ discard block |
||
| 25 | 25 | |
| 26 | 26 | final protected function setLink($url = null) |
| 27 | 27 | { |
| 28 | - if(!isset($url)) |
|
| 29 | - $url = URLDecode::getBase(); |
|
| 28 | + if(!isset($url)) {
|
|
| 29 | + $url = URLDecode::getBase(); |
|
| 30 | + } |
|
| 30 | 31 | $this->xml->channel->addChild('link', $url); |
| 31 | 32 | } |
| 32 | 33 | |
@@ -50,8 +51,9 @@ discard block |
||
| 50 | 51 | |
| 51 | 52 | final protected function setCopyright($copyright = null) |
| 52 | 53 | { |
| 53 | - if(!isset($copyright)) |
|
| 54 | - $copyright = 'Copyright ' . date('Y') . ' Jacob Emerick'; |
|
| 54 | + if(!isset($copyright)) {
|
|
| 55 | + $copyright = 'Copyright ' . date('Y') . ' Jacob Emerick';
|
|
| 56 | + } |
|
| 55 | 57 | $this->xml->channel->addChild('copyright', $copyright); |
| 56 | 58 | } |
| 57 | 59 | |
@@ -66,8 +68,9 @@ discard block |
||
| 66 | 68 | |
| 67 | 69 | final protected function setPubDate($date = null) |
| 68 | 70 | { |
| 69 | - if(!isset($date)) |
|
| 70 | - $date = date('r'); |
|
| 71 | + if(!isset($date)) {
|
|
| 72 | + $date = date('r');
|
|
| 73 | + } |
|
| 71 | 74 | $this->xml->channel->addChild('pubDate', $date); |
| 72 | 75 | } |
| 73 | 76 | |
@@ -27,8 +27,9 @@ discard block |
||
| 27 | 27 | parent::__construct(); |
| 28 | 28 | |
| 29 | 29 | $this->post = PostCollector::getPostByURI(URLDecode::getPiece(2)); |
| 30 | - if($this->post == null) |
|
| 31 | - $this->eject(); |
|
| 30 | + if($this->post == null) {
|
|
| 31 | + $this->eject(); |
|
| 32 | + } |
|
| 32 | 33 | |
| 33 | 34 | $this->handle_comment_submit( |
| 34 | 35 | self::$BLOG_SITE_ID, |
@@ -103,8 +104,9 @@ discard block |
||
| 103 | 104 | private function get_series_posts() |
| 104 | 105 | { |
| 105 | 106 | $series_posts = $this->fetch_series_posts(); |
| 106 | - if(count($series_posts) < 1) |
|
| 107 | - return array(); |
|
| 107 | + if(count($series_posts) < 1) {
|
|
| 108 | + return array(); |
|
| 109 | + } |
|
| 108 | 110 | |
| 109 | 111 | $previous_post = new stdclass(); |
| 110 | 112 | $next_post = new stdclass(); |
@@ -132,9 +134,9 @@ discard block |
||
| 132 | 134 | |
| 133 | 135 | $post->url = Loader::getRootUrl('blog') . "{$post_row->category}/{$post_row->path}/"; |
| 134 | 136 | |
| 135 | - if(!$found_current_post) |
|
| 136 | - $previous_post = $post; |
|
| 137 | - else |
|
| 137 | + if(!$found_current_post) {
|
|
| 138 | + $previous_post = $post; |
|
| 139 | + } else |
|
| 138 | 140 | { |
| 139 | 141 | $next_post = $post; |
| 140 | 142 | break; |
@@ -151,8 +153,9 @@ discard block |
||
| 151 | 153 | private $series_posts; |
| 152 | 154 | private function fetch_series_posts() |
| 153 | 155 | { |
| 154 | - if(!isset($this->series_posts)) |
|
| 155 | - $this->series_posts = SeriesCollector::getSeriesForPost($this->post->id); |
|
| 156 | + if(!isset($this->series_posts)) {
|
|
| 157 | + $this->series_posts = SeriesCollector::getSeriesForPost($this->post->id); |
|
| 158 | + } |
|
| 156 | 159 | return $this->series_posts; |
| 157 | 160 | } |
| 158 | 161 | |
@@ -49,8 +49,7 @@ discard block |
||
| 49 | 49 | { |
| 50 | 50 | $this->set_title(sprintf(self::$TITLE_MAIN, ucwords($this->query))); |
| 51 | 51 | $this->set_description(sprintf(self::$DESCRIPTION_MAIN, ucwords($this->query))); |
| 52 | - } |
|
| 53 | - else |
|
| 52 | + } else |
|
| 54 | 53 | { |
| 55 | 54 | $this->set_title(sprintf(self::$TITLE_PAGINATED, ucwords($this->query), $this->page, $this->total_pages)); |
| 56 | 55 | $this->set_description(sprintf(self::$DESCRIPTION_PAGINATED, $this->page, $this->total_pages, ucwords($this->query))); |
@@ -63,22 +62,24 @@ discard block |
||
| 63 | 62 | |
| 64 | 63 | protected function get_introduction() |
| 65 | 64 | { |
| 66 | - if($this->total_pages > 1) |
|
| 67 | - return array( |
|
| 68 | - 'title' => "Posts from search '{$this->query}', page {$this->page} of {$this->total_pages}."); |
|
| 69 | - else if($this->total_pages == 1) |
|
| 70 | - return array( |
|
| 71 | - 'title' => "Posts from search '{$this->query}'."); |
|
| 72 | - else |
|
| 73 | - return array( |
|
| 74 | - 'title' => "Sorry, '{$this->query}' didn't return any posts."); |
|
| 65 | + if($this->total_pages > 1) {
|
|
| 66 | + return array( |
|
| 67 | + 'title' => "Posts from search '{$this->query}', page {$this->page} of {$this->total_pages}.");
|
|
| 68 | + } else if($this->total_pages == 1) {
|
|
| 69 | + return array( |
|
| 70 | + 'title' => "Posts from search '{$this->query}'.");
|
|
| 71 | + } else {
|
|
| 72 | + return array( |
|
| 73 | + 'title' => "Sorry, '{$this->query}' didn't return any posts.");
|
|
| 74 | + } |
|
| 75 | 75 | } |
| 76 | 76 | |
| 77 | 77 | protected function get_page_number() |
| 78 | 78 | { |
| 79 | 79 | $page = URLDecode::getPiece(3); |
| 80 | - if(isset($page) && is_numeric($page)) |
|
| 81 | - return $page; |
|
| 80 | + if(isset($page) && is_numeric($page)) {
|
|
| 81 | + return $page; |
|
| 82 | + } |
|
| 82 | 83 | return 1; |
| 83 | 84 | } |
| 84 | 85 | |
@@ -113,25 +114,29 @@ discard block |
||
| 113 | 114 | |
| 114 | 115 | protected function get_list_next_link() |
| 115 | 116 | { |
| 116 | - if($this->page == 1) |
|
| 117 | - return; |
|
| 118 | - if($this->page == 2) |
|
| 119 | - return Content::instance('URLSafe', "/search/{$this->query}/")->activate(); |
|
| 117 | + if($this->page == 1) {
|
|
| 118 | + return; |
|
| 119 | + } |
|
| 120 | + if($this->page == 2) {
|
|
| 121 | + return Content::instance('URLSafe', "/search/{$this->query}/")->activate();
|
|
| 122 | + } |
|
| 120 | 123 | return Content::instance('URLSafe', "/search/{$this->query}/" . ($this->page - 1) . '/')->activate(); |
| 121 | 124 | } |
| 122 | 125 | |
| 123 | 126 | protected function get_list_prev_link() |
| 124 | 127 | { |
| 125 | - if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count()) |
|
| 126 | - return; |
|
| 128 | + if(($this->page * self::$POSTS_PER_PAGE) >= $this->get_total_post_count()) {
|
|
| 129 | + return; |
|
| 130 | + } |
|
| 127 | 131 | return Content::instance('URLSafe', "/search/{$this->query}/" . ($this->page + 1) . '/')->activate(); |
| 128 | 132 | } |
| 129 | 133 | |
| 130 | 134 | private $total_post_count; |
| 131 | 135 | protected function get_total_post_count() |
| 132 | 136 | { |
| 133 | - if(!isset($this->total_post_count)) |
|
| 134 | - $this->total_post_count = count($this->get_search_result()); |
|
| 137 | + if(!isset($this->total_post_count)) {
|
|
| 138 | + $this->total_post_count = count($this->get_search_result()); |
|
| 139 | + } |
|
| 135 | 140 | return $this->total_post_count; |
| 136 | 141 | } |
| 137 | 142 | |