Passed
Push — master ( 4feda1...7cf784 )
by Fran
07:47
created
src/base/Singleton.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -135,9 +135,11 @@
 block discarded – undo
135 135
                 $cacheService->storeData($cacheFilename, $properties, Cache::JSON);
136 136
             }
137 137
             /** @var \ReflectionProperty $property */
138
-            if (!empty($properties) && is_array($properties)) foreach ($properties as $property => $class) {
138
+            if (!empty($properties) && is_array($properties)) {
139
+                foreach ($properties as $property => $class) {
139 140
                 $this->load($property, true, $class);
140 141
             }
142
+            }
141 143
             $this->setLoaded();
142 144
         } else {
143 145
             Logger::log(get_class($this) . ' already loaded', LOG_INFO);
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
 
21 21
     public function __construct()
22 22
     {
23
-        Logger::log(get_class($this) . ' constructor invoked');
23
+        Logger::log(get_class($this).' constructor invoked');
24 24
         $this->init();
25 25
     }
26 26
 
@@ -90,14 +90,14 @@  discard block
 block discarded – undo
90 90
         $calledClass = get_called_class();
91 91
         try {
92 92
             $instance = InjectorHelper::constructInyectableInstance($variable, $singleton, $classNameSpace, $calledClass);
93
-            $setter = "set" . ucfirst($variable);
93
+            $setter = "set".ucfirst($variable);
94 94
             if (method_exists($calledClass, $setter)) {
95 95
                 $this->$setter($instance);
96
-            } else {
96
+            }else {
97 97
                 $this->$variable = $instance;
98 98
             }
99
-        } catch (\Exception $e) {
100
-            Logger::log($e->getMessage() . ': ' . $e->getFile() . ' [' . $e->getLine() . ']', LOG_ERR);
99
+        }catch (\Exception $e) {
100
+            Logger::log($e->getMessage().': '.$e->getFile().' ['.$e->getLine().']', LOG_ERR);
101 101
         }
102 102
         return $this;
103 103
     }
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
     {
110 110
         if (!$this->isLoaded()) {
111 111
             $filename = sha1(get_class($this));
112
-            $cacheFilename = "reflections" . DIRECTORY_SEPARATOR . substr($filename, 0, 2) . DIRECTORY_SEPARATOR . substr($filename, 2, 2) . DIRECTORY_SEPARATOR . $filename . ".json";
112
+            $cacheFilename = "reflections".DIRECTORY_SEPARATOR.substr($filename, 0, 2).DIRECTORY_SEPARATOR.substr($filename, 2, 2).DIRECTORY_SEPARATOR.$filename.".json";
113 113
             /** @var \PSFS\base\Cache $cacheService */
114 114
             $cacheService = Cache::getInstance();
115 115
             /** @var \PSFS\base\config\Config $configService */
@@ -124,8 +124,8 @@  discard block
 block discarded – undo
124 124
                 $this->load($property, true, $class);
125 125
             }
126 126
             $this->setLoaded();
127
-        } else {
128
-            Logger::log(get_class($this) . ' already loaded', LOG_INFO);
127
+        }else {
128
+            Logger::log(get_class($this).' already loaded', LOG_INFO);
129 129
         }
130 130
     }
131 131
 }
Please login to merge, or discard this patch.
src/base/Service.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -177,7 +177,7 @@
 block discarded – undo
177 177
         $this->url = NULL;
178 178
         $this->params = array();
179 179
         $this->headers = array();
180
-        Logger::log("Context service for " . get_called_class() . " cleared!");
180
+        Logger::log("Context service for ".get_called_class()." cleared!");
181 181
     }
182 182
 
183 183
     /**
Please login to merge, or discard this patch.
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
     /**
117 117
      * Add request param
118 118
      *
119
-     * @param $key
119
+     * @param integer $key
120 120
      * @param null $value
121 121
      *
122 122
      * @return \PSFS\base\Service
@@ -153,8 +153,8 @@  discard block
 block discarded – undo
153 153
     }
154 154
 
155 155
     /**
156
-     * @param $header
157
-     * @param null $content
156
+     * @param string $header
157
+     * @param string $content
158 158
      *
159 159
      * @return $this
160 160
      */
