Passed
Push — master ( 849dc0...25ac3b )
by Fran
03:04
created
src/base/extension/traits/CssTrait.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -32,8 +32,8 @@  discard block
 block discarded – undo
32 32
     protected function compileCss($basePath, $hash)
33 33
     {
34 34
         $debug = Config::getParam('debug');
35
-        $base = $basePath . "css" . DIRECTORY_SEPARATOR;
36
-        if ($debug || !file_exists($base . $hash . ".css")) {
35
+        $base = $basePath."css".DIRECTORY_SEPARATOR;
36
+        if ($debug || !file_exists($base.$hash.".css")) {
37 37
             $data = '';
38 38
             if (0 < count($this->files)) {
39 39
                 $minifier = new CSS();
@@ -42,14 +42,14 @@  discard block
 block discarded – undo
42 42
                 }
43 43
             }
44 44
             if ($debug) {
45
-                AssetsHelper::storeContents($base . $hash . ".css", $data);
46
-            } else {
45
+                AssetsHelper::storeContents($base.$hash.".css", $data);
46
+            }else {
47 47
                 $minifier = new CSS();
48 48
                 $minifier->add($data);
49 49
                 ini_set('max_execution_time', -1);
50 50
                 ini_set('memory_limit', -1);
51 51
                 GeneratorHelper::createDir($base);
52
-                $minifier->minify($base . $hash . ".css");
52
+                $minifier->minify($base.$hash.".css");
53 53
             }
54 54
             unset($minifier);
55 55
         }
@@ -69,21 +69,21 @@  discard block
 block discarded – undo
69 69
         if (file_exists($file)) {
70 70
             $debug = Config::getParam('debug');
71 71
             $pathParts = explode("/", $file);
72
-            $filePath = $this->hash . "_" . $pathParts[count($pathParts) - 1];
73
-            if (!file_exists($base . $filePath) || filemtime($base . $filePath) < filemtime($file) || $debug) {
72
+            $filePath = $this->hash."_".$pathParts[count($pathParts) - 1];
73
+            if (!file_exists($base.$filePath) || filemtime($base.$filePath) < filemtime($file) || $debug) {
74 74
                 //Si tenemos modificaciones tenemos que compilar de nuevo todos los ficheros modificados
75
-                if (file_exists($base . $hash . ".css") && @unlink($base . $hash . ".css") === false) {
76
-                    throw new ConfigException("Can't unlink file " . $base . $hash . ".css");
75
+                if (file_exists($base.$hash.".css") && @unlink($base.$hash.".css") === false) {
76
+                    throw new ConfigException("Can't unlink file ".$base.$hash.".css");
77 77
                 }
78 78
                 $this->loopCssLines($file);
79 79
             }
80 80
             if ($debug) {
81 81
                 $data = file_get_contents($file);
82
-                AssetsHelper::storeContents($base . $filePath, $data);
83
-            } else {
82
+                AssetsHelper::storeContents($base.$filePath, $data);
83
+            }else {
84 84
                 $data .= file_get_contents($file);
85 85
             }
86
-            $this->compiledFiles[] = "/css/" . $filePath;
86
+            $this->compiledFiles[] = "/css/".$filePath;
87 87
         }
88 88
 
89 89
         return $data;
@@ -115,25 +115,25 @@  discard block
 block discarded – undo
115 115
      */
116 116
     protected function extractCssResources($source, $file)
117 117
     {
118
-        Inspector::stats('[CssTrait] Start collecting resources from ' . $file, Inspector::SCOPE_DEBUG);
118
+        Inspector::stats('[CssTrait] Start collecting resources from '.$file, Inspector::SCOPE_DEBUG);
119 119
         $sourceFile = AssetsHelper::extractSourceFilename($source);
120
-        $orig = realpath(dirname($file) . DIRECTORY_SEPARATOR . $sourceFile);
120
+        $orig = realpath(dirname($file).DIRECTORY_SEPARATOR.$sourceFile);
121 121
         $origPart = preg_split('/(\/|\\\)public(\/|\\\)/i', $orig);
122 122
         try {
123 123
             if (count($source) > 1 && array_key_exists(1, $origPart)) {
124
-                $dest = $this->path . $origPart[1];
124
+                $dest = $this->path.$origPart[1];
125 125
                 GeneratorHelper::createDir(dirname($dest));
126 126
                 if (!file_exists($dest) || filemtime($orig) > filemtime($dest)) {
127 127
                     if (@copy($orig, $dest) === FALSE) {
128
-                        throw new \RuntimeException('Can\' copy ' . $dest . '');
128
+                        throw new \RuntimeException('Can\' copy '.$dest.'');
129 129
                     }
130 130
                     Logger::log("$orig copiado a $dest", LOG_INFO);
131 131
                 }
132 132
             }
133
-        } catch (\Exception $e) {
133
+        }catch (\Exception $e) {
134 134
             Logger::log($e->getMessage(), LOG_ERR);
135 135
         }
136
-        Inspector::stats('[CssTrait] End collecting resources from ' . $file, Inspector::SCOPE_DEBUG);
136
+        Inspector::stats('[CssTrait] End collecting resources from '.$file, Inspector::SCOPE_DEBUG);
137 137
     }
138 138
 
139 139
     /**
@@ -147,8 +147,8 @@  discard block
 block discarded – undo
147 147
             foreach ($compiledFiles as $file) {
148 148
                 echo "\t\t<link href='{$file}' rel='stylesheet' media='screen, print'>";
149 149
             }
150
-        } else {
151
-            echo "\t\t<link href='" . $baseUrl . "/css/" . $hash . ".css' rel='stylesheet' " .
150
+        }else {
151
+            echo "\t\t<link href='".$baseUrl."/css/".$hash.".css' rel='stylesheet' ".
152 152
                 //"crossorigin='anonymous' integrity='sha384-" . $sri . "'>";
153 153
                 "media='screen, print'>";
154 154
         }
Please login to merge, or discard this patch.
src/base/Logger.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -17,8 +17,8 @@  discard block
 block discarded – undo
17 17
 
18 18
 
19 19
 if (!defined('LOG_DIR')) {
20
-    GeneratorHelper::createDir(BASE_DIR . DIRECTORY_SEPARATOR . 'logs');
21
-    define('LOG_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'logs');
20
+    GeneratorHelper::createDir(BASE_DIR.DIRECTORY_SEPARATOR.'logs');
21
+    define('LOG_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'logs');
22 22
 }
23 23
 
24 24
 /**
@@ -57,10 +57,10 @@  discard block
 block discarded – undo
57 57
     {
58 58
         $args = func_get_args();
59 59
         list($logger, $debug, $path) = $this->setup($args);
60
-        $this->stream = fopen($path . DIRECTORY_SEPARATOR . date('Ymd') . '.log', 'ab+');
60
+        $this->stream = fopen($path.DIRECTORY_SEPARATOR.date('Ymd').'.log', 'ab+');
61 61
         if (false !== $this->stream && is_resource($this->stream)) {
62 62
             $this->addPushLogger($logger, $debug);
63
-        } else {
63
+        }else {
64 64
             throw new ConfigException(t('Error creating logger'));
65 65
         }
66 66
         $this->logLevel = strtoupper(Config::getParam('log.level', 'NOTICE'));
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
     private function createLoggerPath()
142 142
     {
143 143
         $logger = $this->setLoggerName();
144
-        $path = Config::getParam('default.log.path', LOG_DIR) . DIRECTORY_SEPARATOR . $logger . DIRECTORY_SEPARATOR . date('Y') . DIRECTORY_SEPARATOR . date('m');
144
+        $path = Config::getParam('default.log.path', LOG_DIR).DIRECTORY_SEPARATOR.$logger.DIRECTORY_SEPARATOR.date('Y').DIRECTORY_SEPARATOR.date('m');
145 145
         GeneratorHelper::createDir($path);
146 146
         return $path;
147 147
     }
Please login to merge, or discard this patch.
src/base/dto/Dto.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -57,14 +57,14 @@  discard block
 block discarded – undo
57 57
                             }
58 58
                         }
59 59
                         $dto[$property->getName()] = $value;
60
-                    } else {
60
+                    }else {
61 61
                         $type = InjectorHelper::extractVarType($property->getDocComment());
62 62
                         $dto[$property->getName()] = $this->checkCastedValue($property->getValue($this), $type ?: 'string');
63 63
                     }
64 64
                 }
65 65
             }
66
-        } catch (\Exception $e) {
67
-            Logger::log(get_class($this) . ': ' . $e->getMessage(), LOG_ERR);
66
+        }catch (\Exception $e) {
67
+            Logger::log(get_class($this).': '.$e->getMessage(), LOG_ERR);
68 68
         }
69 69
         return $dto;
70 70
     }
@@ -101,12 +101,12 @@  discard block
 block discarded – undo
101 101
                             $this->$key[] = $dto;
102 102
                         }
103 103
                     }
104
-                } else {
104
+                }else {
105 105
                     $this->$key = clone $this->__cache[$type];
106 106
                     $this->$key->fromArray($value);
107 107
                 }
108 108
             }
109
-        } else {
109
+        }else {
110 110
             $this->castValue($key, $value, $type);
111 111
         }
112 112
     }
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
     /**
134 134
      * @return mixed
135 135
      */
136
-    public function jsonSerialize(): array|string|null
136
+    public function jsonSerialize(): array | string | null
137 137
     {
138 138
         return $this->toArray();
139 139
     }
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
                     $value = (bool)$rawValue;
194 194
                     break;
195 195
             }
196
-        } else {
196
+        }else {
197 197
             $value = $rawValue;
198 198
         }
199 199
         return $value;
Please login to merge, or discard this patch.
src/base/dto/Form.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,11 +46,11 @@
 block discarded – undo
46 46
         foreach ($this->fields as $field) {
47 47
             $array['fields'][] = $field->__toArray();
48 48
         }
49
-        usort($array['fields'], function ($fieldA, $fieldB) {
49
+        usort($array['fields'], function($fieldA, $fieldB) {
50 50
             if ((bool)$fieldA['required'] !== (bool)$fieldB['required']) {
51 51
                 if ((bool)$fieldA['required']) {
52 52
                     return -1;
53
-                } else {
53
+                }else {
54 54
                     return 1;
55 55
                 }
56 56
             }
Please login to merge, or discard this patch.
src/base/dto/Order.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
     {
64 64
         if (preg_match('/^asc$/i', $direction)) {
65 65
             return self::ASC;
66
-        } else {
66
+        }else {
67 67
             return self::DESC;
68 68
         }
69 69
     }
Please login to merge, or discard this patch.
src/base/Router.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -61,8 +61,8 @@  discard block
 block discarded – undo
61 61
      */
62 62
     public function init()
63 63
     {
64
-        list($this->routing, $this->slugs) = $this->cache->getDataFromFile(CONFIG_DIR . DIRECTORY_SEPARATOR . 'urls.json', $this->cacheType, TRUE);
65
-        $this->domains = $this->cache->getDataFromFile(CONFIG_DIR . DIRECTORY_SEPARATOR . 'domains.json', $this->cacheType, TRUE);
64
+        list($this->routing, $this->slugs) = $this->cache->getDataFromFile(CONFIG_DIR.DIRECTORY_SEPARATOR.'urls.json', $this->cacheType, TRUE);
65
+        $this->domains = $this->cache->getDataFromFile(CONFIG_DIR.DIRECTORY_SEPARATOR.'domains.json', $this->cacheType, TRUE);
66 66
         if (empty($this->routing) || Config::getParam('debug', true)) {
67 67
             $this->debugLoad();
68 68
         }
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
             $this->hydrateRouting();
84 84
             $this->simpatize();
85 85
             Logger::log('End routes load');
86
-        } else {
86
+        }else {
87 87
             Logger::log('Routes generation skipped');
88 88
         }
89 89
     }
@@ -100,13 +100,13 @@  discard block
 block discarded – undo
100 100
         try {
101 101
             //Search action and execute
102 102
             return $this->searchAction($route);
103
-        } catch (AccessDeniedException $e) {
104
-            Logger::log(t('Solicitamos credenciales de acceso a zona restringida'), LOG_WARNING, ['file' => $e->getFile() . '[' . $e->getLine() . ']']);
103
+        }catch (AccessDeniedException $e) {
104
+            Logger::log(t('Solicitamos credenciales de acceso a zona restringida'), LOG_WARNING, ['file' => $e->getFile().'['.$e->getLine().']']);
105 105
             return Admin::staticAdminLogon();
106
-        } catch (RouterException $r) {
106
+        }catch (RouterException $r) {
107 107
             Logger::log($r->getMessage(), LOG_WARNING);
108 108
             $code = $r->getCode();
109
-        } catch (Exception $e) {
109
+        }catch (Exception $e) {
110 110
             Logger::log($e->getMessage(), LOG_ERR);
111 111
             throw $e;
112 112
         }
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
      */
125 125
     protected function searchAction($route)
126 126
     {
127
-        Inspector::stats('[Router] Searching action to execute: ' . $route, Inspector::SCOPE_DEBUG);
127
+        Inspector::stats('[Router] Searching action to execute: '.$route, Inspector::SCOPE_DEBUG);
128 128
         //Revisamos si tenemos la ruta registrada
129 129
         $parts = parse_url($route);
130 130
         $path = array_key_exists('path', $parts) ? $parts['path'] : $route;
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
                     }
145 145
 
146 146
                     throw new RouterException(t('Preconditions failed'), 412);
147
-                } catch (Exception $e) {
147
+                }catch (Exception $e) {
148 148
                     Logger::log($e->getMessage(), LOG_ERR);
149 149
                     throw $e;
150 150
                 }
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
                 }
170 170
             }
171 171
             $valid = count($action['requirements']) === $checked;
172
-        } else {
172
+        }else {
173 173
             $valid = true;
174 174
         }
