Passed
Push — master ( 453953...eba0f7 )
by Fran
37:00 queued 23:59
created
src/base/types/Api.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
             $tableMap = $this->getTableMap();
174 174
             try {
175 175
                 $column = $tableMap->getColumnByPhpName($field);
176
-            } catch(\Exception $e) {
176
+            }catch (\Exception $e) {
177 177
                 Logger::log($e->getMessage(), LOG_ERR);
178 178
                 $column = null;
179 179
             }
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
          *
186 186
          * @param ModelCriteria $query
187 187
          */
188
-        private function addOrders(ModelCriteria &$query)
188
+        private function addOrders(ModelCriteria & $query)
189 189
         {
190 190
             $orderAdded = FALSE;
191 191
             foreach ($this->order->getOrders() as $field => $direction) {
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
                     $orderAdded = TRUE;
194 194
                     if ($direction === Order::ASC) {
195 195
                         $query->addAscendingOrderByColumn($column->getPhpName());
196
-                    } else {
196
+                    }else {
197 197
                         $query->addDescendingOrderByColumn($column->getPhpName());
198 198
                     }
199 199
                 }
@@ -208,7 +208,7 @@  discard block
 block discarded – undo
208 208
          *
209 209
          * @param ModelCriteria $query
210 210
          */
211
-        private function addExtraColumns(ModelCriteria &$query)
211
+        private function addExtraColumns(ModelCriteria & $query)
212 212
         {
213 213
             if (!empty($this->extraColumns)) {
214 214
                 foreach ($this->extraColumns as $expression => $columnName) {
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
          *
223 223
          * @param ModelCriteria $query
224 224
          */
225
-        protected function joinTables(ModelCriteria &$query)
225
+        protected function joinTables(ModelCriteria & $query)
226 226
         {
227 227
             //TODO for specific implementations
228 228
         }
@@ -232,7 +232,7 @@  discard block
 block discarded – undo
232 232
          */
233 233
         protected function parseExtraColumns() {
234 234
             $columns = [];
235
-            foreach($this->extraColumns as $key => $columnName) {
235
+            foreach ($this->extraColumns as $key => $columnName) {
236 236
                 $columns[$columnName] = strtolower($columnName);
237 237
             }
238 238
             return $columns;
@@ -243,7 +243,7 @@  discard block
 block discarded – undo
243 243
          *
244 244
          * @param ModelCriteria $query
245 245
          */
246
-        private function addFilters(ModelCriteria &$query)
246
+        private function addFilters(ModelCriteria & $query)
247 247
         {
248 248
             if (count($this->query) > 0) {
249 249
                 foreach ($this->query as $field => $value) {
@@ -261,22 +261,22 @@  discard block
 block discarded – undo
261 261
                             $text = preg_replace('/(\'|\")/', '', $value);
262 262
                             $text = preg_replace('/\ /', '%', $text);
263 263
                             $query->filterBy($tableField, '%'.$text.'%', Criteria::LIKE);
264
-                        } else {
264
+                        }else {
265 265
                             $query->filterBy($tableField, $value, Criteria::EQUAL);
266 266
                         }
267
-                    } elseif(self::API_COMBO_FILTER === $field) {
267
+                    } elseif (self::API_COMBO_FILTER === $field) {
268 268
                         $exp = 'CONCAT(';
269 269
                         $sep = '';
270 270
                         $tablemap = $this->getTableMap();
271
-                        foreach($tablemap->getColumns() as $column) {
272
-                            if($column->isText()) {
273
-                                $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
271
+                        foreach ($tablemap->getColumns() as $column) {
272
+                            if ($column->isText()) {
273
+                                $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
274 274
                                 $sep = ', " ", ';
275 275
                             }
276 276
                         }
277 277
                         $exp .= ")";
278
-                        $query->where($exp . Criteria::LIKE . '"'.$value.'"');
279
-                    } else {
278
+                        $query->where($exp.Criteria::LIKE.'"'.$value.'"');
279
+                    }else {
280 280
                         $tableField = null;
281 281
                     }
282 282
                 }
@@ -298,10 +298,10 @@  discard block
 block discarded – undo
298 298
                 list($page, $limit) = $this->extractPagination();
299 299
                 if ($limit == -1) {
300 300
                     $this->list = $query->find($this->con);
301
-                } else {
301
+                }else {
302 302
                     $this->list = $query->paginate($page, $limit, $this->con);
303 303
                 }
304
-            } catch (\Exception $e) {
304
+            }catch (\Exception $e) {
305 305
                 Logger::log($e->getMessage(), LOG_ERR);
306 306
             }
307 307
         }
@@ -318,7 +318,7 @@  discard block
 block discarded – undo
318 318
                 $this->joinTables($query);
319 319
                 $this->addExtraColumns($query);
320 320
                 $this->model = $query->findPk($pk);
321
-            } catch (\Exception $e) {
321
+            }catch (\Exception $e) {
322 322
                 Logger::getInstance(get_class($this))->errorLog($e->getMessage());
323 323
             }
324 324
         }
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
                     $saved = TRUE;
407 407
                     $model = $this->model->toArray();
408 408
                 }
409
-            } catch (\Exception $e) {
409
+            }catch (\Exception $e) {
410 410
                 Logger::log($e->getMessage(), LOG_ERR);
411 411
             }
412 412
 
@@ -434,7 +434,7 @@  discard block
 block discarded – undo
434 434
                         $this->model->delete($this->con);
435 435
                         $deleted = TRUE;
436 436
                     }
437
-                } catch (\Exception $e) {
437
+                }catch (\Exception $e) {
438 438
                     Logger::getInstance(get_class($this->model))->errorLog($e->getMessage());
439 439
                 }
