Passed
Push — master ( be88e3...87c6b6 )
by Fran
04:17
created
src/base/types/helpers/ApiHelper.php 1 patch
Spacing   +34 added lines, -34 removed lines patch added patch discarded remove patch
@@ -37,19 +37,19 @@  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) {
50
-                    if(!$form->fieldExists($columnMap->getPhpName())) {
49
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
50
+                    if (!$form->fieldExists($columnMap->getPhpName())) {
51 51
                         $fDto = self::parseFormField($domain, $i18NTableMap, $columnMap->getPhpName(), $i18NTableMap->getBehaviors());
52
-                        if(null !== $fDto) {
52
+                        if (null !== $fDto) {
53 53
                             $fDto->pk = false;
54 54
                             $form->addField($fDto);
55 55
                         }
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
         $fDto->entity = $relatedModel;
79 79
         $relatedField = $foreignTable->getColumn($mappedColumn->getRelatedColumnName());
80 80
         $fDto->relatedField = $relatedField->getPhpName();
81
-        $fDto->url = Router::getInstance()->getRoute(strtolower($domain) . '-api-' . $relatedModel);
81
+        $fDto->url = Router::getInstance()->getRoute(strtolower($domain).'-api-'.$relatedModel);
82 82
         return $fDto;
83 83
     }
84 84
 
@@ -185,14 +185,14 @@  discard block
 block discarded – undo
185 185
     {
186 186
         $column = null;
187 187
         try {
188
-            foreach($tableMap->getColumns() as $tableMapColumn) {
188
+            foreach ($tableMap->getColumns() as $tableMapColumn) {
189 189
                 $columnName = $tableMapColumn->getPhpName();
190
-                if(preg_match('/'.$field.'/i', $columnName)) {
190
+                if (preg_match('/'.$field.'/i', $columnName)) {
191 191
                     $column = $tableMapColumn;
192 192
                     break;
193 193
                 }
194 194
             }
195
-        } catch (\Exception $e) {
195
+        }catch (\Exception $e) {
196 196
             Logger::log($e->getMessage(), LOG_DEBUG);
197 197
         }
198 198
         return $column;
@@ -206,18 +206,18 @@  discard block
 block discarded – undo
206 206
     private static function addQueryFilter(ColumnMap $column, ModelCriteria &$query, $value = null)
207 207
     {
208 208
         $tableField = $column->getFullyQualifiedName();
209
-        if(is_array($value)) {
209
+        if (is_array($value)) {
210 210
             $query->add($tableField, $value, Criteria::IN);
211 211
         } elseif (preg_match('/^\[/', $value) && preg_match('/\]$/', $value)) {
212 212
             $query->add($tableField, explode(',', preg_replace('/(\[|\])/', '', $value)), Criteria::IN);
213 213
         } elseif (preg_match('/^(\'|\")(.*)(\'|\")$/', $value)) {
214 214
             $text = preg_replace('/(\'|\")/', '', $value);
215 215
             $text = preg_replace('/\ /', '%', $text);
216
-            $query->add($tableField, '%' . $text . '%', Criteria::LIKE);
217
-        } else {
218
-            if(null !== $column->getValueSet()) {
216
+            $query->add($tableField, '%'.$text.'%', Criteria::LIKE);
217
+        }else {
218
+            if (null !== $column->getValueSet()) {
219 219
                 $valueSet = $column->getValueSet();
220
-                if(in_array($value, $valueSet)) {
220
+                if (in_array($value, $valueSet)) {
221 221
                     $value = array_search($value, $valueSet);
222 222
                 }
223 223
             }
@@ -237,31 +237,31 @@  discard block
 block discarded – undo
237 237
         $sep = '';
238 238
         foreach ($tableMap->getColumns() as $column) {
239 239
             if ($column->isText()) {
240
-                $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
240
+                $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
241 241
                 $sep = ', " ", ';
242 242
             }
243 243
         }
244
-        foreach($tableMap->getRelations() as $relation) {
245
-            if(preg_match('/I18n$/i', $relation->getName())) {
244
+        foreach ($tableMap->getRelations() as $relation) {
245
+            if (preg_match('/I18n$/i', $relation->getName())) {
246 246
                 $localeTableMap = $relation->getLocalTable();
247 247
                 foreach ($localeTableMap->getColumns() as $column) {
248 248
                     if ($column->isText()) {
249
-                        $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
249
+                        $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
250 250
                         $sep = ', " ", ';
251 251
                     }
252 252
                 }
253 253
             }
254 254
         }
255 255
         foreach ($extraColumns as $extra => $name) {
256
-            if(!preg_match("/(COUNT|DISTINCT|SUM|MAX|MIN|GROUP)/i", $extra)) {
257
-                $exp .= $sep . $extra;
256
+            if (!preg_match("/(COUNT|DISTINCT|SUM|MAX|MIN|GROUP)/i", $extra)) {
257
+                $exp .= $sep.$extra;
258 258
                 $sep = ', " ", ';
259 259
             }
260 260
         }
261 261
         $exp .= ")";
262 262
         $text = preg_replace('/(\'|\")/', '', $value);
263 263
         $text = preg_replace('/\ /', '%', $text);
264
-        $query->where($exp . Criteria::LIKE . '"%' . $text . '%"');
264
+        $query->where($exp.Criteria::LIKE.'"%'.$text.'%"');
265 265
     }
266 266
 
267 267
     /**
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
      */
287 287
     public static function extractQuery($modelNameNamespace, ConnectionInterface $con = null)
288 288
     {
289
-        $queryReflector = new \ReflectionClass($modelNameNamespace . "Query");
289
+        $queryReflector = new \ReflectionClass($modelNameNamespace."Query");
290 290
         /** @var \Propel\Runtime\ActiveQuery\ModelCriteria $query */
291 291
         $query = $queryReflector->getMethod('create')->invoke($con);
292 292
 
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
315 315
         } elseif ($mappedColumn->isText()) {
316 316
             if ($mappedColumn->getSize() > 100) {
317 317
                 $fDto = self::createField($field, Field::TEXTAREA_TYPE, $required);
318
-            } else {
318
+            }else {
319 319
                 $fDto = self::generateStringField($field, $required);
320 320
             }
321 321
         } elseif ($mappedColumn->getType() === PropelTypes::BOOLEAN) {
@@ -343,7 +343,7 @@  discard block
 block discarded – undo
343 343
                 $fDto->pk = true;
344 344
             }
345 345
         }
346
-        switch(Config::getParam('api.field.type', TableMap::TYPE_PHPNAME)) {
346
+        switch (Config::getParam('api.field.type', TableMap::TYPE_PHPNAME)) {
347 347
             default:
348 348
             case TableMap::TYPE_PHPNAME:
349 349
                 $fDto->name = $mappedColumn->getPhpName();
@@ -367,7 +367,7 @@  discard block
 block discarded – undo
367 367
      */
368 368
     public static function extractPrimaryKeyColumnName(TableMap $tableMap) {
369 369
         $modelPk = null;
370
-        foreach($tableMap->getPrimaryKeys() as $pk) {
370
+        foreach ($tableMap->getPrimaryKeys() as $pk) {
371 371
             $modelPk = $pk;
372 372
             break;
373 373
         }
@@ -376,15 +376,15 @@  discard block
 block discarded – undo
376 376
 
377 377
     private static function mapResult(ActiveRecordInterface $model, array $data = []) {
378 378
         $result = [];
379
-        foreach($data as $key => $value) {
379
+        foreach ($data as $key => $value) {
380 380
             try {
381 381
                 $realValue = $model->getByName($key);
382
-            } catch(\Exception $e) {
382
+            }catch (\Exception $e) {
383 383
                 $realValue = $value;
384 384
             }
385
-            if(Api::API_MODEL_KEY_FIELD === $key) {
385
+            if (Api::API_MODEL_KEY_FIELD === $key) {
386 386
                 $result[$key] = (integer)$realValue;
387
-            } else {
387
+            }else {
388 388
                 $result[$key] = $realValue;
389 389
             }
390 390
         }
@@ -401,14 +401,14 @@  discard block
 block discarded – undo
401 401
         $objTableMap = get_class($formatter->getTableMap());
402 402
         /** @var TableMapTrait $objTableMap */
403 403
         $objData = $data;
404
-        foreach($objTableMap::getFieldNames() as $field) {
405
-            if(!array_key_exists($field, $objData)) {
404
+        foreach ($objTableMap::getFieldNames() as $field) {
405
+            if (!array_key_exists($field, $objData)) {
406 406
                 $objData[$field] = null;
407 407
             }
408 408
         }
409 409
         $obj = @$formatter->getAllObjectsFromRow($objData);
410 410
         $result = self::mapResult($obj, $data);
411
-        if(!preg_match('/' . $modelPk->getPhpName() . '/i', $query[Api::API_FIELDS_RESULT_FIELD])) {
411
+        if (!preg_match('/'.$modelPk->getPhpName().'/i', $query[Api::API_FIELDS_RESULT_FIELD])) {
412 412
             unset($result[$modelPk->getPhpName()]);
413 413
         }
414 414
         return $result;
Please login to merge, or discard this patch.
src/base/types/traits/Api/MutationTrait.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -75,21 +75,21 @@  discard block
 block discarded – undo
75 75
         if (count($pks) == 1) {
76 76
             $pks = array_keys($pks);
77 77
             return [
78
-                $tableMap::TABLE_NAME . '.' . $pks[0] => Api::API_MODEL_KEY_FIELD
78
+                $tableMap::TABLE_NAME.'.'.$pks[0] => Api::API_MODEL_KEY_FIELD
79 79
             ];
80 80
         } elseif (count($pks) > 1) {
81 81
             $apiPks = [];
82 82
             $principal = '';
83 83
             $sep = 'CONCAT(';
84 84
             foreach ($pks as $pk) {
85
-                $apiPks[$tableMap::TABLE_NAME . '.' . $pk->getName()] = $pk->getPhpName();
86
-                $principal .= $sep . $tableMap::TABLE_NAME . '.' . $pk->getName();
87
-                $sep = ', "' . Api::API_PK_SEPARATOR . '", ';
85
+                $apiPks[$tableMap::TABLE_NAME.'.'.$pk->getName()] = $pk->getPhpName();
86
+                $principal .= $sep.$tableMap::TABLE_NAME.'.'.$pk->getName();
87
+                $sep = ', "'.Api::API_PK_SEPARATOR.'", ';
88 88
             }
89 89
             $principal .= ')';
90 90
             $apiPks[$principal] = Api::API_MODEL_KEY_FIELD;
91 91
             return $apiPks;
92
-        } else {
92
+        }else {
93 93
             throw new ApiException(_('El modelo de la API no está debidamente mapeado, no hay Primary Key o es compuesta'));
94 94
         }
95 95
     }
@@ -112,10 +112,10 @@  discard block
 block discarded – undo
112 112
         $pks = '';
113 113
         $sep = '';
114 114
         foreach ($tableMap->getPrimaryKeys() as $pk) {
115
-            $pks .= $sep . $pk->getFullyQualifiedName();
115
+            $pks .= $sep.$pk->getFullyQualifiedName();
116 116
             $sep = ', "|", ';
117 117
         }
118
-        $this->extraColumns['CONCAT("' . $tableMap->getPhpName() . ' #", ' . $pks . ')'] = Api::API_LIST_NAME_FIELD;
118
+        $this->extraColumns['CONCAT("'.$tableMap->getPhpName().' #", '.$pks.')'] = Api::API_LIST_NAME_FIELD;
119 119
     }
120 120
 
121 121
     /**
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
             }
138 138
             if (null !== $column) {
139 139
                 $this->extraColumns[$column->getFullyQualifiedName()] = Api::API_LIST_NAME_FIELD;
140
-            } else {
140
+            }else {
141 141
                 $this->addClassListName($tableMap);
142 142
             }
143 143
         }
@@ -156,15 +156,15 @@  discard block
 block discarded – undo
156 156
             $this->addPkToList();
157 157
         }
158 158
         if (!empty($this->extraColumns)) {
159
-            if(Config::getParam('api.extrafields.compat', true)) {
159
+            if (Config::getParam('api.extrafields.compat', true)) {
160 160
                 $fields = array_values($this->extraColumns);
161
-            } else {
161
+            }else {
162 162
                 $returnFields = Request::getInstance()->getQuery(Api::API_FIELDS_RESULT_FIELD);
163 163
                 $fields = explode(',', $returnFields ?: '');
164 164
                 $fields[] = self::API_MODEL_KEY_FIELD;
165 165
             }
166 166
             foreach ($this->extraColumns as $expression => $columnName) {
167
-                if(empty($fields) || in_array($columnName, $fields)) {
167
+                if (empty($fields) || in_array($columnName, $fields)) {
168 168
                     $query->withColumn($expression, $columnName);
169 169
                 }
170 170
             }
@@ -196,17 +196,17 @@  discard block
 block discarded – undo
196 196
     {
197 197
         $this->extractApiLang();
198 198
         $model = $this->getModelNamespace();
199
-        $modelI18n = $model . 'I18n';
199
+        $modelI18n = $model.'I18n';
200 200
         if (method_exists($query, 'useI18nQuery')) {
201 201
             $query->useI18nQuery($this->lang);
202
-            $modelI18nTableMapClass = str_replace('\\Models\\', '\\Models\\Map\\', $modelI18n) . 'TableMap';
202
+            $modelI18nTableMapClass = str_replace('\\Models\\', '\\Models\\Map\\', $modelI18n).'TableMap';
203 203
             /** @var TableMap $modelI18nTableMap */
204 204
             $modelI18nTableMap = $modelI18nTableMapClass::getTableMap();
205
-            foreach($modelI18nTableMap->getColumns() as $columnMap) {
206
-                if(!$columnMap->isPrimaryKey()) {
207
-                    $query->withColumn($modelI18nTableMapClass::TABLE_NAME . '.' . $columnMap->getName(), $columnMap->getPhpName());
208
-                } elseif(!$columnMap->isForeignKey()) {
209
-                    $query->withColumn('IFNULL(' . $modelI18nTableMapClass::TABLE_NAME . '.' . $columnMap->getName() . ', "'.$this->lang.'")', $columnMap->getPhpName());
205
+            foreach ($modelI18nTableMap->getColumns() as $columnMap) {
206
+                if (!$columnMap->isPrimaryKey()) {
207
+                    $query->withColumn($modelI18nTableMapClass::TABLE_NAME.'.'.$columnMap->getName(), $columnMap->getPhpName());
208
+                } elseif (!$columnMap->isForeignKey()) {
209
+                    $query->withColumn('IFNULL('.$modelI18nTableMapClass::TABLE_NAME.'.'.$columnMap->getName().', "'.$this->lang.'")', $columnMap->getPhpName());
210 210
                 }
211 211
             }
212 212
         }
@@ -220,20 +220,20 @@  discard block
 block discarded – undo
220 220
         $model->fromArray($data);
221 221
         $tableMap = $this->getTableMap();
222 222
         try {
223
-            if($tableMap->hasRelation($tableMap->getPhpName() . 'I18n'))
223
+            if ($tableMap->hasRelation($tableMap->getPhpName().'I18n'))
224 224
             {
225
-                $relateI18n = $tableMap->getRelation($tableMap->getPhpName() . 'I18n');
225
+                $relateI18n = $tableMap->getRelation($tableMap->getPhpName().'I18n');
226 226
                 $i18NTableMap = $relateI18n->getLocalTable();
227
-                foreach($i18NTableMap->getColumns() as $columnMap) {
228
-                    $method = 'set' . $columnMap->getPhpName();
229
-                    if(!($columnMap->isPrimaryKey() && $columnMap->isForeignKey())
227
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
228
+                    $method = 'set'.$columnMap->getPhpName();
229
+                    if (!($columnMap->isPrimaryKey() && $columnMap->isForeignKey())
230 230
                         &&array_key_exists($columnMap->getPhpName(), $data)
231 231
                         && method_exists($model, $method)) {
232 232
                         $model->$method($data[$columnMap->getPhpName()]);
233 233
                     }
234 234
                 }
235 235
             }
236
-        } catch(\Exception $e) {
236
+        }catch (\Exception $e) {
237 237
             Logger::log($e->getMessage(), LOG_DEBUG);
238 238
         }
239 239
     }
@@ -243,7 +243,7 @@  discard block
 block discarded – undo
243 243
      */
244 244
     protected function checkFieldType() {
245 245
         $configType = Config::getParam('api.field.type');
246
-        switch($configType) {
246
+        switch ($configType) {
247 247
             case 'UpperCamelCase':
248 248
             case TableMap::TYPE_PHPNAME:
249 249
                 $this->fieldType = TableMap::TYPE_PHPNAME;
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.