175 175
         return $valid;
@@ -192,11 +192,11 @@  discard block
 block discarded – undo
192 192
             if ($modules->hasResults()) {
193 193
                 foreach ($modules->getIterator() as $modulePath) {
194 194
                     $module = $modulePath->getBasename();
195
-                    $this->routing = $this->inspectDir($modulesPath . DIRECTORY_SEPARATOR . $module, $module, $this->routing);
195
+                    $this->routing = $this->inspectDir($modulesPath.DIRECTORY_SEPARATOR.$module, $module, $this->routing);
196 196
                 }
197 197
             }
198 198
         }
199
-        $this->cache->storeData(CONFIG_DIR . DIRECTORY_SEPARATOR . 'domains.json', $this->domains, Cache::JSON, TRUE);
199
+        $this->cache->storeData(CONFIG_DIR.DIRECTORY_SEPARATOR.'domains.json', $this->domains, Cache::JSON, TRUE);
200 200
     }
201 201
 
202 202
     /**
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
213 213
             $homeParams = NULL;
214 214
             foreach ($this->routing as $pattern => $params) {
215 215
                 list($method, $route) = RouterHelper::extractHttpRoute($pattern);
216
-                if (preg_match('/' . preg_quote($route, '/') . '$/i', '/' . $home)) {
216
+                if (preg_match('/'.preg_quote($route, '/').'$/i', '/'.$home)) {
217 217
                     $homeParams = $params;
218 218
                 }
219 219
                 unset($method);
@@ -245,18 +245,18 @@  discard block
 block discarded – undo
245 245
     {
246 246
         $baseUrl = $absolute ? Request::getInstance()->getRootUrl() : '';
247 247
         if ('' === $slug) {
248
-            return $baseUrl . '/';
248
+            return $baseUrl.'/';
249 249
         }
250 250
         if (!is_array($this->slugs) || !array_key_exists($slug, $this->slugs)) {
251 251
             throw new RouterException(t('No existe la ruta especificada'));
252 252
         }
253
-        $url = $baseUrl . $this->slugs[$slug];
253
+        $url = $baseUrl.$this->slugs[$slug];
254 254
         if (!empty($params)) {
255 255
             foreach ($params as $key => $value) {
256
-                $url = str_replace('{' . $key . '}', $value, $url);
256
+                $url = str_replace('{'.$key.'}', $value, $url);
257 257
             }
258 258
         } elseif (!empty($this->routing[$this->slugs[$slug]]['default'])) {
259
-            $url = $baseUrl . $this->routing[$this->slugs[$slug]]['default'];
259
+            $url = $baseUrl.$this->routing[$this->slugs[$slug]]['default'];
260 260
         }
261 261
 
262 262
         return preg_replace('/(GET|POST|PUT|DELETE|ALL|HEAD|PATCH)\#\|\#/', '', $url);
@@ -268,7 +268,7 @@  discard block
 block discarded – undo
268 268
      */
