Passed
Push — master ( fbd0b5...1c1b55 )
by Fran
03:49
created
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/Service.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
     protected $isMultipart = false;
71 71
 
72 72
     private function closeConnection() {
73
-        if(null !== $this->con) {
73
+        if (null !== $this->con) {
74 74
             curl_close($this->con);
75 75
         }
76 76
     }
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
      */
234 234
     public function setIsJson($isJson = true) {
235 235
         $this->isJson = $isJson;
236
-        if($isJson) {
236
+        if ($isJson) {
237 237
             $this->setIsMultipart(false);
238 238
         }
239 239
     }
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
      */
252 252
     public function setIsMultipart($isMultipart = true) {
253 253
         $this->isMultipart = $isMultipart;
254
-        if($isMultipart) {
254
+        if ($isMultipart) {
255 255
             $this->setIsJson(false);
256 256
         }
257 257
     }
@@ -271,7 +271,7 @@  discard block
 block discarded – undo
271 271
         $this->url = NULL;
272 272
         $this->params = array();
273 273
         $this->headers = array();
274
-        Logger::log('Context service for ' . static::class . ' cleared!');
274
+        Logger::log('Context service for '.static::class.' cleared!');
275 275
         $this->closeConnection();
276 276
     }
277 277
 
@@ -314,18 +314,18 @@  discard block
 block discarded – undo
314 314
     }
315 315
 
316 316
     protected function applyOptions() {
317
-        if(count($this->options)) {
317
+        if (count($this->options)) {
318 318
             curl_setopt_array($this->con, $this->options);
319 319
         }
320 320
     }
321 321
 
322 322
     protected function applyHeaders() {
323 323
         $headers = [];
324
-        foreach($this->headers as $key => $value) {
325
-            $headers[] = $key . ': ' . $value;
324
+        foreach ($this->headers as $key => $value) {
325
+            $headers[] = $key.': '.$value;
326 326
         }
327 327
         $headers[self::PSFS_TRACK_HEADER] = Logger::getUid();
328
-        if(count($headers)) {
328
+        if (count($headers)) {
329 329
             curl_setopt($this->con, CURLOPT_HTTPHEADER, $headers);
330 330
         }
331 331
     }
@@ -334,10 +334,10 @@  discard block
 block discarded – undo
334 334
      * @return int
335 335
      */
336 336
     private function parseServiceType() {
337
-        if($this->getIsJson()) {
337
+        if ($this->getIsJson()) {
338 338
             return ServiceHelper::TYPE_JSON;
339 339
         }
340
-        if($this->getIsMultipart()) {
340
+        if ($this->getIsMultipart()) {
341 341
             return ServiceHelper::TYPE_MULTIPART;
342 342
         }
343 343
         return ServiceHelper::TYPE_HTTP;
@@ -350,9 +350,9 @@  discard block
 block discarded – undo
350 350
             case Request::VERB_GET:
351 351
             default:
352 352
                 $this->addOption(CURLOPT_CUSTOMREQUEST, Request::VERB_GET);
353
-                if(!empty($this->params)) {
353
+                if (!empty($this->params)) {
354 354
                     $sep = false !== strpos($this->getUrl(), '?') ? '?' : '';
355
-                    $this->url = $this->url . $sep . http_build_query($this->params);
355
+                    $this->url = $this->url.$sep.http_build_query($this->params);
356 356
                 }
357 357
                 break;
358 358
             case Request::VERB_POST:
@@ -384,14 +384,14 @@  discard block
 block discarded – undo
384 384
         $this->applyOptions();
385 385
         $this->applyHeaders();
386 386
         $verbose = null;
387
-        if('debug' === Config::getParam('log.level')) {
387
+        if ('debug' === Config::getParam('log.level')) {
388 388
             curl_setopt($this->con, CURLOPT_VERBOSE, true);
389 389
             $verbose = fopen('php://temp', 'wb+');
390 390
             curl_setopt($this->con, CURLOPT_STDERR, $verbose);
391 391
         }
392 392
         $result = curl_exec($this->con);
393 393
         $this->result = $this->isJson ? json_decode($result, true) : $result;
394
-        if('debug' === Config::getParam('log.level')) {
394
+        if ('debug' === Config::getParam('log.level')) {
395 395
             rewind($verbose);
396 396
             $verboseLog = stream_get_contents($verbose);
397 397
             Logger::log($verboseLog, LOG_DEBUG, [
@@ -401,7 +401,7 @@  discard block
 block discarded – undo
401 401
             ]);
402 402
             $this->info['verbose'] = $verboseLog;
403 403
         }
404
-        Logger::log($this->url . ' response: ', LOG_DEBUG, is_array($this->result) ? $this->result : [$this->result]);
404
+        Logger::log($this->url.' response: ', LOG_DEBUG, is_array($this->result) ? $this->result : [$this->result]);
405 405
         $this->info = array_merge($this->info, curl_getinfo($this->con));
406 406
     }
407 407
 
Please login to merge, or discard this patch.
src/base/Request.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
      */
146 146
     public static function header($name, $default = null)
147 147
     {
148
-        return self::getInstance()->getHeader($name,  $default);
148
+        return self::getInstance()->getHeader($name, $default);
149 149
     }
150 150
 
151 151
     /**
@@ -158,10 +158,10 @@  discard block
 block discarded – undo
158 158
         $header = null;
159 159
         if ($this->hasHeader($name)) {
160 160
             $header = $this->header[$name];
161
-        } else if(array_key_exists('h_' . strtolower($name), $this->query)) {
162
-            $header = $this->query['h_' . strtolower($name)];
163
-        } else if(array_key_exists('HTTP_' . strtoupper(str_replace('-', '_', $name)), $this->server)) {
164
-            $header = $this->server['HTTP_' . strtoupper(str_replace('-', '_', $name))];
161
+        }else if (array_key_exists('h_'.strtolower($name), $this->query)) {
162
+            $header = $this->query['h_'.strtolower($name)];
163
+        }else if (array_key_exists('HTTP_'.strtoupper(str_replace('-', '_', $name)), $this->server)) {
164
+            $header = $this->server['HTTP_'.strtoupper(str_replace('-', '_', $name))];
165 165
         }
166 166
         return $header ?: $default;
167 167
     }
@@ -260,7 +260,7 @@  discard block
 block discarded – undo
260 260
             $url = $this->getServer('HTTP_ORIGIN');
261 261
         }
262 262
         ob_start();
263
-        header('Location: ' . $url);
263
+        header('Location: '.$url);
264 264
         ob_end_clean();
265 265
         Security::getInstance()->updateSession();
266 266
         exit(t('Redireccionando...'));
@@ -305,10 +305,10 @@  discard block
 block discarded – undo
305 305
         $url = $this->getServerName();
306 306
         $protocol = $hasProtocol ? $this->getProtocol() : '';
307 307
         if (!empty($protocol)) {
308
-            $url = $protocol . $url;
308
+            $url = $protocol.$url;
309 309
         }
310 310
         if (!in_array($this->getServer('SERVER_PORT'), [80, 443], true)) {
311
-            $url .= ':' . $this->getServer('SERVER_PORT');
311
+            $url .= ':'.$this->getServer('SERVER_PORT');
312 312
         }
313 313
         return $url;
314 314
     }
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.