[API] หน้าสร้าง Page ให้ Validate URL ของเพจด้วย #380

kaogeek · Dec 8, 2021 · 24c1576 · 24c1576
1 parent 62ee540
commit 24c1576
Show file tree

Hide file tree

Showing 2 changed files with 47 additions and 3 deletions.
diff --git a/api-spanboon/src/api/controllers/PageController.ts b/api-spanboon/src/api/controllers/PageController.ts
@@ -1514,6 +1514,15 @@ export class PageController {
             return res.status(400).send(errorResponse);
         }
 
+        // check dupicate uniqueId
+        if (pages.pageUsername !== undefined && pages.pageUsername !== null && pages.pageUsername !== '') {
+            const isContainsUniqueId = await this.userService.isContainsUniqueId(pages.pageUsername);
+            if (isContainsUniqueId !== undefined && isContainsUniqueId) {
+                const errorResponse = ResponseUtil.getErrorResponse('PageUsername already exists', undefined);
+                return res.status(400).send(errorResponse);
+            }
+        }
+
         const fileName = userObjId + FileUtil.renameFile();
         let assetCreate: Asset;
 
@@ -1959,6 +1968,15 @@ export class PageController {
                 return res.status(400).send(ResponseUtil.getSuccessResponse('Invalid Page Id', undefined));
             }
 
+            // check dupicate uniqueId
+            if (pages.pageUsername !== undefined && pages.pageUsername !== null && pages.pageUsername !== '') {
+                const isContainsUniqueId = await this.userService.isContainsUniqueId(pages.pageUsername, undefined, pageUpdate.pageUsername);
+                if (isContainsUniqueId !== undefined && isContainsUniqueId) {
+                    const errorResponse = ResponseUtil.getErrorResponse('PageUsername already exists', undefined);
+                    return res.status(400).send(errorResponse);
+                }
+            }
+
             let pageName = pages.name;
             let pagesUsername = pages.pageUsername;
             let pageSubTitle = pages.subTitle;

diff --git a/api-spanboon/src/api/services/UserService.ts b/api-spanboon/src/api/services/UserService.ts
@@ -79,16 +79,42 @@ export class UserService {
         }
     }
 
-    public isContainsUniqueId(uniqueId: string): Promise<boolean> {
+    public isContainsUniqueId(uniqueId: string, ignoreUserUniqueId?: string, ignorePageUniqueId?: string): Promise<boolean> {
         if (uniqueId === undefined || uniqueId === null || uniqueId === '') {
             return Promise.resolve(undefined);
         }
 
         return new Promise(async (resolve, reject) => {
             try {
-                const checkUniqueIdUserQuey = { where: { uniqueId } };
+                let checkUniqueIdUserQuey: any = { where: { uniqueId } };
+                if (ignoreUserUniqueId !== undefined && ignoreUserUniqueId !== null && ignoreUserUniqueId !== '') {
+                    checkUniqueIdUserQuey = {
+                        $and: [
+                            {
+                                uniqueId
+                            }, {
+                                uniqueId: {
+                                    $nin: [ignoreUserUniqueId]
+                                }
+                            }
+                        ]
+                    };
+                }
                 const checkUniqueIdUser: User = await this.findOne(checkUniqueIdUserQuey);
-                const checkPageUsernameQuey = { where: { pageUsername: uniqueId } };
+                let checkPageUsernameQuey: any = { where: { pageUsername: uniqueId } };
+                if (ignorePageUniqueId !== undefined && ignorePageUniqueId !== null && ignorePageUniqueId !== '') {
+                    checkPageUsernameQuey = {
+                        $and: [
+                            {
+                                pageUsername: uniqueId
+                            }, {
+                                pageUsername: {
+                                    $nin: [ignorePageUniqueId]
+                                }
+                            }
+                        ]
+                    };
+                }
                 const checkPageUsername: any = await this.pageRepository.findOne(checkPageUsernameQuey);
 
                 if ((checkUniqueIdUser !== null && checkUniqueIdUser !== undefined) || (checkPageUsername !== null && checkPageUsername !== undefined)) {