Merge pull request #492 from bounswe/feature/BE/474/follow-info-for-g…

…ame-list added follow info for game list
bounswe · Nov 21, 2023 · 3188462 · 3188462
2 parents f746f6b + 7166b25
commit 3188462
Show file tree

Hide file tree

Showing 5 changed files with 60 additions and 3 deletions.
diff --git a/ludos/backend/src/controllers/game.controller.ts b/ludos/backend/src/controllers/game.controller.ts
@@ -7,6 +7,7 @@ import {
   HttpStatus,
   NotFoundException,
   Param,
+  ParseBoolPipe,
   ParseIntPipe,
   Post,
   Put,
@@ -139,12 +140,20 @@ export class GameController {
     description: 'ASC or DESC. Default is ASC',
     example: 'ASC',
   })
+  @ApiQuery({
+    name: 'isFollowed',
+    required: false,
+    description: 'Filter by followed games. If false no filter is applied',
+    example: 'true',
+  })
   @ApiOkResponse({
     type: GamePageResponseDto,
   })
+  @ApiBearerAuth()
   @UseInterceptors(new SerializerInterceptor(GamePageResponseDto))
   @Get()
   public async listGames(
+    @Req() req: AuthorizedRequest,
     @Query('page', new DefaultValuePipe(1), ParseIntPipe) page: number = 1,
     @Query('limit', new DefaultValuePipe(10), ParseIntPipe) limit: number = 10,
     @Query('searchKey') searchKey?: string,
@@ -154,6 +163,8 @@ export class GameController {
     @Query('developer') developer?: string,
     @Query('orderByKey') orderByKey?: keyof Game,
     @Query('order') order?: 'ASC' | 'DESC',
+    @Query('isFollowed', new DefaultValuePipe(false), ParseBoolPipe)
+    isFollowed?: boolean,
   ) {
     return await this.gameService.listGames(
       page,
@@ -165,6 +176,8 @@ export class GameController {
       developer,
       orderByKey,
       order,
+      req.user && req.user.id,
+      isFollowed,
     );
   }
 }
diff --git a/ludos/backend/src/dtos/game/response/list.response.ts b/ludos/backend/src/dtos/game/response/list.response.ts
@@ -32,4 +32,7 @@ export class GameListResponseDto {
   @Expose()
   @ApiProperty()
   ageRestriction: string;
+  @Expose()
+  @ApiProperty()
+  isFollowed?: boolean;
 }
diff --git a/ludos/backend/src/repositories/game.repository.ts b/ludos/backend/src/repositories/game.repository.ts
@@ -37,6 +37,8 @@ export class GameRepository extends Repository<Game> {
     developer?: string,
     orderByKey: keyof Game = 'id',
     order: 'ASC' | 'DESC' = 'ASC',
+    userId?: string, // This is not the creator, this is used for follow check
+    isFollowed?: boolean,
   ): Promise<Pagination<Game, IPaginationMeta>> {
     const queryBuilder = this.createQueryBuilder('games').where('1=1');
     if (searchKey) {
@@ -58,10 +60,41 @@ export class GameRepository extends Repository<Game> {
     if (developer) {
       queryBuilder.andWhere('games.developer = :developer', { developer });
     }
+    if (userId && isFollowed) {
+      const subQuery = this.createQueryBuilder()
+        .select('1')
+        .from('game_user_follows', 'guf')
+        .where(`guf.usersId = '${userId}'`)
+        .andWhere('guf.gamesId = games.id')
+        .getQuery();
+      queryBuilder.andWhere(`EXISTS (${subQuery})`);
+    }
     if (orderByKey) {
       queryBuilder.orderBy(`games_${orderByKey}`, order);
     }
-    const games = await paginate<Game>(queryBuilder, { page, limit });
-    return games;
+    const paginationResult = await paginate<Game>(queryBuilder, {
+      page,
+      limit,
+    });
+    if (userId) {
+      await Promise.all(
+        paginationResult.items.map(async (game) => {
+          game.isFollowed = await this.checkIfGameIsFollowed(game.id, userId);
+        }),
+      );
+    }
+    return paginationResult;
+  }
+  private async checkIfGameIsFollowed(
+    gameId: string,
+    userId: string,
+  ): Promise<boolean> {
+    const result = await this.createQueryBuilder()
+      .select('1')
+      .from('game_user_follows', 'guf')
+      .where(`guf.usersId = '${userId}'`)
+      .andWhere('guf.gamesId = :gameId', { gameId })
+      .getExists();
+    return result ? true : false;
   }
 }
diff --git a/ludos/backend/src/services/game.service.ts b/ludos/backend/src/services/game.service.ts
@@ -97,6 +97,8 @@ export class GameService {
     developer?: string,
     orderByKey?: keyof Game,
     order?: 'ASC' | 'DESC',
+    userId?: string,
+    isFollowed?: boolean,
   ): Promise<Pagination<Game, IPaginationMeta>> {
     const tagList = tags ? tags.split(',') : undefined;
     const platformList = platforms ? platforms.split(',') : undefined;
@@ -110,6 +112,8 @@ export class GameService {
       developer,
       orderByKey,
       order,
+      userId,
+      isFollowed,
     );
   }
 }
diff --git a/ludos/backend/test/game.controller.spec.ts b/ludos/backend/test/game.controller.spec.ts
@@ -8,6 +8,7 @@ import { User } from '../src/entities/user.entity';
 import { GameRepository } from '../src/repositories/game.repository';
 import { UserRepository } from '../src/repositories/user.repository';
 import { GameService } from '../src/services/game.service';
+import { AuthorizedRequest } from '../src/interfaces/common/authorized-request.interface';
 import { RatingRepository } from '../src/repositories/rating.repository';
 describe('GameController', () => {
   let gameController: GameController;
@@ -189,7 +190,8 @@ describe('GameController', () => {
       const listSpy = jest
         .spyOn(gameRepository, 'findGames')
         .mockResolvedValue(listResponse);
-      const response = await gameController.listGames();
+      const req = {}
+      const response = await gameController.listGames(req as AuthorizedRequest);
       expect(response).toBe(listResponse);
       expect(listSpy).toHaveBeenCalledWith(
         1,
@@ -201,6 +203,8 @@ describe('GameController', () => {
         undefined,
         undefined,
         undefined,
+        undefined,
+        undefined
       );
     });
   });