@autobusal/routes-order 1.33.3 → 1.33.5

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 1.33.5
+
+### Fixed
+
+- **The summary's TICKET line shows the gross fare.** With a coupon it used to show the discounted figure AND a discount line - "TICKET 40 / DISCOUNT -5 / TOTAL 40", arithmetic that doesn't add up on screen. (Audit BOOK-09-b.)
+
+## 1.33.4
+
+### Added
+
+- **Complimentary in the web checkout.** An operator/employee selling on their OWN operator's route is offered the zero-cost Complimentary option the mobile app already had - the route-aware flag comes from `/api/funds/allowed?route=...` and the ownership rule is re-checked server-side at payment time. (Audit decision BOOK-07; needs obtapi >= 2.4.37.)
+
 ## 1.33.3
 
 ### Fixed

package/Checkout/Checkout.tsx

@@ -114,7 +114,7 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
    * answers it from cache - this reads the answer, it does not fetch it a
    * second time.
    */
-  const { data: paymentsData } = useGetPayments();
+  const { data: paymentsData } = useGetPayments(step2.id);
 
   const singleMethod = getAvailablePayments(paymentsData, isPartner ?? false).length === 1;
 
@@ -434,7 +434,12 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
           step1={ step1 }
           step2={ step2 }
           step3={ step3 }
-          fare={ total.display }
+          /* Claude - 2026-08-22 (audit finding BOOK-09-b): the GROSS
+             fare, not the coupon-adjusted `total` - with a coupon the
+             summary used to read "TICKET 40 / DISCOUNT -5 / TOTAL 40",
+             arithmetic that does not add up on screen. Gross minus the
+             discount line now equals the total the buyer pays. */
+          fare={ getTotal(step2, step3).display }
           addons={ addonItems.map(item => ({ label: item.label, value: `${ item.price } ${ currency }` })) }
           coupon={ coupon }
           total={ grandTotal }
@@ -447,6 +452,7 @@ const Checkout = ({ values, step1, step2, step3, isPartner, t, onStore, onBack }
             <Payments
               selected={ action ?? '' }
               isPartner={ isPartner ?? false }
+              routeId={ step2.id }
               t={ t }
               onChange={ onAction }
             />

package/Step5/Payments/Payments.tsx

@@ -1,7 +1,7 @@
 import { useEffect } from 'react';
 import { TFunction } from 'i18next';
 import { RiSecurePaymentLine } from 'react-icons/ri';
-import { FaRegHandPaper, FaMoneyBill } from 'react-icons/fa';
+import { FaRegHandPaper, FaMoneyBill, FaGift } from 'react-icons/fa';
 import { AiOutlineCreditCard } from 'react-icons/ai';
 import { Inline } from '@autobusal/common';
 import { SubTitle } from '../../styles';
@@ -12,12 +12,13 @@ import { getAvailablePayments } from '../utilities';
 interface Props {
   selected: string
   isPartner: boolean
+  routeId?: number | string
   t: TFunction<'common'>
   onChange: (action: string) => void
 }
 
-const Payments = ({ selected, isPartner, t, onChange }: Props): JSX.Element | null => {
-  const { data, isLoading, isSuccess } = useGetPayments();
+const Payments = ({ selected, isPartner, routeId, t, onChange }: Props): JSX.Element | null => {
+  const { data, isLoading, isSuccess } = useGetPayments(routeId);
 
   const available = getAvailablePayments(data, isPartner);
 
@@ -90,6 +91,13 @@ const Payments = ({ selected, isPartner, t, onChange }: Props): JSX.Element | nu
             </ButtonPayment>
           ) }
 
+          { (data?.complimentary && !isPartner) && (
+            <ButtonPayment type="button" $selected={ selected === 'complimentary' } onClick={ () => onChange('complimentary') }>
+              <FaGift />
+              { t('routes_order.step5.actions.complimentary') }
+            </ButtonPayment>
+          ) }
+
           { (data?.bkt || isPartner) && (
             <ButtonPayment type="button" $selected={ selected === 'bkt' } onClick={ () => onChange('bkt') }>
               <AiOutlineCreditCard />

package/Step5/utilities.ts

@@ -51,6 +51,11 @@ export const getAvailablePayments = (data?: ActionData, isPartner = false): stri
   if (data.reserve && !isPartner) { available.push('reserve'); }
   if (data.funds && !isPartner) { available.push('funds', 'cash'); }
 
+  // Claude - 2026-08-22 (audit decision BOOK-07): an operator/employee on
+  // their own operator's route pays nothing - the flag is route-aware
+  // (services.ts) and re-checked server-side at payment time.
+  if (data.complimentary && !isPartner) { available.push('complimentary'); }
+
   if (data.bkt || isPartner) { available.push('bkt'); }
   if (data.stripe || isPartner) { available.push('stripe'); }
   if (data.mollie || isPartner) { available.push('mollie'); }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/routes-order",
-  "version": "1.33.3",
+  "version": "1.33.5",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts"

package/services.ts

@@ -222,12 +222,20 @@ export const useGetCoupon = (routeId: number, total: number): UseMutationResult<
   })
 );
 
-export const useGetPayments = (): UseQueryResult<ActionData> => (
+/*
+ * Claude - 2026-08-22 (audit decision BOOK-07): asks with the route being
+ * sold, so the API can offer Complimentary to an operator/employee on
+ * their own operator's route. External routes carry string ids and can
+ * never be own inventory, so those simply don't send the parameter.
+ */
+export const useGetPayments = (routeId?: number | string): UseQueryResult<ActionData> => (
   useQuery({
-    queryKey: ['allowed-funds'],
+    queryKey: ['allowed-funds', { routeId }],
     queryFn: async () => (
       await apiClient
-        .get('/api/funds/allowed')
+        .get('/api/funds/allowed', {
+          params: typeof routeId === 'number' ? { route: routeId } : {}
+        })
         .then(response => (
           response.data
         ))

package/types.ts

@@ -42,6 +42,7 @@ export interface ActionData {
   fake: boolean
   reserve: boolean
   funds: boolean
+  complimentary?: boolean
   bkt: boolean
   stripe: boolean
   mollie: boolean