feat(next/api): cascade record duration metrics

leancloud · Aug 9, 2023 · 1316063 · 1316063
1 parent 528733c
commit 1316063
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/next/api/src/controller/ticket.ts b/next/api/src/controller/ticket.ts
@@ -31,7 +31,7 @@ export class TicketController {
   @ResponseBody(TicketListItemResponse)
   getAssociatedTickets(@Ctx() ctx: Context) {
     const ticket = ctx.state.ticket as Ticket;
-    return ticket.getAssociatedTickets({ useMasterKey: true });
+    return ticket.getAssociatedTickets();
   }
 
   @Post(':id/associated-tickets')

diff --git a/next/api/src/model/Ticket.ts b/next/api/src/model/Ticket.ts
@@ -370,15 +370,14 @@ export class Ticket extends Model {
     const associateTickets = await this.getAssociatedTickets();
 
     const result = await Promise.all(
-      (options?.cascade ? [...associateTickets, this] : [this]).map((ticket) => {
+      (options?.cascade ? [...associateTickets, this] : [this]).map(async (ticket) => {
         const updater = new TicketUpdater(ticket);
-        updater.operate(action);
-        return updater.update(operator, options);
+        const updatedTicket = await updater.operate(action).update(operator, options);
+        await durationMetricService.recordOperateTicket(ticket, action);
+        return updatedTicket;
       })
     );
 
-    await durationMetricService.recordOperateTicket(this, action);
-
     return result.find((ticket) => ticket.id === this.id)!;
   }