Passed
Push — Accessing-and-setting-the-site... ( 48995f...ec05f1 )
by Stone
04:16
created
Core/AdminController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,8 +24,8 @@
 block discarded – undo
24 24
 
25 25
     }
26 26
 
27
-    protected function onlyAdmin(){
28
-        if(!$this->auth->isAdmin()){
27
+    protected function onlyAdmin() {
28
+        if (!$this->auth->isAdmin()) {
29 29
             $this->alertBox->setAlert("Only admins can access this", 'error');
30 30
             $this->container->getResponse()->redirect('/admin');
31 31
         }
Please login to merge, or discard this patch.
Core/Dependency/Response.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
             default:
24 24
                 $headerType = 'text/html';
25 25
         }
26
-        $contentType = 'Content-Type: ' . $headerType;
26
+        $contentType = 'Content-Type: '.$headerType;
27 27
 
28 28
         header($contentType);
29 29
     }
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
             }
42 42
         }
43 43
 
44
-        header("location: /" . $url);
44
+        header("location: /".$url);
45 45
         exit(); //after redirect do not execute anything else from the function
46 46
     }
47 47
 }
48 48
\ No newline at end of file
Please login to merge, or discard this patch.
Core/Controller.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -84,12 +84,12 @@  discard block
 block discarded – undo
84 84
         $loadModuleObj = $this->getModuleNamespace($loadModule);
85 85
         //Modules must be children of the Module template
86 86
         if (!is_subclass_of($loadModuleObj, 'Core\Modules\Module')) {
87
-            throw new \ErrorException('Module ' . $loadModuleName . ' must be a sub class of module');
87
+            throw new \ErrorException('Module '.$loadModuleName.' must be a sub class of module');
88 88
         }
89 89
         $loadedModule = new $loadModuleObj($this->container);
90 90
         //we are not allowed to create public modules, they must be a placeholder ready
91 91
         if (!property_exists($this, $loadModuleName)) {
92
-            throw new \ErrorException('the protected or private variable of ' . $loadModuleName . ' is not present');
92
+            throw new \ErrorException('the protected or private variable of '.$loadModuleName.' is not present');
93 93
         }
94 94
         $this->$loadModuleName = $loadedModule;
95 95
     }
@@ -106,23 +106,23 @@  discard block
 block discarded – undo
106 106
         $childClass = new \ReflectionClass(get_class($this));
107 107
         $childClassNamespace = $childClass->getNamespaceName();
108 108
         //check in classNameSpace
