Starbugstone / blogoc

App/Controllers/Ajax/postModification.php

Spacing +4 added lines, -4 removed lines

@@ -26,8 +26,8 @@ 
     {
         $this->onlyAdmin();
         $this->onlyPost();
-        $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);
@@ -44,8 +44,8 @@ 
     {
         $this->onlyAdmin();
         $this->onlyPost();
-        $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->setOnFrontPage(!$state, $postId);

Core/Traits/StringFunctions.php

Spacing +5 added lines, -5 removed lines

@@ -83,7 +83,7 @@ 
 
         //Searching for the page break tag
         $breakTagPosition = strpos($text, "<!-- EndOfExcerptBlogOc -->");
-        if($breakTagPosition > 0){
+        if ($breakTagPosition > 0) {
             return $this->completeDom(substr($text, 0, $breakTagPosition));
         }
 
@@ -91,7 +91,7 @@ 
         $string = preg_split('/(<img[^>]+\>)|(<p[^>]+\>)|(<span[^>]+\>)|\s/', $text, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE);
 
         //The preg split can return false, probably will never happen but just in case.
-        if(!$string)
+        if (!$string)
         {
             throw new \Error("excerpt generation failed");
         }
@@ -104,7 +104,7 @@ 
             $trimed .= $string[$wordCounter];
             if ($wordCounter < $count - 1) {
                 $trimed .= " ";
-            } else {
+            }else {
                 $trimed .= "[...]";
             }
         }
@@ -155,7 +155,7 @@ 
         $result = true;
         foreach ($strings as $string)
         {
-            if(!$this->isAlphaNum($string))
+            if (!$this->isAlphaNum($string))
             {
                 $result = false;
             }
@@ -170,7 +170,7 @@ 
      */
     public function isHexa(string $string):bool
     {
-        return preg_match("/[\da-f]/",$string);
+        return preg_match("/[\da-f]/", $string);
     }
 
     /**

Core/Dependency/Cookie.php

Spacing +2 added lines, -2 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace Core\Dependency;
 
-class Cookie{
+class Cookie {
 
     /**
      * set a cookie
@@ -21,7 +21,7 @@ 
      */
     public function deleteCookie(string $name):void
     {
-        setcookie($name, "", time()-3600); //expire the cookie
+        setcookie($name, "", time() - 3600); //expire the cookie
     }
 
     /**

App/Controllers/Login.php

Spacing +10 added lines, -10 removed lines

@@ -76,7 +76,7 @@ 
     private function populateUserFromId(int $userId)
     {
         $result = $this->userModel->getUserDetailsById($userId);
-        $this->populateUser((array)$result);
+        $this->populateUser((array) $result);
     }
 
     /**
@@ -97,7 +97,7 @@ 
      */
     public function index()
     {
-        if($this->session->isParamSet("user"))
+        if ($this->session->isParamSet("user"))
         {
             //we are already connected, redirect
             $this->response->redirect();
@@ -126,7 +126,7 @@ 
      */
     public function register()
     {
-        if($this->session->isParamSet("user"))
+        if ($this->session->isParamSet("user"))
         {
             //we are already connected, redirect
             $this->response->redirect();
@@ -183,7 +183,7 @@ 
             }
 
             $authUser = $this->userModel->authenticateUser($email, $password);
-            if(!$authUser->success)
+            if (!$authUser->success)
             {
                 $error = true;
                 $loginErrors->global = $authUser->message;
@@ -202,14 +202,14 @@ 
         //we are authenticated here
 
         //populate the user object with returned data
-        $this->populateUser((array)$authUser->user);
+        $this->populateUser((array) $authUser->user);
 
         //if the user wanted to be remembered
-        if($rememberMe)
+        if ($rememberMe)
         {
             $this->rememberedLoginModel->setToken(); //generate a new token
             $rememberMeToken = $this->rememberedLoginModel->rememberMe($this->user->idusers);
-            if($rememberMeToken->success)
+            if ($rememberMeToken->success)
             {
                 //set cookie
                 $this->cookie->setCookie("rememberMe", $rememberMeToken->token, $rememberMeToken->expiry_timestamp);
@@ -292,10 +292,10 @@ 
         $baseUrl = $this->request->getBaseUrl();
         $redirectUrl = $this->removeFromBeginning($refererUrl, $baseUrl);
 
-        if($redirectUrl === "login/register")
+        if ($redirectUrl === "login/register")
         {
             //if we were already on the register page, go to home page
-            $redirectUrl="";
+            $redirectUrl = "";
         }
 
         $this->alertBox->setAlert('Account created, please check your mailbox to activate account');
@@ -308,7 +308,7 @@ 
     public function disconnect()
     {
         $userId = $this->session->get("userId");
-        if($userId)
+        if ($userId)
         {
             $userHash = $this->rememberedLoginModel->getTokenHashFromId($userId);
             $this->rememberedLoginModel->deleteToken($userHash);

App/Models/RoleModel.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 use Core\Container;
 use Core\Model;
 
-class RoleModel extends Model{
+class RoleModel extends Model {
 
     private $roleTbl;
 

Core/Modules/Debug.php

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 use Core\Constant;
 use \Core\Container;
 
-class Debug extends Module{
+class Debug extends Module {
 
     /**
      * Var dump a var

App/Models/UserModel.php

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;
         }
 

App/Controllers/Admin/Home.php

Spacing +6 added lines, -6 removed lines

@@ -96,14 +96,14 @@ 
         $this->onlyUser();
         $this->onlyPost();
 
-        $user = (object)$this->request->getDataFull();
+        $user = (object) $this->request->getDataFull();
         $redirectUrl = "/admin";
 
         if ($user->userId !== $this->session->get("userId") || isset($user->userRoleSelector) || isset($user->locked_out)) {
             //an admin is trying to update a user or form tampered with
             $this->onlyAdmin();
-            $redirectUrl = "/admin/home/view-user/" . $user->userId;
-        } else {
+            $redirectUrl = "/admin/home/view-user/".$user->userId;
+        }else {
             //set the role to the original state for update
             $beforeUser = $this->userModel->getUserDetailsById($user->userId);
             $user->userRoleSelector = $beforeUser->roles_idroles;
@@ -117,13 +117,13 @@ 
         $error = false;
         $registerErrors = new \stdClass();
 
-        if($userId == 1 && $user->userLockedOut == 1)
+        if ($userId == 1 && $user->userLockedOut == 1)
         {
             $error = true;
             $this->alertBox->setAlert("Original admin may not be deactivated", "error");
         }
 
-        if($userId == 1 && $user->userRoleSelector != 2)
+        if ($userId == 1 && $user->userRoleSelector != 2)
         {
             $error = true;
             $this->alertBox->setAlert("Original admin must stay admin", "error");
@@ -204,7 +204,7 @@ 
             throw new \Exception("Error in passed ID");
         }
 
-        if($userId === 1)
+        if ($userId === 1)
         {
             $this->alertBox->setAlert('Original Admin can not be deleted', "error");
             $this->response->redirect("/admin/home/list-users");

App/Controllers/Ajax/User.php

Spacing +5 added lines, -5 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");
         }
@@ -46,8 +46,8 @@ 
     {
         $this->onlyAdmin();
         $this->onlyPost();
-        $state = (bool)($this->request->getData("state") === 'true');
-        $userId = (int)$this->request->getData("userId");
+        $state = (bool) ($this->request->getData("state") === 'true');
+        $userId = (int) $this->request->getData("userId");
 
         $result = array();
         $result["success"] = false;
@@ -55,7 +55,7 @@ 
         $result["userId"] = $userId;
 
         // we can not update the Original Admin activation state
-        if($userId !== 1)
+        if ($userId !== 1)
         {
             $result["success"] = $this->userModel->activateUser(!$state, $userId);
             $result["state"] = !$state;