Skip to content

Commit a073ccc

Browse files
revert some old changes
1 parent 3ebc452 commit a073ccc

File tree

5 files changed

+205
-17260
lines changed

5 files changed

+205
-17260
lines changed

cmd/analyze.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -280,6 +280,7 @@ func getToolName(toolName string, version string) string {
280280
}
281281
}
282282
}
283+
283284
return toolName
284285
}
285286

cmd/upload.go

Lines changed: 2 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -83,33 +83,13 @@ func processSarifAndSendResults(sarifPath string, commitUUID string, projectToke
8383
func processSarif(sarif Sarif) [][]map[string]interface{} {
8484
var codacyIssues []map[string]interface{}
8585
var payloads [][]map[string]interface{}
86-
var modifiedType string
86+
8787
for _, run := range sarif.Runs {
8888
var toolName = getToolName(strings.ToLower(run.Tool.Driver.Name), run.Tool.Driver.Version)
8989
tool, patterns := loadsToolAndPatterns(toolName, false)
9090

9191
for _, result := range run.Results {
92-
if toolName == "pmd" || toolName == "pmd-7" {
93-
var language string
94-
var ruleset string
95-
// Extract ruleset and language from rules properties if available
96-
rules := run.Tool.Driver.Rules[result.RuleIndex]
97-
if rules.HelpURI != "" {
98-
if parts := strings.Split(rules.HelpURI, "pmd_rules_"); len(parts) > 1 {
99-
subParts := strings.SplitN(parts[1], "_", 2)
100-
if len(subParts) == 2 {
101-
language = subParts[0]
102-
rulesetPart := strings.SplitN(subParts[1], ".html", 2)[0]
103-
ruleset = rulesetPart
104-
}
105-
}
106-
}
107-
modifiedType = fmt.Sprintf(tool.Prefix+"category_%s_%s_%s", language, ruleset, result.RuleID)
108-
109-
} else {
110-
modifiedType = tool.Prefix + strings.Replace(result.RuleID, "/", "_", -1)
111-
}
112-
92+
modifiedType := tool.Prefix + strings.Replace(result.RuleID, "/", "_", -1)
11393
pattern := getPatternByID(patterns, modifiedType)
11494
if pattern == nil {
11595
fmt.Printf("Rule '%s' doesn't have a direct mapping on Codacy\n", modifiedType)

0 commit comments

Comments
 (0)