Passed
Push — master ( 9bc0cf...1710e3 )
by Fran
03:14
created
src/services/GeneratorService.php 1 patch
Spacing   +60 added lines, -60 removed lines patch added patch discarded remove patch
@@ -39,24 +39,24 @@  discard block
 block discarded – undo
39 39
      */
40 40
     public static function findTranslations($path, $locale)
41 41
     {
42
-        $localePath = realpath(BASE_DIR . DIRECTORY_SEPARATOR . 'locale');
43
-        $localePath .= DIRECTORY_SEPARATOR . $locale . DIRECTORY_SEPARATOR . 'LC_MESSAGES' . DIRECTORY_SEPARATOR;
42
+        $localePath = realpath(BASE_DIR.DIRECTORY_SEPARATOR.'locale');
43
+        $localePath .= DIRECTORY_SEPARATOR.$locale.DIRECTORY_SEPARATOR.'LC_MESSAGES'.DIRECTORY_SEPARATOR;
44 44
 
45 45
         $translations = array();
46 46
         if (file_exists($path)) {
47 47
             $d = dir($path);
48 48
             while (false !== ($dir = $d->read())) {
49 49
                 GeneratorHelper::createDir($localePath);
50
-                if (!file_exists($localePath . 'translations.po')) {
51
-                    file_put_contents($localePath . 'translations.po', '');
50
+                if (!file_exists($localePath.'translations.po')) {
51
+                    file_put_contents($localePath.'translations.po', '');
52 52
                 }
53
-                $inspect_path = realpath($path . DIRECTORY_SEPARATOR . $dir);
54
-                $cmd_php = "export PATH=\$PATH:/opt/local/bin; xgettext " .
55
-                    $inspect_path . DIRECTORY_SEPARATOR .
53
+                $inspect_path = realpath($path.DIRECTORY_SEPARATOR.$dir);
54
+                $cmd_php = "export PATH=\$PATH:/opt/local/bin; xgettext ".
55
+                    $inspect_path.DIRECTORY_SEPARATOR.
56 56
                     "*.php --from-code=UTF-8 -j -L PHP --debug --force-po -o {$localePath}translations.po";
57
-                if (is_dir($path . DIRECTORY_SEPARATOR . $dir) && preg_match('/^\./', $dir) == 0) {
58
-                    $res = _('Revisando directorio: ') . $inspect_path;
59
-                    $res .= _('Comando ejecutado: ') . $cmd_php;
57
+                if (is_dir($path.DIRECTORY_SEPARATOR.$dir) && preg_match('/^\./', $dir) == 0) {
58
+                    $res = _('Revisando directorio: ').$inspect_path;
59
+                    $res .= _('Comando ejecutado: ').$cmd_php;
60 60
                     $res .= shell_exec($cmd_php);
61 61
                     usleep(10);
62 62
                     $translations[] = $res;
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
      */
78 78
     public function createStructureModule($module, $force = false, $type = "", $apiClass = "")
79 79
     {
80
-        $mod_path = CORE_DIR . DIRECTORY_SEPARATOR;
80
+        $mod_path = CORE_DIR.DIRECTORY_SEPARATOR;
81 81
         $module = ucfirst($module);
82 82
         $this->createModulePath($module, $mod_path);
83 83
         $this->createModulePathTree($module, $mod_path);
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
         // Creates the src folder
99 99
         GeneratorHelper::createDir($mod_path);
100 100
         // Create module path
101
-        GeneratorHelper::createDir($mod_path . $module);
101
+        GeneratorHelper::createDir($mod_path.$module);
102 102
     }
103 103
 
104 104
     /**
@@ -113,16 +113,16 @@  discard block
 block discarded – undo
113 113
         $this->log->addLog("Generamos la estructura");
114 114
         $paths = [
115 115
             "Api", "Api/base", "Config", "Controller", "Models", "Public", "Templates", "Services", "Test", "Doc",
116
-            "Locale", "Locale/" . Config::getParam('default.locale', 'es_ES'), "Locale/" . Config::getParam('default.locale', 'es_ES') . "/LC_MESSAGES"
116
+            "Locale", "Locale/".Config::getParam('default.locale', 'es_ES'), "Locale/".Config::getParam('default.locale', 'es_ES')."/LC_MESSAGES"
117 117
         ];
118
-        $module_path = $mod_path . $module;
118
+        $module_path = $mod_path.$module;
119 119
         foreach ($paths as $path) {
120
-            GeneratorHelper::createDir($module_path . DIRECTORY_SEPARATOR . $path);
120
+            GeneratorHelper::createDir($module_path.DIRECTORY_SEPARATOR.$path);
121 121
         }
122 122
         //Creamos las carpetas de los assets
123 123
         $htmlPaths = array("css", "js", "img", "media", "font");
124 124
         foreach ($htmlPaths as $path) {
125
-            GeneratorHelper::createDir($module_path . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . $path);
125
+            GeneratorHelper::createDir($module_path.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR.$path);
126 126
         }
127 127
     }
128 128
 
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
      */
136 136
     private function createModuleBaseFiles($module, $mod_path, $force = false, $controllerType = '')
137 137
     {
138
-        $module_path = $mod_path . $module;
138
+        $module_path = $mod_path.$module;
139 139
         $this->generateControllerTemplate($module, $module_path, $force, $controllerType);
140 140
         $this->generateServiceTemplate($module, $module_path, $force);
141 141
         $this->genereateAutoloaderTemplate($module, $module_path, $force);
@@ -153,8 +153,8 @@  discard block
 block discarded – undo
153 153
      */
154 154
     private function createModuleModels($module, $path)
155 155
     {
156
-        $module_path = $path . $module;
157
-        $module_path = str_replace(CORE_DIR . DIRECTORY_SEPARATOR, '', $module_path);
156
+        $module_path = $path.$module;
157
+        $module_path = str_replace(CORE_DIR.DIRECTORY_SEPARATOR, '', $module_path);
158 158
 
159 159
         $configGenerator = $this->getConfigGenerator($module_path);
160 160
 
@@ -164,8 +164,8 @@  discard block
 block discarded – undo
164 164
         $configTemplate = $this->tpl->dump("generator/config.propel.template.twig", array(
165 165
             "module" => $module,
166 166
         ));
167
-        $this->writeTemplateToFile($configTemplate, CORE_DIR . DIRECTORY_SEPARATOR . $module_path . DIRECTORY_SEPARATOR . "Config" .
168
-            DIRECTORY_SEPARATOR . "config.php", true);
167
+        $this->writeTemplateToFile($configTemplate, CORE_DIR.DIRECTORY_SEPARATOR.$module_path.DIRECTORY_SEPARATOR."Config".
168
+            DIRECTORY_SEPARATOR."config.php", true);
169 169
         $this->log->addLog("Generado config genérico para propel");
170 170
     }
171 171
 
@@ -187,28 +187,28 @@  discard block
 block discarded – undo
187 187
             "namespace" => preg_replace('/(\\\|\/)/', '\\', $module),
188 188
             "url" => preg_replace('/(\\\|\/)/', '/', $module),
189 189
             "class" => $class,
190
-            "controllerType" => $class . "Base",
190
+            "controllerType" => $class."Base",
191 191
             "is_base" => false
192 192
         ));
193
-        $controller = $this->writeTemplateToFile($controllerBody, $mod_path . DIRECTORY_SEPARATOR . "Controller" .
194
-            DIRECTORY_SEPARATOR . "{$class}Controller.php", $force);
193
+        $controller = $this->writeTemplateToFile($controllerBody, $mod_path.DIRECTORY_SEPARATOR."Controller".
194
+            DIRECTORY_SEPARATOR."{$class}Controller.php", $force);
195 195
 
196 196
         $controllerBody = $this->tpl->dump("generator/controller.template.twig", array(
197 197
             "module" => $module,
198 198
             "namespace" => preg_replace('/(\\\|\/)/', '\\', $module),
199 199
             "url" => preg_replace('/(\\\|\/)/', '/', $module),
200
-            "class" => $class . "Base",
200
+            "class" => $class."Base",
201 201
             "service" => $class,
202 202
             "controllerType" => $controllerType,
203 203
             "is_base" => true,
204 204
             "domain" => $class,
205 205
         ));
206
-        $controllerBase = $this->writeTemplateToFile($controllerBody, $mod_path . DIRECTORY_SEPARATOR . "Controller" .
207
-            DIRECTORY_SEPARATOR . "base" . DIRECTORY_SEPARATOR . "{$class}BaseController.php", true);
206
+        $controllerBase = $this->writeTemplateToFile($controllerBody, $mod_path.DIRECTORY_SEPARATOR."Controller".
207
+            DIRECTORY_SEPARATOR."base".DIRECTORY_SEPARATOR."{$class}BaseController.php", true);
208 208
 
209
-        $filename = $mod_path . DIRECTORY_SEPARATOR . "Test" . DIRECTORY_SEPARATOR . "{$class}Test.php";
209
+        $filename = $mod_path.DIRECTORY_SEPARATOR."Test".DIRECTORY_SEPARATOR."{$class}Test.php";
210 210
         $test = true;
211
-        if(!file_exists($filename)) {
211
+        if (!file_exists($filename)) {
212 212
             $testTemplate = $this->tpl->dump("generator/testCase.template.twig", array(
213 213
                 "module" => $module,
214 214
                 "namespace" => preg_replace('/(\\\|\/)/', '\\', $module),
@@ -229,8 +229,8 @@  discard block
 block discarded – undo
229 229
     private function generateBaseApiTemplate($module, $mod_path, $force = false, $apiClass = "")
230 230
     {
231 231
         $created = true;
232
-        $modelPath = $mod_path . $module . DIRECTORY_SEPARATOR . 'Models';
233
-        $api_path = $mod_path . $module . DIRECTORY_SEPARATOR . 'Api';
232
+        $modelPath = $mod_path.$module.DIRECTORY_SEPARATOR.'Models';
233
+        $api_path = $mod_path.$module.DIRECTORY_SEPARATOR.'Api';
234 234
         if (file_exists($modelPath)) {
235 235
             $dir = dir($modelPath);
236 236
             while ($file = $dir->read()) {
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
         //Generamos el fichero de configuración
259 259
         $this->log->addLog("Generamos fichero vacío de configuración");
260 260
         return $this->writeTemplateToFile("<?php\n\t",
261
-            $mod_path . DIRECTORY_SEPARATOR . "Config" . DIRECTORY_SEPARATOR . "config.php",
261
+            $mod_path.DIRECTORY_SEPARATOR."Config".DIRECTORY_SEPARATOR."config.php",
262 262
             $force);
263 263
     }
264 264
 
@@ -273,10 +273,10 @@  discard block
 block discarded – undo
273 273
         //Generamos el fichero de configuración
274 274
         $this->log->addLog("Generamos ficheros para assets base");
275 275
         $css = $this->writeTemplateToFile("/* CSS3 STYLES */\n\n",
276
-            $mod_path . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . "css" . DIRECTORY_SEPARATOR . "styles.css",
276
+            $mod_path.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR."css".DIRECTORY_SEPARATOR."styles.css",
277 277
             $force);
278 278
         $js = $this->writeTemplateToFile("/* APP MODULE JS */\n\n(function() {\n\t'use strict';\n})();",
279
-            $mod_path . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . "js" . DIRECTORY_SEPARATOR . "app.js",
279
+            $mod_path.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR."js".DIRECTORY_SEPARATOR."app.js",
280 280
             $force);
281 281
         return ($css && $js);
282 282
     }
@@ -298,7 +298,7 @@  discard block
 block discarded – undo
298 298
             "class" => $class,
299 299
         ));
300 300
         return $this->writeTemplateToFile($controller,
301
-            $mod_path . DIRECTORY_SEPARATOR . "Services" . DIRECTORY_SEPARATOR . "{$class}Service.php",
301
+            $mod_path.DIRECTORY_SEPARATOR."Services".DIRECTORY_SEPARATOR."{$class}Service.php",
302 302
             $force);
303 303
     }
304 304
 
@@ -317,13 +317,13 @@  discard block
 block discarded – undo
317 317
             "autoloader" => preg_replace('/(\\\|\/)/', '_', $module),
318 318
             "regex" => preg_replace('/(\\\|\/)/m', '\\\\\\\\\\\\', $module),
319 319
         ));
320
-        $autoload = $this->writeTemplateToFile($autoloader, $mod_path . DIRECTORY_SEPARATOR . "autoload.php", $force);
320
+        $autoload = $this->writeTemplateToFile($autoloader, $mod_path.DIRECTORY_SEPARATOR."autoload.php", $force);
321 321
 
322 322
         $this->log->addLog("Generamos el phpunit");
323 323
         $phpUnitTemplate = $this->tpl->dump("generator/phpunit.template.twig", array(
324 324
             "module" => $module,
325 325
         ));
326
-        $phpunit = $this->writeTemplateToFile($phpUnitTemplate, $mod_path . DIRECTORY_SEPARATOR . "phpunit.xml.dist", $force);
326
+        $phpunit = $this->writeTemplateToFile($phpUnitTemplate, $mod_path.DIRECTORY_SEPARATOR."phpunit.xml.dist", $force);
327 327
         return $autoload && $phpunit;
328 328
     }
329 329
 
@@ -344,7 +344,7 @@  discard block
 block discarded – undo
344 344
             "db" => $this->config->get("db_name"),
345 345
         ));
346 346
         return $this->writeTemplateToFile($schema,
347
-            $mod_path . DIRECTORY_SEPARATOR . "Config" . DIRECTORY_SEPARATOR . "schema.xml",
347
+            $mod_path.DIRECTORY_SEPARATOR."Config".DIRECTORY_SEPARATOR."schema.xml",
348 348
             $force);
349 349
     }
350 350
 
@@ -362,7 +362,7 @@  discard block
 block discarded – undo
362 362
             "namespace" => preg_replace('/(\\\|\/)/', '', $module),
363 363
         ));
