marcus/pdf-umbenenner
1
0
Fork 0
Code Issues 14 Releases 2 Activity

Review-Fix: Korrekte Fehlermeldung bei fehlgeschlagenem DB-Reset

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
Marcus van Elst
2026-04-23 14:07:37 +02:00
parent b41b4112c4
commit 83f6d63c27
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java
+7 -1
View File
@@ -100,6 +100,10 @@ public final class GuiBatchRunTab {
/** Spec: hint shown when a second start attempt is made while a run is active. */ /** 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."; 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. */ /** Spec: German startup error shown when the saved configuration is unusable. */
static final String NO_SAVED_CONFIGURATION_HINT = static final String NO_SAVED_CONFIGURATION_HINT =
"Bitte speichern Sie die Konfiguration, bevor ein Verarbeitungslauf gestartet wird."; "Bitte speichern Sie die Konfiguration, bevor ein Verarbeitungslauf gestartet wird.";
@@ -711,7 +715,9 @@ public final class GuiBatchRunTab {
// skipped due to FAILED_FINAL status. // skipped due to FAILED_FINAL status.
boolean started = coordinator.startReprocessing(configPath, snapshot); boolean started = coordinator.startReprocessing(configPath, snapshot);
if (!started) { 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; return;
} }
LOG.info("GUI-Erneut-Verarbeiten: Mini-Lauf gestartet für {} Dokument(e), " LOG.info("GUI-Erneut-Verarbeiten: Mini-Lauf gestartet für {} Dokument(e), "