Merge pull request #353 from tighten/drift/allow-fqcn-in-views

Allow blade files to have FQCN
tighten · Jan 5, 2024 · 27ccf82 · 27ccf82
2 parents 9471884 + 1f1e198
commit 27ccf82
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/Linters/QualifiedNamesOnlyForClassName.php b/src/Linters/QualifiedNamesOnlyForClassName.php
@@ -14,6 +14,10 @@ class QualifiedNamesOnlyForClassName extends BaseLinter
 
     public function lint(Parser $parser)
     {
+        if (preg_match('/\.blade\.php$/i', $this->filename)) {
+            return [];
+        }
+
         $traverser = new NodeTraverser;
 
         $fqcnExtends = new FindingVisitor(function (Node $node) {

diff --git a/tests/Linting/Linters/QualifiedNamesOnlyForClassNameTest.php b/tests/Linting/Linters/QualifiedNamesOnlyForClassNameTest.php
@@ -220,4 +220,18 @@ public function dispatchStripeEvent(Request \$request)
 
         $this->assertEmpty($lints);
     }
+
+    /** @test */
+    public function does_not_trigger_on_blade_files()
+    {
+        $file = <<<'file'
+            <?php $propertyTypeOptions = array_merge(['' => 'Select a property type'], App\Models\Property::$types); ?>
+        file;
+
+        $lints = (new TLint)->lint(
+            new QualifiedNamesOnlyForClassName($file, '.blade.php')
+        );
+
+        $this->assertEmpty($lints);
+    }
 }