Passed
Push — master ( 1016bd...db9634 )
by Fran
04:53
created
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.