Merge pull request #404 from Quickchive/fix/remove-category-content-duplication-check

fix: 콘텐츠 저장 시에도 카테고리 중복검사 로직 수정

Author: stae1102

src/contents/contents.service.ts

@@ -87,17 +87,23 @@ export class ContentsService {
     const content = new Content();
 
     if (categoryId) {
-      const category = await this.categoryRepository.findById(
-        categoryId,
-        entityManager,
-      );
+      const [category, subCategories] = await Promise.all([
+        (async () => {
+          const category = await this.categoryRepository.findById(categoryId);
 
-      if (!category) throw new NotFoundException('Category not found');
+          if (!category) {
+            throw new NotFoundException('카테고리가 존재하지 않습니다.');
+          }
 
-      await checkContentDuplicateAndAddCategorySaveLog(
-        link,
-        category,
-        userInDb,
+          return category;
+        })(),
+        this.categoryRepository.findByParentId(categoryId),
+      ]);
+
+      await this.isDuplicatedContents(
+        content.id,
+        [category, ...subCategories],
+        content.link,
       );
 
       content.category = category;
@@ -223,16 +229,12 @@ export class ContentsService {
         this.categoryRepository.findByParentId(categoryId),
       ]);
 
-      const isDuplicated = await this.isDuplicatedContents(
+      await this.isDuplicatedContents(
         content.id,
         [category, ...subCategories],
         content.link,
       );
 
-      if (isDuplicated) {
-        throw new ConflictException('이미 저장된 컨텐츠입니다.');
-      }
-
       await this.contentRepository.updateOne(
         {
           id: content.id,
@@ -495,6 +497,8 @@ export class ContentsService {
       },
     });
 
-    return existingContents.length > 0;
+    if (existingContents.length > 0) {
+      throw new ConflictException('이미 저장된 컨텐츠입니다.');
+    }
   }
 }