fix: modify user controller hierarchy

skku-royals · Mar 26, 2024 · 91ba4f1 · 91ba4f1
1 parent a3e81ec
commit 91ba4f1
Showing 1 changed file with 14 additions and 14 deletions.
diff --git a/backend/app/src/user/user.controller.ts b/backend/app/src/user/user.controller.ts
@@ -87,20 +87,6 @@ export class UserController {
     }
   }
 
-  @Roles(Role.Admin)
-  @Put(':userId')
-  async updateUser(
-    @Param('userId', ParseIntPipe) userId: number,
-    @Body() userDTO: UpdateUserDTO,
-    @Req() req: AuthenticatedRequest
-  ): Promise<ReducedUserDTO> {
-    try {
-      return await this.userService.updateUser(req.user.id, userId, userDTO)
-    } catch (error) {
-      BusinessExceptionHandler(error)
-    }
-  }
-
   @Put('profile-image')
   @UseInterceptors(FileInterceptor('image', IMAGE_OPTIONS))
   async updateCurrentUserProfileImage(
@@ -118,6 +104,20 @@ export class UserController {
     }
   }
 
+  @Roles(Role.Admin)
+  @Put(':userId')
+  async updateUser(
+    @Param('userId', ParseIntPipe) userId: number,
+    @Body() userDTO: UpdateUserDTO,
+    @Req() req: AuthenticatedRequest
+  ): Promise<ReducedUserDTO> {
+    try {
+      return await this.userService.updateUser(req.user.id, userId, userDTO)
+    } catch (error) {
+      BusinessExceptionHandler(error)
+    }
+  }
+
   @Roles(Role.Admin)
   @Post('temp')
   async createTempUser(