Starbugstone / blogoc

Core/Model.php

Spacing +4 added lines, -4 removed lines

@@ -139,7 +139,7 @@ 
             $table = $reflect->getShortName(); //this is to only get the model name, otherwise we get the full namespace
             //since our models all end with Model, we should remove it.
             $table = $this->removeFromEnd($table, 'Model');
-            $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
+            $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
             $table = strtolower($table); //the database names are in lowercase
         }
 
@@ -162,7 +162,7 @@ 
         }
 
         //if we are here, then table doesn't exist, check for view
-        $view = 'v_' . $table;
+        $view = 'v_'.$table;
         $stmt->bindValue(':table', $view, PDO::PARAM_STR);
         $stmt->execute();
         $exists = $stmt->rowCount() > 0; //will return 1 if table exists or 0 if non existant
@@ -186,7 +186,7 @@ 
     protected function getTablePrefix($table)
     {
         if (Config::TABLE_PREFIX != '') {
-            $table = Config::TABLE_PREFIX . '_' . $table;
+            $table = Config::TABLE_PREFIX.'_'.$table;
         }
         return $table;
     }
@@ -255,7 +255,7 @@ 
     protected function getRowById($rowId, $table = null)
     {
         $tableName = $this->getTable($table);
-        $idName = 'id' . $tableName;
+        $idName = 'id'.$tableName;
         $sql = "SELECT * FROM $tableName WHERE $idName = :rowId";
         $this->query($sql);
         $this->bind(':rowId', $rowId);

App/Controllers/Admin/Post.php

Braces +3 added lines, -1 removed lines

@@ -211,10 +211,12 @@
             $this->alertBox->setAlert("empty slug not allowed", "error");
         }
 
-        if ($postSlug != $originalPostSlug) //if the slug has been updated
+        if ($postSlug != $originalPostSlug) {
+            //if the slug has been updated
         {
             if (!$this->postModel->isPostSlugUnique($postSlug)) {
                 $error = true;
+        }
                 $originalPostSlug = $this->postModel->getPostSlugFromId($postId);
                 $this->alertBox->setAlert("Slug not unique", "error");
             }

Spacing +8 added lines, -8 removed lines

@@ -166,10 +166,10 @@ 
 
         //checking result and redirecting
         if ($postId != null) {
-            $this->alertBox->setAlert("Post " . $title . " Created");
-            $this->container->getResponse()->redirect("admin/post/modify/" . $postSlug);
+            $this->alertBox->setAlert("Post ".$title." Created");
+            $this->container->getResponse()->redirect("admin/post/modify/".$postSlug);
         }
-        $this->alertBox->setAlert("Error creating " . $title, "error");
+        $this->alertBox->setAlert("Error creating ".$title, "error");
         $this->container->getResponse()->redirect("admin/post/new");
 
     }
@@ -237,11 +237,11 @@ 
 
         //checking result and redirecting
         if ($postUpdate) {
-            $this->alertBox->setAlert("Post " . $title . " Updated");
-            $this->container->getResponse()->redirect("admin/post/modify/" . $postSlug);
+            $this->alertBox->setAlert("Post ".$title." Updated");
+            $this->container->getResponse()->redirect("admin/post/modify/".$postSlug);
         }
-        $this->alertBox->setAlert("Error updating " . $title, "error");
-        $this->container->getResponse()->redirect("admin/post/modify/" . $originalPostSlug);
+        $this->alertBox->setAlert("Error updating ".$title, "error");
+        $this->container->getResponse()->redirect("admin/post/modify/".$originalPostSlug);
     }
 
 
@@ -258,7 +258,7 @@ 
         $removedPost = $this->postModel->deletePost($postId);
 
         if ($removedPost) {
-            $this->alertBox->setAlert("Post " . $postTitle . " deleted");
+            $this->alertBox->setAlert("Post ".$postTitle." deleted");
         }
 
         $this->response->redirect("admin/post/list/");

App/Controllers/Admin/Category.php

Spacing +5 added lines, -5 removed lines

@@ -103,10 +103,10 @@ 
 
         //checking result and redirecting
         if ($categoryUpdate) {
-            $this->alertBox->setAlert("Category " . $categoryName . " updated");
+            $this->alertBox->setAlert("Category ".$categoryName." updated");
             $this->container->getResponse()->redirect("/admin/category/list/");
         }
