Passed
Push — master ( b37f55...96b935 )
by Fran
04:33
created
src/base/types/Api.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
      *
187 187
      * @param ModelCriteria $query
188 188
      */
189
-    private function addOrders(ModelCriteria &$query)
189
+    private function addOrders(ModelCriteria & $query)
190 190
     {
191 191
         $orderAdded = FALSE;
192 192
         $tableMap = $this->getTableMap();
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
                 $orderAdded = TRUE;
196 196
                 if ($direction === Order::ASC) {
197 197
                     $query->addAscendingOrderByColumn($column->getPhpName());
198
-                } else {
198
+                }else {
199 199
                     $query->addDescendingOrderByColumn($column->getPhpName());
200 200
                 }
201 201
             }
@@ -210,8 +210,8 @@  discard block
 block discarded – undo
210 210
         $pks = $tableMap->getPrimaryKeys();
211 211
         if (count($pks) == 1) {
212 212
             $pks = array_keys($pks);
213
-            return $tableMap::TABLE_NAME . '.' . $pks[0];
214
-        } else {
213
+            return $tableMap::TABLE_NAME.'.'.$pks[0];
214
+        }else {
215 215
             throw new ApiException(_('El modelo de la API no está debidamente mapeado, no hay Primary Key o es compuesta'));
216 216
         }
217 217
     }
@@ -242,7 +242,7 @@  discard block
 block discarded – undo
242 242
             }
243 243
             if (null !== $column) {
244 244
                 $this->extraColumns[$column->getFullyQualifiedName()] = self::API_LIST_NAME_FIELD;
245
-            } else {
245
+            }else {
246 246
                 $this->addClassListName($tableMap);
247 247
             }
248 248
         }
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
      *
254 254
      * @param ModelCriteria $query
255 255
      */
256
-    private function addExtraColumns(ModelCriteria &$query)
256
+    private function addExtraColumns(ModelCriteria & $query)
257 257
     {
258 258
         if (self::API_ACTION_LIST === $this->action) {
259 259
             $this->addDefaultListField();
@@ -271,7 +271,7 @@  discard block
 block discarded – undo
271 271
      *
272 272
      * @param ModelCriteria $query
273 273
      */
274
-    protected function joinTables(ModelCriteria &$query)
274
+    protected function joinTables(ModelCriteria & $query)
275 275
     {
276 276
         //TODO for specific implementations
277 277
     }
@@ -293,14 +293,14 @@  discard block
 block discarded – undo
293 293
      *
294 294
      * @param ModelCriteria $query
295 295
      */
296
-    private function addFilters(ModelCriteria &$query)
296
+    private function addFilters(ModelCriteria & $query)
297 297
     {
298 298
         if (count($this->query) > 0) {
299 299
             $tableMap = $this->getTableMap();
300 300
             foreach ($this->query as $field => $value) {
301 301
                 if (self::API_COMBO_FIELD === $field) {
302 302
                     ApiHelper::composerComboField($tableMap, $query, $this->extraColumns, $value);
303
-                } else {
303
+                }else {
304 304
                     ApiHelper::addModelField($tableMap, $query, $field, $value);
305 305
                 }
306 306
             }
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
     /**
311 311
      * @param ModelCriteria $query
312 312
      */
313
-    private function checkReturnFields(ModelCriteria &$query)
313
+    private function checkReturnFields(ModelCriteria & $query)
314 314
     {
315 315
         $returnFields = $this->getRequest()->getQuery('__fields');
316 316
         if (null !== $returnFields) {
@@ -346,10 +346,10 @@  discard block
 block discarded – undo
346 346
             list($page, $limit) = $this->extractPagination();
347 347
             if ($limit == -1) {
348 348
                 $this->list = $query->find($this->con);
349
-            } else {
349
+            }else {
350 350
                 $this->list = $query->paginate($page, $limit, $this->con);
351 351
             }
352
-        } catch (\Exception $e) {
352
+        }catch (\Exception $e) {
353 353
             Logger::log($e->getMessage(), LOG_ERR);
354 354
         }
355 355
     }
@@ -366,7 +366,7 @@  discard block
 block discarded – undo
366 366
             $this->joinTables($query);
367 367
             $this->addExtraColumns($query);
368 368
             $this->model = $query->findPk($pk);
369
-        } catch (\Exception $e) {
369
+        }catch (\Exception $e) {
370 370
             Logger::getInstance(get_class($this))->errorLog($e->getMessage());
371 371
         }
372 372
     }
@@ -457,8 +457,8 @@  discard block
 block discarded – undo
457 457
                 $saved = TRUE;
458 458
                 $model = $this->model->toArray();
459 459
             }
460
-        } catch (\Exception $e) {
461
-            $model = _('Ha ocurrido un error intentando guardar el elemento: ') . $e->getMessage();
460
+        }catch (\Exception $e) {
461
+            $model = _('Ha ocurrido un error intentando guardar el elemento: ').$e->getMessage();
462 462
             Logger::log($e->getMessage(), LOG_ERR);
463 463
         }
