Merge pull request #329 from StefanMaron/ImproveHandlingOnNamespaces

Resolve false positive on LC0003/LC0005 with use of namespaces
StefanMaron · Nov 13, 2023 · 6678bbc · 6678bbc
2 parents d2ea8b2 + 4703382
commit 6678bbc
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Design/Rule0003DoNotUseObjectIDsInVariablesOrProperties.cs b/Design/Rule0003DoNotUseObjectIDsInVariablesOrProperties.cs
@@ -68,10 +68,10 @@ private void CheckForObjectIDsInVariablesOrProperties(SyntaxNodeAnalysisContext
 
                 if (property.PropertyKind != PropertyKind.Permissions && property.PropertyKind != PropertyKind.AccessByPermission)
                 {
-                    if (ctx.Node.ToString().Trim('"').ToUpper() != property.ValueText.ToUpper())
+                    if (ctx.Node.GetLastToken().ToString().Trim('"').ToUpper() != property.ValueText.ToUpper())
                         ctx.ReportDiagnostic(Diagnostic.Create(DiagnosticDescriptors.Rule0003DoNotUseObjectIDsInVariablesOrProperties, ctx.Node.GetLocation(), new object[] { ctx.Node.ToString().Trim('"'), property.ValueText }));
 
-                    if (ctx.Node.ToString().Trim('"') != property.ValueText)
+                    if (ctx.Node.GetLastToken().ToString().Trim('"') != property.ValueText)
                         ctx.ReportDiagnostic(Diagnostic.Create(DiagnosticDescriptors.Rule0005VariableCasingShouldNotDIfferFromDeclaration, ctx.Node.GetLocation(), new object[] { property.ValueText, "" }));
                 }
             }