From 0c5774f69507a9c7dc45a7c2c13efa526da2b2be Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Tr=C3=B6ger?= <martin@troeger-online.net>
Date: Mon, 20 Apr 2026 12:47:10 +0200
Subject: [PATCH] install.sh: interaktive Abfragen bei Installation verbessert
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- .env-Verschiebung aus Projektverzeichnis wird angeboten und bei Bestätigung automatisch ausgeführt
- Lokales Foto-Verzeichnis wird interaktiv abgefragt (mit Hinweis falls es bereits existiert)
- Ausführlicher Hinweis nach Anlegen der Default-.env mit Pflichtfeldern und nächsten Schritten
- bats-Hinweis entfernt (nur für Entwickler relevant)
- Tests: Umlaut-Mismatch in security.bats behoben, teardown() für Lock-Isolation ergänzt

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 install.sh               | 51 +++++++++++++++++++++++++++++++++-------
 tests/helpers/setup.bash |  5 ++++
 tests/security.bats      |  2 +-
 3 files changed, 49 insertions(+), 9 deletions(-)

diff --git a/install.sh b/install.sh
index f4095c0..3c62d94 100755
--- a/install.sh
+++ b/install.sh
@@ -37,8 +37,17 @@ CONFIG_ENV="$CONFIG_DIR/.env"
 
 if [[ -f "$ENV_FILE" ]]; then
     echo "Hinweis: .env im Projektverzeichnis gefunden."
-    echo "  Bitte nach $CONFIG_ENV verschieben:"
-    echo "  cp .env $CONFIG_ENV && chmod 600 $CONFIG_ENV"
+    read -r -p "  Jetzt nach $CONFIG_ENV verschieben? [J/n]: " MOVE_ENV
+    if [[ "${MOVE_ENV,,}" != "n" ]]; then
+        mkdir -p "$CONFIG_DIR"
+        cp "$ENV_FILE" "$CONFIG_ENV"
+        chmod 600 "$CONFIG_ENV"
+        rm "$ENV_FILE"
+        echo "  Erledigt: .env wurde verschoben."
+    else
+        echo "  Nicht verschoben. Bitte manuell ausfuehren:"
+        echo "  cp .env $CONFIG_ENV && chmod 600 $CONFIG_ENV"
+    fi
 fi
 
 if [[ -f "$CONFIG_ENV" ]]; then
@@ -49,6 +58,18 @@ if [[ -f "$CONFIG_ENV" ]]; then
     set +a
 fi
 
+### Lokales Foto-Verzeichnis interaktiv abfragen
+
+echo ""
+if [[ -d "$LOCAL_PHOTO_DIR" ]]; then
+    read -r -p "Lokales Foto-Verzeichnis [${LOCAL_PHOTO_DIR}] (Verzeichnis existiert bereits): " INPUT_PHOTO_DIR
+else
+    read -r -p "Lokales Foto-Verzeichnis [${LOCAL_PHOTO_DIR}]: " INPUT_PHOTO_DIR
+fi
+if [[ -n "$INPUT_PHOTO_DIR" ]]; then
+    LOCAL_PHOTO_DIR="$INPUT_PHOTO_DIR"
+fi
+
 ### Konfiguration anzeigen
 
 echo ""
@@ -84,11 +105,6 @@ if ! command -v zenity >/dev/null 2>&1 && ! command -v kdialog >/dev/null 2>&1;
     echo "  (Ohne Dialog-Tool wird ein Text-Fallback verwendet)"
 fi
 
-if ! command -v bats >/dev/null 2>&1; then
-    echo "Hinweis: 'bats' nicht gefunden (nur fuer Tests benoetigt)."
-    echo "  sudo apt install bats"
-fi
-
 ### Verzeichnisse pruefen
 
 if [ ! -d "$LOCAL_PHOTO_DIR" ]; then
@@ -166,8 +182,27 @@ if [ ! -f "$CONFIG_ENV" ]; then
     cp "$SCRIPT_DIR/.env.example" "$CONFIG_ENV"
     chmod 600 "$CONFIG_ENV"
     echo ""
-    echo "WICHTIG: Konfiguration anpassen:"
+    echo "=========================================================="
+    echo "WICHTIG: Konfiguration anpassen, bevor du Darktable startest"
+    echo "=========================================================="
+    echo ""
+    echo "Eine Vorlage wurde angelegt:"
+    echo "  $CONFIG_ENV"
+    echo ""
+    echo "Mindestens diese Felder musst du eintragen:"
+    echo "  SERVER_USER      - dein SSH-Benutzer auf dem Server"
+    echo "  SERVER_IP        - IP-Adresse oder Hostname des Servers"
+    echo "  SERVER_DB_DIR    - Pfad zur Darktable-Datenbank auf dem Server"
+    echo "  SERVER_PHOTO_DIR - Pfad zum Fotoverzeichnis auf dem Server"
+    echo ""
+    echo "LOCAL_PHOTO_DIR ist bereits auf '${LOCAL_PHOTO_DIR}' gesetzt."
+    echo ""
+    echo "Jetzt bearbeiten:"
     echo "  nano $CONFIG_ENV"
+    echo ""
+    echo "Danach install.sh erneut ausfuehren, damit die Verbindung"
+    echo "zum Server geprueft wird."
+    echo "=========================================================="
 fi
 
 ### Desktop-Shortcuts installieren
diff --git a/tests/helpers/setup.bash b/tests/helpers/setup.bash
index d167205..a7ecbc3 100644
--- a/tests/helpers/setup.bash
+++ b/tests/helpers/setup.bash
@@ -23,6 +23,11 @@ SYNC_BIN=$HOME/.local/bin/darktable_sync.sh
 EOF
 }
 
+# Raeumt nach jedem Test auf (verhindert Lock-Leakage zwischen Tests)
+teardown() {
+    rm -rf "$CONFIG_DIR/sync.lock"
+}
+
 # Fuehrt ein Script mit dem Stubs-Verzeichnis vorne im PATH aus
 run_with_stubs() {
     run env PATH="$STUBS_DIR:$PATH" "$@"
diff --git a/tests/security.bats b/tests/security.bats
index 3b2f6bd..31a5459 100644
--- a/tests/security.bats
+++ b/tests/security.bats
@@ -80,7 +80,7 @@ EOF
     mkdir -p "$CONFIG_DIR/sync.lock"
     run_with_stubs env SSH_STUB_FAIL=0 bash "$SYNC_SCRIPT"
     [ "$status" -eq 1 ]
-    [[ "$output" == *"laeuft bereits"* ]]
+    [[ "$output" == *"läuft bereits"* ]]
     rmdir "$CONFIG_DIR/sync.lock"
 }