Fix error check breaking stats (#892)

egibs · web-flow · commit d6b45265f9a0 · 2025-04-29T10:13:39.000-07:00
Signed-off-by: egibs &lt;20933572+egibs@users.noreply.github.com&gt;
diff --git a/pkg/action/scan.go b/pkg/action/scan.go
@@ -756,7 +756,7 @@ func Scan(ctx context.Context, c malcontent.Config) (*malcontent.Report, error)
 		}
 		return true
 	})
-	if scanCtx.Err() != nil && c.Stats && c.Renderer.Name() != "JSON" && c.Renderer.Name() != "YAML" {
+	if scanCtx.Err() == nil && c.Stats && c.Renderer.Name() != "JSON" && c.Renderer.Name() != "YAML" {
 		err = render.Statistics(&c, r)
 		if err != nil {
 			return r, fmt.Errorf("stats: %w", err)

Original file line number	Diff line number	Diff line change
`@@ -756,7 +756,7 @@ func Scan(ctx context.Context, c malcontent.Config) (*malcontent.Report, error)`
`756`	`756`	`}`
`757`	`757`	`return true`
`758`	`758`	`})`
`759`		`- if scanCtx.Err() != nil && c.Stats && c.Renderer.Name() != "JSON" && c.Renderer.Name() != "YAML" {`
	`759`	`+ if scanCtx.Err() == nil && c.Stats && c.Renderer.Name() != "JSON" && c.Renderer.Name() != "YAML" {`
`760`	`760`	`err = render.Statistics(&c, r)`
`761`	`761`	`if err != nil {`
`762`	`762`	`return r, fmt.Errorf("stats: %w", err)`