364 364
         return $this->writeTemplateToFile($build_properties,
365
-            $mod_path . DIRECTORY_SEPARATOR . "Config" . DIRECTORY_SEPARATOR . "propel.yml",
365
+            $mod_path.DIRECTORY_SEPARATOR."Config".DIRECTORY_SEPARATOR."propel.yml",
366 366
             $force);
367 367
     }
368 368
 
@@ -380,7 +380,7 @@  discard block
 block discarded – undo
380 380
             "module" => $module,
381 381
         ));
382 382
         return $this->writeTemplateToFile($index,
383
-            $mod_path . DIRECTORY_SEPARATOR . "Templates" . DIRECTORY_SEPARATOR . "index.html.twig",
383
+            $mod_path.DIRECTORY_SEPARATOR."Templates".DIRECTORY_SEPARATOR."index.html.twig",
384 384
             $force);
385 385
     }
386 386
 
@@ -398,11 +398,11 @@  discard block
 block discarded – undo
398 398
             try {
399 399
                 $this->cache->storeData($filename, $fileContent, Cache::TEXT, true);
400 400
                 $created = true;
401
-            } catch (\Exception $e) {
401
+            }catch (\Exception $e) {
402 402
                 Logger::log($e->getMessage(), LOG_ERR);
403 403
             }