464 464
 
@@ -488,7 +488,7 @@  discard block
 block discarded – undo
488 488
                     $this->model->delete($this->con);
489 489
                     $deleted = TRUE;
490 490
                 }
491
-            } catch (\Exception $e) {
491
+            }catch (\Exception $e) {
492 492
                 $message = _('Ha ocurrido un error intentando eliminar el elemento, por favor verifica que no tenga otros elementos relacionados');
493 493
                 Logger::getInstance(get_class($this->model))->errorLog($e->getMessage());
494 494
             }
@@ -523,14 +523,14 @@  discard block
 block discarded – undo
523 523
                     $updated = TRUE;
524 524
                     $status = 200;
525 525
                     $model = $this->model->toArray();
526
-                } else {
526
+                }else {
527 527
                     $model = _('Ha ocurrido un error intentando actualizar el elemento, por favor revisa los logs');
528 528
                 }
529
-            } catch (\Exception $e) {
529
+            }catch (\Exception $e) {
530 530
                 $model = $e->getMessage();
531 531
                 Logger::getInstance(get_class($this->model))->errorLog($e->getMessage());
532 532
             }
533
-        } else {
533
+        }else {
534 534
             $model = _('Ha ocurrido un error intentando actualizar el elemento, por favor revisa los logs');
535 535
         }
536 536
 
@@ -542,7 +542,7 @@  discard block
 block discarded – undo
542 542
      */
543 543
     private function extractQuery()
544 544
     {
545
-        $queryReflector = new \ReflectionClass($this->getModelNamespace() . "Query");
545
+        $queryReflector = new \ReflectionClass($this->getModelNamespace()."Query");
546 546
         /** @var \Propel\Runtime\ActiveQuery\ModelCriteria $query */
547 547
         $query = $queryReflector->getMethod('create')->invoke($this->con);
548 548
 
@@ -586,7 +586,7 @@  discard block
 block discarded – undo
586 586
         if (null !== $this->con && $this->con->inTransaction()) {
587 587
             if ($status === 200) {
588 588
                 $this->con->commit();
589
-            } else {
589
+            }else {
590 590
                 $this->con->rollBack();
591 591
             }
592 592
         }
@@ -638,8 +638,8 @@  discard block
 block discarded – undo
638 638
             "domain" => $this->getDomain(),
639 639
             "listLabel" => self::API_LIST_NAME_FIELD,
640 640
             'modelId' => self::API_MODEL_KEY_FIELD,
641
-            'formUrl' => preg_replace('/\/\{(.*)\}$/i', '', $this->getRoute(strtolower('admin-api-form-' . $this->getDomain() . '-' . $this->getApi()), TRUE)),
642
-            "url" => preg_replace('/\/\{(.*)\}$/i', '', $this->getRoute(strtolower($this->getDomain() . '-' . 'api-' . $this->getApi() . "-pk"), TRUE)),
641
+            'formUrl' => preg_replace('/\/\{(.*)\}$/i', '', $this->getRoute(strtolower('admin-api-form-'.$this->getDomain().'-'.$this->getApi()), TRUE)),
642
+            "url" => preg_replace('/\/\{(.*)\}$/i', '', $this->getRoute(strtolower($this->getDomain().'-'.'api-'.$this->getApi()."-pk"), TRUE)),
643 643
         ), [], '');
644 644
     }
645 645
 
@@ -679,7 +679,7 @@  discard block
 block discarded – undo
679 679
                 $total = $this->list->getNbResults();
680 680
                 $pages = $this->list->getLastPage();
681 681
             }
682
-        } catch (\Exception $e) {
682
+        }catch (\Exception $e) {
683 683
             Logger::getInstance(get_class($this))->errorLog($e->getMessage());
684 684
         }
685 685
 
@@ -698,7 +698,7 @@  discard block
 block discarded – undo
698 698
         $return = array();
699 699
         if (NULL === $model && method_exists($model, 'toArray')) {
700 700
             $code = 404;
701
-        } else {
701
+        }else {
702 702
             $return = $model->toArray();
703 703
         }
704 704
 
@@ -713,9 +713,9 @@  discard block
 block discarded – undo
713 713
         $pks = '';
714 714
         $sep = '';
715 715
         foreach ($tableMap->getPrimaryKeys() as $pk) {
716
-            $pks .= $sep . $pk->getFullyQualifiedName();
716
+            $pks .= $sep.$pk->getFullyQualifiedName();
717 717
             $sep = ', "|", ';
718 718
         }
719
-        $this->extraColumns['CONCAT("' . $tableMap->getPhpName() . ' #", ' . $pks . ')'] = self::API_LIST_NAME_FIELD;
719
+        $this->extraColumns['CONCAT("'.$tableMap->getPhpName().' #", '.$pks.')'] = self::API_LIST_NAME_FIELD;
720 720
     }
721 721
 }
Please login to merge, or discard this patch.