fix: make grantedUsers optional in IPage interface and handle potenti…

…al null in page route
weseek · Jan 8, 2025 · a92911c · a92911c
1 parent eeabc23
commit a92911c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/apps/app/src/server/routes/apiv3/page/index.ts b/apps/app/src/server/routes/apiv3/page/index.ts
@@ -530,7 +530,7 @@ module.exports = (crowi) => {
     } = page;
     let isGrantNormalized = false;
     try {
-      const grantedUsersId = grantedUsers.map(ref => getIdForRef(ref));
+      const grantedUsersId = grantedUsers?.map(ref => getIdForRef(ref));
       isGrantNormalized = await pageGrantService.isGrantNormalized(req.user, path, grant, grantedUsersId, grantedGroups, false, false);
     }
     catch (err) {

diff --git a/packages/core/src/interfaces/page.ts b/packages/core/src/interfaces/page.ts
@@ -28,7 +28,7 @@ export type IPage = {
   descendantCount: number,
   isEmpty: boolean,
   grant: PageGrant,
-  grantedUsers: Ref<IUser>[],
+  grantedUsers?: Ref<IUser>[],
   grantedGroups: IGrantedGroup[],
   lastUpdateUser?: Ref<IUser>,
   liker: Ref<IUser>[],