Completed
Pull Request — master (#1095)
by
unknown
02:49
created
src/Commands/CommandMakeCommand.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
@@ -100,7 +100,7 @@
 block discarded – undo
100 100
 
101 101
         $commandPath = GenerateConfigReader::read('command');
102 102
 
103
-        return $path . $commandPath->getPath() . '/' . $this->getFileName() . '.php';
103
+        return $path.$commandPath->getPath().'/'.$this->getFileName().'.php';
104 104
     }
105 105
 
106 106
     /**
Please login to merge, or discard this patch.
src/Commands/FactoryMakeCommand.php 3 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   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 
72 72
         $factoryPath = GenerateConfigReader::read('factory');
73 73
 
74
-        return $path . $factoryPath->getPath() . '/' . $this->getFileName();
74
+        return $path.$factoryPath->getPath().'/'.$this->getFileName();
75 75
     }
76 76
 
77 77
     /**
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
      */
82 82
     public function getClass()
83 83
     {
84
-        $studlyName = Str::studly( $this->argument($this->argumentName));
84
+        $studlyName = Str::studly($this->argument($this->argumentName));
85 85
 
86 86
         if (!str_contains($studlyName, 'Factory'))
87 87
             $studlyName .= 'Factory';
@@ -105,11 +105,11 @@  discard block
 block discarded – undo
105 105
 
106 106
             $namespace = $this->laravel['modules']->config('namespace');
107 107
 
108
-            $namespace .= '\\' . $module->getStudlyName();
108
+            $namespace .= '\\'.$module->getStudlyName();
109 109
 
110
-            $namespace .= '\\' .  $modules->config('paths.generator.model.path', 'Entities');
110
+            $namespace .= '\\'.$modules->config('paths.generator.model.path', 'Entities');
111 111
 
112
-            $namespace .= '\\' .  str_replace('Factory', '', $this->argument($this->argumentName));
112
+            $namespace .= '\\'.str_replace('Factory', '', $this->argument($this->argumentName));
113 113
 
114 114
             if (class_exists($namespace))
115 115
                 $model = $namespace;
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
      */
136 136
     private function getFileName()
137 137
     {
138
-        return $this->getClass() . '.php';
138
+        return $this->getClass().'.php';
139 139
     }
140 140
 
141 141
 
@@ -169,9 +169,9 @@  discard block
 block discarded – undo
169 169
     {
170 170
         $namespace = $this->laravel['modules']->config('namespace');
171 171
 
172
-        $namespace .= '\\' . $module->getStudlyName();
172
+        $namespace .= '\\'.$module->getStudlyName();
173 173
 
174
-        $namespace .= '\\' . $this->getDefaultNamespace();
174
+        $namespace .= '\\'.$this->getDefaultNamespace();
175 175
 
176 176
         $namespace = str_replace('/', '\\', $namespace);
177 177
 
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -83,8 +83,9 @@  discard block
 block discarded – undo
83 83
     {
84 84
         $studlyName = Str::studly( $this->argument($this->argumentName));
85 85
 
86
-        if (!str_contains($studlyName, 'Factory'))
87
-            $studlyName .= 'Factory';
86
+        if (!str_contains($studlyName, 'Factory')) {
87
+                    $studlyName .= 'Factory';
88
+        }
88 89
 
89 90
         return $studlyName;
90 91
     }
@@ -111,8 +112,9 @@  discard block
 block discarded – undo
111 112
 
112 113
             $namespace .= '\\' .  str_replace('Factory', '', $this->argument($this->argumentName));
113 114
 
114
-            if (class_exists($namespace))
115
-                $model = $namespace;
115
+            if (class_exists($namespace)) {
116
+                            $model = $namespace;
117
+            }
116 118
 
117 119
         }
118 120
 
Please login to merge, or discard this patch.
src/Commands/MiddlewareMakeCommand.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
@@ -78,7 +78,7 @@
 block discarded – undo
78 78
 
79 79
         $middlewarePath = GenerateConfigReader::read('filter');
80 80
 
81
-        return $path . $middlewarePath->getPath() . '/' . $this->getFileName() . '.php';
81
+        return $path.$middlewarePath->getPath().'/'.$this->getFileName().'.php';
82 82
     }