404
-        } else {
405
-            Logger::log($filename . _(' not exists or cant write'), LOG_ERR);
404
+        }else {
405
+            Logger::log($filename._(' not exists or cant write'), LOG_ERR);
406 406
         }
407 407
         return $created;
408 408
     }
@@ -431,7 +431,7 @@  discard block
 block discarded – undo
431 431
         ));
432 432
 
433 433
         return $this->writeTemplateToFile($controller,
434
-            $mod_path . DIRECTORY_SEPARATOR . 'base' . DIRECTORY_SEPARATOR . "{$api}BaseApi.php", true);
434
+            $mod_path.DIRECTORY_SEPARATOR.'base'.DIRECTORY_SEPARATOR."{$api}BaseApi.php", true);
435 435
     }
436 436
 
437 437
     /**
@@ -454,7 +454,7 @@  discard block
 block discarded – undo
454 454
             "class" => $class,
455 455
         ));
456 456
 
457
-        return $this->writeTemplateToFile($controller, $mod_path . DIRECTORY_SEPARATOR . "{$api}.php", $force);
457
+        return $this->writeTemplateToFile($controller, $mod_path.DIRECTORY_SEPARATOR."{$api}.php", $force);
458 458
     }
459 459
 
460 460
     /**
@@ -468,12 +468,12 @@  discard block
 block discarded – undo
468 468
     {
469 469
         if (file_exists($filename_path)) {
470 470
             $destfolder = basename($filename_path);
471
-            if (!file_exists(WEB_DIR . $dest . DIRECTORY_SEPARATOR . $destfolder) || $debug || $force) {
471
+            if (!file_exists(WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder) || $debug || $force) {
472 472
                 if (is_dir($filename_path)) {
473
-                    self::copyr($filename_path, WEB_DIR . $dest . DIRECTORY_SEPARATOR . $destfolder);
474
-                } else {
475
-                    if (@copy($filename_path, WEB_DIR . $dest . DIRECTORY_SEPARATOR . $destfolder) === FALSE) {
476
-                        throw new ConfigException("Can't copy " . $filename_path . " to " . WEB_DIR . $dest . DIRECTORY_SEPARATOR . $destfolder);
473
+                    self::copyr($filename_path, WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder);
474
+                }else {
475
+                    if (@copy($filename_path, WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder) === FALSE) {
476
+                        throw new ConfigException("Can't copy ".$filename_path." to ".WEB_DIR.$dest.DIRECTORY_SEPARATOR.$destfolder);
477 477
                     }
478 478
                 }
479 479
             }
@@ -492,10 +492,10 @@  discard block
 block discarded – undo
492 492
         GeneratorHelper::createDir($dst);
493 493
         while (false !== ($file = readdir($dir))) {
494 494
             if (($file != '.') && ($file != '..')) {
495
-                if (is_dir($src . '/' . $file)) {
496
-                    self::copyr($src . '/' . $file, $dst . '/' . $file);
497
-                } elseif (@copy($src . '/' . $file, $dst . '/' . $file) === false) {
498
-                    throw new ConfigException("Can't copy " . $src . " to " . $dst);
495
+                if (is_dir($src.'/'.$file)) {
496
+                    self::copyr($src.'/'.$file, $dst.'/'.$file);
497
+                } elseif (@copy($src.'/'.$file, $dst.'/'.$file) === false) {
498
+                    throw new ConfigException("Can't copy ".$src." to ".$dst);
499 499
                 }
500 500
             }
501 501
         }
@@ -508,12 +508,12 @@  discard block
 block discarded – undo
508 508
      */
