From 83d9ea3ccb98a9ae0ff5a15aa309747c04b9211c Mon Sep 17 00:00:00 2001 From: Amy0104 <97265214+Amy0104@users.noreply.github.com> Date: Mon, 25 Apr 2022 19:03:30 +0800 Subject: [PATCH] [Fix][UI Next][V1.0.0-Beta] Remove the default values for node name and child node. (#9760) --- .../src/locales/modules/en_US.ts | 1 + .../src/locales/modules/zh_CN.ts | 1 + .../components/node/fields/use-child-node.ts | 22 +++++++++---------- .../components/node/tasks/use-sub-process.ts | 3 +-- 4 files changed, 14 insertions(+), 13 deletions(-) diff --git a/dolphinscheduler-ui-next/src/locales/modules/en_US.ts b/dolphinscheduler-ui-next/src/locales/modules/en_US.ts index 41d26035d..1f605dae0 100644 --- a/dolphinscheduler-ui-next/src/locales/modules/en_US.ts +++ b/dolphinscheduler-ui-next/src/locales/modules/en_US.ts @@ -631,6 +631,7 @@ const project = { workflow_name: 'Workflow Name', workflow_name_tips: 'Please select a workflow (required)', child_node: 'Child Node', + child_node_tips: 'Please select a child node (required)', run_flag: 'Run flag', normal: 'Normal', prohibition_execution: 'Prohibition execution', diff --git a/dolphinscheduler-ui-next/src/locales/modules/zh_CN.ts b/dolphinscheduler-ui-next/src/locales/modules/zh_CN.ts index 4525d65d2..03ac1caf6 100644 --- a/dolphinscheduler-ui-next/src/locales/modules/zh_CN.ts +++ b/dolphinscheduler-ui-next/src/locales/modules/zh_CN.ts @@ -627,6 +627,7 @@ const project = { workflow_name: '工作流名称', workflow_name_tips: '请选择工作流(必选)', child_node: '子节点', + child_node_tips: '请选择子节点(必选)', run_flag: '运行标志', normal: '正常', prohibition_execution: '禁止执行', diff --git a/dolphinscheduler-ui-next/src/views/projects/task/components/node/fields/use-child-node.ts b/dolphinscheduler-ui-next/src/views/projects/task/components/node/fields/use-child-node.ts index 7dcee5d03..f9a1be2c0 100644 --- a/dolphinscheduler-ui-next/src/views/projects/task/components/node/fields/use-child-node.ts +++ b/dolphinscheduler-ui-next/src/views/projects/task/components/node/fields/use-child-node.ts @@ -52,10 +52,6 @@ export function useChildNode({ value: option.code })) loading.value = false - if (!model.processDefinitionCode) { - model.processDefinitionCode = options.value[0].value - if (!model.name) model.name = options.value[0].label - } } const getProcessListByCode = async (processCode: number) => { if (!processCode) return @@ -63,10 +59,6 @@ export function useChildNode({ model.definition = res } - const onChange = (code: number, options: { label: string }) => { - if (!model.name) model.name = options.label - } - onMounted(() => { if (from === 1 && processName) { getProcessListByCode(processName) @@ -80,10 +72,18 @@ export function useChildNode({ span: 24, name: t('project.node.child_node'), props: { - loading: loading, - 'on-update:value': onChange + loading: loading }, options: options, - class: 'select-child-node' + class: 'select-child-node', + validate: { + trigger: ['input', 'blur'], + required: true, + validator(unuse: any, value: number) { + if (!value) { + return Error(t('project.node.child_node_tips')) + } + } + } } } diff --git a/dolphinscheduler-ui-next/src/views/projects/task/components/node/tasks/use-sub-process.ts b/dolphinscheduler-ui-next/src/views/projects/task/components/node/tasks/use-sub-process.ts index 2f8ed0dd0..7338da437 100644 --- a/dolphinscheduler-ui-next/src/views/projects/task/components/node/tasks/use-sub-process.ts +++ b/dolphinscheduler-ui-next/src/views/projects/task/components/node/tasks/use-sub-process.ts @@ -45,8 +45,7 @@ export function useSubProcess({ failRetryTimes: 0, workerGroup: 'default', delayTime: 0, - timeout: 30, - processDefinitionCode: 0 + timeout: 30 } as INodeData) let extra: IJsonItem[] = [] -- GitLab