From f128291dfe4932d08ec52f0c6b041b45c584384e Mon Sep 17 00:00:00 2001
From: mindesbunister <github_service@egonetix.de>
Date: Mon, 6 Oct 2025 09:44:49 +0200
Subject: [PATCH] Fix script exit bug caused by ((var++)) with set -e
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Critical bug fix:
- Script was exiting after 'Application Configuration Status:'
- Issue: ((total_browsers++)) returns 1 when var is 0
- With 'set -e', non-zero return causes immediate exit
- Solution: Use var=$((var + 1)) instead of ((var++))

Changed all increment operations:
- ((total_browsers++)) → total_browsers=$((total_browsers + 1))
- ((configured_browsers++)) → configured_browsers=$((configured_browsers + 1))

This prevents script termination and allows proper browser detection
to complete and display configuration status for all browsers.
---
 one-button-optimizer.sh | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/one-button-optimizer.sh b/one-button-optimizer.sh
index 7c9c4a3..c5b9eef 100755
--- a/one-button-optimizer.sh
+++ b/one-button-optimizer.sh
@@ -561,43 +561,43 @@ analyze_and_prompt() {
         
         # Check each browser
         if [[ -d "$user_home/.mozilla/firefox" ]]; then
-            ((total_browsers++))
+            total_browsers=$((total_browsers + 1))
             if grep -q "tmpfs-cache" "$user_home/.mozilla/firefox"/*default*/prefs.js 2>/dev/null; then
                 echo "   ✅ Firefox: Configured"
-                ((configured_browsers++))
+                configured_browsers=$((configured_browsers + 1))
             else
                 echo "   ⚠️  Firefox: Not configured"
             fi
         fi
         
         if [[ -d "$user_home/.config/BraveSoftware" ]]; then
-            ((total_browsers++))
+            total_browsers=$((total_browsers + 1))
             if [[ -f "$user_home/.local/share/applications/brave-browser.desktop" ]] && \
                grep -q "tmpfs-cache" "$user_home/.local/share/applications/brave-browser.desktop" 2>/dev/null; then
                 echo "   ✅ Brave: Configured"
-                ((configured_browsers++))
+                configured_browsers=$((configured_browsers + 1))
             else
                 echo "   ⚠️  Brave: Not configured"
             fi
         fi
         
         if [[ -d "$user_home/.config/google-chrome" ]]; then
-            ((total_browsers++))
+            total_browsers=$((total_browsers + 1))
             if [[ -f "$user_home/.local/share/applications/google-chrome.desktop" ]] && \
                grep -q "tmpfs-cache" "$user_home/.local/share/applications/google-chrome.desktop" 2>/dev/null; then
                 echo "   ✅ Chrome: Configured"
-                ((configured_browsers++))
+                configured_browsers=$((configured_browsers + 1))
             else
                 echo "   ⚠️  Chrome: Not configured"
             fi
         fi
         
         if [[ -d "$user_home/.config/chromium" ]]; then
-            ((total_browsers++))
+            total_browsers=$((total_browsers + 1))
             if [[ -f "$user_home/.local/share/applications/chromium-browser.desktop" ]] && \
                grep -q "tmpfs-cache" "$user_home/.local/share/applications/chromium-browser.desktop" 2>/dev/null; then
                 echo "   ✅ Chromium: Configured"
-                ((configured_browsers++))
+                configured_browsers=$((configured_browsers + 1))
             else
                 echo "   ⚠️  Chromium: Not configured"
             fi