Passed
Push — Showing-Posts ( a59326...b4fc92 )
by Stone
01:47
created
Core/Traits/StringFunctions.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 
85 85
         //Searching for the page break tag
86 86
         $breakTagPosition = strpos($text, "<!-- EndOfExcerptBlogOc -->");
87
-        if($breakTagPosition > 0){
87
+        if ($breakTagPosition > 0) {
88 88
             return $this->completeDom(substr($text, 0, $breakTagPosition));
89 89
         }
90 90
 
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
         $string = preg_split('/(<img[^>]+\>)|(<p[^>]+\>)|(<span[^>]+\>)|\s/', $text, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE);
93 93
 
94 94
         //The preg split can return false, probably will never happen but just in case.
95
-        if(!$string)
95
+        if (!$string)
96 96
         {
97 97
             throw new \Error("excerpt generation failed");
98 98
         }
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
             $trimed .= $string[$wordCounter];
106 106
             if ($wordCounter < $count - 1) {
107 107
                 $trimed .= " ";
108
-            } else {
108
+            }else {
109 109
                 $trimed .= "[...]";
110 110
             }
111 111
         }
Please login to merge, or discard this patch.
App/Controllers/Ajax/PostVerification.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
         $postSlug = $this->request->getData("postSlug");
27 27
 
28 28
         $data = false;
