Completed
Push — master ( 7a7060...5183c9 )
by Nicolas
02:35 queued 21s
created
src/Commands/RuleMakeCommand.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
     }
60 60
 
61 61
     /**
62
-     * @return mixed
62
+     * @return string
63 63
      */
64 64
     protected function getTemplateContents()
65 65
     {
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
     }
74 74
 
75 75
     /**
76
-     * @return mixed
76
+     * @return string
77 77
      */
78 78
     protected function getDestinationFilePath()
79 79
     {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@
 block discarded – undo
67 67
 
68 68
         $rulePath = $this->laravel['modules']->config('paths.generator.rules');
69 69
 
70
-        return $path . $rulePath . '/' . $this->getFileName() . '.php';
70
+        return $path.$rulePath.'/'.$this->getFileName().'.php';
71 71
     }
72 72
 
73 73
     /**
Please login to merge, or discard this patch.
src/Commands/MailMakeCommand.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
 
66 66
         $mailPath = $this->laravel['modules']->config('paths.generator.emails', 'Emails');
67 67
 
68
-        return $path . $mailPath . '/' . $this->getFileName() . '.php';
68
+        return $path.$mailPath.'/'.$this->getFileName().'.php';
69 69
     }
70 70
 
71 71
     /**
Please login to merge, or discard this patch.
src/Commands/NotificationMakeCommand.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
 
46 46
         $mailPath = $this->laravel['modules']->config('paths.generator.notifications', 'Notifications');
47 47
 
48
-        return $path . $mailPath . '/' . $this->getFileName() . '.php';
48
+        return $path.$mailPath.'/'.$this->getFileName().'.php';
49 49
     }
50 50
 
51 51
     /**
Please login to merge, or discard this patch.
src/Commands/JobMakeCommand.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
 
66 66
         $jobPath = $this->laravel['modules']->config('paths.generator.jobs');
67 67
 
68
-        return $path . $jobPath . '/' . $this->getFileName() . '.php';
68
+        return $path.$jobPath.'/'.$this->getFileName().'.php';
69 69
     }
70 70
 
71 71
     /**
Please login to merge, or discard this patch.
src/Commands/EventMakeCommand.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         $module = $this->laravel['modules']->findOrFail($this->getModuleName());
32 32
 
33 33
         return (new Stub('/event.stub', [
34
-            'NAMESPACE' => $this->getClassNamespace($module) . "\\" . config('modules.paths.generator.event'),
34
+            'NAMESPACE' => $this->getClassNamespace($module)."\\".config('modules.paths.generator.event'),
35 35
             "CLASS"     => $this->getClass(),
36 36
         ]))->render();
37 37
     }
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
         $path       = $this->laravel['modules']->getModulePath($this->getModuleName());
42 42
         $seederPath = $this->laravel['modules']->config('paths.generator.event');
43 43
 
44
-        return $path . $seederPath . '/' . $this->getFileName() . '.php';
44
+        return $path.$seederPath.'/'.$this->getFileName().'.php';
45 45
     }
46 46
 
47 47
     /**
Please login to merge, or discard this patch.
src/helpers.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-if (! function_exists('module_path')) {
3
+if (!function_exists('module_path')) {
4 4
     function module_path($name)
5 5
     {
6 6
         $module = app('modules')->find($name);
@@ -9,7 +9,7 @@  discard block
 block discarded – undo
9 9
     }
10 10
 }
11 11
 
12
-if (! function_exists('public_path')) {
12
+if (!function_exists('public_path')) {
13 13
     /**
14 14
      * Get the path to the public folder.
15 15
      *
@@ -18,6 +18,6 @@  discard block
 block discarded – undo
18 18
      */
19 19
     function public_path($path = '')
20 20
     {
21
-        return app()->make('path.public') . ($path ? DIRECTORY_SEPARATOR . ltrim($path, DIRECTORY_SEPARATOR) : $path);
21
+        return app()->make('path.public').($path ? DIRECTORY_SEPARATOR . ltrim($path, DIRECTORY_SEPARATOR) : $path);
22 22
     }
23 23
 }
Please login to merge, or discard this patch.
src/Commands/MigrationMakeCommand.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
     /**
68 68
      * @throws \InvalidArgumentException
69 69
      *
70
-     * @return mixed
70
+     * @return Stub
71 71
      */
72 72
     protected function getTemplateContents()
73 73
     {
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
     }
108 108
 
109 109
     /**
110
-     * @return mixed
110
+     * @return string
111 111
      */
112 112
     protected function getDestinationFilePath()
113 113
     {
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 
114 114
         $generatorPath = $this->laravel['modules']->config('paths.generator.migration');
115 115
 
116
-        return $path . $generatorPath . '/' . $this->getFileName() . '.php';
116
+        return $path.$generatorPath.'/'.$this->getFileName().'.php';
117 117
     }
118 118
 
119 119
     /**
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
      */
122 122
     private function getFileName()
