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