109
-        if (class_exists($childClassNamespace . '\\Modules\\' . $loadModule)) {
110
-            $this->addToDevHelper('module ' . $loadModule . ' loaded', $childClassNamespace . '\\' . $loadModule);
111
-            return $childClassNamespace . '\\' . $loadModule;
109
+        if (class_exists($childClassNamespace.'\\Modules\\'.$loadModule)) {
110
+            $this->addToDevHelper('module '.$loadModule.' loaded', $childClassNamespace.'\\'.$loadModule);
111
+            return $childClassNamespace.'\\'.$loadModule;
112 112
         }
113 113
         //check in app
114
-        if (class_exists('App\\Modules\\' . $loadModule)) {
115
-            $this->addToDevHelper('module ' . $loadModule . ' loaded', 'App\\Modules\\' . $loadModule);
116
-            return 'App\\Modules\\' . $loadModule;
114
+        if (class_exists('App\\Modules\\'.$loadModule)) {
115
+            $this->addToDevHelper('module '.$loadModule.' loaded', 'App\\Modules\\'.$loadModule);
116
+            return 'App\\Modules\\'.$loadModule;
117 117
         }
118 118
         //check in core, send error popup if overcharged
119
-        if (class_exists('Core\\Modules\\' . $loadModule)) {
120
-            $this->addToDevHelper('module ' . $loadModule . ' loaded', 'Core\\Modules\\' . $loadModule);
121
-            return 'Core\\Modules\\' . $loadModule;
119
+        if (class_exists('Core\\Modules\\'.$loadModule)) {
120
+            $this->addToDevHelper('module '.$loadModule.' loaded', 'Core\\Modules\\'.$loadModule);
121
+            return 'Core\\Modules\\'.$loadModule;
122 122
         }
123 123
 
124 124
         //if we are here then no module found
125
-        throw new \ErrorException('module ' . $loadModule . ' does not exist or not loaded');
125
+        throw new \ErrorException('module '.$loadModule.' does not exist or not loaded');
126 126
 
127 127
     }
128 128
 
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
     public function getView($template)
148 148
     {
149 149
         $twig = $this->container->getTemplate();
150
-        return $twig->render($template . '.twig', $this->data);
150
+        return $twig->render($template.'.twig', $this->data);
151 151
     }
152 152
 
153 153
     /**
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
             $this->devHelper();
170 170
         }
171 171
         $twig = $this->container->getTemplate();
172
-        $twig->display($template . '.twig', $this->data);
172
+        $twig->display($template.'.twig', $this->data);
173 173
     }
174 174
 
175 175
     /**
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
     protected function addToDevHelper($name, $var)
202 202
     {
203 203
         //only populate if in dev environment
204
-        if (Config::DEV_ENVIRONMENT){
204
+        if (Config::DEV_ENVIRONMENT) {
205 205
             $classMethods = [];
206 206
             $classMethods[$name] = $var;
207 207
             if (!isset($this->data['dev_info'])) {
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
@@ -91,7 +91,7 @@
 block discarded – undo
91 91
      * return the entire session superglobal. mainly for debugging purposes
92 92
      * @return mixed
93 93
      */
94
-    public function getAllSessionVars(){
94
+    public function getAllSessionVars() {
95 95
         return $_SESSION;
96 96
     }
97 97
 
Please login to merge, or discard this patch.
Core/Model.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
      * fetches the result from an executed query
100 100
      * @return array
101 101
      */
102
-    protected function fetchAll(){
102
+    protected function fetchAll() {
103 103
         return $this->stmt->fetchAll();
104 104
     }
105 105
 
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
      * returns a single line from the executed query
108 108
      * @return mixed
109 109
      */
110
-    protected function fetch(){
110
+    protected function fetch() {
111 111
         return $this->stmt->fetch();
112 112
     }
113 113
 
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
         if ($table === null) {
133 133
             $reflect = new \ReflectionClass(get_class($this));
134 134
             $table = $reflect->getShortName(); //this is to only get the model name, otherwise we get the full namespace
135
-            $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
135
+            $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
136 136
             $table = strtolower($table); //the database names are in lowercase
137 137
         }
138 138
 
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
         }
152 152
 
153 153
         //if we are here, then table doesn't exist, check for view
154
-        $view = 'v_' . $table;
154
+        $view = 'v_'.$table;
155 155
         $stmt->bindValue(':table', $view, PDO::PARAM_STR);
156 156
         $stmt->execute();
157 157
         $exists = $stmt->rowCount() > 0; //will return 1 if table exists or 0 if non existant
@@ -172,8 +172,8 @@  discard block
 block discarded – undo
172 172
      * @param $table string the table name
173 173
      * @return string
174 174
      */
175
-    protected function getTablePrefix($table){
176
-        if(Config::TABLE_PREFIX != '')
175
+    protected function getTablePrefix($table) {
176
+        if (Config::TABLE_PREFIX != '')
177 177
         {
178 178
             $table = Config::TABLE_PREFIX.'_'.$table;
179 179
         }
@@ -244,7 +244,7 @@  discard block
 block discarded – undo
244 244
     protected function getRowById($rowId, $table = ''): array
245 245
     {
246 246
         $tableName = $this->getTable($table);
247
-        $idName = 'id' . $tableName;
247
+        $idName = 'id'.$tableName;
248 248
         $sql = "SELECT * FROM $tableName WHERE $idName = :rowId";
249 249
         $this->query($sql);
250 250
         $this->bind(':rowId', $rowId);
Please login to merge, or discard this patch.