Compare commits

..

2 Commits

1 changed files with 14 additions and 17 deletions

View File

@ -113,17 +113,14 @@ public class DokumentServiceImpl implements DokumentService {
public Auftragsbestaetigung erstelleAuftragsbestaetigung(String angebotsNummer) { public Auftragsbestaetigung erstelleAuftragsbestaetigung(String angebotsNummer) {
Angebot angebot = ladeAngebot(angebotsNummer); Angebot angebot = ladeAngebot(angebotsNummer);
// Doppelte AB zuerst (PH-DP-05 / MT-DP-06): wenn bereits eine AB für // Duplikat-Check zuerst (PH-DP-05 / MT-DP-06): wenn bereits eine AB existiert,
// dieses Angebot existiert, ist das Angebot zwangsläufig UEBERFUEHRT // ist das Angebot zwangsläufig UEBERFUEHRT und beide Prüfungen würden anschlagen.
// und die Status-Prüfung würde ebenfalls anschlagen. Die Spec verlangt // Die Spec verlangt in diesem Szenario explizit DuplicateDocumentException.
// in diesem Szenario eindeutig DuplicateDocumentException, deshalb
// muss diese Prüfung vor dem allgemeinen Status-Check stehen.
if (repository.existsFolgedokument(angebotsNummer)) { if (repository.existsFolgedokument(angebotsNummer)) {
throw new DuplicateDocumentException(angebotsNummer); throw new DuplicateDocumentException(angebotsNummer);
} }
// Allgemeine Status-Prüfung (PH-DP-04): blockiert auch andere // Allgemeiner Status-Check (PH-DP-04): greift bei anderen Nicht-OFFEN-Zuständen.
// Nicht-OFFEN-Zustände (z. B. ABGELEHNT, falls künftig ergänzt).
if (angebot.getStatus() != DokumentStatus.OFFEN) { if (angebot.getStatus() != DokumentStatus.OFFEN) {
throw new IllegalStateException( throw new IllegalStateException(
"Angebot " + angebotsNummer + " ist nicht mehr OFFEN (Status: " + "Angebot " + angebotsNummer + " ist nicht mehr OFFEN (Status: " +