From 4686eff0d4b54bc7ec0f77e2e9a5c911bfefd8ba Mon Sep 17 00:00:00 2001
From: nobody <nobody+git-spam@afra-berlin.de>
Date: Sun, 18 Aug 2024 00:40:23 +0200
Subject: [PATCH] FIX: config spec "Fixes only" mode

---
 src/config_spec.rs | 39 +++++++++++++++++++--------------------
 1 file changed, 19 insertions(+), 20 deletions(-)

diff --git a/src/config_spec.rs b/src/config_spec.rs
index c694020..3365dd1 100644
--- a/src/config_spec.rs
+++ b/src/config_spec.rs
@@ -231,27 +231,26 @@ pub fn interactive_check(spec: &CfgSpec, config: KV) -> Result<(), Box<dyn Error
     let mut all_valid: bool = true;
     for group in spec.groups {
         group_header(group.name, group.description);
-        let todo = needs_info(&config, group);
-        // TODO: add distinction between edit all / edit necessary only
-        let choices = if !todo.is_empty() {
-            println!(
-                "{}",
-                style::notice(&format!(
-                    "The following fields need adjustment: {}",
-                    todo.join(", ")
-                ))
-            );
-            &["[F]ixes only", "[E]dit all", "[L]ist all", "[S]kip group"][..]
-        } else {
-            println!(
-                "{}",
-                style::notice(
-                    "This group looks fine. OK to [S]kip, unless you want to adjust values here."
-                )
-            );
-            &["[S]kip group", "[E]dit all", "[L]ist all"][..]
-        };
         loop {
+            let todo = needs_info(&config, group);
+            let choices = if !todo.is_empty() {
+                println!(
+                    "{}",
+                    style::notice(&format!(
+                        "The following fields need adjustment: {}",
+                        todo.join(", ")
+                    ))
+                );
+                &["[F]ixes only", "[E]dit all", "[L]ist all", "[S]kip group"][..]
+            } else {
+                println!(
+                    "{}",
+                    style::notice(
+                        "This group looks fine. OK to [S]kip, unless you want to adjust values here."
+                    )
+                );
+                &["[S]kip group", "[E]dit all", "[L]ist all"][..]
+            };
             let choice = prompt_action(choices);
             match choice {
                 'L' => {