-        $this->alertBox->setAlert("Error updating " . $categoryName, "error");
+        $this->alertBox->setAlert("Error updating ".$categoryName, "error");
         $this->container->getResponse()->redirect("/admin/category/list/");
     }
 
@@ -123,7 +123,7 @@ 
         $removedCategory = $this->categoryModel->delete($categoryId);
 
         if ($removedCategory) {
-            $this->alertBox->setAlert("Category " . $categoryName . " deleted");
+            $this->alertBox->setAlert("Category ".$categoryName." deleted");
         }
 
         $this->response->redirect("/admin/category/list/");
@@ -168,10 +168,10 @@ 
 
         //checking result and redirecting
         if ($categoryNew) {
-            $this->alertBox->setAlert("Category " . $categoryName . " created");
+            $this->alertBox->setAlert("Category ".$categoryName." created");
             $this->container->getResponse()->redirect("/admin/category/list/");
         }
-        $this->alertBox->setAlert("Error creating " . $categoryName, "error");
+        $this->alertBox->setAlert("Error creating ".$categoryName, "error");
         $this->container->getResponse()->redirect("/admin/category/list/");
     }
 }
\ No newline at end of file

App/Controllers/Admin/Tag.php

Spacing +5 added lines, -5 removed lines

@@ -89,10 +89,10 @@ 
 
         //checking result and redirecting
         if ($tagUpdate) {
-            $this->alertBox->setAlert("Tag " . $tagName . " updated");
+            $this->alertBox->setAlert("Tag ".$tagName." updated");
             $this->response->redirect("/admin/tag/list/");
         }
-        $this->alertBox->setAlert("Error updating " . $tagName, "error");
+        $this->alertBox->setAlert("Error updating ".$tagName, "error");
         $this->response->redirect("/admin/tag/list/");
     }
 
@@ -109,7 +109,7 @@ 
         $removedTag = $this->tagModel->delete($tagId);
 
         if ($removedTag) {
-            $this->alertBox->setAlert("Tag " . $tagName . " deleted");
+            $this->alertBox->setAlert("Tag ".$tagName." deleted");
         }
 
         $this->response->redirect("/admin/tag/list/");
@@ -145,10 +145,10 @@ 
 
         //checking result and redirecting
         if ($tagNew) {
-            $this->alertBox->setAlert("Tag " . $tagName . " created");
+            $this->alertBox->setAlert("Tag ".$tagName." created");
             $this->response->redirect("/admin/tag/list/");
         }
-        $this->alertBox->setAlert("Error creating " . $tagName, "error");
+        $this->alertBox->setAlert("Error creating ".$tagName, "error");
         $this->response->redirect("/admin/tag/list/");
     }
 }
\ No newline at end of file

App/Controllers/Ajax/PostVerification.php

Braces +3 added lines, -1 removed lines

@@ -36,9 +36,11 @@
 
         $data = $postModel->isPostSlugUnique($postSlug);
 
