update multiple accept logic

src/popo/reservation/place/reserve.place.controller.ts

@@ -26,6 +26,7 @@ import { Roles } from '../../../auth/authroization/roles.decorator';
 import { RolesGuard } from '../../../auth/authroization/roles.guard';
 import { PlaceService } from '../../place/place.service';
 import {JwtPayload} from "../../../auth/strategies/jwt.payload";
+import { ReservePlace } from './reserve.place.entity';
 
 @ApiTags('Place Reservation')
 @Controller('reservation-place')
@@ -196,8 +197,16 @@ export class ReservePlaceController {
     @Body() body: AcceptPlaceReservationListDto,
     @Query('sendEmail') sendEmail?: string,
   ) {
+    const reservations: ReservePlace[]  = [];
     for (const reservation_uuid of body.uuid_list) {
       const reservation = await this.reservePlaceService.findOneByUuidOrFail(reservation_uuid);
+      reservations.push(reservation);
+    }
+
+    // early created reservation should be processed first
+    reservations.sort((a, b) => (a.created_at > b.created_at ? 1 : -1));
+
+    for (const reservation of reservations) {
       await this.reservePlaceService.checkReservationPossible(
         {
           place_id: reservation.place_id,
@@ -208,7 +217,7 @@ export class ReservePlaceController {
         reservation.booker_id,
       )
       const response = await this.reservePlaceService.updateStatus(
-        reservation_uuid,
+        reservation.uuid,
         ReservationStatus.accept,
       );
 

src/popo/reservation/place/reserve.place.service.ts

@@ -29,6 +29,7 @@ export class ReservePlaceService {
     private readonly placeService: PlaceService,
   ) {}
 
+  // TODO: delete this code, after concurrent check logic is fully validated
   async isReservationOverlap(
     place_id: string,
     date: string,
@@ -131,6 +132,7 @@ export class ReservePlaceService {
 
     const targetPlace = await this.placeService.findOneByUuidOrFail(place_id);
 
+    // TODO: delete this code, after concurrent check logic is fully validated
     // Reservation Overlap Check
     // const isReservationOverlap = await this.isReservationOverlap(
     //   place_id,
@@ -148,7 +150,7 @@ export class ReservePlaceService {
     const isConcurrentPossible = await this.isReservationConcurrent(place_id, targetPlace.max_concurrent_reservation, date, start_time, end_time);
     if (!isConcurrentPossible) {
       throw new BadRequestException(
-        `"${targetPlace.name}" 장소에 이미 ${targetPlace.max_concurrent_reservation}개 예약이 있어 ${date} ${start_time} ~ ${end_time}에는 예약이 불가능 합니다.`
+        `"${targetPlace.name}" 장소에 이미 승인된 ${targetPlace.max_concurrent_reservation}개 예약이 있어 ${date} ${start_time} ~ ${end_time}에는 예약이 불가능 합니다.`
       )
     }