3028868
/
SE1_Team_2
5
5
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Compare commits

base: 3028868:54c1a482b462558d0bba9795e2859c79e019850c
Branches Tags
3028868:main
3028868:feature/produktverwaltung
3028868:Programmoberfläche
3028868:feature/kundenverwaltung
..
compare: 3028868:d0a53379dbf23fef6b18cf3759812989961cb6b8
Branches Tags
3028868:main
3028868:feature/produktverwaltung
3028868:Programmoberfläche
3028868:feature/kundenverwaltung

No commits in common. "54c1a482b462558d0bba9795e2859c79e019850c" and "d0a53379dbf23fef6b18cf3759812989961cb6b8" have entirely different histories.
54c1a482b4 ... d0a53379db

1 changed files with 17 additions and 14 deletions
Show Stats Expand all files Collapse all files

29
src/main/java/de/hsmannheim/faktura/dokumente/service/DokumentServiceImpl.java
View File

@ -38,9 +38,9 @@ public class DokumentServiceImpl implements DokumentService {
* Alle Abhängigkeiten werden injiziert (Dependency Injection).
*/
public DokumentServiceImpl(DokumentRepository repository,
ProductLookup productLookup,
CustomerLookup customerLookup,
PdfExportService pdfExportService) {
ProductLookup productLookup,
CustomerLookup customerLookup,
PdfExportService pdfExportService) {
this.repository = repository;
this.productLookup = productLookup;
this.customerLookup = customerLookup;
@ -113,18 +113,21 @@ public class DokumentServiceImpl implements DokumentService {
public Auftragsbestaetigung erstelleAuftragsbestaetigung(String angebotsNummer) {
Angebot angebot = ladeAngebot(angebotsNummer);
// 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.
// 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.
if (repository.existsFolgedokument(angebotsNummer)) {
throw new DuplicateDocumentException(angebotsNummer);
}
// Allgemeiner Status-Check (PH-DP-04): greift bei anderen Nicht-OFFEN-Zuständen.
// Allgemeine Status-Prüfung (PH-DP-04): blockiert auch andere
// Nicht-OFFEN-Zustände (z. B. ABGELEHNT, falls künftig ergänzt).
if (angebot.getStatus() != DokumentStatus.OFFEN) {
throw new IllegalStateException(
"Angebot " + angebotsNummer + " ist nicht mehr OFFEN (Status: " +
angebot.getStatus() + "). Auftragsbestätigung kann nicht erzeugt werden.");
"Angebot " + angebotsNummer + " ist nicht mehr OFFEN (Status: " +
angebot.getStatus() + "). Auftragsbestätigung kann nicht erzeugt werden.");
}
// AB erzeugen – Positionen 1:1 übernehmen
@ -158,8 +161,8 @@ public class DokumentServiceImpl implements DokumentService {
if (ab.getStatus() != DokumentStatus.BESTAETIGT) {
throw new IllegalStateException(
"Auftragsbestätigung " + auftragsNummer + " hat nicht den Status BESTAETIGT " +
"(aktueller Status: " + ab.getStatus() + ").");
"Auftragsbestätigung " + auftragsNummer + " hat nicht den Status BESTAETIGT " +
"(aktueller Status: " + ab.getStatus() + ").");
}
// Positionen ohne Preis-Snapshot (PH-DP-07)
@ -191,8 +194,8 @@ public class DokumentServiceImpl implements DokumentService {
if (ls.getStatus() != DokumentStatus.GELIEFERT) {
throw new IllegalStateException(
"Lieferschein " + lieferscheinNummer + " hat nicht den Status GELIEFERT " +
"(aktueller Status: " + ls.getStatus() + ").");
"Lieferschein " + lieferscheinNummer + " hat nicht den Status GELIEFERT " +
"(aktueller Status: " + ls.getStatus() + ").");
}
// Kundendaten für Pflichtangaben holen