Passed
Push — Auth ( 6598c8...9ea218 )
by Stone
02:12
created
App/Controllers/Ajax/postModification.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,8 +28,8 @@  discard block
 block discarded – undo
28 28
         if (!$this->container->getRequest()->isPost()) {
29 29
             throw new \Core\JsonException('Call is not post');
30 30
         }
31
-        $state = (bool)($this->request->getData("state") === 'true');
32
-        $postId = (int)$this->request->getData("postId");
31
+        $state = (bool) ($this->request->getData("state") === 'true');
32
+        $postId = (int) $this->request->getData("postId");
33 33
 
34 34
         $result = array();
35 35
         $result["success"] = $this->postModule->setPublished(!$state, $postId);
@@ -48,8 +48,8 @@  discard block
 block discarded – undo
48 48
         if (!$this->container->getRequest()->isPost()) {
49 49
             throw new \Core\JsonException('Call is not post');
50 50
         }
51
-        $state = (bool)($this->request->getData("state") === 'true'); //TODO test this element
52
-        $postId = (int)$this->request->getData("postId");
51
+        $state = (bool) ($this->request->getData("state") === 'true'); //TODO test this element
52
+        $postId = (int) $this->request->getData("postId");
53 53
 
54 54
         $result = array();
55 55
         $result["success"] = $this->postModule->setOnFrontPage(!$state, $postId);
Please login to merge, or discard this patch.
Core/Dependency/Session.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@
 block discarded – undo
92 92
      * return the entire session superglobal.
93 93
      * @return mixed
94 94
      */
95
-    public function getAllSessionVars(){
95
+    public function getAllSessionVars() {
96 96
         return $_SESSION;
97 97
     }
98 98
 
Please login to merge, or discard this patch.
App/Controllers/Ajax/ImageUpload.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -44,24 +44,24 @@  discard block
 block discarded – undo
44 44
     private function getFilename(string $folder, string $file): string
45 45
     {
46 46
         //slugify the file name to avoid security errors or bugs with special characters.
47
-        $fileName = pathinfo($file, PATHINFO_FILENAME );
48
-        $fileExtension = pathinfo($file, PATHINFO_EXTENSION );
47
+        $fileName = pathinfo($file, PATHINFO_FILENAME);
48
+        $fileExtension = pathinfo($file, PATHINFO_EXTENSION);
49 49
         $slugify = new Slugify();
50 50
         $fileName = $slugify->slugify($fileName);
51 51
         //if the filename has only special chars, the slugify will be empty, create a unique ID
52
-        if($fileName ==="")
52
+        if ($fileName === "")
53 53
         {
54 54
             $fileName = uniqid();
55 55
         }
56 56
         $file = $fileName.".".$fileExtension;
57
-        $fileUrl = $folder . $file;
57
+        $fileUrl = $folder.$file;
58 58
         $docRoot = $this->request->getDocumentRoot();
59
-        $filePath = $docRoot . "/public/" . $fileUrl;
59
+        $filePath = $docRoot."/public/".$fileUrl;
60 60
         if (file_exists($filePath) !== 1) {
61 61
             $fileNum = 0;
62 62
             while (file_exists($filePath)) {
63
-                $fileUrl = $folder . $fileNum . "_" . $file;
64
-                $filePath = $docRoot . "/public/" . $fileUrl;
63
+                $fileUrl = $folder.$fileNum."_".$file;
64
+                $filePath = $docRoot."/public/".$fileUrl;
65 65
                 $fileNum += 1;
66 66
             }
67 67
         }
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 
86 86
             // Respond to the successful upload with JSON.
87 87
             echo json_encode(array('location' => $filetowrite));
88
-        } else {
88
+        }else {
89 89
             // Notify editor that the upload failed
90 90
             echo json_encode(array('error' => 'Upload failed, file might be too big'));
91 91
 
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 
120 120
             // Respond to the successful upload with JSON.
121 121
             echo json_encode(array('location' => $filetowrite));
122
-        } else {
122
+        }else {
123 123
             // Notify editor that the upload failed
124 124
             header("HTTP/1.1 500 Server Error");
125 125
         }
Please login to merge, or discard this patch.
App/Models/UserModel.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -204,9 +204,11 @@
 block discarded – undo
204 204
 
205 205
         $user = $this->getUserDetailsByEmail($email);
206 206
 
207
-        if ($user === false) //no user exists
207
+        if ($user === false) {
208
+            //no user exists
208 209
         {
209 210
             $response->message = "email doesn't exist, register a new account?";
211
+        }
210 212
             return $response;
211 213
         }
212 214
 
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
         $user = $this->fetch();
147 147
         $linkValidTime = strtotime($user->reset_password_hash_generation_datetime);
148 148
         $currentTime = time();
149
-        if($currentTime-$linkValidTime > Constant::PASSWORD_RESET_DURATION*60)
149
+        if ($currentTime - $linkValidTime > Constant::PASSWORD_RESET_DURATION * 60)
150 150
         {
151 151
             //token is no longer valid
152 152
             return false;
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
         //check if email is valid for sanity
167 167
         if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
168 168
             $email = htmlspecialchars($email);
169
-            throw new BlogocException("invalid email " . $email);
169
+            throw new BlogocException("invalid email ".$email);
170 170
         }
171 171
         $sql = $this->baseSqlSelect();
172 172
         $sql .= "
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
         $this->bind(':roles_idroles', 1);
215 215
         $this->execute();
216 216
 
217
-        return (int)$this->dbh->lastInsertId();
217
+        return (int) $this->dbh->lastInsertId();
218 218
     }
