krz/domain-dig

an ios app for DNS & SSL analysis

clone: git clone https://gitbay.org/krz/domain-dig.git

23ef8ef9e632b205675b0fda8200635201d2a889

unsigned

author: Christian Cleberg <hello@cleberg.net> · 2026-07-20T21:32:41Z

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(-)

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 {