Passed
Push — master ( 5fae1c...f113c6 )
by Fran
10:43 queued 08:12
created
src/base/types/helpers/ApiFormHelper.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -19,11 +19,11 @@
 block discarded – undo
19 19
     public static function checkApiActions($namespace, $domain, $api) {
20 20
         $actions = [];
21 21
         $reflector = new ReflectionClass($namespace);
22
-        if(null !== $reflector) {
23
-            foreach($reflector->getMethods(ReflectionMethod::IS_PUBLIC) as $apiAction) {
22
+        if (null !== $reflector) {
23
+            foreach ($reflector->getMethods(ReflectionMethod::IS_PUBLIC) as $apiAction) {
24 24
                 $docComments = $apiAction->getDocComment();
25 25
                 $action = AnnotationHelper::extractAction($docComments);
26
-                if(null !== $action) {
26
+                if (null !== $action) {
27 27
                     list($route, $info) = RouterHelper::extractRouteInfo($apiAction, $api, $domain);
28 28
                     list($method, $cleanRoute) = RouterHelper::extractHttpRoute($route);
29 29
                     $formAction = new FormAction();
Please login to merge, or discard this patch.
src/base/types/helpers/InjectorHelper.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -31,10 +31,10 @@  discard block
 block discarded – undo
31 31
                 $label = self::getLabel($property->getDocComment());
32 32
                 $values = self::getValues($property->getDocComment());
33 33
                 $isArray = (bool)preg_match('/\[\]$/', $instanceType);
34
-                if($isArray) {
34
+                if ($isArray) {
35 35
                     $instanceType = str_replace('[]', '', $instanceType);
36 36
                 }
37
-                if($instanceType === '\\DateTime' || !Router::exists($instanceType)) {
37
+                if ($instanceType === '\\DateTime' || !Router::exists($instanceType)) {
38 38
                     list($type, $format) = DocumentorHelper::translateSwaggerFormats($instanceType);
39 39
                     $variables[$property->getName()] = [
40 40
                         'type' => $type,
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
                         'required' => $isRequired,
43 43
                         'description' => $label,
44 44
                     ];
45
-                } else {
45
+                }else {
46 46
                     $instance = new ReflectionClass($instanceType);
47 47
                     $variables[$property->getName()] = [
48 48
                         'is_array' => $isArray,
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
                         'properties' => self::extractVariables($instance),
52 52
                     ];
53 53
                 }
54
-                if(!empty($values)){
54
+                if (!empty($values)) {
55 55
                     $variables[$property->getName()]['enum'] = $values;
56 56
                 }
57 57
             }
@@ -155,13 +155,13 @@  discard block
 block discarded – undo
155 155
      */
156 156
     public static function constructInjectableInstance($variable, $singleton, $classNameSpace, $calledClass)
157 157
     {
158
-        Logger::log('Create inyectable instance for ' . $classNameSpace);
158
+        Logger::log('Create inyectable instance for '.$classNameSpace);
159 159
         $reflector = new ReflectionClass($calledClass);
160 160
         $property = $reflector->getProperty($variable);
161 161
         $varInstanceType = (null === $classNameSpace) ? InjectorHelper::extractVarType($property->getDocComment()) : $classNameSpace;
162 162
         if (true === $singleton && method_exists($varInstanceType, 'getInstance')) {
163 163
             $instance = $varInstanceType::getInstance();
164
-        } else {
164
+        }else {
165 165
             $instance = new $varInstanceType();
166 166
         }
167 167
         return $instance;
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
     public static function getClassProperties($class)
176 176
     {
177 177
         $properties = [];
178
-        Logger::log('Extracting annotations properties from class ' . $class);
178
+        Logger::log('Extracting annotations properties from class '.$class);
179 179
         $selfReflector = new ReflectionClass($class);
180 180
         if (false !== $selfReflector->getParentClass()) {
181 181
             $properties = self::getClassProperties($selfReflector->getParentClass()->getName());
Please login to merge, or discard this patch.
src/base/types/helpers/AnnotationHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@
 block discarded – undo
92 92
      * @return string|null
93 93
      */
94 94
     public static function extractFromDoc($needle, $comments, $default = null) {
95
-        preg_match('/@' . $needle . '\ (.*)(\n|\r)/im', $comments, $matches);
95
+        preg_match('/@'.$needle.'\ (.*)(\n|\r)/im', $comments, $matches);
96 96
         return (count($matches) > 0) ? $matches[1] : $default;
97 97
     }
98 98
 }
Please login to merge, or discard this patch.
src/base/types/traits/Helper/FieldHelperTrait.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -122,27 +122,27 @@  discard block
 block discarded – undo
122 122
         $behaviors = $tableMap->getBehaviors();
123 123
         foreach ($map::getFieldNames() as $field) {
124 124
             $fDto = self::parseFormField($domain, $tableMap, $field, $behaviors);
125
-            if(null !== $fDto) {
125
+            if (null !== $fDto) {
126 126
                 $form->addField($fDto);
127 127
             }
128 128
         }
129 129
 
130
-        if(array_key_exists('i18n', $behaviors)) {
131
-            $relateI18n = $tableMap->getRelation($tableMap->getPhpName() . 'I18n');
132
-            if(null !== $relateI18n) {
130
+        if (array_key_exists('i18n', $behaviors)) {
131
+            $relateI18n = $tableMap->getRelation($tableMap->getPhpName().'I18n');
132
+            if (null !== $relateI18n) {
133 133
                 $i18NTableMap = $relateI18n->getLocalTable();
134
-                foreach($i18NTableMap->getColumns() as $columnMap) {
134
+                foreach ($i18NTableMap->getColumns() as $columnMap) {
135 135
                     $columnName = self::getColumnMapName($columnMap);
136
-                    if(!$form->fieldExists($columnName)) {
136
+                    if (!$form->fieldExists($columnName)) {
137 137
                         /** @var Field $fDto */
138 138
                         $fDto = self::parseFormField($domain, $i18NTableMap, $columnMap->getPhpName(), $i18NTableMap->getBehaviors());
139
-                        if(null !== $fDto) {
139
+                        if (null !== $fDto) {
140 140
                             $fDto->pk = false;
141 141
                             $fDto->required = true;
142
-                            if(strtolower($fDto->name) === 'locale') {
142
+                            if (strtolower($fDto->name) === 'locale') {
143 143
                                 $fDto->type = Field::COMBO_TYPE;
144 144
                                 $languages = explode(',', Config::getParam('i18n.locales', Config::getParam('default.language', 'es_ES')));
145
-                                foreach($languages as $language) {
145
+                                foreach ($languages as $language) {
146 146
                                     $fDto->data[] = [
147 147
                                         $fDto->name => $language,
148 148
                                         'Label' => t($language),
@@ -169,14 +169,14 @@  discard block
 block discarded – undo
169 169
     {
170 170
         $column = null;
171 171
         try {
172
-            foreach($tableMap->getColumns() as $tableMapColumn) {
172
+            foreach ($tableMap->getColumns() as $tableMapColumn) {
173 173
                 $columnName = $tableMapColumn->getPhpName();
174
-                if(preg_match('/^'.$field.'$/i', $columnName)) {
174
+                if (preg_match('/^'.$field.'$/i', $columnName)) {
175 175
                     $column = $tableMapColumn;
176 176
                     break;
177 177
                 }
178 178
             }
179
-        } catch (\Exception $e) {
179
+        }catch (\Exception $e) {
180 180
             Logger::log($e->getMessage(), LOG_DEBUG);
181 181
         }
182 182
         return $column;
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
      */
188 188
     public static function getFieldTypes() {
189 189
         $configType = Config::getParam('api.field.case', TableMap::TYPE_PHPNAME);
190
-        switch($configType) {
190
+        switch ($configType) {
191 191
             default:
192 192
             case 'UpperCamelCase':
193 193
             case TableMap::TYPE_PHPNAME:
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
@@ -44,8 +44,8 @@  discard block
 block discarded – undo
44 44
         //Generating the templates translations
45 45
         $translations = $this->tpl->regenerateTemplates();
46 46
 
47
-        $localePath = realpath(BASE_DIR . DIRECTORY_SEPARATOR . 'locale');
48
-        $localePath .= DIRECTORY_SEPARATOR . $locale . DIRECTORY_SEPARATOR . 'LC_MESSAGES' . DIRECTORY_SEPARATOR;
47
+        $localePath = realpath(BASE_DIR.DIRECTORY_SEPARATOR.'locale');
48
+        $localePath .= DIRECTORY_SEPARATOR.$locale.DIRECTORY_SEPARATOR.'LC_MESSAGES'.DIRECTORY_SEPARATOR;
49 49
 
50 50
         //xgettext localizations
51 51
         $translations = array_merge($translations, I18nHelper::findTranslations(SOURCE_DIR, $locale));
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
         $translations = array_merge($translations, I18nHelper::findTranslations(CACHE_DIR, $locale));
54 54
 
55 55
         $translations[] = "msgfmt {$localePath}translations.po -o {$localePath}translations.mo";
56
-        $translations[] = shell_exec('export PATH=\$PATH:/opt/local/bin:/bin:/sbin; msgfmt '. $localePath . 'translations.po -o ' . $localePath . 'translations.mo');
56
+        $translations[] = shell_exec('export PATH=\$PATH:/opt/local/bin:/bin:/sbin; msgfmt '.$localePath.'translations.po -o '.$localePath.'translations.mo');
57 57
         return $this->render('translations.html.twig', array(
58 58
             'translations' => $translations,
59 59
         ));
Please login to merge, or discard this patch.
src/base/types/traits/Generator/StructureTrait.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
         // Creates the src folder
25 25
         GeneratorHelper::createDir($modPath);
26 26
         // Create module path
27
-        GeneratorHelper::createDir($modPath . $module);
27
+        GeneratorHelper::createDir($modPath.$module);
28 28
     }
29 29
 
30 30
     /**
@@ -40,16 +40,16 @@  discard block
 block discarded – undo
40 40
         Logger::log("Generamos la estructura");
41 41
         $paths = [
42 42
             "Api", "Config", "Controller", "Models", "Public", "Templates", "Services", "Test", "Doc",
43
-            "Locale", "Locale/" . Config::getParam('default.locale', 'es_ES'), "Locale/" . Config::getParam('default.locale', 'es_ES') . "/LC_MESSAGES"
43
+            "Locale", "Locale/".Config::getParam('default.locale', 'es_ES'), "Locale/".Config::getParam('default.locale', 'es_ES')."/LC_MESSAGES"
44 44
         ];
45
-        $modulePath = $modPath . $module;
45
+        $modulePath = $modPath.$module;
46 46
         foreach ($paths as $path) {
47
-            GeneratorHelper::createDir($modulePath . DIRECTORY_SEPARATOR . $path);
47
+            GeneratorHelper::createDir($modulePath.DIRECTORY_SEPARATOR.$path);
48 48
         }
49 49
         //Creamos las carpetas de los assets
50 50
         $htmlPaths = array("css", "js", "img", "media", "font");
51 51
         foreach ($htmlPaths as $path) {
52
-            GeneratorHelper::createDir($modulePath . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . $path);
52
+            GeneratorHelper::createDir($modulePath.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR.$path);
53 53
         }
54 54
     }
55 55
 
@@ -64,10 +64,10 @@  discard block
 block discarded – undo
64 64
         //Generamos el fichero de configuración
65 65
         Logger::log("Generamos ficheros para assets base");
66 66
         $css = $this->writeTemplateToFile("/* CSS3 STYLES */\n\n",
67
-            $modPath . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . "css" . DIRECTORY_SEPARATOR . "styles.css",
67
+            $modPath.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR."css".DIRECTORY_SEPARATOR."styles.css",
68 68
             $force);
69 69
         $js = $this->writeTemplateToFile("/* APP MODULE JS */\n\n(function() {\n\t'use strict';\n})();",
70
-            $modPath . DIRECTORY_SEPARATOR . "Public" . DIRECTORY_SEPARATOR . "js" . DIRECTORY_SEPARATOR . "app.js",
70
+            $modPath.DIRECTORY_SEPARATOR."Public".DIRECTORY_SEPARATOR."js".DIRECTORY_SEPARATOR."app.js",
71 71
             $force);
72 72
         return ($css && $js);
73 73
     }
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
             "class" => $class,
90 90
         ));
91 91
         return $this->writeTemplateToFile($controller,
92
-            $modPath . DIRECTORY_SEPARATOR . "Services" . DIRECTORY_SEPARATOR . "{$class}Service.php",
92
+            $modPath.DIRECTORY_SEPARATOR."Services".DIRECTORY_SEPARATOR."{$class}Service.php",
93 93
             $force);
94 94
     }
95 95
 
@@ -108,13 +108,13 @@  discard block
 block discarded – undo
108 108
             "autoloader" => preg_replace('/(\\\|\/)/', '_', $module),
109 109
             "regex" => preg_replace('/(\\\|\/)/m', '\\\\\\\\\\\\', $module),
110 110
         ));
111
-        $autoload = $this->writeTemplateToFile($autoloader, $mod_path . DIRECTORY_SEPARATOR . "autoload.php", $force);
111
+        $autoload = $this->writeTemplateToFile($autoloader, $mod_path.DIRECTORY_SEPARATOR."autoload.php", $force);
112 112
 
113 113
         Logger::log("Generamos el phpunit");
114 114
         $phpUnitTemplate = $this->tpl->dump("generator/phpunit.template.twig", array(
115 115
             "module" => $module,
116 116
         ));
117
-        $phpunit = $this->writeTemplateToFile($phpUnitTemplate, $mod_path . DIRECTORY_SEPARATOR . "phpunit.xml.dist", $force);
117
+        $phpunit = $this->writeTemplateToFile($phpUnitTemplate, $mod_path.DIRECTORY_SEPARATOR."phpunit.xml.dist", $force);
118 118
         return $autoload && $phpunit;
119 119
     }
120 120
 }
Please login to merge, or discard this patch.
src/base/types/traits/TemplateTrait.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -28,11 +28,11 @@
 block discarded – undo
28 28
             try {
29 29
                 Cache::getInstance()->storeData($filename, $fileContent, Cache::TEXT, true);
30 30
                 $created = true;
31
-            } catch (\Exception $e) {
31
+            }catch (\Exception $e) {
32 32
                 Logger::log($e->getMessage(), LOG_ERR);
33 33
             }
34
-        } else {
35
-            Logger::log($filename . t(' not exists or cant write'), LOG_ERR);
34
+        }else {
35
+            Logger::log($filename.t(' not exists or cant write'), LOG_ERR);
36 36
         }
37 37
         return $created;
38 38
     }
Please login to merge, or discard this patch.
src/services/DocumentorService.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
      */
45 45
     public function extractApiEndpoints(array $module)
46 46
     {
47
-        $modulePath = $module['path'] . DIRECTORY_SEPARATOR . 'Api';
47
+        $modulePath = $module['path'].DIRECTORY_SEPARATOR.'Api';
48 48
         $moduleName = $module['name'];
49 49
         $endpoints = [];
50 50
         if (file_exists($modulePath)) {
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
                 /** @var SplFileInfo $file */
55 55
                 foreach ($finder as $file) {
56 56
                     $filename = str_replace([$modulePath, '/'], ['', '\\'], $file->getPathname());
57
-                    $namespace = "\\{$moduleName}\\Api" . str_replace('.php', '', $filename);
57
+                    $namespace = "\\{$moduleName}\\Api".str_replace('.php', '', $filename);
58 58
                     $info = $this->extractApiInfo($namespace, $moduleName);
59 59
                     if (!empty($info)) {
60 60
                         $endpoints[$namespace] = $info;
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
                         if (NULL !== $mInfo) {
85 85
                             $info[] = $mInfo;
86 86
                         }
87
-                    } catch (Exception $e) {
87
+                    }catch (Exception $e) {
88 88
                         Logger::log($e->getMessage(), LOG_ERR);
89 89
                     }
90 90
                 }
Please login to merge, or discard this patch.
src/base/types/traits/Api/SwaggerFormaterTrait.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -86,10 +86,10 @@  discard block
 block discarded – undo
86 86
         $formatted = [
87 87
             "swagger" => "2.0",
88 88
             "host" => preg_replace('/^(http|https)\:\/\/(.*)\/$/i', '$2', Router::getInstance()->getRoute('', true)),
89
-            "basePath" => '/' . $module['name'] . '/api',
89
+            "basePath" => '/'.$module['name'].'/api',
90 90
             "schemes" => [Request::getInstance()->getServer('HTTPS') === 'on' ? 'https' : 'http'],
91 91
             "info" => [
92
-                "title" => t('Documentación API módulo ') . $module['name'],
92
+                "title" => t('Documentación API módulo ').$module['name'],
93 93
                 "version" => Config::getParam('api.version', '1.0.0'),
94 94
                 "contact" => [
95 95
                     "name" => Config::getParam("author", "Fran López"),
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
         foreach ($endpoints as $model) {
102 102
             foreach ($model as $endpoint) {
103 103
                 if (!preg_match('/^\/(admin|api)\//i', $endpoint['url']) && strlen($endpoint['url'])) {
104
-                    $url = preg_replace('/\/' . $module['name'] . '\/api/i', '', $endpoint['url']);
104
+                    $url = preg_replace('/\/'.$module['name'].'\/api/i', '', $endpoint['url']);
105 105
                     $description = $endpoint['description'];
106 106
                     $method = strtolower($endpoint['method']);
107 107
                     $paths[$url][$method] = [
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
             }
143 143
         }
144 144
         ksort($dtos);
145
-        uasort($paths, function ($path1, $path2) {
145
+        uasort($paths, function($path1, $path2) {
146 146
             $key1 = array_keys($path1)[0];
147 147
             $key2 = array_keys($path2)[0];
148 148
             return strcmp($path1[$key1]['tags'][0], $path2[$key2]['tags'][0]);
@@ -166,13 +166,13 @@  discard block
 block discarded – undo
166 166
                     $modelDto['objects'][$dtoName][$param] = [
167 167
                         'type' => 'array',
168 168
                         'items' => [
169
-                            '$ref' => '#/definitions/' . $info['type'],
169
+                            '$ref' => '#/definitions/'.$info['type'],
170 170
                         ]
171 171
                     ];
172
-                } else {
172
+                }else {
173 173
                     $modelDto['objects'][$dtoName][$param] = [
174 174
                         'type' => 'object',
175
-                        '$ref' => '#/definitions/' . $info['type'],
175
+                        '$ref' => '#/definitions/'.$info['type'],
176 176
                     ];
177 177
                 }
178 178
                 $modelDto['objects'][$info['class']] = $info['properties'];
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
                 if (array_key_exists('objects', $paramDto)) {
181 181
                     $modelDto['objects'] = array_merge($modelDto['objects'], $paramDto['objects']);
182 182
                 }
183
-            } else {
183
+            }else {
184 184
                 $modelDto['objects'][$dtoName][$param] = $info;
185 185
             }
186 186
         }
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
211 211
                     $return = $this->extractReturn($modelNamespace, $docComments);
212 212
                     $url = array_pop($route);
213 213
                     $methodInfo = [
214
-                        'url' => str_replace('/' . $module . '/api', '', $url),
214
+                        'url' => str_replace('/'.$module.'/api', '', $url),
215 215
                         'method' => $info['http'],
216 216
                         'description' => $info['label'],
217 217
                         'return' => $return,
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
                     $this->setRequestParams($method, $methodInfo, $modelNamespace, $docComments);
223 223
                     $this->setQueryParams($method, $methodInfo);
224 224
                     $this->setRequestHeaders($reflection, $methodInfo);
225
-                } catch (Exception $e) {
225
+                }catch (Exception $e) {
226 226
                     Logger::log($e->getMessage(), LOG_ERR);
227 227
                 }
228 228
             }
@@ -322,7 +322,7 @@  discard block
 block discarded – undo
322 322
             foreach ($method->getParameters() as $parameter) {
323 323
                 $parameterName = $parameter->getName();
324 324
                 $types = [];
325
-                preg_match_all('/\@param\ (.*)\ \$' . $parameterName . '$/im', $docComments, $types);
325
+                preg_match_all('/\@param\ (.*)\ \$'.$parameterName.'$/im', $docComments, $types);
326 326
                 if (count($types) > 1 && count($types[1]) > 0) {
327 327
                     $methodInfo['parameters'][$parameterName] = $types[1][0];
328 328
                 }
Please login to merge, or discard this patch.