mts-booking-library 3.3.0 → 3.4.0

package/lib/booking/journeyBooking.d.ts

@@ -2,7 +2,7 @@ import { ErrorResponse } from "../types/ErrorResponse";
 import { CreateUpdateCartRequest } from "../types/common/Cart";
 import { EditPassengersDetailsRequest, GetBuyerPassengersDetailsResponse } from "../types/common/Person";
 import { AddReductionRequest } from "../types/common/Reduction";
-import { BusMatrix } from "../types/journeys/BusMatrix";
+import { BusMatrix, SeatChange } from "../types/journeys/BusMatrix";
 import { JourneyCart } from "../types/journeys/JourneyCart";
 import { JourneySearchRequest, JourneySearchResult } from "../types/journeys/JourneySearch";
 import { ReductionTrip } from "../types/journeys/Trip";
@@ -165,11 +165,24 @@ export declare class JourneyBooking extends Booking {
     getAssignedSeats(tripId: number, options?: ApiCallOptions): Promise<ErrorResponse | number[]>;
     /**
      * @description This method shall be called when the user wants to change the assigned seats of a trip.
+     *
+     * Two shapes are supported. Callers should prefer `seatChanges` when
+     * the cart has more than one passenger on the trip — the BE pairs by
+     * {@link SeatChange.oldSeatId}, which uniquely identifies the cart's
+     * underlying TripBusLayoutCell. The legacy `newSeatsIds` form is a
+     * full positional snapshot of every passenger's new seat; the BE
+     * zips it against EF-ordered TripBusLayoutCells, which can reassign
+     * the wrong passenger when the two orderings disagree.
+     *
+     * Send only one of the two arguments per call. If both are present
+     * the BE prefers `seatChanges`.
      * @param tripId The id of the trip for which the seats should be changed
-     * @param newSeatsIds The ids of the new seats, as found the {@link BusMatrix}
+     * @param seatsOrChanges Either the legacy `newSeatsIds` array (every
+     *   passenger's new seat, in order) or an explicit list of
+     *   {@link SeatChange} pairs (only the seats that change).
      * @returns An {@link ErrorResponse} object in case of error, true otherwise.
      */
-    changeSeatsOfTrip(tripId: number, newSeatsIds: number[], options?: ApiCallOptions): Promise<ErrorResponse | boolean>;
+    changeSeatsOfTrip(tripId: number, seatsOrChanges: number[] | SeatChange[], options?: ApiCallOptions): Promise<ErrorResponse | boolean>;
     /**
      * @description This method shall be called when the user wants to retrieve the trips in the cart, for adding a available reduction
      * to a specific trip (rather than to the whole cart).

package/lib/booking/journeyBooking.js

@@ -462,13 +462,26 @@ var JourneyBooking = /** @class */ (function (_super) {
     };
     /**
      * @description This method shall be called when the user wants to change the assigned seats of a trip.
+     *
+     * Two shapes are supported. Callers should prefer `seatChanges` when
+     * the cart has more than one passenger on the trip — the BE pairs by
+     * {@link SeatChange.oldSeatId}, which uniquely identifies the cart's
+     * underlying TripBusLayoutCell. The legacy `newSeatsIds` form is a
+     * full positional snapshot of every passenger's new seat; the BE
+     * zips it against EF-ordered TripBusLayoutCells, which can reassign
+     * the wrong passenger when the two orderings disagree.
+     *
+     * Send only one of the two arguments per call. If both are present
+     * the BE prefers `seatChanges`.
      * @param tripId The id of the trip for which the seats should be changed
-     * @param newSeatsIds The ids of the new seats, as found the {@link BusMatrix}
+     * @param seatsOrChanges Either the legacy `newSeatsIds` array (every
+     *   passenger's new seat, in order) or an explicit list of
+     *   {@link SeatChange} pairs (only the seats that change).
      * @returns An {@link ErrorResponse} object in case of error, true otherwise.
      */
-    JourneyBooking.prototype.changeSeatsOfTrip = function (tripId, newSeatsIds, options) {
+    JourneyBooking.prototype.changeSeatsOfTrip = function (tripId, seatsOrChanges, options) {
         return __awaiter(this, void 0, void 0, function () {
-            var seatSelectionStatus, url;
+            var seatSelectionStatus, usesExplicitShape, body, url;
             var _this = this;
             return __generator(this, function (_a) {
                 // First check that it is possible to call this API
@@ -479,12 +492,19 @@ var JourneyBooking = /** @class */ (function (_super) {
                 if (!seatSelectionStatus || !seatSelectionStatus[0]) {
                     throw Error("The status of the cart does not allow to call this API");
                 }
+                usesExplicitShape = seatsOrChanges.length > 0 && typeof seatsOrChanges[0] === "object";
+                body = {
+                    tripId: tripId,
+                    cartGuid: this.cart.guid
+                };
+                if (usesExplicitShape) {
+                    body.seatChanges = seatsOrChanges;
+                }
+                else {
+                    body.newSeatsIds = seatsOrChanges;
+                }
                 url = "".concat(this.config.API_ENDPOINT, "/v3_booking/carts/seats");
-                return [2 /*return*/, this.callPostApi(url, {
-                        tripId: tripId,
-                        newSeatsIds: newSeatsIds,
-                        cartGuid: this.cart.guid
-                    }, options).then(function (response) {
+                return [2 /*return*/, this.callPostApi(url, body, options).then(function (response) {
                         if ((0, ErrorResponse_1.objectIsMTSErrorResponse)(response))
                             return response;
                         // Update the booking process status

package/lib/index.d.ts

@@ -16,7 +16,7 @@ export { ErrorResponse, objectIsMTSErrorResponse } from "./types/ErrorResponse";
 export { City } from "./types/common/City";
 export { Extra, GetExtrasResponse, GetExtrasForBookingResponse } from "./types/common/Extra";
 export * from "./types/common/dates";
-export { BusLayoutCell, BusMatrix, BusCellTypes, SeatStatus } from "./types/journeys/BusMatrix";
+export { BusLayoutCell, BusMatrix, BusCellTypes, SeatChange, SeatStatus } from "./types/journeys/BusMatrix";
 export { JourneyCart, JourneyBookingType, TripBookingInfo } from "./types/journeys/JourneyCart";
 export { JourneyInfo } from "./types/journeys/JourneyInfo";
 export { JourneySearchRequest, JourneySearchResult, DEFAULT_JOURNEY_SEARCH } from "./types/journeys/JourneySearch";

package/lib/types/journeys/BusMatrix.d.ts

@@ -18,6 +18,17 @@ export type BusLayoutCell = {
  * - On the third dimension, the are the columns.
  */
 export type BusMatrix = BusLayoutCell[][][];
+/**
+ * Explicit (oldSeatId, newSeatId) pair for a single seat change in a
+ * cart. Used by `JourneyBooking.changeSeatsOfTrip` instead of the
+ * legacy positional `newSeatsIds` snapshot — the BE pairs by
+ * `oldSeatId`, so multi-passenger swaps no longer depend on array
+ * order and can never reassign the wrong passenger.
+ */
+export type SeatChange = {
+    oldSeatId: number;
+    newSeatId: number;
+};
 export declare enum BusCellTypes {
     SEAT = "SEAT",
     SELECTED_SEAT = "SELECTED_SEAT",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mts-booking-library",
-  "version": "3.3.0",
+  "version": "3.4.0",
   "description": "Library for using MyTicketSolution Booking API",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",