psfs / core

src/base/Cache.php

Doc Comments +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
 
         /**
          * Método que guarda en fichero los datos pasados
-         * @param $path
+         * @param string $path
          * @param $data
          * @param int $transform
          * @param boolean $absolutePath

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
                 if (null !== $function && $this->hasExpiredCache($path, $expires)) {
                     $data = call_user_func($function);
                     $this->storeData($path, $data, $transform);
-                } else {
+                }else {
                     $data = $this->getDataFromFile($path, $transform);
                 }
             }

src/base/extension/TemplateFunctions.php

Doc Comments +2 added lines, -2 removed lines

@@ -130,7 +130,7 @@ 
          * Método que copia los recursos recursivamente
          * @param string $dest
          * @param boolean $force
-         * @param $filename_path
+         * @param string $filename_path
          * @param boolean $debug
          */
         private static function copyResources($dest, $force, $filename_path, $debug)
@@ -155,7 +155,7 @@ 
          * @param string $path
          * @param $domains
          *
-         * @return mixed
+         * @return string
          */
         private static function extractPathname($path, $domains)
         {

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
             $router = Router::getInstance();
             try {
                 return $router->getRoute($path, $absolute, $params);
-            } catch (\Exception $e)
+            }catch (\Exception $e)
             {
                 return $router->getRoute('', $absolute, $params);
             }

Braces +6 added lines, -3 removed lines

