Passed
Pull Request — master (#335)
by Valentin
04:47 queued 38s
created
src/Prototype/src/NodeVisitors/ClassNode/LocateStatements.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,8 +27,8 @@
 block discarded – undo
27 27
      */
28 28
     public function enterNode(Node $node)
29 29
     {
30
-        if ($node instanceof Node\Stmt\Use_) {
31
-            foreach ($node->uses as $use) {
30
+        if ($node instanceof Node\Stmt\Use_){
31
+            foreach ($node->uses as $use){
32 32
                 $this->imports[] = [
33 33
                     'name'  => implode('\\', $use->name->parts),
34 34
                     'alias' => $use->alias->name ?? null,
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,8 +27,10 @@
 block discarded – undo
27 27
      */
28 28
     public function enterNode(Node $node)
29 29
     {
30
-        if ($node instanceof Node\Stmt\Use_) {
31
-            foreach ($node->uses as $use) {
30
+        if ($node instanceof Node\Stmt\Use_)
31
+        {
32
+            foreach ($node->uses as $use)
33
+            {
32 34
                 $this->imports[] = [
33 35
                     'name'  => implode('\\', $use->name->parts),
34 36
                     'alias' => $use->alias->name ?? null,
Please login to merge, or discard this patch.
src/Prototype/src/NodeVisitors/ClassNode/LocateVariables.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -25,16 +25,16 @@
 block discarded – undo
25 25
      */
26 26
     public function enterNode(Node $node)
27 27
     {
28
-        if ($node instanceof Node\Stmt\Class_) {
29
-            foreach ($node->stmts as $stmt) {
30
-                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name === '__construct') {
28
+        if ($node instanceof Node\Stmt\Class_){
29
+            foreach ($node->stmts as $stmt){
30
+                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name === '__construct'){
31 31
                     return $stmt;
32 32
                 }
33 33
             }
34 34
             return NodeTraverser::DONT_TRAVERSE_CHILDREN;
35 35
         }
36 36
 
37
-        if ($node instanceof Node\Expr\Variable) {
37
+        if ($node instanceof Node\Expr\Variable){
38 38
             $this->vars[] = $node->name;
39 39
         }
40 40
 
Please login to merge, or discard this patch.
Braces   +8 added lines, -4 removed lines patch added patch discarded remove patch
@@ -25,16 +25,20 @@
 block discarded – undo
25 25
      */
26 26
     public function enterNode(Node $node)
27 27
     {
28
-        if ($node instanceof Node\Stmt\Class_) {
29
-            foreach ($node->stmts as $stmt) {
30
-                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name === '__construct') {
28
+        if ($node instanceof Node\Stmt\Class_)
29
+        {
30
+            foreach ($node->stmts as $stmt)
31
+            {
32
+                if ($stmt instanceof Node\Stmt\ClassMethod && $stmt->name === '__construct')
33
+                {
31 34
                     return $stmt;
32 35
                 }
33 36
             }
34 37
             return NodeTraverser::DONT_TRAVERSE_CHILDREN;
35 38
         }
36 39
 
37
-        if ($node instanceof Node\Expr\Variable) {
40
+        if ($node instanceof Node\Expr\Variable)
41
+        {
38 42
             $this->vars[] = $node->name;
39 43
         }
40 44
 
Please login to merge, or discard this patch.
src/Prototype/tests/ClassNode/ConflictResolver/ConflictResolverTest.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
     {
23 23
         $i = new Injector();
24 24
 
25
-        $filename = __DIR__ . '/Fixtures/TestClass.php';
25
+        $filename = __DIR__.'/Fixtures/TestClass.php';
26 26
         $r = $i->injectDependencies(
27 27
             file_get_contents($filename),
28 28
             $this->getDefinition(
@@ -35,16 +35,16 @@  discard block
 block discarded – undo
35 35
             )
36 36
         );
37 37
 
38
-        $this->assertStringContainsString(Fixtures\Test::class . ';', $r);
38
+        $this->assertStringContainsString(Fixtures\Test::class.';', $r);
39 39
         $this->assertRegExp('/@var Test[\s|\r\n]/', $r);
40 40
         $this->assertStringContainsString('@param Test $test', $r);
41 41
 
42
-        $this->assertStringContainsString(Fixtures\SubFolder\Test::class . ' as Test2;', $r);
43
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class . ';', $r);
42
+        $this->assertStringContainsString(Fixtures\SubFolder\Test::class.' as Test2;', $r);
43
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class.';', $r);
44 44
         $this->assertRegExp('/@var Test2[\s|\r\n]/', $r);
45 45
         $this->assertStringContainsString('@param Test2 $test2', $r);
46 46
 
47
-        $this->assertStringContainsString(Fixtures\ATest3::class . ';', $r);
47
+        $this->assertStringContainsString(Fixtures\ATest3::class.';', $r);
48 48
         $this->assertRegExp('/@var ATest3[\s|\r\n]/', $r);
49 49
         $this->assertStringContainsString('@param ATest3 $test3', $r);
50 50
     }
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
      */
58 58
     public static function assertRegExp(string $pattern, string $string, string $message = ''): void
59 59
     {
60
-        if (\method_exists(Assert::class, 'assertMatchesRegularExpression')) {
60
+        if (\method_exists(Assert::class, 'assertMatchesRegularExpression')){
61 61
             Assert::assertMatchesRegularExpression($pattern, $string, $message);
62 62
 
63 63
             return;
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
     {
74 74
         $i = new Injector();
75 75
 
76
-        $filename = __DIR__ . '/Fixtures/TestClassWithImports.php';
76
+        $filename = __DIR__.'/Fixtures/TestClassWithImports.php';
77 77
         $r = $i->injectDependencies(
78 78
             file_get_contents($filename),
79 79
             $this->getDefinition(
@@ -86,18 +86,18 @@  discard block
 block discarded – undo
86 86
             )
87 87
         );
88 88
 
89
-        $this->assertStringContainsString(Fixtures\Test::class . ' as FTest;', $r);
90
-        $this->assertStringNotContainsString(Fixtures\Test::class . ';', $r);
89
+        $this->assertStringContainsString(Fixtures\Test::class.' as FTest;', $r);
90
+        $this->assertStringNotContainsString(Fixtures\Test::class.';', $r);
91 91
         $this->assertRegExp('/@var FTest[\s|\r\n]/', $r);
92 92
         $this->assertStringContainsString('@param FTest $test', $r);
93 93
 
94
-        $this->assertStringContainsString(Fixtures\SubFolder\Test::class . ' as TestAlias;', $r);
95
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class . ';', $r);
94
+        $this->assertStringContainsString(Fixtures\SubFolder\Test::class.' as TestAlias;', $r);
95
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class.';', $r);
96 96
         $this->assertRegExp('/@var TestAlias[\s|\r\n]/', $r);
97 97
         $this->assertStringContainsString('@param TestAlias $test2', $r);
98 98
 
99
-        $this->assertStringContainsString(Fixtures\ATest3::class . ' as ATest;', $r);
100
-        $this->assertStringNotContainsString(Fixtures\ATest3::class . ';', $r);
99
+        $this->assertStringContainsString(Fixtures\ATest3::class.' as ATest;', $r);
100
+        $this->assertStringNotContainsString(Fixtures\ATest3::class.';', $r);
101 101
         $this->assertRegExp('/@var ATest[\s|\r\n]/', $r);
102 102
         $this->assertStringContainsString('@param ATest $test3', $r);
103 103
     }
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
     {
110 110
         $i = new Injector();
111 111
 
112
-        $filename = __DIR__ . '/Fixtures/ChildClass.php';
112
+        $filename = __DIR__.'/Fixtures/ChildClass.php';
113 113
         $r = $i->injectDependencies(
114 114
             file_get_contents($filename),
115 115
             $this->getDefinition(
@@ -122,17 +122,17 @@  discard block
 block discarded – undo
122 122
             )
123 123
         );
124 124
 
125
-        $this->assertStringContainsString(Fixtures\Test::class . ';', $r);
125
+        $this->assertStringContainsString(Fixtures\Test::class.';', $r);
126 126
         $this->assertRegExp('/@var Test[\s|\r\n]/', $r);
127 127
         $this->assertStringContainsString('@param Test $test', $r);
128 128
 
129
-        $this->assertStringContainsString(Fixtures\SubFolder\Test::class . ' as Test2;', $r);
130
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class . ';', $r);
129
+        $this->assertStringContainsString(Fixtures\SubFolder\Test::class.' as Test2;', $r);
130
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class.';', $r);
131 131
         $this->assertRegExp('/@var Test2[\s|\r\n]/', $r);
132 132
         $this->assertStringContainsString('@param Test2 $test2', $r);
133 133
 
134
-        $this->assertStringContainsString(Fixtures\ATest3::class . ' as ATestAlias;', $r);
135
-        $this->assertStringNotContainsString(Fixtures\ATest3::class . ';', $r);
134
+        $this->assertStringContainsString(Fixtures\ATest3::class.' as ATestAlias;', $r);
135
+        $this->assertStringNotContainsString(Fixtures\ATest3::class.';', $r);
136 136
         $this->assertRegExp('/@var ATestAlias[\s|\r\n]/', $r);
137 137
         $this->assertStringContainsString('@param ATestAlias $test3', $r);
138 138
     }
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
     {
142 142
         $i = new Injector();
143 143
 
144
-        $filename = __DIR__ . '/Fixtures/DuplicatePropertyClass.php';
144
+        $filename = __DIR__.'/Fixtures/DuplicatePropertyClass.php';
145 145
         $r = $i->injectDependencies(
146 146
             file_get_contents($filename),
147 147
             $this->getDefinition(
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
             )
153 153
         );
154 154
 
155
-        $this->assertStringContainsString(Fixtures\Test::class . ';', $r);
155
+        $this->assertStringContainsString(Fixtures\Test::class.';', $r);
156 156
         $this->assertRegExp('/@var Test[\s|\r\n]/', $r);
157 157
         $this->assertStringContainsString('@param Test $test', $r);
158 158
         $this->assertStringContainsString('__construct(Test $test)', $r);
Please login to merge, or discard this patch.