219 219
 
220 220
     /**
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
         }
246 246
 
247 247
         if ($this->isAccountPasswordBlocked($user)) {
248
-            $response->message = "too many bad passwords, account is blocked for " . Constant::LOCKOUT_MINUTES . " minutes";
248
+            $response->message = "too many bad passwords, account is blocked for ".Constant::LOCKOUT_MINUTES." minutes";
249 249
             return $response;
250 250
         }
251 251
 
Please login to merge, or discard this patch.
Core/Dependency/Cookie.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -2,7 +2,7 @@  discard block
 block discarded – undo
2 2
 
3 3
 namespace Core\Dependency;
4 4
 
5
-class Cookie{
5
+class Cookie {
6 6
 
7 7
     /**
8 8
      * set a cookie
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
      */
22 22
     public function deleteCookie(string $name)
23 23
     {
24
-        setcookie($name, "", time()-3600); //expire the cookie
24
+        setcookie($name, "", time() - 3600); //expire the cookie
25 25
     }
26 26
 
27 27
     /**
Please login to merge, or discard this patch.
Core/Container.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -50,10 +50,10 @@  discard block
 block discarded – undo
50 50
         $twigOptions = [];
51 51
         if (!Config::DEV_ENVIRONMENT) {
52 52
             $twigOptions = [
53
-                'cache' => dirname(__DIR__) . '/Cache'
53
+                'cache' => dirname(__DIR__).'/Cache'
54 54
             ];
55 55
         }
56
-        $loader = new \Twig_Loader_Filesystem(dirname(__DIR__) . '/App/Views');
56
+        $loader = new \Twig_Loader_Filesystem(dirname(__DIR__).'/App/Views');
57 57
         $twig = new \Twig_Environment($loader, $twigOptions);
58 58
 
59 59
         return $twig;
@@ -68,13 +68,13 @@  discard block
 block discarded – undo
68 68
         if ($this->dbh) {
69 69
             return $this->dbh;
70 70
         }
71
-        $dsn = "mysql:host=" . Config::DB_HOST . ";dbname=" . Config::DB_NAME . ";charset=utf8"; //Creating the Data Source name
71
+        $dsn = "mysql:host=".Config::DB_HOST.";dbname=".Config::DB_NAME.";charset=utf8"; //Creating the Data Source name
72 72
         $opt = [
73 73
             PDO::ATTR_PERSISTENT => true,
74 74
             PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
75 75
             PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_OBJ
76 76
         ];
77
-        $this->dbh = new PDO($dsn, Config::DB_USER, Config::DB_PASSWORD, $opt);;
77
+        $this->dbh = new PDO($dsn, Config::DB_USER, Config::DB_PASSWORD, $opt); ;
78 78
         return $this->dbh;
79 79
     }
80 80
 
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
      */