509 509
     private function getPropelPaths($module_path)
510 510
     {
511
-        $moduleDir = CORE_DIR . DIRECTORY_SEPARATOR . $module_path;
511
+        $moduleDir = CORE_DIR.DIRECTORY_SEPARATOR.$module_path;
512 512
         GeneratorHelper::createDir($moduleDir);
513 513
         $moduleDir = realpath($moduleDir);
514
-        $configDir = $moduleDir . DIRECTORY_SEPARATOR . 'Config';
515
-        $sqlDir = $moduleDir . DIRECTORY_SEPARATOR . 'Config' . DIRECTORY_SEPARATOR . 'Sql';
516
-        $migrationDir = $moduleDir . DIRECTORY_SEPARATOR . 'Config' . DIRECTORY_SEPARATOR . 'Migrations';
514
+        $configDir = $moduleDir.DIRECTORY_SEPARATOR.'Config';
515
+        $sqlDir = $moduleDir.DIRECTORY_SEPARATOR.'Config'.DIRECTORY_SEPARATOR.'Sql';
516
+        $migrationDir = $moduleDir.DIRECTORY_SEPARATOR.'Config'.DIRECTORY_SEPARATOR.'Migrations';
517 517
         $paths = [
518 518
             'projectDir' => $moduleDir,
519 519
             'outputDir' => $moduleDir,
@@ -578,9 +578,9 @@  discard block
 block discarded – undo
578 578
     private function setupManager(GeneratorConfig $configGenerator, AbstractManager &$manager, $workingDir = CORE_DIR)
579 579
     {
580 580
         $manager->setGeneratorConfig($configGenerator);
581
-        $schemaFile = new \SplFileInfo($configGenerator->getSection('paths')['schemaDir'] . DIRECTORY_SEPARATOR . 'schema.xml');
581
+        $schemaFile = new \SplFileInfo($configGenerator->getSection('paths')['schemaDir'].DIRECTORY_SEPARATOR.'schema.xml');
582 582
         $manager->setSchemas([$schemaFile]);
583
-        $manager->setLoggerClosure(function ($message) {
583
+        $manager->setLoggerClosure(function($message) {
584 584
             Logger::log($message, LOG_INFO);
585 585
         });
586 586
         $manager->setWorkingDirectory($workingDir);
Please login to merge, or discard this patch.
src/controller/I18nController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -42,8 +42,8 @@  discard block
 block discarded – undo
42 42
         //Generating the templates translations
43 43
         $translations = $this->tpl->regenerateTemplates();
44 44
 
45
-        $localePath = realpath(BASE_DIR . DIRECTORY_SEPARATOR . 'locale');
46
-        $localePath .= DIRECTORY_SEPARATOR . $locale . DIRECTORY_SEPARATOR . 'LC_MESSAGES' . DIRECTORY_SEPARATOR;
45
+        $localePath = realpath(BASE_DIR.DIRECTORY_SEPARATOR.'locale');
46
+        $localePath .= DIRECTORY_SEPARATOR.$locale.DIRECTORY_SEPARATOR.'LC_MESSAGES'.DIRECTORY_SEPARATOR;
47 47
 
48 48
         //xgettext localizations
49 49
         $translations = array_merge($translations, GeneratorService::findTranslations(SOURCE_DIR, $locale));
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
         $translations = array_merge($translations, GeneratorService::findTranslations(CACHE_DIR, $locale));
52 52
 
53 53
         $translations[] = "msgfmt {$localePath}translations.po -o {$localePath}translations.mo";
54
-        $translations[] = shell_exec('export PATH=\$PATH:/opt/local/bin:/bin:/sbin; msgfmt '. $localePath . 'translations.po -o ' . $localePath . 'translations.mo');
54
+        $translations[] = shell_exec('export PATH=\$PATH:/opt/local/bin:/bin:/sbin; msgfmt '.$localePath.'translations.po -o '.$localePath.'translations.mo');
55 55
         return $this->render('translations.html.twig', array(
56 56
             'translations' => $translations,
57 57
         ));
Please login to merge, or discard this patch.
src/base/config/ConfigForm.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -83,9 +83,9 @@
 block discarded – undo
83 83
             'class' => 'btn-warning md-default',
84 84
             'icon' => 'fa-plus',
85 85
         ];
86
-        if(Config::getParam('admin.version', 'v1') === 'v1') {
87
-            $add['onclick'] = 'javascript:addNewField(document.getElementById("' . $this->getName() . '"));';
88
-        } else {
86
+        if (Config::getParam('admin.version', 'v1') === 'v1') {
87
+            $add['onclick'] = 'javascript:addNewField(document.getElementById("'.$this->getName().'"));';
88
+        }else {
89 89
             $add['ng-click'] = 'addNewField()';
90 90
         }
91 91
         $this->addButton('submit', _('Guardar configuración'), 'submit', array(
Please login to merge, or discard this patch.
src/base/types/Form.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -69,8 +69,8 @@  discard block
 block discarded – undo
69 69
     public function add($name, array $value = array())
70 70
     {
71 71
         $this->fields[$name] = $value;
72
-        $this->fields[$name]['name'] = $this->getName() . '['.$name.']';
73
-        $this->fields[$name]['id'] = $this->getName() . '_' . $name;
72
+        $this->fields[$name]['name'] = $this->getName().'['.$name.']';
73
+        $this->fields[$name]['id'] = $this->getName().'_'.$name;
74 74
         $this->fields[$name]['placeholder'] = array_key_exists('placeholder', $value) ? $value['placeholder'] : $name;
75 75
         $this->fields[$name]['hasLabel'] = array_key_exists('hasLabel', $value) ? $value['hasLabel'] : true;
76 76
         return $this;
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
         }
134 134
         if ('' !== $hashOrig) {
135 135
             $this->crfs = sha1($hashOrig);
136
-            $this->add($this->getName() . '_token', array(
136
+            $this->add($this->getName().'_token', array(
137 137
                 'type' => 'hidden',
138 138
                 'value' => $this->crfs,
139 139
             ));
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
     public function isValid()
203 203
     {
204 204
         $valid = true;
205
-        $tokenField = $this->getName() . '_token';
205
+        $tokenField = $this->getName().'_token';
206 206
         //Controlamos CSRF token
207 207
         if (!$this->existsFormToken($tokenField)) {
208 208
             $this->errors[$tokenField] = t('Formulario no válido');
@@ -276,7 +276,7 @@  discard block
 block discarded – undo
276 276
         $data = array();
277 277
         if (count($this->fields) > 0) {
278 278
             foreach ($this->fields as $key => $field) {
279
-                if (self::SEPARATOR !== $key && $key !== ($this->getName() . '_token')) {
279
+                if (self::SEPARATOR !== $key && $key !== ($this->getName().'_token')) {
280 280
                     $data[$key] = array_key_exists('value', $field) ? $field['value'] : null;
281 281
                 }
282 282
             }
@@ -390,8 +390,8 @@  discard block
 block discarded – undo
390 390
             $model = $this->getHydratedModel();
391 391
             $model->save();
392 392
             $save = true;
393
-            Logger::log(get_class($this->model) . ' guardado con id ' . $this->model->getPrimaryKey(), LOG_INFO);
394
-        } catch (\Exception $e) {
393
+            Logger::log(get_class($this->model).' guardado con id '.$this->model->getPrimaryKey(), LOG_INFO);
394
+        }catch (\Exception $e) {
395 395
             Logger::log($e->getMessage(), LOG_ERR);
396 396
             throw new FormException($e->getMessage(), $e->getCode(), $e);
397 397
         }
@@ -410,10 +410,10 @@  discard block
 block discarded – undo
410 410
         if (null === $value) {
411 411
             $isEmpty = true;
412 412
             // Empty Array check
413
-        } else if (is_array($value) && 0 === count($value)) {
413
+        }else if (is_array($value) && 0 === count($value)) {
414 414
             $isEmpty = true;
415 415
             // Empty string check
416
-        } else if ('' === preg_replace('/(\ |\r|\n)/m', '', $value)) {
416
+        }else if ('' === preg_replace('/(\ |\r|\n)/m', '', $value)) {
417 417
             $isEmpty = true;
418 418
         }
419 419
 
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
             && array_key_exists($key, $field)
441 441
             && !array_key_exists('error', $field[$key])
442 442
             && !empty($field['value'])
443
-            && preg_match('/' . $field['pattern'] . '/', $field['value']) === 0
443
+            && preg_match('/'.$field['pattern'].'/', $field['value']) === 0
444 444
         ) {
445 445
             $this->setError($key, str_replace('%s', "<strong>{$key}</strong>", t('El campo %s no tiene un formato válido')));
446 446
             $field['error'] = $this->getError($key);
@@ -462,10 +462,10 @@  discard block
 block discarded – undo
462 462
         if (array_key_exists($key, $data[$formName])) {
463 463
             if (preg_match('/id/i', $key) && ($data[$formName][$key] === 0 || $data[$formName][$key] === '%' || $data[$formName][$key] === '')) {
464 464
                 $field['value'] = null;
465
-            } else {
465
+            }else {
466 466
                 $field['value'] = $data[$formName][$key];
467 467
             }
468
-        } else {
468
+        }else {
469 469
             unset($field['value']);
470 470
         }
471 471
         return array($data, $field);
@@ -479,8 +479,8 @@  discard block
 block discarded – undo
479 479
      */
480 480
     private function hydrateModelField($key, $value)
481 481
     {
482
-        $setter = 'set' . ucfirst($key);
483
-        $getter = 'get' . ucfirst($key);
482
+        $setter = 'set'.ucfirst($key);
483
+        $getter = 'get'.ucfirst($key);
484 484
         if (method_exists($this->model, $setter)) {
485 485
             if (method_exists($this->model, $getter)) {
486 486
                 $tmp = $this->model->$getter();
@@ -505,16 +505,16 @@  discard block
 block discarded – undo
505 505
     {
506 506
         //Extraemos el dato del modelo relacionado si existe el getter
507 507
         $method = null;
508
-        if (array_key_exists('class_data', $field) && method_exists($val, 'get' . $field['class_data'])) {
509
-            $classMethod = 'get' . $field['class_data'];
508
+        if (array_key_exists('class_data', $field) && method_exists($val, 'get'.$field['class_data'])) {
509
+            $classMethod = 'get'.$field['class_data'];
510 510
             $class = $val->$classMethod();
511
-            if (array_key_exists('class_id', $field) && method_exists($class, 'get' . $field['class_id'])) {
512
-                $method = 'get' . $field['class_id'];
511
+            if (array_key_exists('class_id', $field) && method_exists($class, 'get'.$field['class_id'])) {
512
+                $method = 'get'.$field['class_id'];
513 513
                 $data[] = $class->$method();
514
-            } else {
514
+            }else {
515 515
                 $data[] = $class->getPrimaryKey();
516 516
             }
517
-        } else {
517
+        }else {
518 518
             $data[] = $val;
519 519
         }
520 520
 
@@ -530,7 +530,7 @@  discard block
 block discarded – undo
530 530
     private function extractModelFieldValue($key, $field)
531 531
     {
532 532
         //Extraemos el valor del campo del modelo
533
-        $method = 'get' . ucfirst($key);
533
+        $method = 'get'.ucfirst($key);
534 534
         $type = (array_key_exists('type', $field)) ? $field['type'] : 'text';
535 535
         //Extraemos los campos del modelo
536 536
         if (method_exists($this->model, $method)) {
@@ -540,16 +540,16 @@  discard block
 block discarded – undo
540 540
                 //Si es una relación múltiple
541 541
                 if ($value instanceof ObjectCollection) {
542 542
                     $field = $this->computeModelFieldValue($field, $value, $type);
543
-                } else { //O una relación unitaria
543
+                }else { //O una relación unitaria
544 544
                     if (method_exists($value, '__toString')) {
545 545
                         $field['value'] = $value;
546 546
                     } elseif ($value instanceof \DateTime) {
547 547
                         $field['value'] = $value->format('Y-m-d H:i:s');
548
-                    } else {
548
+                    }else {
549 549
                         $field['value'] = $value->getPrimaryKey();
550 550
                     }
551 551
                 }
552
-            } else {
552
+            }else {
553 553
                 $field['value'] = $value;
554 554
             }
555 555
         }
Please login to merge, or discard this patch.