Merge pull request #17439 from tamasvajk/feature/include-razor-files-…

…in-fromSource C#: Include `.razor` files in `File::fromSource`
github · Sep 12, 2024 · 9250c29 · 9250c29
2 parents a4c1ec7 + da3c5f4
commit 9250c29
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/csharp/ql/integration-tests/all-platforms/cshtml/Files.ql b/csharp/ql/integration-tests/all-platforms/cshtml/Files.ql
@@ -1,5 +1,5 @@
 import csharp
 
 from File f
-where f.fromSource() or f.getExtension() = "cshtml"
+where f.fromSource()
 select f
diff --git a/csharp/ql/lib/semmle/code/csharp/File.qll b/csharp/ql/lib/semmle/code/csharp/File.qll
@@ -61,7 +61,7 @@ class File extends Container, Impl::File {
 
   /** Holds if this file contains source code. */
   final predicate fromSource() {
-    this.getExtension() = ["cs", "cshtml"] and
+    this.getExtension() = ["cs", "cshtml", "razor"] and
     not this.isStub()
   }