🐛 Fix scopes for user, groups routes

staart · Nov 15, 2020 · c3dcb4a · c3dcb4a
1 parent 1804bf3
commit c3dcb4a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/src/modules/groups/groups.controller.ts b/src/modules/groups/groups.controller.ts
@@ -43,7 +43,7 @@ export class GroupController {
   }
 
   @Get(':groupId')
-  @Scopes('group-{id}:read-info')
+  @Scopes('group-{groupId}:read-info')
   async get(
     @Param('groupId', ParseIntPipe) id: number,
   ): Promise<Expose<groups>> {
@@ -52,7 +52,7 @@ export class GroupController {
 
   @Patch(':groupId')
   @AuditLog('update-info')
-  @Scopes('group-{id}:write-info')
+  @Scopes('group-{groupId}:write-info')
   async update(
     @Body() data: UpdateGroupDto,
     @Param('groupId', ParseIntPipe) id: number,
@@ -62,7 +62,7 @@ export class GroupController {
 
   @Put(':groupId')
   @AuditLog('update-info')
-  @Scopes('group-{id}:write-info')
+  @Scopes('group-{groupId}:write-info')
   async replace(
     @Body() data: ReplaceGroupDto,
     @Param('groupId', ParseIntPipe) id: number,
@@ -72,7 +72,7 @@ export class GroupController {
 
   @Delete(':groupId')
   @AuditLog('delete')
-  @Scopes('group-{id}:delete')
+  @Scopes('group-{groupId}:delete')
   async remove(
     @Param('groupId', ParseIntPipe) id: number,
   ): Promise<Expose<groups>> {

diff --git a/src/modules/users/users.controller.ts b/src/modules/users/users.controller.ts
@@ -37,13 +37,13 @@ export class UserController {
   }
 
   @Get(':userId')
-  @Scopes('user-{id}:read-info')
+  @Scopes('user-{userId}:read-info')
   async get(@Param('userId', ParseIntPipe) id: number): Promise<Expose<users>> {
     return this.usersService.getUser(Number(id));
   }
 
   @Patch(':userId')
-  @Scopes('user-{id}:write-info')
+  @Scopes('user-{userId}:write-info')
   async update(
     @Param('userId', ParseIntPipe) id: number,
     @Body() data: UpdateUserDto,
@@ -52,15 +52,15 @@ export class UserController {
   }
 
   @Delete(':userId')
-  @Scopes('user-{id}:deactivate')
+  @Scopes('user-{userId}:deactivate')
   async remove(
     @Param('userId', ParseIntPipe) id: number,
   ): Promise<Expose<users>> {
     return this.usersService.deactivateUser(Number(id));
   }
 
   @Post(':userId/merge-request')
-  @Scopes('user-{id}:merge')
+  @Scopes('user-{userId}:merge')
   @RateLimit({
     points: 10,
     duration: 60,
@@ -74,7 +74,7 @@ export class UserController {
   }
 
   @Post('merge')
-  @Scopes('user-{id}:merge')
+  @Scopes('user-{userId}:merge')
   @RateLimit({
     points: 10,
     duration: 60,