Passed
Push — master ( 4e78e3...fbd0b5 )
by Fran
03:13
created
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
         }
@@ -161,15 +161,15 @@  discard block
 block discarded – undo
161 161
             $this->addPkToList();
162 162
         }
163 163
         if (!empty($this->extraColumns)) {
164
-            if(Config::getParam('api.extrafields.compat', true)) {
164
+            if (Config::getParam('api.extrafields.compat', true)) {
165 165
                 $fields = array_values($this->extraColumns);
166
-            } else {
166
+            }else {
167 167
                 $returnFields = Request::getInstance()->getQuery(Api::API_FIELDS_RESULT_FIELD);
168 168
                 $fields = explode(',', $returnFields ?: '');
169 169
                 $fields[] = self::API_MODEL_KEY_FIELD;
170 170
             }
171 171
             foreach ($this->extraColumns as $expression => $columnName) {
172
-                if(empty($fields) || in_array($columnName, $fields)) {
172
+                if (empty($fields) || in_array($columnName, $fields)) {
173 173
                     $query->withColumn($expression, $columnName);
174 174
                 }
175 175
             }
@@ -201,17 +201,17 @@  discard block
 block discarded – undo
201 201
     {
202 202
         $this->extractApiLang();
203 203
         $model = $this->getModelNamespace();
204
-        $modelI18n = $model . 'I18n';
204
+        $modelI18n = $model.'I18n';
205 205
         if (method_exists($query, 'useI18nQuery')) {
206 206
             $query->useI18nQuery($this->lang);
207
-            $modelI18nTableMapClass = str_replace('\\Models\\', '\\Models\\Map\\', $modelI18n) . 'TableMap';
207
+            $modelI18nTableMapClass = str_replace('\\Models\\', '\\Models\\Map\\', $modelI18n).'TableMap';
208 208
             /** @var TableMap $modelI18nTableMap */
209 209
             $modelI18nTableMap = $modelI18nTableMapClass::getTableMap();
210
-            foreach($modelI18nTableMap->getColumns() as $columnMap) {
211
-                if(!$columnMap->isPrimaryKey()) {
210
+            foreach ($modelI18nTableMap->getColumns() as $columnMap) {
211
+                if (!$columnMap->isPrimaryKey()) {
212 212
                     $query->withColumn($columnMap->getFullyQualifiedName(), ApiHelper::getColumnMapName($columnMap));
213
-                } elseif(!$columnMap->isForeignKey()) {
214
-                    $query->withColumn('IFNULL(' . $columnMap->getFullyQualifiedName() . ', "'.$this->lang.'")', ApiHelper::getColumnMapName($columnMap));
213
+                } elseif (!$columnMap->isForeignKey()) {
214
+                    $query->withColumn('IFNULL('.$columnMap->getFullyQualifiedName().', "'.$this->lang.'")', ApiHelper::getColumnMapName($columnMap));
215 215
                 }
216 216
             }
217 217
         }
@@ -225,21 +225,21 @@  discard block
 block discarded – undo
225 225
         $model->fromArray($data, ApiHelper::getFieldTypes());
226 226
         $tableMap = $this->getTableMap();
227 227
         try {
228
-            if($tableMap->hasRelation($tableMap->getPhpName() . 'I18n'))
228
+            if ($tableMap->hasRelation($tableMap->getPhpName().'I18n'))
229 229
             {
230
-                $relateI18n = $tableMap->getRelation($tableMap->getPhpName() . 'I18n');
230
+                $relateI18n = $tableMap->getRelation($tableMap->getPhpName().'I18n');
231 231
                 $i18NTableMap = $relateI18n->getLocalTable();
232
-                foreach($i18NTableMap->getColumns() as $columnMap) {
233
-                    $method = 'set' . $columnMap->getPhpName();
232
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
233
+                    $method = 'set'.$columnMap->getPhpName();
234 234
                     $dtoColumnName = ApiHelper::getColumnMapName($columnMap);
235
-                    if(array_key_exists($dtoColumnName, $data)
235
+                    if (array_key_exists($dtoColumnName, $data)
236 236
                         && method_exists($model, $method)
237 237
                         && !($columnMap->isPrimaryKey() && $columnMap->isForeignKey())) {
238 238
                         $model->$method($data[$dtoColumnName]);
239 239
                     }
240 240
                 }
241 241
             }
242
-        } catch(\Exception $e) {
242
+        }catch (\Exception $e) {
243 243
             Logger::log($e->getMessage(), LOG_DEBUG);
244 244
         }
245 245
     }
Please login to merge, or discard this patch.
src/base/types/traits/OutputTrait.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
                 $this->setStatusCode(Template::STATUS_OK);
122 122
                 break;
123 123
             default:
124
-                $this->setStatusCode('HTTP/1.0 ' .($status ?: 200));
124
+                $this->setStatusCode('HTTP/1.0 '.($status ?: 200));
125 125
                 break;
126 126
         }
127 127
         return $this;
@@ -135,11 +135,11 @@  discard block
 block discarded – undo
135 135
     private function setReponseHeaders($contentType = 'text/html', array $cookies = array())
136 136
     {
137 137
         $powered = Config::getParam('poweredBy', 'PSFS');
138
-        header('X-Powered-By: ' . $powered);
138
+        header('X-Powered-By: '.$powered);
139 139
         ResponseHelper::setStatusHeader($this->getStatusCode());
140 140
         ResponseHelper::setAuthHeaders($this->isPublicZone());
141 141
         ResponseHelper::setCookieHeaders($cookies);
142
-        header('Content-type: ' . $contentType);
142
+        header('Content-type: '.$contentType);
143 143
 
144 144
     }
145 145
 
@@ -152,20 +152,20 @@  discard block
 block discarded – undo
152 152
      */
153 153
     public function output($output = '', $contentType = 'text/html', array $cookies = array())
154 154
     {
155
-        if(!self::isTest()) {
155
+        if (!self::isTest()) {
156 156
             Logger::log('Start output response');
157 157
             ob_start();
158 158
             $this->setReponseHeaders($contentType, $cookies);
159
-            header('Content-length: ' . strlen($output));
160
-            header('CRC: ' . crc32($output));
159
+            header('Content-length: '.strlen($output));
160
+            header('CRC: '.crc32($output));
161 161
 
162 162
             $needCache = Cache::needCache();
163 163
             $cache = Cache::getInstance();
164 164
             list($path, $cacheDataName) = $cache->getRequestCacheHash();
165 165
             if (null !== $cacheDataName && false !== $needCache && $this->getStatusCode() === Template::STATUS_OK) {
166 166
                 Logger::log('Saving output response into cache');
167
-                $cache->storeData('json' . DIRECTORY_SEPARATOR . $path . $cacheDataName, $output);
168
-                $cache->storeData('json' . DIRECTORY_SEPARATOR . $path . $cacheDataName . '.headers', headers_list(), Cache::JSON);
167
+                $cache->storeData('json'.DIRECTORY_SEPARATOR.$path.$cacheDataName, $output);
168
+                $cache->storeData('json'.DIRECTORY_SEPARATOR.$path.$cacheDataName.'.headers', headers_list(), Cache::JSON);
169 169
             } elseif (Request::getInstance()->getMethod() !== 'GET') {
170 170
                 $cache->flushCache();
171 171
             }
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
             ob_end_clean();
176 176
             Logger::log('End output response');
177 177
             $this->closeRender();
178
-        } else {
178
+        }else {
179 179
             return $output;
180 180
         }
181 181
     }
@@ -188,11 +188,11 @@  discard block
 block discarded – undo
188 188
         Logger::log('Close template render');
189 189
         $uri = Request::requestUri();
190 190
         Security::getInstance()->setSessionKey('lastRequest', array(
191
-            'url' => Request::getInstance()->getRootUrl() . $uri,
191
+            'url' => Request::getInstance()->getRootUrl().$uri,
192 192
             'ts' => microtime(true),
193 193
         ));
194 194
         Security::getInstance()->updateSession();
195
-        Logger::log('End request: ' . $uri, LOG_INFO);
195
+        Logger::log('End request: '.$uri, LOG_INFO);
196 196
         exit;
197 197
     }
198 198
 
@@ -230,17 +230,17 @@  discard block
 block discarded – undo
230 230
         /////////////////////////////////////////////////////////////
231 231
         // Date in the past sets the value to already have been expired.
232 232
         header('Expires: Sat, 26 Jul 1997 05:00:00 GMT');
233
-        header('Last-Modified: ' . gmdate('D, d M Y H:i:s') . ' GMT');
233
+        header('Last-Modified: '.gmdate('D, d M Y H:i:s').' GMT');
234 234
         header('Cache-Control: no-store, no-cache, must-revalidate'); // HTTP/1.1
235 235
         header('Cache-Control: pre-check=0, post-check=0, max-age=0'); // HTTP/1.1
236 236
         header('Pragma: no-cache');
237 237
         header('Expires: 0');
238 238
         header('Content-Transfer-Encoding: binary');
239
-        header('Content-type: ' . $content);
240
-        header('Content-length: ' . strlen($data));
241
-        header('Content-Disposition: attachment; filename="' . $filename . '"');
239
+        header('Content-type: '.$content);
240
+        header('Content-length: '.strlen($data));
241
+        header('Content-Disposition: attachment; filename="'.$filename.'"');
242 242
         header('Access-Control-Expose-Headers: Filename');
243
-        header('Filename: ' . $filename);
243
+        header('Filename: '.$filename);
244 244
         echo $data;
245 245
         ob_flush();
246 246
         ob_end_clean();
Please login to merge, or discard this patch.
src/base/types/Api.php 1 patch
Spacing   +36 added lines, -36 removed lines patch added patch discarded remove patch
@@ -82,20 +82,20 @@  discard block
 block discarded – undo
82 82
     public function init()
83 83
     {
84 84
         parent::init();
85
-        Logger::log(static::class . ' init', LOG_DEBUG);
85
+        Logger::log(static::class.' init', LOG_DEBUG);
86 86
         $this->domain = $this->getDomain();
87 87
         $this->hydrateRequestData();
88 88
         $this->hydrateOrders();
89
-        if($this instanceof CustomApi === false) {
89
+        if ($this instanceof CustomApi === false) {
90 90
             $this->createConnection($this->getTableMap());
91 91
         }
92 92
         $this->checkFieldType();
93 93
         $this->setLoaded(true);
94
-        Logger::log(static::class . ' loaded', LOG_DEBUG);
94
+        Logger::log(static::class.' loaded', LOG_DEBUG);
95 95
     }
96 96
 
97 97
     private function checkActions($method) {
98
-        switch($method) {
98
+        switch ($method) {
99 99
             default:
100 100
             case 'modelList': $this->action = self::API_ACTION_LIST; break;
101 101
             case 'get': $this->action = self::API_ACTION_GET; break;
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
     protected function hydrateOrders()
113 113
     {
114 114
         if (count($this->query)) {
115
-            Logger::log(static::class . ' gathering query string', LOG_DEBUG);
115
+            Logger::log(static::class.' gathering query string', LOG_DEBUG);
116 116
             foreach ($this->query as $key => $value) {
117 117
                 if ($key === self::API_ORDER_FIELD && is_array($value)) {
118 118
                     foreach ($value as $field => $direction) {
@@ -129,10 +129,10 @@  discard block
 block discarded – undo
129 129
      */
130 130
     protected function extractPagination()
131 131
     {
132
-        Logger::log(static::class . ' extract pagination start', LOG_DEBUG);
132
+        Logger::log(static::class.' extract pagination start', LOG_DEBUG);
133 133
         $page = array_key_exists(self::API_PAGE_FIELD, $this->query) ? $this->query[self::API_PAGE_FIELD] : 1;
134 134
         $limit = array_key_exists(self::API_LIMIT_FIELD, $this->query) ? $this->query[self::API_LIMIT_FIELD] : 100;
135
-        Logger::log(static::class . ' extract pagination end', LOG_DEBUG);
135
+        Logger::log(static::class.' extract pagination end', LOG_DEBUG);
136 136
         return array($page, (int)$limit);
137 137
     }
138 138
 
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
      */
144 144
     private function addOrders(ModelCriteria &$query)
145 145
     {
146
-        Logger::log(static::class . ' extract orders start ', LOG_DEBUG);
146
+        Logger::log(static::class.' extract orders start ', LOG_DEBUG);
147 147
         $orderAdded = FALSE;
148 148
         $tableMap = $this->getTableMap();
149 149
         foreach ($this->order->getOrders() as $field => $direction) {
@@ -151,18 +151,18 @@  discard block
 block discarded – undo
151 151
                 $orderAdded = TRUE;
152 152
                 if ($direction === Order::ASC) {
153 153
                     $query->addAscendingOrderByColumn($column->getPhpName());
154
-                } else {
154
+                }else {
155 155
                     $query->addDescendingOrderByColumn($column->getPhpName());
156 156
                 }
157 157
             }
158 158
         }
159 159
         if (!$orderAdded) {
160 160
             $pks = $this->getPkDbName();
161
-            foreach(array_keys($pks) as $key) {
161
+            foreach (array_keys($pks) as $key) {
162 162
                 $query->addAscendingOrderByColumn($key);
163 163
             }
164 164
         }
165
-        Logger::log(static::class . ' extract orders end', LOG_DEBUG);
165
+        Logger::log(static::class.' extract orders end', LOG_DEBUG);
166 166
     }
167 167
 
168 168
     /**
@@ -177,7 +177,7 @@  discard block
 block discarded – undo
177 177
             foreach ($this->query as $field => $value) {
178 178
                 if (self::API_COMBO_FIELD === $field) {
179 179
                     ApiHelper::composerComboField($tableMap, $query, $this->extraColumns, $value);
180
-                } elseif(!preg_match('/^__/', $field)) {
180
+                } elseif (!preg_match('/^__/', $field)) {
181 181
                     ApiHelper::addModelField($tableMap, $query, $field, $value);
182 182
                 }
183 183
             }
@@ -198,10 +198,10 @@  discard block
 block discarded – undo
198 198
             list($page, $limit) = $this->extractPagination();
199 199
             if ($limit === -1) {
200 200
                 $this->list = $query->find($this->con);
201
-            } else {
201
+            }else {
202 202
                 $this->list = $query->paginate($page, $limit, $this->con);
203 203
             }
204
-        } catch (\Exception $e) {
204
+        }catch (\Exception $e) {
205 205
             Logger::log($e->getMessage(), LOG_ERR);
206 206
         }
207 207
     }
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
         $code = 200;
221 221
         list($return, $total, $pages) = $this->getList();
222 222
         $message = null;
223
-        if(!$total) {
223
+        if (!$total) {
224 224
             $message = i18n::_('No se han encontrado elementos para la búsqueda');
225 225
         }
226 226
 
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
246 246
         $pages = 1;
247 247
         $message = null;
248 248
         list($code, $return) = $this->getSingleResult($pk);
249
-        if($code !== 200) {
249
+        if ($code !== 200) {
250 250
             $message = i18n::_('No se ha encontrado el elemento solicitado');
251 251
         }
252 252
 
@@ -275,13 +275,13 @@  discard block
 block discarded – undo
275 275
                 $status = 200;
276 276
                 $saved = TRUE;
277 277
                 $model = $this->model->toArray($this->fieldType ?: TableMap::TYPE_PHPNAME, true, [], true);
278
-            } else {
278
+            }else {
279 279
                 $message = i18n::_('No se ha podido guardar el modelo seleccionado');
280 280
             }
281
-        } catch (\Exception $e) {
282
-            $message = i18n::_('Ha ocurrido un error intentando guardar el elemento: ') .'<br>'. $e->getMessage();
281
+        }catch (\Exception $e) {
282
+            $message = i18n::_('Ha ocurrido un error intentando guardar el elemento: ').'<br>'.$e->getMessage();
283 283
             $context = [];
284
-            if(null !== $e->getPrevious()) {
284
+            if (null !== $e->getPrevious()) {
285 285
                 $context[] = $e->getPrevious()->getMessage();
286 286
             }
287 287
             Logger::log($e->getMessage(), LOG_CRIT, $context);
@@ -317,18 +317,18 @@  discard block
 block discarded – undo
317 317
                     $updated = TRUE;
318 318
                     $status = 200;
319 319
                     $model = $this->model->toArray($this->fieldType ?: TableMap::TYPE_PHPNAME, true, [], true);
320
-                } else {
320
+                }else {
321 321
                     $message = i18n::_('Ha ocurrido un error intentando actualizar el elemento, por favor revisa los logs');
322 322
                 }
323
-            } catch (\Exception $e) {
323
+            }catch (\Exception $e) {
324 324
                 $message = i18n::_('Ha ocurrido un error intentando actualizar el elemento, por favor revisa los logs');
325 325
                 $context = [];
326
-                if(null !== $e->getPrevious()) {
326
+                if (null !== $e->getPrevious()) {
327 327
                     $context[] = $e->getPrevious()->getMessage();
328 328
                 }
329 329
                 Logger::log($e->getMessage(), LOG_CRIT, $context);
330 330
             }
331
-        } else {
331
+        }else {
332 332
             $message = i18n::_('No se ha encontrado el modelo al que se hace referencia para actualizar');
333 333
         }
334 334
 
@@ -356,15 +356,15 @@  discard block
 block discarded – undo
356 356
                 $this->con->beginTransaction();
357 357
                 $this->hydrateModel($pk);
358 358
                 if (NULL !== $this->model) {
359
-                    if(method_exists('clearAllReferences', $this->model)) {
359
+                    if (method_exists('clearAllReferences', $this->model)) {
360 360
                         $this->model->clearAllReferences(true);
361 361
                     }
362 362
                     $this->model->delete($this->con);
363 363
                     $deleted = TRUE;
364 364
                 }
365
-            } catch (\Exception $e) {
365
+            }catch (\Exception $e) {
366 366
                 $context = [];
367
-                if(null !== $e->getPrevious()) {
367
+                if (null !== $e->getPrevious()) {
368 368
                     $context[] = $e->getPrevious()->getMessage();
369 369
                 }
370 370
                 Logger::log($e->getMessage(), LOG_CRIT, $context);
@@ -393,7 +393,7 @@  discard block
 block discarded – undo
393 393
             $this->saveBulk();
394 394
             $saved = true;
395 395
             $status = 200;
396
-        } catch(\Exception $e) {
396
+        }catch (\Exception $e) {
397 397
             Logger::log($e->getMessage(), LOG_CRIT, $this->getRequest()->getData());
398 398
             $message = i18n::_('Bulk insert rolled back');
399 399
         }
@@ -415,7 +415,7 @@  discard block
 block discarded – undo
415 415
 
416 416
         /** @var CustomerTableMap $tableMap */
417 417
         $modelPk = ApiHelper::extractPrimaryKeyColumnName($this->getTableMap());
418
-        foreach($this->list->getData() as $data) {
418
+        foreach ($this->list->getData() as $data) {
419 419
             $return[] = ApiHelper::mapArrayObject($this->getModelNamespace(), $modelPk, $this->query, $data);
420 420
         }
421 421
         return $return;
@@ -432,23 +432,23 @@  discard block
 block discarded – undo
432 432
         try {
433 433
             $this->paginate();
434 434
             if (null !== $this->list) {
435
-                if(array_key_exists(self::API_FIELDS_RESULT_FIELD, $this->query) && Config::getParam('api.field.types')) {
435
+                if (array_key_exists(self::API_FIELDS_RESULT_FIELD, $this->query) && Config::getParam('api.field.types')) {
436 436
                     $return = $this->extractDataWithFormat();
437
-                } else {
437
+                }else {
438 438
                     $return = $this->list->toArray(null, false, $this->fieldType ?: TableMap::TYPE_PHPNAME, false);
439 439
                 }
440 440
                 $total = 0;
441 441
                 $pages = 0;
442
-                if($this->list instanceof PropelModelPager) {
442
+                if ($this->list instanceof PropelModelPager) {
443 443
                     $total = $this->list->getNbResults();
444 444
                     $pages = $this->list->getLastPage();
445
-                } elseif($this->list instanceof ArrayCollection) {
445
+                } elseif ($this->list instanceof ArrayCollection) {
446 446
                     $total = count($return);
447 447
                     $pages = 1;
448 448
                 }
449 449
             }
450
-        } catch (\Exception $e) {
451
-            Logger::log(get_class($this) . ': ' . $e->getMessage(), LOG_ERR);
450
+        }catch (\Exception $e) {
451
+            Logger::log(get_class($this).': '.$e->getMessage(), LOG_ERR);
452 452
         }
453 453
 
454 454
         return array($return, $total, $pages);
@@ -466,7 +466,7 @@  discard block
 block discarded – undo
466 466
         $return = array();
467 467
         if (NULL === $model || !method_exists($model, 'toArray')) {
468 468
             $code = 404;
469
-        } else {
469
+        }else {
470 470
             $return = $model->toArray($this->fieldType ?: TableMap::TYPE_PHPNAME, true, [], true);
471 471
         }
472 472
 
Please login to merge, or discard this patch.