From 865040845bf804deaa6e5a85ecadacc21aedb8b9 Mon Sep 17 00:00:00 2001
From: mindesbunister <github_service@egonetix.de>
Date: Mon, 6 Oct 2025 09:49:31 +0200
Subject: [PATCH] Fix all ((configured_count++)) instances causing script exit
 during configuration

- Replace ((configured_count++)) with configured_count=$((configured_count + 1))
- This fixes script exiting prematurely after configuring first application
- Issue: ((var++)) returns old value (0) which causes exit with set -euo pipefail
- Fixed 14 instances across the configure_applications_for_tmpfs function
---
 one-button-optimizer.sh | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/one-button-optimizer.sh b/one-button-optimizer.sh
index c5b9eef..855fa09 100755
--- a/one-button-optimizer.sh
+++ b/one-button-optimizer.sh
@@ -952,7 +952,7 @@ configure_applications_for_tmpfs() {
                 chown -R $current_user:$current_user /tmp/tmpfs-cache/browser/firefox
                 
                 success "  ✅ Firefox configured to use tmpfs cache"
-                ((configured_count++))
+                configured_count=$((configured_count + 1))
             fi
         fi
     done
@@ -979,7 +979,7 @@ BRAVEEOF
         chown $current_user:$current_user "$brave_desktop" 2>/dev/null || true
         chmod +x "$brave_desktop" 2>/dev/null || true
         success "  ✅ Brave Browser configured to use tmpfs cache"
-        ((configured_count++))
+        configured_count=$((configured_count + 1))
     fi
     
     # Google Chrome
@@ -1004,7 +1004,7 @@ CHROMEOF
         chown $current_user:$current_user "$chrome_desktop" 2>/dev/null || true
         chmod +x "$chrome_desktop" 2>/dev/null || true
         success "  ✅ Google Chrome configured to use tmpfs cache"
-        ((configured_count++))
+        configured_count=$((configured_count + 1))
     fi
     
     # Chromium
@@ -1029,7 +1029,7 @@ CHROMIUMEOF
         chown $current_user:$current_user "$chromium_desktop" 2>/dev/null || true
         chmod +x "$chromium_desktop" 2>/dev/null || true
         success "  ✅ Chromium configured to use tmpfs cache"
-        ((configured_count++))
+        configured_count=$((configured_count + 1))
     fi
     
     # Development Tools Configuration
@@ -1039,7 +1039,7 @@ CHROMIUMEOF
     # NPM
     if command -v npm &>/dev/null; then
         sudo -u $current_user npm config set cache /tmp/tmpfs-cache/development/npm 2>/dev/null && \
-        success "  ✅ NPM cache configured" && ((configured_count++))
+        success "  ✅ NPM cache configured" && configured_count=$((configured_count + 1))
     fi
     
     # Pip
@@ -1050,7 +1050,7 @@ CHROMIUMEOF
 cache-dir = /tmp/tmpfs-cache/development/pip
 PIPEOF
         chown $current_user:$current_user "$user_home/.config/pip/pip.conf" 2>/dev/null || true
-        success "  ✅ Pip cache configured" && ((configured_count++))
+        success "  ✅ Pip cache configured" && configured_count=$((configured_count + 1))
     fi
     
     # KDE/Plasma Configuration
@@ -1066,7 +1066,7 @@ PIPEOF
         ln -sf /tmp/tmpfs-cache/kde/thumbnails "$user_home/.cache/thumbnails"
         chown -R $current_user:$current_user /tmp/tmpfs-cache/kde/thumbnails
         success "  ✅ KDE thumbnail cache linked to tmpfs"
-        ((configured_count++))
+        configured_count=$((configured_count + 1))
     fi
     
     echo ""