francis94c / blog

libraries/Blogger.php

Spacing +13 added lines, -13 removed lines

@@ -74,8 +74,8 @@ 
    * Constructor
    * @param mixed $params associative array of parameters. See README.md
    */
-  function __construct($params=null) {
-    $this->ci =& /** @scrutinizer ignore-call */ get_instance();
+  function __construct($params = null) {
+    $this->ci = & /** @scrutinizer ignore-call */ get_instance();
     $this->ci->load->database();
     $this->table_name = self::TABLE_PREFIX . (isset($params["name"]) ? "_" . $params["name"] : "");
     $this->ci->load->database();
@@ -99,7 +99,7 @@ 
    *
    * @return bool                            True on Success, False if Not.
    */
-  public function install(string $blogName=null, string $adminTableName=null, string $adminIdColumnName=null, int $adminIdColumnConstraint=null): bool {
+  public function install(string $blogName = null, string $adminTableName = null, string $adminIdColumnName = null, int $adminIdColumnConstraint = null): bool {
     $blogName = $blogName === null ? $this->table_name : self::TABLE_PREFIX . "_" . $blogName;
     $this->ci->load->dbforge();
     $this->ci->dbforge->add_field("id");
@@ -215,7 +215,7 @@ 
    *
    * @return bool              True  if sucessfull without errors, false if not.
    */
-  public function loadEditor(string $callback, int $postId=null, bool $w3css=true): bool {
+  public function loadEditor(string $callback, int $postId = null, bool $w3css = true): bool {
     $this->loadScripts($w3css);
     $this->ci->load->helper("form");
     $data = array(
@@ -245,7 +245,7 @@ 
    *                       inputs. These are public string constants declared in
    *                       this file.
    */
-  public function savePost(int $posterId=null): string {
+  public function savePost(int $posterId = null): string {
     $action = $this->ci->security->xss_clean($this->ci->input->post("action"));
     switch ($action) {
       case "save":
@@ -269,7 +269,7 @@ 
    *
    * @return string             Action taken during the pocess; Blogger::CREATE Or Blogger::EDIT
    */
-  private function handleSavePost(int $posterId=null): string {
+  private function handleSavePost(int $posterId = null): string {
     $id = $this->ci->security->xss_clean($this->ci->input->post("id"));
     if ($id != "") {
       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;
@@ -287,7 +287,7 @@ 
 
    * @return string           Action reached while processing form inputs.
    */
-  private function handlePublishPost(int $posterId=null): string {
+  private function handlePublishPost(int $posterId = null): string {
     $id = $this->ci->security->xss_clean($this->ci->input->post("id"));
     if ($id == "") return self::ABORT;
     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;
@@ -309,7 +309,7 @@ 
    *
    * @return array           Array of posts for a given page.
    */
-  public function getPosts(int $page, int $limit, bool $filter=false, bool $hits=false): array {
+  public function getPosts(int $page, int $limit, bool $filter = false, bool $hits = false): array {
     return $this->ci->bmanager->getPosts($page, $limit, $filter, $hits);
   }
   /**
@@ -335,7 +335,7 @@ 
    *
    * @return bool                True if successful, false if not.
    */
-  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 {
+  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 {
     if ($view === null || $empty_view === null) $this->ci->load->bind("francis94c/blog", $blogger);
     $posts = $this->getPosts($page, $limit, $filter, $hits);
     if (count($posts) == 0) {
@@ -364,7 +364,7 @@ 
    *
    * @return array          [description]
    */
-  public function getRecentPosts($limit=5, $filter=false): array {
+  public function getRecentPosts($limit = 5, $filter = false): array {
     return $this->ci->bmanager->getRecentPosts($limit, $filter);
   }
   /**
@@ -373,7 +373,7 @@ 
    * @param  [type] $view [description]
    * @return [type]       [description]
    */
-  public function renderPost($post, $view=null) {
+  public function renderPost($post, $view = null) {
     if (!is_array($post)) $post = $this->ci->bmanager->getPost($post);
     if (!$post) return false;
     $post["content"] = $this->ci->parsedown->text($post["content"]);
@@ -409,7 +409,7 @@ 
    * @param  [type] $postId [description]
    * @return [type]         [description]
    */
-  public function getPost($postId, $hit=true) {
+  public function getPost($postId, $hit = true) {
     return $this->ci->bmanager->getPost($postId, $hit);
   }
   /**
@@ -445,7 +445,7 @@ 
    * @param  boolean $filter [description]
    * @return [type]          [description]
    */
-  public function searchPosts($words, $page, $limit=0, $filter=false) {
+  public function searchPosts($words, $page, $limit = 0, $filter = false) {
     return $this->ci->bmanager->searchPosts($words, $page, $limit, $filter);
   }
 }

Braces +36 added lines, -12 removed lines

@@ -139,7 +139,9 @@ 
     }
     $this->ci->dbforge->add_field("date_created TIMESTAMP DEFAULT CURRENT_TIMESTAMP");
     $attributes = array('ENGINE' => 'InnoDB');
-    if (!$this->ci->dbforge->create_table($blogName, true, $attributes)) return false;
+    if (!$this->ci->dbforge->create_table($blogName, true, $attributes)) {
+      return false;
+    }
     return true;
   }
   /**
@@ -253,10 +255,14 @@ 
       case "publish":
         return $this->handlePublishPost($posterId);
       case "createAndPublish":
-        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;
+        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;
+        }
         return self::ABORT;
       case "delete":
-        if ($this->ci->bmanager->deletePost($this->ci->security->xss_clean($this->ci->input->post("id")))) return self::DELETE;
+        if ($this->ci->bmanager->deletePost($this->ci->security->xss_clean($this->ci->input->post("id")))) {
+          return self::DELETE;
+        }
         return self::ABORT;
       default:
         return self::NO_ACTION;
@@ -272,10 +278,14 @@ 
   private function handleSavePost(int $posterId=null): string {
     $id = $this->ci->security->xss_clean($this->ci->input->post("id"));
     if ($id != "") {
-      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;
+      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;
+      }
       return self::EDIT;
     } else {
-      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;
+      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;
+      }
       return self::CREATE;
     }
   }
@@ -289,9 +299,15 @@ 
    */
   private function handlePublishPost(int $posterId=null): string {
     $id = $this->ci->security->xss_clean($this->ci->input->post("id"));
-    if ($id == "") return self::ABORT;
-    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;
-    if (!$this->ci->bmanager->publishPost($id, true)) return self::ABORT;
+    if ($id == "") {
+      return self::ABORT;
+    }
+    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;
+    }
+    if (!$this->ci->bmanager->publishPost($id, true)) {
+      return self::ABORT;
+    }
     return self::PUBLISH;
   }
   /**
@@ -336,7 +352,9 @@ 
    * @return bool                True if successful, false if not.
    */
   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 {
-    if ($view === null || $empty_view === null) $this->ci->load->bind("francis94c/blog", $blogger);
+    if ($view === null || $empty_view === null) {
+      $this->ci->load->bind("francis94c/blog", $blogger);
+    }
     $posts = $this->getPosts($page, $limit, $filter, $hits);
     if (count($posts) == 0) {
       if ($empty_view === null) { $blogger->load->view("empty"); } else {
@@ -374,8 +392,12 @@ 
    * @return [type]       [description]
    */
   public function renderPost($post, $view=null) {
-    if (!is_array($post)) $post = $this->ci->bmanager->getPost($post);
-    if (!$post) return false;
+    if (!is_array($post)) {
+      $post = $this->ci->bmanager->getPost($post);
+    }
+    if (!$post) {
+      return false;
+    }
     $post["content"] = $this->ci->parsedown->text($post["content"]);
     if ($view == null) {
       $this->ci->load->splint("francis94c/blog", "-post_item", $post);
@@ -393,7 +415,9 @@ 
     $data = array();
     $data["title"] = $post["title"];
     $data["description"] = substr($post["content"], 0, 154);
-    if (isset($post["share_image"])) $data["image_link"] = $post["share_image"];
+    if (isset($post["share_image"])) {
+      $data["image_link"] = $post["share_image"];
+    }
     $data["url"] = current_url();
     return $this->ci->load->splint(self::PACKAGE, "-meta_og", $data, true);
   }