Completed
Pull Request — master (#71)
by
unknown
02:04 queued 13s
created
src/Commands/RouteCommand.php 3 patches
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -1,8 +1,6 @@
 block discarded – undo
1 1
 <?php namespace Wn\Generators\Commands;
2 2
 
3 3
 
4
-use InvalidArgumentException;
5
-
6 4
 class RouteCommand extends BaseCommand {
7 5
 
8 6
 	protected $signature = 'wn:route
Please login to merge, or discard this patch.
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -5,13 +5,13 @@
 block discarded – undo
5 5
 
6 6
 class RouteCommand extends BaseCommand {
7 7
 
8
-	protected $signature = 'wn:route
8
+    protected $signature = 'wn:route
9 9
 		{resource : Name of the resource.}
10 10
         {--controller= : Name of the RESTful controller.}
11 11
         {--laravel= : Use Laravel style route definitions}
12 12
     ';
13 13
 
14
-	protected $description = 'Generates RESTful routes.';
14
+    protected $description = 'Generates RESTful routes.';
15 15
 
16 16
     public function handle()
17 17
     {
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
     protected function getController()
68 68
     {
69 69
         $controller = $this->option('controller');
70
-        if(! $controller){
70
+        if (!$controller) {
71 71
             $controller = ucwords(\Illuminate\Support\Str::plural(\Illuminate\Support\Str::camel($this->argument('resource')))) . 'Controller';
72 72
         }
73 73
         return $controller;
Please login to merge, or discard this patch.
lumen-test/tests/acceptance/RouteCommandCept.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@
 block discarded – undo
62 62
 
63 63
 
64 64
 $I->wantTo('run wn:routes in Lumen 5.3+');
65
-if(!file_exists('./routes')) {
65
+if (!file_exists('./routes')) {
66 66
     mkdir('./routes');
67 67
 }
68 68
 $I->writeToFile('./routes/web.php', '<?php
Please login to merge, or discard this patch.
src/Commands/MigrationCommand.php 2 patches
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
         if(! $file){
38 38
             $file = date('Y_m_d_His_') . $snakeName . '_table';
39 39
             $this->deleteOldMigration($snakeName);
40
-        }else{
40
+        } else{
41 41
             $this->deleteOldMigration($file);
42 42
         }
43 43
 
@@ -47,7 +47,9 @@  discard block
 block discarded – undo
47 47
     protected function deleteOldMigration($fileName)
48 48
     {
49 49
         foreach (new \DirectoryIterator("./database/migrations/") as $fileInfo){
50
-            if($fileInfo->isDot()) continue;
50
+            if($fileInfo->isDot()) {
51
+                continue;
52
+            }
51 53
 
52 54
             if(strpos($fileInfo->getFilename(), $fileName) !== FALSE){
53 55
                 unlink($fileInfo->getPathname());
Please login to merge, or discard this patch.
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -34,10 +34,10 @@  discard block
 block discarded – undo
34 34
             ->get();
35 35
 
36 36
         $file = $this->option('file');
37
-        if(! $file){
37
+        if (!$file) {
38 38
             $file = date('Y_m_d_His_') . $snakeName . '_table';
39 39
             $this->deleteOldMigration($snakeName);
40
-        }else{
40
+        } else {
41 41
             $this->deleteOldMigration($file);
42 42
         }
43 43
 
@@ -46,10 +46,10 @@  discard block
 block discarded – undo
46 46
 
47 47
     protected function deleteOldMigration($fileName)
48 48
     {
49
-        foreach (new \DirectoryIterator("./database/migrations/") as $fileInfo){
50
-            if($fileInfo->isDot()) continue;
49
+        foreach (new \DirectoryIterator("./database/migrations/") as $fileInfo) {
50
+            if ($fileInfo->isDot()) continue;
51 51
 
52
-            if(strpos($fileInfo->getFilename(), $fileName) !== FALSE){
52
+            if (strpos($fileInfo->getFilename(), $fileName) !== FALSE) {
53 53
                 unlink($fileInfo->getPathname());
54 54
             }
55 55
         }
@@ -58,12 +58,12 @@  discard block
 block discarded – undo
58 58
     protected function getSchema()
59 59
     {
60 60
         $schema = $this->option('schema');
61
-        if(! $schema){
61
+        if (!$schema) {
62 62
             return $this->spaces(12) . "// Schema declaration";
63 63
         }
64 64
 
65 65
         $items = $schema;
66
-        if( ! $this->option('parsed')){
66
+        if (!$this->option('parsed')) {
67 67
             $items = $this->getArgumentParser('schema')->parse($schema);
68 68
         }
69 69
 
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
         $name = $parts[0]['name'];
98 98
         $parts[1]['args'] = array_merge(["'{$name}'"], $parts[1]['args']);
99 99
         unset($parts[0]);
100
-        $parts = array_map(function($part){
100
+        $parts = array_map(function($part) {
101 101
             return '->' . $part['name'] . '(' . implode(', ', $part['args']) . ')';
102 102
         }, $parts);
103 103
         return "            \$table" . implode('', $parts) . ';';
@@ -106,12 +106,12 @@  discard block
 block discarded – undo
106 106
     protected function getConstraints()
107 107
     {
108 108
         $keys = $this->option('keys');
109
-        if(! $keys){
109
+        if (!$keys) {
110 110
             return $this->spaces(12) . "// Constraints declaration";
111 111
         }
112 112
 
113 113
         $items = $keys;
114
-        if(! $this->option('parsed')){
114
+        if (!$this->option('parsed')) {
115 115
             $items = $this->getArgumentParser('foreign-keys')->parse($keys);
116 116
         }
117 117
 
@@ -125,10 +125,10 @@  discard block
 block discarded – undo
125 125
 
126 126
     protected function getConstraintDeclaration($key)
127 127
     {
128
-        if(! $key['column']){
128
+        if (!$key['column']) {
129 129
             $key['column'] = 'id';
130 130
         }
131
-        if(! $key['table']){
131
+        if (!$key['table']) {
132 132
             $key['table'] = \Illuminate\Support\Str::plural(substr($key['name'], 0, count($key['name']) - 4));
133 133
         }
134 134
 
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
             ])
141 141
             ->get();
142 142
 
143
-        if($key['on_delete']){
143
+        if ($key['on_delete']) {
144 144
             $constraint .= PHP_EOL . $this->getTemplate('migration/on-constraint')
145 145
                     ->with([
146 146
                         'event' => 'Delete',
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
                     ->get();
150 150
         }
151 151
 
152
-        if($key['on_update']){
152
+        if ($key['on_update']) {
153 153
             $constraint .= PHP_EOL . $this->getTemplate('migration/on-constraint')
154 154
                     ->with([
155 155
                         'event' => 'Update',
Please login to merge, or discard this patch.