29
-        if($this->isAlphaNum($postSlug))
29
+        if ($this->isAlphaNum($postSlug))
30 30
         {
31 31
             $slugModel = new SlugModel($this->container);
32 32
 
Please login to merge, or discard this patch.
App/Controllers/Ajax/ImageUpload.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -43,13 +43,13 @@  discard block
 block discarded – undo
43 43
     private function getFilename(string $folder, string $file):string
44 44
     {
45 45
 
46
-        $fileUrl = $folder . $file;
46
+        $fileUrl = $folder.$file;
47 47
         $docRoot = $this->request->getDocumentRoot();
48 48
         $filePath = $docRoot."/public/".$fileUrl;
49
-        if(file_exists($filePath) !== 1)
49
+        if (file_exists($filePath) !== 1)
50 50
         {
51 51
             $fileNum = 0;
52
-            while(file_exists($filePath))
52
+            while (file_exists($filePath))
53 53
             {
54 54
                 $fileUrl = $folder.$fileNum."_".$file;
55 55
                 $filePath = $docRoot."/public/".$fileUrl;
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
 
77 77
             // Respond to the successful upload with JSON.
78 78
             echo json_encode(array('location' => $filetowrite));
79
-        } else {
79
+        }else {
80 80
             // Notify editor that the upload failed
81 81
             echo json_encode(array('error' => 'Upload failed, file might be too big'));
82 82
 
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 
111 111
             // Respond to the successful upload with JSON.
112 112
             echo json_encode(array('location' => $filetowrite));
113
-        } else {
113
+        }else {
114 114
             // Notify editor that the upload failed
115 115
             header("HTTP/1.1 500 Server Error");
116 116
         }
Please login to merge, or discard this patch.
App/Controllers/Post.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 use Core\Controller;
9 9
 use Core\Container;
10 10
 
11
-class Post extends Controller{
11
+class Post extends Controller {
12 12
 
13 13
     protected $siteConfig;
14 14
 
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
      * @throws \Twig_Error_Runtime
27 27
      * @throws \Twig_Error_Syntax
28 28
      */
29
-    public function viewPost(string $slug){
29
+    public function viewPost(string $slug) {
30 30
 
31 31
         $tagModel = new TagModel($this->container);
32 32
         $postModel = new PostModel($this->container);
Please login to merge, or discard this patch.
App/Models/UserModel.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@
 block discarded – undo
4 4
 
5 5
 use Core\Model;
6 6
 
7
-class UserModel extends Model{
7
+class UserModel extends Model {
8 8
 
9 9
     public function getAuthorDetails(int $authorId)
10 10
     {
Please login to merge, or discard this patch.
App/Models/TagModel.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@
 block discarded – undo
67 67
         $this->query($sql);
68 68
         $this->bind(":tag", $tag);
69 69
         $this->execute();
70
-        return (int)$this->dbh->lastInsertId();
70
+        return (int) $this->dbh->lastInsertId();
71 71
 
72 72
     }
73 73
 
Please login to merge, or discard this patch.
Core/Model.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
      * fetches the result from an executed query
102 102
      * @return array
103 103
      */
104
-    protected function fetchAll(){
104
+    protected function fetchAll() {
105 105
         return $this->stmt->fetchAll();
106 106
     }
107 107
 
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
      * returns a single line from the executed query
110 110
      * @return mixed
111 111
      */
112
-    protected function fetch(){
112
+    protected function fetch() {
113 113
         return $this->stmt->fetch();
114 114
     }
115 115
 
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
             $table = $reflect->getShortName(); //this is to only get the model name, otherwise we get the full namespace
137 137
             //since our models all end with Model, we should remove it.
138 138
             $table = $this->removeFromEnd($table, 'Model');
139
-            $table = $table . 's'; //adding the s since the table should be plural. Might be some special case where the plural isn't just with an s
139
+            $table = $table.'s'; //adding the s since the table should be plural. Might be some special case where the plural isn't just with an s
140 140
             $table = strtolower($table); //the database names are in lowercase
141 141
         }
142 142
 
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
         }
156 156
 
157 157
         //if we are here, then table doesn't exist, check for view
158
-        $view = 'v_' . $table;
158
+        $view = 'v_'.$table;
159 159
         $stmt->bindValue(':table', $view, PDO::PARAM_STR);
160 160
         $stmt->execute();
161 161
         $exists = $stmt->rowCount() > 0; //will return 1 if table exists or 0 if non existant
@@ -176,8 +176,8 @@  discard block
 block discarded – undo
176 176
      * @param $table string the table name
177 177
      * @return string
178 178
      */
179
-    protected function getTablePrefix($table){
180
-        if(Config::TABLE_PREFIX != '')
179
+    protected function getTablePrefix($table) {
180
+        if (Config::TABLE_PREFIX != '')
181 181
         {
182 182
             $table = Config::TABLE_PREFIX.'_'.$table;
183 183
         }
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
     protected function getRowById($rowId, $table = null)
249 249
     {
250 250
         $tableName = $this->getTable($table);
251
-        $idName = 'id' . $tableName;
251
+        $idName = 'id'.$tableName;
252 252
         $sql = "SELECT * FROM $tableName WHERE $idName = :rowId";
253 253
         $this->query($sql);
254 254
         $this->bind(':rowId', $rowId);
Please login to merge, or discard this patch.
App/Modules/Pagination.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
             throw new \Exception("Pagination Error", "404");
27 27
         }
28 28
         $pageNo = $this->removeFromBeginning($page, "page-");
29
-        if(!filter_var($pageNo, FILTER_VALIDATE_INT)){
29
+        if (!filter_var($pageNo, FILTER_VALIDATE_INT)) {
30 30
             throw new \Exception("Invalid page number");
31 31
         }
32 32
         $offset = ($pageNo - 1) * $rowsPerPage;
Please login to merge, or discard this patch.
App/Controllers/Admin/Post.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
         $onFrontpage = $posts["isOnFrontPage"];
117 117
         $idUser = $userSessionId;
118 118
 
119
-        if(!is_int($idUser) || $idUser === null)
119
+        if (!is_int($idUser) || $idUser === null)
120 120
         {
121 121
             throw new \Error("Invalid userID");
122 122
         }
@@ -150,10 +150,10 @@  discard block
 block discarded – undo
150 150
 
151 151
         //checking result and redirecting
152 152
         if ($postId != null) {
153
-            $this->alertBox->setAlert("Post " . $title . " Created");
154
-            $this->container->getResponse()->redirect("admin/post/modify/" . $postSlug);
153
+            $this->alertBox->setAlert("Post ".$title." Created");
154
+            $this->container->getResponse()->redirect("admin/post/modify/".$postSlug);
155 155
         }
156
-        $this->alertBox->setAlert("Error creating " . $title, "error");
156
+        $this->alertBox->setAlert("Error creating ".$title, "error");
157 157
         $this->container->getResponse()->redirect("admin/post/new");
158 158
 
159 159
     }
@@ -222,10 +222,10 @@  discard block
 block discarded – undo
222 222
 
223 223
         //checking result and redirecting
224 224
         if ($postUpdate) {
225
-            $this->alertBox->setAlert("Post " . $title . " Updated");
226
-            $this->container->getResponse()->redirect("admin/post/modify/" . $postSlug);
225
+            $this->alertBox->setAlert("Post ".$title." Updated");
226
+            $this->container->getResponse()->redirect("admin/post/modify/".$postSlug);
227 227
         }
228
-        $this->alertBox->setAlert("Error updating " . $title, "error");
229
-        $this->container->getResponse()->redirect("admin/post/modify/" . $originalPostSlug);
228
+        $this->alertBox->setAlert("Error updating ".$title, "error");
229
+        $this->container->getResponse()->redirect("admin/post/modify/".$originalPostSlug);
230 230
     }
231 231
 }
232 232
\ No newline at end of file
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -196,10 +196,12 @@
 block discarded – undo
196 196
             $this->alertBox->setAlert("empty slug not allowed", "error");
197 197
         }
198 198
 
199
-        if ($postSlug != $originalPostSlug) //if the slug has been updated
199
+        if ($postSlug != $originalPostSlug) {
200
+            //if the slug has been updated
200 201
         {
201 202
             if (!$this->slugModel->isUnique($postSlug, "posts", "posts_slug")) {
202 203
                 $error = true;
204
+        }
203 205
                 $originalPostSlug = $this->slugModel->getSlugFromId($postId, "posts", "idposts", "posts_slug");
204 206
                 $this->alertBox->setAlert("Slug not unique", "error");
205 207
             }
Please login to merge, or discard this patch.