Passed
Push — master ( 633df7...87ee59 )
by Fran
03:08
created
src/base/types/helpers/I18nHelper.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
         $locale = Request::header('X-API-LANG', $default);
29 29
         if (empty($locale)) {
30 30
             $locale = Security::getInstance()->getSessionKey(self::PSFS_SESSION_LANGUAGE_KEY);
31
-            if(empty($locale) && array_key_exists('HTTP_ACCEPT_LANGUAGE', $_SERVER)) {
31
+            if (empty($locale) && array_key_exists('HTTP_ACCEPT_LANGUAGE', $_SERVER)) {
32 32
                 $BrowserLocales = explode(",", str_replace("-", "_", $_SERVER["HTTP_ACCEPT_LANGUAGE"])); // brosers use en-US, Linux uses en_US
33 33
                 for ($i = 0, $ct = count($BrowserLocales); $i < $ct; $i++) {
34 34
                     list($BrowserLocales[$i]) = explode(";", $BrowserLocales[$i]); //trick for "en;q=0.8"
@@ -44,8 +44,8 @@  discard block
 block discarded – undo
44 44
         // TODO check more en locales
45 45
         if (strtolower($locale) === 'en') {
46 46
             $locale = 'en_GB';
47
-        } else {
48
-            $locale = $locale . '_' . strtoupper($locale);
47
+        }else {
48
+            $locale = $locale.'_'.strtoupper($locale);
49 49
         }
50 50
         $default_locales = explode(',', Config::getParam('i18n.locales', ''));
51 51
         if (!in_array($locale, array_merge($default_locales, self::$langs))) {
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
         $translations = array();
67 67
         if (file_exists($absoluteTranslationFileName)) {
68 68
             @include($absoluteTranslationFileName);
69
-        } else {
69
+        }else {
70 70
             Cache::getInstance()->storeData($absoluteTranslationFileName, "<?php \$translations = array();\n", Cache::TEXT, TRUE);
71 71
         }
72 72
 
@@ -81,13 +81,13 @@  discard block
 block discarded – undo
81 81
     public static function setLocale($default = null)
82 82
     {
83 83
         $locale = self::extractLocale($default);
84
-        Logger::log('Set locale to project [' . $locale . ']');
84
+        Logger::log('Set locale to project ['.$locale.']');
85 85
         // Load translations
86
-        putenv("LC_ALL=" . $locale);
86
+        putenv("LC_ALL=".$locale);
87 87
         setlocale(LC_ALL, $locale);
88 88
         // Load the locale path
89
-        $locale_path = BASE_DIR . DIRECTORY_SEPARATOR . 'locale';
90
-        Logger::log('Set locale dir ' . $locale_path);
89
+        $locale_path = BASE_DIR.DIRECTORY_SEPARATOR.'locale';
90
+        Logger::log('Set locale dir '.$locale_path);
91 91
         GeneratorHelper::createDir($locale_path);
92 92
         bindtextdomain('translations', $locale_path);
93 93
         textdomain('translations');
@@ -123,9 +123,9 @@  discard block
 block discarded – undo
123 123
      */
124 124
     public static function checkI18Class($namespace) {
125 125
         $isI18n = false;
126
-        if(preg_match('/I18n$/i', $namespace)) {
126
+        if (preg_match('/I18n$/i', $namespace)) {
127 127
             $parentClass = preg_replace('/I18n$/i', '', $namespace);
128
-            if(Router::exists($parentClass)) {
128
+            if (Router::exists($parentClass)) {
129 129
                 $isI18n = true;
130 130
             }
131 131
         }
@@ -151,12 +151,12 @@  discard block
 block discarded – undo
151 151
             ['i', 'i', 'i', 'i', 'I', 'I', 'I', 'I'],
152 152
             ['o', 'o', 'o', 'o', 'O', 'O', 'O', 'O'],
153 153
             ['u', 'u', 'u', 'u', 'U', 'U', 'U', 'U'],
154
-            ['n', 'N', 'c', 'C',],
154
+            ['n', 'N', 'c', 'C', ],
155 155
         ];
156 156
 
157 157
         $text = filter_var($string, FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW);
158
-        for($i = 0, $total = count($from); $i < $total; $i++) {
159
-            $text = str_replace($from[$i],$to[$i], $text);
158
+        for ($i = 0, $total = count($from); $i < $total; $i++) {
159
+            $text = str_replace($from[$i], $to[$i], $text);
160 160
         }
161 161
 
162 162
         return $text;
Please login to merge, or discard this patch.
src/controller/ConfigController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         foreach ($domains as $domain => $routes) {
32 32
             $pDomain = str_replace('@', '', $domain);
33 33
             $pDomain = str_replace('/', '', $pDomain);
34
-            $response[] = strtolower($pDomain) . '.api.secret';
34
+            $response[] = strtolower($pDomain).'.api.secret';
35 35
         }
36 36
         return $this->json($response);
37 37
     }
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
      */
46 46
     public function config()
47 47
     {
48
-        Logger::log("Config loaded executed by " . $this->getRequest()->getRequestUri());
48
+        Logger::log("Config loaded executed by ".$this->getRequest()->getRequestUri());
49 49
         /* @var $form \PSFS\base\config\ConfigForm */
50 50
         $form = new ConfigForm(Router::getInstance()->getRoute('admin-config'), Config::$required, Config::$optional, Config::getInstance()->dumpConfig());
51 51
         $form->build();
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
                 }
83 83
                 Security::getInstance()->setFlash("callback_message", t("Configuración actualizada correctamente"));
84 84
                 Security::getInstance()->setFlash("callback_route", $this->getRoute("admin-config", true));
85
-            } else {
85
+            }else {
86 86
                 throw new \HttpException(t('Error al guardar la configuración, prueba a cambiar los permisos'), 403);
87 87
             }
88 88
         }
Please login to merge, or discard this patch.
src/base/types/CustomAuthApi.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@
 block discarded – undo
12 12
 
13 13
     public function init()
14 14
     {
15
-        if(!Security::getInstance()->isLogged()) {
15
+        if (!Security::getInstance()->isLogged()) {
16 16
             throw new ApiException(t('Resource not authorized'), 401);
17 17
         }
18 18
         parent::init();
Please login to merge, or discard this patch.
src/base/types/traits/Api/ConnectionTrait.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     {
33 33
         $this->con = Propel::getConnection($tableMap::DATABASE_NAME);
34 34
         $this->con->beginTransaction();
35
-        if(method_exists($this->con, 'useDebug')) {
35
+        if (method_exists($this->con, 'useDebug')) {
36 36
             Logger::log('Enabling debug queries mode', LOG_INFO);
37 37
             $this->con->useDebug(Config::getParam('debug'));
38 38
         }
@@ -45,12 +45,12 @@  discard block
 block discarded – undo
45 45
      */
46 46
     protected function closeTransaction($status)
47 47
     {
48
-        if(null !== $this->con) {
48
+        if (null !== $this->con) {
49 49
             $this->traceDebugQuery();
50 50
             if (null !== $this->con && $this->con->inTransaction()) {
51 51
                 if ($status === 200) {
52 52
                     $this->con->commit();
53
-                } else {
53
+                }else {
54 54
                     $this->con->rollBack();
55 55
                 }
56 56
             }
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
     protected function traceDebugQuery()
65 65
     {
66 66
         if (Config::getParam('debug')) {
67
-            Logger::log($this->con->getLastExecutedQuery()?: 'Empty Query', LOG_DEBUG);
67
+            Logger::log($this->con->getLastExecutedQuery() ?: 'Empty Query', LOG_DEBUG);
68 68
         }
69 69
     }
70 70
 
@@ -72,13 +72,13 @@  discard block
 block discarded – undo
72 72
      * Checks if the connection has a transaction initialized
73 73
      */
74 74
     protected function checkTransaction() {
75
-        if(null !== $this->con && !$this->con->inTransaction()) {
75
+        if (null !== $this->con && !$this->con->inTransaction()) {
76 76
             $this->con->beginTransaction();
77 77
         }
78
-        if(null !== $this->con && $this->con->inTransaction()) {
78
+        if (null !== $this->con && $this->con->inTransaction()) {
79 79
             $this->items++;
80 80
         }
81
-        if($this->items >= Config::getParam('api.block.limit', 1000)) {
81
+        if ($this->items >= Config::getParam('api.block.limit', 1000)) {
82 82
             $this->con->commit();
83 83
             $this->items = 0;
84 84
         }
Please login to merge, or discard this patch.
src/base/types/helpers/ServiceHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
     }
16 16
 
17 17
     public static function parseRawData($type, array $params = []) {
18
-        switch($type) {
18
+        switch ($type) {
19 19
             default:
20 20
             case self::TYPE_HTTP:
21 21
                 $parsedParams = http_build_query($params);
Please login to merge, or discard this patch.
src/base/extension/TemplateFunctions.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -39,13 +39,13 @@  discard block
 block discarded – undo
39 39
 
40 40
         $filePath = '';
41 41
         if (!file_exists($filePath)) {
42
-            $filePath = BASE_DIR . $string;
42
+            $filePath = BASE_DIR.$string;
43 43
         }
44 44
         $filenamePath = AssetsParser::findDomainPath($string, $filePath);
45 45
 
46 46
         $filePath = self::processAsset($string, $name, $return, $filenamePath);
47 47
         $basePath = Config::getParam('resources.cdn.url', Request::getInstance()->getRootUrl());
48
-        $returnPath = empty($name) ? $basePath . '/' . $filePath : $name;
48
+        $returnPath = empty($name) ? $basePath.'/'.$filePath : $name;
49 49
         return $return ? $returnPath : '';
50 50
     }
51 51
 
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
         $router = Router::getInstance();
63 63
         try {
64 64
             return $router->getRoute($path, $absolute, $params);
65
-        } catch (\Exception $e) {
65
+        }catch (\Exception $e) {
66 66
             return $router->getRoute('', $absolute, $params);
67 67
         }
68 68
     }
@@ -205,9 +205,9 @@  discard block
 block discarded – undo
205 205
     {
206 206
         $data = file_get_contents($filenamePath);
207 207
         if (!empty($name)) {
208
-            file_put_contents(WEB_DIR . DIRECTORY_SEPARATOR . $name, $data);
208
+            file_put_contents(WEB_DIR.DIRECTORY_SEPARATOR.$name, $data);
209 209
         }else {
210
-            file_put_contents($base . $filePath, $data);
210
+            file_put_contents($base.$filePath, $data);
211 211
         }
212 212
     }
213 213
 
@@ -226,9 +226,9 @@  discard block
 block discarded – undo
226 226
         if (file_exists($filenamePath)) {
227 227
             list($base, $htmlBase, $filePath) = AssetsParser::calculateAssetPath($string, $name, $return, $filenamePath);
228 228
             //Creamos el directorio si no existe
229
-            GeneratorHelper::createDir($base . $htmlBase);
229
+            GeneratorHelper::createDir($base.$htmlBase);
230 230
             //Si se ha modificado
231
-            if (!file_exists($base . $filePath) || filemtime($base . $filePath) < filemtime($filenamePath)) {
231
+            if (!file_exists($base.$filePath) || filemtime($base.$filePath) < filemtime($filenamePath)) {
232 232
                 if ($htmlBase === 'css') {
233 233
                     self::processCssLines($filenamePath);
234 234
                 }
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -206,7 +206,7 @@
 block discarded – undo
206 206
         $data = file_get_contents($filenamePath);
207 207
         if (!empty($name)) {
208 208
             file_put_contents(WEB_DIR . DIRECTORY_SEPARATOR . $name, $data);
209
-        }else {
209
+        } else {
210 210
             file_put_contents($base . $filePath, $data);
211 211
         }
212 212
     }
Please login to merge, or discard this patch.
src/base/types/helpers/RouterHelper.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -19,11 +19,11 @@  discard block
 block discarded – undo
19 19
     public static function getClassToCall(array $action)
20 20
     {
21 21
         Logger::log('Getting class to call for executing the request action', LOG_DEBUG, $action);
22
-        $actionClass = class_exists($action['class']) ? $action['class'] : "\\" . $action['class'];
22
+        $actionClass = class_exists($action['class']) ? $action['class'] : "\\".$action['class'];
23 23
         $reflectionClass = new \ReflectionClass($actionClass);
24
-        if($reflectionClass->hasMethod('getInstance')) {
24
+        if ($reflectionClass->hasMethod('getInstance')) {
25 25
             $class = $reflectionClass->getMethod('getInstance')->invoke(null, $action['method']);
26
-        } else {
26
+        }else {
27 27
             $class = new $actionClass;
28 28
         }
29 29
         return $class;
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
         $expr = preg_quote($expr, '/');
113 113
         $expr = str_replace('###', '(.*)', $expr);
114 114
         $expr2 = preg_replace('/\(\.\*\)$/', '', $expr);
115
-        $matched = preg_match('/^' . $expr . '\/?$/i', $path) || preg_match('/^' . $expr2 . '?$/i', $path);
115
+        $matched = preg_match('/^'.$expr.'\/?$/i', $path) || preg_match('/^'.$expr2.'?$/i', $path);
116 116
         return $matched;
117 117
     }
118 118
 
@@ -123,8 +123,8 @@  discard block
 block discarded – undo
123 123
      */
124 124
     public static function extractDomainInfo(\ReflectionClass $class, $domain)
125 125
     {
126
-        $path = dirname($class->getFileName()) . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR;
127
-        $path = realpath($path) . DIRECTORY_SEPARATOR;
126
+        $path = dirname($class->getFileName()).DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR;
127
+        $path = realpath($path).DIRECTORY_SEPARATOR;
128 128
         $templatesPath = 'templates';
129 129
         $publicPath = 'public';
130 130
         $modelsPath = 'models';
@@ -134,13 +134,13 @@  discard block
 block discarded – undo
134 134
             $modelsPath = ucfirst($modelsPath);
135 135
         }
136 136
         if ($class->hasConstant('TPL')) {
137
-            $templatesPath .= DIRECTORY_SEPARATOR . $class->getConstant('TPL');
137
+            $templatesPath .= DIRECTORY_SEPARATOR.$class->getConstant('TPL');
138 138
         }
139 139
         return [
140 140
             'base' => $path,
141
-            'template' => $path . $templatesPath,
142
-            'model' => $path . $modelsPath,
143
-            'public' => $path . $publicPath,
141
+            'template' => $path.$templatesPath,
142
+            'model' => $path.$modelsPath,
143
+            'public' => $path.$publicPath,
144 144
         ];
145 145
     }
146 146
 
@@ -164,12 +164,12 @@  discard block
 block discarded – undo
164 164
             foreach ($parameters as $param) {
165 165
                 if ($param->isOptional() && !is_array($param->getDefaultValue())) {
166 166
                     $params[$param->getName()] = $param->getDefaultValue();
167
-                    $default = str_replace('{' . $param->getName() . '}', $param->getDefaultValue(), $regex);
168
-                } elseif(!$param->isOptional()) {
167
+                    $default = str_replace('{'.$param->getName().'}', $param->getDefaultValue(), $regex);
168
+                } elseif (!$param->isOptional()) {
169 169
                     $requirements[] = $param->getName();
170 170
                 }
171 171
             }
172
-        } else {
172
+        }else {
173 173
             $default = $regex;
174 174
         }
175 175
 
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
             $default = str_replace('{__DOMAIN__}', $module, $default);
252 252
             $httpMethod = self::extractReflectionHttpMethod($docComments);
253 253
             $label = self::extractReflectionLabel(str_replace('{__API__}', $api, $docComments));
254
-            $route = $httpMethod . "#|#" . $regex;
254
+            $route = $httpMethod."#|#".$regex;
255 255
             $route = preg_replace('/(\\r|\\f|\\t|\\n)/', '', $route);
256 256
             $info = [
257 257
                 'method' => $method->getName(),
Please login to merge, or discard this patch.
src/base/types/helpers/RequestHelper.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -45,10 +45,10 @@  discard block
 block discarded – undo
45 45
                 if (!headers_sent()) {
46 46
                     // TODO include this headers in Template class output method
47 47
                     header('Access-Control-Allow-Credentials: true');
48
-                    header('Access-Control-Allow-Origin: ' . Request::getInstance()->getServer('HTTP_ORIGIN', '*'));
48
+                    header('Access-Control-Allow-Origin: '.Request::getInstance()->getServer('HTTP_ORIGIN', '*'));
49 49
                     header('Vary: Origin');
50 50
                     header('Access-Control-Allow-Methods: GET, POST, DELETE, PUT, PATCH, OPTIONS');
51
-                    header('Access-Control-Allow-Headers: ' . implode(', ', self::getCorsHeaders()));
51
+                    header('Access-Control-Allow-Headers: '.implode(', ', self::getCorsHeaders()));
52 52
                 }
53 53
                 if (Request::getInstance()->getMethod() === Request::VERB_OPTIONS) {
54 54
                     Logger::log('Returning OPTIONS header confirmation for CORS pre flight requests', LOG_DEBUG);
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
                         return $ip;
79 79
                     }
80 80
                 }
81
-            } else {
81
+            }else {
82 82
                 if (self::validateIpAddress($_SERVER['HTTP_X_FORWARDED_FOR'])) {
83 83
                     return $_SERVER['HTTP_X_FORWARDED_FOR'];
84 84
                 }
Please login to merge, or discard this patch.
src/base/types/traits/Api/ApiTrait.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -96,15 +96,15 @@  discard block
 block discarded – undo
96 96
     protected function hydrateBulkRequest() {
97 97
         $class = new \ReflectionClass($this->getModelNamespace());
98 98
         $this->list = [];
99
-        foreach($this->data as $item) {
100
-            if(is_array($item)) {
101
-                if(count($this->list) < Config::getParam('api.block.limit', 1000)) {
99
+        foreach ($this->data as $item) {
100
+            if (is_array($item)) {
101
+                if (count($this->list) < Config::getParam('api.block.limit', 1000)) {
102 102
                     /** @var ActiveRecordInterface $model */
103 103
                     $model = $class->newInstance();
104 104
                     $this->hydrateModelFromRequest($model, $item);
105 105
                     $this->list[] = $model;
106
-                } else {
107
-                    Logger::log(t('Max items per bulk insert raised'), LOG_WARNING, count($this->data) . t('items'));
106
+                }else {
107
+                    Logger::log(t('Max items per bulk insert raised'), LOG_WARNING, count($this->data).t('items'));
108 108
                 }
109 109
             }
110 110
         }
@@ -115,12 +115,12 @@  discard block
 block discarded – undo
115 115
      */
116 116
     protected function saveBulk() {
117 117
         $tablemap = $this->getTableMap();
118
-        foreach($this->list as &$model) {
118
+        foreach ($this->list as &$model) {
119 119
             $con = Propel::getWriteConnection($tablemap::DATABASE_NAME);
120 120
             try {
121 121
                 $model->save($con);
122 122
                 $con->commit();
123
-            } catch(\Exception $e) {
123
+            }catch (\Exception $e) {
124 124
                 Logger::log($e->getMessage(), LOG_ERR, $model->toArray());
125 125
                 $con->rollBack();
126 126
             }
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
     protected function exportList() {
134 134
         $list = [];
135 135
         /** @var ActiveRecordInterface $item */
136
-        foreach($this->list as $item) {
136
+        foreach ($this->list as $item) {
137 137
             $list[] = $item->toArray();
138 138
         }
139 139
         return $list;
@@ -157,18 +157,18 @@  discard block
 block discarded – undo
157 157
      */
158 158
     protected function findPk(ModelCriteria $query, $primaryKey) {
159 159
         $pks = explode(Api::API_PK_SEPARATOR, urldecode($primaryKey));
160
-        if(count($pks) === 1 && !empty($pks[0])) {
160
+        if (count($pks) === 1 && !empty($pks[0])) {
161 161
             $query->filterByPrimaryKey($pks[0]);
162
-        } else {
162
+        }else {
163 163
             $item = 0;
164
-            foreach($this->getPkDbName() as $phpName) {
164
+            foreach ($this->getPkDbName() as $phpName) {
165 165
                 try {
166 166
                     $query->filterBy($phpName, $pks[$item]);
167 167
                     $item++;
168
-                    if($item >= count($pks)) {
168
+                    if ($item >= count($pks)) {
169 169
                         break;
170 170
                     }
171
-                } catch(\Exception $e) {
171
+                }catch (\Exception $e) {
172 172
                     Logger::log($e->getMessage(), LOG_DEBUG);
173 173
                 }
174 174
             }
@@ -198,8 +198,8 @@  discard block
 block discarded – undo
198 198
         try {
199 199
             $query = $this->prepareQuery();
200 200
             $this->model = $this->findPk($query, $primaryKey);
201
-        } catch (\Exception $e) {
202
-            Logger::log(get_class($this) . ': ' . $e->getMessage(), LOG_ERR);
201
+        }catch (\Exception $e) {
202
+            Logger::log(get_class($this).': '.$e->getMessage(), LOG_ERR);
203 203
         }
204 204
     }
205 205
 
Please login to merge, or discard this patch.