From fa185d4515e5c204a41dcb7922eec6ed3e25fe38 Mon Sep 17 00:00:00 2001
From: Raphael Numbus <raphael@numbus.eu>
Date: Sun, 7 Dec 2025 11:39:23 +0100
Subject: [PATCH] Seems to fix the recap section

---
 deploy.sh | 20 ++++++--------------
 1 file changed, 6 insertions(+), 14 deletions(-)

diff --git a/deploy.sh b/deploy.sh
index bf1a3b5..1c5b9cb 100755
--- a/deploy.sh
+++ b/deploy.sh
@@ -405,10 +405,14 @@ The mirror will be created using the size of the smaller disk, and any extra spa
       elif (( num_selected == 1 )); then # If only one data disk is selected, it is a content disk
         num_content=1
         num_parity=0
+        NUMBER_OF_CONTENT_DISKS=$num_content
+        NUMBER_OF_PARITY_DISKS=$num_parity
       elif (( num_selected > 2 )); then # If more than two data disks are selected
         # 1 parity for up to 2 content disks.
         num_parity=$(( (num_selected + 2) / 3 ))
         num_content=$(( num_selected - num_parity ))
+        NUMBER_OF_CONTENT_DISKS=$num_content
+        NUMBER_OF_PARITY_DISKS=$num_parity
       fi
 
       if (( num_selected != 2 )); then
@@ -445,18 +449,6 @@ The mirror will be created using the size of the smaller disk, and any extra spa
   [[ -n "$BOOT_DISK_1" ]] && NUMBER_OF_BOOT_DISKS=$((NUMBER_OF_BOOT_DISKS + 1)) && export BOOT_DISK_1
   [[ -n "$BOOT_DISK_2" ]] && NUMBER_OF_BOOT_DISKS=$((NUMBER_OF_BOOT_DISKS + 1)) && export BOOT_DISK_2
 
-  NUMBER_OF_CONTENT_DISKS=0
-  for i in {1..6}; do
-    disk_var="CONTENT_DISK_$i"
-    [[ -n "${!disk_var}" ]] && NUMBER_OF_CONTENT_DISKS=$((NUMBER_OF_CONTENT_DISKS + 1))
-  done
-
-  NUMBER_OF_PARITY_DISKS=0
-  for i in {1..3}; do
-    disk_var="PARITY_DISK_$i"
-    [[ -n "${!disk_var}" ]] && NUMBER_OF_PARITY_DISKS=$((NUMBER_OF_PARITY_DISKS + 1))
-  done
-
   RECAP_CONTENT=$(cat <<EOF
 ### Disk Configuration Summary
 
@@ -467,11 +459,11 @@ Please review the selected disk layout before proceeding.
 $( [[ -n "$BOOT_DISK_2" ]] && echo "* **Boot 2:** \`$BOOT_DISK_2\`" || echo "* **Boot 2:** *Not configured*")
 
 **Data Disks ($NUMBER_OF_CONTENT_DISKS):**
-$(for i in {1..6}; do disk_var="CONTENT_DISK_$i"; [[ -n "${!disk_var}" ]] && echo "* **Data $i:** \`${!disk_var}\`"; done)
+$(for i in $(seq 1 $NUMBER_OF_CONTENT_DISKS); do disk_var="CONTENT_DISK_$i"; [[ -n "${!disk_var}" ]] && echo "* **Data $i:** \`${!disk_var}\`"; done)
 $( [[ $NUMBER_OF_CONTENT_DISKS -eq 0 ]] && echo "* *Not configured*")
 
 **Parity Disks ($NUMBER_OF_PARITY_DISKS):**
-$(for i in {1..3}; do disk_var="PARITY_DISK_$i"; [[ -n "${!disk_var}" ]] && echo "* **Parity $i:** \`${!disk_var}\`"; done)
+$(for i in $(seq 1 $NUMBER_OF_PARITY_DISKS); do disk_var="PARITY_DISK_$i"; [[ -n "${!disk_var}" ]] && echo "* **Parity $i:** \`${!disk_var}\`"; done)
 $( [[ $NUMBER_OF_PARITY_DISKS -eq 0 ]] && echo "* *Not configured*")
 EOF
 )