Please login to merge, or discard this patch.
src/autoload.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -21,11 +21,11 @@
 block discarded – undo
21 21
             $path = str_replace("\\", DIRECTORY_SEPARATOR, $class);
22 22
 
23 23
             // filepath
24
-            $abs_path = SOURCE_DIR . DIRECTORY_SEPARATOR . $path . ".php";
24
+            $abs_path = SOURCE_DIR.DIRECTORY_SEPARATOR.$path.".php";
25 25
             if (!file_exists($abs_path)) {
26
-                pre('→ ' . $class);
27
-                pre('→ ' . $path);
28
-                pre('→ ' . $abs_path);
26
+                pre('→ '.$class);
27
+                pre('→ '.$path);
28
+                pre('→ '.$abs_path);
29 29
             }
30 30
 
31 31
             // require the file
Please login to merge, or discard this patch.
src/base/Cache.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 
122 122
     /**
123 123
      * Método que guarda en fichero los datos pasados
124
-     * @param $path
124
+     * @param string $path
125 125
      * @param $data
126 126
      * @param int $transform
127 127
      * @param boolean $absolutePath
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
 
157 157
     /**
158 158
      * Método estático que revisa si se necesita cachear la respuesta de un servicio o no
159
-     * @return integer|boolean
159
+     * @return integer
160 160
      */
161 161
     public static function needCache()