-        if ($data === false) //slug is not unique, but could be from the same post
+        if ($data === false) {
+            //slug is not unique, but could be from the same post
         {
             $slugOfId = $postModel->getPostSlugFromId($postId);
+        }
             if ($slugOfId === $postSlug) {
                 //it's the same post, return true
                 $data = true;

Core/Modules/AlertBox.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
         //make sure we have the right type or throw an error
 
         if (!in_array($type, $this->allowedTypes)) {
-            throw new \Exception("Invalid toastr alert type " . $type);
+            throw new \Exception("Invalid toastr alert type ".$type);
         }
 
         $message = htmlspecialchars($message); //avoid any injection

App/Controllers/Ajax/ImageUpload.php

Spacing +9 added lines, -9 removed lines

@@ -44,24 +44,24 @@ 
     private function getFilename(string $folder, string $file): string
     {
         //slugify the file name to avoid security errors or bugs with special characters.
-        $fileName = pathinfo($file, PATHINFO_FILENAME );
-        $fileExtension = pathinfo($file, PATHINFO_EXTENSION );
+        $fileName = pathinfo($file, PATHINFO_FILENAME);
+        $fileExtension = pathinfo($file, PATHINFO_EXTENSION);
         $slugify = new Slugify();
         $fileName = $slugify->slugify($fileName);
         //if the filename has only special chars, the slugify will be empty, create a unique ID
-        if($fileName ==="")
+        if ($fileName === "")
         {
             $fileName = uniqid();
         }
         $file = $fileName.".".$fileExtension;
-        $fileUrl = $folder . $file;
+        $fileUrl = $folder.$file;
         $docRoot = $this->request->getDocumentRoot();
-        $filePath = $docRoot . "/public/" . $fileUrl;
+        $filePath = $docRoot."/public/".$fileUrl;
         if (file_exists($filePath) !== 1) {
             $fileNum = 0;
             while (file_exists($filePath)) {
-                $fileUrl = $folder . $fileNum . "_" . $file;
-                $filePath = $docRoot . "/public/" . $fileUrl;
+                $fileUrl = $folder.$fileNum."_".$file;
+                $filePath = $docRoot."/public/".$fileUrl;
                 $fileNum += 1;
             }
         }
@@ -85,7 +85,7 @@ 
 
             // Respond to the successful upload with JSON.
             echo json_encode(array('location' => $filetowrite));
-        } else {
+        }else {
             // Notify editor that the upload failed
             echo json_encode(array('error' => 'Upload failed, file might be too big'));
 
@@ -119,7 +119,7 @@ 
 
             // Respond to the successful upload with JSON.
             echo json_encode(array('location' => $filetowrite));
-        } else {
+        }else {
             // Notify editor that the upload failed
             header("HTTP/1.1 500 Server Error");
         }

App/Models/UserModel.php

Braces +3 added lines, -1 removed lines

@@ -204,9 +204,11 @@
 
         $user = $this->getUserDetailsByEmail($email);
 
-        if ($user === false) //no user exists
+        if ($user === false) {
+            //no user exists
         {
             $response->message = "email doesn't exist, register a new account?";
+        }
             return $response;
         }
 

Spacing +4 added lines, -4 removed lines

@@ -171,7 +171,7 @@ 
         //check if email is valid for sanity
         if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
             $email = htmlspecialchars($email);
-            throw new BlogocException("invalid email " . $email);
+            throw new BlogocException("invalid email ".$email);
         }
         $sql = $this->baseSqlSelect();
         $sql .= "
@@ -214,7 +214,7 @@ 
         $this->bind(':roles_idroles', 1); //we set to one, should probably get from database and config
         $this->execute();
 
-        return (int)$this->dbh->lastInsertId();
+        return (int) $this->dbh->lastInsertId();
     }
 
     /**
@@ -224,7 +224,7 @@ 
      */
     public function updateUser(\stdClass $user)
     {
-        $sql="
+        $sql = "
             UPDATE $this->userTbl
             SET
               name=:name,
@@ -274,7 +274,7 @@ 
         }
 
         if ($this->isAccountPasswordBlocked($user)) {
-            $response->message = "too many bad passwords, account is blocked for " . Constant::LOCKOUT_MINUTES . " minutes";
+            $response->message = "too many bad passwords, account is blocked for ".Constant::LOCKOUT_MINUTES." minutes";
             return $response;
         }
 

Core/Container.php

Spacing +5 added lines, -5 removed lines

@@ -50,10 +50,10 @@ 
         $twigOptions = [];
         if (!Config::DEV_ENVIRONMENT) {
             $twigOptions = [
-                'cache' => dirname(__DIR__) . '/Cache'
+                'cache' => dirname(__DIR__).'/Cache'
             ];
         }
-        $loader = new \Twig_Loader_Filesystem(dirname(__DIR__) . '/App/Views');
+        $loader = new \Twig_Loader_Filesystem(dirname(__DIR__).'/App/Views');
         $twig = new \Twig_Environment($loader, $twigOptions);
 
         return $twig;
@@ -68,13 +68,13 @@ 
         if ($this->dbh) {
             return $this->dbh;
         }
-        $dsn = "mysql:host=" . Config::DB_HOST . ";dbname=" . Config::DB_NAME . ";charset=utf8"; //Creating the Data Source name
+        $dsn = "mysql:host=".Config::DB_HOST.";dbname=".Config::DB_NAME.";charset=utf8"; //Creating the Data Source name
         $opt = [
             PDO::ATTR_PERSISTENT => true,
             PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
             PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_OBJ
         ];
-        $this->dbh = new PDO($dsn, Config::DB_USER, Config::DB_PASSWORD, $opt);;
+        $this->dbh = new PDO($dsn, Config::DB_USER, Config::DB_PASSWORD, $opt); ;
         return $this->dbh;
     }
 
@@ -120,7 +120,7 @@ 
      */
     public function getCookie(): Dependency\Cookie
     {
-        if(!$this->cookie){
+        if (!$this->cookie) {
             $this->cookie = new Cookie();
         }
         return $this->cookie;