Passed
Push — master ( 472f53...0ee5cf )
by Fran
02:53
created
src/base/types/traits/Form/FormSecurityTrait.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
         }
30 30
         if ('' !== $hashOrig) {
31 31
             $this->crfs = sha1($hashOrig);
32
-            $this->add($this->getName() . '_token', array(
32
+            $this->add($this->getName().'_token', array(
33 33
                 'type' => 'hidden',
34 34
                 'value' => $this->crfs,
35 35
             ));
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
     public function isValid()
44 44
     {
45 45
         $valid = true;
46
-        $tokenField = $this->getName() . '_token';
46
+        $tokenField = $this->getName().'_token';
47 47
         // Check crfs token
48 48
         if (!$this->existsFormToken($tokenField)) {
49 49
             $this->errors[$tokenField] = t('Formulario no válido');
Please login to merge, or discard this patch.
src/base/types/traits/Api/MutationTrait.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
         if (count($pks) === 1) {
81 81
             $pks = array_keys($pks);
82 82
             return [
83
-                $tableMap::TABLE_NAME . '.' . $pks[0] => Api::API_MODEL_KEY_FIELD
83
+                $tableMap::TABLE_NAME.'.'.$pks[0] => Api::API_MODEL_KEY_FIELD
84 84
             ];
85 85
         }
86 86
         if (count($pks) > 1) {
@@ -88,9 +88,9 @@  discard block
 block discarded – undo
88 88
             $principal = '';
89 89
             $sep = 'CONCAT(';
90 90
             foreach ($pks as $pk) {
91
-                $apiPks[$tableMap::TABLE_NAME . '.' . $pk->getName()] = $pk->getPhpName();
92
-                $principal .= $sep . $tableMap::TABLE_NAME . '.' . $pk->getName();
93
-                $sep = ', "' . Api::API_PK_SEPARATOR . '", ';
91
+                $apiPks[$tableMap::TABLE_NAME.'.'.$pk->getName()] = $pk->getPhpName();
92
+                $principal .= $sep.$tableMap::TABLE_NAME.'.'.$pk->getName();
93
+                $sep = ', "'.Api::API_PK_SEPARATOR.'", ';
94 94
             }
95 95
             $principal .= ')';
96 96
             $apiPks[$principal] = Api::API_MODEL_KEY_FIELD;
@@ -117,10 +117,10 @@  discard block
 block discarded – undo
117 117
         $pks = '';
118 118
         $sep = '';
119 119
         foreach ($tableMap->getPrimaryKeys() as $pk) {
120
-            $pks .= $sep . $pk->getFullyQualifiedName();
120
+            $pks .= $sep.$pk->getFullyQualifiedName();
121 121
             $sep = ', "|", ';
122 122
         }
123
-        $this->extraColumns['CONCAT("' . $tableMap->getPhpName() . ' #", ' . $pks . ')'] = Api::API_LIST_NAME_FIELD;
123
+        $this->extraColumns['CONCAT("'.$tableMap->getPhpName().' #", '.$pks.')'] = Api::API_LIST_NAME_FIELD;
124 124
     }
125 125
 
126 126
     /**
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
             }
143 143
             if (null !== $column) {
144 144
                 $this->extraColumns[$column->getFullyQualifiedName()] = Api::API_LIST_NAME_FIELD;
145
-            } else {
145
+            }else {
146 146
                 $this->addClassListName($tableMap);
147 147
             }
148 148
         }
@@ -160,15 +160,15 @@  discard block
 block discarded – undo
160 160
             $this->addPkToList();
161 161
         }
162 162
         if (!empty($this->extraColumns)) {
163
-            if(Config::getParam('api.extrafields.compat', true)) {
163
+            if (Config::getParam('api.extrafields.compat', true)) {
164 164
                 $fields = array_values($this->extraColumns);
165
-            } else {
165
+            }else {
166 166
                 $returnFields = Request::getInstance()->getQuery(Api::API_FIELDS_RESULT_FIELD);
167 167
                 $fields = explode(',', $returnFields ?: '');
168 168
                 $fields[] = self::API_MODEL_KEY_FIELD;
169 169
             }
170 170
             foreach ($this->extraColumns as $expression => $columnName) {
171
-                if(empty($fields) || in_array($columnName, $fields)) {
171
+                if (empty($fields) || in_array($columnName, $fields)) {
172 172
                     $query->withColumn($expression, $columnName);
173 173
                 }
174 174
             }
@@ -199,17 +199,17 @@  discard block
 block discarded – undo
199 199
     {
200 200
         $this->extractApiLang();
201 201
         $model = $this->getModelNamespace();
202
-        $modelI18n = $model . 'I18n';
202
+        $modelI18n = $model.'I18n';
203 203
         if (method_exists($query, 'useI18nQuery')) {
204 204
             $query->useI18nQuery($this->lang);
205
-            $i18nMapClass = str_replace('\\Models\\', '\\Models\\Map\\', $modelI18n) . 'TableMap';
205
+            $i18nMapClass = str_replace('\\Models\\', '\\Models\\Map\\', $modelI18n).'TableMap';
206 206
             /** @var TableMap $modelI18nTableMap */
207 207
             $modelI18nTableMap = $i18nMapClass::getTableMap();
208
-            foreach($modelI18nTableMap->getColumns() as $columnMap) {
209
-                if(!$columnMap->isPrimaryKey()) {
208
+            foreach ($modelI18nTableMap->getColumns() as $columnMap) {
209
+                if (!$columnMap->isPrimaryKey()) {
210 210
                     $query->withColumn($columnMap->getFullyQualifiedName(), ApiHelper::getColumnMapName($columnMap));
211
-                } elseif(!$columnMap->isForeignKey()) {
212
-                    $query->withColumn('IFNULL(' . $columnMap->getFullyQualifiedName() . ', "'.$this->lang.'")', ApiHelper::getColumnMapName($columnMap));
211
+                } elseif (!$columnMap->isForeignKey()) {
212
+                    $query->withColumn('IFNULL('.$columnMap->getFullyQualifiedName().', "'.$this->lang.'")', ApiHelper::getColumnMapName($columnMap));
213 213
                 }
214 214
             }
215 215
         }
@@ -223,22 +223,22 @@  discard block
 block discarded – undo
223 223
         $model->fromArray($data, ApiHelper::getFieldTypes());
224 224
         $tableMap = $this->getTableMap();
225 225
         try {
226
-            if($tableMap->hasRelation($tableMap->getPhpName() . 'I18n'))
226
+            if ($tableMap->hasRelation($tableMap->getPhpName().'I18n'))
227 227
             {
228
-                $relateI18n = $tableMap->getRelation($tableMap->getPhpName() . 'I18n');
228
+                $relateI18n = $tableMap->getRelation($tableMap->getPhpName().'I18n');
229 229
                 $i18NTableMap = $relateI18n->getLocalTable();
230 230
                 $model->setLocale($this->lang);
231
-                foreach($i18NTableMap->getColumns() as $columnMap) {
232
-                    $method = 'set' . $columnMap->getPhpName();
231
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
232
+                    $method = 'set'.$columnMap->getPhpName();
233 233
                     $dtoColumnName = ApiHelper::getColumnMapName($columnMap);
234
-                    if(array_key_exists($dtoColumnName, $data)
234
+                    if (array_key_exists($dtoColumnName, $data)
235 235
                         && method_exists($model, $method)
236 236
                         && !($columnMap->isPrimaryKey() && $columnMap->isForeignKey())) {
237 237
                         $model->$method($data[$dtoColumnName]);
238 238
                     }
239 239
                 }
240 240
             }
241
-        } catch(\Exception $e) {
241
+        }catch (\Exception $e) {
242 242
             Logger::log($e->getMessage(), LOG_DEBUG);
243 243
         }
244 244
     }
Please login to merge, or discard this patch.
src/base/types/traits/Helper/FieldMapperHelperTrait.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -22,15 +22,15 @@  discard block
 block discarded – undo
22 22
      */
23 23
     private static function mapResult(ActiveRecordInterface $model, array $data = []) {
24 24
         $result = [];
25
-        foreach($data as $key => $value) {
25
+        foreach ($data as $key => $value) {
26 26
             try {
27 27
                 $realValue = $model->getByName($key);
28
-            } catch(\Exception $e) {
28
+            }catch (\Exception $e) {
29 29
                 $realValue = $value;
30 30
             }
31
-            if(Api::API_MODEL_KEY_FIELD === $key) {
31
+            if (Api::API_MODEL_KEY_FIELD === $key) {
32 32
                 $result[$key] = (integer)$realValue;
33
-            } else {
33
+            }else {
34 34
                 $result[$key] = $realValue;
35 35
             }
36 36
         }
@@ -55,14 +55,14 @@  discard block
 block discarded – undo
55 55
         $objTableMap = get_class($formatter->getTableMap());
56 56
         /** @var TableMapTrait $objTableMap */
57 57
         $objData = $data;
58
-        foreach($objTableMap::getFieldNames() as $field) {
59
-            if(!array_key_exists($field, $objData)) {
58
+        foreach ($objTableMap::getFieldNames() as $field) {
59
+            if (!array_key_exists($field, $objData)) {
60 60
                 $objData[$field] = null;
61 61
             }
62 62
         }
63 63
         $obj = @$formatter->getAllObjectsFromRow($objData);
64 64
         $result = self::mapResult($obj, $data);
65
-        if(!preg_match('/' . $modelPk->getPhpName() . '/i', $query[Api::API_FIELDS_RESULT_FIELD])) {
65
+        if (!preg_match('/'.$modelPk->getPhpName().'/i', $query[Api::API_FIELDS_RESULT_FIELD])) {
66 66
             unset($result[$modelPk->getPhpName()]);
67 67
         }
68 68
         return $result;
@@ -73,11 +73,11 @@  discard block
 block discarded – undo
73 73
      * @return string
74 74
      */
75 75
     public static function getColumnMapName(ColumnMap $field) {
76
-        switch(self::getFieldTypes()) {
76
+        switch (self::getFieldTypes()) {
77 77
             default:
78 78
             case 'UpperCamelCase':
79 79
             case TableMap::TYPE_PHPNAME:
80
-                $columnName =$field->getPhpName();
80
+                $columnName = $field->getPhpName();
81 81
                 break;
82 82
             case 'camelCase':
83 83
             case 'lowerCamelCase':
Please login to merge, or discard this patch.
src/base/types/traits/Helper/FieldModelHelperTrait.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         $fDto->entity = $relatedModel;
32 32
         $relatedField = $foreignTable->getColumn($mappedColumn->getRelatedColumnName());
33 33
         $fDto->relatedField = $relatedField->getPhpName();
34
-        $fDto->url = Router::getInstance()->getRoute(strtolower($domain) . '-api-' . $relatedModel);
34
+        $fDto->url = Router::getInstance()->getRoute(strtolower($domain).'-api-'.$relatedModel);
35 35
         return $fDto;
36 36
     }
37 37
 
@@ -43,18 +43,18 @@  discard block
 block discarded – undo
43 43
     private static function addQueryFilter(ColumnMap $column, ModelCriteria &$query, $value = null)
44 44
     {
45 45
         $tableField = $column->getFullyQualifiedName();
46
-        if(is_array($value)) {
46
+        if (is_array($value)) {
47 47
             $query->add($tableField, $value, Criteria::IN);
48 48
         } elseif (preg_match('/^\[/', $value) && preg_match('/\]$/', $value)) {
49 49
             $query->add($tableField, explode(',', preg_replace('/(\[|\])/', '', $value)), Criteria::IN);
50 50
         } elseif (preg_match('/^(\'|\")(.*)(\'|\")$/', $value)) {
51 51
             $text = preg_replace('/(\'|\")/', '', $value);
52 52
             $text = preg_replace('/\ /', '%', $text);
53
-            $query->add($tableField, '%' . $text . '%', Criteria::LIKE);
54
-        } else {
55
-            if(null !== $column->getValueSet()) {
53
+            $query->add($tableField, '%'.$text.'%', Criteria::LIKE);
54
+        }else {
55
+            if (null !== $column->getValueSet()) {
56 56
                 $valueSet = $column->getValueSet();
57
-                if(in_array($value, $valueSet)) {
57
+                if (in_array($value, $valueSet)) {
58 58
                     $value = array_search($value, $valueSet);
59 59
                 }
60 60
             }
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
      */
85 85
     public static function extractQuery($modelNameNamespace, ConnectionInterface $con = null)
86 86
     {
87
-        $queryReflector = new \ReflectionClass($modelNameNamespace . "Query");
87
+        $queryReflector = new \ReflectionClass($modelNameNamespace."Query");
88 88
         /** @var \Propel\Runtime\ActiveQuery\ModelCriteria $query */
89 89
         $query = $queryReflector->getMethod('create')->invoke($con);
90 90
 
@@ -103,31 +103,31 @@  discard block
 block discarded – undo
103 103
         $sep = '';
104 104
         foreach ($tableMap->getColumns() as $column) {
105 105
             if ($column->isText()) {
106
-                $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
106
+                $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
107 107
                 $sep = ', " ", ';
108 108
             }
109 109
         }
110
-        foreach($tableMap->getRelations() as $relation) {
111
-            if(preg_match('/I18n$/i', $relation->getName())) {
110
+        foreach ($tableMap->getRelations() as $relation) {
111
+            if (preg_match('/I18n$/i', $relation->getName())) {
112 112
                 $localeTableMap = $relation->getLocalTable();
113 113
                 foreach ($localeTableMap->getColumns() as $column) {
114 114
                     if ($column->isText()) {
115
-                        $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
115
+                        $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
116 116
                         $sep = ', " ", ';
117 117
                     }
118 118
                 }
119 119
             }
120 120
         }
121 121
         foreach (array_keys($extraColumns) as $extra) {
122
-            if(!preg_match("/(COUNT|DISTINCT|SUM|MAX|MIN|GROUP)/i", $extra)) {
123
-                $exp .= $sep . $extra;
122
+            if (!preg_match("/(COUNT|DISTINCT|SUM|MAX|MIN|GROUP)/i", $extra)) {
123
+                $exp .= $sep.$extra;
124 124
                 $sep = ', " ", ';
125 125
             }
126 126
         }
127 127
         $exp .= ")";
128 128
         $text = preg_replace('/(\'|\")/', '', $value);
129 129
         $text = preg_replace('/\ /', '%', $text);
130
-        $query->where($exp . Criteria::LIKE . '"%' . $text . '%"');
130
+        $query->where($exp.Criteria::LIKE.'"%'.$text.'%"');
131 131
     }
132 132
 
133 133
     /**
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
      */
137 137
     public static function extractPrimaryKeyColumnName(TableMap $tableMap) {
138 138
         $modelPk = null;
139
-        foreach($tableMap->getPrimaryKeys() as $pk) {
139
+        foreach ($tableMap->getPrimaryKeys() as $pk) {
140 140
             $modelPk = $pk;
141 141
             break;
142 142
         }
Please login to merge, or discard this patch.
src/base/types/traits/Helper/FieldHelperTrait.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -122,20 +122,20 @@  discard block
 block discarded – undo
122 122
         $behaviors = $tableMap->getBehaviors();
123 123
         foreach ($map::getFieldNames() as $field) {
124 124
             $fDto = self::parseFormField($domain, $tableMap, $field, $behaviors);
125
-            if(null !== $fDto) {
125
+            if (null !== $fDto) {
126 126
                 $form->addField($fDto);
127 127
             }
128 128
         }
129 129
 
130
-        if(array_key_exists('i18n', $behaviors)) {
131
-            $relateI18n = $tableMap->getRelation($tableMap->getPhpName() . 'I18n');
132
-            if(null !== $relateI18n) {
130
+        if (array_key_exists('i18n', $behaviors)) {
131
+            $relateI18n = $tableMap->getRelation($tableMap->getPhpName().'I18n');
132
+            if (null !== $relateI18n) {
133 133
                 $i18NTableMap = $relateI18n->getLocalTable();
134
-                foreach($i18NTableMap->getColumns() as $columnMap) {
134
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
135 135
                     $columnName = self::getColumnMapName($columnMap);
136
-                    if(!$form->fieldExists($columnName)) {
136
+                    if (!$form->fieldExists($columnName)) {
137 137
                         $fDto = self::parseFormField($domain, $i18NTableMap, $columnMap->getPhpName(), $i18NTableMap->getBehaviors());
138
-                        if(null !== $fDto) {
138
+                        if (null !== $fDto) {
139 139
                             $fDto->pk = false;
140 140
                             $fDto->required = true;
141 141
                             $form->addField($fDto);
@@ -158,14 +158,14 @@  discard block
 block discarded – undo
158 158
     {
159 159
         $column = null;
160 160
         try {
161
-            foreach($tableMap->getColumns() as $tableMapColumn) {
161
+            foreach ($tableMap->getColumns() as $tableMapColumn) {
162 162
                 $columnName = $tableMapColumn->getPhpName();
163
-                if(preg_match('/^'.$field.'$/i', $columnName)) {
163
+                if (preg_match('/^'.$field.'$/i', $columnName)) {
164 164
                     $column = $tableMapColumn;
165 165
                     break;
166 166
                 }
167 167
             }
168
-        } catch (\Exception $e) {
168
+        }catch (\Exception $e) {
169 169
             Logger::log($e->getMessage(), LOG_DEBUG);
170 170
         }
171 171
         return $column;
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
176 176
      */
177 177
     public static function getFieldTypes() {
178 178
         $configType = Config::getParam('api.field.case', TableMap::TYPE_PHPNAME);
179
-        switch($configType) {
179
+        switch ($configType) {
180 180
             default:
181 181
             case 'UpperCamelCase':
182 182
             case TableMap::TYPE_PHPNAME:
Please login to merge, or discard this patch.
src/base/dto/Form.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -42,10 +42,10 @@  discard block
 block discarded – undo
42 42
             $array['fields'][] = $field->__toArray();
43 43
         }
44 44
         usort($array['fields'], function($fieldA, $fieldB) {
45
-            if((bool)$fieldA['required'] !== (bool)$fieldB['required']) {
46
-                if((bool)$fieldA['required']) {
45
+            if ((bool)$fieldA['required'] !== (bool)$fieldB['required']) {
46
+                if ((bool)$fieldA['required']) {
47 47
                     return -1;
48
-                } else {
48
+                }else {
49 49
                     return 1;
50 50
                 }
51 51
             }
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
             }
57 57
             return ($aOrder < $bOrder) ? -1 : 1;
58 58
         });
59
-        foreach($this->actions as $action) {
59
+        foreach ($this->actions as $action) {
60 60
             $array['actions'][] = $action->__toArray();
61 61
         }
62 62
         return $array;
Please login to merge, or discard this patch.
src/base/extension/AssetsTokenParser.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@
 block discarded – undo
141 141
         $tmp = [];
142 142
         if (null === $node) {
143 143
             $node = $value;
144
-        } else {
144
+        }else {
145 145
             $tmp = $this->getTmpAttribute($node);
146 146
         }
147 147
         if (null !== $node) {
Please login to merge, or discard this patch.
src/base/types/helpers/DeployHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@
 block discarded – undo
17 17
     public static function updateCacheVar() {
18 18
         $now = new \DateTime(null, new \DateTimeZone(Config::getParam('project.timezone', 'Europe/Madrid')));
19 19
         $config = Config::getInstance()->dumpConfig();
20
-        $config[self::CACHE_VAR_TAG] = 'v' . $now->format('Ymd.His');
20
+        $config[self::CACHE_VAR_TAG] = 'v'.$now->format('Ymd.His');
21 21
         Config::save($config);
22 22
         return $config[self::CACHE_VAR_TAG];
23 23
     }
Please login to merge, or discard this patch.
src/controller/GeneratorController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
      */
29 29
     public function generateModule()
30 30
     {
31
-        Logger::log("Arranque generador de módulos al solicitar " . $this->getRequest()->getRequestUri());
31
+        Logger::log("Arranque generador de módulos al solicitar ".$this->getRequest()->getRequestUri());
32 32
         /* @var $form \PSFS\base\config\ConfigForm */
33 33
         $form = new ModuleForm();
34 34
         $form->build();
@@ -60,8 +60,8 @@  discard block
 block discarded – undo
60 60
                 $this->gen->createStructureModule($module, false, $type, $apiClass);
61 61
                 Security::getInstance()->setFlash("callback_message", str_replace("%s", $module, t("Módulo %s generado correctamente")));
62 62
                 // Security::getInstance()->setFlash("callback_route", $this->getRoute("admin-module", true));
63
-            } catch (\Exception $e) {
64
-                Logger::log($e->getMessage() . " [" . $e->getFile() . ":" . $e->getLine() . "]");
63
+            }catch (\Exception $e) {
64
+                Logger::log($e->getMessage()." [".$e->getFile().":".$e->getLine()."]");
65 65
                 Security::getInstance()->setFlash("callback_message", htmlentities($e->getMessage()));
66 66
             }
67 67
         }
Please login to merge, or discard this patch.