francis94c / blog

phpunit/BlogTest.php

Spacing +6 added lines, -6 removed lines

@@ -19,7 +19,7 @@ 
    * Prerquisites for the Unit Tests.
    */
   public static function setUpBeforeClass(): void {
-    self::$ci =& get_instance();
+    self::$ci = & get_instance();
     self::$ci->load->database('mysqli://root@localhost/test_db');
     $queries = [
       "CREATE TABLE IF NOT EXISTS admins (id INT(7) AUTO_INCREMENT PRIMARY KEY, name VARCHAR(20) NOT NULL, password TEXT NOT NULL) Engine=InnoDB;",
@@ -78,7 +78,7 @@ 
    */
   public function testEmptyBlog(): void {
     self::$ci->blogger->setBlog("test_blog");
-    $this->setOutputCallback(function ($output) {
+    $this->setOutputCallback(function($output) {
       $this->assertRegExp("/<h3 class=\"w3-center w3-margin\">No Posts.<\/h3>/", $output);
       $this->assertRegExp("/<div class=\"w3-padding\">/", $output);
     });
@@ -96,7 +96,7 @@ 
   public function testEditor(): void {
 
     // === Collect Output ===
-    $this->setOutputCallback(function () {});
+    $this->setOutputCallback(function() {});
     $this->assertTrue(self::$ci->blogger->loadEditor("my_callback")); // Outputs Editor HTML.
     $o = $this->getActualOutput();
     // ==/ Collect Output ===
@@ -109,7 +109,7 @@ 
     $this->assertRegExp("/\/my_callback/", $o);
 
     // Reset Output Callback.
-    $this->setOutputCallback(function ($o) { return $o;});
+    $this->setOutputCallback(function($o) { return $o; });
 
     $this->expectOutputRegex("/value=\"1\"\/>/");
     $this->assertTrue(self::$ci->blogger->loadEditor("my_callback", 1)); // Outputs Editor HTML.
@@ -237,7 +237,7 @@ 
    */
   public function testEditPostUI(): void {
     // === Collect Output ===
-    $this->setOutputCallback(function () {});
+    $this->setOutputCallback(function() {});
     $this->assertTrue(self::$ci->blogger->loadEditor("a_callback", 1)); // Outputs Editor HTML.
     $o = $this->getActualOutput();
     // ==/ Collect Output ===
@@ -370,7 +370,7 @@ 
    */
   public function testRenderPostItems(): void {
     // === Collect Output ===
-    $this->setOutputCallback(function () {});
+    $this->setOutputCallback(function() {});
     $this->assertTrue(self::$ci->blogger->renderPostItems("../splints/" . self::PACKAGE . "/phpunit/views/test_post_card_item", "the_gunners"));
     $o = $this->getActualOutput();
     // ==/ Collect Output ===

libraries/Blogger.php

Spacing +14 added lines, -14 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        Array of posts ordered by ID (Most recent first).
    */
-  public function getRecentPosts(int $limit=5, bool $filter=false): array {
+  public function getRecentPosts(int $limit = 5, bool $filter = false): array {
     return $this->ci->bmanager->getRecentPosts($limit, $filter);
   }
   /**
@@ -378,7 +378,7 @@ 
    *
    * @return bool             True if successful, false if not.
    */
-  public function renderPost($post, string $view=null): bool {
+  public function renderPost($post, string $view = null): bool {
     if (!is_array($post)) $post = $this->ci->bmanager->getPost($post);
     if (!$post) return false;
     $post["content"] = $this->ci->parsedown->text($post["content"]);
@@ -417,7 +417,7 @@ 
    *
    * @return int          Posts count.
    */
-  public function getPostsCount(bool $filter=true): int {
+  public function getPostsCount(bool $filter = true): int {
     return $this->ci->bmanager->getPostsCount($filter);
   }
   /**
@@ -429,7 +429,7 @@ 
    *
    * @return array          An associative array of a single post.
    */
-  public function getPost($postId, $hit=true) {
+  public function getPost($postId, $hit = true) {
     return $this->ci->bmanager->getPost($postId, $hit);
   }
   /**
@@ -476,7 +476,7 @@ 
    *
    * @return array           Array of matched posts.
    */
-  public function searchPosts(string $words, int $page, int $limit=5, bool $filter=false): array {
+  public function searchPosts(string $words, int $page, int $limit = 5, bool $filter = false): array {
     return $this->ci->bmanager->searchPosts($words, $page, $limit, $filter);
   }
 }

models/BlogManager.php

Spacing +10 added lines, -10 removed lines

@@ -11,7 +11,7 @@ 
 
   function __construct() {
     parent::__construct();
-    $this->ci =& get_instance();
+    $this->ci = & get_instance();
     $this->ci->load->database();
   }
   /**
@@ -44,7 +44,7 @@ 
    *                         database. Returns 0 if the post couldn't be
    *                         created.
    */
-  function createPost(string $title, string $content, int $adminId=null): int {
+  function createPost(string $title, string $content, int $adminId = null): int {
     $this->ci->load->helper("url");
     $data = array(
       "title"   => $title,
@@ -63,7 +63,7 @@ 
    * @param  [type] $adminId [description]
    * @return [type]          [description]
    */
-  function createAndPublishPost($title, $content, $adminId=null) {
+  function createAndPublishPost($title, $content, $adminId = null) {
     $data = array(
       "title"          => $title,
       "content"        => $content,
@@ -91,7 +91,7 @@ 
    *
    * @return array           Array of posts for a given page.
    */
-  function getPosts(int $page=1, int $limit=5, bool $filter=false, bool $hits=false): array {
+  function getPosts(int $page = 1, int $limit = 5, bool $filter = false, bool $hits = false): array {
     if ($limit != 0) $this->db->limit($limit, ($page * $limit) - $limit);
     if ($filter) $this->db->where("published", 1);
     if ($hits) {
@@ -107,7 +107,7 @@ 
    * @param  boolean $filter [description]
    * @return [type]          [description]
    */
-  function getRecentPosts($limit=5, $filter=false) {
+  function getRecentPosts($limit = 5, $filter = false) {
     if ($limit != null && $limit != null) $this->db->limit($limit);
     if ($filter) $this->db->where("published", 1);
     $this->db->order_by("id", "DESC");
@@ -122,7 +122,7 @@ 
    * @return bool            True on success, False if not.
    */
   function savePost($postId, $title, $content): bool {
-    $data = array (
+    $data = array(
       "title"   => $title,
       "content" => $content,
       "slug"    => url_title($title)
@@ -136,7 +136,7 @@ 
    * @param  int   $postId ID of the post to retrieve.
    * @return array An associative array for the Posts's data.
    */
-  function getPost($postId, $hit=true) {
+  function getPost($postId, $hit = true) {
     if (is_numeric($postId)) {
       $this->db->where("id", $postId);
     } else {
@@ -153,7 +153,7 @@ 
           $this->db->update($this->table_name);
         }
       }
-      $post =  $query->result_array()[0];
+      $post = $query->result_array()[0];
       $images = array();
       // Fetch all images in post.
       preg_match("/<img\s[^>]*?src\s*=\s*['\"]([^'\"]*?)['\"][^>]*?>/", $post["content"], $images);
@@ -195,7 +195,7 @@ 
    * [getPostsCount get the total number of posts in the database.]
    * @return [type] [description]
    */
-  function getPostsCount(bool $filter=true): int {
+  function getPostsCount(bool $filter = true): int {
     if ($filter) $this->db->where("published", 1);
     $this->db->select("COUNT(title) as posts");
     return (int) $this->db->get($this->table_name)->result()[0]->posts;
@@ -217,7 +217,7 @@ 
    * @param  boolean $filter [description]
    * @return [type]          [description]
    */
-  function searchPosts($words, $page, $limit=5, $filter=false) {
+  function searchPosts($words, $page, $limit = 5, $filter = false) {
     if ($limit != 0) $this->db->limit($limit, ($page * $limit) - $limit);
     if ($filter) $this->db->where("published", 1);
     $this->db->like("title", $words);

Braces +48 added lines, -16 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;
   }
   /**
@@ -92,8 +104,12 @@ 
    * @return array           Array of posts for a given page.
    */
   function getPosts(int $page=1, int $limit=5, bool $filter=false, bool $hits=false): array {
-    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 {
@@ -108,8 +124,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();
   }
@@ -147,7 +167,9 @@ 
       if ($hit) {
         if ($this->ci->config->item("blogger_hits") === null ||
         $this->ci->config->item("blogger_hits") === true) {
-          if (!is_numeric($postId)) $postId = $query->result()[0]->id;
+          if (!is_numeric($postId)) {
+            $postId = $query->result()[0]->id;
+          }
           $this->db->where("id", $postId);
           $this->db->set("hits", "hits+1", FALSE);
           $this->db->update($this->table_name);
@@ -162,7 +184,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;
     }
@@ -176,7 +200,9 @@ 
   function getHits(int $postId): int {
     $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;
   }
   /**
@@ -196,7 +222,9 @@ 
    * @return [type] [description]
    */
   function getPostsCount(bool $filter=true): int {
-    if ($filter) $this->db->where("published", 1);
+    if ($filter) {
+      $this->db->where("published", 1);
+    }
     $this->db->select("COUNT(title) as posts");
     return (int) $this->db->get($this->table_name)->result()[0]->posts;
   }
@@ -218,8 +246,12 @@ 
    * @return [type]          [description]
    */
   function searchPosts($words, $page, $limit=5, $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();