From 3da3260eb57e1b1f8dd6e0beff4e6a77bec674f6 Mon Sep 17 00:00:00 2001
From: balibabu <cike8899@users.noreply.github.com>
Date: Mon, 5 Aug 2024 12:03:47 +0800
Subject: [PATCH] fix: Fixed the issue that the related form value does not
 change after selecting the freedom field of the model #1804 (#1805)

### What problem does this PR solve?
fix: Fixed the issue that the related form value does not change after
selecting the freedom field of the model #1804

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
---
 web/src/components/llm-select/index.tsx          |  8 +++++++-
 web/src/components/llm-setting-items/index.tsx   | 16 +++++++++++++---
 .../components/knowledge-file/model.ts           |  2 +-
 web/src/pages/flow/hooks.ts                      | 16 +++++++++++++++-
 4 files changed, 36 insertions(+), 6 deletions(-)

diff --git a/web/src/components/llm-select/index.tsx b/web/src/components/llm-select/index.tsx
index 61d37b599..e3b348be9 100644
--- a/web/src/components/llm-select/index.tsx
+++ b/web/src/components/llm-select/index.tsx
@@ -17,7 +17,13 @@ const LLMSelect = ({ id, value, onChange }: IProps) => {
   );
 
   return (
-    <Popover content={content} trigger="click" placement="left" arrow={false}>
+    <Popover
+      content={content}
+      trigger="click"
+      placement="left"
+      arrow={false}
+      destroyTooltipOnHide
+    >
       <Select
         style={{ width: '100%' }}
         dropdownStyle={{ display: 'none' }}
diff --git a/web/src/components/llm-setting-items/index.tsx b/web/src/components/llm-setting-items/index.tsx
index c31d7aabe..3cf2ec0cb 100644
--- a/web/src/components/llm-setting-items/index.tsx
+++ b/web/src/components/llm-setting-items/index.tsx
@@ -28,9 +28,13 @@ const LlmSettingItems = ({ prefix, formItemLayout = {} }: IProps) => {
   const handleParametersChange = useCallback(
     (value: ModelVariableType) => {
       const variable = settledModelVariableMap[value];
-      form?.setFieldsValue(variable);
+      let nextVariable: Record<string, any> = variable;
+      if (prefix) {
+        nextVariable = { [prefix]: variable };
+      }
+      form.setFieldsValue(nextVariable);
     },
-    [form],
+    [form, prefix],
   );
 
   const memorizedPrefix = useMemo(() => (prefix ? [prefix] : []), [prefix]);
@@ -46,7 +50,13 @@ const LlmSettingItems = ({ prefix, formItemLayout = {} }: IProps) => {
         {...formItemLayout}
         rules={[{ required: true, message: t('modelMessage') }]}
       >
-        <Select options={[...modelOptions[LlmModelType.Chat], ...modelOptions[LlmModelType.Image2text],]} showSearch/>
+        <Select
+          options={[
+            ...modelOptions[LlmModelType.Chat],
+            ...modelOptions[LlmModelType.Image2text],
+          ]}
+          showSearch
+        />
       </Form.Item>
       <Divider></Divider>
       <Form.Item
diff --git a/web/src/pages/add-knowledge/components/knowledge-file/model.ts b/web/src/pages/add-knowledge/components/knowledge-file/model.ts
index 4ba63c67c..508f5d660 100644
--- a/web/src/pages/add-knowledge/components/knowledge-file/model.ts
+++ b/web/src/pages/add-knowledge/components/knowledge-file/model.ts
@@ -101,7 +101,7 @@ const model: DvaModel<KFModelState> = {
       function* ({ payload }, { call, put }) {
         yield put({ type: 'getKfList', payload: { kb_id: payload } });
       },
-      { type: 'poll', delay: 5000 }, // TODO: Provide type support for this effect
+      { type: 'poll', delay: 15000 }, // TODO: Provide type support for this effect
     ],
     *updateDocumentStatus({ payload = {} }, { call, put }) {
       const { data } = yield call(
diff --git a/web/src/pages/flow/hooks.ts b/web/src/pages/flow/hooks.ts
index 1c4bc3b41..46e407a4b 100644
--- a/web/src/pages/flow/hooks.ts
+++ b/web/src/pages/flow/hooks.ts
@@ -252,8 +252,22 @@ export const useHandleFormValuesChange = (id?: string) => {
   const updateNodeForm = useGraphStore((state) => state.updateNodeForm);
   const handleValuesChange = useCallback(
     (changedValues: any, values: any) => {
+      let nextValues: any = values;
+      // Fixed the issue that the related form value does not change after selecting the freedom field of the model
+      if (
+        Object.keys(changedValues).length === 1 &&
+        'parameter' in changedValues &&
+        changedValues['parameter'] in settledModelVariableMap
+      ) {
+        nextValues = {
+          ...values,
+          ...settledModelVariableMap[
+            changedValues['parameter'] as keyof typeof settledModelVariableMap
+          ],
+        };
+      }
       if (id) {
-        updateNodeForm(id, values);
+        updateNodeForm(id, nextValues);
       }
     },
     [updateNodeForm, id],