From 23ef8ef9e632b205675b0fda8200635201d2a889 Mon Sep 17 00:00:00 2001 From: Christian Cleberg Date: Mon, 20 Jul 2026 16:32:41 -0500 Subject: fix: extract nested ternary in batch quickStatus Closes the last swift:S3358 in the new-code period. The hasChanges branch nested a severity check inside the impactClassification ternary; split into if/else. No behavior change. --- DomainDig/DomainViewModel.swift | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'DomainDig') diff --git a/DomainDig/DomainViewModel.swift b/DomainDig/DomainViewModel.swift index 256c1e1..f7d1d7e 100644 --- a/DomainDig/DomainViewModel.swift +++ b/DomainDig/DomainViewModel.swift @@ -2975,7 +2975,11 @@ final class DomainViewModel { let riskAssessment = entry?.changeSummary?.riskAssessment ?? DomainInsightEngine.analyze(snapshot: payload.snapshot).riskAssessment let quickStatus: String if entry?.changeSummary?.hasChanges == true { - quickStatus = entry?.changeSummary?.impactClassification == .critical ? "Critical" : (entry?.changeSummary?.severity == .high ? "High" : "Changed") + if entry?.changeSummary?.impactClassification == .critical { + quickStatus = "Critical" + } else { + quickStatus = entry?.changeSummary?.severity == .high ? "High" : "Changed" + } } else if certificateWarningLevel != .none { quickStatus = certificateWarningLevel == .critical ? "Critical" : "Warning" } else if riskAssessment.level == .high { -- cgit v1.2.3