83 83
 
84 84
     /**
Please login to merge, or discard this patch.
src/Commands/PolicyMakeCommand.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
@@ -76,7 +76,7 @@
 block discarded – undo
76 76
 
77 77
         $policyPath = GenerateConfigReader::read('policies');
78 78
 
79
-        return $path . $policyPath->getPath() . '/' . $this->getFileName() . '.php';
79
+        return $path.$policyPath->getPath().'/'.$this->getFileName().'.php';
80 80
     }
81 81
 
82 82
     /**
Please login to merge, or discard this patch.
src/Commands/ProviderMakeCommand.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   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
         /** @var Module $module */
76 76
         $module = $this->laravel['modules']->findOrFail($this->getModuleName());
77 77
 
78
-        return (new Stub('/' . $stub . '.stub', [
78
+        return (new Stub('/'.$stub.'.stub', [
79 79
             'NAMESPACE'         => $this->getClassNamespace($module),
80 80
             'CLASS'             => $this->getClass(),
81 81
             'LOWER_NAME'        => $module->getLowerName(),
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 
101 101
         $generatorPath = GenerateConfigReader::read('provider');
102 102
 
103
-        return $path . $generatorPath->getPath() . '/' . $this->getFileName() . '.php';
103
+        return $path.$generatorPath->getPath().'/'.$this->getFileName().'.php';
104 104
     }
105 105
 
106 106
     /**
Please login to merge, or discard this patch.
src/Commands/RequestMakeCommand.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
@@ -78,7 +78,7 @@
 block discarded – undo
78 78
 
79 79
         $requestPath = GenerateConfigReader::read('request');
80 80
 
81
-        return $path . $requestPath->getPath() . '/' . $this->getFileName() . '.php';
81
+        return $path.$requestPath->getPath().'/'.$this->getFileName().'.php';
82 82
     }
83 83
 
84 84
     /**
Please login to merge, or discard this patch.
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
@@ -76,7 +76,7 @@
 block discarded – undo
76 76
 
77 77
         $rulePath = GenerateConfigReader::read('rules');
78 78
 
79
-        return $path . $rulePath->getPath() . '/' . $this->getFileName() . '.php';
79
+        return $path.$rulePath->getPath().'/'.$this->getFileName().'.php';
80 80
     }
81 81
 
82 82
     /**
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
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 
117 117
         $generatorPath = GenerateConfigReader::read('migration');
118 118
 
119
-        return $path . $generatorPath->getPath() . '/' . $this->getFileName() . '.php';
119
+        return $path.$generatorPath->getPath().'/'.$this->getFileName().'.php';
120 120
     }
121 121
 
122 122
     /**
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
      */
125 125
     private function getFileName()
126 126
     {
127
-        return date('Y_m_d_His_') . $this->getSchemaName();
127
+        return date('Y_m_d_His_').$this->getSchemaName();
128 128
     }
129 129
 
130 130
     /**
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
@@ -53,8 +53,8 @@  discard block
 block discarded – undo
53 53
 
54 54
         $string = '';
55 55
         foreach ($pieces as $i => $piece) {
56
-            if ($i+1 < count($pieces)) {
57
-                $string .= strtolower($piece) . '_';
56
+            if ($i + 1 < count($pieces)) {
57
+                $string .= strtolower($piece).'_';
58 58
             } else {
59 59
                 $string .= Str::plural(strtolower($piece));
60 60
             }
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
     private function handleOptionalMigrationOption()
96 96
     {
97 97
         if ($this->option('migration') === true) {
98
-            $migrationName = 'create_' . $this->createMigrationName() . '_table';
98
+            $migrationName = 'create_'.$this->createMigrationName().'_table';
99 99
             $this->call('module:make-migration', ['name' => $migrationName, 'module' => $this->argument('module')]);
100 100
         }
101 101
     }
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 
129 129
         $modelPath = GenerateConfigReader::read('model');
130 130
 
131
-        return $path . $modelPath->getPath() . '/' . $this->getModelName() . '.php';
131
+        return $path.$modelPath->getPath().'/'.$this->getModelName().'.php';
132 132
     }
133 133
 
134 134
     /**
Please login to merge, or discard this patch.