francis94c / blog

models/BlogManager.php

Braces +42 added lines, -14 removed lines

@@ -51,9 +51,15 @@ 
       "content" => $content,
       "slug"    => url_title($title)
     );
-    if (is_numeric($title)) $data["slug"] = "_" . $data["slug"];
-    if ($adminId !== null) $data["poster_id"] = $adminId;
-    if ($this->db->insert($this->table_name, $data)) return $this->db->insert_id();
+    if (is_numeric($title)) {
+      $data["slug"] = "_" . $data["slug"];
+    }
+    if ($adminId !== null) {
+      $data["poster_id"] = $adminId;
+    }
+    if ($this->db->insert($this->table_name, $data)) {
+      return $this->db->insert_id();
+    }
     return 0;
   }
   /**
@@ -71,9 +77,15 @@ 
       "published"      => 1,
       "date_published" => date("Y-m-d H:i:s")
     );
-    if (is_numeric($title)) $data["slug"] = "_" . $data["slug"];
-    if ($adminId != null) $data["poster_id"] = $adminId;
-    if ($this->db->insert($this->table_name, $data)) return $this->db->insert_id();
+    if (is_numeric($title)) {
+      $data["slug"] = "_" . $data["slug"];
+    }
+    if ($adminId != null) {
+      $data["poster_id"] = $adminId;
+    }
+    if ($this->db->insert($this->table_name, $data)) {
+      return $this->db->insert_id();
+    }
     return false;
   }
   /**
@@ -87,8 +99,12 @@ 
    * @return array           Array of posts for a given page.
    */
   function getPosts($page=1, $limit=5, $filter=false, $hits=false) {
-    if ($limit != 0) $this->db->limit($limit, ($page * $limit) - $limit);
-    if ($filter) $this->db->where("published", 1);
+    if ($limit != 0) {
+      $this->db->limit($limit, ($page * $limit) - $limit);
+    }
+    if ($filter) {
+      $this->db->where("published", 1);
+    }
     if ($hits) {
       $this->db->order_by("hits", "DESC");
     } else {
@@ -103,8 +119,12 @@ 
    * @return [type]          [description]
    */
   function getRecentPosts($limit=5, $filter=false) {
-    if ($limit != null && $limit != null) $this->db->limit($limit);
-    if ($filter) $this->db->where("published", 1);
+    if ($limit != null && $limit != null) {
+      $this->db->limit($limit);
+    }
+    if ($filter) {
+      $this->db->where("published", 1);
+    }
     $this->db->order_by("id", "DESC");
     return $this->db->get($this->table_name)->result_array();
   }
@@ -156,7 +176,9 @@ 
       $src = array();
       // Get the contents of the src tag.
       preg_match("/(http|https):\/\/[a-zA-Z0-9-._\/]+/", $share_image, $src);
-      if (count($src) == 0) return $post;
+      if (count($src) == 0) {
+        return $post;
+      }
       $post["share_image"] = $src[0];
       return $post;
     }
@@ -170,7 +192,9 @@ 
   function getHits($postId) {
     $this->db->where("id", $postId);
     $query = $this->db->get($this->table_name);
-    if ($query->num_rows() > 0) return $query->result()[0]->hits;
+    if ($query->num_rows() > 0) {
+      return $query->result()[0]->hits;
+    }
     return 0;
   }
   /**
@@ -211,8 +235,12 @@ 
    * @return [type]          [description]
    */
   function searchPosts($words, $page, $limit=0, $filter=false) {
-    if ($limit != 0) $this->db->limit($limit, ($page * $limit) - $limit);
-    if ($filter) $this->db->where("published", 1);
+    if ($limit != 0) {
+      $this->db->limit($limit, ($page * $limit) - $limit);
+    }
+    if ($filter) {
+      $this->db->where("published", 1);
+    }
     $this->db->like("title", $words);
     $this->db->or_like("content", $words);
     return $this->db->get($this->table_name)->result_array();

libraries/Blogger.php

Braces +27 added lines, -9 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;
   }
   /**
@@ -252,7 +254,9 @@ 
     } elseif ($action == "publish" || $action == "createAndPublish") {
       if ($action == "publish") {
         $id = $this->ci->security->xss_clean($this->ci->input->post("id"));
-        if ($id == "") return self::ABORT;
+        if ($id == "") {
+          return self::ABORT;
+        }
         $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);
         $this->ci->bmanager->publishPost($id, true);
         return self::PUBLISH;
@@ -263,7 +267,9 @@ 
         return self::ABORT;
       }
     } elseif ($action == "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::NO_ACTION;
   }
@@ -274,10 +280,14 @@ 
   private function handleSavePost(): 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;
     }
   }
@@ -310,7 +320,9 @@ 
    * @return [type]              [description]
    */
   public function renderPostItems($view=null, $callback=null, $empty_view=null, $page=1, $limit=5, $filter=false, $hits=false, $slug=true) {
-    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 {
@@ -345,8 +357,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);
@@ -364,7 +380,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);
   }