From 84de1e4c57a8c796664ac226f3f64abd0176301d Mon Sep 17 00:00:00 2001
From: Dan D'Avella <dan.davella@pixee.ai>
Date: Thu, 9 Jan 2025 09:23:43 -0500
Subject: [PATCH] Use proper finding IDs for sonar and semgrep SARIF (#956)

---
 src/codemodder/result.py           | 1 +
 src/core_codemods/sonar/results.py | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/codemodder/result.py b/src/codemodder/result.py
index 69d86bba..a5aff86f 100644
--- a/src/codemodder/result.py
+++ b/src/codemodder/result.py
@@ -105,6 +105,7 @@ def from_sarif(
         )
 
     @classmethod
+    @abstractmethod
     def rule_url_from_id(cls, result: dict, run: dict, rule_id: str) -> str:
         raise NotImplementedError
 
diff --git a/src/core_codemods/sonar/results.py b/src/core_codemods/sonar/results.py
index d6f34811..dbbe122a 100644
--- a/src/core_codemods/sonar/results.py
+++ b/src/core_codemods/sonar/results.py
@@ -69,7 +69,7 @@ def from_result(cls, result: dict) -> Self:
             locations=locations,
             codeflows=all_flows,
             finding=Finding(
-                id=rule_id,
+                id=finding_id,
                 rule=Rule(
                     id=rule_id,
                     name=name,