psfs / core

src/base/types/helpers/SecurityHelper.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         if (!Config::getInstance()->checkTryToSaveConfig()
             && (preg_match('/^\/(admin|setup\-admin)/i', $route) || NULL !== Config::getInstance()->get('restricted'))
         ) {
-            if(!file_exists(CONFIG_DIR . DIRECTORY_SEPARATOR . 'admins.json')) {
+            if (!file_exists(CONFIG_DIR.DIRECTORY_SEPARATOR.'admins.json')) {
                 //Si no hay fichero de usuarios redirigimos directamente al gestor
                 return UserController::showAdminManager();
             }

src/base/types/Api.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
     namespace PSFS\base\types;
 
-    use Propel\Runtime\ActiveQuery\Criteria;
     use Propel\Runtime\ActiveQuery\ModelCriteria;
     use Propel\Runtime\ActiveRecord\ActiveRecordInterface;
     use Propel\Runtime\Connection\ConnectionInterface;

Spacing +31 added lines, -31 removed lines

@@ -168,7 +168,7 @@ 
          *
          * @param ModelCriteria $query
          */
-        private function addOrders(ModelCriteria &$query)
+        private function addOrders(ModelCriteria & $query)
         {
             $orderAdded = FALSE;
             $tableMap = $this->getTableMap();
@@ -177,7 +177,7 @@ 
                     $orderAdded = TRUE;
                     if ($direction === Order::ASC) {
                         $query->addAscendingOrderByColumn($column->getPhpName());
-                    } else {
+                    }else {
                         $query->addDescendingOrderByColumn($column->getPhpName());
                     }
                 }
@@ -191,19 +191,19 @@ 
          * Method that add a new field with the Label of the row
          */
         private function addDefaultListField() {
-            if(!in_array(self::API_LIST_NAME_FIELD, $this->extraColumns)) {
+            if (!in_array(self::API_LIST_NAME_FIELD, $this->extraColumns)) {
                 $tableMap = $this->getTableMap();
                 $column = null;
-                if($tableMap->hasColumn('NAME')) {
+                if ($tableMap->hasColumn('NAME')) {
                     $column = $tableMap->getColumn('NAME');
-                } elseif($tableMap->hasColumn('TITLE')) {
+                } elseif ($tableMap->hasColumn('TITLE')) {
                     $column = $tableMap->getColumn('TITLE');
-                } elseif($tableMap->hasColumn('LABEL')) {
+                } elseif ($tableMap->hasColumn('LABEL')) {
                     $column = $tableMap->getColumn('LABEL');
                 }
-                if(null !== $column) {
+                if (null !== $column) {
                     $this->extraColumns[$column->getFullyQualifiedName()] = self::API_LIST_NAME_FIELD;
-                } else {
+                }else {
                     $this->addClassListName($tableMap);
                 }
             }
@@ -214,7 +214,7 @@ 
          *
          * @param ModelCriteria $query
          */
-        private function addExtraColumns(ModelCriteria &$query)
+        private function addExtraColumns(ModelCriteria & $query)
         {
             $this->addDefaultListField();
             if (!empty($this->extraColumns)) {
@@ -229,7 +229,7 @@ 
          *
          * @param ModelCriteria $query
          */
-        protected function joinTables(ModelCriteria &$query)
+        protected function joinTables(ModelCriteria & $query)
         {
             //TODO for specific implementations
         }
@@ -239,7 +239,7 @@ 
          */
         protected function parseExtraColumns() {
             $columns = [];
-            foreach($this->extraColumns as $key => $columnName) {
+            foreach ($this->extraColumns as $key => $columnName) {
                 $columns[$columnName] = strtolower($columnName);
             }
             return $columns;
@@ -250,14 +250,14 @@ 
          *
          * @param ModelCriteria $query
          */
-        private function addFilters(ModelCriteria &$query)
+        private function addFilters(ModelCriteria & $query)
         {
             if (count($this->query) > 0) {
                 $tableMap = $this->getTableMap();
                 foreach ($this->query as $field => $value) {
-                    if(self::API_COMBO_FIELD === $field) {
+                    if (self::API_COMBO_FIELD === $field) {
                         ApiHelper::composerComboField($tableMap, $query, $this->extraColumns, $value);
-                    } else {
+                    }else {
                         ApiHelper::addModelField($tableMap, $query, $field, $value);
                     }
                 }
@@ -279,10 +279,10 @@ 
                 list($page, $limit) = $this->extractPagination();
                 if ($limit == -1) {
                     $this->list = $query->find($this->con);
-                } else {
+                }else {
                     $this->list = $query->paginate($page, $limit, $this->con);
                 }
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 Logger::log($e->getMessage(), LOG_ERR);
             }
         }
@@ -299,7 +299,7 @@ 
                 $this->joinTables($query);
                 $this->addExtraColumns($query);
                 $this->model = $query->findPk($pk);
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 Logger::getInstance(get_class($this))->errorLog($e->getMessage());
             }
         }
@@ -387,8 +387,8 @@ 
                     $saved = TRUE;
                     $model = $this->model->toArray();
                 }
-            } catch (\Exception $e) {
-                $model = _('Ha ocurrido un error intentando guardar el elemento: ') . $e->getMessage();
+            }catch (\Exception $e) {
+                $model = _('Ha ocurrido un error intentando guardar el elemento: ').$e->getMessage();
                 Logger::log($e->getMessage(), LOG_ERR);
             }
 
@@ -417,7 +417,7 @@ 
                         $this->model->delete($this->con);
                         $deleted = TRUE;
                     }
-                } catch (\Exception $e) {
+                }catch (\Exception $e) {
                     $message = _('Ha ocurrido un error intentando eliminar el elemento, por favor verifica que no tenga otros elementos relacionados');
                     Logger::getInstance(get_class($this->model))->errorLog($e->getMessage());
                 }
@@ -451,14 +451,14 @@ 
                         $updated = TRUE;
                         $status = 200;
                         $model = $this->model->toArray();
-                    } else {
+                    }else {
                         $model = _('Ha ocurrido un error intentando actualizar el elemento, por favor revisa los logs');
                     }
-                } catch(\Exception $e) {
+                }catch (\Exception $e) {
                     $model = $e->getMessage();
                     Logger::getInstance(get_class($this->model))->errorLog($e->getMessage());
                 }
-            } else {
+            }else {
                 $model = _('Ha ocurrido un error intentando actualizar el elemento, por favor revisa los logs');
             }
 
@@ -470,7 +470,7 @@ 
          */
         private function extractQuery()
         {
-            $queryReflector = new \ReflectionClass($this->getModelNamespace() . "Query");
+            $queryReflector = new \ReflectionClass($this->getModelNamespace()."Query");
             /** @var \Propel\Runtime\ActiveQuery\ModelCriteria $query */
             $query = $queryReflector->getMethod('create')->invoke($this->con);
 
@@ -514,7 +514,7 @@ 
             if (null !== $this->con && $this->con->inTransaction()) {
                 if ($status === 200) {
                     $this->con->commit();
-                } else {
+                }else {
                     $this->con->rollBack();
                 }
             }
@@ -558,7 +558,7 @@ 
             return $this->render('api.admin.html.twig', array(
                 "api"    => $this->getApi(),
                 "domain" => $this->domain,
-                "url"    => preg_replace('/\/\{(.*)\}$/i', '', $this->getRoute(strtolower('api-' . $this->getApi() . "-pk"), TRUE)),
+                "url"    => preg_replace('/\/\{(.*)\}$/i', '', $this->getRoute(strtolower('api-'.$this->getApi()."-pk"), TRUE)),
             ), [], '');
         }
 
@@ -592,12 +592,12 @@ 
             $pages = 0;
             try {
                 $this->paginate();
-                if(null !== $this->list) {
+                if (null !== $this->list) {
                     $return = $this->list->toArray();
                     $total = $this->list->getNbResults();
                     $pages = $this->list->getLastPage();
                 }
-            } catch (\Exception $e) {
+            }catch (\Exception $e) {
                 Logger::getInstance(get_class($this))->errorLog($e->getMessage());
             }
 
@@ -616,7 +616,7 @@ 
             $return = array();
             if (NULL === $model && method_exists($model, 'toArray')) {
                 $code = 404;
-            } else {
+            }else {
                 $return = $model->toArray();
             }
 
@@ -631,10 +631,10 @@ 
             $pks = '';
             $sep = '';
             foreach ($tableMap->getPrimaryKeys() as $pk) {
-                $pks .= $sep . $pk->getFullyQualifiedName();
+                $pks .= $sep.$pk->getFullyQualifiedName();
                 $sep = ', "|", ';
             }
-            $this->extraColumns['CONCAT("' . $tableMap->getPhpName() . ' #", ' . $pks . ')'] = self::API_LIST_NAME_FIELD;
+            $this->extraColumns['CONCAT("'.$tableMap->getPhpName().' #", '.$pks.')'] = self::API_LIST_NAME_FIELD;
         }
 
         /**

src/base/types/helpers/ApiHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -192,7 +192,7 @@
 
     /**
      * @param ColumnMap $column
-     * @param ModelCriteria $quwuery
+     * @param ModelCriteria $query
      * @param mixed $value
      */
     private static function addQueryFilter(ColumnMap $column, ModelCriteria &$query, $value = null) {

Spacing +15 added lines, -15 removed lines

@@ -45,9 +45,9 @@ 
                 $fDto = self::generatePasswordField($field, $required);
             } elseif (in_array($mappedColumn->getType(), [PropelTypes::TIMESTAMP])) {
                 $fDto = self::createField($field, Field::TEXT_TYPE, $required);
-            } elseif(in_array($mappedColumn->getType(), [PropelTypes::ENUM, PropelTypes::SET])) {
+            } elseif (in_array($mappedColumn->getType(), [PropelTypes::ENUM, PropelTypes::SET])) {
                 $fDto = self::generateEnumField($field, $required);
-                foreach($mappedColumn->getValueSet() as $value) {
+                foreach ($mappedColumn->getValueSet() as $value) {
                     $fDto->data[] = [
                         $field => $value,
                         "Label" => _($value),
@@ -55,7 +55,7 @@ 
                 }
             }
 
-            if(null !== $fDto) {
+            if (null !== $fDto) {
                 $form->addField($fDto);
             }
         }
@@ -78,7 +78,7 @@ 
         $fDto->entity = $relatedModel;
         $relatedField = $foreignTable->getColumn($mappedColumn->getRelatedColumnName());
         $fDto->relatedField = $relatedField->getPhpName();
-        $fDto->url = Router::getInstance()->getRoute('api-' . $relatedModel);
+        $fDto->url = Router::getInstance()->getRoute('api-'.$relatedModel);
         return $fDto;
     }
 
@@ -183,7 +183,7 @@ 
     public static function checkFieldExists(TableMap $tableMap, $field) {
         try {
             $column = $tableMap->getColumnByPhpName($field);
-        } catch(\Exception $e) {
+        }catch (\Exception $e) {
             Logger::log($e->getMessage(), LOG_ERR);
             $column = null;
         }
@@ -195,7 +195,7 @@ 
      * @param ModelCriteria $quwuery
      * @param mixed $value
      */
-    private static function addQueryFilter(ColumnMap $column, ModelCriteria &$query, $value = null) {
+    private static function addQueryFilter(ColumnMap $column, ModelCriteria & $query, $value = null) {
         $tableField = $column->getPhpName();
         if (preg_match('/^<=/', $value)) {
             $query->filterBy($tableField, substr($value, 2, strlen($value)), Criteria::LESS_EQUAL);
@@ -209,7 +209,7 @@ 
             $text = preg_replace('/(\'|\")/', '', $value);
             $text = preg_replace('/\ /', '%', $text);
             $query->filterBy($tableField, '%'.$text.'%', Criteria::LIKE);
-        } else {
+        }else {
             $query->filterBy($tableField, $value, Criteria::EQUAL);
         }
     }
@@ -220,23 +220,23 @@ 
      * @param array $extraColumns
      * @param mixed $value
      */
-    public static function composerComboField(TableMap $tableMap, ModelCriteria &$query, array $extraColumns = [], $value = null) {
+    public static function composerComboField(TableMap $tableMap, ModelCriteria & $query, array $extraColumns = [], $value = null) {
         $exp = 'CONCAT(';
         $sep = '';
-        foreach($tableMap->getColumns() as $column) {
-            if($column->isText()) {
-                $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
+        foreach ($tableMap->getColumns() as $column) {
+            if ($column->isText()) {
+                $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
                 $sep = ', " ", ';
             }
         }
-        foreach($extraColumns as $extra => $name) {
-            $exp .= $sep . $extra;
+        foreach ($extraColumns as $extra => $name) {
+            $exp .= $sep.$extra;
             $sep = ', " ", ';
         }
         $exp .= ")";
         $text = preg_replace('/(\'|\")/', '', $value);
         $text = preg_replace('/\ /', '%', $text);
-        $query->where($exp . Criteria::LIKE . '"%' . $text . '%"');
+        $query->where($exp.Criteria::LIKE.'"%'.$text.'%"');
     }
 
     /**
@@ -246,7 +246,7 @@ 
      * @param string $field
      * @param mixed $value
      */
-    public static function addModelField(TableMap $tableMap, ModelCriteria &$query, $field, $value = null) {
+    public static function addModelField(TableMap $tableMap, ModelCriteria & $query, $field, $value = null) {
         if ($column = self::checkFieldExists($tableMap, $field)) {
             self::addQueryFilter($column, $query, $value);
         }