Konfigurator: Abrechnungs-Integritaet + Motiv-Katalog nur mit Motiven
All checks were successful
Build & Deploy / deploy (push) Successful in 28s
All checks were successful
Build & Deploy / deploy (push) Successful in 28s
Fehlbelastung behoben: PayPal belastete bisher den CLIENT-Quote (quote.gross_total, mit der frei getippten Menge gerechnet), waehrend der Auftrag serverseitig aus entries.length angelegt/bepreist wird. Wichen die auseinander (Menge > 25, Sync pausiert, Menge nachtraeglich geaendert), zahlte der Kunde z. B. fuer 150 Karten, angelegt wurden 15. - PayPal belastet jetzt ausschliesslich den serverseitig bestaetigten Auftragsbetrag (ergebnis.gross_total), nie den Client-Quote. - Checkout wird blockiert, solange entries.length !== quantity, mit klarer Meldung (Extern-/Datei-/preislose Produkte ausgenommen). Motiv: Die Option "Motiv aus unserer Auswahl" wird nur noch angezeigt, wenn fuer das Format Motive hinterlegt sind; sonst automatisch "Ohne Motiv". Vor dem Laden bleibt die Option sichtbar (kein Aufblitzen). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
@@ -306,6 +306,7 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
|
|
||||||
const [motifMode, setMotifMode] = useState<MotifMode>(entwurf.motifMode ?? 'katalog');
|
const [motifMode, setMotifMode] = useState<MotifMode>(entwurf.motifMode ?? 'katalog');
|
||||||
const [motifs, setMotifs] = useState<Motif[]>([]);
|
const [motifs, setMotifs] = useState<Motif[]>([]);
|
||||||
|
const [motifeGeladen, setMotifeGeladen] = useState(false);
|
||||||
const [motifId, setMotifId] = useState<number | null>(entwurf.motifId ?? null);
|
const [motifId, setMotifId] = useState<number | null>(entwurf.motifId ?? null);
|
||||||
// Gestaltungsservice ist jetzt eine der sich gegenseitig ausschließenden
|
// Gestaltungsservice ist jetzt eine der sich gegenseitig ausschließenden
|
||||||
// Motiv-Optionen – daher abgeleitet, kein eigener Zustand mehr.
|
// Motiv-Optionen – daher abgeleitet, kein eigener Zustand mehr.
|
||||||
@@ -359,7 +360,7 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
|
|
||||||
const [sendet, setSendet] = useState(false);
|
const [sendet, setSendet] = useState(false);
|
||||||
const [bestellFehler, setBestellFehler] = useState<string[]>([]);
|
const [bestellFehler, setBestellFehler] = useState<string[]>([]);
|
||||||
const [bestellung, setBestellung] = useState<{ nummer: string; zahlung: string; bezahlt: boolean; extern?: boolean } | null>(null);
|
const [bestellung, setBestellung] = useState<{ nummer: string; zahlung: string; bezahlt: boolean; betrag: number; extern?: boolean } | null>(null);
|
||||||
|
|
||||||
const [settings, setSettings] = useState<PublicSettings | null>(null);
|
const [settings, setSettings] = useState<PublicSettings | null>(null);
|
||||||
const [preisKatalog, setPreisKatalog] = useState<Record<string, PriceItem>>({});
|
const [preisKatalog, setPreisKatalog] = useState<Record<string, PriceItem>>({});
|
||||||
@@ -403,7 +404,7 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
|
|
||||||
// … Motive, Einstellungen und Preise dagegen optional. Sie dürfen den
|
// … Motive, Einstellungen und Preise dagegen optional. Sie dürfen den
|
||||||
// Konfigurator nicht blockieren, wenn Felder fehlen (z. B. vor dem Bootstrap).
|
// Konfigurator nicht blockieren, wenn Felder fehlen (z. B. vor dem Bootstrap).
|
||||||
fetchMotifs().then(setMotifs).catch(() => setMotifs([]));
|
fetchMotifs().then(setMotifs).catch(() => setMotifs([])).finally(() => setMotifeGeladen(true));
|
||||||
fetchSettings().then(setSettings).catch(() => setSettings(null));
|
fetchSettings().then(setSettings).catch(() => setSettings(null));
|
||||||
fetchPriceItems()
|
fetchPriceItems()
|
||||||
.then((items) => setPreisKatalog(Object.fromEntries(items.map((i) => [i.key, i]))))
|
.then((items) => setPreisKatalog(Object.fromEntries(items.map((i) => [i.key, i]))))
|
||||||
@@ -502,6 +503,17 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
[motifs, benoetigteAusrichtung],
|
[motifs, benoetigteAusrichtung],
|
||||||
);
|
);
|
||||||
const gewaehltesMotiv = sichtbareMotive.find((m) => m.id === motifId) ?? null;
|
const gewaehltesMotiv = sichtbareMotive.find((m) => m.id === motifId) ?? null;
|
||||||
|
// Katalog-Auswahl nur anbieten, wenn (nach dem Laden) auch Motive hinterlegt
|
||||||
|
// sind. Vor dem Laden bleibt die Option sichtbar (Standard), damit sie nicht
|
||||||
|
// aufblitzt und wieder verschwindet.
|
||||||
|
const katalogVerfuegbar = sichtbareMotive.length > 0;
|
||||||
|
const katalogOptionSichtbar = !motifeGeladen || katalogVerfuegbar;
|
||||||
|
|
||||||
|
// Ist „Motiv aus unserer Auswahl" gewählt, aber es gibt (nach dem Laden) keine
|
||||||
|
// Motive für dieses Format, automatisch auf „Ohne Motiv" wechseln.
|
||||||
|
useEffect(() => {
|
||||||
|
if (motifeGeladen && !katalogVerfuegbar && motifMode === 'katalog') setMotifMode('ohne');
|
||||||
|
}, [motifeGeladen, katalogVerfuegbar, motifMode]);
|
||||||
|
|
||||||
// Produkt aus dem Direktlink (z. B. /briefe) vorwählen, sobald geladen –
|
// Produkt aus dem Direktlink (z. B. /briefe) vorwählen, sobald geladen –
|
||||||
// überschreibt auch ein evtl. zwischengespeichertes anderes Produkt.
|
// überschreibt auch ein evtl. zwischengespeichertes anderes Produkt.
|
||||||
@@ -845,7 +857,7 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
.then(() => {
|
.then(() => {
|
||||||
if (abgebrochen || !paypalRef.current) return;
|
if (abgebrochen || !paypalRef.current) return;
|
||||||
return rendereButtons(paypalRef.current, {
|
return rendereButtons(paypalRef.current, {
|
||||||
betrag: quote?.gross_total ?? 0,
|
betrag: bestellung.betrag,
|
||||||
beschreibung: `Skrift ${gewaehltesProdukt?.name ?? ''} – ${bestellung.nummer}`,
|
beschreibung: `Skrift ${gewaehltesProdukt?.name ?? ''} – ${bestellung.nummer}`,
|
||||||
onApproved: async (referenz) => {
|
onApproved: async (referenz) => {
|
||||||
await confirmPayment(bestellung.nummer, referenz);
|
await confirmPayment(bestellung.nummer, referenz);
|
||||||
@@ -857,7 +869,7 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
.catch((err) => setBestellFehler([err.message ?? 'PayPal konnte nicht geladen werden.']));
|
.catch((err) => setBestellFehler([err.message ?? 'PayPal konnte nicht geladen werden.']));
|
||||||
|
|
||||||
return () => { abgebrochen = true; };
|
return () => { abgebrochen = true; };
|
||||||
}, [bestellung, settings, quote?.gross_total, gewaehltesProdukt?.name]);
|
}, [bestellung, settings, gewaehltesProdukt?.name]);
|
||||||
|
|
||||||
const mwst = quote?.mwst_percent ?? settings?.mwst_percent ?? 19;
|
const mwst = quote?.mwst_percent ?? settings?.mwst_percent ?? 19;
|
||||||
|
|
||||||
@@ -935,8 +947,16 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
if (!paypalMoeglich && zahlung === 'paypal') setZahlung('rechnung');
|
if (!paypalMoeglich && zahlung === 'paypal') setZahlung('rechnung');
|
||||||
}, [paypalMoeglich, zahlung]);
|
}, [paypalMoeglich, zahlung]);
|
||||||
|
|
||||||
|
// Sicherheitsnetz gegen Fehlbelastung: Die produzierte Menge (entries_count,
|
||||||
|
// serverseitig = entries.length) MUSS der berechneten/bezahlten Menge
|
||||||
|
// entsprechen. Weicht sie ab (z. B. Excel mit weniger Zeilen als gewählte
|
||||||
|
// Menge, oder nachträglich geänderte Menge), wird die Bestellung blockiert –
|
||||||
|
// sonst zahlt der Kunde für mehr Karten, als tatsächlich angelegt werden.
|
||||||
|
// Extern-/Datei-/Preislose Produkte haben ein anderes Mengenmodell → ausgenommen.
|
||||||
|
const mengeStimmt = externModus || dateiModus || keinePreise || entries.length === quantity;
|
||||||
|
|
||||||
// Ohne Preis (Extern-Modus/Datei-Produkt) gibt es keinen quote – dort reicht die Zustimmung.
|
// Ohne Preis (Extern-Modus/Datei-Produkt) gibt es keinen quote – dort reicht die Zustimmung.
|
||||||
const bestellbereit = agb && datenschutz && final && (!!quote || keinePreise);
|
const bestellbereit = agb && datenschutz && final && (!!quote || keinePreise) && mengeStimmt;
|
||||||
|
|
||||||
async function weiter() {
|
async function weiter() {
|
||||||
// Beim Verlassen des Text-Schritts prüfen, ob das Schriftstück zu viele
|
// Beim Verlassen des Text-Schritts prüfen, ob das Schriftstück zu viele
|
||||||
@@ -1128,6 +1148,10 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
nummer: ergebnis.order_number,
|
nummer: ergebnis.order_number,
|
||||||
zahlung: ergebnis.payment_method,
|
zahlung: ergebnis.payment_method,
|
||||||
bezahlt: ergebnis.payment_method !== 'paypal',
|
bezahlt: ergebnis.payment_method !== 'paypal',
|
||||||
|
// Serverseitig bestätigter Betrag – NUR dieser wird per PayPal belastet,
|
||||||
|
// nie der Client-Quote (der könnte von der tatsächlich angelegten
|
||||||
|
// Auftragsmenge abweichen und den Kunden falsch belasten).
|
||||||
|
betrag: ergebnis.gross_total,
|
||||||
extern: externModus,
|
extern: externModus,
|
||||||
});
|
});
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
@@ -1379,8 +1403,10 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
{/* Einheitliche Radio-Karten in fester Reihenfolge – es ist immer
|
{/* Einheitliche Radio-Karten in fester Reihenfolge – es ist immer
|
||||||
genau eine Option gewählt (auch der Gestaltungsservice). */}
|
genau eine Option gewählt (auch der Gestaltungsservice). */}
|
||||||
<div className="sk-options" style={{ marginTop: 'var(--sk-space-4)' }}>
|
<div className="sk-options" style={{ marginTop: 'var(--sk-space-4)' }}>
|
||||||
<Option name="motif" selected={motifMode === 'katalog'} onSelect={() => setMotifMode('katalog')}
|
{katalogOptionSichtbar && (
|
||||||
title="Motiv aus unserer Auswahl" desc="Wir zeigen Ihnen passende Motive" />
|
<Option name="motif" selected={motifMode === 'katalog'} onSelect={() => setMotifMode('katalog')}
|
||||||
|
title="Motiv aus unserer Auswahl" desc="Wir zeigen Ihnen passende Motive" />
|
||||||
|
)}
|
||||||
<Option name="motif" selected={motifMode === 'ohne'} onSelect={() => setMotifMode('ohne')}
|
<Option name="motif" selected={motifMode === 'ohne'} onSelect={() => setMotifMode('ohne')}
|
||||||
title="Ohne Motiv" desc="Blanko-Karte, nur beschrieben" />
|
title="Ohne Motiv" desc="Blanko-Karte, nur beschrieben" />
|
||||||
<Option name="motif" selected={motifMode === 'upload'} onSelect={() => setMotifMode('upload')}
|
<Option name="motif" selected={motifMode === 'upload'} onSelect={() => setMotifMode('upload')}
|
||||||
@@ -2065,6 +2091,22 @@ export default function Configurator({ vorwahlProdukt }: { vorwahlProdukt?: stri
|
|||||||
</span>
|
</span>
|
||||||
</label>
|
</label>
|
||||||
|
|
||||||
|
{!mengeStimmt && (
|
||||||
|
<div role="alert" style={{
|
||||||
|
border: '1px solid var(--sk-danger, #c62828)',
|
||||||
|
background: 'rgba(198,40,40,0.08)',
|
||||||
|
color: 'var(--sk-danger, #c62828)',
|
||||||
|
borderRadius: 'var(--sk-radius, 10px)',
|
||||||
|
padding: 'var(--sk-space-3) var(--sk-space-4)',
|
||||||
|
margin: 'var(--sk-space-4) 0 0',
|
||||||
|
fontSize: 'var(--sk-size-sm)',
|
||||||
|
}}>
|
||||||
|
Ihre hinterlegten Empfänger/Dokumente ({entries.length}) stimmen nicht mit der
|
||||||
|
gewählten Menge ({quantity}) überein. Bitte gleichen Sie die Menge oder Ihre
|
||||||
|
Empfängerliste an, bevor Sie kostenpflichtig bestellen.
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
|
|
||||||
{bestellFehler.length > 0 && (
|
{bestellFehler.length > 0 && (
|
||||||
<ul className="sk-error" style={{ margin: 'var(--sk-space-3) 0 0', paddingLeft: '1.1em' }}>
|
<ul className="sk-error" style={{ margin: 'var(--sk-space-3) 0 0', paddingLeft: '1.1em' }}>
|
||||||
{bestellFehler.map((f) => <li key={f}>{f}</li>)}
|
{bestellFehler.map((f) => <li key={f}>{f}</li>)}
|
||||||
|
|||||||
Reference in New Issue
Block a user