psfs / core

src/base/types/interfaces/ControllerInterface.php

Doc Comments +6 added lines

@@ -8,7 +8,13 @@
  */
 interface ControllerInterface
 {
+    /**
+     * @return string|null
+     */
     public function render($template, array $vars = [], $cookies = []);
 
+    /**
+     * @return void
+     */
     public function response($response, $type = 'text/html');
 }

src/services/AdminServices.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 use PSFS\base\config\Config;
 use PSFS\base\Security;
 use PSFS\base\Service;
-use PSFS\controller\Admin;
 use Symfony\Component\Finder\Finder;
 
 class AdminServices extends Service

Braces +12 added lines, -4 removed lines

@@ -61,11 +61,13 @@ 
      */
     private function parseAdmins(&$admins)
     {
-        if (!empty($admins)) foreach ($admins as &$admin) {
+        if (!empty($admins)) {
+            foreach ($admins as &$admin) {
             if (isset($admin["profile"])) {
                 switch ($admin["profile"]) {
                     case Security::MANAGER_ID_TOKEN:
                         $admin['class'] = 'warning';
+        }
                         break;
                     case Security::ADMIN_ID_TOKEN:
                         $admin['class'] = 'info';
@@ -95,7 +97,9 @@ 
             $size = $file->getSize() / 8 / 1024;
             $time = date("c", $file->getMTime());
             $dateTime = new \DateTime($time);
-            if (!isset($logs[$dateTime->format("Y")])) $logs[$dateTime->format("Y")] = array();
+            if (!isset($logs[$dateTime->format("Y")])) {
+                $logs[$dateTime->format("Y")] = array();
+            }
             $logs[$dateTime->format("Y")][$dateTime->format("m")][$time] = array(
                 "filename" => $file->getFilename(),
                 "size" => round($size, 3)
@@ -136,7 +140,9 @@ 
                 $detailLog[] = array_merge(array(
                     "log" => $line,
                 ), $detail);
-                if (count($detailLog) >= 1000) break;
+                if (count($detailLog) >= 1000) {
+                    break;
+                }
             }
             $log = $detailLog;
         }
@@ -160,7 +166,9 @@ 
 
                 $detail = json_decode($match[0][0], TRUE);
             }
-            if (empty($detail)) $detail = array();
+            if (empty($detail)) {
+                $detail = array();
+            }
             preg_match_all('/\>\ (.*):/i', $line, $match);
 
             $type = (isset($match[1][0])) ? $match[1][0] : '';

Spacing +5 added lines, -5 removed lines

@@ -34,7 +34,7 @@ 
     {
         $platform = trim(Config::getInstance()->get("platform.name"));
         header('HTTP/1.1 401 Unauthorized');
-        header('WWW-Authenticate: Basic Realm="' . $platform . '"');
+        header('WWW-Authenticate: Basic Realm="'.$platform.'"');
         echo _("Zona restringida");
         exit();
     }
@@ -75,7 +75,7 @@ 
                         $admin['class'] = 'primary';
                         break;
                 }
-            } else {
+            }else {
                 $admin["class"] = "primary";
             }
         }
@@ -92,7 +92,7 @@ 
         $logs = array();
         /** @var \SplFileInfo $file */
         foreach ($files as $file) {
-            $size = $file->getSize() / 8 / 1024;
+            $size = $file->getSize()/8/1024;
             $time = date("c", $file->getMTime());
             $dateTime = new \DateTime($time);
             if (!isset($logs[$dateTime->format("Y")])) $logs[$dateTime->format("Y")] = array();
@@ -128,7 +128,7 @@ 
             $time = date("c", $file->getMTime());
             $dateTime = new \DateTime($time);
             $monthOpen = $dateTime->format("m");
-            $content = file($file->getPath() . DIRECTORY_SEPARATOR . $file->getFilename());
+            $content = file($file->getPath().DIRECTORY_SEPARATOR.$file->getFilename());
             krsort($content);
             $detailLog = array();
             foreach ($content as &$line) {
@@ -181,7 +181,7 @@ 
                     break;
             }
 
-        } catch (\Exception $e) {
+        }catch (\Exception $e) {
             $detail = array(
                 "type" => "danger",
             );

src/base/config/LoginForm.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
             ))
             ->addButton('submit', _("Acceder como {{username}}"))
             ->addButton("cancel", _("Cancelar"), "button", array(
-                "onclick" => "javacript:location.href = \"" . Router::getInstance()->getRoute('') . "\";",
+                "onclick" => "javacript:location.href = \"".Router::getInstance()->getRoute('')."\";",
                 "class" => "btn-link",
             ));
     }

