Starbugstone / blogoc

App/Controllers/Ajax/ImageUpload.php

Spacing +6 added lines, -6 removed lines

@@ -43,14 +43,14 @@ 
     private function getFilename(string $folder, string $file): string
     {
 
-        $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;
             }
         }
@@ -74,7 +74,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'));
 
@@ -108,7 +108,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/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/User.php

Spacing +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
 use Core\JsonException;
 use Core\Traits\StringFunctions;
 
-class User  extends AjaxController{
+class User  extends AjaxController {
 
     use StringFunctions;
 
@@ -25,7 +25,7 @@ 
         //the router needs a parameter with get functions else throsw a wobbly
         //we pass a get variable and call the /controller/function/get?bla
         //for better use and security, we must pass "get" as the parameter
-        if(!$this->startsWith(strtolower($get),"get"))
+        if (!$this->startsWith(strtolower($get), "get"))
         {
             throw new JsonException("invalid call");
         }

App/Controllers/Login.php

Spacing +3 added lines, -3 removed lines

@@ -60,7 +60,7 @@ 
     private function populateUserFromId(int $userId)
     {
         $result = $this->userModel->getUserDetailsById($userId);
-        $this->populateUser((array)$result);
+        $this->populateUser((array) $result);
     }
 
     /**
@@ -163,7 +163,7 @@ 
         }
 
         $authUser = $this->userModel->authenticateUser($email, $password);
-        if(!$authUser)
+        if (!$authUser)
         {
             $error = true;
             $loginErrors->password = "Incorrect Password";
@@ -178,7 +178,7 @@ 
         //we are authenticated here
 
         //populate the user object with returned data
-        $this->populateUser((array)$authUser);
+        $this->populateUser((array) $authUser);
         $this->setUserSession();
 
         //if all is valid, redirect to user admin page

Core/Controller.php

Spacing +15 added lines, -15 removed lines

@@ -106,12 +106,12 @@ 
         $loadModuleObj = $this->getModuleNamespace($loadModule);
         //Modules must be children of the Module template
         if (!is_subclass_of($loadModuleObj, 'Core\Modules\Module')) {
-            throw new \ErrorException('Module ' . $loadModuleName . ' must be a sub class of module');
+            throw new \ErrorException('Module '.$loadModuleName.' must be a sub class of module');
         }
         $loadedModule = new $loadModuleObj($this->container);
         //we are not allowed to create public modules, they must be a placeholder ready
         if (!property_exists($this, $loadModuleName)) {
-            throw new \ErrorException('the protected or private variable of ' . $loadModuleName . ' is not present');
+            throw new \ErrorException('the protected or private variable of '.$loadModuleName.' is not present');
         }
         $this->$loadModuleName = $loadedModule;
     }
@@ -128,23 +128,23 @@ 
         $childClass = new \ReflectionClass(get_class($this));
         $childClassNamespace = $childClass->getNamespaceName();
         //check in classNameSpace
-        if (class_exists($childClassNamespace . '\\Modules\\' . $loadModule)) {
-            $this->addToDevHelper('module ' . $loadModule . ' loaded', $childClassNamespace . '\\' . $loadModule);
-            return $childClassNamespace . '\\' . $loadModule;
+        if (class_exists($childClassNamespace.'\\Modules\\'.$loadModule)) {
+            $this->addToDevHelper('module '.$loadModule.' loaded', $childClassNamespace.'\\'.$loadModule);
+            return $childClassNamespace.'\\'.$loadModule;
         }
         //check in app
-        if (class_exists('App\\Modules\\' . $loadModule)) {
-            $this->addToDevHelper('module ' . $loadModule . ' loaded', 'App\\Modules\\' . $loadModule);
-            return 'App\\Modules\\' . $loadModule;
+        if (class_exists('App\\Modules\\'.$loadModule)) {
+            $this->addToDevHelper('module '.$loadModule.' loaded', 'App\\Modules\\'.$loadModule);
+            return 'App\\Modules\\'.$loadModule;
         }
         //check in core, send error popup if overcharged
-        if (class_exists('Core\\Modules\\' . $loadModule)) {
-            $this->addToDevHelper('module ' . $loadModule . ' loaded', 'Core\\Modules\\' . $loadModule);
-            return 'Core\\Modules\\' . $loadModule;
+        if (class_exists('Core\\Modules\\'.$loadModule)) {
+            $this->addToDevHelper('module '.$loadModule.' loaded', 'Core\\Modules\\'.$loadModule);
+            return 'Core\\Modules\\'.$loadModule;
         }
 
         //if we are here then no module found
-        throw new \ErrorException('module ' . $loadModule . ' does not exist or not loaded');
+        throw new \ErrorException('module '.$loadModule.' does not exist or not loaded');
 
     }
 
@@ -169,7 +169,7 @@ 
     public function getView($template)
     {
         $twig = $this->container->getTemplate();
-        return $twig->render($template . '.twig', $this->data);
+        return $twig->render($template.'.twig', $this->data);
     }
 
     /**
@@ -195,7 +195,7 @@ 
         }
 
         $twig = $this->container->getTemplate();
-        $twig->display($template . '.twig', $this->data);
+        $twig->display($template.'.twig', $this->data);
     }
 
     /**
@@ -227,7 +227,7 @@ 
     protected function addToDevHelper($name, $var)
     {
         //only populate if in dev environment
-        if (Config::DEV_ENVIRONMENT){
+        if (Config::DEV_ENVIRONMENT) {
             $classMethods = [];
             $classMethods[$name] = $var;
             if (!isset($this->data['dev_info'])) {

App/Controllers/Ajax/postModification.php

Spacing +4 added lines, -4 removed lines

@@ -28,8 +28,8 @@ 
         if (!$this->container->getRequest()->isPost()) {
             throw new \Core\JsonException('Call is not post');
         }
-        $state = (bool)($this->request->getData("state") === 'true');
-        $postId = (int)$this->request->getData("postId");
+        $state = (bool) ($this->request->getData("state") === 'true');
+        $postId = (int) $this->request->getData("postId");
 
         $result = array();
         $result["success"] = $this->postModule->setPublished(!$state, $postId);
@@ -48,8 +48,8 @@ 
         if (!$this->container->getRequest()->isPost()) {
             throw new \Core\JsonException('Call is not post');
         }
-        $state = (bool)($this->request->getData("state") === 'true'); //TODO test this element
-        $postId = (int)$this->request->getData("postId");
+        $state = (bool) ($this->request->getData("state") === 'true'); //TODO test this element
+        $postId = (int) $this->request->getData("postId");
 
         $result = array();
         $result["success"] = $this->postModule->setOnFrontPage(!$state, $postId);

Core/Dependency/Session.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
      * return the entire session superglobal.
      * @return mixed
      */
-    public function getAllSessionVars(){
+    public function getAllSessionVars() {
         return $_SESSION;
     }
 

App/Models/UserModel.php

Spacing +2 added lines, -2 removed lines

@@ -68,7 +68,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 = "
             SELECT idusers, username, avatar, email, surname, name, creation_date, last_update, locked_out, role_name, role_level
@@ -124,7 +124,7 @@ 
         $this->bind(':roles_idroles', 1);
         $this->execute();
 
-        return (int)$this->dbh->lastInsertId();
+        return (int) $this->dbh->lastInsertId();
 
     }
 }
\ No newline at end of file