Passed
Push — master ( 355b9e...54a74b )
by Francis
01:27
created
views/post_edit.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
   </textarea>
48 48
 </div>
49 49
 <?php
50
-$ci =& get_instance();
50
+$ci = & get_instance();
51 51
 $ci->load->splint("francis94c/ci-parsedown", "+Parsedown", null, "parsedown");
52 52
 ?>
53 53
 <div class="w3-padding w3-margin w3-border w3-round" id="preview">
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
  * @return [type] [description]
77 77
  */
78 78
 function saveAndPublish() {
79
-  document.getElementById("action").value = <?=$type == "edit" ? "\"publish\"" : "\"createAndPublish\"";?>;
79
+  document.getElementById("action").value = <?=$type == "edit" ? "\"publish\"" : "\"createAndPublish\""; ?>;
80 80
   document.getElementById("postForm").submit();
81 81
 }
82 82
 /**
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 </script>
105 105
 <input type="hidden" name="id" value="<?=$type == "edit" ? $id : ""?>"/>
106 106
 <input id="action" type="hidden" name="action" value="publish"/>
107
-<?=form_close();?>
107
+<?=form_close(); ?>
108 108
 <button onclick="save();" class="w3-margin w3-button w3-teal w3-round w3-hover-theme"><?=$type == "edit" ? "Save" : "Create"?></button>
109 109
 <button onclick="promptPublish();" class="w3-margin w3-button w3-teal w3-hover-theme w3-round"><?=$type == "edit" ? "Save and Publish" : "Create and Publish"?></button>
110 110
 <?php if ($type == "edit") {?>
Please login to merge, or discard this patch.
unit_tests/BlogTest.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
    * Prerquisites for the Unit Tests.
20 20
    */