162 162
     {
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
      */
30 30
     private function saveTextToFile($data, $path, $absolute = false)
31 31
     {
32
-        $absolutePath = ($absolute) ? $path : CACHE_DIR . DIRECTORY_SEPARATOR . $path;
32
+        $absolutePath = ($absolute) ? $path : CACHE_DIR.DIRECTORY_SEPARATOR.$path;
33 33
         $filename = basename($absolutePath);
34 34
         GeneratorHelper::createDir(str_replace($filename, "", $absolutePath));
35 35
         if (false === file_put_contents($absolutePath, $data)) {
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
     public function getDataFromFile($path, $transform = Cache::TEXT, $absolute = false)
49 49
     {
50 50
         $data = null;
51
-        $absolutePath = ($absolute) ? $path : CACHE_DIR . DIRECTORY_SEPARATOR . $path;
51
+        $absolutePath = ($absolute) ? $path : CACHE_DIR.DIRECTORY_SEPARATOR.$path;
52 52
         if (file_exists($absolutePath)) {
53 53
             $data = file_get_contents($absolutePath);
54 54
         }
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
      */
65 65
     private function hasExpiredCache($path, $expires = 300, $absolute = false)
66 66
     {
67
-        $absolutePath = ($absolute) ? $path : CACHE_DIR . DIRECTORY_SEPARATOR . $path;
67
+        $absolutePath = ($absolute) ? $path : CACHE_DIR.DIRECTORY_SEPARATOR.$path;
68 68
         $lasModificationDate = filemtime($absolutePath);
69 69
         return ($lasModificationDate + $expires <= time());
70 70
     }
@@ -144,11 +144,11 @@  discard block
 block discarded – undo
144 144
     public function readFromCache($path, $expires = 300, callable $function, $transform = Cache::TEXT)
145 145
     {
146 146
         $data = null;
147
-        if (file_exists(CACHE_DIR . DIRECTORY_SEPARATOR . $path)) {
147
+        if (file_exists(CACHE_DIR.DIRECTORY_SEPARATOR.$path)) {
148 148
             if (null !== $function && $this->hasExpiredCache($path, $expires)) {
149 149
                 $data = call_user_func($function);
150 150
                 $this->storeData($path, $data, $transform);
151
-            } else {
151
+            }else {
152 152
                 $data = $this->getDataFromFile($path, $transform);
153 153
             }
154 154
         }
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
         $hash = "";
195 195
         $action = Security::getInstance()->getSessionKey("__CACHE__");
196 196
         if (null !== $action && $action["cache"] > 0) {
197
-            $hash = $action["http"] . " " . $action["slug"];
197
+            $hash = $action["http"]." ".$action["slug"];
198 198
         }
199 199
         return sha1($hash);
200 200
     }
Please login to merge, or discard this patch.
src/controller/I18nController.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -38,8 +38,8 @@
 block discarded – undo
38 38
         //Generating the templates translations
39 39
         $translations = $this->tpl->regenerateTemplates();
40 40
 
41
-        $locale_path = realpath(BASE_DIR . DIRECTORY_SEPARATOR . 'locale');
42
-        $locale_path .= DIRECTORY_SEPARATOR . $locale . DIRECTORY_SEPARATOR . 'LC_MESSAGES' . DIRECTORY_SEPARATOR;
41
+        $locale_path = realpath(BASE_DIR.DIRECTORY_SEPARATOR.'locale');
42
+        $locale_path .= DIRECTORY_SEPARATOR.$locale.DIRECTORY_SEPARATOR.'LC_MESSAGES'.DIRECTORY_SEPARATOR;
43 43
 
44 44
         //xgettext localizations
45 45
         $translations = array_merge($translations, GeneratorService::findTranslations(SOURCE_DIR, $locale));
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,9 @@
 block discarded – undo
33 33
     public function getTranslations($locale)
34 34
     {
35 35
         //Default locale
36
-        if (null === $locale) $locale = $this->config->get("default_language");
36
+        if (null === $locale) {
37
+            $locale = $this->config->get("default_language");
38
+        }
37 39
 
38 40
         //Generating the templates translations
39 41
         $translations = $this->tpl->regenerateTemplates();
Please login to merge, or discard this patch.
src/base/Logger.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
      * @param string $msg
89 89
      * @param array $context
90 90
      *
91
-     * @return bool
91
+     * @return boolean|null
92 92
      */
93 93
     public function debugLog($msg = '', $context = [])
94 94
     {
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 
98 98
     /**
99 99
      * Método que escribe un log de Error
100
-     * @param $msg
100
+     * @param string $msg
101 101
      * @param array $context
102 102
      *
103 103
      * @return bool
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
 
110 110
     /**
111 111
      * Método que escribe un log de Warning
112
-     * @param $msg
112
+     * @param string $msg
113 113
      * @param array $context
114 114
      * @return bool
115 115
      */
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -14,8 +14,8 @@  discard block
 block discarded – undo
14 14
 
15 15
 
16 16
 if (!defined("LOG_DIR")) {
17
-    GeneratorHelper::createDir(BASE_DIR . DIRECTORY_SEPARATOR . 'logs');
18
-    define("LOG_DIR", BASE_DIR . DIRECTORY_SEPARATOR . 'logs');
17
+    GeneratorHelper::createDir(BASE_DIR.DIRECTORY_SEPARATOR.'logs');
18
+    define("LOG_DIR", BASE_DIR.DIRECTORY_SEPARATOR.'logs');
19 19
 }
20 20
 
21 21
 /**
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
         $config = Config::getInstance();
49 49
         $args = func_get_args();
50 50
         list($logger, $debug, $path) = $this->setup($config, $args);
51
-        $this->stream = fopen($path . DIRECTORY_SEPARATOR . date("Ymd") . ".log", "a+");
51
+        $this->stream = fopen($path.DIRECTORY_SEPARATOR.date("Ymd").".log", "a+");
52 52
         $this->addPushLogger($logger, $debug, $config);
53 53
         $this->log_level = Config::getInstance()->get('log.level') ?: 'info';
54 54
     }
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
     private function createLoggerPath(Config $config)
203 203
     {
204 204
         $logger = $this->setLoggerName($config);
205
-        $path = LOG_DIR . DIRECTORY_SEPARATOR . $logger . DIRECTORY_SEPARATOR . date('Y') . DIRECTORY_SEPARATOR . date('m');
205
+        $path = LOG_DIR.DIRECTORY_SEPARATOR.$logger.DIRECTORY_SEPARATOR.date('Y').DIRECTORY_SEPARATOR.date('m');
206 206
         GeneratorHelper::createDir($path);
207 207
 
208 208
         return $path;
Please login to merge, or discard this patch.
src/base/Router.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -393,8 +393,10 @@
 block discarded – undo
393 393
             throw new RouterException(_("No existe la ruta especificada"));
394 394
         }
395 395
         $url = ($absolute) ? Request::getInstance()->getRootUrl() . $this->slugs[$slug] : $this->slugs[$slug];
396
-        if (!empty($params)) foreach ($params as $key => $value) {
396
+        if (!empty($params)) {
397
+            foreach ($params as $key => $value) {
397 398
             $url = str_replace("{" . $key . "}", $value, $url);
399
+        }
398 400
         } elseif (!empty($this->routing[$this->slugs[$slug]]["default"])) {
399 401
             $url = ($absolute) ? Request::getInstance()->getRootUrl() . $this->routing[$this->slugs[$slug]]["default"] : $this->routing[$this->slugs[$slug]]["default"];
400 402
         }
Please login to merge, or discard this patch.
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -60,12 +60,12 @@  discard block
 block discarded – undo
60 60
      */
61 61
     public function init()
62 62
     {
63
-        if (!file_exists(CONFIG_DIR . DIRECTORY_SEPARATOR . "urls.json") || Config::getInstance()->getDebugMode()) {
63
+        if (!file_exists(CONFIG_DIR.DIRECTORY_SEPARATOR."urls.json") || Config::getInstance()->getDebugMode()) {
64 64
             $this->hydrateRouting();
65 65
             $this->simpatize();
66
-        } else {
67
-            list($this->routing, $this->slugs) = $this->cache->getDataFromFile(CONFIG_DIR . DIRECTORY_SEPARATOR . "urls.json", Cache::JSON, TRUE);
68
-            $this->domains = $this->cache->getDataFromFile(CONFIG_DIR . DIRECTORY_SEPARATOR . "domains.json", Cache::JSON, TRUE);
66
+        }else {
67
+            list($this->routing, $this->slugs) = $this->cache->getDataFromFile(CONFIG_DIR.DIRECTORY_SEPARATOR."urls.json", Cache::JSON, TRUE);
68
+            $this->domains = $this->cache->getDataFromFile(CONFIG_DIR.DIRECTORY_SEPARATOR."domains.json", Cache::JSON, TRUE);
69 69
             $this->checkExternalModules(false);
70 70
         }
71 71
     }
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
                 "success" => FALSE,
91 91
                 "error" => $e->getMessage(),
92 92
             )), 'application/json');
93
-        } else {
93
+        }else {
94 94
             return $template->render('error.html.twig', array(
95 95
                 'exception' => $e,
96 96
                 'trace' => $e->getTraceAsString(),
@@ -141,12 +141,12 @@  discard block
 block discarded – undo
141 141
             SecurityHelper::checkRestrictedAccess($route);
142 142
             //Search action and execute
143 143
             $this->searchAction($route);
144
-        } catch (AccessDeniedException $e) {
144
+        }catch (AccessDeniedException $e) {
145 145
             Logger::log(_('Solicitamos credenciales de acceso a zona restringida'));
146 146
             return Admin::staticAdminLogon($route);
147
-        } catch (RouterException $r) {
147
+        }catch (RouterException $r) {
148 148
             Logger::log($r->getMessage(), LOG_WARNING);
149
-        } catch (\Exception $e) {
149
+        }catch (\Exception $e) {
150 150
             Logger::log($e->getMessage(), LOG_ERR);
151 151
             throw $e;
152 152
         }
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
      */
164 164
     protected function searchAction($route)
165 165
     {
166
-        Logger::log('Searching action to execute: ' . $route, LOG_INFO);
166
+        Logger::log('Searching action to execute: '.$route, LOG_INFO);
167 167
         //Revisamos si tenemos la ruta registrada
168 168
         $parts = parse_url($route);
169 169
         $path = (array_key_exists('path', $parts)) ? $parts['path'] : $route;
@@ -177,7 +177,7 @@  discard block
 block discarded – undo
177 177
                 $class = RouterHelper::getClassToCall($action);
178 178
                 try {
179 179
                     $this->executeCachedRoute($route, $action, $class, $get);
180
-                } catch (\Exception $e) {
180
+                }catch (\Exception $e) {
181 181
                     Logger::log($e->getMessage(), LOG_ERR);
182 182
                     throw new RouterException($e->getMessage(), 404, $e);
183 183
                 }
@@ -206,17 +206,17 @@  discard block
 block discarded – undo
206 206
             $externalModules = explode(',', $externalModules);
207 207
             foreach ($externalModules as &$module) {
208 208
                 $module = preg_replace('/(\\\|\/)/', DIRECTORY_SEPARATOR, $module);
209
-                $externalModulePath = VENDOR_DIR . DIRECTORY_SEPARATOR . $module . DIRECTORY_SEPARATOR . 'src';
209
+                $externalModulePath = VENDOR_DIR.DIRECTORY_SEPARATOR.$module.DIRECTORY_SEPARATOR.'src';
210 210
                 if (file_exists($externalModulePath)) {
211 211
                     $externalModule = $this->finder->directories()->in($externalModulePath)->depth(0);
212 212
                     if (!empty($externalModule)) {
213 213
                         foreach ($externalModule as $modulePath) {
214 214
                             $extModule = $modulePath->getBasename();
215
-                            $moduleAutoloader = realpath($externalModulePath . DIRECTORY_SEPARATOR . $extModule . DIRECTORY_SEPARATOR . 'autoload.php');
215
+                            $moduleAutoloader = realpath($externalModulePath.DIRECTORY_SEPARATOR.$extModule.DIRECTORY_SEPARATOR.'autoload.php');
216 216
                             if (file_exists($moduleAutoloader)) {
217 217
                                 @include $moduleAutoloader;
218
-                                if($hydrateRoute) {
219
-                                    $this->routing = $this->inspectDir($externalModulePath . DIRECTORY_SEPARATOR . $extModule, '\\' . $extModule, $this->routing);
218
+                                if ($hydrateRoute) {
219
+                                    $this->routing = $this->inspectDir($externalModulePath.DIRECTORY_SEPARATOR.$extModule, '\\'.$extModule, $this->routing);
220 220
                                 }
221 221
                             }
222 222
                         }
@@ -238,11 +238,11 @@  discard block
 block discarded – undo
238 238
             $modules = $this->finder->directories()->in($modulesPath)->depth(0);
239 239
             foreach ($modules as $modulePath) {
240 240
                 $module = $modulePath->getBasename();
241
-                $this->routing = $this->inspectDir($modulesPath . DIRECTORY_SEPARATOR . $module, $module, $this->routing);
241
+                $this->routing = $this->inspectDir($modulesPath.DIRECTORY_SEPARATOR.$module, $module, $this->routing);
242 242
             }
243 243
         }
244 244
         $this->checkExternalModules();
245
-        $this->cache->storeData(CONFIG_DIR . DIRECTORY_SEPARATOR . "domains.json", $this->domains, Cache::JSON, TRUE);
245
+        $this->cache->storeData(CONFIG_DIR.DIRECTORY_SEPARATOR."domains.json", $this->domains, Cache::JSON, TRUE);
246 246
     }
247 247
 
248 248
     /**
@@ -257,7 +257,7 @@  discard block
 block discarded – undo
257 257
             $home_params = NULL;
258 258
             foreach ($this->routing as $pattern => $params) {
259 259
                 list($method, $route) = RouterHelper::extractHttpRoute($pattern);
260
-                if (preg_match("/" . preg_quote($route, "/") . "$/i", "/" . $home)) {
260
+                if (preg_match("/".preg_quote($route, "/")."$/i", "/".$home)) {
261 261
                     $home_params = $params;
262 262
                 }
263 263
             }
@@ -282,7 +282,7 @@  discard block
 block discarded – undo
282 282
         $files = $this->finder->files()->in($origen)->path('/(controller|api)/i')->depth(1)->name("*.php");
283 283
         foreach ($files as $file) {
284 284
             $filename = str_replace("/", '\\', str_replace($origen, '', $file->getPathname()));
285
-            $routing = $this->addRouting($namespace . str_replace('.php', '', $filename), $routing, $namespace);
285
+            $routing = $this->addRouting($namespace.str_replace('.php', '', $filename), $routing, $namespace);
286 286
         }
287 287
         $this->finder = new Finder();
288 288
 
@@ -351,7 +351,7 @@  discard block
 block discarded – undo
351 351
             if (!$this->domains) {
352 352
                 $this->domains = [];
353 353
             }
354
-            $domain = "@" . $class->getConstant("DOMAIN") . "/";
354
+            $domain = "@".$class->getConstant("DOMAIN")."/";
355 355
             if (!array_key_exists($domain, $this->domains)) {
356 356
                 $this->domains[$domain] = RouterHelper::extractDomainInfo($class, $domain);
357 357
             }
@@ -366,11 +366,11 @@  discard block
 block discarded – undo
366 366
      */
367 367
     public function simpatize()
368 368
     {
369
-        $translationFileName = "translations" . DIRECTORY_SEPARATOR . "routes_translations.php";
370
-        $absoluteTranslationFileName = CACHE_DIR . DIRECTORY_SEPARATOR . $translationFileName;
369
+        $translationFileName = "translations".DIRECTORY_SEPARATOR."routes_translations.php";
370
+        $absoluteTranslationFileName = CACHE_DIR.DIRECTORY_SEPARATOR.$translationFileName;
371 371
         $this->generateSlugs($absoluteTranslationFileName);
372 372
         GeneratorHelper::createDir(CONFIG_DIR);
373
-        Cache::getInstance()->storeData(CONFIG_DIR . DIRECTORY_SEPARATOR . "urls.json", array($this->routing, $this->slugs), Cache::JSON, TRUE);
373
+        Cache::getInstance()->storeData(CONFIG_DIR.DIRECTORY_SEPARATOR."urls.json", array($this->routing, $this->slugs), Cache::JSON, TRUE);
374 374
 
375 375
         return $this;
376 376
     }
@@ -388,16 +388,16 @@  discard block
 block discarded – undo
388 388
     public function getRoute($slug = '', $absolute = FALSE, $params = [])
389 389
     {
390 390
         if (strlen($slug) === 0) {
391
-            return ($absolute) ? Request::getInstance()->getRootUrl() . '/' : '/';
391
+            return ($absolute) ? Request::getInstance()->getRootUrl().'/' : '/';
392 392
         }
393 393
         if (NULL === $slug || !array_key_exists($slug, $this->slugs)) {
394 394
             throw new RouterException(_("No existe la ruta especificada"));
395 395
         }
396
-        $url = ($absolute) ? Request::getInstance()->getRootUrl() . $this->slugs[$slug] : $this->slugs[$slug];
396
+        $url = ($absolute) ? Request::getInstance()->getRootUrl().$this->slugs[$slug] : $this->slugs[$slug];
397 397
         if (!empty($params)) foreach ($params as $key => $value) {
398
-            $url = str_replace("{" . $key . "}", $value, $url);
398
+            $url = str_replace("{".$key."}", $value, $url);
399 399
         } elseif (!empty($this->routing[$this->slugs[$slug]]["default"])) {
400
-            $url = ($absolute) ? Request::getInstance()->getRootUrl() . $this->routing[$this->slugs[$slug]]["default"] : $this->routing[$this->slugs[$slug]]["default"];
400
+            $url = ($absolute) ? Request::getInstance()->getRootUrl().$this->routing[$this->slugs[$slug]]["default"] : $this->routing[$this->slugs[$slug]]["default"];
401 401
         }
402 402
 
403 403
         return preg_replace('/(GET|POST|PUT|DELETE|ALL)\#\|\#/', '', $url);
@@ -458,19 +458,19 @@  discard block
 block discarded – undo
458 458
      */
459 459
     protected function executeCachedRoute($route, $action, $class, $params = NULL)
460 460
     {
461
-        Logger::log('Executing route ' . $route, LOG_INFO);
461
+        Logger::log('Executing route '.$route, LOG_INFO);
462 462
         Security::getInstance()->setSessionKey("__CACHE__", $action);
463 463
         $cache = Cache::needCache();
464 464
         $execute = TRUE;
465 465
         if (FALSE !== $cache && Config::getInstance()->getDebugMode() === FALSE) {
466 466
             $cacheDataName = $this->cache->getRequestCacheHash();
467
-            $tmpDir = substr($cacheDataName, 0, 2) . DIRECTORY_SEPARATOR . substr($cacheDataName, 2, 2) . DIRECTORY_SEPARATOR;
468
-            $cachedData = $this->cache->readFromCache("json" . DIRECTORY_SEPARATOR . $tmpDir . $cacheDataName,
469
-                $cache, function () {
467
+            $tmpDir = substr($cacheDataName, 0, 2).DIRECTORY_SEPARATOR.substr($cacheDataName, 2, 2).DIRECTORY_SEPARATOR;
468
+            $cachedData = $this->cache->readFromCache("json".DIRECTORY_SEPARATOR.$tmpDir.$cacheDataName,
469
+                $cache, function() {
470 470
                 });
471 471
             if (NULL !== $cachedData) {
472
-                $headers = $this->cache->readFromCache("json" . DIRECTORY_SEPARATOR . $tmpDir . $cacheDataName . ".headers",
473
-                    $cache, function () {
472
+                $headers = $this->cache->readFromCache("json".DIRECTORY_SEPARATOR.$tmpDir.$cacheDataName.".headers",
473
+                    $cache, function() {
474 474
                     }, Cache::JSON);
475 475
                 Template::getInstance()->renderCache($cachedData, $headers);
476 476
                 $execute = FALSE;
Please login to merge, or discard this patch.
src/base/types/helpers/RouterHelper.php 3 patches
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -56,8 +56,10 @@  discard block
 block discarded – undo
56 56
         $_route = explode("/", $url['path']);
57 57
         $_pattern = explode("/", $pattern);
58 58
         $get = array();
59
-        if (!empty($_pattern)) foreach ($_pattern as $index => $component) {
59
+        if (!empty($_pattern)) {
60
+            foreach ($_pattern as $index => $component) {
60 61
             $_get = array();
62
+        }
61 63
             preg_match_all('/^\{(.*)\}$/i', $component, $_get);
62 64
             if (!empty($_get[1]) && isset($_route[$index])) {
63 65
                 $get[array_pop($_get[1])] = $_route[$index];
@@ -151,12 +153,16 @@  discard block
 block discarded – undo
151 153
         $params = [];
152 154
         $parameters = $method->getParameters();
153 155
         /** @var \ReflectionParameter $param */
154
-        if (count($parameters) > 0) foreach ($parameters as $param) {
156
+        if (count($parameters) > 0) {
157
+            foreach ($parameters as $param) {
155 158
             if ($param->isOptional() && !is_array($param->getDefaultValue())) {
156 159
                 $params[$param->getName()] = $param->getDefaultValue();
160
+        }
157 161
                 $default = str_replace('{' . $param->getName() . '}', $param->getDefaultValue(), $regex);
158 162
             }
159
-        } else $default = $regex;
163
+        } else {
164
+            $default = $regex;
165
+        }
160 166
 
161 167
         return array($regex, $default, $params);
162 168
     }
Please login to merge, or discard this patch.
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
     /**
141 141
      * Método que extrae los parámetros de una función
142 142
      *
143
-     * @param array $sr
143
+     * @param string[] $sr
144 144
      * @param \ReflectionMethod $method
145 145
      *
146 146
      * @return array
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
      *
208 208
      * @param string $docComments
209 209
      *
210
-     * @return bool
210
+     * @return string
211 211
      */
212 212
     public static function extractReflectionCacheability($docComments)
213 213
     {
Please login to merge, or discard this patch.
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
     public static function getClassToCall($action)
23 23
     {
24 24
         Logger::log('Getting class to call for executing the request action', LOG_DEBUG, $action);
25
-        $actionClass = class_exists($action["class"]) ? $action["class"] : "\\" . $action["class"];
25
+        $actionClass = class_exists($action["class"]) ? $action["class"] : "\\".$action["class"];
26 26
         $class = (method_exists($actionClass, "getInstance")) ? $actionClass::getInstance() : new $actionClass;
27 27
         return $class;
28 28
     }
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
         $expr = preg_quote($expr, '/');
109 109
         $expr = str_replace('###', '(.*)', $expr);
110 110
         $expr2 = preg_replace('/\(\.\*\)$/', '', $expr);
111
-        $matched = preg_match('/^' . $expr . '\/?$/i', $path) || preg_match('/^' . $expr2 . '?$/i', $path);
111
+        $matched = preg_match('/^'.$expr.'\/?$/i', $path) || preg_match('/^'.$expr2.'?$/i', $path);
112 112
         return $matched;
113 113
     }
114 114
 
@@ -119,8 +119,8 @@  discard block
 block discarded – undo
119 119
      */
120 120
     public static function extractDomainInfo(\ReflectionClass $class, $domain)
121 121
     {
122
-        $path = dirname($class->getFileName()) . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR;
123
-        $path = realpath($path) . DIRECTORY_SEPARATOR;
122
+        $path = dirname($class->getFileName()).DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR;
123
+        $path = realpath($path).DIRECTORY_SEPARATOR;
124 124
         $tpl_path = "templates";
125 125
         $public_path = "public";
126 126
         $model_path = "models";
@@ -130,13 +130,13 @@  discard block
 block discarded – undo
130 130
             $model_path = ucfirst($model_path);
131 131
         }
132 132
         if ($class->hasConstant("TPL")) {
133
-            $tpl_path .= DIRECTORY_SEPARATOR . $class->getConstant("TPL");
133
+            $tpl_path .= DIRECTORY_SEPARATOR.$class->getConstant("TPL");
134 134
         }
135 135
         return [
136 136
             "base" => $path,
137
-            "template" => $path . $tpl_path,
138
-            "model" => $path . $model_path,
139
-            "public" => $path . $public_path,
137
+            "template" => $path.$tpl_path,
138
+            "model" => $path.$model_path,
139
+            "public" => $path.$public_path,
140 140
         ];
141 141
     }
142 142
 
@@ -158,9 +158,9 @@  discard block
 block discarded – undo
158 158
         if (count($parameters) > 0) foreach ($parameters as $param) {
159 159
             if ($param->isOptional() && !is_array($param->getDefaultValue())) {
160 160
                 $params[$param->getName()] = $param->getDefaultValue();
161
-                $default = str_replace('{' . $param->getName() . '}', $param->getDefaultValue(), $regex);
161
+                $default = str_replace('{'.$param->getName().'}', $param->getDefaultValue(), $regex);
162 162
             }
163
-        } else $default = $regex;
163
+        }else $default = $regex;
164 164
 
165 165
         return array($regex, $default, $params);
166 166
     }
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
             $default = str_replace('{__DOMAIN__}', $module, $default);
241 241
             $httpMethod = RouterHelper::extractReflectionHttpMethod($docComments);
242 242
             $label = RouterHelper::extractReflectionLabel(str_replace('{__API__}', $api, $docComments));
243
-            $route = $httpMethod . "#|#" . $regex;
243
+            $route = $httpMethod."#|#".$regex;
244 244
             $info = [
245 245
                 "method" => $method->getName(),
246 246
                 "params" => $params,
Please login to merge, or discard this patch.
src/base/types/helpers/InjectorHelper.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -54,13 +54,13 @@  discard block
 block discarded – undo
54 54
      */
55 55
     public static function constructInyectableInstance($variable, $singleton, $classNameSpace, $calledClass)
56 56
     {
57
-        Logger::log('Create inyectable instance for ' . $classNameSpace);
57
+        Logger::log('Create inyectable instance for '.$classNameSpace);
58 58
         $reflector = new \ReflectionClass($calledClass);
59 59
         $property = $reflector->getProperty($variable);
60 60
         $varInstanceType = (null === $classNameSpace) ? InjectorHelper::extractVarType($property->getDocComment()) : $classNameSpace;
61 61
         if (true === $singleton && method_exists($varInstanceType, "getInstance")) {
62 62
             $instance = $varInstanceType::getInstance();
63
-        } else {
63
+        }else {
64 64
             $instance = new $varInstanceType();
65 65
         }
66 66
         return $instance;
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
     public static function getClassProperties($class)
75 75
     {
76 76
         $properties = [];
77
-        Logger::log('Extracting annotations properties from class ' . $class);
77
+        Logger::log('Extracting annotations properties from class '.$class);
78 78
         $selfReflector = new \ReflectionClass($class);
79 79
         if (false !== $selfReflector->getParentClass()) {
80 80
             $properties = self::getClassProperties($selfReflector->getParentClass()->getName());
Please login to merge, or discard this patch.