Fixed metadata controllers returning tags, genres, publishers & developers for deleted games

Author: Alfagun74

CHANGELOG.md

@@ -9,6 +9,7 @@
 - Added ``TESTING_LOG_HTTP_TRAFFIC_ENABLED`` parameter to help analyze network traffic
 - Fixed ``early_access`` being required in UpdateGameDto inconsistency in OpenAPI contract
 - Fixed ``name`` not being mentioned in MetadataProviderDto inconsistency in OpenAPI contract
+- Fixed metadata controllers returning tags, genres, publishers & developers for deleted games
 
 
 ### Thanks

src/modules/metadata/developers/developers.metadata.controller.ts

@@ -50,7 +50,7 @@ export class DeveloperController {
   ): Promise<Paginated<DeveloperMetadata>> {
     const queryBuilder = this.developerRepository
       .createQueryBuilder("developer")
-      .leftJoinAndSelect("developer.games", "games")
+      .leftJoinAndSelect("developer.games", "games", "games.deleted_at IS NULL")
       .where("developer.provider_slug = :provider_slug", {
         provider_slug: "gamevault",
       })

src/modules/metadata/genres/genres.metadata.controller.ts

@@ -50,7 +50,7 @@ export class GenreController {
   ): Promise<Paginated<GenreMetadata>> {
     const queryBuilder = this.genreRepository
       .createQueryBuilder("genre")
-      .leftJoinAndSelect("genre.games", "games")
+      .leftJoinAndSelect("genre.games", "games", "games.deleted_at IS NULL")
       .where("genre.provider_slug = :provider_slug", {
         provider_slug: "gamevault",
       })

src/modules/metadata/publishers/publishers.metadata.controller.ts

@@ -50,7 +50,7 @@ export class PublisherController {
   ): Promise<Paginated<PublisherMetadata>> {
     const queryBuilder = this.publisherRepository
       .createQueryBuilder("publisher")
-      .leftJoinAndSelect("publisher.games", "games")
+      .leftJoinAndSelect("publisher.games", "games", "games.deleted_at IS NULL")
       .where("publisher.provider_slug = :provider_slug", {
         provider_slug: "gamevault",
       })

src/modules/metadata/tags/tags.metadata.controller.ts

@@ -50,7 +50,7 @@ export class TagsController {
   ): Promise<Paginated<TagMetadata>> {
     const queryBuilder = this.tagRepository
       .createQueryBuilder("tag")
-      .leftJoinAndSelect("tag.games", "games")
+      .leftJoinAndSelect("tag.games", "games", "games.deleted_at IS NULL")
       .where("tag.provider_slug = :provider_slug", {
         provider_slug: "gamevault",
       })