From 73309d78902bfb8a19f435fda5e9f3fed3c62a1a Mon Sep 17 00:00:00 2001
From: Nick Zeng <Nick.Zeng@wpic.co>
Date: Mon, 30 Mar 2026 10:23:43 +0800
Subject: [PATCH] fix role key error

---
 frontend/src/components/RolePermissionModal.vue           | 8 ++++----
 .../src/components/__tests__/RolePermissionModal.spec.ts  | 7 +++++--
 frontend/src/stores/role-manage.ts                        | 6 +++---
 3 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/frontend/src/components/RolePermissionModal.vue b/frontend/src/components/RolePermissionModal.vue
index 5299cd2..06bcb1c 100644
--- a/frontend/src/components/RolePermissionModal.vue
+++ b/frontend/src/components/RolePermissionModal.vue
@@ -43,7 +43,7 @@ const methodColorMap: Record<string, string> = {
 const overrideColumns = [
   { title: '方法', key: 'method', width: 80 },
   { title: '路径', dataIndex: 'path', key: 'path' },
-  { title: '允许', key: 'allow', width: 80 },
+  { title: '允许', key: 'allowed', width: 80 },
   { title: '操作', key: 'action', width: 80 },
 ]
 
@@ -113,7 +113,7 @@ function addOverride() {
     route_id: route.id,
     method: route.method,
     path: route.path,
-    allow: true,
+    allowed: true,
   })
   addingRouteId.value = null
 }
@@ -211,8 +211,8 @@ watch(
                   {{ record.method }}
                 </a-tag>
               </template>
-              <template v-if="column.key === 'allow'">
-                <a-switch v-model:checked="record.allow" size="small" />
+              <template v-if="column.key === 'allowed'">
+                <a-switch v-model:checked="record.allowed" size="small" />
               </template>
               <template v-if="column.key === 'action'">
                 <a-button type="link" size="small" danger @click="removeOverride(index)">
diff --git a/frontend/src/components/__tests__/RolePermissionModal.spec.ts b/frontend/src/components/__tests__/RolePermissionModal.spec.ts
index 777f2f9..daef2e9 100644
--- a/frontend/src/components/__tests__/RolePermissionModal.spec.ts
+++ b/frontend/src/components/__tests__/RolePermissionModal.spec.ts
@@ -52,7 +52,7 @@ const mockRoutes = [
 ]
 
 const mockOverrides = [
-  { route_id: 1, method: 'GET', path: '/api/v1/users', allow: true },
+  { route_id: 1, method: 'GET', path: '/api/v1/users', allowed: true },
 ]
 
 function setupMocks(overrides: {
@@ -60,7 +60,10 @@ function setupMocks(overrides: {
   roleOverrides?: typeof mockOverrides
 } = {}) {
   vi.mocked(api.get).mockImplementation((url: string) => {
-    if (url.match(/\/roles\/\d+\/route-groups/)) return Promise.resolve(overrides.groupIds ?? [1, 2]) as never
+    if (url.match(/\/roles\/\d+\/route-groups/)) {
+      const ids = overrides.groupIds ?? [1, 2]
+      return Promise.resolve(ids.map((id) => ({ id }))) as never
+    }
     if (url.match(/\/roles\/\d+\/route-overrides/)) return Promise.resolve(overrides.roleOverrides ?? mockOverrides) as never
     if (url === '/api/v1/route-groups') return Promise.resolve(mockGroups) as never
     if (url === '/api/v1/routes') return Promise.resolve(mockRoutes) as never
diff --git a/frontend/src/stores/role-manage.ts b/frontend/src/stores/role-manage.ts
index 60921b6..6ab8e2e 100644
--- a/frontend/src/stores/role-manage.ts
+++ b/frontend/src/stores/role-manage.ts
@@ -13,7 +13,7 @@ export interface RouteOverride {
   route_id: number
   method: string
   path: string
-  allow: boolean
+  allowed: boolean
 }
 
 export const useRoleManageStore = defineStore('role-manage', () => {
@@ -34,8 +34,8 @@ export const useRoleManageStore = defineStore('role-manage', () => {
   }
 
   async function fetchRoleRouteGroups(roleId: number) {
-    const data = await api.get<number[]>(`/api/v1/roles/${roleId}/route-groups`)
-    return data
+    const data = await api.get<{ id: number }[]>(`/api/v1/roles/${roleId}/route-groups`)
+    return data.map((g) => g.id)
   }
 
   async function setRoleRouteGroups(roleId: number, groupIds: number[]) {