@ticketboothapp/booking 1.2.78 → 1.2.79

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ticketboothapp/booking",
-  "version": "1.2.78",
+  "version": "1.2.79",
   "private": false,
   "sideEffects": [
     "**/*.css",

package/src/components/booking/AdminChangeBookingFlow.tsx

@@ -936,6 +936,9 @@ export function AdminChangeBookingFlow({
   } | null>(null);
   const [changeQuoteLoading, setChangeQuoteLoading] = useState(false);
   const [changeQuoteFetchError, setChangeQuoteFetchError] = useState<string | null>(null);
+  /** Dedupe quote calls while user input payload is unchanged. */
+  const lastCompletedQuoteInputsKeyRef = useRef<string | null>(null);
+  const inFlightQuoteInputsKeyRef = useRef<string | null>(null);
   const selfServePricingConfirmed =
     suppressSelfServeCurrencyUi &&
     latestChangeQuote != null &&
@@ -3696,6 +3699,40 @@ export function AdminChangeBookingFlow({
     !!lastName.trim();
 
   const isChangeQuoteBlocked = isCustomerSelfServeChange && latestChangeQuote?.canProceed === false;
+  const changeQuoteInputsKey = useMemo(() => JSON.stringify({
+    bookingReference: initialValues?.bookingReference?.trim() ?? '',
+    lastName: lastName.trim().toLowerCase(),
+    optionId: selectedAvailability?.productOptionId?.trim() || activeOptions[0]?.optionId || '',
+    dateTime: selectedAvailability?.dateTime ?? '',
+    availabilityId: selectedAvailability?.availabilityId ?? null,
+    pickupLocationId: pickupLocationId ?? null,
+    returnAvailabilityId: selectedReturnOption?.returnAvailabilityId ?? null,
+    quantities,
+    addOnSelections,
+    adminCustomLinesAsAdditionalAdjustments,
+    feReceiptSubtotal: adminFeAuthoritativeReceipt.subtotal,
+    feReceiptTax: adminFeAuthoritativeReceipt.tax,
+    feReceiptTotal: adminFeAuthoritativeReceipt.total,
+    feReceiptLineItems: adminFeAuthoritativeReceipt.lineItems,
+    useAdminFeAuthoritativeQuote,
+  }), [
+    initialValues?.bookingReference,
+    lastName,
+    selectedAvailability?.productOptionId,
+    selectedAvailability?.dateTime,
+    selectedAvailability?.availabilityId,
+    activeOptions,
+    pickupLocationId,
+    selectedReturnOption?.returnAvailabilityId,
+    quantities,
+    addOnSelections,
+    adminCustomLinesAsAdditionalAdjustments,
+    adminFeAuthoritativeReceipt.subtotal,
+    adminFeAuthoritativeReceipt.tax,
+    adminFeAuthoritativeReceipt.total,
+    adminFeAuthoritativeReceipt.lineItems,
+    useAdminFeAuthoritativeQuote,
+  ]);
   const requiresReturnInChangeFlow = isCustomerSelfServeChange && !!initialValues?.returnAvailabilityId?.trim();
   const missingRequiredReturnSelection = requiresReturnInChangeFlow && !selectedReturnOption;
 
@@ -4174,6 +4211,17 @@ export function AdminChangeBookingFlow({
       setLatestChangeQuote(null);
       setChangeQuoteLoading(false);
       setChangeQuoteFetchError(null);
+      lastCompletedQuoteInputsKeyRef.current = null;
+      inFlightQuoteInputsKeyRef.current = null;
+      return;
+    }
+
+    if (
+      changeQuoteInputsKey &&
+      (inFlightQuoteInputsKeyRef.current === changeQuoteInputsKey ||
+        lastCompletedQuoteInputsKeyRef.current === changeQuoteInputsKey)
+    ) {
+      setChangeQuoteLoading(false);
       return;
     }
 
@@ -4192,6 +4240,7 @@ export function AdminChangeBookingFlow({
     const bookingReferenceForQuote = initialValues.bookingReference.trim();
     const timer = window.setTimeout(() => {
       void (async () => {
+        inFlightQuoteInputsKeyRef.current = changeQuoteInputsKey;
         const bookingItems = Object.entries(quantities)
           .filter(([, count]) => count > 0)
           .map(([category, count]) => ({ category, count }));
@@ -4218,11 +4267,18 @@ export function AdminChangeBookingFlow({
               previousReturnAvailabilityId: initialValues.returnAvailabilityId ?? null,
             },
           };
+          const deterministicClientProposedTotal = useAdminFeAuthoritativeQuote
+            ? adminFeAuthoritativeReceipt.total
+            : changeFlowNewBookingTotal;
+          const deterministicAmountDue = originalReceipt
+            ? roundMoney(adminFeAuthoritativeReceipt.total - originalReceipt.total)
+            : roundMoney(changeFlowClientEstimateDue);
           const quote = useAdminFeAuthoritativeQuote
             ? await quoteChangeBookingAdminFeReceipt({
                 ...quoteRequestBase,
                 feReceipt: adminFeAuthoritativeReceipt,
-                feAmountDueMajorUnits: roundMoney(changeFlowClientEstimateDue),
+                feAmountDueMajorUnits: deterministicAmountDue,
+                clientProposedTotal: deterministicClientProposedTotal,
               })
             : await quoteChangeBooking(quoteRequestBase);
           if (seq !== changeQuoteRequestSeq.current) return;
@@ -4242,11 +4298,16 @@ export function AdminChangeBookingFlow({
               tax: effectiveTax,
             }, currency),
           );
+          lastCompletedQuoteInputsKeyRef.current = changeQuoteInputsKey;
         } catch (e) {
           if (seq !== changeQuoteRequestSeq.current) return;
           setLatestChangeQuote(null);
           setChangeQuoteFetchError(e instanceof Error ? e.message : 'Failed to get price for this change');
+          lastCompletedQuoteInputsKeyRef.current = null;
         } finally {
+          if (inFlightQuoteInputsKeyRef.current === changeQuoteInputsKey) {
+            inFlightQuoteInputsKeyRef.current = null;
+          }
           if (seq === changeQuoteRequestSeq.current) {
             setChangeQuoteLoading(false);
           }
@@ -4283,6 +4344,8 @@ export function AdminChangeBookingFlow({
     adminCustomLinesAsAdditionalAdjustments,
     adminFeAuthoritativeReceipt,
     useAdminFeAuthoritativeQuote,
+    changeQuoteInputsKey,
+    originalReceipt?.total,
   ]);
 
   // Auto-select product option when date is selected: most popular if set, otherwise first available.
@@ -4817,13 +4880,19 @@ export function AdminChangeBookingFlow({
             ? { manualLineAdjustments: adminCustomLinesAsAdditionalAdjustments }
             : {}),
           clientProposedTotal:
-            latestChangeQuote?.serverDisplay?.total ?? changeFlowNewBookingTotal,
+            useAdminFeAuthoritativeQuote
+              ? adminFeAuthoritativeReceipt.total
+              : (latestChangeQuote?.serverDisplay?.total ?? changeFlowNewBookingTotal),
         };
         const quote = useAdminFeAuthoritativeQuote
           ? await quoteChangeBookingAdminFeReceipt({
               ...quoteRequestBase,
               feReceipt: adminFeAuthoritativeReceipt,
-              feAmountDueMajorUnits: roundMoney(changeFlowClientEstimateDue),
+              feAmountDueMajorUnits: roundMoney(
+                originalReceipt
+                  ? adminFeAuthoritativeReceipt.total - originalReceipt.total
+                  : changeFlowClientEstimateDue
+              ),
             })
           : await quoteChangeBooking(quoteRequestBase);
         const quoteSlice = sliceChangeQuoteForUi(