File tree Expand file tree Collapse file tree 9 files changed +10
-19
lines changed Expand file tree Collapse file tree 9 files changed +10
-19
lines changed Original file line number Diff line number Diff line change @@ -91,8 +91,7 @@ public abstract function b();
9191}
9292
9393PHP
94- ]
95- ,
94+ ],
9695
9796 'Multiple declarations in different namespaces: prefix each namespace. ' => <<<'PHP'
9897<?php
Original file line number Diff line number Diff line change @@ -82,8 +82,7 @@ class A
8282}
8383
8484PHP
85- ]
86- ,
85+ ],
8786
8887 'Multiple declarations in different namespaces: prefix each namespace. ' => <<<'PHP'
8988<?php
Original file line number Diff line number Diff line change @@ -70,8 +70,7 @@ final class A
7070}
7171
7272PHP
73- ]
74- ,
73+ ],
7574
7675 'Multiple declarations in different namespaces: prefix each namespace. ' => <<<'PHP'
7776<?php
Original file line number Diff line number Diff line change @@ -81,8 +81,7 @@ public function a();
8181}
8282
8383PHP
84- ]
85- ,
84+ ],
8685
8786 'Multiple declarations in different namespaces: prefix each namespace. ' => <<<'PHP'
8887<?php
Original file line number Diff line number Diff line change @@ -103,8 +103,7 @@ class B extends \Humbug\Foo\A
103103}
104104
105105PHP
106- ]
107- ,
106+ ],
108107
109108 'Declaration in a different namespace imported via a use statement: prefix the namespace. ' => <<<'PHP'
110109<?php
Original file line number Diff line number Diff line change @@ -85,8 +85,7 @@ public function a()
8585}
8686
8787PHP
88- ]
89- ,
88+ ],
9089
9190 'Multiple declarations in different namespaces: prefix each namespace. ' => <<<'PHP'
9291<?php
Original file line number Diff line number Diff line change @@ -175,8 +175,7 @@ class B
175175}
176176
177177PHP
178- ]
179- ,
178+ ],
180179
181180 'Multiple declarations in different namespaces: prefix each namespace. ' => <<<'PHP'
182181<?php
Original file line number Diff line number Diff line change @@ -97,7 +97,6 @@ private function shouldPrefixStmt(Namespace_ $namespace): bool
9797 return true ;
9898 }
9999 /** @var Class_ $firstStmt */
100-
101100 $ className = (string ) $ namespace ->name .'\\' .$ firstStmt ->name ;
102101
103102 return false === in_array ($ className , $ this ->whitelist );
Original file line number Diff line number Diff line change @@ -46,7 +46,7 @@ final class StringScalarPrefixer extends NodeVisitorAbstract
4646
4747 /**
4848 * @param string $prefix
49- * @param string[] $whitelistedFunctions
49+ * @param string[] $whitelistedFunctions
5050 * @param string[] $whitelist
5151 * @param callable $globalWhitelister
5252 * @param FullyQualifiedNameResolver $nameResolver
@@ -82,7 +82,6 @@ private function shouldPrefixScalar(Node $node): bool
8282 return false ;
8383 }
8484 /** @var String_ $node */
85-
8685 $ parentNode = AppendParentNode::getParent ($ node );
8786
8887 if (false === ($ parentNode instanceof Arg) || false === AppendParentNode::hasParent ($ parentNode )) {
@@ -91,10 +90,10 @@ private function shouldPrefixScalar(Node $node): bool
9190
9291 $ argParent = AppendParentNode::getParent ($ parentNode );
9392
94- return (
93+ return
9594 $ argParent instanceof FuncCall
9695 && in_array ((string ) $ argParent ->name , $ this ->whitelistedFunctions )
97- ) ;
96+ ;
9897 }
9998
10099 private function prefixStringScalar (String_ $ string ): Node
You can’t perform that action at this time.
0 commit comments