269 269
     private function checkPreActions($class, $method)
270 270
     {
271
-        $preAction = 'pre' . ucfirst($method);
271
+        $preAction = 'pre'.ucfirst($method);
272 272
         if (method_exists($class, $preAction)) {
273 273
             Inspector::stats('[Router] Pre action invoked', Inspector::SCOPE_DEBUG);
274 274
             try {
@@ -276,7 +276,7 @@  discard block
 block discarded – undo
276 276
                     Logger::log(t('Pre action failed'), LOG_ERR, [error_get_last()]);
277 277
                     error_clear_last();
278 278
                 }
279
-            } catch (Exception $e) {
279
+            }catch (Exception $e) {
280 280
                 Logger::log($e->getMessage(), LOG_ERR, [$class, $method]);
281 281
             }
282 282
         }
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
      */
294 294
     protected function executeCachedRoute($route, $action, $class, $params = NULL)
295 295
     {
296
-        Inspector::stats('[Router] Executing route ' . $route, Inspector::SCOPE_DEBUG);
296
+        Inspector::stats('[Router] Executing route '.$route, Inspector::SCOPE_DEBUG);
297 297
         $action['params'] = array_merge($action['params'], $params, Request::getInstance()->getQueryParams());
298 298
         Security::getInstance()->setSessionKey(Cache::CACHE_SESSION_VAR, $action);
299 299
         $cache = Cache::needCache();
@@ -301,15 +301,15 @@  discard block
 block discarded – undo
301 301
         $return = null;
302 302
         if (FALSE !== $cache && $action['http'] === 'GET' && Config::getParam('debug') === FALSE) {
303 303
             list($path, $cacheDataName) = $this->cache->getRequestCacheHash();
304
-            $cachedData = $this->cache->readFromCache('json' . DIRECTORY_SEPARATOR . $path . $cacheDataName, $cache);
304
+            $cachedData = $this->cache->readFromCache('json'.DIRECTORY_SEPARATOR.$path.$cacheDataName, $cache);
305 305
             if (NULL !== $cachedData) {
306
-                $headers = $this->cache->readFromCache('json' . DIRECTORY_SEPARATOR . $path . $cacheDataName . '.headers', $cache, null, Cache::JSON);
306
+                $headers = $this->cache->readFromCache('json'.DIRECTORY_SEPARATOR.$path.$cacheDataName.'.headers', $cache, null, Cache::JSON);
307 307
                 Template::getInstance()->renderCache($cachedData, $headers);
308 308
                 $execute = FALSE;
309 309
             }
310 310
         }
311 311
         if ($execute) {
312
-            Inspector::stats('[Router] Start executing action ' . $route, Inspector::SCOPE_DEBUG);
312
+            Inspector::stats('[Router] Start executing action '.$route, Inspector::SCOPE_DEBUG);
313 313
             $this->checkPreActions($class, $action['method']);
314 314
             $return = call_user_func_array([$class, $action['method']], $params);
315 315
             if (false === $return) {
Please login to merge, or discard this patch.
src/Dispatcher.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -79,18 +79,18 @@
 block discarded – undo
79 79
                 if (!Request::getInstance()->isFile()) {
80 80
                     return $this->router->execute($uri ?: $this->actualUri);
81 81
                 }
82
-            } else {
82
+            }else {
83 83
                 return ConfigController::getInstance()->config();
84 84
             }
85
-        } catch (AdminCredentialsException $a) {
85
+        }catch (AdminCredentialsException $a) {
86 86
             return UserController::showAdminManager();
87
-        } catch (SecurityException $s) {
87
+        }catch (SecurityException $s) {
88 88
             return $this->security->notAuthorized($this->actualUri);
89
-        } catch (RouterException $r) {
89
+        }catch (RouterException $r) {
90 90
             return $this->router->httpNotFound($r);
91
-        } catch (ApiException $a) {
91
+        }catch (ApiException $a) {
92 92
             return $this->router->httpNotFound($a, true);
93
-        } catch (\Exception $e) {
93
+        }catch (\Exception $e) {
94 94
             return $this->dumpException($e);
95 95
         }
96 96
     }
Please login to merge, or discard this patch.