21 21
   public static function setUpBeforeClass(): void {
22
-    self::$ci =& get_instance();
22
+    self::$ci = & get_instance();
23 23
     self::$ci->load->database('mysqli://root@localhost/test_db');
24 24
     $queries = [
25 25
       "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 @@  discard block
 block discarded – undo
78 78
    */
79 79
   public function testEmptyBlog(): void {
80 80
     self::$ci->blogger->setBlog("test_blog");
81
-    $this->setOutputCallback(function ($output) {
81
+    $this->setOutputCallback(function($output) {
82 82
       $this->assertRegExp("/<h3 class=\"w3-center w3-margin\">No Posts.<\/h3>/", $output);
83 83
       $this->assertRegExp("/<div class=\"w3-padding\">/", $output);
84 84
     });
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
   public function testEditor(): void {
97 97
 
98 98
     // === Collect Output ===
99
-    $this->setOutputCallback(function () {});
99
+    $this->setOutputCallback(function() {});
100 100
     $this->assertTrue(self::$ci->blogger->loadEditor("my_callback")); // Outputs Editor HTML.
101 101
     $o = $this->getActualOutput();
102 102
     // ==/ Collect Output ===
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
     $this->assertRegExp("/\/my_callback/", $o);
110 110
 
111 111
     // Reset Output Callback.
112
-    $this->setOutputCallback(function ($o) { return $o;});
112
+    $this->setOutputCallback(function($o) { return $o; });
113 113
 
114 114
     $this->expectOutputRegex("/value=\"1\"\/>/");
115 115
     $this->assertTrue(self::$ci->blogger->loadEditor("my_callback", 1)); // Outputs Editor HTML.
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
    */
238 238
   public function testEditPostUI(): void {
239 239
     // === Collect Output ===
240
-    $this->setOutputCallback(function () {});
240
+    $this->setOutputCallback(function() {});
241 241
     $this->assertTrue(self::$ci->blogger->loadEditor("a_callback", 1)); // Outputs Editor HTML.
242 242
     $o = $this->getActualOutput();
243 243
     // ==/ Collect Output ===
@@ -354,7 +354,7 @@  discard block
 block discarded – undo
354 354
    */
355 355
   public function testRenderPostItems(): void {
356 356
     // === Collect Output ===
357
-    $this->setOutputCallback(function () {});
357
+    $this->setOutputCallback(function() {});
358 358
     $this->assertTrue(self::$ci->blogger->renderPostItems("../splints/" . self::PACKAGE . "/unit_tests/views/test_post_card_item", "the_gunners"));
359 359
     $o = $this->getActualOutput();
360 360
     // ==/ Collect Output ===
Please login to merge, or discard this patch.
models/BlogManager.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@  discard block
 block discarded – undo
11 11
 
12 12
   function __construct() {
13 13
     parent::__construct();
14
-    $this->ci =& get_instance();
14
+    $this->ci = & get_instance();
15 15
     $this->ci->load->database();
16 16
   }
17 17
   /**
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
    *                         database. Returns 0 if the post couldn't be
45 45
    *                         created.
46 46
    */
47
-  function createPost(string $title, string $content, int $adminId=null): int {
47
+  function createPost(string $title, string $content, int $adminId = null): int {
48 48
     $this->ci->load->helper("url");
49 49
     $data = array(
50 50
       "title"   => $title,
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
    * @param  [type] $adminId [description]
64 64
    * @return [type]          [description]
65 65
    */
66
-  function createAndPublishPost($title, $content, $adminId=null) {
66
+  function createAndPublishPost($title, $content, $adminId = null) {
67 67
     $data = array(
68 68
       "title"          => $title,
69 69
       "content"        => $content,
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
    *
92 92
    * @return array           Array of posts for a given page.
93 93
    */
94
-  function getPosts(int $page=1, int $limit=5, bool $filter=false, bool $hits=false): array {
94
+  function getPosts(int $page = 1, int $limit = 5, bool $filter = false, bool $hits = false): array {
95 95
     if ($limit != 0) $this->db->limit($limit, ($page * $limit) - $limit);
96 96
     if ($filter) $this->db->where("published", 1);
97 97
     if ($hits) {
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
    * @param  boolean $filter [description]
108 108
    * @return [type]          [description]
109 109
    */
110
-  function getRecentPosts($limit=5, $filter=false) {
110
+  function getRecentPosts($limit = 5, $filter = false) {
111 111
     if ($limit != null && $limit != null) $this->db->limit($limit);
112 112
     if ($filter) $this->db->where("published", 1);
113 113
     $this->db->order_by("id", "DESC");
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
    * @return bool            True on success, False if not.
123 123
    */
124 124
   function savePost($postId, $title, $content): bool {
125
-    $data = array (
125
+    $data = array(
126 126
       "title"   => $title,
127 127
       "content" => $content,
128 128
       "slug"    => url_title($title)
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
    * @param  int   $postId ID of the post to retrieve.
137 137
    * @return array An associative array for the Posts's data.
138 138
    */
139
-  function getPost($postId, $hit=true) {
139
+  function getPost($postId, $hit = true) {
140 140
     if (is_numeric($postId)) {
141 141
       $this->db->where("id", $postId);
142 142
     } else {
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
           $this->db->update($this->table_name);
154 154
         }
155 155
       }
156
-      $post =  $query->result_array()[0];
156
+      $post = $query->result_array()[0];
157 157
       $images = array();
158 158
       // Fetch all images in post.
159 159
       preg_match("/<img\s[^>]*?src\s*=\s*['\"]([^'\"]*?)['\"][^>]*?>/", $post["content"], $images);
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
    * [getPostsCount get the total number of posts in the database.]
196 196
    * @return [type] [description]
197 197
    */
198
-  function getPostsCount(bool $filter=true): int {
198
+  function getPostsCount(bool $filter = true): int {
199 199
     if ($filter) $this->db->where("published", 1);
200 200
     $this->db->select("COUNT(title) as posts");
201 201
     return (int) $this->db->get($this->table_name)->result()[0]->posts;
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
    * @param  boolean $filter [description]
218 218
    * @return [type]          [description]
219 219
    */
220
-  function searchPosts($words, $page, $limit=0, $filter=false) {
220
+  function searchPosts($words, $page, $limit = 0, $filter = false) {
221 221
     if ($limit != 0) $this->db->limit($limit, ($page * $limit) - $limit);
222 222
     if ($filter) $this->db->where("published", 1);
223 223
     $this->db->like("title", $words);
Please login to merge, or discard this patch.
libraries/Blogger.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -74,8 +74,8 @@  discard block
 block discarded – undo
74 74
    * Constructor
75 75
    * @param mixed $params associative array of parameters. See README.md
76 76
    */
77
-  function __construct($params=null) {
78
-    $this->ci =& /** @scrutinizer ignore-call */ get_instance();
77
+  function __construct($params = null) {
78
+    $this->ci = & /** @scrutinizer ignore-call */ get_instance();
79 79
     $this->ci->load->database();
80 80
     $this->table_name = self::TABLE_PREFIX . (isset($params["name"]) ? "_" . $params["name"] : "");
81 81
     $this->ci->load->database();
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
    *
100 100
    * @return bool                            True on Success, False if Not.
101 101
    */
102
-  public function install(string $blogName=null, string $adminTableName=null, string $adminIdColumnName=null, int $adminIdColumnConstraint=null): bool {
102
+  public function install(string $blogName = null, string $adminTableName = null, string $adminIdColumnName = null, int $adminIdColumnConstraint = null): bool {
103 103
     $blogName = $blogName === null ? $this->table_name : self::TABLE_PREFIX . "_" . $blogName;
104 104
     $this->ci->load->dbforge();
105 105
     $this->ci->dbforge->add_field("id");
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
    *
216 216
    * @return bool              True  if sucessfull without errors, false if not.
217 217
    */
218
-  public function loadEditor(string $callback, int $postId=null, bool $w3css=true): bool {
218
+  public function loadEditor(string $callback, int $postId = null, bool $w3css = true): bool {
219 219
     $this->loadScripts($w3css);
220 220
     $this->ci->load->helper("form");
221 221
     $data = array(
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
    *                       inputs. These are public string constants declared in
246 246
    *                       this file.
247 247
    */
248
-  public function savePost(int $posterId=null): string {
248
+  public function savePost(int $posterId = null): string {
249 249
     $action = $this->ci->security->xss_clean($this->ci->input->post("action"));
250 250
     switch ($action) {
251 251
       case "save":
@@ -269,7 +269,7 @@  discard block
 block discarded – undo
269 269
    *
270 270
    * @return string             Action taken during the pocess; Blogger::CREATE Or Blogger::EDIT
271 271
    */
272
-  private function handleSavePost(int $posterId=null): string {
272
+  private function handleSavePost(int $posterId = null): string {
273 273
     $id = $this->ci->security->xss_clean($this->ci->input->post("id"));
274 274
     if ($id != "") {
275 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;
@@ -287,7 +287,7 @@  discard block
 block discarded – undo
287 287
 
288 288
    * @return string           Action reached while processing form inputs.
289 289
    */
290
-  private function handlePublishPost(int $posterId=null): string {
290
+  private function handlePublishPost(int $posterId = null): string {
291 291
     $id = $this->ci->security->xss_clean($this->ci->input->post("id"));
292 292
     if ($id == "") return self::ABORT;
293 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;
@@ -309,7 +309,7 @@  discard block
 block discarded – undo
309 309
    *
310 310
    * @return array           Array of posts for a given page.
311 311
    */
312
-  public function getPosts(int $page, int $limit, bool $filter=false, bool $hits=false): array {
312
+  public function getPosts(int $page, int $limit, bool $filter = false, bool $hits = false): array {
313 313
     return $this->ci->bmanager->getPosts($page, $limit, $filter, $hits);
314 314
   }
315 315
   /**
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
    *
336 336
    * @return bool                True if successful, false if not.
337 337
    */
338
-  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 {
338
+  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 339
     if ($view === null || $empty_view === null) $this->ci->load->bind("francis94c/blog", $blogger);
340 340
     $posts = $this->getPosts($page, $limit, $filter, $hits);
341 341
     if (count($posts) == 0) {
@@ -364,7 +364,7 @@  discard block
 block discarded – undo
364 364
    *
365 365
    * @return array        Array of posts ordered by ID (Most recent first).
366 366
    */
367
-  public function getRecentPosts(int $limit=5, bool $filter=false): array {
367
+  public function getRecentPosts(int $limit = 5, bool $filter = false): array {
368 368
     return $this->ci->bmanager->getRecentPosts($limit, $filter);
369 369
   }
370 370
   /**
@@ -378,7 +378,7 @@  discard block
 block discarded – undo
378 378
    *
379 379
    * @return bool             True if successful, false if not.
380 380
    */
381
-  public function renderPost($post, string $view=null): bool {
381
+  public function renderPost($post, string $view = null): bool {
382 382
     if (!is_array($post)) $post = $this->ci->bmanager->getPost($post);
383 383
     if (!$post) return false;
384 384
     $post["content"] = $this->ci->parsedown->text($post["content"]);
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
417 417
    *
418 418
    * @return int          Posts count.
419 419
    */
420
-  public function getPostsCount(bool $filter=true): int {
420
+  public function getPostsCount(bool $filter = true): int {
421 421
     return $this->ci->bmanager->getPostsCount($filter);
422 422
   }
423 423
   /**
@@ -429,7 +429,7 @@  discard block
 block discarded – undo
429 429
    *
430 430
    * @return array          An associative array of a single post.
431 431
    */
432
-  public function getPost($postId, $hit=true) {
432
+  public function getPost($postId, $hit = true) {
433 433
     return $this->ci->bmanager->getPost($postId, $hit);
434 434
   }
435 435
   /**
@@ -467,7 +467,7 @@  discard block
 block discarded – undo
467 467
    * @param  boolean $filter [description]
468 468
    * @return [type]          [description]
469 469
    */
470
-  public function searchPosts($words, $page, $limit=0, $filter=false) {
470
+  public function searchPosts($words, $page, $limit = 0, $filter = false) {
471 471
     return $this->ci->bmanager->searchPosts($words, $page, $limit, $filter);
472 472
   }
473 473
 }
Please login to merge, or discard this patch.