bug #4586 BracesFixer - handle dynamic static method call (SpacePossum)

This PR was merged into the 2.15 branch.

Discussion
----------

BracesFixer - handle dynamic static method call

closes #4451

Commits
-------

239938d BracesFixer - handle dynamic static method call

Author: keradus

src/Fixer/Basic/BracesFixer.php

@@ -424,7 +424,7 @@ static function ($item) {
                         // and it is not a `Foo::{bar}()` situation
                         !($nestToken->equals('}') && $nextNonWhitespaceNestToken->equals('(')) &&
                         // and it is not a `${"a"}->...` and `${"b{$foo}"}->...` situation
-                        !($nestToken->equals('}') && $tokens[$nestIndex - 1]->equalsAny(['"', "'", [T_CONSTANT_ENCAPSED_STRING]])) &&
+                        !($nestToken->equals('}') && $tokens[$nestIndex - 1]->equalsAny(['"', "'", [T_CONSTANT_ENCAPSED_STRING], [T_VARIABLE]])) &&
                         // and next token is not a closing tag that would break heredoc/nowdoc syntax
                         !($tokens[$nestIndex - 1]->isGivenKind(T_END_HEREDOC) && $nextNonWhitespaceNestToken->isGivenKind(T_CLOSE_TAG))
                     ) {

tests/Fixer/Basic/BracesFixerTest.php

@@ -5248,4 +5248,19 @@ function foo()
             ],
         ];
     }
+
+    public function testDynamicStaticMethodCallNotTouched()
+    {
+        $this->doTest(
+            '<?php
+SomeClass::{$method}(new \stdClass());
+SomeClass::{\'test\'}(new \stdClass());
+
+function example()
+{
+    SomeClass::{$method}(new \stdClass());
+    SomeClass::{\'test\'}(new \stdClass());
+}'
+        );
+    }
 }