#15 Fix parser with duplicate class label in file content

Author: dimitriBouteille

src/Helpers/Parser.php

@@ -25,10 +25,28 @@ public static function findClassName(?string $fileContent): ?string
             throw new \InvalidArgumentException('The content does not contain PHP code.');
         }
 
-        if (preg_match('#^namespace\s+(.+?);.*class\s+(\w+).+;$#sm', $fileContent, $m)) {
-            return $m[1].'\\'.$m[2];
+        $class = null;
+        $i = 0;
+        for (;$i < count($tokens);$i++) {
+            if ($tokens[$i][0] === T_CLASS) {
+                for ($j = $i + 1;$j < count($tokens);$j++) {
+                    if ($tokens[$j] === '{') {
+                        $class = $tokens[$i + 2][1];
+                    }
+                }
+            }
         }
 
-        return null;
+        if ($class === null || $class === '') {
+            return null;
+        }
+
+        $namespace =  null;
+        if (preg_match('#(^|\s)namespace(.*?)\s*;#sm', $fileContent, $m)) {
+            $namespace = $m[2] ?? null;
+            $namespace = $namespace !== null ? trim($namespace) : null;
+        }
+
+        return $namespace ? $namespace . "\\" . $class : $class;
     }
 }

tests/Helpers/ParserTest.php

@@ -43,7 +43,7 @@ public static function providerFindClassName(): \Generator
             return $fileContent;
         };
 
-        yield 'Light php file' => [
+       yield 'Light php file' => [
             $load('source-1.php'),
             'App\Routes\MyRoute',
         ];
@@ -57,6 +57,11 @@ public static function providerFindClassName(): \Generator
             $load('source-3.php'),
             'App\Routes\MyRoute',
         ];
+
+        yield 'With class string in file content' => [
+            $load('source-4.php'),
+            'App\Routes\MyRoute',
+        ];
     }
 
     /**

tests/fixtures/route-classes/source-4.php

@@ -0,0 +1,26 @@
+<?php
+/**
+ * Copyright (c) 2024 Dimitri BOUTEILLE (https://github.com/dimitriBouteille)
+ * See LICENSE.txt for license details.
+ *
+ * Author: Dimitri BOUTEILLE <[email protected]>
+ */
+
+namespace App\Routes;
+
+use Dbout\WpRestApi\Attributes\Action;
+use Dbout\WpRestApi\Attributes\Route;
+use Dbout\WpRestApi\Enums\Method;
+
+#[Route(
+    namespace: 'app/v2',
+    route: 'document/(?P<documentId>\d+)'
+)]
+class MyRoute
+{
+    #[Action(Method::GET)]
+    public function get(): \WP_REST_Response
+    {
+        throw new \Exception('Invalid builder class type.');
+    }
+}