src/base/dto/Order.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
     {
         if (preg_match('/^asc$/i', $direction)) {
             return Order::ASC;
-        } else {
+        }else {
             return Order::DESC;
         }
     }

src/base/types/AuthApi.php

Spacing +2 added lines, -2 removed lines

@@ -26,13 +26,13 @@
     {
         $namespace = explode('\\', $this->getModelTableMap());
         $module = strtolower($namespace[0]);
-        $secret = Config::getInstance()->get($module . '.api.secret');
+        $secret = Config::getInstance()->get($module.'.api.secret');
         if (NULL === $secret) {
             $secret = Config::getInstance()->get("api.secret");
         }
         if (NULL === $secret) {
             $auth = TRUE;
-        } else {
+        }else {
             $token = Request::getInstance()->getHeader('X-API-SEC-TOKEN');
             if (array_key_exists('API_TOKEN', $this->query)) {
                 $token = $this->query['API_TOKEN'];

src/base/extension/AssetsNode.php

Spacing +3 added lines, -3 removed lines

@@ -30,16 +30,16 @@
         $scripts = $this->getNode("scripts");
 
         //Creamos el parser
-        $compiler->addDebugInfo($scripts)->write('$parser = new \\PSFS\\base\\extension\\AssetsParser(\'' . $this->type . '\')')
+        $compiler->addDebugInfo($scripts)->write('$parser = new \\PSFS\\base\\extension\\AssetsParser(\''.$this->type.'\')')
             ->raw(";\n");
 
         //Asociamos el hash
-        $compiler->write('$parser->setHash(\'' . $this->hash . '\')')
+        $compiler->write('$parser->setHash(\''.$this->hash.'\')')
             ->raw(";\n");
 
         //Asociamos los ficheros
         foreach ($scripts->getAttribute("value") as $value) {
-            $compiler->write('$parser->addFile(\'' . $value . '\')')->raw(";\n");
+            $compiler->write('$parser->addFile(\''.$value.'\')')->raw(";\n");
         }
 
         //Procesamos los ficheros

src/base/extension/AssetsTokenParser.php

Spacing +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
         $tmp = array();
         if (NULL === $node) {
             $node = $value;
-        } else {
+        }else {
             $tmp = $this->getTmpAttribute($node);
         }
         $tmp[] = $value->getAttribute("value");

src/base/Service.php

Doc Comments +3 added lines, -3 removed lines

@@ -116,7 +116,7 @@ 
     /**
      * Add request param
      *
-     * @param $key
+     * @param integer $key
      * @param null $value
      *
      * @return \PSFS\base\Service
@@ -153,8 +153,8 @@ 
     }
 
     /**
-     * @param $header
-     * @param null $content
+     * @param string $header
+     * @param string $content
      *
      * @return $this
      */

Spacing +22 added lines, -22 removed lines

@@ -69,7 +69,7 @@ 
     protected $isMultipart = false;
 
     private function closeConnection() {
-        if(null !== $this->con) {
+        if (null !== $this->con) {
             curl_close($this->con);
         }
     }
@@ -232,7 +232,7 @@ 
      */
     public function setIsJson($isJson = true) {
         $this->isJson = $isJson;
-        if($isJson) {
+        if ($isJson) {
             $this->setIsMultipart(false);
         }
     }
@@ -249,7 +249,7 @@ 
      */
     public function setIsMultipart($isMultipart = true) {
         $this->isMultipart = $isMultipart;
-        if($isMultipart) {
+        if ($isMultipart) {
             $this->setIsJson(false);
         }
     }
@@ -269,7 +269,7 @@ 
         $this->url = NULL;
         $this->params = array();
         $this->headers = array();
-        Logger::log('Context service for ' . static::class . ' cleared!');
+        Logger::log('Context service for '.static::class.' cleared!');
         $this->closeConnection();
     }
 
@@ -312,18 +312,18 @@ 
     }
 
     protected function applyOptions() {
-        if(count($this->options)) {
+        if (count($this->options)) {
             curl_setopt_array($this->con, $this->options);
         }
     }
 
     protected function applyHeaders() {
         $headers = [];
-        foreach($this->headers as $key => $value) {
-            $headers[] = $key . ': ' . $value;
+        foreach ($this->headers as $key => $value) {
+            $headers[] = $key.': '.$value;
         }
         $headers[self::PSFS_TRACK_HEADER] = Logger::getUid();
-        if(count($headers)) {
+        if (count($headers)) {
             curl_setopt($this->con, CURLOPT_HTTPHEADER, $headers);
         }
     }
@@ -334,18 +334,18 @@ 
             case Request::VERB_GET:
             default:
                 $this->addOption(CURLOPT_CUSTOMREQUEST, Request::VERB_GET);
-                if(!empty($this->params)) {
+                if (!empty($this->params)) {
                     $sep = !preg_match('/\?/', $this->getUrl()) ? '?' : '';
-                    $this->url = $this->url . $sep . http_build_query($this->params);
+                    $this->url = $this->url.$sep.http_build_query($this->params);
                 }
                 break;
             case Request::VERB_POST:
                 $this->addOption(CURLOPT_CUSTOMREQUEST, Request::VERB_POST);
-                if($this->getIsJson()) {
+                if ($this->getIsJson()) {
                     $this->addOption(CURLOPT_POSTFIELDS, json_encode($this->params));
-                } elseif($this->getIsMultipart()) {
+                } elseif ($this->getIsMultipart()) {
                     $this->addOption(CURLOPT_POSTFIELDS, $this->params);
-                } else {
+                }else {
                     $this->addOption(CURLOPT_POSTFIELDS, http_build_query($this->params));
                 }
                 break;
@@ -355,21 +355,21 @@ 
             case Request::VERB_PUT:
                 $this->addOption(CURLOPT_CUSTOMREQUEST, Request::VERB_PUT);
 
-                if($this->getIsJson()) {
+                if ($this->getIsJson()) {
                     $this->addOption(CURLOPT_POSTFIELDS, json_encode($this->params));
-                } elseif($this->getIsMultipart()) {
+                } elseif ($this->getIsMultipart()) {
                     $this->addOption(CURLOPT_POSTFIELDS, $this->params);
-                } else {
+                }else {
                     $this->addOption(CURLOPT_POSTFIELDS, http_build_query($this->params));
                 }
                 break;
             case Request::VERB_PATCH:
                 $this->addOption(CURLOPT_CUSTOMREQUEST, Request::VERB_PATCH);
-                if($this->getIsJson()) {
+                if ($this->getIsJson()) {
                     $this->addOption(CURLOPT_POSTFIELDS, json_encode($this->params));
-                } elseif($this->getIsMultipart()) {
+                } elseif ($this->getIsMultipart()) {
                     $this->addOption(CURLOPT_POSTFIELDS, $this->params);
-                } else {
+                }else {
                     $this->addOption(CURLOPT_POSTFIELDS, http_build_query($this->params));
                 }
                 break;
@@ -386,14 +386,14 @@ 
         $this->setDefaults();
         $this->applyOptions();
         $this->applyHeaders();
-        if('debug' === Config::getParam('log.level')) {
+        if ('debug' === Config::getParam('log.level')) {
             curl_setopt($this->con, CURLOPT_VERBOSE, true);
             $verbose = fopen('php://temp', 'w+');
             curl_setopt($this->con, CURLOPT_STDERR, $verbose);
         }
         $result = curl_exec($this->con);
         $this->result = $this->isJson ? json_decode($result, true) : $result;
-        if('debug' === Config::getParam('log.level')) {
+        if ('debug' === Config::getParam('log.level')) {
             rewind($verbose);
             $verboseLog = stream_get_contents($verbose);
             Logger::log($verboseLog, LOG_DEBUG, [
@@ -403,7 +403,7 @@ 
             ]);
             $this->info['verbose'] = $verboseLog;
         }
-        Logger::log($this->url . ' response: ', LOG_DEBUG, is_array($this->result) ? $this->result : [$this->result]);
+        Logger::log($this->url.' response: ', LOG_DEBUG, is_array($this->result) ? $this->result : [$this->result]);
         $this->info = array_merge($this->info, curl_getinfo($this->con));
     }
 

src/base/types/helpers/AdminHelper.php

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,7 @@ 
     {
         $routes = [];
         foreach ($systemRoutes as $route => $params) {
-            if(isset($params['http']) && 'GET' === $params['http'] && preg_match('/^\/admin(\/|$)/', $params['default'])) {
+            if (isset($params['http']) && 'GET' === $params['http'] && preg_match('/^\/admin(\/|$)/', $params['default'])) {
                 $module = strtoupper($params['module']);
                 $mode = $params["visible"] ? 'visible' : 'hidden';
                 $routes[$module][$mode][] = [
@@ -41,11 +41,11 @@ 
                 ];
             }
         }
-        foreach($routes as $module => &$route) {
-            if(array_key_exists('visible', $route)) {
+        foreach ($routes as $module => &$route) {
+            if (array_key_exists('visible', $route)) {
                 uasort($route["visible"], '\PSFS\base\types\helpers\AdminHelper::sortByLabel');
             }
-            if(array_key_exists('hidden', $route)) {
+            if (array_key_exists('hidden', $route)) {
                 uasort($route["hidden"], '\PSFS\base\types\helpers\AdminHelper::sortByLabel');
             }
         }