Passed
Push — master ( dd9457...9bc0cf )
by Fran
03:07
created
src/base/types/helpers/I18nHelper.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
         $locale = Request::header('X-API-LANG', $default);
29 29
         if (empty($locale)) {
30 30
             $locale = Security::getInstance()->getSessionKey(self::PSFS_SESSION_LANGUAGE_KEY);
31
-            if(empty($locale) && array_key_exists('HTTP_ACCEPT_LANGUAGE', $_SERVER)) {
31
+            if (empty($locale) && array_key_exists('HTTP_ACCEPT_LANGUAGE', $_SERVER)) {
32 32
                 $BrowserLocales = explode(",", str_replace("-", "_", $_SERVER["HTTP_ACCEPT_LANGUAGE"])); // brosers use en-US, Linux uses en_US
33 33
                 for ($i = 0, $ct = count($BrowserLocales); $i < $ct; $i++) {
34 34
                     list($BrowserLocales[$i]) = explode(";", $BrowserLocales[$i]); //trick for "en;q=0.8"
@@ -44,8 +44,8 @@  discard block
 block discarded – undo
44 44
         // TODO check more en locales
45 45
         if (strtolower($locale) === 'en') {
46 46
             $locale = 'en_GB';
47
-        } else {
48
-            $locale = $locale . '_' . strtoupper($locale);
47
+        }else {
48
+            $locale = $locale.'_'.strtoupper($locale);
49 49
         }
50 50
         $default_locales = explode(',', Config::getParam('i18n.locales', ''));
51 51
         if (!in_array($locale, array_merge($default_locales, self::$langs))) {
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
         $translations = array();
67 67
         if (file_exists($absoluteTranslationFileName)) {
68 68
             @include($absoluteTranslationFileName);
69
-        } else {
69
+        }else {
70 70
             Cache::getInstance()->storeData($absoluteTranslationFileName, "<?php \$translations = array();\n", Cache::TEXT, TRUE);
71 71
         }
72 72
 
@@ -81,13 +81,13 @@  discard block
 block discarded – undo
81 81
     public static function setLocale($default = null)
82 82
     {
83 83
         $locale = self::extractLocale($default);
84
-        Logger::log('Set locale to project [' . $locale . ']');
84
+        Logger::log('Set locale to project ['.$locale.']');
85 85
         // Load translations
86
-        putenv("LC_ALL=" . $locale);
86
+        putenv("LC_ALL=".$locale);
87 87
         setlocale(LC_ALL, $locale);
88 88
         // Load the locale path
89
-        $locale_path = BASE_DIR . DIRECTORY_SEPARATOR . 'locale';
90
-        Logger::log('Set locale dir ' . $locale_path);
89
+        $locale_path = BASE_DIR.DIRECTORY_SEPARATOR.'locale';
90
+        Logger::log('Set locale dir '.$locale_path);
91 91
         GeneratorHelper::createDir($locale_path);
92 92
         bindtextdomain('translations', $locale_path);
93 93
         textdomain('translations');
@@ -123,9 +123,9 @@  discard block
 block discarded – undo
123 123
      */
124 124
     public static function checkI18Class($namespace) {
125 125
         $isI18n = false;
126
-        if(preg_match('/I18n$/i', $namespace)) {
126
+        if (preg_match('/I18n$/i', $namespace)) {
127 127
             $parentClass = preg_replace('/I18n$/i', '', $namespace);
128
-            if(Router::exists($parentClass)) {
128
+            if (Router::exists($parentClass)) {
129 129
                 $isI18n = true;
130 130
             }
131 131
         }
@@ -151,12 +151,12 @@  discard block
 block discarded – undo
151 151
             ['i', 'i', 'i', 'i', 'I', 'I', 'I', 'I'],
152 152
             ['o', 'o', 'o', 'o', 'O', 'O', 'O', 'O'],
153 153
             ['u', 'u', 'u', 'u', 'U', 'U', 'U', 'U'],
154
-            ['n', 'N', 'c', 'C',],
154
+            ['n', 'N', 'c', 'C', ],
155 155
         ];
156 156
 
157 157
         $text = filter_var($string, FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW);
158
-        for($i = 0, $total = count($from); $i < $total; $i++) {
159
-            $text = str_replace($from[$i],$to[$i], $text);
158
+        for ($i = 0, $total = count($from); $i < $total; $i++) {
159
+            $text = str_replace($from[$i], $to[$i], $text);
160 160
         }
161 161
 
162 162
         return $text;
Please login to merge, or discard this patch.
src/controller/ConfigController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         foreach ($domains as $domain => $routes) {
32 32
             $pDomain = str_replace('@', '', $domain);
33 33
             $pDomain = str_replace('/', '', $pDomain);
34
-            $response[] = strtolower($pDomain) . '.api.secret';
34
+            $response[] = strtolower($pDomain).'.api.secret';
35 35
         }
36 36
         return $this->json($response);
37 37
     }
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
      */
46 46
     public function config()
47 47
     {
48
-        Logger::log("Config loaded executed by " . $this->getRequest()->getRequestUri());
48
+        Logger::log("Config loaded executed by ".$this->getRequest()->getRequestUri());
49 49
         /* @var $form \PSFS\base\config\ConfigForm */
50 50
         $form = new ConfigForm(Router::getInstance()->getRoute('admin-config'), Config::$required, Config::$optional, Config::getInstance()->dumpConfig());
51 51
         $form->build();
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
                 }
83 83
                 Security::getInstance()->setFlash("callback_message", t("Configuración actualizada correctamente"));
84 84
                 Security::getInstance()->setFlash("callback_route", $this->getRoute("admin-config", true));
85
-            } else {
85
+            }else {
86 86
                 throw new \HttpException(t('Error al guardar la configuración, prueba a cambiar los permisos'), 403);
87 87
             }
88 88
         }
Please login to merge, or discard this patch.
src/base/types/CustomAuthApi.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@
 block discarded – undo
12 12
 
13 13
     public function init()
14 14
     {
15
-        if(!Security::getInstance()->isLogged()) {
15
+        if (!Security::getInstance()->isLogged()) {
16 16
             throw new ApiException(t('Resource not authorized'), 401);
17 17
         }
18 18
         parent::init();
Please login to merge, or discard this patch.
src/base/types/traits/Api/ConnectionTrait.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     {
33 33
         $this->con = Propel::getConnection($tableMap::DATABASE_NAME);
34 34
         $this->con->beginTransaction();
35
-        if(method_exists($this->con, 'useDebug')) {
35
+        if (method_exists($this->con, 'useDebug')) {
36 36
             Logger::log('Enabling debug queries mode', LOG_INFO);
37 37
             $this->con->useDebug(Config::getParam('debug'));
38 38
         }
@@ -45,12 +45,12 @@  discard block
 block discarded – undo
45 45
      */
46 46
     protected function closeTransaction($status)
47 47
     {
48
-        if(null !== $this->con) {
48
+        if (null !== $this->con) {
49 49
             $this->traceDebugQuery();
50 50
             if (null !== $this->con && $this->con->inTransaction()) {
51 51
                 if ($status === 200) {
52 52
                     $this->con->commit();
53
-                } else {
53
+                }else {
54 54
                     $this->con->rollBack();
55 55
                 }
56 56
             }
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
     protected function traceDebugQuery()
65 65
     {
66 66
         if (Config::getParam('debug')) {
67
-            Logger::log($this->con->getLastExecutedQuery()?: 'Empty Query', LOG_DEBUG);
67
+            Logger::log($this->con->getLastExecutedQuery() ?: 'Empty Query', LOG_DEBUG);
68 68
         }
69 69
     }
70 70
 
@@ -72,13 +72,13 @@  discard block
 block discarded – undo
72 72
      * Checks if the connection has a transaction initialized
73 73
      */
74 74
     protected function checkTransaction() {
75
-        if(null !== $this->con && !$this->con->inTransaction()) {
75
+        if (null !== $this->con && !$this->con->inTransaction()) {
76 76
             $this->con->beginTransaction();
77 77
         }
78
-        if(null !== $this->con && $this->con->inTransaction()) {
78
+        if (null !== $this->con && $this->con->inTransaction()) {
79 79
             $this->items++;
80 80
         }
81
-        if($this->items >= Config::getParam('api.block.limit', 1000)) {
81
+        if ($this->items >= Config::getParam('api.block.limit', 1000)) {
82 82
             $this->con->commit();
83 83
             $this->items = 0;
84 84
         }
Please login to merge, or discard this patch.
src/services/GeneratorService.php 1 patch
Spacing   +60 added lines, -60 removed lines patch added patch discarded remove patch
@@ -45,24 +45,24 @@  discard block
 block discarded – undo
45 45
      */
46 46
     public static function findTranslations($path, $locale)
47 47
     {
48
-        $locale_path = realpath(BASE_DIR . DIRECTORY_SEPARATOR . 'locale');
49
-        $locale_path .= DIRECTORY_SEPARATOR . $locale . DIRECTORY_SEPARATOR . 'LC_MESSAGES' . DIRECTORY_SEPARATOR;
48
+        $locale_path = realpath(BASE_DIR.DIRECTORY_SEPARATOR.'locale');
49
+        $locale_path .= DIRECTORY_SEPARATOR.$locale.DIRECTORY_SEPARATOR.'LC_MESSAGES'.DIRECTORY_SEPARATOR;
50 50
 
51 51
         $translations = array();
52 52
         if (file_exists($path)) {
53 53
             $d = dir($path);
54 54
             while (false !== ($dir = $d->read())) {
55 55
                 GeneratorHelper::createDir($locale_path);
56
-                if (!file_exists($locale_path . 'translations.po')) {
57
-                    file_put_contents($locale_path . 'translations.po', '');
56
+                if (!file_exists($locale_path.'translations.po')) {
57
+                    file_put_contents($locale_path.'translations.po', '');
58 58
                 }
59
-                $inspect_path = realpath($path . DIRECTORY_SEPARATOR . $dir);
60
-                $cmd_php = "export PATH=\$PATH:/opt/local/bin; xgettext " .
61
-                    $inspect_path . DIRECTORY_SEPARATOR .
59
+                $inspect_path = realpath($path.DIRECTORY_SEPARATOR.$dir);
60
+                $cmd_php = "export PATH=\$PATH:/opt/local/bin; xgettext ".
61
+                    $inspect_path.DIRECTORY_SEPARATOR.
62 62
                     "*.php --from-code=UTF-8 -j -L PHP --debug --force-po -o {$locale_path}translations.po";
63
-                if (is_dir($path . DIRECTORY_SEPARATOR . $dir) && preg_match('/^\./', $dir) == 0) {
64
-                    $res = _('Revisando directorio: ') . $inspect_path;
65
-                    $res .= _('Comando ejecutado: ') . $cmd_php;
63
+                if (is_dir($path.DIRECTORY_SEPARATOR.$dir) && preg_match('/^\./', $dir) == 0) {
64
+                    $res = _('Revisando directorio: ').$inspect_path;
65
+                    $res .= _('Comando ejecutado: ').$cmd_php;
66 66
                     $res .= shell_exec($cmd_php);
67 67
                     usleep(10);
68 68
                     $translations[] = $res;
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
      */
84 84
     public function createStructureModule($module, $force = false, $type = "", $apiClass = "")
85 85
     {
86
-        $mod_path = CORE_DIR . DIRECTORY_SEPARATOR;
86
+        $mod_path = CORE_DIR.DIRECTORY_SEPARATOR;
87 87
         $module = ucfirst($module);
88 88
         $this->createModulePath($module, $mod_path);
89 89
         $this->createModulePathTree($module, $mod_path);
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
         // Creates the src folder
105 105
         GeneratorHelper::createDir($mod_path);
106 106
         // Create module path
107
-        GeneratorHelper::createDir($mod_path . $module);
107
+        GeneratorHelper::createDir($mod_path.$module);
108 108
     }
109 109
 
110 110
     /**
@@ -119,16 +119,16 @@  discard block
 block discarded – undo
119 119
         $this->log->addLog("Generamos la estructura");
120 120
         $paths = [
121 121
             "Api", "Api/base", "Config", "Controller", "Models", "Public", "Templates", "Services", "Test", "Doc",
122
-            "Locale", "Locale/" . Config::getParam('default.locale', 'es_ES'), "Locale/" . Config::getParam('default.locale', 'es_ES') . "/LC_MESSAGES"
122
+            "Locale", "Locale/".Config::getParam('default.locale', 'es_ES'), "Locale/".Config::getParam('default.locale', 'es_ES')."/LC_MESSAGES"
123 123
         ];
124
-        $module_path = $mod_path . $module;
124
+        $module_path = $mod_path.$module;
125 125
         foreach ($paths as $path) {
126
-            GeneratorHelper::createDir($module_path . DIRECTORY_SEPARATOR . $path);
126
+            GeneratorHelper::createDir($module_path.DIRECTORY_SEPARATOR.$path);
127 127
         }
128 128
         //Creamos las carpetas de los assets
129 129
         $htmlPaths = array("css", "js", "img", "media", "font");
130 130
         foreach ($htmlPaths as $path) {
131
-            GeneratorHelper::createDir($module_path . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . $path);
131
+            GeneratorHelper::createDir($module_path.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR.$path);
132 132
         }
133 133
     }
134 134
 
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
      */
142 142
     private function createModuleBaseFiles($module, $mod_path, $force = false, $controllerType = '')
143 143
     {
144
-        $module_path = $mod_path . $module;
144
+        $module_path = $mod_path.$module;
145 145
         $this->generateControllerTemplate($module, $module_path, $force, $controllerType);
146 146
         $this->generateServiceTemplate($module, $module_path, $force);
147 147
         $this->genereateAutoloaderTemplate($module, $module_path, $force);
@@ -159,8 +159,8 @@  discard block
 block discarded – undo
159 159
      */
160 160
     private function createModuleModels($module, $path)
161 161
     {
162
-        $module_path = $path . $module;
163
-        $module_path = str_replace(CORE_DIR . DIRECTORY_SEPARATOR, '', $module_path);
162
+        $module_path = $path.$module;
163
+        $module_path = str_replace(CORE_DIR.DIRECTORY_SEPARATOR, '', $module_path);
164 164
 
165 165
         $configGenerator = $this->getConfigGenerator($module_path);
166 166
 
@@ -170,8 +170,8 @@  discard block
 block discarded – undo
170 170
         $configTemplate = $this->tpl->dump("generator/config.propel.template.twig", array(
171 171
             "module" => $module,
172 172
         ));
173
-        $this->writeTemplateToFile($configTemplate, CORE_DIR . DIRECTORY_SEPARATOR . $module_path . DIRECTORY_SEPARATOR . "Config" .
174
-            DIRECTORY_SEPARATOR . "config.php", true);
173
+        $this->writeTemplateToFile($configTemplate, CORE_DIR.DIRECTORY_SEPARATOR.$module_path.DIRECTORY_SEPARATOR."Config".
174
+            DIRECTORY_SEPARATOR."config.php", true);
175 175
         $this->log->addLog("Generado config genérico para propel");
176 176
     }
177 177
 
@@ -193,28 +193,28 @@  discard block
 block discarded – undo
193 193
             "namespace" => preg_replace('/(\\\|\/)/', '\\', $module),
194 194
             "url" => preg_replace('/(\\\|\/)/', '/', $module),
195 195
             "class" => $class,
196
-            "controllerType" => $class . "Base",
196
+            "controllerType" => $class."Base",
197 197
             "is_base" => false
198 198
         ));
199
-        $controller = $this->writeTemplateToFile($controllerBody, $mod_path . DIRECTORY_SEPARATOR . "Controller" .
200
-            DIRECTORY_SEPARATOR . "{$class}Controller.php", $force);
199
+        $controller = $this->writeTemplateToFile($controllerBody, $mod_path.DIRECTORY_SEPARATOR."Controller".
200
+            DIRECTORY_SEPARATOR."{$class}Controller.php", $force);
201 201
 
202 202
         $controllerBody = $this->tpl->dump("generator/controller.template.twig", array(
203 203
             "module" => $module,
204 204
             "namespace" => preg_replace('/(\\\|\/)/', '\\', $module),
205 205
             "url" => preg_replace('/(\\\|\/)/', '/', $module),
206
-            "class" => $class . "Base",
206
+            "class" => $class."Base",
207 207
             "service" => $class,
208 208
             "controllerType" => $controllerType,
209 209
             "is_base" => true,
210 210
             "domain" => $class,
211 211
         ));
212
-        $controllerBase = $this->writeTemplateToFile($controllerBody, $mod_path . DIRECTORY_SEPARATOR . "Controller" .
213
-            DIRECTORY_SEPARATOR . "base" . DIRECTORY_SEPARATOR . "{$class}BaseController.php", true);
212
+        $controllerBase = $this->writeTemplateToFile($controllerBody, $mod_path.DIRECTORY_SEPARATOR."Controller".
213
+            DIRECTORY_SEPARATOR."base".DIRECTORY_SEPARATOR."{$class}BaseController.php", true);
214 214
 
215
-        $filename = $mod_path . DIRECTORY_SEPARATOR . "Test" . DIRECTORY_SEPARATOR . "{$class}Test.php";
215
+        $filename = $mod_path.DIRECTORY_SEPARATOR."Test".DIRECTORY_SEPARATOR."{$class}Test.php";
216 216
         $test = true;
217
-        if(!file_exists($filename)) {
217
+        if (!file_exists($filename)) {
218 218
             $testTemplate = $this->tpl->dump("generator/testCase.template.twig", array(
219 219
                 "module" => $module,
220 220
                 "namespace" => preg_replace('/(\\\|\/)/', '\\', $module),
@@ -235,8 +235,8 @@  discard block
 block discarded – undo
235 235
     private function generateBaseApiTemplate($module, $mod_path, $force = false, $apiClass = "")
236 236
     {
237 237
         $created = true;
238
-        $modelPath = $mod_path . $module . DIRECTORY_SEPARATOR . 'Models';
239
-        $api_path = $mod_path . $module . DIRECTORY_SEPARATOR . 'Api';
238
+        $modelPath = $mod_path.$module.DIRECTORY_SEPARATOR.'Models';
239
+        $api_path = $mod_path.$module.DIRECTORY_SEPARATOR.'Api';
240 240
         if (file_exists($modelPath)) {
241 241
             $dir = dir($modelPath);
242 242
             while ($file = $dir->read()) {
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
         //Generamos el fichero de configuración
265 265
         $this->log->addLog("Generamos fichero vacío de configuración");
266 266
         return $this->writeTemplateToFile("<?php\n\t",
267
-            $mod_path . DIRECTORY_SEPARATOR . "Config" . DIRECTORY_SEPARATOR . "config.php",
267
+            $mod_path.DIRECTORY_SEPARATOR."Config".DIRECTORY_SEPARATOR."config.php",
268 268
             $force);
269 269
     }
270 270
 
@@ -279,10 +279,10 @@  discard block
 block discarded – undo
279 279
         //Generamos el fichero de configuración
280 280
         $this->log->addLog("Generamos ficheros para assets base");
281 281
         $css = $this->writeTemplateToFile("/* CSS3 STYLES */\n\n",
282
-            $mod_path . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . "css" . DIRECTORY_SEPARATOR . "styles.css",
282
+            $mod_path.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR."css".DIRECTORY_SEPARATOR."styles.css",
283 283
             $force);
284 284
         $js = $this->writeTemplateToFile("/* APP MODULE JS */\n\n(function() {\n\t'use strict';\n})();",
285
-            $mod_path . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . "js" . DIRECTORY_SEPARATOR . "app.js",
285
+            $mod_path.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR."js".DIRECTORY_SEPARATOR."app.js",
286 286
             $force);
287 287
         return ($css && $js);
288 288
     }
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
             "class" => $class,
305 305
         ));
306 306
         return $this->writeTemplateToFile($controller,
307
-            $mod_path . DIRECTORY_SEPARATOR . "Services" . DIRECTORY_SEPARATOR . "{$class}Service.php",
307
+            $mod_path.DIRECTORY_SEPARATOR."Services".DIRECTORY_SEPARATOR."{$class}Service.php",
308 308
             $force);
309 309
     }
310 310
 
@@ -323,13 +323,13 @@  discard block
 block discarded – undo
323 323
             "autoloader" => preg_replace('/(\\\|\/)/', '_', $module),
324 324
             "regex" => preg_replace('/(\\\|\/)/m', '\\\\\\\\\\\\', $module),
325 325
         ));
326
-        $autoload = $this->writeTemplateToFile($autoloader, $mod_path . DIRECTORY_SEPARATOR . "autoload.php", $force);
326
+        $autoload = $this->writeTemplateToFile($autoloader, $mod_path.DIRECTORY_SEPARATOR."autoload.php", $force);
327 327
 
328 328
         $this->log->addLog("Generamos el phpunit");
329 329
         $phpUnitTemplate = $this->tpl->dump("generator/phpunit.template.twig", array(
330 330
             "module" => $module,
331 331
         ));
332
-        $phpunit = $this->writeTemplateToFile($phpUnitTemplate, $mod_path . DIRECTORY_SEPARATOR . "phpunit.xml.dist", $force);
332
+        $phpunit = $this->writeTemplateToFile($phpUnitTemplate, $mod_path.DIRECTORY_SEPARATOR."phpunit.xml.dist", $force);
333 333
         return $autoload && $phpunit;
334 334
     }
335 335
 
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
             "db" => $this->config->get("db_name"),
351 351
         ));
352 352
         return $this->writeTemplateToFile($schema,
353
-            $mod_path . DIRECTORY_SEPARATOR . "Config" . DIRECTORY_SEPARATOR . "schema.xml",
353
+            $mod_path.DIRECTORY_SEPARATOR."Config".DIRECTORY_SEPARATOR."schema.xml",
354 354
             $force);
355 355
     }
356 356
 
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
             "namespace" => preg_replace('/(\\\|\/)/', '', $module),
369 369
         ));
370 370
         return $this->writeTemplateToFile($build_properties,
371
-            $mod_path . DIRECTORY_SEPARATOR . "Config" . DIRECTORY_SEPARATOR . "propel.yml",
371
+            $mod_path.DIRECTORY_SEPARATOR."Config".DIRECTORY_SEPARATOR."propel.yml",
372 372
             $force);
373 373
     }
374 374
 
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
             "module" => $module,
387 387
         ));
388 388
         return $this->writeTemplateToFile($index,
389
-            $mod_path . DIRECTORY_SEPARATOR . "Templates" . DIRECTORY_SEPARATOR . "index.html.twig",
389
+            $mod_path.DIRECTORY_SEPARATOR."Templates".DIRECTORY_SEPARATOR."index.html.twig",
390 390
             $force);
391 391
     }
392 392
 
@@ -404,11 +404,11 @@  discard block
 block discarded – undo
404 404
             try {
405 405
                 $this->cache->storeData($filename, $fileContent, Cache::TEXT, true);
406 406
                 $created = true;
407
-            } catch (\Exception $e) {
407
+            }catch (\Exception $e) {
408 408
                 Logger::log($e->getMessage(), LOG_ERR);
409 409
             }
410
-        } else {
411
-            Logger::log($filename . _(' not exists or cant write'), LOG_ERR);
410
+        }else {
411
+            Logger::log($filename._(' not exists or cant write'), LOG_ERR);
412 412
         }
413 413
         return $created;
414 414
     }
@@ -437,7 +437,7 @@  discard block
 block discarded – undo
437 437
         ));
438 438
 
439 439
         return $this->writeTemplateToFile($controller,
440
-            $mod_path . DIRECTORY_SEPARATOR . 'base' . DIRECTORY_SEPARATOR . "{$api}BaseApi.php", true);
440
+            $mod_path.DIRECTORY_SEPARATOR.'base'.DIRECTORY_SEPARATOR."{$api}BaseApi.php", true);
441 441
     }
442 442
 
443 443
     /**
@@ -460,7 +460,7 @@  discard block
 block discarded – undo
460 460
             "class" => $class,
461 461
         ));
462 462
 
463
-        return $this->writeTemplateToFile($controller, $mod_path . DIRECTORY_SEPARATOR . "{$api}.php", $force);
463
+        return $this->writeTemplateToFile($controller, $mod_path.DIRECTORY_SEPARATOR."{$api}.php", $force);
464 464
     }
465 465
 
466 466
     /**
@@ -474,12 +474,12 @@  discard block
 block discarded – undo
474 474
     {
475 475
         if (file_exists($filename_path)) {
476 476
             $destfolder = basename($filename_path);
477
-            if (!file_exists(WEB_DIR . $dest . DIRECTORY_SEPARATOR . $destfolder) || $debug || $force) {
477
+            if (!file_exists(WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder) || $debug || $force) {
478 478
                 if (is_dir($filename_path)) {
479
-                    self::copyr($filename_path, WEB_DIR . $dest . DIRECTORY_SEPARATOR . $destfolder);
480
-                } else {
481
-                    if (@copy($filename_path, WEB_DIR . $dest . DIRECTORY_SEPARATOR . $destfolder) === FALSE) {
482
-                        throw new ConfigException("Can't copy " . $filename_path . " to " . WEB_DIR . $dest . DIRECTORY_SEPARATOR . $destfolder);
479
+                    self::copyr($filename_path, WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder);
480
+                }else {
481
+                    if (@copy($filename_path, WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder) === FALSE) {
482
+                        throw new ConfigException("Can't copy ".$filename_path." to ".WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder);
483 483
                     }
484 484
                 }
485 485
             }
@@ -498,10 +498,10 @@  discard block
 block discarded – undo
498 498
         GeneratorHelper::createDir($dst);
499 499
         while (false !== ($file = readdir($dir))) {
500 500
             if (($file != '.') && ($file != '..')) {
501
-                if (is_dir($src . '/' . $file)) {
502
-                    self::copyr($src . '/' . $file, $dst . '/' . $file);
503
-                } elseif (@copy($src . '/' . $file, $dst . '/' . $file) === false) {
504
-                    throw new ConfigException("Can't copy " . $src . " to " . $dst);
501
+                if (is_dir($src.'/'.$file)) {
502
+                    self::copyr($src.'/'.$file, $dst.'/'.$file);
503
+                } elseif (@copy($src.'/'.$file, $dst.'/'.$file) === false) {
504
+                    throw new ConfigException("Can't copy ".$src." to ".$dst);
505 505
                 }
506 506
             }
507 507
         }
@@ -514,12 +514,12 @@  discard block
 block discarded – undo
514 514
      */
515 515
     private function getPropelPaths($module_path)
516 516
     {
517
-        $moduleDir = CORE_DIR . DIRECTORY_SEPARATOR . $module_path;
517
+        $moduleDir = CORE_DIR.DIRECTORY_SEPARATOR.$module_path;
518 518
         GeneratorHelper::createDir($moduleDir);
519 519
         $moduleDir = realpath($moduleDir);
520
-        $configDir = $moduleDir . DIRECTORY_SEPARATOR . 'Config';
521
-        $sqlDir = $moduleDir . DIRECTORY_SEPARATOR . 'Config' . DIRECTORY_SEPARATOR . 'Sql';
522
-        $migrationDir = $moduleDir . DIRECTORY_SEPARATOR . 'Config' . DIRECTORY_SEPARATOR . 'Migrations';
520
+        $configDir = $moduleDir.DIRECTORY_SEPARATOR.'Config';
521
+        $sqlDir = $moduleDir.DIRECTORY_SEPARATOR.'Config'.DIRECTORY_SEPARATOR.'Sql';
522
+        $migrationDir = $moduleDir.DIRECTORY_SEPARATOR.'Config'.DIRECTORY_SEPARATOR.'Migrations';
523 523
         $paths = [
524 524
             'projectDir' => $moduleDir,
525 525
             'outputDir' => $moduleDir,
@@ -584,9 +584,9 @@  discard block
 block discarded – undo
584 584
     private function setupManager(GeneratorConfig $configGenerator, AbstractManager &$manager, $workingDir = CORE_DIR)
585 585
     {
586 586
         $manager->setGeneratorConfig($configGenerator);
587
-        $schemaFile = new \SplFileInfo($configGenerator->getSection('paths')['schemaDir'] . DIRECTORY_SEPARATOR . 'schema.xml');
587
+        $schemaFile = new \SplFileInfo($configGenerator->getSection('paths')['schemaDir'].DIRECTORY_SEPARATOR.'schema.xml');
588 588
         $manager->setSchemas([$schemaFile]);
589
-        $manager->setLoggerClosure(function ($message) {
589
+        $manager->setLoggerClosure(function($message) {
590 590
             Logger::log($message, LOG_INFO);
591 591
         });
592 592
         $manager->setWorkingDirectory($workingDir);
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
@@ -83,20 +83,20 @@  discard block
 block discarded – undo
83 83
     public function init()
84 84
     {
85 85
         parent::init();
86
-        Logger::log(static::class . ' init', LOG_DEBUG);
86
+        Logger::log(static::class.' init', LOG_DEBUG);
87 87
         $this->domain = $this->getDomain();
88 88
         $this->hydrateRequestData();
89 89
         $this->hydrateOrders();
90
-        if($this instanceof CustomApi === false) {
90
+        if ($this instanceof CustomApi === false) {
91 91
             $this->createConnection($this->getTableMap());
92 92
         }
93 93
         $this->checkFieldType();
94 94
         $this->setLoaded(true);
95
-        Logger::log(static::class . ' loaded', LOG_DEBUG);
95
+        Logger::log(static::class.' loaded', LOG_DEBUG);
96 96
     }
97 97
 
98 98
     private function checkActions($method) {
99
-        switch($method) {
99
+        switch ($method) {
100 100
             default:
101 101
             case 'modelList': $this->action = self::API_ACTION_LIST; break;
102 102
             case 'get': $this->action = self::API_ACTION_GET; break;
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
     protected function hydrateOrders()
114 114
     {
115 115
         if (count($this->query)) {
116
-            Logger::log(static::class . ' gathering query string', LOG_DEBUG);
116
+            Logger::log(static::class.' gathering query string', LOG_DEBUG);
117 117
             foreach ($this->query as $key => $value) {
118 118
                 if ($key === self::API_ORDER_FIELD && is_array($value)) {
119 119
                     foreach ($value as $field => $direction) {
@@ -130,10 +130,10 @@  discard block
 block discarded – undo
130 130
      */
131 131
     protected function extractPagination()
132 132
     {
133
-        Logger::log(static::class . ' extract pagination start', LOG_DEBUG);
133
+        Logger::log(static::class.' extract pagination start', LOG_DEBUG);
134 134
         $page = array_key_exists(self::API_PAGE_FIELD, $this->query) ? $this->query[self::API_PAGE_FIELD] : 1;
135 135
         $limit = array_key_exists(self::API_LIMIT_FIELD, $this->query) ? $this->query[self::API_LIMIT_FIELD] : 100;
136
-        Logger::log(static::class . ' extract pagination end', LOG_DEBUG);
136
+        Logger::log(static::class.' extract pagination end', LOG_DEBUG);
137 137
         return array($page, $limit);
138 138
     }
139 139
 
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
      */
145 145
     private function addOrders(ModelCriteria &$query)
146 146
     {
147
-        Logger::log(static::class . ' extract orders start ', LOG_DEBUG);
147
+        Logger::log(static::class.' extract orders start ', LOG_DEBUG);
148 148
         $orderAdded = FALSE;
149 149
         $tableMap = $this->getTableMap();
150 150
         foreach ($this->order->getOrders() as $field => $direction) {
@@ -152,17 +152,17 @@  discard block
 block discarded – undo
152 152
                 $orderAdded = TRUE;
153 153
                 if ($direction === Order::ASC) {
154 154
                     $query->addAscendingOrderByColumn($column->getPhpName());
155
-                } else {
155
+                }else {
156 156
                     $query->addDescendingOrderByColumn($column->getPhpName());
157 157
                 }
158 158
             }
159 159
         }
160 160
         if (!$orderAdded) {
161
-            foreach($this->getPkDbName() as $pk => $phpName) {
161
+            foreach ($this->getPkDbName() as $pk => $phpName) {
162 162
                 $query->addAscendingOrderByColumn($pk);
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);
@@ -394,7 +394,7 @@  discard block
 block discarded – undo
394 394
             $this->saveBulk();
395 395
             $saved = true;
396 396
             $status = 200;
397
-        } catch(\Exception $e) {
397
+        }catch (\Exception $e) {
398 398
             Logger::log($e->getMessage(), LOG_CRIT, $this->getRequest()->getData());
399 399
             $message = i18n::_('Bulk insert rolled back');
400 400
         }
@@ -416,7 +416,7 @@  discard block
 block discarded – undo
416 416
 
417 417
         /** @var CustomerTableMap $tableMap */
418 418
         $modelPk = ApiHelper::extractPrimaryKeyColumnName($this->getTableMap());
419
-        foreach($this->list->getData() as $data) {
419
+        foreach ($this->list->getData() as $data) {
420 420
             $return[] = ApiHelper::mapArrayObject($this->getModelNamespace(), $modelPk, $this->query, $data);
421 421
         }
422 422
         return $return;
@@ -433,23 +433,23 @@  discard block
 block discarded – undo
433 433
         try {
434 434
             $this->paginate();
435 435
             if (null !== $this->list) {
436
-                if(array_key_exists(self::API_FIELDS_RESULT_FIELD, $this->query) && Config::getParam('api.field.types')) {
436
+                if (array_key_exists(self::API_FIELDS_RESULT_FIELD, $this->query) && Config::getParam('api.field.types')) {
437 437
                     $return = $this->extractDataWithFormat();
438
-                } else {
438
+                }else {
439 439
                     $return = $this->list->toArray(null, false, $this->fieldType ?: TableMap::TYPE_PHPNAME, false);
440 440
                 }
441 441
                 $total = 0;
442 442
                 $pages = 0;
443
-                if($this->list instanceof PropelModelPager) {
443
+                if ($this->list instanceof PropelModelPager) {
444 444
                     $total = $this->list->getNbResults();
445 445
                     $pages = $this->list->getLastPage();
446
-                } elseif($this->list instanceof ArrayCollection) {
446
+                } elseif ($this->list instanceof ArrayCollection) {
447 447
                     $total = count($return);
448 448
                     $pages = 1;
449 449
                 }
450 450
             }
451
-        } catch (\Exception $e) {
452
-            Logger::log(get_class($this) . ': ' . $e->getMessage(), LOG_ERR);
451
+        }catch (\Exception $e) {
452
+            Logger::log(get_class($this).': '.$e->getMessage(), LOG_ERR);
453 453
         }
454 454
 
455 455
         return array($return, $total, $pages);
@@ -467,7 +467,7 @@  discard block
 block discarded – undo
467 467
         $return = array();
468 468
         if (NULL === $model || !method_exists($model, 'toArray')) {
469 469
             $code = 404;
470
-        } else {
470
+        }else {
471 471
             $return = $model->toArray($this->fieldType ?: TableMap::TYPE_PHPNAME, true, [], true);
472 472
         }
473 473
 
Please login to merge, or discard this patch.
src/base/types/helpers/ServiceHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
     }
16 16
 
17 17
     public static function parseRawData($type, array $params = []) {
18
-        switch($type) {
18
+        switch ($type) {
19 19
             default:
20 20
             case self::TYPE_HTTP:
21 21
                 $parsedParams = http_build_query($params);
Please login to merge, or discard this patch.
src/base/types/helpers/ApiHelper.php 1 patch
Spacing   +34 added lines, -34 removed lines patch added patch discarded remove patch
@@ -37,19 +37,19 @@  discard block
 block discarded – undo
37 37
         $behaviors = $tableMap->getBehaviors();
38 38
         foreach ($map::getFieldNames() as $field) {
39 39
             $fDto = self::parseFormField($domain, $tableMap, $field, $behaviors);
40
-            if(null !== $fDto) {
40
+            if (null !== $fDto) {
41 41
                 $form->addField($fDto);
42 42
             }
43 43
         }
44 44
 
45
-        if(array_key_exists('i18n', $behaviors)) {
46
-            $relateI18n = $tableMap->getRelation($tableMap->getPhpName() . 'I18n');
47
-            if(null !== $relateI18n) {
45
+        if (array_key_exists('i18n', $behaviors)) {
46
+            $relateI18n = $tableMap->getRelation($tableMap->getPhpName().'I18n');
47
+            if (null !== $relateI18n) {
48 48
                 $i18NTableMap = $relateI18n->getLocalTable();
49
-                foreach($i18NTableMap->getColumns() as $columnMap) {
50
-                    if(!$form->fieldExists($columnMap->getPhpName())) {
49
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
50
+                    if (!$form->fieldExists($columnMap->getPhpName())) {
51 51
                         $fDto = self::parseFormField($domain, $i18NTableMap, $columnMap->getPhpName(), $i18NTableMap->getBehaviors());
52
-                        if(null !== $fDto) {
52
+                        if (null !== $fDto) {
53 53
                             $fDto->pk = false;
54 54
                             $form->addField($fDto);
55 55
                         }
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
         $fDto->entity = $relatedModel;
79 79
         $relatedField = $foreignTable->getColumn($mappedColumn->getRelatedColumnName());
80 80
         $fDto->relatedField = $relatedField->getPhpName();
81
-        $fDto->url = Router::getInstance()->getRoute(strtolower($domain) . '-api-' . $relatedModel);
81
+        $fDto->url = Router::getInstance()->getRoute(strtolower($domain).'-api-'.$relatedModel);
82 82
         return $fDto;
83 83
     }
84 84
 
@@ -185,14 +185,14 @@  discard block
 block discarded – undo
185 185
     {
186 186
         $column = null;
187 187
         try {
188
-            foreach($tableMap->getColumns() as $tableMapColumn) {
188
+            foreach ($tableMap->getColumns() as $tableMapColumn) {
189 189
                 $columnName = $tableMapColumn->getPhpName();
190
-                if(preg_match('/'.$field.'/i', $columnName)) {
190
+                if (preg_match('/'.$field.'/i', $columnName)) {
191 191
                     $column = $tableMapColumn;
192 192
                     break;
193 193
                 }
194 194
             }
195
-        } catch (\Exception $e) {
195
+        }catch (\Exception $e) {
196 196
             Logger::log($e->getMessage(), LOG_DEBUG);
197 197
         }
198 198
         return $column;
@@ -206,18 +206,18 @@  discard block
 block discarded – undo
206 206
     private static function addQueryFilter(ColumnMap $column, ModelCriteria &$query, $value = null)
207 207
     {
208 208
         $tableField = $column->getFullyQualifiedName();
209
-        if(is_array($value)) {
209
+        if (is_array($value)) {
210 210
             $query->add($tableField, $value, Criteria::IN);
211 211
         } elseif (preg_match('/^\[/', $value) && preg_match('/\]$/', $value)) {
212 212
             $query->add($tableField, explode(',', preg_replace('/(\[|\])/', '', $value)), Criteria::IN);
213 213
         } elseif (preg_match('/^(\'|\")(.*)(\'|\")$/', $value)) {
214 214
             $text = preg_replace('/(\'|\")/', '', $value);
215 215
             $text = preg_replace('/\ /', '%', $text);
216
-            $query->add($tableField, '%' . $text . '%', Criteria::LIKE);
217
-        } else {
218
-            if(null !== $column->getValueSet()) {
216
+            $query->add($tableField, '%'.$text.'%', Criteria::LIKE);
217
+        }else {
218
+            if (null !== $column->getValueSet()) {
219 219
                 $valueSet = $column->getValueSet();
220
-                if(in_array($value, $valueSet)) {
220
+                if (in_array($value, $valueSet)) {
221 221
                     $value = array_search($value, $valueSet);
222 222
                 }
223 223
             }
@@ -237,31 +237,31 @@  discard block
 block discarded – undo
237 237
         $sep = '';
238 238
         foreach ($tableMap->getColumns() as $column) {
239 239
             if ($column->isText()) {
240
-                $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
240
+                $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
241 241
                 $sep = ', " ", ';
242 242
             }
243 243
         }
244
-        foreach($tableMap->getRelations() as $relation) {
245
-            if(preg_match('/I18n$/i', $relation->getName())) {
244
+        foreach ($tableMap->getRelations() as $relation) {
245
+            if (preg_match('/I18n$/i', $relation->getName())) {
246 246
                 $localeTableMap = $relation->getLocalTable();
247 247
                 foreach ($localeTableMap->getColumns() as $column) {
248 248
                     if ($column->isText()) {
249
-                        $exp .= $sep . 'IFNULL(' . $column->getFullyQualifiedName() . ',"")';
249
+                        $exp .= $sep.'IFNULL('.$column->getFullyQualifiedName().',"")';
250 250
                         $sep = ', " ", ';
251 251
                     }
252 252
                 }
253 253
             }
254 254
         }
255 255
         foreach ($extraColumns as $extra => $name) {
256
-            if(!preg_match("/(COUNT|DISTINCT|SUM|MAX|MIN|GROUP)/i", $extra)) {
257
-                $exp .= $sep . $extra;
256
+            if (!preg_match("/(COUNT|DISTINCT|SUM|MAX|MIN|GROUP)/i", $extra)) {
257
+                $exp .= $sep.$extra;
258 258
                 $sep = ', " ", ';
259 259
             }
260 260
         }
261 261
         $exp .= ")";
262 262
         $text = preg_replace('/(\'|\")/', '', $value);
263 263
         $text = preg_replace('/\ /', '%', $text);
264
-        $query->where($exp . Criteria::LIKE . '"%' . $text . '%"');
264
+        $query->where($exp.Criteria::LIKE.'"%'.$text.'%"');
265 265
     }
266 266
 
267 267
     /**
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
      */
287 287
     public static function extractQuery($modelNameNamespace, ConnectionInterface $con = null)
288 288
     {
289
-        $queryReflector = new \ReflectionClass($modelNameNamespace . "Query");
289
+        $queryReflector = new \ReflectionClass($modelNameNamespace."Query");
290 290
         /** @var \Propel\Runtime\ActiveQuery\ModelCriteria $query */
291 291
         $query = $queryReflector->getMethod('create')->invoke($con);
292 292
 
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
315 315
         } elseif ($mappedColumn->isText()) {
316 316
             if ($mappedColumn->getSize() > 100) {
317 317
                 $fDto = self::createField($field, Field::TEXTAREA_TYPE, $required);
318
-            } else {
318
+            }else {
319 319
                 $fDto = self::generateStringField($field, $required);
320 320
             }
321 321
         } elseif ($mappedColumn->getType() === PropelTypes::BOOLEAN) {
@@ -343,7 +343,7 @@  discard block
 block discarded – undo
343 343
                 $fDto->pk = true;
344 344
             }
345 345
         }
346
-        switch(Config::getParam('api.field.type', TableMap::TYPE_PHPNAME)) {
346
+        switch (Config::getParam('api.field.type', TableMap::TYPE_PHPNAME)) {
347 347
             default:
348 348
             case TableMap::TYPE_PHPNAME:
349 349
                 $fDto->name = $mappedColumn->getPhpName();
@@ -367,7 +367,7 @@  discard block
 block discarded – undo
367 367
      */
368 368
     public static function extractPrimaryKeyColumnName(TableMap $tableMap) {
369 369
         $modelPk = null;
370
-        foreach($tableMap->getPrimaryKeys() as $pk) {
370
+        foreach ($tableMap->getPrimaryKeys() as $pk) {
371 371
             $modelPk = $pk;
372 372
             break;
373 373
         }
@@ -376,15 +376,15 @@  discard block
 block discarded – undo
376 376
 
377 377
     private static function mapResult(ActiveRecordInterface $model, array $data = []) {
378 378
         $result = [];
379
-        foreach($data as $key => $value) {
379
+        foreach ($data as $key => $value) {
380 380
             try {
381 381
                 $realValue = $model->getByName($key);
382
-            } catch(\Exception $e) {
382
+            }catch (\Exception $e) {
383 383
                 $realValue = $value;
384 384
             }
385
-            if(Api::API_MODEL_KEY_FIELD === $key) {
385
+            if (Api::API_MODEL_KEY_FIELD === $key) {
386 386
                 $result[$key] = (integer)$realValue;
387
-            } else {
387
+            }else {
388 388
                 $result[$key] = $realValue;
389 389
             }
390 390
         }
@@ -401,14 +401,14 @@  discard block
 block discarded – undo
401 401
         $objTableMap = get_class($formatter->getTableMap());
402 402
         /** @var TableMapTrait $objTableMap */
403 403
         $objData = $data;
404
-        foreach($objTableMap::getFieldNames() as $field) {
405
-            if(!array_key_exists($field, $objData)) {
404
+        foreach ($objTableMap::getFieldNames() as $field) {
405
+            if (!array_key_exists($field, $objData)) {
406 406
                 $objData[$field] = null;
407 407
             }
408 408
         }
409 409
         $obj = @$formatter->getAllObjectsFromRow($objData);
410 410
         $result = self::mapResult($obj, $data);
411
-        if(!preg_match('/' . $modelPk->getPhpName() . '/i', $query[Api::API_FIELDS_RESULT_FIELD])) {
411
+        if (!preg_match('/'.$modelPk->getPhpName().'/i', $query[Api::API_FIELDS_RESULT_FIELD])) {
412 412
             unset($result[$modelPk->getPhpName()]);
413 413
         }
414 414
         return $result;
Please login to merge, or discard this patch.
src/base/types/traits/Api/MutationTrait.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -75,21 +75,21 @@  discard block
 block discarded – undo
75 75
         if (count($pks) == 1) {
76 76
             $pks = array_keys($pks);
77 77
             return [
78
-                $tableMap::TABLE_NAME . '.' . $pks[0] => Api::API_MODEL_KEY_FIELD
78
+                $tableMap::TABLE_NAME.'.'.$pks[0] => Api::API_MODEL_KEY_FIELD
79 79
             ];
80 80
         } elseif (count($pks) > 1) {
81 81
             $apiPks = [];
82 82
             $principal = '';
83 83
             $sep = 'CONCAT(';
84 84
             foreach ($pks as $pk) {
85
-                $apiPks[$tableMap::TABLE_NAME . '.' . $pk->getName()] = $pk->getPhpName();
86
-                $principal .= $sep . $tableMap::TABLE_NAME . '.' . $pk->getName();
87
-                $sep = ', "' . Api::API_PK_SEPARATOR . '", ';
85
+                $apiPks[$tableMap::TABLE_NAME.'.'.$pk->getName()] = $pk->getPhpName();
86
+                $principal .= $sep.$tableMap::TABLE_NAME.'.'.$pk->getName();
87
+                $sep = ', "'.Api::API_PK_SEPARATOR.'", ';
88 88
             }
89 89
             $principal .= ')';
90 90
             $apiPks[$principal] = Api::API_MODEL_KEY_FIELD;
91 91
             return $apiPks;
92
-        } else {
92
+        }else {
93 93
             throw new ApiException(_('El modelo de la API no está debidamente mapeado, no hay Primary Key o es compuesta'));
94 94
         }
95 95
     }
@@ -112,10 +112,10 @@  discard block
 block discarded – undo
112 112
         $pks = '';
113 113
         $sep = '';
114 114
         foreach ($tableMap->getPrimaryKeys() as $pk) {
115
-            $pks .= $sep . $pk->getFullyQualifiedName();
115
+            $pks .= $sep.$pk->getFullyQualifiedName();
116 116
             $sep = ', "|", ';
117 117
         }
118
-        $this->extraColumns['CONCAT("' . $tableMap->getPhpName() . ' #", ' . $pks . ')'] = Api::API_LIST_NAME_FIELD;
118
+        $this->extraColumns['CONCAT("'.$tableMap->getPhpName().' #", '.$pks.')'] = Api::API_LIST_NAME_FIELD;
119 119
     }
120 120
 
121 121
     /**
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
             }
138 138
             if (null !== $column) {
139 139
                 $this->extraColumns[$column->getFullyQualifiedName()] = Api::API_LIST_NAME_FIELD;
140
-            } else {
140
+            }else {
141 141
                 $this->addClassListName($tableMap);
142 142
             }
143 143
         }
@@ -156,15 +156,15 @@  discard block
 block discarded – undo
156 156
             $this->addPkToList();
157 157
         }
158 158
         if (!empty($this->extraColumns)) {
159
-            if(Config::getParam('api.extrafields.compat', true)) {
159
+            if (Config::getParam('api.extrafields.compat', true)) {
160 160
                 $fields = array_values($this->extraColumns);
161
-            } else {
161
+            }else {
162 162
                 $returnFields = Request::getInstance()->getQuery(Api::API_FIELDS_RESULT_FIELD);
163 163
                 $fields = explode(',', $returnFields ?: '');
164 164
                 $fields[] = self::API_MODEL_KEY_FIELD;
165 165
             }
166 166
             foreach ($this->extraColumns as $expression => $columnName) {
167
-                if(empty($fields) || in_array($columnName, $fields)) {
167
+                if (empty($fields) || in_array($columnName, $fields)) {
168 168
                     $query->withColumn($expression, $columnName);
169 169
                 }
170 170
             }
@@ -196,17 +196,17 @@  discard block
 block discarded – undo
196 196
     {
197 197
         $this->extractApiLang();
198 198
         $model = $this->getModelNamespace();
199
-        $modelI18n = $model . 'I18n';
199
+        $modelI18n = $model.'I18n';
200 200
         if (method_exists($query, 'useI18nQuery')) {
201 201
             $query->useI18nQuery($this->lang);
202
-            $modelI18nTableMapClass = str_replace('\\Models\\', '\\Models\\Map\\', $modelI18n) . 'TableMap';
202
+            $modelI18nTableMapClass = str_replace('\\Models\\', '\\Models\\Map\\', $modelI18n).'TableMap';
203 203
             /** @var TableMap $modelI18nTableMap */
204 204
             $modelI18nTableMap = $modelI18nTableMapClass::getTableMap();
205
-            foreach($modelI18nTableMap->getColumns() as $columnMap) {
206
-                if(!$columnMap->isPrimaryKey()) {
207
-                    $query->withColumn($modelI18nTableMapClass::TABLE_NAME . '.' . $columnMap->getName(), $columnMap->getPhpName());
208
-                } elseif(!$columnMap->isForeignKey()) {
209
-                    $query->withColumn('IFNULL(' . $modelI18nTableMapClass::TABLE_NAME . '.' . $columnMap->getName() . ', "'.$this->lang.'")', $columnMap->getPhpName());
205
+            foreach ($modelI18nTableMap->getColumns() as $columnMap) {
206
+                if (!$columnMap->isPrimaryKey()) {
207
+                    $query->withColumn($modelI18nTableMapClass::TABLE_NAME.'.'.$columnMap->getName(), $columnMap->getPhpName());
208
+                } elseif (!$columnMap->isForeignKey()) {
209
+                    $query->withColumn('IFNULL('.$modelI18nTableMapClass::TABLE_NAME.'.'.$columnMap->getName().', "'.$this->lang.'")', $columnMap->getPhpName());
210 210
                 }
211 211
             }
212 212
         }
@@ -220,20 +220,20 @@  discard block
 block discarded – undo
220 220
         $model->fromArray($data);
221 221
         $tableMap = $this->getTableMap();
222 222
         try {
223
-            if($tableMap->hasRelation($tableMap->getPhpName() . 'I18n'))
223
+            if ($tableMap->hasRelation($tableMap->getPhpName().'I18n'))
224 224
             {
225
-                $relateI18n = $tableMap->getRelation($tableMap->getPhpName() . 'I18n');
225
+                $relateI18n = $tableMap->getRelation($tableMap->getPhpName().'I18n');
226 226
                 $i18NTableMap = $relateI18n->getLocalTable();
227
-                foreach($i18NTableMap->getColumns() as $columnMap) {
228
-                    $method = 'set' . $columnMap->getPhpName();
229
-                    if(!($columnMap->isPrimaryKey() && $columnMap->isForeignKey())
227
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
228
+                    $method = 'set'.$columnMap->getPhpName();
229
+                    if (!($columnMap->isPrimaryKey() && $columnMap->isForeignKey())
230 230
                         &&array_key_exists($columnMap->getPhpName(), $data)
231 231
                         && method_exists($model, $method)) {
232 232
                         $model->$method($data[$columnMap->getPhpName()]);
233 233
                     }
234 234
                 }
235 235
             }
236
-        } catch(\Exception $e) {
236
+        }catch (\Exception $e) {
237 237
             Logger::log($e->getMessage(), LOG_DEBUG);
238 238
         }
239 239
     }
@@ -243,7 +243,7 @@  discard block
 block discarded – undo
243 243
      */
244 244
     protected function checkFieldType() {
245 245
         $configType = Config::getParam('api.field.type');
246
-        switch($configType) {
246
+        switch ($configType) {
247 247
             case 'UpperCamelCase':
248 248
             case TableMap::TYPE_PHPNAME:
249 249
                 $this->fieldType = TableMap::TYPE_PHPNAME;
Please login to merge, or discard this patch.