440 440
             }
@@ -478,7 +478,7 @@  discard block
 block discarded – undo
478 478
          */
479 479
         private function extractQuery()
480 480
         {
481
-            $queryReflector = new \ReflectionClass($this->getModelNamespace() . "Query");
481
+            $queryReflector = new \ReflectionClass($this->getModelNamespace()."Query");
482 482
             /** @var \Propel\Runtime\ActiveQuery\ModelCriteria $query */
483 483
             $query = $queryReflector->getMethod('create')->invoke($this->con);
484 484
 
@@ -522,7 +522,7 @@  discard block
 block discarded – undo
522 522
             if (null !== $this->con && $this->con->inTransaction()) {
523 523
                 if ($status === 200) {
524 524
                     $this->con->commit();
525
-                } else {
525
+                }else {
526 526
                     $this->con->rollBack();
527 527
                 }
528 528
             }
@@ -566,7 +566,7 @@  discard block
 block discarded – undo
566 566
             return $this->render('api.admin.html.twig', array(
567 567
                 "api"    => $this->getApi(),
568 568
                 "domain" => $this->domain,
569
-                "url"    => preg_replace('/\/\{(.*)\}$/i', '', $this->getRoute(strtolower('api-' . $this->getApi() . "-pk"), TRUE)),
569
+                "url"    => preg_replace('/\/\{(.*)\}$/i', '', $this->getRoute(strtolower('api-'.$this->getApi()."-pk"), TRUE)),
570 570
             ), [], '');
571 571
         }
572 572
 
@@ -600,12 +600,12 @@  discard block
 block discarded – undo
600 600
             $pages = 0;
601 601
             try {
602 602
                 $this->paginate();
603
-                if(null !== $this->list) {
603
+                if (null !== $this->list) {
604 604
                     $return = $this->list->toArray();
605 605
                     $total = $this->list->getNbResults();
606 606
                     $pages = $this->list->getLastPage();
607 607
                 }
608
-            } catch (\Exception $e) {
608
+            }catch (\Exception $e) {
609 609
                 Logger::getInstance(get_class($this))->errorLog($e->getMessage());
610 610
             }
611 611
 
@@ -624,7 +624,7 @@  discard block
 block discarded – undo
624 624
             $return = array();
625 625
             if (NULL === $model && method_exists($model, 'toArray')) {
626 626
                 $code = 404;
627
-            } else {
627
+            }else {
628 628
                 $return = $model->toArray();
629 629
             }
630 630
 
Please login to merge, or discard this patch.