PullRequest: 1165 fix: 作业与执行记录增加鉴权

Merge branch 'fix/relativeModalAuthCheck of git@code.alipay.com:oceanbase/oceanbase-developer-center.git into dev-4.4.1

https://code.alipay.com/oceanbase/oceanbase-developer-center/pull_requests/1165


Reviewed-by: 晓康 <xxk268858@oceanbase.com>


* fix: 作业与执行记录增加鉴权
* fix: 完善权限判断
* refactor: 优化写法

Author: 之瑛

src/component/RelativeResourceModal/index.less

@@ -7,13 +7,18 @@
     align-items: start;
     flex-direction: column;
     cursor: pointer;
-    .title {
+    .title,
+    .disabledTitle {
       font-size: 12px;
       line-height: 20px;
       font-weight: 600;
-      &:hover {
-        color: var(--text-color-link);
-      }
+    }
+    .title:hover {
+      color: var(--text-color-link);
+    }
+    .disabledTitle {
+      color: var(--text-color-hint);
+      cursor: not-allowed;
     }
   }
   .recordScheduleName {
@@ -38,11 +43,14 @@
       white-space: nowrap;
       overflow: hidden;
       text-overflow: ellipsis;
+
+      .titleText {
+        font-weight: 600;
+      }
     }
     .disabledTitle {
       font-size: 12px;
       line-height: 20px;
-      font-weight: 600;
       cursor: not-allowed;
       color: var(--text-color-hint);
     }

src/component/RelativeResourceModal/index.tsx

@@ -132,6 +132,12 @@ const RelativeResourceModal: React.FC<IRelativeResourceModalProps> = ({
     }
   }, [open, id, mode]);
 
+  // 判断是否可以访问详情：个人空间无权限，否则检查项目权限
+  const getCanAccessDetail = (record: IResourceDependencyItem): boolean => {
+    const isPersonalSpace = mode === EEntityType.USER || login.isPrivateSpace();
+    return isPersonalSpace ? false : !!record?.project?.currentUserResourceRoles?.length;
+  };
+
   const EResourceTypeConfig = useMemo(
     () => ({
       [EResourceType.TASKS]: {
@@ -178,25 +184,24 @@ const RelativeResourceModal: React.FC<IRelativeResourceModalProps> = ({
             setCurrentRecord(record);
             setDetailVisible(true);
           };
-          // 个人空间对应的资源依赖弹窗，不校验项目权限，直接按无权限展示
-          const isDeleteUser = mode === EEntityType.USER || login.isPrivateSpace();
-          const hasProjectAuth = isDeleteUser
-            ? false
-            : !!record?.project?.currentUserResourceRoles?.length;
-          const hint = isDeleteUser
+          const canAccessDetail = getCanAccessDetail(record);
+          const isPersonalSpace = mode === EEntityType.USER || login.isPrivateSpace();
+          const hint = isPersonalSpace
             ? '无访问权限，无法查看工单详情'
             : formatMessage({
                 id: 'src.component.RelativeResourceModal.94DC18D3',
                 defaultMessage: '暂无所属的项目访问权限，无法查看工单详情，请联系管理员',
               });
           return (
             <div className={styles.taskName}>
-              <Tooltip title={hasProjectAuth ? '' : hint}>
+              <Tooltip title={canAccessDetail ? '' : hint}>
                 <div
-                  className={hasProjectAuth ? styles.title : styles.disabledTitle}
-                  onClick={hasProjectAuth ? handleClick : () => {}}
+                  className={canAccessDetail ? styles.title : styles.disabledTitle}
+                  onClick={canAccessDetail ? handleClick : () => {}}
                 >
-                  <Tooltip title={hasProjectAuth ? text : ''}>{text || '-'}</Tooltip>
+                  <Tooltip title={canAccessDetail ? text : ''}>
+                    <span className={styles.titleText}>{text || '-'}</span>
+                  </Tooltip>
                 </div>
               </Tooltip>
               <div>
@@ -247,14 +252,20 @@ const RelativeResourceModal: React.FC<IRelativeResourceModalProps> = ({
         key: 'name',
         ellipsis: true,
         render: (text: string, record: IResourceDependencyItem) => {
+          const canAccessDetail = getCanAccessDetail(record);
+
           const handleClick = () => {
+            if (!canAccessDetail) {
+              return;
+            }
             setCurrentRecord(record);
             setScheduleDetailVisible(true);
           };
-
           return (
             <div className={styles.scheduleName} onClick={handleClick}>
-              <div className={styles.title}>{text || '-'}</div>
+              <div className={canAccessDetail ? styles.title : styles.disabledTitle}>
+                {text || '-'}
+              </div>
               <div>
                 <TaskTitle record={record} />
               </div>
@@ -301,14 +312,19 @@ const RelativeResourceModal: React.FC<IRelativeResourceModalProps> = ({
         width: 92,
         ellipsis: true,
         render: (text: string, record: IResourceDependencyItem) => {
+          const canAccessDetail = getCanAccessDetail(record);
           const handleClick = () => {
+            if (!canAccessDetail) {
+              return;
+            }
             setCurrentRecord(record);
             setExecuteDetailVisible(true);
           };
-
           return (
             <div className={styles.taskName} onClick={handleClick}>
-              {text ? `#${text}` : '-'}
+              <div className={canAccessDetail ? styles.title : styles.disabledTitle}>
+                {text ? `#${text}` : '-'}
+              </div>
             </div>
           );
         },