@@ -139,7 +139,9 @@ discard block |
||
139 | 139 | } |
140 | 140 | $this->ci->dbforge->add_field("date_created TIMESTAMP DEFAULT CURRENT_TIMESTAMP"); |
141 | 141 | $attributes = array('ENGINE' => 'InnoDB'); |
142 | - if (!$this->ci->dbforge->create_table($blogName, true, $attributes)) return false; |
|
142 | + if (!$this->ci->dbforge->create_table($blogName, true, $attributes)) { |
|
143 | + return false; |
|
144 | + } |
|
143 | 145 | return true; |
144 | 146 | } |
145 | 147 | /** |
@@ -253,10 +255,14 @@ discard block |
||
253 | 255 | case "publish": |
254 | 256 | return $this->handlePublishPost($posterId); |
255 | 257 | case "createAndPublish": |
256 | - if ($this->ci->bmanager->createAndPublishPost($this->ci->security->xss_clean($this->ci->input->post("title")), $this->ci->security->xss_clean($this->ci->input->post("editor")), $posterId) !== false) return self::CREATE_AND_PUBLISH; |
|
258 | + if ($this->ci->bmanager->createAndPublishPost($this->ci->security->xss_clean($this->ci->input->post("title")), $this->ci->security->xss_clean($this->ci->input->post("editor")), $posterId) !== false) { |
|
259 | + return self::CREATE_AND_PUBLISH; |
|
260 | + } |
|
257 | 261 | return self::ABORT; |
258 | 262 | case "delete": |
259 | - if ($this->ci->bmanager->deletePost($this->ci->security->xss_clean($this->ci->input->post("id")))) return self::DELETE; |
|
263 | + if ($this->ci->bmanager->deletePost($this->ci->security->xss_clean($this->ci->input->post("id")))) { |
|
264 | + return self::DELETE; |
|
265 | + } |
|
260 | 266 | return self::ABORT; |
261 | 267 | default: |
262 | 268 | return self::NO_ACTION; |
@@ -272,10 +278,14 @@ discard block |
||
272 | 278 | private function handleSavePost(int $posterId=null): string { |
273 | 279 | $id = $this->ci->security->xss_clean($this->ci->input->post("id")); |
274 | 280 | if ($id != "") { |
275 | - if (!$this->ci->bmanager->savePost($this->ci->input->post("id"), $this->ci->security->xss_clean($this->ci->input->post("title")), $this->ci->security->xss_clean($this->ci->input->post("editor")), $posterId)) return self::ABORT; |
|
281 | + if (!$this->ci->bmanager->savePost($this->ci->input->post("id"), $this->ci->security->xss_clean($this->ci->input->post("title")), $this->ci->security->xss_clean($this->ci->input->post("editor")), $posterId)) { |
|
282 | + return self::ABORT; |
|
283 | + } |
|
276 | 284 | return self::EDIT; |
277 | 285 | } else { |
278 | - if ($this->ci->bmanager->createPost($this->ci->security->xss_clean($this->ci->input->post("title")), $this->ci->security->xss_clean($this->ci->input->post("editor")), $posterId) == 0) return self::ABORT; |
|
286 | + if ($this->ci->bmanager->createPost($this->ci->security->xss_clean($this->ci->input->post("title")), $this->ci->security->xss_clean($this->ci->input->post("editor")), $posterId) == 0) { |
|
287 | + return self::ABORT; |
|
288 | + } |
|
279 | 289 | return self::CREATE; |
280 | 290 | } |
281 | 291 | } |
@@ -289,9 +299,15 @@ discard block |
||
289 | 299 | */ |
290 | 300 | private function handlePublishPost(int $posterId=null): string { |
291 | 301 | $id = $this->ci->security->xss_clean($this->ci->input->post("id")); |
292 | - if ($id == "") return self::ABORT; |
|
293 | - if (!$this->ci->bmanager->savePost($id, $this->ci->security->xss_clean($this->ci->input->post("title")), $this->ci->security->xss_clean($this->ci->input->post("editor")), $posterId)) return self::ABORT; |
|
294 | - if (!$this->ci->bmanager->publishPost($id, true)) return self::ABORT; |
|
302 | + if ($id == "") { |
|
303 | + return self::ABORT; |
|
304 | + } |
|
305 | + if (!$this->ci->bmanager->savePost($id, $this->ci->security->xss_clean($this->ci->input->post("title")), $this->ci->security->xss_clean($this->ci->input->post("editor")), $posterId)) { |
|
306 | + return self::ABORT; |
|
307 | + } |
|
308 | + if (!$this->ci->bmanager->publishPost($id, true)) { |
|
309 | + return self::ABORT; |
|
310 | + } |
|
295 | 311 | return self::PUBLISH; |
296 | 312 | } |
297 | 313 | /** |
@@ -336,7 +352,9 @@ discard block |
||
336 | 352 | * @return bool True if successful, false if not. |
337 | 353 | */ |
338 | 354 | public function renderPostItems(string $view=null, string $callback=null, string $empty_view=null, int $page=1, int $limit=5, bool $filter=false, bool $hits=false, bool $slug=true): bool { |
339 | - if ($view === null || $empty_view === null) $this->ci->load->bind("francis94c/blog", $blogger); |
|
355 | + if ($view === null || $empty_view === null) { |
|
356 | + $this->ci->load->bind("francis94c/blog", $blogger); |
|
357 | + } |
|
340 | 358 | $posts = $this->getPosts($page, $limit, $filter, $hits); |
341 | 359 | if (count($posts) == 0) { |
342 | 360 | if ($empty_view === null) { $blogger->load->view("empty"); } else { |
@@ -379,8 +397,12 @@ discard block |
||
379 | 397 | * @return bool True if successful, false if not. |
380 | 398 | */ |
381 | 399 | public function renderPost($post, $view=null): bool { |
382 | - if (!is_array($post)) $post = $this->ci->bmanager->getPost($post); |
|
383 | - if (!$post) return false; |
|
400 | + if (!is_array($post)) { |
|
401 | + $post = $this->ci->bmanager->getPost($post); |
|
402 | + } |
|
403 | + if (!$post) { |
|
404 | + return false; |
|
405 | + } |
|
384 | 406 | $post["content"] = $this->ci->parsedown->text($post["content"]); |
385 | 407 | if ($view === null) { |
386 | 408 | $this->ci->load->splint("francis94c/blog", "-post_item", $post); |
@@ -403,7 +425,9 @@ discard block |
||
403 | 425 | $data = array(); |
404 | 426 | $data["title"] = $post["title"]; |
405 | 427 | $data["description"] = substr($post["content"], 0, 154); |
406 | - if (isset($post["share_image"])) $data["image_link"] = $post["share_image"]; |
|
428 | + if (isset($post["share_image"])) { |
|
429 | + $data["image_link"] = $post["share_image"]; |
|
430 | + } |
|
407 | 431 | $data["url"] = current_url(); |
408 | 432 | // Return Meta OG View String. |
409 | 433 | return $this->ci->load->splint(self::PACKAGE, "-meta_og", $data, true); |