spiral / framework

src/Tokenizer/src/Reflection/ReflectionFile.php

Spacing +65 added lines, -65 removed lines

@@ -112,7 +112,7 @@ 
 
     public function __construct(
         private readonly string $filename
-    ) {
+    ){
         $this->tokens = Tokenizer::getTokens($filename);
         $this->countTokens = \count($this->tokens);
 
@@ -141,7 +141,7 @@ 
      */
     public function getClasses(): array
     {
-        if (!isset($this->declarations['T_CLASS'])) {
+        if (!isset($this->declarations['T_CLASS'])){
             return [];
         }
 
@@ -153,7 +153,7 @@ 
      */
     public function getTraits(): array
     {
-        if (!isset($this->declarations['T_TRAIT'])) {
+        if (!isset($this->declarations['T_TRAIT'])){
             return [];
         }
 
@@ -165,7 +165,7 @@ 
      */
     public function getInterfaces(): array
     {
-        if (!isset($this->declarations['T_INTERFACE'])) {
+        if (!isset($this->declarations['T_INTERFACE'])){
             return [];
         }
 
@@ -196,7 +196,7 @@ 
      */
     public function getInvocations(): array
     {
-        if (empty($this->invocations)) {
+        if (empty($this->invocations)){
             $this->locateInvocations($this->getTokens());
         }
 
@@ -224,12 +224,12 @@ 
      */
     protected function locateDeclarations()
     {
-        foreach ($this->getTokens() as $tokenID => $token) {
-            if (!\in_array($token[self::TOKEN_TYPE], self::$processTokens)) {
+        foreach ($this->getTokens() as $tokenID => $token){
+            if (!\in_array($token[self::TOKEN_TYPE], self::$processTokens)){
                 continue;
             }
 
-            switch ($token[self::TOKEN_TYPE]) {
+            switch ($token[self::TOKEN_TYPE]){
                 case T_NAMESPACE:
                     $this->registerNamespace($tokenID);
                     break;
@@ -245,17 +245,17 @@ 
                 case T_CLASS:
                 case T_TRAIT:
                 case T_INTERFACE:
-                    if ($this->isClassNameConst($tokenID)) {
+                    if ($this->isClassNameConst($tokenID)){
                         // PHP5.5 ClassName::class constant
                         continue 2;
                     }
 
-                    if ($this->isAnonymousClass($tokenID)) {
+                    if ($this->isAnonymousClass($tokenID)){
                         // PHP7.0 Anonymous classes new class ('foo', 'bar')
                         continue 2;
                     }
 
-                    if (!$this->isCorrectDeclaration($tokenID)) {
+                    if (!$this->isCorrectDeclaration($tokenID)){
                         // PHP8.0 Named parameters ->foo(class: 'bar')
                         continue 2;
                     }
@@ -272,7 +272,7 @@ 
         }
 
         //Dropping empty namespace
-        if (isset($this->namespaces[''])) {
+        if (isset($this->namespaces[''])){
             $this->namespaces['\\'] = $this->namespaces[''];
             unset($this->namespaces['']);
         }
@@ -286,14 +286,14 @@ 
         $namespace = '';
         $localID = $tokenID + 1;
 
-        do {
+        do{
             $token = $this->tokens[$localID++];
-            if ($token[self::TOKEN_CODE] === '{') {
+            if ($token[self::TOKEN_CODE] === '{'){
                 break;
             }
 
             $namespace .= $token[self::TOKEN_CODE];
-        } while (
+        }while (
             isset($this->tokens[$localID])
             && $this->tokens[$localID][self::TOKEN_CODE] !== '{'
             && $this->tokens[$localID][self::TOKEN_CODE] !== ';'
@@ -303,13 +303,13 @@ 
         $namespace = \trim($namespace);
 
         $uses = [];
-        if (isset($this->namespaces[$namespace])) {
+        if (isset($this->namespaces[$namespace])){
             $uses = $this->namespaces[$namespace];
         }
 
-        if ($this->tokens[$localID][self::TOKEN_CODE] === ';') {
+        if ($this->tokens[$localID][self::TOKEN_CODE] === ';'){
             $endingID = \count($this->tokens) - 1;
-        } else {
+        }else{
             $endingID = $this->endingToken($tokenID);
         }
 
@@ -329,20 +329,20 @@ 
 
         $class = '';
         $localAlias = null;
-        for ($localID = $tokenID + 1; $this->tokens[$localID][self::TOKEN_CODE] !== ';'; ++$localID) {
-            if ($this->tokens[$localID][self::TOKEN_TYPE] == T_AS) {
+        for ($localID = $tokenID + 1; $this->tokens[$localID][self::TOKEN_CODE] !== ';'; ++$localID){
+            if ($this->tokens[$localID][self::TOKEN_TYPE] == T_AS){
                 $localAlias = '';
                 continue;
             }
 
-            if ($localAlias === null) {
+            if ($localAlias === null){
                 $class .= $this->tokens[$localID][self::TOKEN_CODE];
-            } else {
+            }else{
                 $localAlias .= $this->tokens[$localID][self::TOKEN_CODE];
             }
         }
 
-        if (empty($localAlias)) {
+        if (empty($localAlias)){
             $names = explode('\\', $class);
             $localAlias = end($names);
         }
@@ -355,9 +355,9 @@ 
      */
     private function registerFunction(int $tokenID): void
     {
-        foreach ($this->declarations as $declarations) {
-            foreach ($declarations as $location) {
-                if ($tokenID >= $location[self::O_TOKEN] && $tokenID <= $location[self::C_TOKEN]) {
+        foreach ($this->declarations as $declarations){
+            foreach ($declarations as $location){
+                if ($tokenID >= $location[self::O_TOKEN] && $tokenID <= $location[self::C_TOKEN]){
                     //We are inside class, function is method
                     return;
                 }
@@ -365,14 +365,14 @@ 
         }
 
         $localID = $tokenID + 1;
-        while ($this->tokens[$localID][self::TOKEN_TYPE] !== T_STRING) {
+        while ($this->tokens[$localID][self::TOKEN_TYPE] !== T_STRING){
             //Fetching function name
             ++$localID;
         }
 
         $name = $this->tokens[$localID][self::TOKEN_CODE];
-        if (!empty($namespace = $this->activeNamespace($tokenID))) {
-            $name = $namespace . self::NS_SEPARATOR . $name;
+        if (!empty($namespace = $this->activeNamespace($tokenID))){
+            $name = $namespace.self::NS_SEPARATOR.$name;
         }
 
         $this->functions[$name] = [
@@ -388,13 +388,13 @@ 
     private function registerDeclaration(int $tokenID, int $tokenType): void
     {
         $localID = $tokenID + 1;
-        while ($this->tokens[$localID][self::TOKEN_TYPE] !== T_STRING) {
+        while ($this->tokens[$localID][self::TOKEN_TYPE] !== T_STRING){
             ++$localID;
         }
 
         $name = $this->tokens[$localID][self::TOKEN_CODE];
-        if (!empty($namespace = $this->activeNamespace($tokenID))) {
-            $name = $namespace . self::NS_SEPARATOR . $name;
+        if (!empty($namespace = $this->activeNamespace($tokenID))){
+            $name = $namespace.self::NS_SEPARATOR.$name;
         }
 
         $this->declarations[\token_name($tokenType)][$name] = [
@@ -416,7 +416,7 @@ 
     /**
      * Check if token ID represents anonymous class creation, e.g. `new class ('foo', 'bar')`.
      */
-    private function isAnonymousClass(int|string $tokenID): bool
+    private function isAnonymousClass(int | string $tokenID): bool
     {
         return $this->tokens[$tokenID][self::TOKEN_TYPE] === T_CLASS
             && isset($this->tokens[$tokenID - 2])
@@ -426,7 +426,7 @@ 
     /**
      * Check if token ID represents named parameter with name `class`, e.g. `foo(class: SomeClass::name)`.
      */
-    private function isCorrectDeclaration(int|string $tokenID): bool
+    private function isCorrectDeclaration(int | string $tokenID): bool
     {
         return \in_array($this->tokens[$tokenID][self::TOKEN_TYPE], [T_CLASS, T_TRAIT, T_INTERFACE], true)
             && isset($this->tokens[$tokenID + 2])
@@ -459,24 +459,24 @@ 
 
         //Tokens used to re-enable token detection
         $stopTokens = [T_STRING, T_WHITESPACE, T_DOUBLE_COLON, T_OBJECT_OPERATOR, T_NS_SEPARATOR];
-        foreach ($tokens as $tokenID => $token) {
+        foreach ($tokens as $tokenID => $token){
             $tokenType = $token[self::TOKEN_TYPE];
 
             //We are not indexing function declarations or functions called from $objects.
-            if (\in_array($tokenType, [T_FUNCTION, T_OBJECT_OPERATOR, T_NEW])) {
+            if (\in_array($tokenType, [T_FUNCTION, T_OBJECT_OPERATOR, T_NEW])){
                 if (
                     empty($argumentsTID)
                     && (
                         empty($invocationTID)
                         || $this->getSource($invocationTID, $tokenID - 1) !== '$this'
                     )
-                ) {
+                ){
                     //Not a call, function declaration, or object method
                     $ignore = true;
                     continue;
                 }
-            } elseif ($ignore) {
-                if (!\in_array($tokenType, $stopTokens)) {
+            } elseif ($ignore){
+                if (!\in_array($tokenType, $stopTokens)){
                     //Returning to search
                     $ignore = false;
                 }
@@ -484,13 +484,13 @@ 
             }
 
             //We are inside function, and there is "(", indexing arguments.
-            if (!empty($invocationTID) && ($tokenType === '(' || $tokenType === '[')) {
-                if (empty($argumentsTID)) {
+            if (!empty($invocationTID) && ($tokenType === '(' || $tokenType === '[')){
+                if (empty($argumentsTID)){
                     $argumentsTID = $tokenID;
                 }
 
                 ++$level;
-                if ($level != 1) {
+                if ($level != 1){
                     //Not arguments beginning, but arguments part
                     $arguments[$tokenID] = $token;
                 }
@@ -499,16 +499,16 @@ 
             }
 
             //We are inside function arguments and ")" met.
-            if (!empty($invocationTID) && ($tokenType === ')' || $tokenType === ']')) {
+            if (!empty($invocationTID) && ($tokenType === ')' || $tokenType === ']')){
                 --$level;
-                if ($level == -1) {
+                if ($level == -1){
                     $invocationTID = false;
                     $level = 0;
                     continue;
                 }
 
                 //Function fully indexed, we can process it now.
-                if ($level == 0) {
+                if ($level == 0){
                     $this->registerInvocation(
                         $invocationTID,
                         $argumentsTID,
@@ -520,7 +520,7 @@ 
                     //Closing search
                     $arguments = [];
                     $argumentsTID = $invocationTID = false;
-                } else {
+                }else{
                     //Not arguments beginning, but arguments part
                     $arguments[$tokenID] = $token;
                 }
@@ -529,13 +529,13 @@ 
             }
 
             //Still inside arguments.
-            if (!empty($invocationTID) && !empty($level)) {
+            if (!empty($invocationTID) && !empty($level)){
                 $arguments[$tokenID] = $token;
                 continue;
             }
 
             //Nothing valuable to remember, will be parsed later.
-            if (!empty($invocationTID) && \in_array($tokenType, $stopTokens)) {
+            if (!empty($invocationTID) && \in_array($tokenType, $stopTokens)){
                 continue;
             }
 
@@ -545,7 +545,7 @@ 
                 || $tokenType == T_STATIC
                 || $tokenType == T_NS_SEPARATOR
                 || ($tokenType == T_VARIABLE && $token[self::TOKEN_CODE] === '$this')
-            ) {
+            ){
                 $invocationTID = $tokenID;
                 $level = 0;
 
@@ -574,7 +574,7 @@ 
 
         [$class, $operator, $name] = $this->fetchContext($invocationID, $argumentsID);
 
-        if (!empty($operator) && empty($class)) {
+        if (!empty($operator) && empty($class)){
             //Non detectable
             return;
         }
@@ -600,17 +600,17 @@ 
         $name = \trim($this->getSource($invocationTID, $argumentsTID), '( ');
 
         //Let's try to fetch all information we need
-        if (\str_contains($name, '->')) {
+        if (\str_contains($name, '->')){
             $operator = '->';
-        } elseif (\str_contains($name, '::')) {
+        } elseif (\str_contains($name, '::')){
             $operator = '::';
         }
 
-        if (!empty($operator)) {
+        if (!empty($operator)){
             [$class, $name] = \explode($operator, $name);
 
             //We now have to clarify class name
-            if (\in_array($class, ['self', 'static', '$this'])) {
+            if (\in_array($class, ['self', 'static', '$this'])){
                 $class = $this->activeDeclaration($invocationTID);
             }
         }
@@ -623,9 +623,9 @@ 
      */
     private function activeDeclaration(int $tokenID): string
     {
-        foreach ($this->declarations as $declarations) {
-            foreach ($declarations as $name => $position) {
-                if ($tokenID >= $position[self::O_TOKEN] && $tokenID <= $position[self::C_TOKEN]) {
+        foreach ($this->declarations as $declarations){
+            foreach ($declarations as $name => $position){
+                if ($tokenID >= $position[self::O_TOKEN] && $tokenID <= $position[self::C_TOKEN]){
                     return $name;
                 }
             }
@@ -640,8 +640,8 @@ 
      */
     private function activeNamespace(int $tokenID): string
     {
-        foreach ($this->namespaces as $namespace => $position) {
-            if ($tokenID >= $position[self::O_TOKEN] && $tokenID <= $position[self::C_TOKEN]) {
+        foreach ($this->namespaces as $namespace => $position){
+            if ($tokenID >= $position[self::O_TOKEN] && $tokenID <= $position[self::C_TOKEN]){
                 return $namespace;
             }
         }
@@ -662,18 +662,18 @@ 
     private function endingToken(int $tokenID): int
     {
         $level = null;
-        for ($localID = $tokenID; $localID < $this->countTokens; ++$localID) {
+        for ($localID = $tokenID; $localID < $this->countTokens; ++$localID){
             $token = $this->tokens[$localID];
-            if ($token[self::TOKEN_CODE] === '{') {
+            if ($token[self::TOKEN_CODE] === '{'){
                 ++$level;
                 continue;
             }
 
-            if ($token[self::TOKEN_CODE] === '}') {
+            if ($token[self::TOKEN_CODE] === '}'){
                 --$level;
             }
 
-            if ($level === 0) {
+            if ($level === 0){
                 break;
             }
         }
@@ -686,7 +686,7 @@ 
      */
     private function lineNumber(int $tokenID): int
     {
-        while (empty($this->tokens[$tokenID][self::TOKEN_LINE])) {
+        while (empty($this->tokens[$tokenID][self::TOKEN_LINE])){
             --$tokenID;
         }
 
@@ -699,7 +699,7 @@ 
     private function getSource(int $startID, int $endID): string
     {
         $result = '';
-        for ($tokenID = $startID; $tokenID <= $endID; ++$tokenID) {
+        for ($tokenID = $startID; $tokenID <= $endID; ++$tokenID){
             //Collecting function usage src
             $result .= $this->tokens[$tokenID][self::TOKEN_CODE];
         }

src/Tokenizer/tests/Classes/ClassWithHeredoc.php

Spacing +2 added lines, -2 removed lines

@@ -19,13 +19,13 @@
 {
 
 }
-class;FOO;
+class; FOO;
 
         <<<'class'
 class FooBar 
 {
 
 }
-class ;FOO;
+class; FOO;
     }
 }
\ No newline at end of file

src/Tokenizer/tests/ReflectionFileTest.php

Spacing +7 added lines, -7 removed lines

@@ -16,17 +16,17 @@ 
         $this->assertContains(TestTrait::class, $reflection->getTraits());
         $this->assertContains(TestInterface::class, $reflection->getInterfaces());
 
-        $this->assertSame([__NAMESPACE__ . '\hello'], $reflection->getFunctions());
+        $this->assertSame([__NAMESPACE__.'\hello'], $reflection->getFunctions());
 
         $functionA = null;
         $functionB = null;
 
-        foreach ($reflection->getInvocations() as $invocation) {
-            if ($invocation->getName() == 'test_function_a') {
+        foreach ($reflection->getInvocations() as $invocation){
+            if ($invocation->getName() == 'test_function_a'){
                 $functionA = $invocation;
             }
 
-            if ($invocation->getName() == 'test_function_b') {
+            if ($invocation->getName() == 'test_function_b'){
                 $functionB = $invocation;
             }
         }
@@ -53,7 +53,7 @@ 
 
     public function testReflectionFileWithNamedParameters(): void
     {
-        $reflection = new ReflectionFile(__DIR__ . '/Classes/ClassWithNamedParameter.php');
+        $reflection = new ReflectionFile(__DIR__.'/Classes/ClassWithNamedParameter.php');
 
         $this->assertSame([
             'Spiral\Tests\Tokenizer\Classes\ClassWithNamedParameter',
@@ -62,7 +62,7 @@ 
 
     public function testReflectionFileAnonymousClass(): void
     {
-        $reflection = new ReflectionFile(__DIR__ . '/Classes/ClassWithAnonymousClass.php');
+        $reflection = new ReflectionFile(__DIR__.'/Classes/ClassWithAnonymousClass.php');
 
         $this->assertSame([
             'Spiral\Tests\Tokenizer\Classes\ClassWithAnonymousClass',
@@ -71,7 +71,7 @@ 
 
     public function testReflectionFileWithHeredoc(): void
     {
-        $reflection = new ReflectionFile(__DIR__ . '/Classes/ClassWithHeredoc.php');
+        $reflection = new ReflectionFile(__DIR__.'/Classes/ClassWithHeredoc.php');
 
         $this->assertSame([
             'Spiral\Tests\Tokenizer\Classes\ClassWithHeredoc',