From 3ca8c6ef737e1125a662f975ff2814a8a7e796a5 Mon Sep 17 00:00:00 2001
From: zhangzhanwei <zhanwei.zhang@fit2cloud.com>
Date: Tue, 22 Jul 2025 16:03:55 +0800
Subject: [PATCH] perf: Morefilled system and model

---
 ui/src/views/model/component/ModelCard.vue    | 15 +++++-
 ui/src/views/system-chat-user/group/index.vue | 50 ++++++++-----------
 ui/src/views/system/role/index.vue            | 37 ++++++--------
 ui/src/views/system/workspace/index.vue       | 29 +++++------
 4 files changed, 64 insertions(+), 67 deletions(-)

diff --git a/ui/src/views/model/component/ModelCard.vue b/ui/src/views/model/component/ModelCard.vue
index d834f3823..d3470ab76 100644
--- a/ui/src/views/model/component/ModelCard.vue
+++ b/ui/src/views/model/component/ModelCard.vue
@@ -64,7 +64,9 @@
       </div>
     </div>
 
-    <template #mouseEnter>
+    <template #mouseEnter
+      v-if="MoreFilledPermission(model.id)"
+    >
       <el-dropdown trigger="click" v-if="!isShared">
         <el-button text @click.stop>
           <el-icon>
@@ -147,10 +149,21 @@ const props = defineProps<{
   apiType: 'systemShare' | 'workspace' | 'systemManage'
 }>()
 
+const apiType = props.apiType
+
+const isSystemShare = computed(() => {
+  return apiType==='systemShare'
+})
+
 const permissionPrecise = computed(() => {
   return permissionMap['model'][props.apiType]
 })
 
