if no access modifier exist the field is private

SonarSource · Feb 17, 2023 · 5e94096 · 5e94096
1 parent c906d02
commit 5e94096
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/analyzers/src/SonarAnalyzer.CSharp/Rules/StaticFieldVisible.cs b/analyzers/src/SonarAnalyzer.CSharp/Rules/StaticFieldVisible.cs
@@ -49,7 +49,8 @@ public sealed class StaticFieldVisible : SonarDiagnosticAnalyzer
             : Enumerable.Empty<Diagnostic>();
 
         private static bool FieldIsRelevant(FieldDeclarationSyntax node) =>
-            !node.Modifiers.Any(SyntaxKind.PrivateKeyword)
+            node.Modifiers.Any()
+            && !node.Modifiers.Any(SyntaxKind.PrivateKeyword)
             && node.Modifiers.Any(SyntaxKind.StaticKeyword)
             && !node.Modifiers.Any(SyntaxKind.ReadOnlyKeyword);
 

diff --git a/analyzers/tests/SonarAnalyzer.UnitTest/TestCases/StaticFieldVisible.cs b/analyzers/tests/SonarAnalyzer.UnitTest/TestCases/StaticFieldVisible.cs
@@ -20,8 +20,7 @@ public class StaticFieldVisible
 
         private static double Pi8 = 3.14;
         private double Pi9 = 3.14;
-        static double Pi10 = 3.14; // Noncompliant "internal" is the default if no access modifier is specified
-                                   // https://learn.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/access-modifiers
+        static double Pi10 = 3.14; // Compliant - if not access modifier exist the field is private
         public readonly double Pi11 = 3.14;
 
         [ThreadStatic]