Starbugstone / blogoc

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;

App/Modules/SendMail.php

Spacing +4 added lines, -4 removed lines

@@ -8,7 +8,7 @@ 
 use Swift_Message;
 use Swift_SmtpTransport;
 
-class SendMail extends Module{
+class SendMail extends Module {
 
     private $siteConfig;
 
@@ -24,7 +24,7 @@ 
 
         // Create the Transport for mail sending
         //$config = $this->siteConfig->getSiteConfig();
-        $this->transport = (new Swift_SmtpTransport($this->siteConfig["SMTP_server"], (int)$this->siteConfig["SMTP_port"]))
+        $this->transport = (new Swift_SmtpTransport($this->siteConfig["SMTP_server"], (int) $this->siteConfig["SMTP_port"]))
             ->setUsername($this->siteConfig["SMTP_user"])
             ->setPassword($this->siteConfig["SMTP_pass"])
         ;
@@ -69,7 +69,7 @@ 
         $message = "<h1>Message from <a href='".$this->container->getRequest()->getBaseUrl()."'>".$this->siteConfig["site_name"]."</a></h1>";
         $message .= "<p>You have asked to reset your password, please click <a href=\"".$url."\">Here</a> to define a new password</p>";
 
-        $this->send($to, "Define New Password", $message );
+        $this->send($to, "Define New Password", $message);
 
     }
 
@@ -89,7 +89,7 @@ 
         $message .= "<p>You have sucsessfuly created an account, now all you need to do is <a href=\"".$url."\">Create your new password</a></p>";
         $message .= "<p>Have fun</p>";
 
-        $this->send($to, "Define Password at ".$this->siteConfig["site_name"], $message );
+        $this->send($to, "Define Password at ".$this->siteConfig["site_name"], $message);
 
     }
 }
\ No newline at end of file