+const MoreFilledPermission = (id: any) => {
+  return permissionPrecise.value.modify(id) || 
+  permissionPrecise.value.delete(id) || isSystemShare.value
+}
+
 const downModel = ref<Model>()
 
 const currentModel = computed(() => {
diff --git a/ui/src/views/system-chat-user/group/index.vue b/ui/src/views/system-chat-user/group/index.vue
index 4fd165f8a..ff3f5b7c2 100644
--- a/ui/src/views/system-chat-user/group/index.vue
+++ b/ui/src/views/system-chat-user/group/index.vue
@@ -63,7 +63,9 @@
                     <div class="flex-between">
                       <span class="ellipsis" :title="row.name">{{ row.name }}</span>
                       <div @click.stop v-show="mouseId === row.id">
-                        <el-dropdown :teleported="false" trigger="click">
+                        <el-dropdown :teleported="false" trigger="click"
+                          v-if="editPermission() || dlePermission()"
+                        >
                           <el-button text>
                             <el-icon class="color-secondary">
                               <MoreFilled />
@@ -74,20 +76,7 @@
                               <el-dropdown-item
                                 @click.stop="createOrUpdate(row)"
                                 class="p-8"
-                                v-if="
-                                  hasPermission(
-                                    new ComplexPermission(
-                                      [RoleConst.ADMIN, RoleConst.WORKSPACE_MANAGE],
-                                      [
-                                        PermissionConst.WORKSPACE_USER_GROUP_EDIT,
-                                        PermissionConst.USER_GROUP_EDIT,
-                                      ],
-                                      [],
-                                      'OR',
-                                    ),
-                                    'OR',
-                                  )
-                                "
+                                v-if="editPermission()"
                               >
                                 <el-icon>
                                   <EditPen />
@@ -97,20 +86,7 @@
                               <el-dropdown-item
                                 @click.stop="deleteGroup(row)"
                                 class="border-t p-8"
-                                v-if="
-                                  hasPermission(
-                                    new ComplexPermission(
-                                      [RoleConst.ADMIN, RoleConst.WORKSPACE_MANAGE],
-                                      [
-                                        PermissionConst.WORKSPACE_USER_GROUP_DELETE,
-                                        PermissionConst.USER_GROUP_DELETE,
-                                      ],
-                                      [],
-                                      'OR',
-                                    ),
-                                    'OR',
-                                  )
-                                "
+                                v-if="dlePermission()"
                               >
                                 <el-icon>
                                   <Delete />
@@ -307,6 +283,22 @@ async function getUserGroupList() {
   }
 }
 
+const editPermission = () => {
+  return hasPermission(new ComplexPermission(
+                      [RoleConst.ADMIN, RoleConst.WORKSPACE_MANAGE],
+                      [PermissionConst.WORKSPACE_USER_GROUP_EDIT,
+                       PermissionConst.USER_GROUP_EDIT,],[],
+                       'OR',),'OR',)
+}
+
+const dlePermission = () => {
+  return hasPermission(new ComplexPermission(
+                      [RoleConst.ADMIN, RoleConst.WORKSPACE_MANAGE],
+                      [PermissionConst.WORKSPACE_USER_GROUP_DELETE,
+                       PermissionConst.USER_GROUP_DELETE,],[],
+                       'OR',),'OR',)
+}
+
 onMounted(async () => {
   await getUserGroupList()
   current.value = list.value[0]
diff --git a/ui/src/views/system/role/index.vue b/ui/src/views/system/role/index.vue
index 4ed4f8f3f..2937ee6e7 100644
--- a/ui/src/views/system/role/index.vue
+++ b/ui/src/views/system/role/index.vue
@@ -72,7 +72,8 @@
                           >({{ roleTypeMap[row.type as RoleTypeEnum] }})</span
                         >
                       </span>
-                      <div @click.stop v-show="mouseId === row.id">
+                      <div @click.stop v-show="mouseId === row.id" 
+                        v-if="editPermission() || delPermission()">
                         <el-dropdown :teleported="false" trigger="click">
                           <el-button text>
                             <el-icon class="color-secondary">
@@ -84,17 +85,7 @@
                               <el-dropdown-item
                                 @click.stop="createOrUpdateRole(row)"
                                 class="p-8"
-                                v-if="
-                                  hasPermission(
-                                    new ComplexPermission(
-                                      [RoleConst.ADMIN],
-                                      [PermissionConst.ROLE_EDIT],
-                                      [],
-                                      'OR',
-                                    ),
-                                    'OR',
-                                  )
-                                "
+                                v-if="editPermission()"
                               >
                                 <el-icon><EditPen /></el-icon>
                                 {{ $t('common.rename') }}
@@ -102,17 +93,7 @@
                               <el-dropdown-item
                                 @click.stop="deleteRole(row)"
                                 class="border-t p-8"
-                                v-if="
-                                  hasPermission(
-                                    new ComplexPermission(
-                                      [RoleConst.ADMIN],
-                                      [PermissionConst.ROLE_DELETE],
-                                      [],
-                                      'OR',
-                                    ),
-                                    'OR',
-                                  )
-                                "
+                                v-if="delPermission()"
                               >
                                 <el-icon><Delete /></el-icon>
                                 {{ $t('common.delete') }}
@@ -205,6 +186,16 @@ async function getRole() {
   }
 }
 
+const editPermission = () => {
+  return hasPermission(new ComplexPermission([RoleConst.ADMIN],
+                      [PermissionConst.ROLE_EDIT],[],'OR',),'OR',)
+}
+
+const delPermission = () => {
+  return hasPermission(new ComplexPermission([RoleConst.ADMIN],
+                      [PermissionConst.ROLE_DELETE],[],'OR',),'OR',)
+}
+
 onMounted(async () => {
   await getRole()
   currentRole.value = internalRoleList.value[0]
diff --git a/ui/src/views/system/workspace/index.vue b/ui/src/views/system/workspace/index.vue
index c6290da22..f9bda2531 100644
--- a/ui/src/views/system/workspace/index.vue
+++ b/ui/src/views/system/workspace/index.vue
@@ -43,7 +43,9 @@
                     <div class="flex-between">
                       <span class="ellipsis" :title="row.name">{{ row.name }}</span>
                       <div @click.stop v-show="mouseId === row.id">
-                        <el-dropdown :teleported="false" trigger="click">
+                        <el-dropdown :teleported="false" trigger="click"
+                          v-if="editPermission() || dlePermission()"  
+                        >
                           <el-button text>
                             <el-icon class="color-secondary">
                               <MoreFilled />
@@ -54,12 +56,7 @@
                               <el-dropdown-item
                                 @click.stop="createOrUpdateWorkspace(row)"
                                 class="p-8"
-                                v-if="
-                                  hasPermission(
-                                    [RoleConst.ADMIN, PermissionConst.WORKSPACE_EDIT],
-                                    'OR',
-                                  )
-                                "
+                                v-if="editPermission()"
                               >
                                 <el-icon><EditPen /></el-icon>
                                 {{ $t('common.rename') }}
@@ -67,13 +64,7 @@
                               <el-dropdown-item
                                 @click.stop="deleteWorkspace(row)"
                                 class="border-t p-8"
-                                v-if="
-                                  row.id !== 'default' &&
-                                  hasPermission(
-                                    [RoleConst.ADMIN, PermissionConst.WORKSPACE_DELETE],
-                                    'OR',
-                                  )
-                                "
+                                v-if="dlePermission()"
                               >
                                 <el-icon><Delete /></el-icon>
                                 {{ $t('common.delete') }}
@@ -144,6 +135,16 @@ onMounted(async () => {
   currentWorkspace.value = list.value[0]
 })
 
+const editPermission = () => {
+  return hasPermission([RoleConst.ADMIN, 
+  PermissionConst.WORKSPACE_EDIT],'OR',)
+}
+
+const dlePermission = () => {
+  return hasPermission([RoleConst.ADMIN, 
+  PermissionConst.WORKSPACE_DELETE],'OR',)
+}
+
 async function refresh(workspace?: WorkspaceItem) {
   await getWorkspace()
   // 创建后选中新建的