Passed
Pull Request — master (#57)
by Stone
03:27
created
App/Controllers/Admin/Config.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@
 block discarded – undo
36 36
         }
37 37
         if ($success) {
38 38
             $this->alertBox->setAlert('Configuration updates successfully');
39
-        } else {
39
+        }else {
40 40
             $this->alertBox->setAlert('error in configuration update', 'error');
41 41
         }
42 42
         $this->container->getResponse()->redirect('admin/config');
Please login to merge, or discard this patch.
App/Controllers/Ajax/Config.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
             if (!$configModel->updateConfig($update->name, $update->value)) {
39 39
                 $result['success'] = false;
40 40
                 $result['errorId'][] = $update->name;
41
-            } else {
41
+            }else {
42 42
                 $result['successId'][] = $update->name;
43 43
             }
44 44
         }
Please login to merge, or discard this patch.
App/Models/PostModel.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
         $this->query($sql);
88 88
         if ($select != null) {
89 89
             foreach ($select as $col => $val) {
90
-                $this->bind(":" . $col, $val);
90
+                $this->bind(":".$col, $val);
91 91
             }
92 92
         }
93 93
         $this->bind(":limit", $limit);
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
         $this->query($sql);
122 122
         if ($select != null) {
123 123
             foreach ($select as $col => $val) {
124
-                $this->bind(":" . $col, $val);
124
+                $this->bind(":".$col, $val);
125 125
             }
126 126
         }
127 127
         $this->execute();
@@ -291,7 +291,7 @@  discard block
 block discarded – undo
291 291
 
292 292
         $this->execute();
293 293
 
294
-        return (int)$this->dbh->lastInsertId();
294
+        return (int) $this->dbh->lastInsertId();
295 295
     }
296 296
 