121 121
     public function getCookie(): Dependency\Cookie
122 122
     {
123
-        if(!$this->cookie){
123
+        if (!$this->cookie) {
124 124
             $this->cookie = new Cookie();
125 125
         }
126 126
         return $this->cookie;
Please login to merge, or discard this patch.
App/Modules/SiteConfig.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
         $categories = $categoryModel->getCategories();
49 49
         foreach ($categories as $category) {
50 50
             $data += [
51
-                $category->category_name => '/category/posts/' . $category->categories_slug
51
+                $category->category_name => '/category/posts/'.$category->categories_slug
52 52
             ];
53 53
         }
54 54
         return $data;
@@ -66,11 +66,11 @@  discard block
 block discarded – undo
66 66
         $session = $this->container->getSession();
67 67
 
68 68
         $userToken = $cookie->getCookie("rememberMe");
69
-        if($userToken)
69
+        if ($userToken)
70 70
         {
71 71
             //we have a rememberMe Hash, login
72 72
             $rememberedLogin = $rememberedLoginModel->findByToken($userToken);
73
-            if($rememberedLogin){
73
+            if ($rememberedLogin) {
74 74
                 //we have a hash, login
75 75
                 $user = $userModel->getUserDetailsById($rememberedLogin->users_idusers);
76 76
                 $session->regenerateSessionId(); //regenerate the ID to avoid session ghosting
Please login to merge, or discard this patch.
App/Models/Remembered_loginModel.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@
 block discarded – undo
27 27
     {
28 28
         if ($token_value) {
29 29
             $this->token = $token_value;
30
-        } else {
30
+        }else {
31 31
             $this->token = $this->generateToken();
32 32
         }
33 33
     }
Please login to merge, or discard this patch.
App/Controllers/Login.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
     private function populateUserFromId(int $userId)
80 80
     {
81 81
         $result = $this->userModel->getUserDetailsById($userId);
82
-        $this->populateUser((array)$result);
82
+        $this->populateUser((array) $result);
83 83
     }
84 84
 
85 85
     /**
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
      */
101 101
     public function index()
102 102
     {
103
-        if($this->session->isParamSet("user"))
103
+        if ($this->session->isParamSet("user"))
104 104
         {
105 105
             //we are already connected, redirect
106 106
             $this->response->redirect();
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
      */
130 130
     public function register()
131 131
     {
132
-        if($this->session->isParamSet("user"))
132
+        if ($this->session->isParamSet("user"))
133 133
         {
134 134
             //we are already connected, redirect
135 135
             $this->response->redirect();
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
             }
190 190
 
191 191
             $authUser = $this->userModel->authenticateUser($email, $password);
192
-            if(!$authUser->success)
192
+            if (!$authUser->success)
193 193
             {
194 194
                 $error = true;
195 195
                 $loginErrors->global = $authUser->message;
@@ -208,14 +208,14 @@  discard block
 block discarded – undo
208 208
         //we are authenticated here
209 209
 
210 210
         //populate the user object with returned data
211
-        $this->populateUser((array)$authUser->user);
211
+        $this->populateUser((array) $authUser->user);
212 212
 
213 213
         //if the user wanted to be remembered
214
-        if($rememberMe)
214
+        if ($rememberMe)
215 215
         {
216 216
             $this->rememberedLoginModel->setToken(); //generate a new token
217 217
             $rememberMeToken = $this->rememberedLoginModel->rememberMe($this->user->idusers);
218
-            if($rememberMeToken->success)
218
+            if ($rememberMeToken->success)
219 219
             {
220 220
                 //set cookie
221 221
                 $this->cookie->setCookie("rememberMe", $rememberMeToken->token, $rememberMeToken->expiry_timestamp);
Please login to merge, or discard this patch.