123 123
     {
124
-        return date('Y_m_d_His_') . $this->getSchemaName();
124
+        return date('Y_m_d_His_').$this->getSchemaName();
125 125
     }
126 126
 
127 127
     /**
Please login to merge, or discard this patch.
src/Commands/ModelMakeCommand.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
     }
60 60
 
61 61
     /**
62
-     * @return mixed
62
+     * @return string
63 63
      */
64 64
     protected function getTemplateContents()
65 65
     {
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
     }
79 79
 
80 80
     /**
81
-     * @return mixed
81
+     * @return string
82 82
      */
83 83
     protected function getDestinationFilePath()
84 84
     {
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
     }
91 91
 
92 92
     /**
93
-     * @return mixed|string
93
+     * @return string
94 94
      */
95 95
     private function getModelName()
96 96
     {
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -52,8 +52,8 @@  discard block
 block discarded – undo
52 52
 
53 53
         $string = '';
54 54
         foreach ($pieces as $i => $piece) {
55
-            if ($i+1 < count($pieces)) {
56
-                $string .= strtolower($piece) . '_';
55
+            if ($i + 1 < count($pieces)) {
56
+                $string .= strtolower($piece).'_';
57 57
             } else {
58 58
                 $string .= Str::plural(strtolower($piece));
59 59
             }
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
     private function handleOptionalMigrationOption()
95 95
     {
96 96
         if ($this->option('migration') === true) {
97
-            $migrationName = 'create_' . $this->createMigrationName() . '_table';
97
+            $migrationName = 'create_'.$this->createMigrationName().'_table';
98 98
             $this->call('module:make-migration', ['name' => $migrationName, 'module' => $this->argument('module')]);
99 99
         }
100 100
     }
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 
128 128
         $seederPath = $this->laravel['modules']->config('paths.generator.model');
129 129
 
130
-        return $path . $seederPath . '/' . $this->getModelName() . '.php';
130
+        return $path.$seederPath.'/'.$this->getModelName().'.php';
131 131
     }
132 132
 
133 133
     /**
Please login to merge, or discard this patch.
src/Module.php 2 patches
Doc Comments   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
      *
43 43
      * @param Container $app
44 44
      * @param $name
45
-     * @param $path
45
+     * @param string $path
46 46
      */
47 47
     public function __construct(Container $app, $name, $path)
48 48
     {
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
     /**
228 228
      * Get a specific data from composer.json file by given the key.
229 229
      *
230
-     * @param $key
230
+     * @param string $key
231 231
      * @param null $default
232 232
      *
233 233
      * @return mixed
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
     /**
301 301
      * Determine whether the given status same with the current module status.
302 302
      *
303
-     * @param $status
303
+     * @param integer $status
304 304
      *
305 305
      * @return bool
306 306
      */
@@ -354,9 +354,9 @@  discard block
 block discarded – undo
354 354
     /**
355 355
      * Set active state for current module.
356 356
      *
357
-     * @param $active
357
+     * @param integer $active
358 358
      *
359
-     * @return bool
359
+     * @return integer
360 360
      */
361 361
     public function setActive($active)
362 362
     {
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
     {
187 187
         $lowerName = $this->getLowerName();
188 188
 
189
-        $langPath = $this->getPath() . '/Resources/lang';
189
+        $langPath = $this->getPath().'/Resources/lang';
190 190
 
191 191
         if (is_dir($langPath)) {
192 192
             $this->loadTranslationsFrom($langPath, $lowerName);
@@ -206,8 +206,8 @@  discard block
 block discarded – undo
206 206
             $file = 'module.json';
207 207
         }
208 208
 
209
-        return array_get($this->moduleJson, $file, function () use ($file) {
210
-            return $this->moduleJson[$file] = new Json($this->getPath() . '/' . $file, $this->app['files']);
209
+        return array_get($this->moduleJson, $file, function() use ($file) {
210
+            return $this->moduleJson[$file] = new Json($this->getPath().'/'.$file, $this->app['files']);
211 211
         });
212 212
     }
213 213
 
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
      */
259 259
     protected function fireEvent($event)
260 260
     {
261
-        $this->app['events']->fire(sprintf('modules.%s.' . $event, $this->getLowerName()), [$this]);
261
+        $this->app['events']->fire(sprintf('modules.%s.'.$event, $this->getLowerName()), [$this]);
262 262
     }
263 263
     /**
264 264
      * Register the aliases from this module.
@@ -283,7 +283,7 @@  discard block
 block discarded – undo
283 283
     protected function registerFiles()
284 284
     {
285 285
         foreach ($this->get('files', []) as $file) {
286
-            include $this->path . '/' . $file;
286
+            include $this->path.'/'.$file;
287 287
         }
288 288
     }
289 289
 
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
      */
407 407
     public function getExtraPath(string $path) : string
408 408
     {
409
-        return $this->getPath() . '/' . $path;
409
+        return $this->getPath().'/'.$path;
410 410
     }
411 411
 
412 412
     /**
Please login to merge, or discard this patch.