Passed
Push — master ( dd9457...9bc0cf )
by Fran
03:07
created
src/base/Security.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
             session_start();
74 74
         }
75 75
         // Fix for phpunits
76
-        if(!isset($_SESSION)) {
76
+        if (!isset($_SESSION)) {
77 77
             $_SESSION = [];
78 78
         }
79 79
     }
@@ -127,11 +127,11 @@  discard block
 block discarded – undo
127 127
     public static function save($user)
128 128
     {
129 129
         $saved = true;
130
-        $admins = Cache::getInstance()->getDataFromFile(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json', Cache::JSONGZ, true) ?: [];
131
-        $admins[$user['username']]['hash'] = sha1($user['username'] . $user['password']);
130
+        $admins = Cache::getInstance()->getDataFromFile(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json', Cache::JSONGZ, true) ?: [];
131
+        $admins[$user['username']]['hash'] = sha1($user['username'].$user['password']);
132 132
         $admins[$user['username']]['profile'] = $user['profile'];
133 133
 
134
-        Cache::getInstance()->storeData(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json', $admins, Cache::JSONGZ, true);
134
+        Cache::getInstance()->storeData(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json', $admins, Cache::JSONGZ, true);
135 135
         return $saved;
136 136
     }
137 137
 
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
      */
175 175
     public function getAdmins()
176 176
     {
177
-        return Cache::getInstance()->getDataFromFile(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json', Cache::JSONGZ, true);
177
+        return Cache::getInstance()->getDataFromFile(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json', Cache::JSONGZ, true);
178 178
     }
179 179
 
180 180
     /**
@@ -199,9 +199,9 @@  discard block
 block discarded – undo
199 199
                 }
200 200
                 if (!empty($user) && !empty($admins[$user])) {
201 201
                     $auth = $admins[$user]['hash'];
202
-                    $this->authorized = ($auth === sha1($user . $pass));
202
+                    $this->authorized = ($auth === sha1($user.$pass));
203 203
                     if ($this->authorized) {
204
-                        $this->updateAdmin($user , $admins[$user]['profile']);
204
+                        $this->updateAdmin($user, $admins[$user]['profile']);
205 205
                         ResponseHelper::setCookieHeaders([
206 206
                             [
207 207
                                 'name' => $this->getHash(),
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
                         ]);
213 213
                         $this->setSessionKey(self::LOGGED_USER_TOKEN, base64_encode("{$user}:{$pass}"));
214 214
                     }
215
-                } else {
215
+                }else {
216 216
                     $this->admin = null;
217 217
                     $this->setSessionKey(self::ADMIN_ID_TOKEN, null);
218 218
                 }
Please login to merge, or discard this patch.
src/base/types/helpers/ApiFormHelper.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -17,11 +17,11 @@  discard block
 block discarded – undo
17 17
     public static function checkApiActions($namespace, $domain, $api) {
18 18
         $actions = [];
19 19
         $reflector = new \ReflectionClass($namespace);
20
-        if(null !== $reflector) {
21
-            foreach($reflector->getMethods(\ReflectionMethod::IS_PUBLIC) as $apiAction) {
20
+        if (null !== $reflector) {
21
+            foreach ($reflector->getMethods(\ReflectionMethod::IS_PUBLIC) as $apiAction) {
22 22
                 $docComments = $apiAction->getDocComment();
23 23
                 $action = self::extractAction($docComments);
24
-                if(null !== $action) {
24
+                if (null !== $action) {
25 25
                     list($route, $info) = RouterHelper::extractRouteInfo($apiAction, $api, $domain);
26 26
                     list($method, $cleanRoute) = RouterHelper::extractHttpRoute($route);
27 27
                     $formAction = new FormAction();
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
     {
45 45
         $action = null;
46 46
         if (false !== preg_match('/@action\s+([^\s]+)/', $doc, $matches)) {
47
-            if(count($matches) > 1) {
47
+            if (count($matches) > 1) {
48 48
                 list(, $action) = $matches;
49 49
             }
50 50
         }
Please login to merge, or discard this patch.
src/bootstrap.php 2 patches
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -6,20 +6,20 @@  discard block
 block discarded – undo
6 6
 
7 7
 defined('SOURCE_DIR') or define('SOURCE_DIR', __DIR__);
8 8
 if (preg_match('/vendor/', SOURCE_DIR)) {
9
-    defined('BASE_DIR') or define('BASE_DIR', SOURCE_DIR . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR . '..');
10
-    defined('CORE_DIR') or define('CORE_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'src');
11
-} else {
12
-    defined('BASE_DIR') or define('BASE_DIR', SOURCE_DIR . DIRECTORY_SEPARATOR . '..');
13
-    defined('CORE_DIR') or define('CORE_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'modules');
9
+    defined('BASE_DIR') or define('BASE_DIR', SOURCE_DIR.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'..');
10
+    defined('CORE_DIR') or define('CORE_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'src');
11
+}else {
12
+    defined('BASE_DIR') or define('BASE_DIR', SOURCE_DIR.DIRECTORY_SEPARATOR.'..');
13
+    defined('CORE_DIR') or define('CORE_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'modules');
14 14
 }
15
-defined('VENDOR_DIR') or define('VENDOR_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'vendor');
16
-defined('LOG_DIR') or define('LOG_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'logs');
17
-defined('CACHE_DIR') or define('CACHE_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'cache');
18
-defined('CONFIG_DIR') or define('CONFIG_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'config');
19
-defined('WEB_DIR') or define('WEB_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'html');
20
-defined('LOCALE_DIR') or define('LOCALE_DIR', BASE_DIR . DIRECTORY_SEPARATOR . 'locale');
15
+defined('VENDOR_DIR') or define('VENDOR_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'vendor');
16
+defined('LOG_DIR') or define('LOG_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'logs');
17
+defined('CACHE_DIR') or define('CACHE_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'cache');
18
+defined('CONFIG_DIR') or define('CONFIG_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'config');
19
+defined('WEB_DIR') or define('WEB_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'html');
20
+defined('LOCALE_DIR') or define('LOCALE_DIR', BASE_DIR.DIRECTORY_SEPARATOR.'locale');
21 21
 
22
-if(!class_exists(bootstrap::class)) {
22
+if (!class_exists(bootstrap::class)) {
23 23
     /**
24 24
      * Class Bootstrap
25 25
      * @package PSFS
@@ -27,9 +27,9 @@  discard block
 block discarded – undo
27 27
     class bootstrap {
28 28
         protected static $loaded = false;
29 29
         public static function load() {
30
-            if(!self::$loaded) {
30
+            if (!self::$loaded) {
31 31
                 defined('PSFS_BOOTSTRAP_LOADED') or define('PSFS_BOOTSTRAP_LOADED', true);
32
-                if(class_exists("\\PSFS\\base\\Logger")) \PSFS\base\Logger::log('Bootstrap initialized', LOG_INFO);
32
+                if (class_exists("\\PSFS\\base\\Logger")) \PSFS\base\Logger::log('Bootstrap initialized', LOG_INFO);
33 33
                 self::$loaded = true;
34 34
             }
35 35
         }
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,9 @@
 block discarded – undo
29 29
         public static function load() {
30 30
             if(!self::$loaded) {
31 31
                 defined('PSFS_BOOTSTRAP_LOADED') or define('PSFS_BOOTSTRAP_LOADED', true);
32
-                if(class_exists("\\PSFS\\base\\Logger")) \PSFS\base\Logger::log('Bootstrap initialized', LOG_INFO);
32
+                if(class_exists("\\PSFS\\base\\Logger")) {
33
+                    \PSFS\base\Logger::log('Bootstrap initialized', LOG_INFO);
34
+                }
33 35
                 self::$loaded = true;
34 36
             }
35 37
         }
Please login to merge, or discard this patch.
src/base/extension/CustomTranslateExtension.php 2 patches
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -27,22 +27,22 @@  discard block
 block discarded – undo
27 27
      * @param bool $use_base
28 28
      */
29 29
     protected static function checkLoad($custom_key = null, $force_reload = false, $use_base = false) {
30
-        if($force_reload) self::dropInstance();
30
+        if ($force_reload) self::dropInstance();
31 31
         self::$locale = I18nHelper::extractLocale(Security::getInstance()->getSessionKey(I18nHelper::PSFS_SESSION_LANGUAGE_KEY));
32 32
         self::$generate = (boolean)Config::getParam('i18n.autogenerate', false);
33
-        if(!$use_base) {
33
+        if (!$use_base) {
34 34
             $custom_key = $custom_key ?: Security::getInstance()->getSessionKey(self::CUSTOM_LOCALE_SESSION_KEY);
35 35
         }
36
-        if(null !== $custom_key) {
37
-            Logger::log('[' . self::class . '] Custom key detected: ' . $custom_key, LOG_INFO);
38
-            self::$filename = implode(DIRECTORY_SEPARATOR, [LOCALE_DIR, 'custom', $custom_key, self::$locale . '.json']);
39
-        } else {
40
-            self::$filename = implode(DIRECTORY_SEPARATOR, [LOCALE_DIR, 'custom', self::$locale . '.json']);
36
+        if (null !== $custom_key) {
37
+            Logger::log('['.self::class.'] Custom key detected: '.$custom_key, LOG_INFO);
38
+            self::$filename = implode(DIRECTORY_SEPARATOR, [LOCALE_DIR, 'custom', $custom_key, self::$locale.'.json']);
39
+        }else {
40
+            self::$filename = implode(DIRECTORY_SEPARATOR, [LOCALE_DIR, 'custom', self::$locale.'.json']);
41 41
         }
42
-        if(file_exists(self::$filename)) {
43
-            Logger::log('[' . self::class . '] Custom locale detected: ' . $custom_key . ' [' . self::$locale . ']', LOG_INFO);
42
+        if (file_exists(self::$filename)) {
43
+            Logger::log('['.self::class.'] Custom locale detected: '.$custom_key.' ['.self::$locale.']', LOG_INFO);
44 44
             self::$translations = json_decode(file_get_contents(self::$filename), true);
45
-        } elseif(null !== $custom_key) {
45
+        } elseif (null !== $custom_key) {
46 46
             self::checkLoad(null, $force_reload, true);
47 47
         }
48 48
     }
@@ -83,12 +83,12 @@  discard block
 block discarded – undo
83 83
      */
84 84
     public static function _($message, $custom_key = null, $force_reload = false) {
85 85
         self::checkLoad($custom_key, $force_reload);
86
-        if(array_key_exists($message, self::$translations)) {
87
-            $translation =  self::$translations[$message];
88
-        } else {
86
+        if (array_key_exists($message, self::$translations)) {
87
+            $translation = self::$translations[$message];
88
+        }else {
89 89
             $translation = gettext($message);
90 90
         }
91
-        if(self::$generate) {
91
+        if (self::$generate) {
92 92
             self::generate($message, $translation);
93 93
         }
94 94
         return $translation;
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
      * @param string $translation
100 100
      */
101 101
     protected static function generate($message, $translation) {
102
-        if(!array_key_exists($message, self::$translations)) {
102
+        if (!array_key_exists($message, self::$translations)) {
103 103
             self::$translations[$message] = $translation;
104 104
         }
105 105
         file_put_contents(self::$filename, json_encode(array_unique(self::$translations), JSON_UNESCAPED_UNICODE | JSON_PRETTY_PRINT));
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -27,7 +27,9 @@
 block discarded – undo
27 27
      * @param bool $use_base
28 28
      */
29 29
     protected static function checkLoad($custom_key = null, $force_reload = false, $use_base = false) {
30
-        if($force_reload) self::dropInstance();
30
+        if($force_reload) {
31
+            self::dropInstance();
32
+        }
31 33
         self::$locale = I18nHelper::extractLocale(Security::getInstance()->getSessionKey(I18nHelper::PSFS_SESSION_LANGUAGE_KEY));
32 34
         self::$generate = (boolean)Config::getParam('i18n.autogenerate', false);
33 35
         if(!$use_base) {
Please login to merge, or discard this patch.
src/base/dto/JsonResponse.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@
 block discarded – undo
43 43
         $this->success = $result;
44 44
         $this->total = $total ?: (is_array($data) ? count($data) : 0);
45 45
         $this->pages = $pages;
46
-        if(null !== $message) {
46
+        if (null !== $message) {
47 47
             $this->message = $message;
48 48
         }
49 49
     }
Please login to merge, or discard this patch.
src/base/types/helpers/SlackHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
             'text' => 'PSFS Error notifier',
29 29
             'attachments' => [
30 30
                 [
31
-                    "text" => $file . (strlen($line) ? ' [' . $line . ']' : ''),
31
+                    "text" => $file.(strlen($line) ? ' ['.$line.']' : ''),
32 32
                     "color" => "danger",
33 33
                     "title" => $message,
34 34
                     'fallback' => 'PSFS Error notifier',
Please login to merge, or discard this patch.
src/base/types/traits/SystemTrait.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
                 $use /= 1024;
39 39
                 break;
40 40
             case "MBytes":
41
-                $use /= (1024 * 1024);
41
+                $use /= (1024*1024);
42 42
                 break;
43 43
             case "Bytes":
44 44
             default:
@@ -62,34 +62,34 @@  discard block
 block discarded – undo
62 62
     protected function bindWarningAsExceptions()
63 63
     {
64 64
         Logger::log('Added handlers for errors');
65
-        if(Config::getParam('debug')) {
65
+        if (Config::getParam('debug')) {
66 66
             Logger::log('Setting error_reporting as E_ALL');
67 67
             ini_set('error_reporting', E_ALL);
68 68
             ini_set('display_errors', 1);
69 69
         }
70 70
         //Warning & Notice handler
71
-        set_error_handler(function ($errno, $errstr, $errfile, $errline) {
71
+        set_error_handler(function($errno, $errstr, $errfile, $errline) {
72 72
             Logger::log($errstr, LOG_CRIT, ['file' => $errfile, 'line' => $errline, 'errno' => $errno]);
73 73
             return true;
74 74
         }, E_ALL | E_STRICT | E_DEPRECATED | E_USER_DEPRECATED | E_ERROR | E_CORE_ERROR | E_COMPILE_ERROR);
75 75
 
76
-        register_shutdown_function(function () {
76
+        register_shutdown_function(function() {
77 77
             $error = error_get_last() or json_last_error() or preg_last_error() or \DateTime::getLastErrors();
78
-            if( $error !== NULL) {
78
+            if ($error !== NULL) {
79 79
                 $errno   = $error["type"];
80 80
                 $errfile = $error["file"];
81 81
                 $errline = $error["line"];
82 82
                 $errstr  = $error["message"];
83 83
                 Logger::log($errstr, LOG_CRIT, ['file' => $errfile, 'line' => $errline, 'errno' => $errno]);
84
-                if(null !== Config::getParam('log.slack.hook')) {
84
+                if (null !== Config::getParam('log.slack.hook')) {
85 85
                     SlackHelper::getInstance()->trace($errstr, $errfile, $errline, $error);
86 86
                 }
87 87
             }
88 88
 
89
-            if(self::getTs() > 10 && null !== Config::getParam('log.slack.hook')) {
89
+            if (self::getTs() > 10 && null !== Config::getParam('log.slack.hook')) {
90 90
                 SlackHelper::getInstance()->trace('Slow service endpoint', '', '', [
91
-                    'time' => round(self::getTs(), 3) . ' secs',
92
-                    'memory' => round(self::getMem('MBytes'), 3) . ' Mb',
91
+                    'time' => round(self::getTs(), 3).' secs',
92
+                    'memory' => round(self::getMem('MBytes'), 3).' Mb',
93 93
                 ]);
94 94
             }
95 95
             return false;
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
         Logger::log('Initializing stats (mem + ts)');
105 105
         if (null !== $_SERVER && array_key_exists('REQUEST_TIME_FLOAT', $_SERVER)) {
106 106
             $this->ts = (float)$_SERVER['REQUEST_TIME_FLOAT'];
107
-        } else {
107
+        }else {
108 108
             $this->ts = PSFS_START_TS;
109 109
         }
110 110
         $this->mem = PSFS_START_MEM;
Please login to merge, or discard this patch.
src/functions.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -48,14 +48,14 @@
 block discarded – undo
48 48
     /* @var $file SplFileInfo */
49 49
     foreach ($finder as $file) {
50 50
         $path = $file->getRealPath();
51
-        if(!in_array($path, $loaded_files)) {
51
+        if (!in_array($path, $loaded_files)) {
52 52
             $loaded_files[] = $path;
53 53
             require_once($path);
54 54
         }
55 55
     }
56 56
 }
57 57
 
58
-if(!function_exists('t')) {
58
+if (!function_exists('t')) {
59 59
     function t($message, $key = null, $reload = false) {
60 60
         return CustomTranslateExtension::_($message, $key, $reload);
61 61
     }
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,9 +34,10 @@
 block discarded – undo
34 34
     function getallheaders()
35 35
     {
36 36
         $headers = [];
37
-        foreach ($_SERVER as $h => $v)
38
-            if (preg_match('/HTTP_(.+)/', $h, $hp)) {
37
+        foreach ($_SERVER as $h => $v) {
38
+                    if (preg_match('/HTTP_(.+)/', $h, $hp)) {
39 39
                 $headers[$hp[1]] = $v;
40
+        }
40 41
             }
41 42
         return $headers;
42 43
     }
Please login to merge, or discard this patch.
src/base/Singleton.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
      */
23 23
     public function __construct()
24 24
     {
25
-        Logger::log(static::class . ' constructor invoked');
25
+        Logger::log(static::class.' constructor invoked');
26 26
         $this->init();
27 27
     }
28 28
 
@@ -81,14 +81,14 @@  discard block
 block discarded – undo
81 81
         $calledClass = static::class;
82 82
         try {
83 83
             $instance = InjectorHelper::constructInyectableInstance($variable, $singleton, $classNameSpace, $calledClass);
84
-            $setter = 'set' . ucfirst($variable);
84
+            $setter = 'set'.ucfirst($variable);
85 85
             if (method_exists($calledClass, $setter)) {
86 86
                 $this->$setter($instance);
87
-            } else {
87
+            }else {
88 88
                 $this->$variable = $instance;
89 89
             }
90
-        } catch (\Exception $e) {
91
-            Logger::log($e->getMessage() . ': ' . $e->getFile() . ' [' . $e->getLine() . ']', LOG_ERR);
90
+        }catch (\Exception $e) {
91
+            Logger::log($e->getMessage().': '.$e->getFile().' ['.$e->getLine().']', LOG_ERR);
92 92
             throw $e;
93 93
         }
94 94
         return $this;
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
     {
104 104
         if (!$this->isLoaded()) {
105 105
             $filename = sha1(get_class($this));
106
-            $cacheFilename = 'reflections' . DIRECTORY_SEPARATOR . substr($filename, 0, 2) . DIRECTORY_SEPARATOR . substr($filename, 2, 2) . DIRECTORY_SEPARATOR . $filename . ".json";
106
+            $cacheFilename = 'reflections'.DIRECTORY_SEPARATOR.substr($filename, 0, 2).DIRECTORY_SEPARATOR.substr($filename, 2, 2).DIRECTORY_SEPARATOR.$filename.".json";
107 107
             /** @var \PSFS\base\Cache $cacheService */
108 108
             $cacheService = Cache::getInstance();
109 109
             /** @var \PSFS\base\config\Config $configService */
@@ -121,8 +121,8 @@  discard block
 block discarded – undo
121 121
                 }
122 122
             }
123 123
             $this->setLoaded();
124
-        } else {
125
-            Logger::log(get_class($this) . ' already loaded', LOG_INFO);
124
+        }else {
125
+            Logger::log(get_class($this).' already loaded', LOG_INFO);
126 126
         }
127 127
     }
128 128
 }
Please login to merge, or discard this patch.