Daniel/webshop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(wizard): prevent duplicate register item addition on step transition
All checks were successful
Staging Build / build (push) Successful in 3m4s
Details

Browse Source
This commit is contained in:
DanielS
2026-08-18 16:26:39 +02:00
parent 8f3d6e5569
commit 409466af9b
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
shop/components/order-wizard.tsx
View File

@@ -542,7 +542,12 @@ export function OrderWizard({
const nextStep = () => { const nextStep = () => {
if (step === 3 && allCategoriesFilled && productValidationErrors.length === 0) { if (step === 3 && allCategoriesFilled && productValidationErrors.length === 0) {
if (hasActiveSelection) { const isDirtyNewDevice = deviceName.trim() !== '' || licenseNumber.trim() !== '' || Object.values(moduleQuantities).some(q => q > 0)
if (editingIdx !== null) {
addToBasket()
} else if (basketItems.length === 0 && hasActiveSelection) {
addToBasket()
} else if (isDirtyNewDevice && hasActiveSelection) {
addToBasket() addToBasket()
} }
} }
@@ -738,7 +743,8 @@ export function OrderWizard({
setIsSubmitting(true) setIsSubmitting(true)
try { try {
let finalItems = [...basketItems] let finalItems = [...basketItems]
if (hasActiveSelection && allCategoriesFilled && productValidationErrors.length === 0) { const isDirtyNewDevice = deviceName.trim() !== '' || licenseNumber.trim() !== '' || Object.values(moduleQuantities).some(q => q > 0)
if ((editingIdx !== null || basketItems.length === 0 || isDirtyNewDevice) && hasActiveSelection && allCategoriesFilled && productValidationErrors.length === 0) {
const normLicense = licenseNumber.trim().toUpperCase() const normLicense = licenseNumber.trim().toUpperCase()
if (normLicense) { if (normLicense) {
const isDuplicateInBasket = basketItems.some( const isDuplicateInBasket = basketItems.some(