@@ -141,7 +141,7 @@ 
                     if (is_dir($filename_path)) {
                         Config::createDir(WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder);
                         Template::copyr($filename_path, WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder);
-                    }else {
+                    } else {
                         if (@copy($filename_path, WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder) === FALSE) {
                             throw new ConfigException("Can't copy ".$filename_path." to ".WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder);
                         }
@@ -198,8 +198,11 @@ 
         private static function putResourceContent($name, $filename_path, $base, $file_path)
         {
             $data = file_get_contents($filename_path);
-            if (!empty($name)) file_put_contents(WEB_DIR.DIRECTORY_SEPARATOR.$name, $data);
-            else file_put_contents($base.$file_path, $data);
+            if (!empty($name)) {
+                file_put_contents(WEB_DIR.DIRECTORY_SEPARATOR.$name, $data);
+            } else {
+                file_put_contents($base.$file_path, $data);
+            }
         }
 
         /**

src/base/Request.php

Doc Comments +5 added lines, -1 removed lines

@@ -95,6 +95,10 @@ 
      * @return string|null
      */
     public static function header($name) { return self::getInstance()->getHeader($name); }
+
+    /**
+     * @return string
+     */
     public function getHeader($name)
     {
         $header = null;
@@ -107,7 +111,7 @@ 
 
     /**
      * Método que devuelve la url solicitada
-     * @return string|null
+     * @return string
      */
     public static function requestUri(){ return self::getInstance()->getRequestUri(); }
 

Spacing +6 added lines, -6 removed lines

@@ -4,14 +4,14 @@ 
 
 use PSFS\base\types\SingletonTrait;
 
-if(!function_exists("getallheaders"))
+if (!function_exists("getallheaders"))
 {
     function getallheaders()
     {
         $headers = array();
-        foreach($_SERVER as $h=>$v)
-            if(preg_match('/HTTP_(.+)/',$h,$hp))
-                $headers[$hp[1]]=$v;
+        foreach ($_SERVER as $h=>$v)
+            if (preg_match('/HTTP_(.+)/', $h, $hp))
+                $headers[$hp[1]] = $v;
         return $headers;
     }
 }
@@ -109,12 +109,12 @@ 
      * Método que devuelve la url solicitada
      * @return string|null
      */
-    public static function requestUri(){ return self::getInstance()->getRequestUri(); }
+    public static function requestUri() { return self::getInstance()->getRequestUri(); }
 
     /**
      * @return string
      */
-    public function getRequestUri(){ return array_key_exists('REQUEST_URI', $this->server) ? $this->server['REQUEST_URI'] : ''; }
+    public function getRequestUri() { return array_key_exists('REQUEST_URI', $this->server) ? $this->server['REQUEST_URI'] : ''; }
 
     /**
      * Método que devuelve el idioma de la petición

Braces +9 added lines, -4 removed lines

@@ -9,9 +9,10 @@ 
     function getallheaders()
     {
         $headers = array();
-        foreach($_SERVER as $h=>$v)
-            if(preg_match('/HTTP_(.+)/',$h,$hp))
+        foreach($_SERVER as $h=>$v) {
+                    if(preg_match('/HTTP_(.+)/',$h,$hp))
                 $headers[$hp[1]]=$v;
+        }
         return $headers;
     }
 }
@@ -180,7 +181,9 @@ 
      */
     public function redirect($url = null)
     {
-        if (null === $url) $url = $this->server['HTTP_ORIGIN'];
+        if (null === $url) {
+            $url = $this->server['HTTP_ORIGIN'];
+        }
         ob_start();
         header('Location: '.$url);
         ob_end_clean();
@@ -226,7 +229,9 @@ 
         $host = $this->getServerName();
         $protocol = $protocol ? $this->getProtocol() : '';
         $url = '';
-        if (!empty($host) && !empty($protocol)) $url = $protocol.$host;
+        if (!empty($host) && !empty($protocol)) {
+            $url = $protocol.$host;
+        }
         return $url;
     }
 

src/base/Router.php

Doc Comments +2 added lines, -2 removed lines

@@ -588,7 +588,7 @@ 
         /**
          * Método que extrae los parámetros de una función
          *
-         * @param array $sr
+         * @param string[] $sr
          * @param \ReflectionMethod $method
          *
          * @return array
@@ -642,7 +642,7 @@ 
          *
          * @param string $docComments
          *
-         * @return bool
+         * @return string
          */
         private function extractReflectionCacheability($docComments)
         {

Braces +12 added lines, -4 removed lines

@@ -260,8 +260,10 @@ 
             $_route = explode("/", $url['path']);
             $_pattern = explode("/", $pattern);
             $get = array();
-            if (!empty($_pattern)) foreach ($_pattern as $index => $component) {
+            if (!empty($_pattern)) {
+                foreach ($_pattern as $index => $component) {
                 $_get = array();
+            }
                 preg_match_all('/^\{(.*)\}$/i', $component, $_get);
                 if (!empty($_get[1]) && isset($_route[$index])) {
                     $get[array_pop($_get[1])] = $_route[$index];
@@ -476,8 +478,10 @@ 
                 throw new RouterException(_("No existe la ruta especificada"));
             }
             $url = ($absolute) ? Request::getInstance()->getRootUrl() . $this->slugs[$slug] : $this->slugs[$slug];
-            if (!empty($params)) foreach ($params as $key => $value) {
+            if (!empty($params)) {
+                foreach ($params as $key => $value) {
                 $url = str_replace("{" . $key . "}", $value, $url);
+            }
             } elseif (!empty($this->routing[$this->slugs[$slug]]["default"])) {
                 $url = ($absolute) ? Request::getInstance()->getRootUrl() . $this->routing[$this->slugs[$slug]]["default"] : $this->routing[$this->slugs[$slug]]["default"];
             }
@@ -599,12 +603,16 @@ 
             $default = '';
             $params = array();
             $parameters = $method->getParameters();
-            if (count($parameters) > 0) foreach ($parameters as $param) {
+            if (count($parameters) > 0) {
+                foreach ($parameters as $param) {
                 if ($param->isOptional() && !is_array($param->getDefaultValue())) {
                     $params[$param->getName()] = $param->getDefaultValue();
+            }
                     $default = str_replace('{' . $param->getName() . '}', $param->getDefaultValue(), $regex);
                 }
-            } else $default = $regex;
+            } else {
+                $default = $regex;
+            }
 
             return array($regex, $default, $params);
         }

Spacing +41 added lines, -41 removed lines

@@ -59,12 +59,12 @@ 
          */
         public function init()
         {
-            if (!file_exists(CONFIG_DIR . DIRECTORY_SEPARATOR . "urls.json") || Config::getInstance()->getDebugMode()) {
+            if (!file_exists(CONFIG_DIR.DIRECTORY_SEPARATOR."urls.json") || Config::getInstance()->getDebugMode()) {
                 $this->hydrateRouting();
                 $this->simpatize();
-            } else {
-                list($this->routing, $this->slugs) = $this->cache->getDataFromFile(CONFIG_DIR . DIRECTORY_SEPARATOR . "urls.json", Cache::JSON, TRUE);
-                $this->domains = $this->cache->getDataFromFile(CONFIG_DIR . DIRECTORY_SEPARATOR . "domains.json", Cache::JSON, TRUE);
+            }else {
+                list($this->routing, $this->slugs) = $this->cache->getDataFromFile(CONFIG_DIR.DIRECTORY_SEPARATOR."urls.json", Cache::JSON, TRUE);
+                $this->domains = $this->cache->getDataFromFile(CONFIG_DIR.DIRECTORY_SEPARATOR."domains.json", Cache::JSON, TRUE);
             }
         }
 
@@ -84,7 +84,7 @@ 
                     "success" => FALSE,
                     "error"   => $e->getMessage(),
                 )), 'application/json');
-            } else {
+            }else {
                 if (NULL === $e) {
                     $e = new \Exception(_('Página no encontrada'), 404);
                 }
@@ -124,24 +124,24 @@ 
 
                 //Search action and execute
                 return $this->searchAction($route);
-            } catch (AccessDeniedException $e) {
+            }catch (AccessDeniedException $e) {
                 Logger::getInstance()->debugLog(_('Solicitamos credenciales de acceso a zona restringida'));
                 if ('login' === Config::getInstance()->get('admin_login')) {
                     return $this->redirectLogin($route);
-                } else {
+                }else {
                     return $this->sentAuthHeader();
                 }
-            } catch (RouterException $r) {
+            }catch (RouterException $r) {
                 if (FALSE !== preg_match('/\/$/', $route)) {
                     if (preg_match('/admin/', $route)) {
                         $default = Config::getInstance()->get('admin_action');
-                    } else {
+                    }else {
                         $default = Config::getInstance()->get('home_action');
                     }
 
                     return $this->execute($this->getRoute($default));
                 }
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 Logger::getInstance()->errorLog($e->getMessage());
                 throw $e;
             }
@@ -157,7 +157,7 @@ 
             $corsEnabled = Config::getInstance()->get('cors.enabled');
             $request = Request::getInstance();
             if (NULL !== $corsEnabled && null !== $request->getServer('HTTP_REFERER')) {
-                if($corsEnabled == '*' || preg_match($corsEnabled, $request->getServer('HTTP_REFERER'))) {
+                if ($corsEnabled == '*' || preg_match($corsEnabled, $request->getServer('HTTP_REFERER'))) {
                     if (!$this->headersSent) {
                         header("Access-Control-Allow-Credentials: true");
                         header("Access-Control-Allow-Origin: *");
@@ -165,8 +165,8 @@ 
                         header("Access-Control-Allow-Headers: Access-Control-Allow-Methods, Access-Control-Allow-Headers, Access-Control-Allow-Origin, Origin, X-Requested-With, Content-Type, Accept, Authorization");
                         $this->headersSent = true;
                     }
-                    if(Request::getInstance()->getMethod() == 'OPTIONS') {
-                        header( "HTTP/1.1 200 OK" );
+                    if (Request::getInstance()->getMethod() == 'OPTIONS') {
+                        header("HTTP/1.1 200 OK");
                         exit();
                     }
                 }
@@ -195,7 +195,7 @@ 
                     $class = $this->getClassToCall($action);
                     try {
                         return $this->executeCachedRoute($route, $action, $class, $get);
-                    } catch (\Exception $e) {
+                    }catch (\Exception $e) {
                         Logger::getInstance()->debugLog($e->getMessage(), array($e->getFile(), $e->getLine()));
                         throw new RouterException($e->getMessage(), 404, $e);
                     }
@@ -283,13 +283,13 @@ 
             if (file_exists($modules)) {
                 $this->routing = $this->inspectDir($modules, "", $this->routing);
             }
-            $this->cache->storeData(CONFIG_DIR . DIRECTORY_SEPARATOR . "domains.json", $this->domains, Cache::JSON, TRUE);
+            $this->cache->storeData(CONFIG_DIR.DIRECTORY_SEPARATOR."domains.json", $this->domains, Cache::JSON, TRUE);
             $home = Config::getInstance()->get('home_action');
             if (NULL !== $home || $home !== '') {
                 $home_params = NULL;
                 foreach ($this->routing as $pattern => $params) {
                     list($method, $route) = $this->extractHttpRoute($pattern);
-                    if (preg_match("/" . preg_quote($route, "/") . "$/i", "/" . $home)) {
+                    if (preg_match("/".preg_quote($route, "/")."$/i", "/".$home)) {
                         $home_params = $params;
                     }
                 }
@@ -314,7 +314,7 @@ 
             $files = $this->finder->files()->in($origen)->path('/(controller|api)/i')->name("*.php");
             foreach ($files as $file) {
                 $filename = str_replace("/", '\\', str_replace($origen, '', $file->getPathname()));
-                $routing = $this->addRouting($namespace . str_replace('.php', '', $filename), $routing);
+                $routing = $this->addRouting($namespace.str_replace('.php', '', $filename), $routing);
             }
             $this->finder = new Finder();
 
@@ -355,7 +355,7 @@ 
                                 $httpMethod = $this->extractReflectionHttpMethod($docComments);
                                 $visible = $this->extractReflectionVisibility($docComments);
                                 $expiration = $this->extractReflectionCacheability($docComments);
-                                $routing[$httpMethod . "#|#" . $regex] = array(
+                                $routing[$httpMethod."#|#".$regex] = array(
                                     "class"   => $namespace,
                                     "method"  => $method->getName(),
                                     "params"  => $params,
@@ -385,9 +385,9 @@ 
         {
             //Calculamos los dominios para las plantillas
             if ($class->hasConstant("DOMAIN")) {
-                $domain = "@" . $class->getConstant("DOMAIN") . "/";
-                $path = dirname($class->getFileName()) . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR;
-                $path = realpath($path) . DIRECTORY_SEPARATOR;
+                $domain = "@".$class->getConstant("DOMAIN")."/";
+                $path = dirname($class->getFileName()).DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR;
+                $path = realpath($path).DIRECTORY_SEPARATOR;
                 $tpl_path = "templates";
                 $public_path = "public";
                 $model_path = "models";
@@ -397,12 +397,12 @@ 
                     $model_path = ucfirst($model_path);
                 }
                 if ($class->hasConstant("TPL")) {
-                    $tpl_path .= DIRECTORY_SEPARATOR . $class->getConstant("TPL");
+                    $tpl_path .= DIRECTORY_SEPARATOR.$class->getConstant("TPL");
                 }
                 $this->domains[$domain] = array(
-                    "template" => $path . $tpl_path,
-                    "model"    => $path . $model_path,
-                    "public"   => $path . $public_path,
+                    "template" => $path.$tpl_path,
+                    "model"    => $path.$model_path,
+                    "public"   => $path.$public_path,
                 );
             }
 
@@ -415,11 +415,11 @@ 
          */
         private function simpatize()
         {
-            $translationFileName = "translations" . DIRECTORY_SEPARATOR . "routes_translations.php";
-            $absoluteTranslationFileName = CACHE_DIR . DIRECTORY_SEPARATOR . $translationFileName;
+            $translationFileName = "translations".DIRECTORY_SEPARATOR."routes_translations.php";
+            $absoluteTranslationFileName = CACHE_DIR.DIRECTORY_SEPARATOR.$translationFileName;
             $this->generateSlugs($absoluteTranslationFileName);
             Config::createDir(CONFIG_DIR);
-            Cache::getInstance()->storeData(CONFIG_DIR . DIRECTORY_SEPARATOR . "urls.json", array($this->routing, $this->slugs), Cache::JSON, TRUE);
+            Cache::getInstance()->storeData(CONFIG_DIR.DIRECTORY_SEPARATOR."urls.json", array($this->routing, $this->slugs), Cache::JSON, TRUE);
 
             return $this;
         }
@@ -470,16 +470,16 @@ 
         public function getRoute($slug = '', $absolute = FALSE, $params = NULL)
         {
             if (strlen($slug) === 0) {
-                return ($absolute) ? Request::getInstance()->getRootUrl() . '/' : '/';
+                return ($absolute) ? Request::getInstance()->getRootUrl().'/' : '/';
             }
             if (NULL === $slug || !array_key_exists($slug, $this->slugs)) {
                 throw new RouterException(_("No existe la ruta especificada"));
             }
-            $url = ($absolute) ? Request::getInstance()->getRootUrl() . $this->slugs[$slug] : $this->slugs[$slug];
+            $url = ($absolute) ? Request::getInstance()->getRootUrl().$this->slugs[$slug] : $this->slugs[$slug];
             if (!empty($params)) foreach ($params as $key => $value) {
-                $url = str_replace("{" . $key . "}", $value, $url);
+                $url = str_replace("{".$key."}", $value, $url);
             } elseif (!empty($this->routing[$this->slugs[$slug]]["default"])) {
-                $url = ($absolute) ? Request::getInstance()->getRootUrl() . $this->routing[$this->slugs[$slug]]["default"] : $this->routing[$this->slugs[$slug]]["default"];
+                $url = ($absolute) ? Request::getInstance()->getRootUrl().$this->routing[$this->slugs[$slug]]["default"] : $this->routing[$this->slugs[$slug]]["default"];
             }
 
             return preg_replace('/(GET|POST|PUT|DELETE|ALL)\#\|\#/', '', $url);
@@ -497,12 +497,12 @@ 
                 if (preg_match('/^\/admin(\/|$)/', $routePattern)) {
                     if (preg_match('/^PSFS/', $params["class"])) {
                         $profile = "superadmin";
-                    } else {
+                    }else {
                         $profile = "admin";
                     }
                     if (!empty($params["default"]) && preg_match('/(GET|ALL)/i', $httpMethod)) {
                         $_profile = ($params["visible"]) ? $profile : 'adminhidden';
-                        if(!array_key_exists($_profile, $routes)) {
+                        if (!array_key_exists($_profile, $routes)) {
                             $routes[$_profile] = array();
                         }
                         $routes[$_profile][] = $params["slug"];
@@ -571,7 +571,7 @@ 
             $expr = preg_quote($expr, '/');
             $expr = str_replace('###', '(.*)', $expr);
             $expr2 = preg_replace('/\(\.\*\)$/', '', $expr);
-            $matched = preg_match('/^' . $expr . '\/?$/i', $path) || preg_match('/^' . $expr2 . '?$/i', $path);
+            $matched = preg_match('/^'.$expr.'\/?$/i', $path) || preg_match('/^'.$expr2.'?$/i', $path);
 
             return $matched;
         }
@@ -609,9 +609,9 @@ 
             if (count($parameters) > 0) foreach ($parameters as $param) {
                 if ($param->isOptional() && !is_array($param->getDefaultValue())) {
                     $params[$param->getName()] = $param->getDefaultValue();
-                    $default = str_replace('{' . $param->getName() . '}', $param->getDefaultValue(), $regex);
+                    $default = str_replace('{'.$param->getName().'}', $param->getDefaultValue(), $regex);
                 }
-            } else $default = $regex;
+            }else $default = $regex;
 
             return array($regex, $default, $params);
         }
@@ -668,16 +668,16 @@ 
          */
         protected function executeCachedRoute($route, $action, $class, $params = NULL)
         {
-            Logger::getInstance()->debugLog(_('Ruta resuelta para ') . $route);
+            Logger::getInstance()->debugLog(_('Ruta resuelta para ').$route);
             $this->session->setSessionKey("__CACHE__", $action);
             $cache = Cache::needCache();
             $execute = TRUE;
             if (FALSE !== $cache && Config::getInstance()->getDebugMode() === FALSE) {
                 $cacheDataName = $this->cache->getRequestCacheHash();
-                $cachedData = $this->cache->readFromCache("templates" . DIRECTORY_SEPARATOR . $cacheDataName, $cache, function () {
+                $cachedData = $this->cache->readFromCache("templates".DIRECTORY_SEPARATOR.$cacheDataName, $cache, function() {
                 });
                 if (NULL !== $cachedData) {
-                    $headers = $this->cache->readFromCache("templates" . DIRECTORY_SEPARATOR . $cacheDataName . ".headers", $cache, function () {
+                    $headers = $this->cache->readFromCache("templates".DIRECTORY_SEPARATOR.$cacheDataName.".headers", $cache, function() {
                     }, Cache::JSON);
                     Template::getInstance()->renderCache($cachedData, $headers);
                     $execute = FALSE;
@@ -726,7 +726,7 @@ 
             $translations = array();
             if (file_exists($absoluteTranslationFileName)) {
                 include($absoluteTranslationFileName);
-            } else {
+            }else {
                 Cache::getInstance()->storeData($absoluteTranslationFileName, "<?php \$translations = array();\n", Cache::TEXT, TRUE);
             }
 

src/base/Security.php

Doc Comments +1 added lines, -1 removed lines

@@ -371,7 +371,7 @@
          * @param array $parts
          * @param int $partLength
          *
-         * @return array
+         * @return string[]
          */
         private static function extractTsAndMod(array &$parts, $partLength)
         {

Spacing +16 added lines, -16 removed lines

@@ -78,13 +78,13 @@ 
         public static function save($user)
         {
             $admins = array();
-            if (file_exists(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json')) {
-                $admins = json_decode(file_get_contents(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json'), TRUE);
+            if (file_exists(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json')) {
+                $admins = json_decode(file_get_contents(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json'), TRUE);
             }
-            $admins[$user['username']]['hash'] = sha1($user['username'] . $user['password']);
+            $admins[$user['username']]['hash'] = sha1($user['username'].$user['password']);
             $admins[$user['username']]['profile'] = $user['profile'];
 
-            return (FALSE !== file_put_contents(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json', json_encode($admins, JSON_PRETTY_PRINT)));
+            return (FALSE !== file_put_contents(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json', json_encode($admins, JSON_PRETTY_PRINT)));
         }
 
         /**
@@ -104,8 +104,8 @@ 
         public function getAdmins()
         {
             $admins = array();
-            if (file_exists(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json')) {
-                $admins = json_decode(file_get_contents(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json'), TRUE);
+            if (file_exists(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json')) {
+                $admins = json_decode(file_get_contents(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json'), TRUE);
             }
 
             return $admins;
@@ -124,7 +124,7 @@ 
         {
             if (!$this->authorized) {
                 $request = Request::getInstance();
-                if (!file_exists(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json')) {
+                if (!file_exists(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json')) {
                     //Si no hay fichero de usuarios redirigimos directamente al gestor
                     return Router::getInstance()->getAdmin()->adminers();
                 }
@@ -137,7 +137,7 @@ 
                 }
                 if (!empty($user) && !empty($admins[$user])) {
                     $auth = $admins[$user]['hash'];
-                    $this->authorized = ($auth == sha1($user . $pass));
+                    $this->authorized = ($auth == sha1($user.$pass));
                     $this->admin = array(
                         'alias'   => $user,
                         'profile' => $admins[$user]['profile'],
@@ -354,12 +354,12 @@ 
             $axis = 0;
             $parts = array();
             try {
-                $partLength = floor(strlen($token) / 10);
-                for ($i = 0, $ct = ceil(strlen($token) / $partLength); $i < $ct; $i++) {
+                $partLength = floor(strlen($token)/10);
+                for ($i = 0, $ct = ceil(strlen($token)/$partLength); $i < $ct; $i++) {
                     $parts[] = substr($token, $axis, $partLength);
                     $axis += $partLength;
                 }
-            } catch(\Exception $e) {
+            }catch (\Exception $e) {
                 $partLength = 0;
             }
 
@@ -399,7 +399,7 @@ 
             $decoded = NULL;
             list($partLength, $parts) = self::extractTokenParts($token);
             list($ts, $mod) = self::extractTsAndMod($parts, $partLength);
-            $hashMod = substr(strtoupper(sha1($module)), strlen($ts) / 2, strlen($ts) * 2);
+            $hashMod = substr(strtoupper(sha1($module)), strlen($ts)/2, strlen($ts)*2);
             if (time() - (integer)$ts < 300 && $hashMod === $mod) {
                 $decoded = implode('', $parts);
             }
@@ -416,16 +416,16 @@ 
         public static function generateToken($secret, $module = 'PSFS')
         {
             $ts = time();
-            $hashModule = substr(strtoupper(sha1($module)), strlen($ts) / 2, strlen($ts) * 2);
+            $hashModule = substr(strtoupper(sha1($module)), strlen($ts)/2, strlen($ts)*2);
             $hash = hash('sha256', $secret);
-            $insert = floor(strlen($hash) / strlen($ts));
+            $insert = floor(strlen($hash)/strlen($ts));
             $j = 0;
             $token = '';
             for ($i = 0, $ct = strlen($ts); $i < $ct; $i++) {
-                $token .= substr($ts, $i, 1) . substr($hash, $j, $insert) . substr($hashModule, $i, 2);
+                $token .= substr($ts, $i, 1).substr($hash, $j, $insert).substr($hashModule, $i, 2);
                 $j += $insert;
             }
-            $token .= substr($hash, ($insert * strlen($ts)), strlen($hash) - ($insert * strlen($ts)));
+            $token .= substr($hash, ($insert*strlen($ts)), strlen($hash) - ($insert*strlen($ts)));
             return $token;
         }
 

src/base/Service.php

Doc Comments +2 added lines, -2 removed lines

@@ -125,8 +125,8 @@
         }
 
         /**
-         * @param $header
-         * @param null $content
+         * @param string $header
+         * @param string $content
          *
          * @return $this
          */

Spacing +2 added lines, -2 removed lines

@@ -177,7 +177,7 @@ 
             $this->url = NULL;
             $this->params = array();
             $this->headers = array();
-            $this->log->debugLog(_("Context service for " . get_called_class() . " cleared!"));
+            $this->log->debugLog(_("Context service for ".get_called_class()." cleared!"));
         }
 
         /**
@@ -208,7 +208,7 @@ 
 
         protected function setOpts()
         {
-            switch(strtoupper($this->type)) {
+            switch (strtoupper($this->type)) {
                 case 'GET':
                 default:
                     curl_setopt($this->con, CURLOPT_CUSTOMREQUEST, "GET");

src/base/Singleton.php

Doc Comments +3 added lines, -3 removed lines

@@ -175,10 +175,10 @@
 
     /**
      * Create the depecency injected
-     * @param $variable
-     * @param $singleton
+     * @param string $variable
+     * @param boolean $singleton
      * @param $classNameSpace
-     * @param $calledClass
+     * @param string $calledClass
      * @return mixed
      */
     private function constructInyectableInstance($variable, $singleton, $classNameSpace, $calledClass)

Spacing +3 added lines, -3 removed lines

@@ -98,10 +98,10 @@ 
             $setter = "set".ucfirst($variable);
             if (method_exists($calledClass, $setter)) {
                 $this->$setter($instance);
-            } else {
+            }else {
                 $this->$variable = $instance;
             }
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             Logger::getInstance()->errorLog($e->getMessage());
         }
         return $this;
@@ -189,7 +189,7 @@ 
         if (true === $singleton && method_exists($varInstanceType, "getInstance")) {
             $instance = $varInstanceType::getInstance();
             return $instance;
-        } else {
+        }else {
             $instance = new $varInstanceType();
             return $instance;
         }

Braces +4 added lines, -2 removed lines

@@ -125,11 +125,13 @@
                 $cacheService->storeData($cacheFilename, $properties, Cache::JSON);
             }
             /** @var \ReflectionProperty $property */
-            if (!empty($properties) && is_array($properties)) foreach ($properties as $property => $class) {
+            if (!empty($properties) && is_array($properties)) {
+                foreach ($properties as $property => $class) {
                 $this->load($property, true, $class);
             }
+            }
             $this->setLoaded();
-        }else {
+        } else {
             $logService->warningLog(get_class($this)." ya cargada");
         }
     }

src/base/types/Api.php

Doc Comments +4 added lines, -4 removed lines

@@ -259,7 +259,7 @@ 
          *
          * @param null|string $pk
          *
-         * @return JsonResponse JSON
+         * @return string|null JSON
          */
         public function get($pk = NULL)
         {
@@ -303,7 +303,7 @@ 
          * @POST
          * @ROUTE /api/{__API__}
          *
-         * @return JsonResponse JSON
+         * @return string|null JSON
          */
         public function post()
         {
@@ -333,7 +333,7 @@ 
          *
          * @param string $pk
          *
-         * @return JsonResponse JSON
+         * @return string|null JSON
          */
         public function delete($pk = NULL)
         {
@@ -362,7 +362,7 @@ 
          *
          * @param string $pk
          *
-         * @return JsonResponse
+         * @return string|null
          *
          */
         public function put($pk)

Spacing +18 added lines, -18 removed lines

@@ -179,7 +179,7 @@ 
          *
          * @param ModelCriteria $query
          */
-        private function addOrders(ModelCriteria &$query)
+        private function addOrders(ModelCriteria & $query)
         {
             $orderAdded = FALSE;
             foreach ($this->order->getOrders() as $field => $direction) {
@@ -187,7 +187,7 @@ 
                     $orderAdded = TRUE;
                     if ($direction === Order::ASC) {
                         $query->addAscendingOrderByColumn($field);
-                    } else {
+                    }else {
                         $query->addDescendingOrderByColumn($field);
                     }
                 }
@@ -201,9 +201,9 @@ 
          * Add extra columns to pagination query
          * @param ModelCriteria $query
          */
-        private function addExtraColumns(ModelCriteria &$query)
+        private function addExtraColumns(ModelCriteria & $query)
         {
-            if(!empty($this->extraColumns)) {
+            if (!empty($this->extraColumns)) {
                 foreach ($this->extraColumns as $expression => $columnName) {
                     $query->withColumn($expression, $columnName);
                 }
@@ -214,7 +214,7 @@ 
          * Method that allow joins between models
          * @param ModelCriteria $query
          */
-        protected function joinTables(ModelCriteria &$query)
+        protected function joinTables(ModelCriteria & $query)
         {
             //TODO for specific implementations
         }
@@ -224,7 +224,7 @@ 
          *
          * @param ModelCriteria $query
          */
-        private function addFilters(ModelCriteria &$query)
+        private function addFilters(ModelCriteria & $query)
         {
             if (count($this->query) > 0) {
                 foreach ($this->query as $field => $value) {
@@ -241,7 +241,7 @@ 
                         } elseif (preg_match('/^(\'|\")(.*)(\'|\")$/', $value)) {
                             $text = preg_replace('/(\'|\")/', '', $value);
                             $query->filterBy($tableField, $text, Criteria::LIKE);
-                        } else {
+                        }else {
                             $query->filterBy($tableField, $value, Criteria::EQUAL);
                         }
                     }
@@ -264,10 +264,10 @@ 
                 list($page, $limit) = $this->extractPagination();
                 if ($limit == -1) {
                     $this->list = $query->find($this->con);
-                } else {
+                }else {
                     $this->list = $query->paginate($page, $limit, $this->con);
                 }
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 Logger::getInstance(get_class($this))->errorLog($e->getMessage());
             }
         }
@@ -284,7 +284,7 @@ 
                 $this->joinTables($query);
                 $this->addExtraColumns($query);
                 $this->model = $query->findPk($pk);
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 Logger::getInstance(get_class($this))->errorLog($e->getMessage());
             }
         }
@@ -308,7 +308,7 @@ 
             $pages = 1;
             if (NULL === $pk || '' === $pk) {
                 list($return, $total, $pages) = $this->getList();
-            } else {
+            }else {
                 list($code, $return) = $this->getSingleResult($pk);
 
             }
@@ -359,7 +359,7 @@ 
                     $saved = TRUE;
                     $model = $this->model->toArray();
                 }
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 jpre($e->getMessage(), true);
                 Logger::getInstance()->errorLog($e->getMessage());
             }
@@ -388,7 +388,7 @@ 
                         $this->model->delete($this->con);
                         $deleted = TRUE;
                     }
-                } catch (\Exception $e) {
+                }catch (\Exception $e) {
                     Logger::getInstance(get_class($this->model))->errorLog($e->getMessage());
                 }
             }
@@ -431,7 +431,7 @@ 
          */
         private function extractQuery()
         {
-            $queryReflector = new \ReflectionClass($this->getModelNamespace() . "Query");
+            $queryReflector = new \ReflectionClass($this->getModelNamespace()."Query");
             /** @var \Propel\Runtime\ActiveQuery\ModelCriteria $query */
             $query = $queryReflector->getMethod('create')->invoke(NULL);
 
@@ -475,7 +475,7 @@ 
             if ($this->con->inTransaction()) {
                 if ($status === 200) {
                     $this->con->commit();
-                } else {
+                }else {
                     $this->con->rollBack();
                 }
             }
@@ -518,7 +518,7 @@ 
             return $this->render('api.admin.html.twig', array(
                 "api"    => $this->getApi(),
                 "domain" => $this->domain,
-                "url"    => preg_replace('/\/$/', '', $this->getRoute(strtolower('api-' . $this->getApi() . "-pk"), TRUE)),
+                "url"    => preg_replace('/\/$/', '', $this->getRoute(strtolower('api-'.$this->getApi()."-pk"), TRUE)),
             ));
         }
 
@@ -554,7 +554,7 @@ 
                 $return = $this->list->toArray();
                 $total = $this->list->getNbResults();
                 $pages = $this->list->getLastPage();
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 Logger::getInstance(get_class($this))->errorLog($e->getMessage());
             }
 
@@ -574,7 +574,7 @@ 
             $return = array();
             if (NULL === $model) {
                 $code = 404;
-            } else {
+            }else {
                 $return = $model->toArray();
             }
             return array($code, $return);

src/base/types/Form.php

Doc Comments +6 added lines, -2 removed lines

@@ -42,6 +42,10 @@ 
          * @return Form
          */
         public function setEncType($enctype) { $this->enctype = $enctype; return $this; }
+
+        /**
+         * @param string|null $action
+         */
         public function setAction($action) { $this->action = $action; return $this; }
         public function setMethod($method) { $this->method = $method; return $this; }
         public function setAttrs(array $attrs) { $this->attrs = $attrs; return $this; }
@@ -466,7 +470,7 @@ 
         }
 
         /**
-         * @param $token_field
+         * @param string $token_field
          * @return bool
          */
         protected function existsFormToken($token_field) {
@@ -494,7 +498,7 @@ 
         /**
          * Método que extrae el valor de un campo del formulario del modelo
          * @param $field
-         * @param $value
+         * @param ObjectCollection $value
          * @param $type
          * @return mixed
          */

Spacing +10 added lines, -10 removed lines

@@ -12,7 +12,7 @@ 
     use PSFS\base\Singleton;
     use PSFS\base\types\interfaces\FormType;
 
-    abstract class Form extends Singleton implements FormType{
+    abstract class Form extends Singleton implements FormType {
 
         /**
          * Variables de formulario
@@ -31,7 +31,7 @@ 
 
         public function __construct($model = null)
         {
-            if(null !== $model) {
+            if (null !== $model) {
                 $this->model = $model;
             }
         }
@@ -122,7 +122,7 @@ 
          */
         public function getFieldValue($name) {
             $value = null;
-            if (null !== $this->getField($name)){
+            if (null !== $this->getField($name)) {
                 $field = $this->getField($name);
                 $value = (array_key_exists('value', $field) && null !== $field['value']) ? $field['value'] : null;
             }
@@ -323,7 +323,7 @@ 
                 $model->save();
                 $save = true;
                 Logger::getInstance()->infoLog(get_class($this->model)." guardado con id ".$this->model->getPrimaryKey());
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 Logger::getInstance()->errorLog($e->getMessage());
                 throw new FormException($e->getMessage(), $e->getCode(), $e);
             }
@@ -369,10 +369,10 @@ 
             if (array_key_exists($key, $data[$form_name])) {
                 if (preg_match("/id/i", $key) && ($data[$form_name][$key] === 0 || $data[$form_name][$key] === "%" || $data[$form_name][$key] === "")) {
                     $field["value"] = null;
-                } else {
+                }else {
                     $field["value"] = $data[$form_name][$key];
                 }
-            } else {
+            }else {
                 unset($field["value"]);
             }
             return array($data, $field);
@@ -416,10 +416,10 @@ 
                 if (array_key_exists('class_id', $field) && method_exists($class, "get".$field["class_id"])) {
                     $method = "get".$field["class_id"];
                     $data[] = $class->$method();
-                } else {
+                }else {
                     $data[] = $class->getPrimaryKey();
                 }
-            } else {
+            }else {
                 $data[] = $val;
             }
 
@@ -449,11 +449,11 @@ 
                             $field["value"] = $value;
                         } elseif ($value instanceof \DateTime) {
                             $field["value"] = $value->format("Y-m-d H:i:s");
-                        } else {
+                        }else {
                             $field["value"] = $value->getPrimaryKey();
                         }
                     }
-                } else {
+                }else {
                     $field["value"] = $value;
                 }
             }

Braces +4 added lines, -2 removed lines

@@ -157,7 +157,9 @@ 
             if ($valid) {
                 if (count($this->fields) > 0) {
                     foreach ($this->fields as $key => &$field) {
-                        if ($key === self::SEPARATOR) continue;
+                        if ($key === self::SEPARATOR) {
+                            continue;
+                        }
                         list($field, $valid) = $this->checkFieldValidation($field, $key);
                     }
                 }
@@ -444,7 +446,7 @@ 
                     //Si es una relación múltiple
                     if ($value instanceof ObjectCollection) {
                         $field = $this->computeModelFieldValue($field, $value, $type);
-                    }else { //O una relación unitaria
+                    } else { //O una relación unitaria
                         if (method_exists($value, "__toString")) {
                             $field["value"] = $value;
                         } elseif ($value instanceof \DateTime) {