Passed
Pull Request — master (#361)
by Valentin
06:29
created
src/Prototype/src/Command/ListCommand.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
     public function perform(): void
23 23
     {
24 24
         $prototyped = $this->locator->getTargetClasses();
25
-        if ($prototyped === []) {
25
+        if ($prototyped === []){
26 26
             $this->writeln('<comment>No prototyped classes found.</comment>');
27 27
 
28 28
             return;
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 
31 31
         $grid = $this->table(['Class:', 'Property:', 'Target:']);
32 32
 
33
-        foreach ($prototyped as $class) {
33
+        foreach ($prototyped as $class){
34 34
             $proto = $this->getPrototypeProperties($class, $prototyped);
35 35
 
36 36
             $grid->addRow([$class->getName(), $this->mergeNames($proto), $this->mergeTargets($proto)]);
Please login to merge, or discard this patch.
src/Prototype/src/Command/InjectCommand.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     public function perform(): void
33 33
     {
34 34
         $prototyped = $this->locator->getTargetClasses();
35
-        if ($prototyped === []) {
35
+        if ($prototyped === []){
36 36
             $this->writeln('<comment>No prototyped classes found.</comment>');
37 37
 
38 38
             return;
@@ -40,18 +40,18 @@  discard block
 block discarded – undo
40 40
 
41 41
         $targets = [];
42 42
 
43
-        foreach ($prototyped as $class) {
43
+        foreach ($prototyped as $class){
44 44
             $proto = $this->getPrototypeProperties($class, $prototyped);
45
-            if (empty($proto)) {
45
+            if (empty($proto)){
46 46
                 $modified = $this->modify($class, $proto);
47
-                if ($modified !== null) {
47
+                if ($modified !== null){
48 48
                     $targets[] = $modified;
49 49
                 }
50 50
                 continue;
51 51
             }
52 52
 
53
-            foreach ($proto as $target) {
54
-                if ($target instanceof \Throwable) {
53
+            foreach ($proto as $target){
54
+                if ($target instanceof \Throwable){
55 55
                     $targets[] = [
56 56
                         $class->getName(),
57 57
                         $target->getMessage(),
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
                     continue 2;
61 61
                 }
62 62
 
63
-                if ($target === null) {
63
+                if ($target === null){
64 64
                     continue 2;
65 65
                 }
66 66
             }
@@ -68,14 +68,14 @@  discard block
 block discarded – undo
68 68
             $targets[] = [$class->getName(), $this->mergeNames($proto), $this->mergeTargets($proto)];
69 69
 
70 70
             $modified = $this->modify($class, $proto);
71
-            if ($modified !== null) {
71
+            if ($modified !== null){
72 72
                 $targets[] = $modified;
73 73
             }
74 74
         }
75 75
 
76
-        if (!empty($targets)) {
76
+        if (!empty($targets)){
77 77
             $grid = $this->table(['Class:', 'Property:', 'Target:']);
78
-            foreach ($targets as $target) {
78
+            foreach ($targets as $target){
79 79
                 $grid->addRow($target);
80 80
             }
81 81
 
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
     private function modify(\ReflectionClass $class, array $proto): ?array
87 87
     {
88 88
         $classDefinition = $this->extractor->extract($class->getFilename(), $proto);
89
-        try {
89
+        try{
90 90
             $modified = (new Injector())->injectDependencies(
91 91
                 file_get_contents($class->getFileName()),
92 92
                 $classDefinition,
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 
96 96
             file_put_contents($class->getFileName(), $modified);
97 97
             return null;
98
-        } catch (\Throwable $e) {
98
+        }catch (\Throwable $e){
99 99
             return [$class->getName(), $e->getMessage(), "{$e->getFile()}:L{$e->getLine()}"];
100 100
         }
101 101
     }
Please login to merge, or discard this patch.
src/Prototype/src/Command/AbstractCommand.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
         $results = [$this->readProperties($class)];
59 59
 
60 60
         $parent = $class->getParentClass();
61
-        while ($parent instanceof \ReflectionClass && isset($all[$parent->getName()])) {
61
+        while ($parent instanceof \ReflectionClass && isset($all[$parent->getName()])){
62 62
             $results[] = $this->readProperties($parent);
63 63
             $parent = $parent->getParentClass();
64 64
         }
@@ -91,8 +91,8 @@  discard block
 block discarded – undo
91 91
     {
92 92
         $result = [];
93 93
 
94
-        foreach ($properties as $target) {
95
-            if ($target instanceof \Throwable) {
94
+        foreach ($properties as $target){
95
+            if ($target instanceof \Throwable){
96 96
                 $result[] = sprintf(
97 97
                     '<fg=red>%s [f: %s, l: %s]</fg=red>',
98 98
                     $target->getMessage(),
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
                 continue;
103 103
             }
104 104
 
105
-            if ($target === null) {
105
+            if ($target === null){
106 106
                 $result[] = '<fg=yellow>undefined</fg=yellow>';
107 107
                 continue;
108 108
             }
@@ -115,16 +115,16 @@  discard block
 block discarded – undo
115 115
 
116 116
     private function readProperties(\ReflectionClass $class): array
117 117
     {
118
-        if (isset($this->cache[$class->getFileName()])) {
118
+        if (isset($this->cache[$class->getFileName()])){
119 119
             $proto = $this->cache[$class->getFileName()];
120
-        } else {
120
+        }else{
121 121
             $proto = $this->getExtractor()->getPrototypeProperties(file_get_contents($class->getFilename()));
122 122
             $this->cache[$class->getFileName()] = $proto;
123 123
         }
124 124
 
125 125
         $result = [];
126
-        foreach ($proto as $name) {
127
-            if (!isset($result[$name])) {
126
+        foreach ($proto as $name){
127
+            if (!isset($result[$name])){
128 128
                 $result[$name] = $this->registry->resolveProperty($name);
129 129
             }
130 130
         }
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 
135 135
     private function reverse(array $results): ?\Generator
136 136
     {
137
-        foreach (array_reverse($results) as $result) {
137
+        foreach (array_reverse($results) as $result){
138 138
             yield from $result;
139 139
         }
140 140
     }
Please login to merge, or discard this patch.