fix: use absolute filepath inside base rule source (#5629)

ldez · web-flow · commit cb43da6f90dd · 2025-03-25T21:05:24.000+01:00
diff --git a/pkg/result/processors/base_rule.go b/pkg/result/processors/base_rule.go
@@ -83,9 +83,9 @@ func (r *baseRule) matchLinter(issue *result.Issue) bool {
 }
 
 func (r *baseRule) matchSource(issue *result.Issue, lineCache *fsutils.LineCache, log logutils.Log) bool {
-	sourceLine, errSourceLine := lineCache.GetLine(issue.RelativePath, issue.Line())
+	sourceLine, errSourceLine := lineCache.GetLine(issue.FilePath(), issue.Line())
 	if errSourceLine != nil {
-		log.Warnf("Failed to get line %s:%d from line cache: %s", issue.RelativePath, issue.Line(), errSourceLine)
+		log.Warnf("Failed to get line %s:%d from line cache: %s", issue.FilePath(), issue.Line(), errSourceLine)
 		return false // can't properly match
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -83,9 +83,9 @@ func (r baseRule) matchLinter(issue result.Issue) bool {`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`func (r baseRule) matchSource(issue result.Issue, lineCache *fsutils.LineCache, log logutils.Log) bool {`
`86`		`- sourceLine, errSourceLine := lineCache.GetLine(issue.RelativePath, issue.Line())`
	`86`	`+ sourceLine, errSourceLine := lineCache.GetLine(issue.FilePath(), issue.Line())`
`87`	`87`	`if errSourceLine != nil {`
`88`		`- log.Warnf("Failed to get line %s:%d from line cache: %s", issue.RelativePath, issue.Line(), errSourceLine)`
	`88`	`+ log.Warnf("Failed to get line %s:%d from line cache: %s", issue.FilePath(), issue.Line(), errSourceLine)`
`89`	`89`	`return false // can't properly match`
`90`	`90`	`}`
`91`	`91`