From 83f6d63c271f922bfd167d2f50d5c518b6cd78a1 Mon Sep 17 00:00:00 2001 From: Marcus van Elst Date: Thu, 23 Apr 2026 14:07:37 +0200 Subject: [PATCH] Review-Fix: Korrekte Fehlermeldung bei fehlgeschlagenem DB-Reset MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Wenn startReprocessing() false zurückgibt, wurde bisher ALREADY_RUNNING_HINT angezeigt – obwohl handleReprocessSelected() isRunning() bereits vorab prüft. Das false bedeutet in diesem Kontext einen Reset-Fehler, nicht einen laufenden Run. Neu: REPROCESS_RESET_FAILED_HINT mit erklärender Meldung. Co-Authored-By: Claude Sonnet 4.6 --- .../adapter/in/gui/batchrun/GuiBatchRunTab.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java b/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java index ff1997c..7802963 100644 --- a/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java +++ b/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java @@ -100,6 +100,10 @@ public final class GuiBatchRunTab { /** Spec: hint shown when a second start attempt is made while a run is active. */ static final String ALREADY_RUNNING_HINT = "Ein Verarbeitungslauf ist bereits aktiv."; + /** Shown when the DB-reset before "Erneut verarbeiten" failed for all selected documents. */ + static final String REPROCESS_RESET_FAILED_HINT = + "Fehler: Status-Reset fehlgeschlagen – Mini-Lauf wurde nicht gestartet."; + /** Spec: German startup error shown when the saved configuration is unusable. */ static final String NO_SAVED_CONFIGURATION_HINT = "Bitte speichern Sie die Konfiguration, bevor ein Verarbeitungslauf gestartet wird."; @@ -711,7 +715,9 @@ public final class GuiBatchRunTab { // skipped due to FAILED_FINAL status. boolean started = coordinator.startReprocessing(configPath, snapshot); if (!started) { - showMessage(ALREADY_RUNNING_HINT); + // handleReprocessSelected() already verified isRunning() == false above, + // so a false return here means the DB reset failed for all fingerprints. + showMessage(REPROCESS_RESET_FAILED_HINT); return; } LOG.info("GUI-Erneut-Verarbeiten: Mini-Lauf gestartet für {} Dokument(e), "