Passed
Push — master ( a14867...1d17df )
by Fran
05:42
created
src/base/types/helpers/ApiHelper.php 1 patch
Spacing   +37 added lines, -37 removed lines patch added patch discarded remove patch
@@ -37,20 +37,20 @@  discard block
 block discarded – undo
37 37
         $behaviors = $tableMap->getBehaviors();
38 38
         foreach ($map::getFieldNames() as $field) {
39 39
             $fDto = self::parseFormField($domain, $tableMap, $field, $behaviors);
40
-            if(null !== $fDto) {
40
+            if (null !== $fDto) {
41 41
                 $form->addField($fDto);
42 42
             }
43 43
         }
44 44
 
45
-        if(array_key_exists('i18n', $behaviors)) {
46
-            $relateI18n = $tableMap->getRelation($tableMap->getPhpName() . 'I18n');
47
-            if(null !== $relateI18n) {
45
+        if (array_key_exists('i18n', $behaviors)) {
46
+            $relateI18n = $tableMap->getRelation($tableMap->getPhpName().'I18n');
47
+            if (null !== $relateI18n) {
48 48
                 $i18NTableMap = $relateI18n->getLocalTable();
49
-                foreach($i18NTableMap->getColumns() as $columnMap) {
49
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
50 50
                     $columnName = self::getColumnMapName($columnMap);
51
-                    if(!$form->fieldExists($columnName)) {
51
+                    if (!$form->fieldExists($columnName)) {
52 52
                         $fDto = self::parseFormField($domain, $i18NTableMap, $columnMap->getPhpName(), $i18NTableMap->getBehaviors());
53
-                        if(null !== $fDto) {
53
+                        if (null !== $fDto) {
54 54
                             $fDto->pk = false;
55 55
                             $form->addField($fDto);
56 56
                         }
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
         $fDto->entity = $relatedModel;
80 80
         $relatedField = $foreignTable->getColumn($mappedColumn->getRelatedColumnName());
81 81
         $fDto->relatedField = $relatedField->getPhpName();
82
-        $fDto->url = Router::getInstance()->getRoute(strtolower($domain) . '-api-' . $relatedModel);
82
+        $fDto->url = Router::getInstance()->getRoute(strtolower($domain).'-api-'.$relatedModel);
83 83
         return $fDto;
84 84
     }
85 85
 
@@ -186,14 +186,14 @@  discard block
 block discarded – undo
186 186
     {
187 187
         $column = null;
188 188
         try {
189
-            foreach($tableMap->getColumns() as $tableMapColumn) {
189
+            foreach ($tableMap->getColumns() as $tableMapColumn) {
190 190
                 $columnName = $tableMapColumn->getPhpName();
191
-                if(preg_match('/^'.$field.'$/i', $columnName)) {
191
+                if (preg_match('/^'.$field.'$/i', $columnName)) {
192 192
                     $column = $tableMapColumn;
193 193
                     break;
194 194
                 }
195 195
             }
196
-        } catch (\Exception $e) {
196
+        }catch (\Exception $e) {
197 197
             Logger::log($e->getMessage(), LOG_DEBUG);
198 198
         }
199 199
         return $column;
@@ -207,18 +207,18 @@  discard block
 block discarded – undo
207 207
     private static function addQueryFilter(ColumnMap $column, ModelCriteria &$query, $value = null)
208 208
     {
209 209
         $tableField = $column->getFullyQualifiedName();
210
-        if(is_array($value)) {
210
+        if (is_array($value)) {
211 211
             $query->add($tableField, $value, Criteria::IN);
212 212
         } elseif (preg_match('/^\[/', $value) && preg_match('/\]$/', $value)) {
213 213
             $query->add($tableField, explode(',', preg_replace('/(\[|\])/', '', $value)), Criteria::IN);
214 214
         } elseif (preg_match('/^(\'|\")(.*)(\'|\")$/', $value)) {
215 215
             $text = preg_replace('/(\'|\")/', '', $value);
216 216
             $text = preg_replace('/\ /', '%', $text);
217
-            $query->add($tableField, '%' . $text . '%', Criteria::LIKE);
218
-        } else {
219
-            if(null !== $column->getValueSet()) {
217
+            $query->add($tableField, '%'.$text.'%', Criteria::LIKE);
218
+        }else {
219
+            if (null !== $column->getValueSet()) {
220 220
                 $valueSet = $column->getValueSet();
221
-                if(in_array($value, $valueSet)) {
221
+                if (in_array($value, $valueSet)) {
222 222
                     $value = array_search($value, $valueSet);
223 223
                 }
224 224
             }
@@ -238,31 +238,31 @@  discard block
 block discarded – undo
238 238
         $sep = '';
239 239
         foreach ($tableMap->getColumns() as $column) {
240 240
             if ($column->isText()) {
241
-                $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
241
+                $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
242 242
                 $sep = ', " ", ';
243 243
             }
244 244
         }
245
-        foreach($tableMap->getRelations() as $relation) {
246
-            if(preg_match('/I18n$/i', $relation->getName())) {
245
+        foreach ($tableMap->getRelations() as $relation) {
246
+            if (preg_match('/I18n$/i', $relation->getName())) {
247 247
                 $localeTableMap = $relation->getLocalTable();
248 248
                 foreach ($localeTableMap->getColumns() as $column) {
249 249
                     if ($column->isText()) {
250
-                        $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
250
+                        $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
251 251
                         $sep = ', " ", ';
252 252
                     }
253 253
                 }
254 254
             }
255 255
         }
256 256
         foreach ($extraColumns as $extra => $name) {
257
-            if(!preg_match("/(COUNT|DISTINCT|SUM|MAX|MIN|GROUP)/i", $extra)) {
258
-                $exp .= $sep . $extra;
257
+            if (!preg_match("/(COUNT|DISTINCT|SUM|MAX|MIN|GROUP)/i", $extra)) {
258
+                $exp .= $sep.$extra;
259 259
                 $sep = ', " ", ';
260 260
             }
261 261
         }
262 262
         $exp .= ")";
263 263
         $text = preg_replace('/(\'|\")/', '', $value);
264 264
         $text = preg_replace('/\ /', '%', $text);
265
-        $query->where($exp . Criteria::LIKE . '"%' . $text . '%"');
265
+        $query->where($exp.Criteria::LIKE.'"%'.$text.'%"');
266 266
     }
267 267
 
268 268
     /**
@@ -287,7 +287,7 @@  discard block
 block discarded – undo
287 287
      */
288 288
     public static function extractQuery($modelNameNamespace, ConnectionInterface $con = null)
289 289
     {
290
-        $queryReflector = new \ReflectionClass($modelNameNamespace . "Query");
290
+        $queryReflector = new \ReflectionClass($modelNameNamespace."Query");
291 291
         /** @var \Propel\Runtime\ActiveQuery\ModelCriteria $query */
292 292
         $query = $queryReflector->getMethod('create')->invoke($con);
293 293
 
@@ -316,7 +316,7 @@  discard block
 block discarded – undo
316 316
         } elseif ($mappedColumn->isText()) {
317 317
             if ($mappedColumn->getSize() > 100) {
318 318
                 $fDto = self::createField($field, Field::TEXTAREA_TYPE, $required);
319
-            } else {
319
+            }else {
320 320
                 $fDto = self::generateStringField($field, $required);
321 321
             }
322 322
         } elseif ($mappedColumn->getType() === PropelTypes::BOOLEAN) {
@@ -344,7 +344,7 @@  discard block
 block discarded – undo
344 344
                 $fDto->pk = true;
345 345
             }
346 346
         }
347
-        switch(Config::getParam('api.field.case', TableMap::TYPE_PHPNAME)) {
347
+        switch (Config::getParam('api.field.case', TableMap::TYPE_PHPNAME)) {
348 348
             default:
349 349
             case TableMap::TYPE_PHPNAME:
350 350
                 $fDto->name = $mappedColumn->getPhpName();
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
      */
369 369
     public static function extractPrimaryKeyColumnName(TableMap $tableMap) {
370 370
         $modelPk = null;
371
-        foreach($tableMap->getPrimaryKeys() as $pk) {
371
+        foreach ($tableMap->getPrimaryKeys() as $pk) {
372 372
             $modelPk = $pk;
373 373
             break;
374 374
         }
@@ -377,15 +377,15 @@  discard block
 block discarded – undo
377 377
 
378 378
     private static function mapResult(ActiveRecordInterface $model, array $data = []) {
379 379
         $result = [];
380
-        foreach($data as $key => $value) {
380
+        foreach ($data as $key => $value) {
381 381
             try {
382 382
                 $realValue = $model->getByName($key);
383
-            } catch(\Exception $e) {
383
+            }catch (\Exception $e) {
384 384
                 $realValue = $value;
385 385
             }
386
-            if(Api::API_MODEL_KEY_FIELD === $key) {
386
+            if (Api::API_MODEL_KEY_FIELD === $key) {
387 387
                 $result[$key] = (integer)$realValue;
388
-            } else {
388
+            }else {
389 389
                 $result[$key] = $realValue;
390 390
             }
391 391
         }
@@ -402,14 +402,14 @@  discard block
 block discarded – undo
402 402
         $objTableMap = get_class($formatter->getTableMap());
403 403
         /** @var TableMapTrait $objTableMap */
404 404
         $objData = $data;
405
-        foreach($objTableMap::getFieldNames() as $field) {
406
-            if(!array_key_exists($field, $objData)) {
405
+        foreach ($objTableMap::getFieldNames() as $field) {
406
+            if (!array_key_exists($field, $objData)) {
407 407
                 $objData[$field] = null;
408 408
             }
409 409
         }
410 410
         $obj = @$formatter->getAllObjectsFromRow($objData);
411 411
         $result = self::mapResult($obj, $data);
412
-        if(!preg_match('/' . $modelPk->getPhpName() . '/i', $query[Api::API_FIELDS_RESULT_FIELD])) {
412
+        if (!preg_match('/'.$modelPk->getPhpName().'/i', $query[Api::API_FIELDS_RESULT_FIELD])) {
413 413
             unset($result[$modelPk->getPhpName()]);
414 414
         }
415 415
         return $result;
@@ -420,7 +420,7 @@  discard block
 block discarded – undo
420 420
      */
421 421
     public static function getFieldTypes() {
422 422
         $configType = Config::getParam('api.field.case');
423
-        switch($configType) {
423
+        switch ($configType) {
424 424
             default:
425 425
             case 'UpperCamelCase':
426 426
             case TableMap::TYPE_PHPNAME:
@@ -446,11 +446,11 @@  discard block
 block discarded – undo
446 446
     }
447 447
 
448 448
     public static function getColumnMapName(ColumnMap $field) {
449
-        switch(self::getFieldTypes()) {
449
+        switch (self::getFieldTypes()) {
450 450
             default:
451 451
             case 'UpperCamelCase':
452 452
             case TableMap::TYPE_PHPNAME:
453
-                $columnName =$field->getPhpName();
453
+                $columnName = $field->getPhpName();
454 454
                 break;
455 455
             case 'camelCase':
456 456
             case 'lowerCamelCase':
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/types/helpers/SlackHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
             'attachments' => [
30 30
                 [
31 31
                     "author_name" => $request->getRootUrl(true),
32
-                    "text" => $file . (strlen($line) ? ' [' . $line . ']' : ''),
32
+                    "text" => $file.(strlen($line) ? ' ['.$line.']' : ''),
33 33
                     "color" => Config::getParam('debug', true) ? 'warning' : "danger",
34 34
                     "title" => $message,
35 35
                     'fallback' => 'PSFS Error notifier',
Please login to merge, or discard this patch.