297 297
     /**
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   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
             $table = $reflect->getShortName(); //this is to only get the model name, otherwise we get the full namespace
140 140
             //since our models all end with Model, we should remove it.
141 141
             $table = $this->removeFromEnd($table, 'Model');
142
-            $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
142
+            $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
143 143
             $table = strtolower($table); //the database names are in lowercase
144 144
         }
145 145
 
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
         }
163 163
 
164 164
         //if we are here, then table doesn't exist, check for view
165
-        $view = 'v_' . $table;
165
+        $view = 'v_'.$table;
166 166
         $stmt->bindValue(':table', $view, PDO::PARAM_STR);
167 167
         $stmt->execute();
168 168
         $exists = $stmt->rowCount() > 0; //will return 1 if table exists or 0 if non existant
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
     protected function getTablePrefix($table)
187 187
     {
188 188
         if (Config::TABLE_PREFIX != '') {
189
-            $table = Config::TABLE_PREFIX . '_' . $table;
189
+            $table = Config::TABLE_PREFIX.'_'.$table;
190 190
         }
191 191
         return $table;
192 192
     }
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
     protected function getRowById($rowId, $table = null)
256 256
     {
257 257
         $tableName = $this->getTable($table);
258
-        $idName = 'id' . $tableName;
258
+        $idName = 'id'.$tableName;
259 259
         $sql = "SELECT * FROM $tableName WHERE $idName = :rowId";
260 260
         $this->query($sql);
261 261
         $this->bind(':rowId', $rowId);
Please login to merge, or discard this patch.
App/Controllers/Admin/Post.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -211,10 +211,12 @@
 block discarded – undo
211 211
             $this->alertBox->setAlert("empty slug not allowed", "error");
212 212
         }
213 213
 
214
-        if ($postSlug != $originalPostSlug) //if the slug has been updated
214
+        if ($postSlug != $originalPostSlug) {
215
+            //if the slug has been updated
215 216
         {
216 217
             if (!$this->postModel->isPostSlugUnique($postSlug)) {
217 218
                 $error = true;
219
+        }
218 220
                 $originalPostSlug = $this->postModel->getPostSlugFromId($postId);
219 221
                 $this->alertBox->setAlert("Slug not unique", "error");
220 222
             }
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -166,10 +166,10 @@  discard block
 block discarded – undo
166 166
 
167 167
         //checking result and redirecting
168 168
         if ($postId != null) {
169
-            $this->alertBox->setAlert("Post " . $title . " Created");
170
-            $this->container->getResponse()->redirect("admin/post/modify/" . $postSlug);
169
+            $this->alertBox->setAlert("Post ".$title." Created");
170
+            $this->container->getResponse()->redirect("admin/post/modify/".$postSlug);
171 171
         }
172
-        $this->alertBox->setAlert("Error creating " . $title, "error");
172
+        $this->alertBox->setAlert("Error creating ".$title, "error");
173 173
         $this->container->getResponse()->redirect("admin/post/new");
174 174
 
175 175
     }
@@ -237,11 +237,11 @@  discard block
 block discarded – undo
237 237
 
238 238
         //checking result and redirecting
239 239
         if ($postUpdate) {
240
-            $this->alertBox->setAlert("Post " . $title . " Updated");
241
-            $this->container->getResponse()->redirect("admin/post/modify/" . $postSlug);
240
+            $this->alertBox->setAlert("Post ".$title." Updated");
241
+            $this->container->getResponse()->redirect("admin/post/modify/".$postSlug);
242 242
         }
243
-        $this->alertBox->setAlert("Error updating " . $title, "error");
244
-        $this->container->getResponse()->redirect("admin/post/modify/" . $originalPostSlug);
243
+        $this->alertBox->setAlert("Error updating ".$title, "error");
244
+        $this->container->getResponse()->redirect("admin/post/modify/".$originalPostSlug);
245 245
     }
246 246
 
247 247
 
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
         $removedPost = $this->postModel->deletePost($postId);
259 259
 
260 260
         if ($removedPost) {
261
-            $this->alertBox->setAlert("Post " . $postTitle . " deleted");
261
+            $this->alertBox->setAlert("Post ".$postTitle." deleted");
262 262
         }
263 263
 
264 264
         $this->response->redirect("admin/post/list/");
Please login to merge, or discard this patch.
App/Controllers/Admin/Category.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -103,10 +103,10 @@  discard block
 block discarded – undo
103 103
 
104 104
         //checking result and redirecting
105 105
         if ($categoryUpdate) {
106
-            $this->alertBox->setAlert("Category " . $categoryName . " updated");
106
+            $this->alertBox->setAlert("Category ".$categoryName." updated");
107 107
             $this->container->getResponse()->redirect("/admin/category/list/");
108 108
         }
109
-        $this->alertBox->setAlert("Error updating " . $categoryName, "error");
109
+        $this->alertBox->setAlert("Error updating ".$categoryName, "error");
110 110
         $this->container->getResponse()->redirect("/admin/category/list/");
111 111
     }
112 112
 
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
         $removedCategory = $this->categoryModel->delete($categoryId);
124 124
 
125 125
         if ($removedCategory) {
126
-            $this->alertBox->setAlert("Category " . $categoryName . " deleted");
126
+            $this->alertBox->setAlert("Category ".$categoryName." deleted");
127 127
         }
128 128
 
129 129
         $this->response->redirect("/admin/category/list/");
@@ -168,10 +168,10 @@  discard block
 block discarded – undo
168 168
 
169 169
         //checking result and redirecting
170 170
         if ($categoryNew) {
171
-            $this->alertBox->setAlert("Category " . $categoryName . " created");
171
+            $this->alertBox->setAlert("Category ".$categoryName." created");
172 172
             $this->container->getResponse()->redirect("/admin/category/list/");
173 173
         }
174
-        $this->alertBox->setAlert("Error creating " . $categoryName, "error");
174
+        $this->alertBox->setAlert("Error creating ".$categoryName, "error");
175 175
         $this->container->getResponse()->redirect("/admin/category/list/");
176 176
     }
177 177
 }
178 178
\ No newline at end of file
Please login to merge, or discard this patch.
App/Controllers/Admin/Tag.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -89,10 +89,10 @@  discard block
 block discarded – undo
89 89
 
90 90
         //checking result and redirecting
91 91
         if ($tagUpdate) {
92
-            $this->alertBox->setAlert("Tag " . $tagName . " updated");
92
+            $this->alertBox->setAlert("Tag ".$tagName." updated");
93 93
             $this->response->redirect("/admin/tag/list/");
94 94
         }
95
-        $this->alertBox->setAlert("Error updating " . $tagName, "error");
95
+        $this->alertBox->setAlert("Error updating ".$tagName, "error");
96 96
         $this->response->redirect("/admin/tag/list/");
97 97
     }
98 98
 
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
         $removedTag = $this->tagModel->delete($tagId);
110 110
 
111 111
         if ($removedTag) {
112
-            $this->alertBox->setAlert("Tag " . $tagName . " deleted");
112
+            $this->alertBox->setAlert("Tag ".$tagName." deleted");
113 113
         }
114 114
 
115 115
         $this->response->redirect("/admin/tag/list/");
@@ -145,10 +145,10 @@  discard block
 block discarded – undo
145 145
 
146 146
         //checking result and redirecting
147 147
         if ($tagNew) {
148
-            $this->alertBox->setAlert("Tag " . $tagName . " created");
148
+            $this->alertBox->setAlert("Tag ".$tagName." created");
149 149
             $this->response->redirect("/admin/tag/list/");
150 150
         }
151
-        $this->alertBox->setAlert("Error creating " . $tagName, "error");
151
+        $this->alertBox->setAlert("Error creating ".$tagName, "error");
152 152
         $this->response->redirect("/admin/tag/list/");
153 153
     }
154 154
 }
155 155
\ No newline at end of file
Please login to merge, or discard this patch.
App/Controllers/Ajax/PostVerification.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,9 +36,11 @@
 block discarded – undo
36 36
 
37 37
         $data = $postModel->isPostSlugUnique($postSlug);
38 38
 
39
-        if ($data === false) //slug is not unique, but could be from the same post
39
+        if ($data === false) {
40
+            //slug is not unique, but could be from the same post
40 41
         {
41 42
             $slugOfId = $postModel->getPostSlugFromId($postId);
43
+        }
42 44
             if ($slugOfId === $postSlug) {
43 45
                 //it's the same post, return true
44 46
                 $data = true;
Please login to merge, or discard this patch.