Compare commits
2 Commits
d0a53379db
...
54c1a482b4
| Author | SHA1 | Date |
|---|---|---|
|
|
54c1a482b4 | |
|
|
51619ceb8f |
|
|
@ -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: " +
|
||||
|
|
|
|||
Loading…
Reference in New Issue