From 94a1d6dc204b406613df44c84d0aebbacf4a379d Mon Sep 17 00:00:00 2001
From: xlsea <m@xlsea.cn>
Date: Tue, 11 Jun 2024 10:14:20 +0800
Subject: [PATCH] =?UTF-8?q?perf(sj=5F1.0.0=5Fbeta1):=20=E4=BC=98=E5=8C=96?=
 =?UTF-8?q?=E6=90=BA=E5=B8=A6=E5=8F=82=E6=95=B0=E8=B7=AF=E7=94=B1=E8=B7=B3?=
 =?UTF-8?q?=E8=BD=AC=E4=BD=93=E9=AA=8C?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 packages/hooks/src/use-table.ts    |  5 ++++
 src/hooks/common/table.ts          |  1 +
 src/typings/naive-ui.d.ts          |  2 +-
 src/views/job/batch/index.vue      | 48 +++++-------------------------
 src/views/job/task/index.vue       |  3 +-
 src/views/retry/log/index.vue      | 14 +++------
 src/views/workflow/batch/index.vue | 25 +++++-----------
 src/views/workflow/task/index.vue  |  2 +-
 8 files changed, 30 insertions(+), 70 deletions(-)

diff --git a/packages/hooks/src/use-table.ts b/packages/hooks/src/use-table.ts
index 6c2ae07..da17be5 100644
--- a/packages/hooks/src/use-table.ts
+++ b/packages/hooks/src/use-table.ts
@@ -29,6 +29,8 @@ export type TableConfig<A extends ApiFn, T, C> = {
   apiFn: A;
   /** api params */
   apiParams?: Parameters<A>[0];
+  /** search params */
+  searchParams?: Parameters<A>[0];
   /** transform api response to table data */
   transformer: Transformer<T, Awaited<ReturnType<A>>>;
   /** columns factory */
@@ -133,6 +135,9 @@ export default function useHookTable<A extends ApiFn, T, C>(config: TableConfig<
   }
 
   if (immediate) {
+    if (config.searchParams) {
+      updateSearchParams(config.searchParams);
+    }
     getData();
   }
 
diff --git a/src/hooks/common/table.ts b/src/hooks/common/table.ts
index 962deaf..5d3c728 100644
--- a/src/hooks/common/table.ts
+++ b/src/hooks/common/table.ts
@@ -36,6 +36,7 @@ export function useTable<A extends NaiveUI.TableApiFn>(config: NaiveUI.NaiveTabl
   } = useHookTable<A, GetTableData<A>, TableColumn<NaiveUI.TableDataWithIndex<GetTableData<A>>>>({
     apiFn,
     apiParams,
+    searchParams: config.searchParams,
     columns: config.columns,
     transformer: res => {
       const { data: records = [], page: current = 1, size = 10, total = 0 } = res.data || {};
diff --git a/src/typings/naive-ui.d.ts b/src/typings/naive-ui.d.ts
index c8e6f25..7074643 100644
--- a/src/typings/naive-ui.d.ts
+++ b/src/typings/naive-ui.d.ts
@@ -42,7 +42,7 @@ declare namespace NaiveUI {
 
   type NaiveTableConfig<A extends TableApiFn> = Pick<
     import('@sa/hooks').TableConfig<A, GetTableData<A>, TableColumn<TableDataWithIndex<GetTableData<A>>>>,
-    'apiFn' | 'apiParams' | 'columns' | 'immediate'
+    'apiFn' | 'apiParams' | 'columns' | 'immediate' | 'searchParams'
   > & {
     /**
      * whether to display the total items count
diff --git a/src/views/job/batch/index.vue b/src/views/job/batch/index.vue
index 8026c45..075c640 100644
--- a/src/views/job/batch/index.vue
+++ b/src/views/job/batch/index.vue
@@ -1,9 +1,8 @@
 <script setup lang="tsx">
 import { NButton, NPopconfirm, NTag } from 'naive-ui';
 import { useBoolean } from '@sa/hooks';
-import { ref, watch } from 'vue';
-import { useRoute } from 'vue-router';
-import { fetchGetJobBatchList, fetchGetJobNameList, fetchJobBatchRetry, fetchJobBatchStop } from '@/service/api';
+import { ref } from 'vue';
+import { fetchGetJobBatchList, fetchJobBatchRetry, fetchJobBatchStop } from '@/service/api';
 import { $t } from '@/locales';
 import { useAppStore } from '@/store/modules/app';
 import { useTable } from '@/hooks/common/table';
@@ -13,12 +12,12 @@ import JobBatchSearch from './modules/job-batch-search.vue';
 import JobBatchDetailDrawer from './modules/job-batch-detail-drawer.vue';
 
 const appStore = useAppStore();
-const route = useRoute();
-
 /** 详情页属性数据 */
 const detailData = ref<Api.JobBatch.JobBatch | null>();
 /** 详情页可见状态 */
 const { bool: detailVisible, setTrue: openDetail } = useBoolean(false);
+const jobName = history.state.jobName;
+const taskBatchStatus = history.state.taskBatchStatus;
 
 const { columnChecks, columns, data, getData, loading, mobilePagination, searchParams, resetSearchParams } = useTable({
   apiFn: fetchGetJobBatchList,
@@ -29,6 +28,10 @@ const { columnChecks, columns, data, getData, loading, mobilePagination, searchP
     jobName: null,
     taskBatchStatus: null
   },
+  searchParams: {
+    jobName,
+    taskBatchStatus
+  },
   columns: () => [
     {
       key: 'id',
@@ -163,41 +166,6 @@ async function handleStopJob(id: string) {
     getData();
   }
 }
-
-/** 处理路由 query 参数变化 */
-async function handleQueryChanged(jobId: number) {
-  if (!jobId) {
-    searchParams.jobName = null;
-  } else {
-    const { data: jobList, error } = await fetchGetJobNameList({ jobId });
-    if (!error && jobList.length > 0) {
-      const jobName = jobList[0].jobName;
-      searchParams.jobName = jobName;
-    }
-  }
-  getData();
-}
-
-watch(
-  () => route.query,
-  () => {
-    if (route.name === 'job_batch' && route.query.jobId) {
-      const jobId = Number(route.query.jobId);
-      handleQueryChanged(jobId);
-    }
-  },
-  { immediate: true }
-);
-
-function initParams() {
-  const taskBatchStatus = history.state.taskBatchStatus;
-  if (taskBatchStatus) {
-    searchParams.taskBatchStatus = taskBatchStatus;
-    getData();
-  }
-}
-
-initParams();
 </script>
 
 <template>
diff --git a/src/views/job/task/index.vue b/src/views/job/task/index.vue
index 9463821..1ee93d3 100644
--- a/src/views/job/task/index.vue
+++ b/src/views/job/task/index.vue
@@ -250,7 +250,8 @@ async function handleTriggerJob(id: string) {
 }
 
 function goToBatch(jobId: string) {
-  routerPushByKey('job_batch', { query: { jobId } });
+  const findItem = data.value.find(item => item.id === jobId)!;
+  routerPushByKey('job_batch', { state: { jobName: findItem.jobName } });
 }
 
 function body(): Api.Job.ExportJob {
diff --git a/src/views/retry/log/index.vue b/src/views/retry/log/index.vue
index bc07353..82b2dc2 100644
--- a/src/views/retry/log/index.vue
+++ b/src/views/retry/log/index.vue
@@ -17,6 +17,7 @@ const appStore = useAppStore();
 const detailData = ref<Api.RetryLog.RetryLog | null>();
 /** 详情页可见状态 */
 const { bool: detailVisible, setTrue: openDetail } = useBoolean(false);
+const retryStatus = history.state.retryStatus;
 
 const { columns, columnChecks, data, getData, loading, mobilePagination, searchParams, resetSearchParams } = useTable({
   apiFn: fetchRetryLogPageList,
@@ -32,6 +33,9 @@ const { columns, columnChecks, data, getData, loading, mobilePagination, searchP
     bizNo: null,
     retryStatus: null
   },
+  searchParams: {
+    retryStatus
+  },
   columns: () => [
     {
       type: 'selection',
@@ -168,16 +172,6 @@ async function loadRetryInfo(row: Api.RetryLog.RetryLog) {
   const res = await fetchRetryLogById(row.id!);
   detailData.value = (res.data as Api.RetryLog.RetryLog) || null;
 }
-
-function initParams() {
-  const retryStatus = history.state.retryStatus;
-  if (retryStatus) {
-    searchParams.retryStatus = retryStatus;
-    getData();
-  }
-}
-
-initParams();
 </script>
 
 <template>
diff --git a/src/views/workflow/batch/index.vue b/src/views/workflow/batch/index.vue
index 1451507..8a4bbfc 100644
--- a/src/views/workflow/batch/index.vue
+++ b/src/views/workflow/batch/index.vue
@@ -1,6 +1,6 @@
 <script setup lang="tsx">
 import { NButton, NPopconfirm, NTag } from 'naive-ui';
-import { useRoute, useRouter } from 'vue-router';
+import { useRouter } from 'vue-router';
 import { fetchGetWorkflowBatchList, fetchStopWorkflowBatch } from '@/service/api';
 import { $t } from '@/locales';
 import { useAppStore } from '@/store/modules/app';
@@ -8,13 +8,10 @@ import { useTable, useTableOperate } from '@/hooks/common/table';
 import { operationReasonRecord, taskBatchStatusRecord } from '@/constants/business';
 import WorkflowBatchSearch from './modules/workflow-batch-search.vue';
 const router = useRouter();
-const route = useRoute();
-
-// 此处可能有问题
-const workflowId =
-  route.query?.workflowId === undefined ? null : Number.parseInt(route.query?.workflowId as string, 10);
 
 const appStore = useAppStore();
+const workflowId = history.state.workflowId;
+const taskBatchStatus = history.state.taskBatchStatus;
 
 const { columns, columnChecks, data, getData, loading, mobilePagination, searchParams, resetSearchParams } = useTable({
   apiFn: fetchGetWorkflowBatchList,
@@ -23,10 +20,14 @@ const { columns, columnChecks, data, getData, loading, mobilePagination, searchP
     size: 10,
     // if you want to use the searchParams in Form, you need to define the following properties, and the value is null
     // the value can not be undefined, otherwise the property in Form will not be reactive
-    workflowId,
+    workflowId: null,
     groupName: null,
     taskBatchStatus: null
   },
+  searchParams: {
+    workflowId,
+    taskBatchStatus
+  },
   columns: () => [
     {
       key: 'id',
@@ -155,16 +156,6 @@ async function handleStop(id: string) {
 function detail(id: string) {
   router.push({ path: '/workflow/form/batch', query: { id } });
 }
-
-function initParams() {
-  const taskBatchStatus = history.state.taskBatchStatus;
-  if (taskBatchStatus) {
-    searchParams.taskBatchStatus = taskBatchStatus;
-    getData();
-  }
-}
-
-initParams();
 </script>
 
 <template>
diff --git a/src/views/workflow/task/index.vue b/src/views/workflow/task/index.vue
index f2a3308..8a75ecf 100644
--- a/src/views/workflow/task/index.vue
+++ b/src/views/workflow/task/index.vue
@@ -238,7 +238,7 @@ function copy(id: string) {
 }
 
 function batch(id: string) {
-  router.push({ path: '/workflow/batch', query: { workflowId: id } });
+  router.push({ path: '/workflow/batch', state: { workflowId: id } });
 }
 
 async function execute(id: string) {