@chevre/domain 21.2.0-alpha.130 → 21.2.0-alpha.131

package/example/src/chevre/searchHoldReservations.ts

@@ -19,18 +19,18 @@ async function main() {
     const stockHolderRepo = new chevre.repository.StockHolder(client, mongoose.connection);
 
     const holdReservations = await stockHolderRepo.search({
-        limit: 100,
-        page: 1
+        limit: 2,
+        page: 2,
+        sort: { 'reservationFor.startDate': chevre.factory.sortType.Ascending }
     });
-    console.log(holdReservations[0]?.reservationFor.aggregateReservation, holdReservations[0]?.objectSize);
+    console.log(holdReservations);
     console.log(holdReservations.length);
 
-    const aggregations = await stockHolderRepo.aggregateByReservationFor({
-        // reservationFor: { id: { $eq: 'xxx' } },
-        limit: 100,
-        page: 1
-    });
-    console.log(aggregations);
+    // const aggregations = await stockHolderRepo.aggregateByReservationFor({
+    //     limit: 100,
+    //     page: 1
+    // });
+    // console.log(aggregations);
 }
 
 main()

package/lib/chevre/repo/mongoose/schemas/holdReservation.js

@@ -60,3 +60,33 @@ schema.index({ 'reservationFor.startDate': -1 }, { name: 'searchByReservationFor
 schema.index({ 'reservationFor.id': 1 }, { name: 'uniqueReservationForId', unique: true });
 schema.index({ 'reservationFor.id': 1, 'reservationFor.startDate': -1 }, { name: 'searchByReservationForId' });
 schema.index({ 'project.id': 1, 'reservationFor.startDate': -1 }, { name: 'searchByProjectId' });
+schema.index({ 'reservations.reservationNumber': 1, 'reservationFor.startDate': -1 }, {
+    name: 'searchByReservationNumber',
+    partialFilterExpression: {
+        'reservations.reservationNumber': { $exists: true }
+    }
+});
+schema.index({ 'reservations.subReservation.id': 1, 'reservationFor.startDate': -1 }, {
+    name: 'searchBySubReservationId',
+    partialFilterExpression: {
+        'reservations.subReservation.id': { $exists: true }
+    }
+});
+schema.index({ 'reservations.subReservation.identifier': 1, 'reservationFor.startDate': -1 }, {
+    name: 'searchBySubReservationIdentifier',
+    partialFilterExpression: {
+        'reservations.subReservation.identifier': { $exists: true }
+    }
+});
+schema.index({ 'reservations.subReservation.reservedTicket.ticketedSeat.seatNumber': 1, 'reservationFor.startDate': -1 }, {
+    name: 'searchBySubReservationSeatNumber',
+    partialFilterExpression: {
+        'reservations.subReservation.reservedTicket.ticketedSeat.seatNumber': { $exists: true }
+    }
+});
+schema.index({ 'reservations.subReservation.reservedTicket.ticketedSeat.seatSection': 1, 'reservationFor.startDate': -1 }, {
+    name: 'searchBySubReservationSeatSection',
+    partialFilterExpression: {
+        'reservations.subReservation.reservedTicket.ticketedSeat.seatSection': { $exists: true }
+    }
+});

package/lib/chevre/repo/stockHolder.d.ts

@@ -43,7 +43,6 @@ export interface ISubReservation {
     };
 }
 export interface IReservationPackage {
-    typeOf: factory.reservationType.ReservationPackage;
     reservationNumber: string;
     subReservation: ISubReservation[];
 }
@@ -148,6 +147,9 @@ export declare class StockHolderRepository {
     search(params: {
         limit?: number;
         page?: number;
+        sort?: {
+            'reservationFor.startDate': factory.sortType;
+        };
         project?: {
             id?: {
                 $eq?: string;

package/lib/chevre/repo/stockHolder.js

@@ -123,7 +123,7 @@ class StockHolderRepository {
                 const reservationCountLte = maximum - addedReservationCount;
                 const subReservations = lockKey.offers.map((offer) => StockHolderRepository.offer2subReservation(offer, lockKey.hasTicketedSeat));
                 const reservationPackage = {
-                    typeOf: factory.reservationType.ReservationPackage,
+                    // typeOf: factory.reservationType.ReservationPackage,
                     reservationNumber: lockKey.holder,
                     subReservation: subReservations
                 };
@@ -181,7 +181,7 @@ class StockHolderRepository {
                 const addedReservationCount = lockKey.offers.length;
                 const subReservations = lockKey.offers.map((offer) => StockHolderRepository.offer2subReservation(offer, lockKey.hasTicketedSeat));
                 const reservationPackage = {
-                    typeOf: factory.reservationType.ReservationPackage,
+                    // typeOf: factory.reservationType.ReservationPackage,
                     reservationNumber: lockKey.holder,
                     subReservation: subReservations
                 };
@@ -264,8 +264,14 @@ class StockHolderRepository {
                 const reservationNumber = params.holder;
                 const updateResult = yield this.holdReservationModel.findOneAndUpdate({
                     'reservationFor.id': { $eq: params.eventId },
-                    'reservations.reservationNumber': { $eq: reservationNumber },
-                    'reservations.subReservation.identifier': { $eq: subReservation.identifier }
+                    'reservations.reservationNumber': {
+                        $exists: true,
+                        $eq: reservationNumber
+                    },
+                    'reservations.subReservation.identifier': {
+                        $exists: true,
+                        $eq: subReservation.identifier
+                    }
                 }, {
                     $inc: { reservationCount: -1 },
                     $pull: { 'reservations.$[reservationPackage].subReservation': { identifier: { $eq: subReservation.identifier } } }
@@ -277,8 +283,23 @@ class StockHolderRepository {
                 })
                     .select({ _id: 1 })
                     .exec();
-                debug('unlock processed. updateResult:', updateResult, 'reservationNumber:', reservationNumber, 'identifier:', subReservation.identifier);
                 // docが存在しなくてもよい
+                debug('unlock processed. updateResult:', updateResult, 'reservationNumber:', reservationNumber, 'identifier:', subReservation.identifier);
+                // subReservationがemptyのreservationsをpull
+                const pullReservationPackageResult = yield this.holdReservationModel.findOneAndUpdate({
+                    'reservationFor.id': { $eq: params.eventId },
+                    'reservations.reservationNumber': { $exists: true, $eq: reservationNumber }
+                }, {
+                    $pull: {
+                        reservations: {
+                            reservationNumber: { $eq: reservationNumber },
+                            subReservation: { $size: 0 }
+                        }
+                    }
+                }, { new: true })
+                    .select({ _id: 1 })
+                    .exec();
+                debug('unlock processed. pullReservationPackageResult:', pullReservationPackageResult, 'reservationNumber:', reservationNumber, 'identifier:', subReservation.identifier);
             }
             else {
                 const field = StockHolderRepository.offer2field(params.offer, params.hasTicketedSeat);
@@ -359,7 +380,12 @@ class StockHolderRepository {
                         $match: { 'reservationFor.id': { $eq: params.eventId } }
                     },
                     {
-                        $match: { 'reservations.subReservation.identifier': { $eq: subReservation.identifier } }
+                        $match: {
+                            'reservations.subReservation.identifier': {
+                                $exists: true,
+                                $eq: subReservation.identifier
+                            }
+                        }
                     }
                 ];
                 const aggregate = this.holdReservationModel.aggregate([
@@ -436,7 +462,12 @@ class StockHolderRepository {
                         $match: { 'reservationFor.id': { $eq: params.eventId } }
                     },
                     {
-                        $match: { 'reservations.subReservation.identifier': { $in: subReservations.map((r) => r.identifier) } }
+                        $match: {
+                            'reservations.subReservation.identifier': {
+                                $exists: true,
+                                $in: subReservations.map((r) => r.identifier)
+                            }
+                        }
                     }
                 ];
                 const aggregate = this.holdReservationModel.aggregate([
@@ -502,8 +533,9 @@ class StockHolderRepository {
     /**
      * 汎用的な検索(mongooseのみ対応)
      */
+    // tslint:disable-next-line:max-func-body-length
     search(params) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p;
         return __awaiter(this, void 0, void 0, function* () {
             const matchStages = [];
             const projectIdEq = (_b = (_a = params.project) === null || _a === void 0 ? void 0 : _a.id) === null || _b === void 0 ? void 0 : _b.$eq;
@@ -516,21 +548,16 @@ class StockHolderRepository {
             }
             const reservationNumberEq = (_e = params.reservationNumber) === null || _e === void 0 ? void 0 : _e.$eq;
             if (typeof reservationNumberEq === 'string') {
-                matchStages.push({ $match: { 'reservations.reservationNumber': { $eq: reservationNumberEq } } });
+                matchStages.push({ $match: { 'reservations.reservationNumber': { $exists: true, $eq: reservationNumberEq } } });
             }
             const identifierEq = (_f = params.identifier) === null || _f === void 0 ? void 0 : _f.$eq;
             if (typeof identifierEq === 'string') {
-                matchStages.push({ $match: { 'reservations.subReservation.identifier': { $eq: identifierEq } } });
+                matchStages.push({ $match: { 'reservations.subReservation.identifier': { $exists: true, $eq: identifierEq } } });
             }
             const idEq = (_g = params.id) === null || _g === void 0 ? void 0 : _g.$eq;
             if (typeof idEq === 'string') {
                 matchStages.push({
-                    $match: {
-                        'reservations.subReservation.id': {
-                            $exists: true,
-                            $eq: idEq
-                        }
-                    }
+                    $match: { 'reservations.subReservation.id': { $exists: true, $eq: idEq } }
                 });
             }
             const seatNumberEq = (_k = (_j = (_h = params.reservedTicket) === null || _h === void 0 ? void 0 : _h.ticketedSeat) === null || _j === void 0 ? void 0 : _j.seatNumber) === null || _k === void 0 ? void 0 : _k.$eq;
@@ -556,8 +583,21 @@ class StockHolderRepository {
                 });
             }
             const aggregate = this.holdReservationModel.aggregate([
-                { $unwind: '$reservations' },
-                { $unwind: '$reservations.subReservation' },
+                ...(typeof ((_p = params.sort) === null || _p === void 0 ? void 0 : _p['reservationFor.startDate']) === 'number')
+                    ? [{ $sort: { 'reservationFor.startDate': params.sort['reservationFor.startDate'] } }]
+                    : [],
+                {
+                    $unwind: {
+                        path: '$reservations',
+                        includeArrayIndex: 'reservationPackageIndex'
+                    }
+                },
+                {
+                    $unwind: {
+                        path: '$reservations.subReservation',
+                        includeArrayIndex: 'subReservationIndex'
+                    }
+                },
                 ...matchStages,
                 {
                     $project: {
@@ -573,7 +613,9 @@ class StockHolderRepository {
                         },
                         reservationNumber: '$reservations.reservationNumber',
                         reservedTicket: '$reservations.subReservation.reservedTicket',
-                        objectSize: { $bsonSize: '$$ROOT' }
+                        objectSize: { $bsonSize: '$$ROOT' },
+                        reservationPackageIndex: '$reservationPackageIndex',
+                        subReservationIndex: '$subReservationIndex'
                     }
                 }
             ]);

package/package.json

@@ -117,5 +117,5 @@
     "postversion": "git push origin --tags",
     "prepublishOnly": "npm run clean && npm run build && npm test && npm run doc"
   },
-  "version": "21.2.0-alpha.130"
+  "version": "21.2.0-alpha.131"
 }